Messages in this thread | | | Date | Thu, 20 May 2021 11:55:15 +0530 | From | skakit@codeauro ... | Subject | Re: [RESEND PATCH V4 2/8] arm64: dts: qcom: sc7280: Add thermal-zones node |
| |
Hi Matthias,
On 2021-05-18 21:57, Matthias Kaehlcke wrote: > On Tue, May 18, 2021 at 05:14:03PM +0530, Vinod Koul wrote: >> On 18-05-21, 11:50, satya priya wrote: >> > Add thermal-zones node for SC7280 SoC. >> > >> > Signed-off-by: satya priya <skakit@codeaurora.org> >> > Reviewed-by: Matthias Kaehlcke <mka@chromium.org> >> > --- >> > Changes in RESEND V4: >> > - No Changes. >> > >> > arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +++ >> > 1 file changed, 3 insertions(+) >> > >> > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi >> > index 2cc4785..2a7d488 100644 >> > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi >> > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi >> > @@ -1125,4 +1125,7 @@ >> > <GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>, >> > <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>; >> > }; >> > + >> > + thermal_zones: thermal-zones { >> > + }; >> >> Empty node..? what am i missing here... > > The 'thermal-zones' node with the SoC thermal zones is usually created > in the <soc>.dtsi, however it doesn't exist yet. The 'Add DT bindings > and device tree nodes for TSENS in SC7280' series > (https://patchwork.kernel.org/project/linux-arm-msm/list/?series=478225) > is creating it, but the series hasn't landed yet. > > The node is needed by other patches in this series that add non-SoC > thermal zones. Hence either an empty node needs to be added (for now) > or this series should 'officially' depend on the TSENS series mentioned > above.
I will add the dependency in cover letter and drop this patch.
Thanks, Satya Priya
| |