From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v4 2/6] thermal: amlogic: Add thermal driver to support G12 SoCs Date: Thu, 29 Aug 2019 11:11:23 -0700 Message-ID: <7hy2zbrgdw.fsf@baylibre.com> References: <20190821222421.30242-1-glaroque@baylibre.com> <20190821222421.30242-3-glaroque@baylibre.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190821222421.30242-3-glaroque@baylibre.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: rui.zhang@intel.com, edubezval@gmail.com, daniel.lezcano@linaro.org Cc: devicetree@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Guillaume La Roque , linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hello thermal maintainers, Guillaume La Roque writes: > Amlogic G12A and G12B SoCs integrate two thermal sensors > with the same design. > One is located close to the DDR controller and the other one is > located close to the PLLs (between the CPU and GPU). > > The calibration data for each of the thermal sensors instance is > stored in a different location within the AO region. > > Implement reading the temperature from each thermal sensor. > > The IP block has more functionality, which may be added to this driver > in the future: > - chip reset when the temperature exceeds a configurable threshold > - up to four interrupts when the temperature has risen above a > configurable threshold > - up to four interrupts when the temperature has fallen below a > configurable threshold > > Signed-off-by: Guillaume La Roque Could we get a review/merge of this driver ( and hopefully queued up for v5.4 ?) This has been reviewed and tested by users on this platform and it's working well. Reviewed-by: Kevin Hilman Tested-by: Kevin Hilman Thanks, Kevin