* [PATCH 0/2] ARM: dts: socfpga: fix booting with SD/MMC @ 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 0 siblings, 0 replies; 20+ messages in thread From: dinguyen at opensource.altera.com @ 2014-10-16 21:03 UTC (permalink / raw) To: linux-arm-kernel From: Dinh Nguyen <dinguyen@opensource.altera.com> The SOCFPGA dev kit was hanging during bootup on the SD/MMC driver loading. The first patch fixes the booting and the 2nd patch adds a regulator node for the SD/MMC driver to use. Dinh Nguyen (2): ARM: dts: socfpga: Fix SD card detect ARM: dts: socfpga: Add a 3.3V fixed regulator node arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 4 +++- arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ 5 files changed, 32 insertions(+), 2 deletions(-) -- 2.0.3 ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 0/2] ARM: dts: socfpga: fix booting with SD/MMC @ 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 0 siblings, 0 replies; 20+ messages in thread From: dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx @ 2014-10-16 21:03 UTC (permalink / raw) To: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, mark.rutland-5wv7dgnIgG8, pawel.moll-5wv7dgnIgG8, jh80.chung-Sze3O3UU22JBDgjK7y7TUQ, dianders-F7+t8E8rja9g9hUCZPvPmw Cc: dinh.linux-Re5JQEeQqe8AvxtiuMwx3w, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ, Dinh Nguyen From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> The SOCFPGA dev kit was hanging during bootup on the SD/MMC driver loading. The first patch fixes the booting and the 2nd patch adds a regulator node for the SD/MMC driver to use. Dinh Nguyen (2): ARM: dts: socfpga: Fix SD card detect ARM: dts: socfpga: Add a 3.3V fixed regulator node arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 4 +++- arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ 5 files changed, 32 insertions(+), 2 deletions(-) -- 2.0.3 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 1/2] ARM: dts: socfpga: Fix SD card detect @ 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 0 siblings, 0 replies; 20+ messages in thread From: dinguyen at opensource.altera.com @ 2014-10-16 21:03 UTC (permalink / raw) To: linux-arm-kernel From: Dinh Nguyen <dinguyen@opensource.altera.com> Without this patch, the booting the SOCFPGA platform would hang at the SDMMC driver loading. There were 2 patches that caused this to happen: - Patch 9795a846e10 "mmc: dw_mmc: remove dw_mci_of_cd_gpio/wp_gpio()" removed looking for "cd-gpios", since mmc_of_parse was getting called. - Patch 3cf890fc42b "mmc: dw_mmc: Pass back errors from mmc_of_parse()" would hang the system at the SDMMC driver loading. This patch will fix booting with SDMMC enabled on SOCFPGA dev kit. Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com> --- arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts index d7296a5..739c3b7 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts @@ -69,7 +69,7 @@ }; &mmc0 { - cd-gpios = <&gpio1 18 0>; + cd = <&gpio1 18 0>; }; &usb1 { -- 2.0.3 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 1/2] ARM: dts: socfpga: Fix SD card detect @ 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 0 siblings, 0 replies; 20+ messages in thread From: dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx @ 2014-10-16 21:03 UTC (permalink / raw) To: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, mark.rutland-5wv7dgnIgG8, pawel.moll-5wv7dgnIgG8, jh80.chung-Sze3O3UU22JBDgjK7y7TUQ, dianders-F7+t8E8rja9g9hUCZPvPmw Cc: dinh.linux-Re5JQEeQqe8AvxtiuMwx3w, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ, Dinh Nguyen From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> Without this patch, the booting the SOCFPGA platform would hang at the SDMMC driver loading. There were 2 patches that caused this to happen: - Patch 9795a846e10 "mmc: dw_mmc: remove dw_mci_of_cd_gpio/wp_gpio()" removed looking for "cd-gpios", since mmc_of_parse was getting called. - Patch 3cf890fc42b "mmc: dw_mmc: Pass back errors from mmc_of_parse()" would hang the system at the SDMMC driver loading. This patch will fix booting with SDMMC enabled on SOCFPGA dev kit. Signed-off-by: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> --- arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts index d7296a5..739c3b7 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts @@ -69,7 +69,7 @@ }; &mmc0 { - cd-gpios = <&gpio1 18 0>; + cd = <&gpio1 18 0>; }; &usb1 { -- 2.0.3 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 0 siblings, 0 replies; 20+ messages in thread From: dinguyen at opensource.altera.com @ 2014-10-16 21:03 UTC (permalink / raw) To: linux-arm-kernel From: Dinh Nguyen <dinguyen@opensource.altera.com> Without the 3.3V regulator node, the SDMMC driver will give these warnings: dw_mmc ff704000.dwmmc0: No vmmc regulator found dw_mmc ff704000.dwmmc0: No vqmmc regulator found This patch adds the regulator node, and points the SD/MMC to the regulator. Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com> --- arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ 5 files changed, 31 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi index 03e8268..8093781 100644 --- a/arch/arm/boot/dts/socfpga_arria5.dtsi +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi @@ -29,7 +29,7 @@ }; }; - dwmmc0 at ff704000 { + mmc0: dwmmc0 at ff704000 { num-slots = <1>; broken-cd; bus-width = <4>; @@ -41,4 +41,13 @@ cpu1-start-addr = <0xffd080c4>; }; }; + + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { + compatible = "regulator-fixed"; + regulator-name = "HPS 3.3V"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + regulator-always-on; + }; }; diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts index 27d551c..b7e4023 100644 --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts @@ -68,6 +68,11 @@ }; }; +&mmc0 { + vmmc-supply = <®ulator_3_3v_hps>; + vqmmc-supply = <®ulator_3_3v_hps>; +}; + &usb1 { status = "okay"; }; diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi index 28c05e7..743bc3b 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi +++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi @@ -48,4 +48,13 @@ cpu1-start-addr = <0xffd080c4>; }; }; + + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { + compatible = "regulator-fixed"; + regulator-name = "HPS 3.3V"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + regulator-always-on; + }; }; diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts index 739c3b7..e1f56ba 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts @@ -70,6 +70,8 @@ &mmc0 { cd = <&gpio1 18 0>; + vmmc-supply = <®ulator_3_3v_hps>; + vqmmc-supply = <®ulator_3_3v_hps>; }; &usb1 { diff --git a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts index d26f155..ea0c454 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts @@ -53,6 +53,11 @@ rxc-skew-ps = <2000>; }; +&mmc0 { + vmmc-supply = <®ulator_3_3v_hps>; + vqmmc-supply = <®ulator_3_3v_hps>; +}; + &usb1 { status = "okay"; }; -- 2.0.3 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 0 siblings, 0 replies; 20+ messages in thread From: dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx @ 2014-10-16 21:03 UTC (permalink / raw) To: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, mark.rutland-5wv7dgnIgG8, pawel.moll-5wv7dgnIgG8, jh80.chung-Sze3O3UU22JBDgjK7y7TUQ, dianders-F7+t8E8rja9g9hUCZPvPmw Cc: dinh.linux-Re5JQEeQqe8AvxtiuMwx3w, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ, Dinh Nguyen From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> Without the 3.3V regulator node, the SDMMC driver will give these warnings: dw_mmc ff704000.dwmmc0: No vmmc regulator found dw_mmc ff704000.dwmmc0: No vqmmc regulator found This patch adds the regulator node, and points the SD/MMC to the regulator. Signed-off-by: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> --- arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ 5 files changed, 31 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi index 03e8268..8093781 100644 --- a/arch/arm/boot/dts/socfpga_arria5.dtsi +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi @@ -29,7 +29,7 @@ }; }; - dwmmc0@ff704000 { + mmc0: dwmmc0@ff704000 { num-slots = <1>; broken-cd; bus-width = <4>; @@ -41,4 +41,13 @@ cpu1-start-addr = <0xffd080c4>; }; }; + + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { + compatible = "regulator-fixed"; + regulator-name = "HPS 3.3V"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + regulator-always-on; + }; }; diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts index 27d551c..b7e4023 100644 --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts @@ -68,6 +68,11 @@ }; }; +&mmc0 { + vmmc-supply = <®ulator_3_3v_hps>; + vqmmc-supply = <®ulator_3_3v_hps>; +}; + &usb1 { status = "okay"; }; diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi index 28c05e7..743bc3b 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi +++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi @@ -48,4 +48,13 @@ cpu1-start-addr = <0xffd080c4>; }; }; + + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { + compatible = "regulator-fixed"; + regulator-name = "HPS 3.3V"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + regulator-always-on; + }; }; diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts index 739c3b7..e1f56ba 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts @@ -70,6 +70,8 @@ &mmc0 { cd = <&gpio1 18 0>; + vmmc-supply = <®ulator_3_3v_hps>; + vqmmc-supply = <®ulator_3_3v_hps>; }; &usb1 { diff --git a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts index d26f155..ea0c454 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts @@ -53,6 +53,11 @@ rxc-skew-ps = <2000>; }; +&mmc0 { + vmmc-supply = <®ulator_3_3v_hps>; + vqmmc-supply = <®ulator_3_3v_hps>; +}; + &usb1 { status = "okay"; }; -- 2.0.3 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 1:43 ` Jaehoon Chung 0 siblings, 0 replies; 20+ messages in thread From: Jaehoon Chung @ 2014-10-17 1:43 UTC (permalink / raw) To: linux-arm-kernel Hi, Dinh. On 10/17/2014 06:03 AM, dinguyen at opensource.altera.com wrote: > From: Dinh Nguyen <dinguyen@opensource.altera.com> > > Without the 3.3V regulator node, the SDMMC driver will give these warnings: > > dw_mmc ff704000.dwmmc0: No vmmc regulator found > dw_mmc ff704000.dwmmc0: No vqmmc regulator found > > This patch adds the regulator node, and points the SD/MMC to the regulator. > > Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com> > --- > arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- > arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ > arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ > arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ > arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ > 5 files changed, 31 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi > index 03e8268..8093781 100644 > --- a/arch/arm/boot/dts/socfpga_arria5.dtsi > +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi > @@ -29,7 +29,7 @@ > }; > }; > > - dwmmc0 at ff704000 { > + mmc0: dwmmc0 at ff704000 { > num-slots = <1>; > broken-cd; > bus-width = <4>; > @@ -41,4 +41,13 @@ > cpu1-start-addr = <0xffd080c4>; > }; > }; > + > + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { > + compatible = "regulator-fixed"; > + regulator-name = "HPS 3.3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + regulator-always-on; Always-on is right? > + }; > }; > diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts > index 27d551c..b7e4023 100644 > --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts > +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts > @@ -68,6 +68,11 @@ > }; > }; > > +&mmc0 { > + vmmc-supply = <®ulator_3_3v_hps>; > + vqmmc-supply = <®ulator_3_3v_hps>; > +}; > + > &usb1 { > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi > index 28c05e7..743bc3b 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi > +++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi > @@ -48,4 +48,13 @@ > cpu1-start-addr = <0xffd080c4>; > }; > }; > + > + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { > + compatible = "regulator-fixed"; > + regulator-name = "HPS 3.3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + regulator-always-on; > + }; > }; > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts > index 739c3b7..e1f56ba 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts > +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts > @@ -70,6 +70,8 @@ > > &mmc0 { > cd = <&gpio1 18 0>; > + vmmc-supply = <®ulator_3_3v_hps>; > + vqmmc-supply = <®ulator_3_3v_hps>; Is vmmc and vqmmc used the common supply? Best Regards, Jaehoon Chung > }; > > &usb1 { > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts > index d26f155..ea0c454 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts > +++ b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts > @@ -53,6 +53,11 @@ > rxc-skew-ps = <2000>; > }; > > +&mmc0 { > + vmmc-supply = <®ulator_3_3v_hps>; > + vqmmc-supply = <®ulator_3_3v_hps>; > +}; > + > &usb1 { > status = "okay"; > }; > ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 1:43 ` Jaehoon Chung 0 siblings, 0 replies; 20+ messages in thread From: Jaehoon Chung @ 2014-10-17 1:43 UTC (permalink / raw) To: dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, mark.rutland-5wv7dgnIgG8, pawel.moll-5wv7dgnIgG8, dianders-F7+t8E8rja9g9hUCZPvPmw Cc: dinh.linux-Re5JQEeQqe8AvxtiuMwx3w, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ Hi, Dinh. On 10/17/2014 06:03 AM, dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org wrote: > From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> > > Without the 3.3V regulator node, the SDMMC driver will give these warnings: > > dw_mmc ff704000.dwmmc0: No vmmc regulator found > dw_mmc ff704000.dwmmc0: No vqmmc regulator found > > This patch adds the regulator node, and points the SD/MMC to the regulator. > > Signed-off-by: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> > --- > arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- > arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ > arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ > arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ > arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ > 5 files changed, 31 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi > index 03e8268..8093781 100644 > --- a/arch/arm/boot/dts/socfpga_arria5.dtsi > +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi > @@ -29,7 +29,7 @@ > }; > }; > > - dwmmc0@ff704000 { > + mmc0: dwmmc0@ff704000 { > num-slots = <1>; > broken-cd; > bus-width = <4>; > @@ -41,4 +41,13 @@ > cpu1-start-addr = <0xffd080c4>; > }; > }; > + > + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { > + compatible = "regulator-fixed"; > + regulator-name = "HPS 3.3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + regulator-always-on; Always-on is right? > + }; > }; > diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts > index 27d551c..b7e4023 100644 > --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts > +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts > @@ -68,6 +68,11 @@ > }; > }; > > +&mmc0 { > + vmmc-supply = <®ulator_3_3v_hps>; > + vqmmc-supply = <®ulator_3_3v_hps>; > +}; > + > &usb1 { > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi > index 28c05e7..743bc3b 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi > +++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi > @@ -48,4 +48,13 @@ > cpu1-start-addr = <0xffd080c4>; > }; > }; > + > + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { > + compatible = "regulator-fixed"; > + regulator-name = "HPS 3.3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + regulator-always-on; > + }; > }; > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts > index 739c3b7..e1f56ba 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts > +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts > @@ -70,6 +70,8 @@ > > &mmc0 { > cd = <&gpio1 18 0>; > + vmmc-supply = <®ulator_3_3v_hps>; > + vqmmc-supply = <®ulator_3_3v_hps>; Is vmmc and vqmmc used the common supply? Best Regards, Jaehoon Chung > }; > > &usb1 { > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts > index d26f155..ea0c454 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts > +++ b/arch/arm/boot/dts/socfpga_cyclone5_sockit.dts > @@ -53,6 +53,11 @@ > rxc-skew-ps = <2000>; > }; > > +&mmc0 { > + vmmc-supply = <®ulator_3_3v_hps>; > + vqmmc-supply = <®ulator_3_3v_hps>; > +}; > + > &usb1 { > status = "okay"; > }; > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 13:37 ` Dinh Nguyen 0 siblings, 0 replies; 20+ messages in thread From: Dinh Nguyen @ 2014-10-17 13:37 UTC (permalink / raw) To: linux-arm-kernel On 10/16/14, 8:43 PM, Jaehoon Chung wrote: > Hi, Dinh. > > On 10/17/2014 06:03 AM, dinguyen at opensource.altera.com wrote: >> From: Dinh Nguyen <dinguyen@opensource.altera.com> >> >> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >> >> dw_mmc ff704000.dwmmc0: No vmmc regulator found >> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >> >> This patch adds the regulator node, and points the SD/MMC to the regulator. >> >> Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com> >> --- >> arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- >> arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ >> arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ >> arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ >> arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ >> 5 files changed, 31 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi >> index 03e8268..8093781 100644 >> --- a/arch/arm/boot/dts/socfpga_arria5.dtsi >> +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi >> @@ -29,7 +29,7 @@ >> }; >> }; >> >> - dwmmc0 at ff704000 { >> + mmc0: dwmmc0 at ff704000 { >> num-slots = <1>; >> broken-cd; >> bus-width = <4>; >> @@ -41,4 +41,13 @@ >> cpu1-start-addr = <0xffd080c4>; >> }; >> }; >> + >> + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { >> + compatible = "regulator-fixed"; >> + regulator-name = "HPS 3.3V"; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-boot-on; >> + regulator-always-on; > > Always-on is right? yes, it's always on. > >> + }; >> }; >> diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts >> index 27d551c..b7e4023 100644 >> --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts >> +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts >> @@ -68,6 +68,11 @@ >> }; >> }; >> >> +&mmc0 { >> + vmmc-supply = <®ulator_3_3v_hps>; >> + vqmmc-supply = <®ulator_3_3v_hps>; >> +}; >> + >> &usb1 { >> status = "okay"; >> }; >> diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi >> index 28c05e7..743bc3b 100644 >> --- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi >> +++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi >> @@ -48,4 +48,13 @@ >> cpu1-start-addr = <0xffd080c4>; >> }; >> }; >> + >> + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { >> + compatible = "regulator-fixed"; >> + regulator-name = "HPS 3.3V"; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-boot-on; >> + regulator-always-on; >> + }; >> }; >> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts >> index 739c3b7..e1f56ba 100644 >> --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts >> +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts >> @@ -70,6 +70,8 @@ >> >> &mmc0 { >> cd = <&gpio1 18 0>; >> + vmmc-supply = <®ulator_3_3v_hps>; >> + vqmmc-supply = <®ulator_3_3v_hps>; > > Is vmmc and vqmmc used the common supply? > Yes.. Dinh ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 13:37 ` Dinh Nguyen 0 siblings, 0 replies; 20+ messages in thread From: Dinh Nguyen @ 2014-10-17 13:37 UTC (permalink / raw) To: Jaehoon Chung, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, mark.rutland-5wv7dgnIgG8, pawel.moll-5wv7dgnIgG8, dianders-F7+t8E8rja9g9hUCZPvPmw Cc: dinh.linux-Re5JQEeQqe8AvxtiuMwx3w, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ On 10/16/14, 8:43 PM, Jaehoon Chung wrote: > Hi, Dinh. > > On 10/17/2014 06:03 AM, dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org wrote: >> From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> >> >> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >> >> dw_mmc ff704000.dwmmc0: No vmmc regulator found >> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >> >> This patch adds the regulator node, and points the SD/MMC to the regulator. >> >> Signed-off-by: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> >> --- >> arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- >> arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ >> arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ >> arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ >> arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ >> 5 files changed, 31 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi >> index 03e8268..8093781 100644 >> --- a/arch/arm/boot/dts/socfpga_arria5.dtsi >> +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi >> @@ -29,7 +29,7 @@ >> }; >> }; >> >> - dwmmc0@ff704000 { >> + mmc0: dwmmc0@ff704000 { >> num-slots = <1>; >> broken-cd; >> bus-width = <4>; >> @@ -41,4 +41,13 @@ >> cpu1-start-addr = <0xffd080c4>; >> }; >> }; >> + >> + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { >> + compatible = "regulator-fixed"; >> + regulator-name = "HPS 3.3V"; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-boot-on; >> + regulator-always-on; > > Always-on is right? yes, it's always on. > >> + }; >> }; >> diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts >> index 27d551c..b7e4023 100644 >> --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts >> +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts >> @@ -68,6 +68,11 @@ >> }; >> }; >> >> +&mmc0 { >> + vmmc-supply = <®ulator_3_3v_hps>; >> + vqmmc-supply = <®ulator_3_3v_hps>; >> +}; >> + >> &usb1 { >> status = "okay"; >> }; >> diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi >> index 28c05e7..743bc3b 100644 >> --- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi >> +++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi >> @@ -48,4 +48,13 @@ >> cpu1-start-addr = <0xffd080c4>; >> }; >> }; >> + >> + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { >> + compatible = "regulator-fixed"; >> + regulator-name = "HPS 3.3V"; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-boot-on; >> + regulator-always-on; >> + }; >> }; >> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts >> index 739c3b7..e1f56ba 100644 >> --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts >> +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts >> @@ -70,6 +70,8 @@ >> >> &mmc0 { >> cd = <&gpio1 18 0>; >> + vmmc-supply = <®ulator_3_3v_hps>; >> + vqmmc-supply = <®ulator_3_3v_hps>; > > Is vmmc and vqmmc used the common supply? > Yes.. Dinh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 16:57 ` Doug Anderson 0 siblings, 0 replies; 20+ messages in thread From: Doug Anderson @ 2014-10-17 16:57 UTC (permalink / raw) To: linux-arm-kernel Dinh, On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen@opensource.altera.com> wrote: > From: Dinh Nguyen <dinguyen@opensource.altera.com> > > Without the 3.3V regulator node, the SDMMC driver will give these warnings: > > dw_mmc ff704000.dwmmc0: No vmmc regulator found > dw_mmc ff704000.dwmmc0: No vqmmc regulator found > > This patch adds the regulator node, and points the SD/MMC to the regulator. > > Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com> > --- > arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- > arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ > arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ > arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ > arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ > 5 files changed, 31 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi > index 03e8268..8093781 100644 > --- a/arch/arm/boot/dts/socfpga_arria5.dtsi > +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi > @@ -29,7 +29,7 @@ > }; > }; > > - dwmmc0 at ff704000 { > + mmc0: dwmmc0 at ff704000 { > num-slots = <1>; > broken-cd; > bus-width = <4>; > @@ -41,4 +41,13 @@ > cpu1-start-addr = <0xffd080c4>; > }; > }; > + > + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { nit: no @0 since there is no "reg" (register) under this node. nit: usually people don't like "_" in node names. ...I would probably do this but I'm not an expert: regulator_3_3v_hps: hps-regulator { This regulator also looks pretty bogus to me. Is this really a regulator that software has no control over? It means you can't fully reset a card but I guess that's OK. I'd also expect this regulator to be defined in the same dts / dtsi file that it's used in. Your current patch says "there's a generic 3.3V regulator on all boards of the socfpga_arria5 class even if they don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is that really true? ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 16:57 ` Doug Anderson 0 siblings, 0 replies; 20+ messages in thread From: Doug Anderson @ 2014-10-17 16:57 UTC (permalink / raw) To: dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx Cc: Rob Herring, Ian Campbell, Kumar Gala, Mark Rutland, Pawel Moll, Jaehoon Chung, Dinh Nguyen, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ Dinh, On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: > From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> > > Without the 3.3V regulator node, the SDMMC driver will give these warnings: > > dw_mmc ff704000.dwmmc0: No vmmc regulator found > dw_mmc ff704000.dwmmc0: No vqmmc regulator found > > This patch adds the regulator node, and points the SD/MMC to the regulator. > > Signed-off-by: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> > --- > arch/arm/boot/dts/socfpga_arria5.dtsi | 11 ++++++++++- > arch/arm/boot/dts/socfpga_arria5_socdk.dts | 5 +++++ > arch/arm/boot/dts/socfpga_cyclone5.dtsi | 9 +++++++++ > arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 2 ++ > arch/arm/boot/dts/socfpga_cyclone5_sockit.dts | 5 +++++ > 5 files changed, 31 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi > index 03e8268..8093781 100644 > --- a/arch/arm/boot/dts/socfpga_arria5.dtsi > +++ b/arch/arm/boot/dts/socfpga_arria5.dtsi > @@ -29,7 +29,7 @@ > }; > }; > > - dwmmc0@ff704000 { > + mmc0: dwmmc0@ff704000 { > num-slots = <1>; > broken-cd; > bus-width = <4>; > @@ -41,4 +41,13 @@ > cpu1-start-addr = <0xffd080c4>; > }; > }; > + > + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { nit: no @0 since there is no "reg" (register) under this node. nit: usually people don't like "_" in node names. ...I would probably do this but I'm not an expert: regulator_3_3v_hps: hps-regulator { This regulator also looks pretty bogus to me. Is this really a regulator that software has no control over? It means you can't fully reset a card but I guess that's OK. I'd also expect this regulator to be defined in the same dts / dtsi file that it's used in. Your current patch says "there's a generic 3.3V regulator on all boards of the socfpga_arria5 class even if they don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is that really true? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 19:19 ` Dinh Nguyen 0 siblings, 0 replies; 20+ messages in thread From: Dinh Nguyen @ 2014-10-17 19:19 UTC (permalink / raw) To: linux-arm-kernel On 10/17/2014 11:57 AM, Doug Anderson wrote: > Dinh, > > On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen@opensource.altera.com> wrote: >> From: Dinh Nguyen <dinguyen@opensource.altera.com> >> >> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >> >> dw_mmc ff704000.dwmmc0: No vmmc regulator found >> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >> >> This patch adds the regulator node, and points the SD/MMC to the regulator. >> [...] >> >> - dwmmc0 at ff704000 { >> + mmc0: dwmmc0 at ff704000 { >> num-slots = <1>; >> broken-cd; >> bus-width = <4>; >> @@ -41,4 +41,13 @@ >> cpu1-start-addr = <0xffd080c4>; >> }; >> }; >> + >> + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { > > nit: no @0 since there is no "reg" (register) under this node. > > nit: usually people don't like "_" in node names. ...I would probably > do this but I'm not an expert: > regulator_3_3v_hps: hps-regulator { Ok..will fix up. > > This regulator also looks pretty bogus to me. Is this really a > regulator that software has no control over? It means you can't fully > reset a card but I guess that's OK. Yes, this is a generic 3.3V regulator that is used for the 3.3V power rail for the SOC, and any IP the needs 3.3V. Schematics are here: http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf And right, I don't see anyway to be able to reset the card. > > I'd also expect this regulator to be defined in the same dts / dtsi > file that it's used in. Your current patch says "there's a generic > 3.3V regulator on all boards of the socfpga_arria5 class even if they > don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is > that really true? > yeah...My original patch had the regulator placed in it's appropriate dts board file, but I saw that as a bit of duplication. I figured that most people are just copying Altera's devkit schematic, that if a new board file shows up that has a different controllable regulator, then the board file can have the new regulator node. Meanwhile, this patch saves adding an additional regulator node for the existing board files. The 3.3V is not just used by MMC, but the for all 3.3V peripherals. Thanks for feedback. Dinh ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 19:19 ` Dinh Nguyen 0 siblings, 0 replies; 20+ messages in thread From: Dinh Nguyen @ 2014-10-17 19:19 UTC (permalink / raw) To: Doug Anderson Cc: Rob Herring, Ian Campbell, Kumar Gala, Mark Rutland, Pawel Moll, Jaehoon Chung, Dinh Nguyen, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ On 10/17/2014 11:57 AM, Doug Anderson wrote: > Dinh, > > On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: >> From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> >> >> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >> >> dw_mmc ff704000.dwmmc0: No vmmc regulator found >> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >> >> This patch adds the regulator node, and points the SD/MMC to the regulator. >> [...] >> >> - dwmmc0@ff704000 { >> + mmc0: dwmmc0@ff704000 { >> num-slots = <1>; >> broken-cd; >> bus-width = <4>; >> @@ -41,4 +41,13 @@ >> cpu1-start-addr = <0xffd080c4>; >> }; >> }; >> + >> + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { > > nit: no @0 since there is no "reg" (register) under this node. > > nit: usually people don't like "_" in node names. ...I would probably > do this but I'm not an expert: > regulator_3_3v_hps: hps-regulator { Ok..will fix up. > > This regulator also looks pretty bogus to me. Is this really a > regulator that software has no control over? It means you can't fully > reset a card but I guess that's OK. Yes, this is a generic 3.3V regulator that is used for the 3.3V power rail for the SOC, and any IP the needs 3.3V. Schematics are here: http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf And right, I don't see anyway to be able to reset the card. > > I'd also expect this regulator to be defined in the same dts / dtsi > file that it's used in. Your current patch says "there's a generic > 3.3V regulator on all boards of the socfpga_arria5 class even if they > don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is > that really true? > yeah...My original patch had the regulator placed in it's appropriate dts board file, but I saw that as a bit of duplication. I figured that most people are just copying Altera's devkit schematic, that if a new board file shows up that has a different controllable regulator, then the board file can have the new regulator node. Meanwhile, this patch saves adding an additional regulator node for the existing board files. The 3.3V is not just used by MMC, but the for all 3.3V peripherals. Thanks for feedback. Dinh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 19:43 ` Doug Anderson 0 siblings, 0 replies; 20+ messages in thread From: Doug Anderson @ 2014-10-17 19:43 UTC (permalink / raw) To: linux-arm-kernel Hi, On Fri, Oct 17, 2014 at 12:19 PM, Dinh Nguyen <dinguyen@opensource.altera.com> wrote: > On 10/17/2014 11:57 AM, Doug Anderson wrote: >> Dinh, >> >> On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen@opensource.altera.com> wrote: >>> From: Dinh Nguyen <dinguyen@opensource.altera.com> >>> >>> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >>> >>> dw_mmc ff704000.dwmmc0: No vmmc regulator found >>> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >>> >>> This patch adds the regulator node, and points the SD/MMC to the regulator. >>> > [...] >>> >>> - dwmmc0 at ff704000 { >>> + mmc0: dwmmc0 at ff704000 { >>> num-slots = <1>; >>> broken-cd; >>> bus-width = <4>; >>> @@ -41,4 +41,13 @@ >>> cpu1-start-addr = <0xffd080c4>; >>> }; >>> }; >>> + >>> + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { >> >> nit: no @0 since there is no "reg" (register) under this node. >> >> nit: usually people don't like "_" in node names. ...I would probably >> do this but I'm not an expert: >> regulator_3_3v_hps: hps-regulator { > > Ok..will fix up. > >> >> This regulator also looks pretty bogus to me. Is this really a >> regulator that software has no control over? It means you can't fully >> reset a card but I guess that's OK. > > Yes, this is a generic 3.3V regulator that is used for the 3.3V power > rail for the SOC, and any IP the needs 3.3V. > > Schematics are here: > http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 > > http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf > > And right, I don't see anyway to be able to reset the card. > >> >> I'd also expect this regulator to be defined in the same dts / dtsi >> file that it's used in. Your current patch says "there's a generic >> 3.3V regulator on all boards of the socfpga_arria5 class even if they >> don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is >> that really true? >> > > yeah...My original patch had the regulator placed in it's appropriate > dts board file, but I saw that as a bit of duplication. I figured that > most people are just copying Altera's devkit schematic, that if a new > board file shows up that has a different controllable regulator, then > the board file can have the new regulator node. Meanwhile, this patch > saves adding an additional regulator node for the existing board files. > > The 3.3V is not just used by MMC, but the for all 3.3V peripherals. The advantage of putting it in each board file is you could name it based on the schematics. I see SoCKit_RevC it is named "VCC3P3" It also allows you to specify the 3.3 supply as the output of a real regulator driver. In s5_soc_devkit it looks like this comes from LTC2978. That's got i2c connections. What happens when you add a driver for that? ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 19:43 ` Doug Anderson 0 siblings, 0 replies; 20+ messages in thread From: Doug Anderson @ 2014-10-17 19:43 UTC (permalink / raw) To: Dinh Nguyen Cc: Rob Herring, Ian Campbell, Kumar Gala, Mark Rutland, Pawel Moll, Jaehoon Chung, Dinh Nguyen, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ Hi, On Fri, Oct 17, 2014 at 12:19 PM, Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: > On 10/17/2014 11:57 AM, Doug Anderson wrote: >> Dinh, >> >> On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: >>> From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> >>> >>> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >>> >>> dw_mmc ff704000.dwmmc0: No vmmc regulator found >>> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >>> >>> This patch adds the regulator node, and points the SD/MMC to the regulator. >>> > [...] >>> >>> - dwmmc0@ff704000 { >>> + mmc0: dwmmc0@ff704000 { >>> num-slots = <1>; >>> broken-cd; >>> bus-width = <4>; >>> @@ -41,4 +41,13 @@ >>> cpu1-start-addr = <0xffd080c4>; >>> }; >>> }; >>> + >>> + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { >> >> nit: no @0 since there is no "reg" (register) under this node. >> >> nit: usually people don't like "_" in node names. ...I would probably >> do this but I'm not an expert: >> regulator_3_3v_hps: hps-regulator { > > Ok..will fix up. > >> >> This regulator also looks pretty bogus to me. Is this really a >> regulator that software has no control over? It means you can't fully >> reset a card but I guess that's OK. > > Yes, this is a generic 3.3V regulator that is used for the 3.3V power > rail for the SOC, and any IP the needs 3.3V. > > Schematics are here: > http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 > > http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf > > And right, I don't see anyway to be able to reset the card. > >> >> I'd also expect this regulator to be defined in the same dts / dtsi >> file that it's used in. Your current patch says "there's a generic >> 3.3V regulator on all boards of the socfpga_arria5 class even if they >> don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is >> that really true? >> > > yeah...My original patch had the regulator placed in it's appropriate > dts board file, but I saw that as a bit of duplication. I figured that > most people are just copying Altera's devkit schematic, that if a new > board file shows up that has a different controllable regulator, then > the board file can have the new regulator node. Meanwhile, this patch > saves adding an additional regulator node for the existing board files. > > The 3.3V is not just used by MMC, but the for all 3.3V peripherals. The advantage of putting it in each board file is you could name it based on the schematics. I see SoCKit_RevC it is named "VCC3P3" It also allows you to specify the 3.3 supply as the output of a real regulator driver. In s5_soc_devkit it looks like this comes from LTC2978. That's got i2c connections. What happens when you add a driver for that? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 20:04 ` Dinh Nguyen 0 siblings, 0 replies; 20+ messages in thread From: Dinh Nguyen @ 2014-10-17 20:04 UTC (permalink / raw) To: linux-arm-kernel On 10/17/2014 02:43 PM, Doug Anderson wrote: > Hi, > > On Fri, Oct 17, 2014 at 12:19 PM, Dinh Nguyen > <dinguyen@opensource.altera.com> wrote: >> On 10/17/2014 11:57 AM, Doug Anderson wrote: >>> Dinh, >>> >>> On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen@opensource.altera.com> wrote: >>>> From: Dinh Nguyen <dinguyen@opensource.altera.com> >>>> >>>> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >>>> >>>> dw_mmc ff704000.dwmmc0: No vmmc regulator found >>>> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >>>> >>>> This patch adds the regulator node, and points the SD/MMC to the regulator. >>>> >> [...] >>>> >>>> - dwmmc0 at ff704000 { >>>> + mmc0: dwmmc0 at ff704000 { >>>> num-slots = <1>; >>>> broken-cd; >>>> bus-width = <4>; >>>> @@ -41,4 +41,13 @@ >>>> cpu1-start-addr = <0xffd080c4>; >>>> }; >>>> }; >>>> + >>>> + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { >>> >>> nit: no @0 since there is no "reg" (register) under this node. >>> >>> nit: usually people don't like "_" in node names. ...I would probably >>> do this but I'm not an expert: >>> regulator_3_3v_hps: hps-regulator { >> >> Ok..will fix up. >> >>> >>> This regulator also looks pretty bogus to me. Is this really a >>> regulator that software has no control over? It means you can't fully >>> reset a card but I guess that's OK. >> >> Yes, this is a generic 3.3V regulator that is used for the 3.3V power >> rail for the SOC, and any IP the needs 3.3V. >> >> Schematics are here: >> http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 >> >> http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf >> >> And right, I don't see anyway to be able to reset the card. >> >>> >>> I'd also expect this regulator to be defined in the same dts / dtsi >>> file that it's used in. Your current patch says "there's a generic >>> 3.3V regulator on all boards of the socfpga_arria5 class even if they >>> don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is >>> that really true? >>> >> >> yeah...My original patch had the regulator placed in it's appropriate >> dts board file, but I saw that as a bit of duplication. I figured that >> most people are just copying Altera's devkit schematic, that if a new >> board file shows up that has a different controllable regulator, then >> the board file can have the new regulator node. Meanwhile, this patch >> saves adding an additional regulator node for the existing board files. >> >> The 3.3V is not just used by MMC, but the for all 3.3V peripherals. > > The advantage of putting it in each board file is you could name it > based on the schematics. I see SoCKit_RevC it is named "VCC3P3" > That's fine. I can edit the patch so that each regulator node goes into it's respective board dts file. > It also allows you to specify the 3.3 supply as the output of a real > regulator driver. In s5_soc_devkit it looks like this comes from > LTC2978. That's got i2c connections. What happens when you add a > driver for that? > Do you mean the c5_soc_devkit? If so, then the 3.3V is coming from the LTC3885 regulator. The LTC2978 is a regulator for the the 3.3V_HPS or 3.3V_REG_HPS which is used by others. So I should change the regulator name from 3.3V_HPS to just 3.3V for the c5_soc_devkit. Dinh ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 20:04 ` Dinh Nguyen 0 siblings, 0 replies; 20+ messages in thread From: Dinh Nguyen @ 2014-10-17 20:04 UTC (permalink / raw) To: Doug Anderson Cc: Rob Herring, Ian Campbell, Kumar Gala, Mark Rutland, Pawel Moll, Jaehoon Chung, Dinh Nguyen, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ On 10/17/2014 02:43 PM, Doug Anderson wrote: > Hi, > > On Fri, Oct 17, 2014 at 12:19 PM, Dinh Nguyen > <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: >> On 10/17/2014 11:57 AM, Doug Anderson wrote: >>> Dinh, >>> >>> On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: >>>> From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> >>>> >>>> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >>>> >>>> dw_mmc ff704000.dwmmc0: No vmmc regulator found >>>> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >>>> >>>> This patch adds the regulator node, and points the SD/MMC to the regulator. >>>> >> [...] >>>> >>>> - dwmmc0@ff704000 { >>>> + mmc0: dwmmc0@ff704000 { >>>> num-slots = <1>; >>>> broken-cd; >>>> bus-width = <4>; >>>> @@ -41,4 +41,13 @@ >>>> cpu1-start-addr = <0xffd080c4>; >>>> }; >>>> }; >>>> + >>>> + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { >>> >>> nit: no @0 since there is no "reg" (register) under this node. >>> >>> nit: usually people don't like "_" in node names. ...I would probably >>> do this but I'm not an expert: >>> regulator_3_3v_hps: hps-regulator { >> >> Ok..will fix up. >> >>> >>> This regulator also looks pretty bogus to me. Is this really a >>> regulator that software has no control over? It means you can't fully >>> reset a card but I guess that's OK. >> >> Yes, this is a generic 3.3V regulator that is used for the 3.3V power >> rail for the SOC, and any IP the needs 3.3V. >> >> Schematics are here: >> http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 >> >> http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf >> >> And right, I don't see anyway to be able to reset the card. >> >>> >>> I'd also expect this regulator to be defined in the same dts / dtsi >>> file that it's used in. Your current patch says "there's a generic >>> 3.3V regulator on all boards of the socfpga_arria5 class even if they >>> don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is >>> that really true? >>> >> >> yeah...My original patch had the regulator placed in it's appropriate >> dts board file, but I saw that as a bit of duplication. I figured that >> most people are just copying Altera's devkit schematic, that if a new >> board file shows up that has a different controllable regulator, then >> the board file can have the new regulator node. Meanwhile, this patch >> saves adding an additional regulator node for the existing board files. >> >> The 3.3V is not just used by MMC, but the for all 3.3V peripherals. > > The advantage of putting it in each board file is you could name it > based on the schematics. I see SoCKit_RevC it is named "VCC3P3" > That's fine. I can edit the patch so that each regulator node goes into it's respective board dts file. > It also allows you to specify the 3.3 supply as the output of a real > regulator driver. In s5_soc_devkit it looks like this comes from > LTC2978. That's got i2c connections. What happens when you add a > driver for that? > Do you mean the c5_soc_devkit? If so, then the 3.3V is coming from the LTC3885 regulator. The LTC2978 is a regulator for the the 3.3V_HPS or 3.3V_REG_HPS which is used by others. So I should change the regulator name from 3.3V_HPS to just 3.3V for the c5_soc_devkit. Dinh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 22:36 ` Doug Anderson 0 siblings, 0 replies; 20+ messages in thread From: Doug Anderson @ 2014-10-17 22:36 UTC (permalink / raw) To: linux-arm-kernel Dinh, On Fri, Oct 17, 2014 at 1:04 PM, Dinh Nguyen <dinguyen@opensource.altera.com> wrote: > On 10/17/2014 02:43 PM, Doug Anderson wrote: >> Hi, >> >> On Fri, Oct 17, 2014 at 12:19 PM, Dinh Nguyen >> <dinguyen@opensource.altera.com> wrote: >>> On 10/17/2014 11:57 AM, Doug Anderson wrote: >>>> Dinh, >>>> >>>> On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen@opensource.altera.com> wrote: >>>>> From: Dinh Nguyen <dinguyen@opensource.altera.com> >>>>> >>>>> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >>>>> >>>>> dw_mmc ff704000.dwmmc0: No vmmc regulator found >>>>> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >>>>> >>>>> This patch adds the regulator node, and points the SD/MMC to the regulator. >>>>> >>> [...] >>>>> >>>>> - dwmmc0 at ff704000 { >>>>> + mmc0: dwmmc0 at ff704000 { >>>>> num-slots = <1>; >>>>> broken-cd; >>>>> bus-width = <4>; >>>>> @@ -41,4 +41,13 @@ >>>>> cpu1-start-addr = <0xffd080c4>; >>>>> }; >>>>> }; >>>>> + >>>>> + regulator_3_3v_hps: fixed_3_3v_hps_regulator at 0 { >>>> >>>> nit: no @0 since there is no "reg" (register) under this node. >>>> >>>> nit: usually people don't like "_" in node names. ...I would probably >>>> do this but I'm not an expert: >>>> regulator_3_3v_hps: hps-regulator { >>> >>> Ok..will fix up. >>> >>>> >>>> This regulator also looks pretty bogus to me. Is this really a >>>> regulator that software has no control over? It means you can't fully >>>> reset a card but I guess that's OK. >>> >>> Yes, this is a generic 3.3V regulator that is used for the 3.3V power >>> rail for the SOC, and any IP the needs 3.3V. >>> >>> Schematics are here: >>> http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 >>> >>> http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf >>> >>> And right, I don't see anyway to be able to reset the card. >>> >>>> >>>> I'd also expect this regulator to be defined in the same dts / dtsi >>>> file that it's used in. Your current patch says "there's a generic >>>> 3.3V regulator on all boards of the socfpga_arria5 class even if they >>>> don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is >>>> that really true? >>>> >>> >>> yeah...My original patch had the regulator placed in it's appropriate >>> dts board file, but I saw that as a bit of duplication. I figured that >>> most people are just copying Altera's devkit schematic, that if a new >>> board file shows up that has a different controllable regulator, then >>> the board file can have the new regulator node. Meanwhile, this patch >>> saves adding an additional regulator node for the existing board files. >>> >>> The 3.3V is not just used by MMC, but the for all 3.3V peripherals. >> >> The advantage of putting it in each board file is you could name it >> based on the schematics. I see SoCKit_RevC it is named "VCC3P3" >> > > That's fine. I can edit the patch so that each regulator node goes into > it's respective board dts file. > >> It also allows you to specify the 3.3 supply as the output of a real >> regulator driver. In s5_soc_devkit it looks like this comes from >> LTC2978. That's got i2c connections. What happens when you add a >> driver for that? >> > > Do you mean the c5_soc_devkit? If so, then the 3.3V is coming from the > LTC3885 regulator. The LTC2978 is a regulator for the the 3.3V_HPS or > 3.3V_REG_HPS which is used by others. Oops, right. c5_soc_devkit_c. I guess I got confused since your regulator name in the dts was "HPS 3.3V". When I saw "3.3V_PM_HPS" I assumed it was the same. On <http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679> I don't see a 3885. Oh, 3855. I see. > So I should change the regulator name from 3.3V_HPS to just 3.3V for the > c5_soc_devkit. Yes, that's my opinion. Matching the schematics can be a huge benefit and can avoid confusion! :) -Doug ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node @ 2014-10-17 22:36 ` Doug Anderson 0 siblings, 0 replies; 20+ messages in thread From: Doug Anderson @ 2014-10-17 22:36 UTC (permalink / raw) To: Dinh Nguyen Cc: Rob Herring, Ian Campbell, Kumar Gala, Mark Rutland, Pawel Moll, Jaehoon Chung, Dinh Nguyen, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx, s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ Dinh, On Fri, Oct 17, 2014 at 1:04 PM, Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: > On 10/17/2014 02:43 PM, Doug Anderson wrote: >> Hi, >> >> On Fri, Oct 17, 2014 at 12:19 PM, Dinh Nguyen >> <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: >>> On 10/17/2014 11:57 AM, Doug Anderson wrote: >>>> Dinh, >>>> >>>> On Thu, Oct 16, 2014 at 2:03 PM, <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> wrote: >>>>> From: Dinh Nguyen <dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> >>>>> >>>>> Without the 3.3V regulator node, the SDMMC driver will give these warnings: >>>>> >>>>> dw_mmc ff704000.dwmmc0: No vmmc regulator found >>>>> dw_mmc ff704000.dwmmc0: No vqmmc regulator found >>>>> >>>>> This patch adds the regulator node, and points the SD/MMC to the regulator. >>>>> >>> [...] >>>>> >>>>> - dwmmc0@ff704000 { >>>>> + mmc0: dwmmc0@ff704000 { >>>>> num-slots = <1>; >>>>> broken-cd; >>>>> bus-width = <4>; >>>>> @@ -41,4 +41,13 @@ >>>>> cpu1-start-addr = <0xffd080c4>; >>>>> }; >>>>> }; >>>>> + >>>>> + regulator_3_3v_hps: fixed_3_3v_hps_regulator@0 { >>>> >>>> nit: no @0 since there is no "reg" (register) under this node. >>>> >>>> nit: usually people don't like "_" in node names. ...I would probably >>>> do this but I'm not an expert: >>>> regulator_3_3v_hps: hps-regulator { >>> >>> Ok..will fix up. >>> >>>> >>>> This regulator also looks pretty bogus to me. Is this really a >>>> regulator that software has no control over? It means you can't fully >>>> reset a card but I guess that's OK. >>> >>> Yes, this is a generic 3.3V regulator that is used for the 3.3V power >>> rail for the SOC, and any IP the needs 3.3V. >>> >>> Schematics are here: >>> http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679 >>> >>> http://www.rocketboards.org/pub/Documentation/ArrowSoCKitEvaluationBoard/SoCKit_RevC.pdf >>> >>> And right, I don't see anyway to be able to reset the card. >>> >>>> >>>> I'd also expect this regulator to be defined in the same dts / dtsi >>>> file that it's used in. Your current patch says "there's a generic >>>> 3.3V regulator on all boards of the socfpga_arria5 class even if they >>>> don't use MMC, but the "socfpga_arria5_socdk" uses it for MMC. Is >>>> that really true? >>>> >>> >>> yeah...My original patch had the regulator placed in it's appropriate >>> dts board file, but I saw that as a bit of duplication. I figured that >>> most people are just copying Altera's devkit schematic, that if a new >>> board file shows up that has a different controllable regulator, then >>> the board file can have the new regulator node. Meanwhile, this patch >>> saves adding an additional regulator node for the existing board files. >>> >>> The 3.3V is not just used by MMC, but the for all 3.3V peripherals. >> >> The advantage of putting it in each board file is you could name it >> based on the schematics. I see SoCKit_RevC it is named "VCC3P3" >> > > That's fine. I can edit the patch so that each regulator node goes into > it's respective board dts file. > >> It also allows you to specify the 3.3 supply as the output of a real >> regulator driver. In s5_soc_devkit it looks like this comes from >> LTC2978. That's got i2c connections. What happens when you add a >> driver for that? >> > > Do you mean the c5_soc_devkit? If so, then the 3.3V is coming from the > LTC3885 regulator. The LTC2978 is a regulator for the the 3.3V_HPS or > 3.3V_REG_HPS which is used by others. Oops, right. c5_soc_devkit_c. I guess I got confused since your regulator name in the dts was "HPS 3.3V". When I saw "3.3V_PM_HPS" I assumed it was the same. On <http://www.rocketboards.org/pub/Documentation/AlteraSoCDevelopmentBoard/c5_soc_devkit_c.pdf?t=1365712679> I don't see a 3885. Oh, 3855. I see. > So I should change the regulator name from 3.3V_HPS to just 3.3V for the > c5_soc_devkit. Yes, that's my opinion. Matching the schematics can be a huge benefit and can avoid confusion! :) -Doug -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2014-10-17 22:36 UTC | newest] Thread overview: 20+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-10-16 21:03 [PATCH 0/2] ARM: dts: socfpga: fix booting with SD/MMC dinguyen at opensource.altera.com 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 2014-10-16 21:03 ` [PATCH 1/2] ARM: dts: socfpga: Fix SD card detect dinguyen at opensource.altera.com 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 2014-10-16 21:03 ` [PATCH 2/2] ARM: dts: socfpga: Add a 3.3V fixed regulator node dinguyen at opensource.altera.com 2014-10-16 21:03 ` dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx 2014-10-17 1:43 ` Jaehoon Chung 2014-10-17 1:43 ` Jaehoon Chung 2014-10-17 13:37 ` Dinh Nguyen 2014-10-17 13:37 ` Dinh Nguyen 2014-10-17 16:57 ` Doug Anderson 2014-10-17 16:57 ` Doug Anderson 2014-10-17 19:19 ` Dinh Nguyen 2014-10-17 19:19 ` Dinh Nguyen 2014-10-17 19:43 ` Doug Anderson 2014-10-17 19:43 ` Doug Anderson 2014-10-17 20:04 ` Dinh Nguyen 2014-10-17 20:04 ` Dinh Nguyen 2014-10-17 22:36 ` Doug Anderson 2014-10-17 22:36 ` Doug Anderson
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.