* [PATCH] arm64: dts: renesas: Remove unneeded status from thermal nodes
@ 2018-09-11 14:37 Geert Uytterhoeven
2018-09-12 9:34 ` Simon Horman
0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2018-09-11 14:37 UTC (permalink / raw)
To: Simon Horman, Magnus Damm, Niklas Söderlund
Cc: linux-renesas-soc, devicetree, Geert Uytterhoeven
The thermal device is supposed to be always enabled. As the default
value of the status property is "okay", there is no need to make this
explicit in SoC-specific .dtsi files where no override is involved.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 1 -
arch/arm64/boot/dts/renesas/r8a7795.dtsi | 1 -
arch/arm64/boot/dts/renesas/r8a7796.dtsi | 1 -
arch/arm64/boot/dts/renesas/r8a77965.dtsi | 1 -
4 files changed, 4 deletions(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
index 046fc937da1441dc..012cbb64246ec144 100644
--- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
@@ -358,7 +358,6 @@
power-domains = <&sysc 32>;
resets = <&cpg 522>;
#thermal-sensor-cells = <1>;
- status = "okay";
};
intc_ex: interrupt-controller@e61c0000 {
diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index 0c6d8859cae397c9..c74ef669a354d68e 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -455,7 +455,6 @@
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
resets = <&cpg 522>;
#thermal-sensor-cells = <1>;
- status = "okay";
};
intc_ex: interrupt-controller@e61c0000 {
diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
index 93a76109882c9fe7..303504f3a5be4158 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
@@ -434,7 +434,6 @@
power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
resets = <&cpg 522>;
#thermal-sensor-cells = <1>;
- status = "okay";
};
intc_ex: interrupt-controller@e61c0000 {
diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
index 77fb909cc83985c5..22dca901182e577c 100644
--- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
@@ -350,7 +350,6 @@
power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
resets = <&cpg 522>;
#thermal-sensor-cells = <1>;
- status = "okay";
};
intc_ex: interrupt-controller@e61c0000 {
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: dts: renesas: Remove unneeded status from thermal nodes
2018-09-11 14:37 [PATCH] arm64: dts: renesas: Remove unneeded status from thermal nodes Geert Uytterhoeven
@ 2018-09-12 9:34 ` Simon Horman
2018-09-14 13:37 ` Simon Horman
0 siblings, 1 reply; 3+ messages in thread
From: Simon Horman @ 2018-09-12 9:34 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Niklas Söderlund, linux-renesas-soc, devicetree
On Tue, Sep 11, 2018 at 04:37:30PM +0200, Geert Uytterhoeven wrote:
> The thermal device is supposed to be always enabled. As the default
> value of the status property is "okay", there is no need to make this
> explicit in SoC-specific .dtsi files where no override is involved.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks,
This looks fine to me but I will wait to see if there are other reviews
before applying.
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: dts: renesas: Remove unneeded status from thermal nodes
2018-09-12 9:34 ` Simon Horman
@ 2018-09-14 13:37 ` Simon Horman
0 siblings, 0 replies; 3+ messages in thread
From: Simon Horman @ 2018-09-14 13:37 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Niklas Söderlund, linux-renesas-soc, devicetree
On Wed, Sep 12, 2018 at 11:34:56AM +0200, Simon Horman wrote:
> On Tue, Sep 11, 2018 at 04:37:30PM +0200, Geert Uytterhoeven wrote:
> > The thermal device is supposed to be always enabled. As the default
> > value of the status property is "okay", there is no need to make this
> > explicit in SoC-specific .dtsi files where no override is involved.
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> Thanks,
>
> This looks fine to me but I will wait to see if there are other reviews
> before applying.
Thanks again, applied for v4.20.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-09-14 18:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-11 14:37 [PATCH] arm64: dts: renesas: Remove unneeded status from thermal nodes Geert Uytterhoeven
2018-09-12 9:34 ` Simon Horman
2018-09-14 13:37 ` Simon Horman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.