devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: am437x-idk-evm: Fix incorrect OPP node names
@ 2020-02-10 21:04 Suman Anna
  2020-02-20 17:51 ` Tony Lindgren
  0 siblings, 1 reply; 2+ messages in thread
From: Suman Anna @ 2020-02-10 21:04 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: Dave Gerlach, Roger Quadros, linux-omap, devicetree, Suman Anna

The commit 337c6c9a69af ("ARM: dts: am437x-idk-evm: Disable
OPP50 for MPU") adjusts couple of OPP nodes defined in the
common am4372.dtsi file, but used outdated node names. This
results in these getting treated as new OPP nodes with missing
properties.

Fix this properly by using the correct node names as updated in
commit b9cb2ba71848 ("ARM: dts: Use - instead of @ for DT OPP
entries for TI SoCs").

Reported-by: Roger Quadros <rogerq@ti.com>
Fixes: 337c6c9a69af ("ARM: dts: am437x-idk-evm: Disable OPP50 for MPU")
Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/am437x-idk-evm.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/am437x-idk-evm.dts b/arch/arm/boot/dts/am437x-idk-evm.dts
index f3ced6df0c9b..9f66f96d09c9 100644
--- a/arch/arm/boot/dts/am437x-idk-evm.dts
+++ b/arch/arm/boot/dts/am437x-idk-evm.dts
@@ -526,11 +526,11 @@
 	 * Supply voltage supervisor on board will not allow opp50 so
 	 * disable it and set opp100 as suspend OPP.
 	 */
-	opp50@300000000 {
+	opp50-300000000 {
 		status = "disabled";
 	};
 
-	opp100@600000000 {
+	opp100-600000000 {
 		opp-suspend;
 	};
 };
-- 
2.23.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ARM: dts: am437x-idk-evm: Fix incorrect OPP node names
  2020-02-10 21:04 [PATCH] ARM: dts: am437x-idk-evm: Fix incorrect OPP node names Suman Anna
@ 2020-02-20 17:51 ` Tony Lindgren
  0 siblings, 0 replies; 2+ messages in thread
From: Tony Lindgren @ 2020-02-20 17:51 UTC (permalink / raw)
  To: Suman Anna; +Cc: Dave Gerlach, Roger Quadros, linux-omap, devicetree

* Suman Anna <s-anna@ti.com> [200210 13:05]:
> The commit 337c6c9a69af ("ARM: dts: am437x-idk-evm: Disable
> OPP50 for MPU") adjusts couple of OPP nodes defined in the
> common am4372.dtsi file, but used outdated node names. This
> results in these getting treated as new OPP nodes with missing
> properties.
> 
> Fix this properly by using the correct node names as updated in
> commit b9cb2ba71848 ("ARM: dts: Use - instead of @ for DT OPP
> entries for TI SoCs").

Applying into fixes thanks.

Tony

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-02-20 17:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-10 21:04 [PATCH] ARM: dts: am437x-idk-evm: Fix incorrect OPP node names Suman Anna
2020-02-20 17:51 ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).