[PATCHv3 5/7] ARM: dts: exynos5422: use wp-gpio method to detect sd-card
linux.amoon at gmail.com
Fri Sep 21 05:31:07 PDT 2018
On Fri, 21 Sep 2018 at 16:42, Krzysztof Kozlowski <krzk at kernel.org> wrote:
> On Thu, 20 Sep 2018 at 19:59, Anand Moon <linux.amoon at gmail.com> wrote:
> > add the Write Protect GPIO property for sdcard, to fix
> > the warning message during boot time.
> > of_get_named_gpiod_flags: can't parse 'wp-gpios' property of node '/soc/mmc at 12220000'
> > Signed-off-by: Anand Moon <linux.amoon at gmail.com>
> > ---
> > New patch to this series.
> > If we keep the wp-gpio GPIO_ACTIVE_LOW sdcard boot into RO mode.
> > ---
> > arch/arm/boot/dts/exynos5422-odroid-core.dtsi | 1 +
> > 1 file changed, 1 insertion(+)
> > diff --git a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
> > index 03fe02ceae66..475c38bbc99e 100644
> > --- a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
> > +++ b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
> > @@ -370,6 +370,7 @@
> > cap-sd-highspeed;
> > max-frequency = <200000000>;
> > cd-gpios = <&gpc2 2 GPIO_ACTIVE_LOW>;
> > + wp-gpios = <&gpc4 0 GPIO_ACTIVE_HIGH>;
> Two issues here:
> 1. You need also proper pinctrl setting.
Thanks for pointing out. Yes proper pinctrl is needed to support this setting.
probably pinctrl sd2_wp is needed to support this feature.
> 2. On schematics the WP is not connected... so how do you know that it
>From XU4_MAIN_REV0.1 schematics page 7 "Micro Socket" their is no
connection to WP pin.
but below schematics page "EXYNOS 5422 MMC UFS" it's shows
> is active high? How did you test it? How do you want to test it on
I just tested this with the DTS change to GPIO_ACTIVE_LOW which
resulted the microSD card booted into RO mode.
So I set this to GPIO_ACTIVE_HIGH just to avoid warning.
I will study the changes requires and then post proper separate patch
to support this feature.
> Best regards,
> > vmmc-supply = <&ldo19_reg>;
> > vqmmc-supply = <&ldo13_reg>;
> > sd-uhs-sdr50;
> > --
> > 2.17.1
More information about the linux-arm-kernel