Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1520484 > unrolled thread

[PATCH 1/2] ARM: dts: rockchip: add the sdmmc pinctrl for rk1108

Started byJacob Chen <jacob-chen@iotwrt.com>
First post2016-11-13 09:20 +0100
Last post2016-11-16 13:50 +0100
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/2] ARM: dts: rockchip: add the sdmmc pinctrl for rk1108 Jacob Chen <jacob-chen@iotwrt.com> - 2016-11-13 09:20 +0100
    Re: [PATCH 1/2] ARM: dts: rockchip: add the sdmmc pinctrl for rk1108 陈豪 <jacobchen110@gmail.com> - 2016-11-13 09:20 +0100
    [PATCH 2/2] ARM: dts: rockchip: enable sdmmc for rk1108-evb Jacob Chen <jacob-chen@iotwrt.com> - 2016-11-13 09:30 +0100
      Re: [PATCH 2/2] ARM: dts: rockchip: enable sdmmc for rk1108-evb Heiko Stuebner <heiko@sntech.de> - 2016-11-16 13:30 +0100
    Re: [PATCH 1/2] ARM: dts: rockchip: add the sdmmc pinctrl for rk1108 Heiko Stuebner <heiko@sntech.de> - 2016-11-16 13:50 +0100

#1520484 — [PATCH 1/2] ARM: dts: rockchip: add the sdmmc pinctrl for rk1108

FromJacob Chen <jacob-chen@iotwrt.com>
Date2016-11-13 09:20 +0100
Subject[PATCH 1/2] ARM: dts: rockchip: add the sdmmc pinctrl for rk1108
Message-ID<sCYb7-65p-3@gated-at.bofh.it>
From: Jacob Chen <jacob2.chen@rock-chips.com>

Signed-off-by: Jacob Chen <jacob2.chen@rock-chips.com>
---
 arch/arm/boot/dts/rk1108.dtsi | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/arch/arm/boot/dts/rk1108.dtsi b/arch/arm/boot/dts/rk1108.dtsi
index 9dccfea..6a06ad7 100644
--- a/arch/arm/boot/dts/rk1108.dtsi
+++ b/arch/arm/boot/dts/rk1108.dtsi
@@ -321,6 +321,31 @@
 			input-enable;
 		};
 
+        sdmmc {
+			sdmmc_clk: sdmmc-clk {
+				rockchip,pins = <3 RK_PC4 RK_FUNC_1 &pcfg_pull_none_drv_4ma>;
+			};
+
+			sdmmc_cmd: sdmmc-cmd {
+				rockchip,pins = <3 RK_PC5 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
+			};
+
+			sdmmc_cd: sdmmc-cd {
+				rockchip,pins = <0 RK_PA1 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
+			};
+
+			sdmmc_bus1: sdmmc-bus1 {
+				rockchip,pins = <3 RK_PC3 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
+			};
+
+			sdmmc_bus4: sdmmc-bus4 {
+				rockchip,pins = <3 RK_PC3 RK_FUNC_1 &pcfg_pull_up_drv_4ma>,
+						<3 RK_PC2 RK_FUNC_1 &pcfg_pull_up_drv_4ma>,
+						<3 RK_PC1 RK_FUNC_1 &pcfg_pull_up_drv_4ma>,
+						<3 RK_PC0 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
+			};
+		};
+
 		i2c1 {
 			i2c1_xfer: i2c1-xfer {
 				rockchip,pins = <2 RK_PD3 RK_FUNC_1 &pcfg_pull_up>,
-- 
2.7.4

[toc] | [next] | [standalone]


#1520485

From陈豪 <jacobchen110@gmail.com>
Date2016-11-13 09:20 +0100
Message-ID<sCYb7-65p-1@gated-at.bofh.it>
In reply to#1520484
2016-11-13 16:13 GMT+08:00 Jacob Chen <jacob-chen@iotwrt.com>:
> From: Jacob Chen <jacob2.chen@rock-chips.com>
>
> Signed-off-by: Jacob Chen <jacob2.chen@rock-chips.com>
> ---
>  arch/arm/boot/dts/rk1108.dtsi | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/arch/arm/boot/dts/rk1108.dtsi b/arch/arm/boot/dts/rk1108.dtsi
> index 9dccfea..6a06ad7 100644
> --- a/arch/arm/boot/dts/rk1108.dtsi
> +++ b/arch/arm/boot/dts/rk1108.dtsi
> @@ -321,6 +321,31 @@
>                         input-enable;
>                 };
>
> +        sdmmc {
> +                       sdmmc_clk: sdmmc-clk {
> +                               rockchip,pins = <3 RK_PC4 RK_FUNC_1 &pcfg_pull_none_drv_4ma>;
> +                       };
> +
> +                       sdmmc_cmd: sdmmc-cmd {
> +                               rockchip,pins = <3 RK_PC5 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
> +                       };
> +
> +                       sdmmc_cd: sdmmc-cd {
> +                               rockchip,pins = <0 RK_PA1 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
> +                       };
> +
> +                       sdmmc_bus1: sdmmc-bus1 {
> +                               rockchip,pins = <3 RK_PC3 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
> +                       };
> +
> +                       sdmmc_bus4: sdmmc-bus4 {
> +                               rockchip,pins = <3 RK_PC3 RK_FUNC_1 &pcfg_pull_up_drv_4ma>,
> +                                               <3 RK_PC2 RK_FUNC_1 &pcfg_pull_up_drv_4ma>,
> +                                               <3 RK_PC1 RK_FUNC_1 &pcfg_pull_up_drv_4ma>,
> +                                               <3 RK_PC0 RK_FUNC_1 &pcfg_pull_up_drv_4ma>;
> +                       };
> +               };
> +
>                 i2c1 {
>                         i2c1_xfer: i2c1-xfer {
>                                 rockchip,pins = <2 RK_PD3 RK_FUNC_1 &pcfg_pull_up>,
> --
> 2.7.4
>

Those patches are based on andy's patch set "Add basic support for
support for Rockchip RK1108 SOC ", assuming he will include
drive-strength functionality in pinctrl.

[toc] | [prev] | [next] | [standalone]


#1520486 — [PATCH 2/2] ARM: dts: rockchip: enable sdmmc for rk1108-evb

FromJacob Chen <jacob-chen@iotwrt.com>
Date2016-11-13 09:30 +0100
Subject[PATCH 2/2] ARM: dts: rockchip: enable sdmmc for rk1108-evb
Message-ID<sCYkN-68Q-5@gated-at.bofh.it>
In reply to#1520484
From: Jacob Chen <jacob2.chen@rock-chips.com>

This patch add sdmmc support for rk1108-evb, now I can load the rootfs
from sdmmc.

Signed-off-by: Jacob Chen <jacob2.chen@rock-chips.com>
---
 arch/arm/boot/dts/rk1108-evb.dts | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/arch/arm/boot/dts/rk1108-evb.dts b/arch/arm/boot/dts/rk1108-evb.dts
index 3956cff..cea26e5 100644
--- a/arch/arm/boot/dts/rk1108-evb.dts
+++ b/arch/arm/boot/dts/rk1108-evb.dts
@@ -56,6 +56,18 @@
 	};
 };
 
+&sdmmc {
+	bus-width = <4>;
+	cap-mmc-highspeed;
+	cap-sd-highspeed;
+	card-detect-delay = <200>;
+	disable-wp;
+	num-slots = <1>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&sdmmc_clk>, <&sdmmc_cmd>, <&sdmmc_cd>, <&sdmmc_bus4>;
+	status = "okay";
+};
+
 &uart0 {
 	status = "okay";
 };
@@ -67,3 +79,12 @@
 &uart2 {
 	status = "okay";
 };
+
+&pinctrl {
+
+	sdmmc {
+		sdmmc_pwr: sdmmc-pwr {
+			rockchip,pins = <3 RK_PB7 RK_FUNC_GPIO &pcfg_pull_up_drv_4ma>;
+		};
+	};
+};
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]


#1523439 — Re: [PATCH 2/2] ARM: dts: rockchip: enable sdmmc for rk1108-evb

FromHeiko Stuebner <heiko@sntech.de>
Date2016-11-16 13:30 +0100
SubjectRe: [PATCH 2/2] ARM: dts: rockchip: enable sdmmc for rk1108-evb
Message-ID<sE7vH-2KY-7@gated-at.bofh.it>
In reply to#1520486
Am Sonntag, 13. November 2016, 16:13:19 CET schrieb Jacob Chen:
> From: Jacob Chen <jacob2.chen@rock-chips.com>
> 
> This patch add sdmmc support for rk1108-evb, now I can load the rootfs
> from sdmmc.
> 
> Signed-off-by: Jacob Chen <jacob2.chen@rock-chips.com>
> ---
>  arch/arm/boot/dts/rk1108-evb.dts | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/rk1108-evb.dts
> b/arch/arm/boot/dts/rk1108-evb.dts index 3956cff..cea26e5 100644
> --- a/arch/arm/boot/dts/rk1108-evb.dts
> +++ b/arch/arm/boot/dts/rk1108-evb.dts
> @@ -56,6 +56,18 @@
>  	};
>  };
> 
> +&sdmmc {
> +	bus-width = <4>;
> +	cap-mmc-highspeed;
> +	cap-sd-highspeed;
> +	card-detect-delay = <200>;
> +	disable-wp;
> +	num-slots = <1>;
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&sdmmc_clk>, <&sdmmc_cmd>, <&sdmmc_cd>, <&sdmmc_bus4>;
> +	status = "okay";
> +};
> +
>  &uart0 {
>  	status = "okay";
>  };
> @@ -67,3 +79,12 @@
>  &uart2 {
>  	status = "okay";
>  };
> +
> +&pinctrl {
> +
> +	sdmmc {
> +		sdmmc_pwr: sdmmc-pwr {
> +			rockchip,pins = <3 RK_PB7 RK_FUNC_GPIO &pcfg_pull_up_drv_4ma>;
> +		};

the above is unsued, and in general please use a fixed regulator as vmmc, as 
all other  Rockchip boards do already, so that the mmc core can handle it 
itself.


Thanks
Heiko

[toc] | [prev] | [next] | [standalone]


#1523443

FromHeiko Stuebner <heiko@sntech.de>
Date2016-11-16 13:50 +0100
Message-ID<sE7P4-2Rf-9@gated-at.bofh.it>
In reply to#1520484
Am Sonntag, 13. November 2016, 16:13:18 CET schrieb Jacob Chen:
> From: Jacob Chen <jacob2.chen@rock-chips.com>
> 
> Signed-off-by: Jacob Chen <jacob2.chen@rock-chips.com>
> ---
>  arch/arm/boot/dts/rk1108.dtsi | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/rk1108.dtsi b/arch/arm/boot/dts/rk1108.dtsi
> index 9dccfea..6a06ad7 100644
> --- a/arch/arm/boot/dts/rk1108.dtsi
> +++ b/arch/arm/boot/dts/rk1108.dtsi
> @@ -321,6 +321,31 @@
>  			input-enable;
>  		};
> 
> +        sdmmc {

I've fixed the intendation here (needed tabs instead of spaces) and moved the 
block to its alphabetically correct position between i2c and uart nodes and 
applied the result to my dts32 branch.


Thanks
Heiko

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web