[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180702175357.GA28713@localhost.localdomain>
Date: Mon, 2 Jul 2018 10:53:57 -0700
From: Eduardo Valentin <edubezval@...il.com>
To: Amit Kucheria <amit.kucheria@...aro.org>
Cc: linux-kernel@...r.kernel.org, rnayak@...eaurora.org,
linux-arm-msm@...r.kernel.org, bjorn.andersson@...aro.org,
smohanad@...eaurora.org, vivek.gautam@...eaurora.org,
andy.gross@...aro.org, Kees Cook <keescook@...omium.org>,
Rob Herring <robh@...nel.org>, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-pm@...r.kernel.org,
linux-soc@...r.kernel.org
Subject: Re: [PATCH v4 0/6] thermal: tsens: Refactoring for TSENSv2 IP
On Mon, Jul 02, 2018 at 06:14:03PM +0530, Amit Kucheria wrote:
> This series is a mixed bag:
> - Some code moves to allow code sharing between various v2.x.y versions of
> the TSENS IP,
> - new qcom,tsens-v2.4.0 DT property for SDM845 and a generic qcom,tsens-v2
> property as a fallback compatible for all v2.x.y platforms,
> - new platform support (sdm845)
> - a cleanup patch and
> - a DT change to have a common way to deal with the SROT and TM registers
> despite slightly different features across the IP family and different
> register offsets.
>
> Rob mentioned offline that we should expose the full version string of the
> TSENS IP (x.y.z) and have a fallback compatible. I hope patch 4 does what
> you were looking for.
Applied patches 1, 3, and 4. Patch 5 needs a description. Patches 2 and
6 go via your arch tree.
>
> Regards,
> Amit
>
> Amit Kucheria (6):
> thermal: tsens: Get rid of unused fields in structure
> dt: qcom: 8996: thermal: Move to DT initialisation
> thermal: tsens: Rename tsens-8996 to tsens-v2 for reuse
> thermal: tsens: Add support for SDM845
> thermal: tsens: Check if we have valid data before reading
> arm64: dts: sdm845: Add tsens nodes
>
> .../devicetree/bindings/thermal/qcom-tsens.txt | 2 ++
> arch/arm64/boot/dts/qcom/msm8996.dtsi | 12 ++++++-
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 16 +++++++++
> drivers/thermal/qcom/Makefile | 2 +-
> drivers/thermal/qcom/{tsens-8996.c => tsens-v2.c} | 39 ++++++++++++----------
> drivers/thermal/qcom/tsens.c | 6 ++++
> drivers/thermal/qcom/tsens.h | 7 ++--
> 7 files changed, 62 insertions(+), 22 deletions(-)
> rename drivers/thermal/qcom/{tsens-8996.c => tsens-v2.c} (66%)
>
> --
> 2.7.4
>
Powered by blists - more mailing lists