Hi, Javi, thanks for your review, will send a version 3 out soon. > -----Original Message----- > From: Javi Merino [mailto:javi.mer...@arm.com] > Sent: Wednesday, October 28, 2015 6:40 PM > To: Chen, Yu C > Cc: Zhang, Rui; edubez...@gmail.com; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org; stable@vger.kernel.org > 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.mer...@arm.com> > > 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@vger.kernel.org> #3.18+ > > Reference:https://bugzilla.kernel.org/show_bug.cgi?id=92431 > > Tested-by: Manuel Krause <manuelkra...@netscape.net> > > Tested-by: szegad <szega...@poczta.onet.pl> > > Tested-by: prash <prash.n....@gmail.com> > > Tested-by: amish <ammdispose-a...@yahoo.com> > > Signed-off-by: Zhang Rui <rui.zh...@intel.com> > > Signed-off-by: Chen Yu <yu.c.c...@intel.com> > > --- > > 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