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 ��.n��������+%������w��{.n�����������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f