Re: [PATCH 2/2] db8500 thermal dirver

2012-07-05 Thread Vincent Guittot
On 20 June 2012 15:04, hongbo.zhang wrote: > From: "hongbo.zhang" > > --- > arch/arm/configs/u8500_defconfig |4 + > arch/arm/mach-ux500/board-mop500.c | 71 > drivers/thermal/Kconfig | 16 + > drivers/thermal/Makefile |

Re: [PATCH 2/2] db8500 thermal dirver

2012-07-05 Thread Vincent Guittot
I sent the mail before finishing the review. additional one will come soon On 5 July 2012 16:13, Vincent Guittot wrote: > On 20 June 2012 15:04, hongbo.zhang wrote: >> From: "hongbo.zhang" >> >> --- >> arch/arm/configs/u8500_defconfig |4 + >> arch/arm/mach-ux500/board-mop500.c

Re: [PATCH 2/2] db8500 thermal dirver

2012-07-05 Thread Vincent Guittot
On 20 June 2012 15:04, hongbo.zhang wrote: > From: "hongbo.zhang" > > --- > arch/arm/configs/u8500_defconfig |4 + > arch/arm/mach-ux500/board-mop500.c | 71 > drivers/thermal/Kconfig | 16 + > drivers/thermal/Makefile |

Re: [PATCH 2/2] db8500 thermal dirver

2012-07-05 Thread Hongbo Zhang
On 5 July 2012 07:20, Linus Walleij wrote: > On Wed, Jun 20, 2012 at 3:04 PM, hongbo.zhang > wrote: > > > /* > > + * Thermal Sensor > > + */ > > + > > +#ifdef CONFIG_DB8500_THERMAL > > Please don't #ifdef the platform data like this, Documentation/CodingStyle > dislikes #ifdefs unless needed an

Re: [PATCH 2/2] db8500 thermal dirver

2012-07-04 Thread Linus Walleij
On Wed, Jun 20, 2012 at 3:04 PM, hongbo.zhang wrote: > /* > + * Thermal Sensor > + */ > + > +#ifdef CONFIG_DB8500_THERMAL Please don't #ifdef the platform data like this, Documentation/CodingStyle dislikes #ifdefs unless needed and this just saves a very marginal piece of code in footprint, and

[PATCH 2/2] db8500 thermal dirver

2012-06-20 Thread hongbo.zhang
From: "hongbo.zhang" --- arch/arm/configs/u8500_defconfig |4 + arch/arm/mach-ux500/board-mop500.c | 71 drivers/thermal/Kconfig | 16 + drivers/thermal/Makefile |4 +- drivers/thermal/cpu_cooling.c|