* [PATCH] ARM: dts: Fix the name of supplies for smsc911x shared by OMAP
@ 2013-12-02 12:53 Florian Vaussard
2013-12-02 19:29 ` Tony Lindgren
0 siblings, 1 reply; 2+ messages in thread
From: Florian Vaussard @ 2013-12-02 12:53 UTC (permalink / raw)
To: Tony Lindgren, Benoit Cousson; +Cc: linux-omap, devicetree, florian.vaussard
drivers/net/ethernet/smsc/smsc911x.c is expecting supplies named
"vdd33a" and "vddvario". Currently the shared DTS file provides
"vmmc" and "vmmc_aux", and the supply lookup will fail:
smsc911x 2c000000.ethernet: Looking up vdd33a-supply from device tree
smsc911x 2c000000.ethernet: Looking up vdd33a-supply property in node /ocp/gpmc@6e000000/ethernet@gpmc failed
smsc911x 2c000000.ethernet: Looking up vddvario-supply from device tree
smsc911x 2c000000.ethernet: Looking up vddvario-supply property in node /ocp/gpmc@6e000000/ethernet@gpmc failed
Fix it!
Tested on OMAP3 Overo platform.
Signed-off-by: Florian Vaussard <florian.vaussard@epfl.ch>
---
arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi b/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi
index 9c18adf..f577b7d 100644
--- a/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi
+++ b/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi
@@ -44,8 +44,8 @@
gpmc,wr-access-ns = <186>;
gpmc,cycle2cycle-samecsen;
gpmc,cycle2cycle-diffcsen;
- vmmc-supply = <&vddvario>;
- vmmc_aux-supply = <&vdd33a>;
+ vddvario-supply = <&vddvario>;
+ vdd33a-supply = <&vdd33a>;
reg-io-width = <4>;
smsc,save-mac-address;
};
--
1.8.1.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ARM: dts: Fix the name of supplies for smsc911x shared by OMAP
2013-12-02 12:53 [PATCH] ARM: dts: Fix the name of supplies for smsc911x shared by OMAP Florian Vaussard
@ 2013-12-02 19:29 ` Tony Lindgren
0 siblings, 0 replies; 2+ messages in thread
From: Tony Lindgren @ 2013-12-02 19:29 UTC (permalink / raw)
To: Florian Vaussard; +Cc: Benoit Cousson, linux-omap, devicetree
* Florian Vaussard <florian.vaussard@epfl.ch> [131202 04:54]:
> drivers/net/ethernet/smsc/smsc911x.c is expecting supplies named
> "vdd33a" and "vddvario". Currently the shared DTS file provides
> "vmmc" and "vmmc_aux", and the supply lookup will fail:
>
> smsc911x 2c000000.ethernet: Looking up vdd33a-supply from device tree
> smsc911x 2c000000.ethernet: Looking up vdd33a-supply property in node /ocp/gpmc@6e000000/ethernet@gpmc failed
> smsc911x 2c000000.ethernet: Looking up vddvario-supply from device tree
> smsc911x 2c000000.ethernet: Looking up vddvario-supply property in node /ocp/gpmc@6e000000/ethernet@gpmc failed
>
> Fix it!
>
> Tested on OMAP3 Overo platform.
OK, looks like we've had it wrong from the start as I just moved it from
the omap3-igep0020.dts file :) Applying into omap-for-v3.14/fixes-take4
Regards,
Tony
> Signed-off-by: Florian Vaussard <florian.vaussard@epfl.ch>
> ---
> arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi b/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi
> index 9c18adf..f577b7d 100644
> --- a/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi
> +++ b/arch/arm/boot/dts/omap-gpmc-smsc911x.dtsi
> @@ -44,8 +44,8 @@
> gpmc,wr-access-ns = <186>;
> gpmc,cycle2cycle-samecsen;
> gpmc,cycle2cycle-diffcsen;
> - vmmc-supply = <&vddvario>;
> - vmmc_aux-supply = <&vdd33a>;
> + vddvario-supply = <&vddvario>;
> + vdd33a-supply = <&vdd33a>;
> reg-io-width = <4>;
> smsc,save-mac-address;
> };
> --
> 1.8.1.2
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-12-02 19:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-02 12:53 [PATCH] ARM: dts: Fix the name of supplies for smsc911x shared by OMAP Florian Vaussard
2013-12-02 19:29 ` 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).