Am Montag, 6. Juli 2015, 09:19:49 schrieb Sascha Hauer: > The thermal code uses int, long and unsigned long for temperatures > in different places. > > Using an unsigned type limits the thermal framework to positive > temperatures without need. Also several drivers currently will report > temperatures near UINT_MAX for temperatures below 0°C. This will probably > immediately shut the machine down due to overtemperature if started below > 0°C. > > 'long' is 64bit on several architectures. This is not needed since INT_MAX > °mC is above the melting point of all known materials. > > Consistently use a plain 'int' for temperatures throughout the thermal code > and the drivers. This only changes the places in the drivers where the > temperature is passed around as pointer, when drivers internally use > another type this is not changed. > > Signed-off-by: Sascha Hauer <[email protected]> > Cc: Zhang Rui <[email protected]> > Cc: Eduardo Valentin <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: Jean Delvare <[email protected]> > Cc: Peter Feuerer <[email protected]> > Cc: Heiko Stuebner <[email protected]> > Cc: Lukasz Majewski <[email protected]> > Cc: Stephen Warren <[email protected]> > Cc: Thierry Reding <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: Guenter Roeck <[email protected]> > Cc: Rafael J. Wysocki <[email protected]> > Cc: Maxime Ripard <[email protected]> > Cc: Darren Hart <[email protected]> > Cc: [email protected] > ---
For > drivers/thermal/rockchip_thermal.c | 10 ++++---- Reviewed-by: Heiko Stuebner <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
