RE: [PATCH 3/3][v2] Thermal: do thermal zone update after a cooling device registered

[Chen, Yu C]

Hi, Javi,
thanks for your review,
will send a version 3 out soon.

> -----Original Message-----
> From: Javi Merino [mailto:javi.merino@xxxxxxx]
> Sent: Wednesday, October 28, 2015 6:40 PM
> To: Chen, Yu C
> Cc: Zhang, Rui; edubezval@xxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; stable@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 3/3][v2] Thermal: do thermal zone update after a cooling
> device registered
> 
> Hi Yu,
> 
> One minor nit below.  Other than that you can add my
> 
> Reviewed-by: Javi Merino <javi.merino@xxxxxxx>
> 
> On Mon, Oct 26, 2015 at 11:15:30AM +0800, Chen Yu wrote:
> > When a new cooling device is registered, we need to update the thermal
> > zone to set the new registered cooling device to a proper state.
> >
> > This fixes a problem that the system is cool, while the fan devices
> > are left running on full speed after boot, if fan device is registered
> > after thermal zone device.
> >
> > Here is the history of why current patch looks like this:
> > https://patchwork.kernel.org/patch/7273041/
> >
> > CC: <stable@xxxxxxxxxxxxxxx> #3.18+
> > Reference:https://bugzilla.kernel.org/show_bug.cgi?id=92431
> > Tested-by: Manuel Krause <manuelkrause@xxxxxxxxxxxx>
> > Tested-by: szegad <szegadlo@xxxxxxxxxxxxxx>
> > Tested-by: prash <prash.n.rao@xxxxxxxxx>
> > Tested-by: amish <ammdispose-arch@xxxxxxxxx>
> > Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx>
> > Signed-off-by: Chen Yu <yu.c.chen@xxxxxxxxx>
> > ---
> > diff --git a/include/linux/thermal.h b/include/linux/thermal.h index
> > 5bcabc7..4298418 100644
> > --- a/include/linux/thermal.h
> > +++ b/include/linux/thermal.h
> > @@ -195,6 +195,7 @@ struct thermal_zone_device {
> >  	int emul_temperature;
> >  	int passive;
> >  	unsigned int forced_passive;
> > +	atomic_t need_update;
> 
> Please document this field in the kernel-doc comment for struct
> thermal_zone_device.
> 

Best Regards,
Yu