Re: [PATCH] thermal: Fix cdev registration with THERMAL_NO_LIMIT on 64bit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tue, 2014-12-09 at 12:22 +0000, Punit Agrawal wrote:
> The size of unsigned long varies between 32 and 64 bit systems while
> the size of phandle arguments is always 32 bits per parameter.
> 
> On 64-bit systems, cooling devices registered via of-thermal apis fail
> to bind when the min/max cooling state is specified as
> THERMAL_NO_LIMIT (-1UL) as there is a mis-match between the value read
> from the device tree (32bit) and the pre-processor define (64bit).
> 
> As we're unlikely to need cooling states larger than 32 bits, and for
> consistency with the size of phandle arguments, explicitly limit
> THERMAL_NO_LIMIT to 32 bits.
> 
> Signed-off-by: Punit Agrawal <punit.agrawal@xxxxxxx>
> ---
>  include/dt-bindings/thermal/thermal.h | 2 +-
>  include/linux/thermal.h               | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/dt-bindings/thermal/thermal.h b/include/dt-bindings/thermal/thermal.h
> index 59822a9..b5e6b00 100644
> --- a/include/dt-bindings/thermal/thermal.h
> +++ b/include/dt-bindings/thermal/thermal.h
> @@ -11,7 +11,7 @@
>  #define _DT_BINDINGS_THERMAL_THERMAL_H
>  
>  /* On cooling devices upper and lower limits */
> -#define THERMAL_NO_LIMIT		(-1UL)
> +#define THERMAL_NO_LIMIT		(~0)
>  
I'm okay with the other change, so Eduardo, what do you think of this?

thanks,
rui
>  #endif
>  
> diff --git a/include/linux/thermal.h b/include/linux/thermal.h
> index ef90838..005586f 100644
> --- a/include/linux/thermal.h
> +++ b/include/linux/thermal.h
> @@ -38,7 +38,7 @@
>  #define THERMAL_CSTATE_INVALID -1UL
>  
>  /* No upper/lower limit requirement */
> -#define THERMAL_NO_LIMIT	THERMAL_CSTATE_INVALID
> +#define THERMAL_NO_LIMIT	((u32)~0)
>  
>  /* Unit conversion macros */
>  #define KELVIN_TO_CELSIUS(t)	(long)(((long)t-2732 >= 0) ?	\


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux