linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: imx7d-sdb: Fix regulator-usb-otg2-vbus node name
@ 2018-03-01 12:52 Leonard Crestez
  2018-03-01 15:29 ` Fabio Estevam
  2018-03-08  8:17 ` Shawn Guo
  0 siblings, 2 replies; 3+ messages in thread
From: Leonard Crestez @ 2018-03-01 12:52 UTC (permalink / raw)
  To: linux-arm-kernel

The two usb-otg regulators for imx7d-sdb are both called
"regulator-usb-otg1-vbus" and they effectively override each other.

This is most likely a copy-paste error.

Fixes: b877039aa1fe ("ARM: dts: imx7d-sdb: Adjust the regulator nodes")

Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
---
 arch/arm/boot/dts/imx7d-sdb.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/imx7d-sdb.dts b/arch/arm/boot/dts/imx7d-sdb.dts
index a7a5dc7..e7d2db8 100644
--- a/arch/arm/boot/dts/imx7d-sdb.dts
+++ b/arch/arm/boot/dts/imx7d-sdb.dts
@@ -75,21 +75,21 @@
 
 	reg_usb_otg1_vbus: regulator-usb-otg1-vbus {
 		compatible = "regulator-fixed";
 		regulator-name = "usb_otg1_vbus";
 		regulator-min-microvolt = <5000000>;
 		regulator-max-microvolt = <5000000>;
 		gpio = <&gpio1 5 GPIO_ACTIVE_HIGH>;
 		enable-active-high;
 	};
 
-	reg_usb_otg2_vbus: regulator-usb-otg1-vbus {
+	reg_usb_otg2_vbus: regulator-usb-otg2-vbus {
 		compatible = "regulator-fixed";
 		regulator-name = "usb_otg2_vbus";
 		regulator-min-microvolt = <5000000>;
 		regulator-max-microvolt = <5000000>;
 		gpio = <&gpio4 7 GPIO_ACTIVE_HIGH>;
 		enable-active-high;
 	};
 
 	reg_can2_3v3: regulator-can2-3v3 {
 		compatible = "regulator-fixed";
-- 
2.7.4

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

* [PATCH] ARM: dts: imx7d-sdb: Fix regulator-usb-otg2-vbus node name
  2018-03-01 12:52 [PATCH] ARM: dts: imx7d-sdb: Fix regulator-usb-otg2-vbus node name Leonard Crestez
@ 2018-03-01 15:29 ` Fabio Estevam
  2018-03-08  8:17 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Fabio Estevam @ 2018-03-01 15:29 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Mar 1, 2018 at 9:52 AM, Leonard Crestez <leonard.crestez@nxp.com> wrote:
> The two usb-otg regulators for imx7d-sdb are both called
> "regulator-usb-otg1-vbus" and they effectively override each other.
>
> This is most likely a copy-paste error.
>
> Fixes: b877039aa1fe ("ARM: dts: imx7d-sdb: Adjust the regulator nodes")
>
> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>

Good catch, thanks:

Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>

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

* [PATCH] ARM: dts: imx7d-sdb: Fix regulator-usb-otg2-vbus node name
  2018-03-01 12:52 [PATCH] ARM: dts: imx7d-sdb: Fix regulator-usb-otg2-vbus node name Leonard Crestez
  2018-03-01 15:29 ` Fabio Estevam
@ 2018-03-08  8:17 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2018-03-08  8:17 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Mar 01, 2018 at 02:52:08PM +0200, Leonard Crestez wrote:
> The two usb-otg regulators for imx7d-sdb are both called
> "regulator-usb-otg1-vbus" and they effectively override each other.
> 
> This is most likely a copy-paste error.
> 
> Fixes: b877039aa1fe ("ARM: dts: imx7d-sdb: Adjust the regulator nodes")
> 
> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>

Applied, thanks.

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

end of thread, other threads:[~2018-03-08  8:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-03-01 12:52 [PATCH] ARM: dts: imx7d-sdb: Fix regulator-usb-otg2-vbus node name Leonard Crestez
2018-03-01 15:29 ` Fabio Estevam
2018-03-08  8:17 ` Shawn Guo

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).