Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751171AbdH1FLI (ORCPT ); Mon, 28 Aug 2017 01:11:08 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:56758 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750735AbdH1FLG (ORCPT ); Mon, 28 Aug 2017 01:11:06 -0400 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Date: Mon, 28 Aug 2017 10:41:05 +0530 From: kgunda@codeaurora.org To: Stephen Boyd Cc: Zhang Rui , Eduardo Valentin , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, iivanov@mm-sol.com, linux-arm-msm@vger.kernel.org, David Collins , linux-arm-msm-owner@vger.kernel.org, Geert Uytterhoeven , Jean Delvare , Lukasz Majewski , Darren Hart , Heiko Stuebner , Peter Feuerer , bjorn.andersson@linaro.org Subject: Re: [PATCH V1] thermal: qcom-spmi-temp-alarm: add support for GEN2 PMIC peripherals In-Reply-To: <20170825231948.GQ21656@codeaurora.org> References: <1499947772-19199-1-git-send-email-kgunda@codeaurora.org> <1502179943.4296.12.camel@intel.com> <5d3f612dca4181b2038e442cfba5594e@codeaurora.org> <1503627189.2774.3.camel@intel.com> <20170825231948.GQ21656@codeaurora.org> Message-ID: <1490647f090d91601bfbabbf56ad053f@codeaurora.org> User-Agent: Roundcube Webmail/1.2.5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1393 Lines: 42 On 2017-08-26 04:49, Stephen Boyd wrote: > On 08/25, Zhang Rui wrote: >> On Thu, 2017-08-17 at 13:12 +0530, kgunda@codeaurora.org wrote: >> > On 2017-08-16 17:53, kgunda@codeaurora.org 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 >> > > > > >> > > > > 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 >> > > > > Signed-off-by: Kiran Gunda >> > > > 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. >> Thanks for that ! > > FWIW, > > Reviewed-by: Stephen Boyd