Re: [PATCH] thermal: rcar: enable to set tripN-temp via DT

From	Eduardo Valentin <edubezval@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] thermal: rcar: enable to set tripN-temp via DT
Date	2015-11-25 04:50 +0100
Message-ID	<qyzfH-BC-7@gated-at.bofh.it> (permalink)
References	(2 earlier) <qyzfH-BC-13@gated-at.bofh.it> <qyzfH-BC-15@gated-at.bofh.it> <qyzfH-BC-17@gated-at.bofh.it> <qyzfH-BC-19@gated-at.bofh.it> <qyzfH-BC-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Morimoto-san,


On Wed, Nov 25, 2015 at 01:45:14AM +0000, Kuninori Morimoto wrote:
> 
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> 
> Current rcar thermal driver is using 90 degrees as trip temp, but it
> should be based on each SoC / platform.
> This patch enables to set trip temp via DT. (It uses db8500-thermal
> style for it)
> It will use 90 degrees as default trip temp if DT doesn't have it.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>  .../devicetree/bindings/thermal/rcar-thermal.txt   |  2 ++
>  drivers/thermal/rcar_thermal.c                     | 34 ++++++++++++++++++++--
>  2 files changed, 33 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt
> index 332e625..6c57f7e 100644
> --- a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt
> +++ b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt
> @@ -18,6 +18,8 @@ Required properties:
>  Option properties:
>  
>  - interrupts		: use interrupt
> +- tripN-temp		: temperature of trip point N. it will use 90000 as default
> +			  if DT doesn't have tripN-temp

First of all, you are creating an entry with is specific to your driver.
That requires it to use proper prefixing.

Besides, your property is already covered by of-thermal. Please convert
your driver to use of-thermal, this way it will give you the flexibility
to configure thermal data in DT.

BR,
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH] thermal: rcar: enable to set tripN-temp via DT Eduardo Valentin <edubezval@gmail.com> - 2015-11-25 04:50 +0100
  Re: [PATCH] thermal: rcar: enable to set tripN-temp via DT Geert Uytterhoeven <geert@linux-m68k.org> - 2015-11-25 11:00 +0100

csiph-web