[PATCH 1/1] ARM: dts: imx6q-bx50v3: disable SD card

[PATCH 1/1] ARM: dts: imx6q-bx50v3: disable SD card

[Ian Ray]

Default boot medium must be internal eMMC, we don't want the SDIO to be
used for these devices.  This commit disables the SD card interface for
GE Healthcare Bx50v3 devices.

Signed-off-by: Ian Ray <ian.ray-JJi787mZWgc@public.gmane.org>
Signed-off-by: Romain Perier <romain.perier-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
---
 arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
index b915837..a530169 100644
--- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi
+++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
@@ -303,6 +303,10 @@
 	};
 };
 
+&usdhc2 {
+	status = "disabled";
+};
+
 &usdhc4 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_usdhc4>;
-- 
2.10.1

--
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

Re: [PATCH 1/1] ARM: dts: imx6q-bx50v3: disable SD card

[Shawn Guo]

Hi Ian,

On Mon, Jan 08, 2018 at 01:29:01PM +0200, Ian Ray wrote:
> Default boot medium must be internal eMMC, we don't want the SDIO to be
> used for these devices.  This commit disables the SD card interface for
> GE Healthcare Bx50v3 devices.
> 
> Signed-off-by: Ian Ray <ian.ray-JJi787mZWgc@public.gmane.org>
> Signed-off-by: Romain Perier <romain.perier-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>

You are submitting a patch authored by yourself.  Why is Romain's SoB
there?

Shawn

> ---
>  arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
> index b915837..a530169 100644
> --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi
> +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
> @@ -303,6 +303,10 @@
>  	};
>  };
>  
> +&usdhc2 {
> +	status = "disabled";
> +};
> +
>  &usdhc4 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_usdhc4>;
> -- 
> 2.10.1
> 
--
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

Re: [PATCH 1/1] ARM: dts: imx6q-bx50v3: disable SD card

[Shawn Guo]

On Mon, Jan 08, 2018 at 01:29:01PM +0200, Ian Ray wrote:
> Default boot medium must be internal eMMC, we don't want the SDIO to be
> used for these devices.  This commit disables the SD card interface for
> GE Healthcare Bx50v3 devices.
> 
> Signed-off-by: Ian Ray <ian.ray-JJi787mZWgc@public.gmane.org>
> Signed-off-by: Romain Perier <romain.perier-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>

Romain's SoB doesn't make too much sense here, and the email address
bounced messages.  So I dropped his SoB and applied patch.

Shawn

> ---
>  arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
> index b915837..a530169 100644
> --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi
> +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
> @@ -303,6 +303,10 @@
>  	};
>  };
>  
> +&usdhc2 {
> +	status = "disabled";
> +};
> +
>  &usdhc4 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_usdhc4>;
> -- 
> 2.10.1
> 
--
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

Re: EXT: Re: [PATCH 1/1] ARM: dts: imx6q-bx50v3: disable SD card

[Ray, Ian (GE Healthcare)]

> On 2 Feb 2018, at 3.04, Shawn Guo <shawnguo@kernel.org> wrote:
> 
> Hi Ian,
> 
> On Mon, Jan 08, 2018 at 01:29:01PM +0200, Ian Ray wrote:
>> Default boot medium must be internal eMMC, we don't want the SDIO to be
>> used for these devices.  This commit disables the SD card interface for
>> GE Healthcare Bx50v3 devices.
>> 
>> Signed-off-by: Ian Ray <ian.ray@ge.com>
>> Signed-off-by: Romain Perier <romain.perier@collabora.com>
> 
> You are submitting a patch authored by yourself.  Why is Romain's SoB
> there?
> 

Sorry for the mess — Romain sent a V1 which was NAK’d because of
non-alphabetical ordering.


> Shawn
> 
>> ---
>> arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 ++++
>> 1 file changed, 4 insertions(+)
>> 
>> diff --git a/arch/arm/boot/dts/imx6q-bx50v3.dtsi b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
>> index b915837..a530169 100644
>> --- a/arch/arm/boot/dts/imx6q-bx50v3.dtsi
>> +++ b/arch/arm/boot/dts/imx6q-bx50v3.dtsi
>> @@ -303,6 +303,10 @@
>> 	};
>> };
>> 
>> +&usdhc2 {
>> +	status = "disabled";
>> +};
>> +
>> &usdhc4 {
>> 	pinctrl-names = "default";
>> 	pinctrl-0 = <&pinctrl_usdhc4>;
>> -- 
>> 2.10.1
>>