Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-28 Thread Sascha Hauer
On Mon, Apr 27, 2015 at 10:36:08PM +0200, Pavel Machek wrote: > On Thu 2015-03-26 16:53:48, Sascha Hauer wrote: > > The thermal framework uses int, long and unsigned long for temperatures > > in millicelsius. The majority of functions uses unsigned long, so change > > the remaining functions to

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-28 Thread Sascha Hauer
On Mon, Apr 27, 2015 at 10:36:08PM +0200, Pavel Machek wrote: On Thu 2015-03-26 16:53:48, Sascha Hauer wrote: The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions to use this

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-27 Thread Pavel Machek
On Thu 2015-03-26 16:53:48, Sascha Hauer wrote: > The thermal framework uses int, long and unsigned long for temperatures > in millicelsius. The majority of functions uses unsigned long, so change > the remaining functions to use this type aswell. Maybe millicelsius_t should be introduced, so

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-27 Thread Pavel Machek
On Thu 2015-03-26 16:53:48, Sascha Hauer wrote: The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions to use this type aswell. Maybe millicelsius_t should be introduced, so that

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-06 Thread Eduardo Valentin
On Fri, Mar 27, 2015 at 08:07:50PM +0100, Sascha Hauer wrote: > On Fri, Mar 27, 2015 at 10:18:14AM +, Punit Agrawal wrote: > > Hi Sascha, > > > > Sascha Hauer writes: > > > > > The thermal framework uses int, long and unsigned long for temperatures > > > in millicelsius. The majority of

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-06 Thread Eduardo Valentin
On Thu, Mar 26, 2015 at 04:53:48PM +0100, Sascha Hauer wrote: > The thermal framework uses int, long and unsigned long for temperatures > in millicelsius. The majority of functions uses unsigned long, so change > the remaining functions to use this type aswell. I believe it make sense to change

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-06 Thread Eduardo Valentin
On Fri, Mar 27, 2015 at 08:07:50PM +0100, Sascha Hauer wrote: On Fri, Mar 27, 2015 at 10:18:14AM +, Punit Agrawal wrote: Hi Sascha, Sascha Hauer s.ha...@pengutronix.de writes: The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-04-06 Thread Eduardo Valentin
On Thu, Mar 26, 2015 at 04:53:48PM +0100, Sascha Hauer wrote: The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions to use this type aswell. I believe it make sense to change all

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-03-27 Thread Sascha Hauer
On Fri, Mar 27, 2015 at 10:18:14AM +, Punit Agrawal wrote: > Hi Sascha, > > Sascha Hauer writes: > > > The thermal framework uses int, long and unsigned long for temperatures > > in millicelsius. The majority of functions uses unsigned long, so change > > the remaining functions to use this

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-03-27 Thread Punit Agrawal
Hi Sascha, Sascha Hauer writes: > The thermal framework uses int, long and unsigned long for temperatures > in millicelsius. The majority of functions uses unsigned long, so change > the remaining functions to use this type aswell. > > Signed-off-by: Sascha Hauer I'd suggest changing to long

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-03-27 Thread Punit Agrawal
Hi Sascha, Sascha Hauer s.ha...@pengutronix.de writes: The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions to use this type aswell. Signed-off-by: Sascha Hauer

Re: [PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-03-27 Thread Sascha Hauer
On Fri, Mar 27, 2015 at 10:18:14AM +, Punit Agrawal wrote: Hi Sascha, Sascha Hauer s.ha...@pengutronix.de writes: The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions

[PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-03-26 Thread Sascha Hauer
The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions to use this type aswell. Signed-off-by: Sascha Hauer --- drivers/thermal/thermal_core.c | 10 +-

[PATCH 01/13] thermal: Make temperatures consistently unsigned long

2015-03-26 Thread Sascha Hauer
The thermal framework uses int, long and unsigned long for temperatures in millicelsius. The majority of functions uses unsigned long, so change the remaining functions to use this type aswell. Signed-off-by: Sascha Hauer s.ha...@pengutronix.de --- drivers/thermal/thermal_core.c | 10 +-