Re: [PATCH] ARM: tegra: add SD wp-gpios to Venice2 DT

[Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1499 bytes --]

On Mon, Apr 28, 2014 at 12:53:39PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> Venice2 can detect write-protect on the SD card. Add the required
> DT entries to allow this.
> 
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> Thierry, could you test this for me; my Venice2 has been sent away for
> a few days for some rework. You should find that if you write-protect
> your SD card, the kernel fails/panics trying to mount it as a r/w root
> fs, whereas if the card is not write-protected, everything still works
> fine. Thanks.
> ---
>  arch/arm/boot/dts/tegra124-venice2.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
> index 84a6ec039e1d..00e741ba6927 100644
> --- a/arch/arm/boot/dts/tegra124-venice2.dts
> +++ b/arch/arm/boot/dts/tegra124-venice2.dts
> @@ -943,6 +943,7 @@
>  	sdhci@0,700b0400 {
>  		cd-gpios = <&gpio TEGRA_GPIO(V, 2) GPIO_ACTIVE_HIGH>;
>  		power-gpios = <&gpio TEGRA_GPIO(R, 0) GPIO_ACTIVE_HIGH>;
> +		wp-gpios = <&gpio TEGRA_GPIO(Q, 4) GPIO_ACTIVE_HIGH>;

I need to make this GPIO_ACTIVE_LOW for this to work properly on my
Venice2. With that I can see the initrd reporting:

	mount: /dev/mmcblk0p1 is write-protected, mounting read-only

So with that changed:

Tested-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]