Re: [PATCH V1] thermal: qcom-spmi-temp-alarm: add support for GEN2 PMIC peripherals

From: Stephen Boyd
Date: Fri Aug 25 2017 - 19:19:59 EST


On 08/25, Zhang Rui wrote:
> On Thu, 2017-08-17 at 13:12 +0530, kgunda@xxxxxxxxxxxxxx wrote:
> > On 2017-08-16 17:53, kgunda@xxxxxxxxxxxxxx wrote:
> > >
> > > On 2017-08-08 13:42, Zhang Rui wrote:
> > > >
> > > > On Thu, 2017-07-13 at 17:39 +0530, Kiran Gunda wrote:
> > > > >
> > > > > From: David Collins <collinsd@xxxxxxxxxxxxxx>
> > > > >
> > > > > Add support for the TEMP_ALARM GEN2 PMIC peripheral
> > > > > subtype.  The
> > > > > GEN2 subtype defines an over temperature state with hysteresis
> > > > > instead of stage in the status register.  There are two GEN2
> > > > > states corresponding to stages 1 and 2.
> > > > >
> > > > > Signed-off-by: David Collins <collinsd@xxxxxxxxxxxxxx>
> > > > > Signed-off-by: Kiran Gunda <kgunda@xxxxxxxxxxxxxx>
> > > > Ivan,
> > > >
> > > > can you please review this patch and let me know your opinion?
> > > >
> > > > thanks,
> > > > rui
> > > Ivan,
> > > Could you please review this patch ?
> > >
> > > Thanks,
> > > Kiran
> > Looks like Ivan is no more reviewing the patches for qcom.
> > Adding Bjorn and Stephen Boyd for the review.
> >
> Given this is a platform specific change, I will queue it for next
> merge window, and let's see if there is any problem reported.
>

FWIW,

Reviewed-by: Stephen Boyd <sboyd@xxxxxxxxxxxxxx>

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project