* [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support
[not found] ` <20160612232456.1621-1-christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
@ 2016-06-12 23:24 ` christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg
[not found] ` <c25d8e5d113240c5ab78771f301d901a-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
[not found] ` <089d827c9de9426eab79a907a2043828@rwthex-s1-a.rwth-ad.de>
0 siblings, 2 replies; 6+ messages in thread
From: christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg @ 2016-06-12 23:24 UTC (permalink / raw)
To: shawnguo-DgEjT+Ai2ygdnm+yROfE0A, kernel-bIcnvbaLZ9MEGnE8C9+IrQ
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA,
grinberg-UTxiZqZC01RS1MOuV/RT9w, fabio.estevam-3arQi8VN3Tc,
Christopher Spinrath
From: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
The Utilite Pro has a mmc card slot connected to the usdhc3
controller. There is no card detection until hardware revision 1.3.
Add support for it and signal the controller with the broken-cd
property that polling has to be used to detect a card.
Signed-off-by: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
Reviewed-by: Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>
---
Notes:
Changes since v2:
- add Fabio's Reviewed-By
Changes since v1:
- enhance commit message to explain to the broken-cd property
arch/arm/boot/dts/imx6q-utilite-pro.dts | 44 +++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/arch/arm/boot/dts/imx6q-utilite-pro.dts b/arch/arm/boot/dts/imx6q-utilite-pro.dts
index 7219745..6199063 100644
--- a/arch/arm/boot/dts/imx6q-utilite-pro.dts
+++ b/arch/arm/boot/dts/imx6q-utilite-pro.dts
@@ -130,6 +130,39 @@
MX6QDL_PAD_SD4_DAT6__UART2_CTS_B 0x1b0b1
>;
};
+
+ pinctrl_usdhc3: usdhc3grp {
+ fsl,pins = <
+ MX6QDL_PAD_SD3_CMD__SD3_CMD 0x17059
+ MX6QDL_PAD_SD3_CLK__SD3_CLK 0x10059
+ MX6QDL_PAD_SD3_DAT0__SD3_DATA0 0x17059
+ MX6QDL_PAD_SD3_DAT1__SD3_DATA1 0x17059
+ MX6QDL_PAD_SD3_DAT2__SD3_DATA2 0x17059
+ MX6QDL_PAD_SD3_DAT3__SD3_DATA3 0x17059
+ >;
+ };
+
+ pinctrl_usdhc3_100mhz: usdhc3grp-100mhz {
+ fsl,pins = <
+ MX6QDL_PAD_SD3_CMD__SD3_CMD 0x170B9
+ MX6QDL_PAD_SD3_CLK__SD3_CLK 0x100B9
+ MX6QDL_PAD_SD3_DAT0__SD3_DATA0 0x170B9
+ MX6QDL_PAD_SD3_DAT1__SD3_DATA1 0x170B9
+ MX6QDL_PAD_SD3_DAT2__SD3_DATA2 0x170B9
+ MX6QDL_PAD_SD3_DAT3__SD3_DATA3 0x170B9
+ >;
+ };
+
+ pinctrl_usdhc3_200mhz: usdhc3grp-200mhz {
+ fsl,pins = <
+ MX6QDL_PAD_SD3_CMD__SD3_CMD 0x170F9
+ MX6QDL_PAD_SD3_CLK__SD3_CLK 0x100F9
+ MX6QDL_PAD_SD3_DAT0__SD3_DATA0 0x170F9
+ MX6QDL_PAD_SD3_DAT1__SD3_DATA1 0x170F9
+ MX6QDL_PAD_SD3_DAT2__SD3_DATA2 0x170F9
+ MX6QDL_PAD_SD3_DAT3__SD3_DATA3 0x170F9
+ >;
+ };
};
&pcie {
@@ -151,3 +184,14 @@
uart-has-rtscts;
status = "okay";
};
+
+&usdhc3 {
+ pinctrl-names = "default", "state_100mhz", "state_200mhz";
+ pinctrl-0 = <&pinctrl_usdhc3>;
+ pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
+ pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
+ no-1-8-v;
+ broken-cd;
+ keep-power-in-suspend;
+ status = "okay";
+};
--
2.8.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] 6+ messages in thread
* Re: [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support
[not found] ` <c25d8e5d113240c5ab78771f301d901a-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
@ 2016-06-15 6:40 ` Igor Grinberg
0 siblings, 0 replies; 6+ messages in thread
From: Igor Grinberg @ 2016-06-15 6:40 UTC (permalink / raw)
To: christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg,
shawnguo-DgEjT+Ai2ygdnm+yROfE0A, kernel-bIcnvbaLZ9MEGnE8C9+IrQ
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA, fabio.estevam-3arQi8VN3Tc
Hi Christopher,
On 06/13/2016 02:24 AM, christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org wrote:
> From: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>
> The Utilite Pro has a mmc card slot connected to the usdhc3
> controller. There is no card detection until hardware revision 1.3.
>
> Add support for it and signal the controller with the broken-cd
> property that polling has to be used to detect a card.
>
> Signed-off-by: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
> Reviewed-by: Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>
> ---
>
> Notes:
> Changes since v2:
> - add Fabio's Reviewed-By
>
> Changes since v1:
> - enhance commit message to explain to the broken-cd property
>
> arch/arm/boot/dts/imx6q-utilite-pro.dts | 44 +++++++++++++++++++++++++++++++++
> 1 file changed, 44 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx6q-utilite-pro.dts b/arch/arm/boot/dts/imx6q-utilite-pro.dts
> index 7219745..6199063 100644
> --- a/arch/arm/boot/dts/imx6q-utilite-pro.dts
> +++ b/arch/arm/boot/dts/imx6q-utilite-pro.dts
[...]
> @@ -151,3 +184,14 @@
> uart-has-rtscts;
> status = "okay";
> };
> +
> +&usdhc3 {
> + pinctrl-names = "default", "state_100mhz", "state_200mhz";
> + pinctrl-0 = <&pinctrl_usdhc3>;
> + pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
> + pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
> + no-1-8-v;
> + broken-cd;
A wast majority of boards produced are of revision >=1.3.
Can we please have the default as revision 1.3 with cd?
And let the patch you have submitted to U-Boot do the job
for older revisions?
> + keep-power-in-suspend;
> + status = "okay";
> +};
>
--
Regards,
Igor.
--
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] 6+ messages in thread
* Re: [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support
[not found] ` <089d827c9de9426eab79a907a2043828-gtPewvpZjL/Ca9uivkkr415UTUQ924AY@public.gmane.org>
@ 2016-06-15 9:16 ` Christopher Spinrath
[not found] ` <712bda47d095416f8b69f4eecbf979a5-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
[not found] ` <575994a0ec924e2db7282a2945d3355f@rwthex-s2-b.rwth-ad.de>
0 siblings, 2 replies; 6+ messages in thread
From: Christopher Spinrath @ 2016-06-15 9:16 UTC (permalink / raw)
To: Igor Grinberg, shawnguo-DgEjT+Ai2ygdnm+yROfE0A,
kernel-bIcnvbaLZ9MEGnE8C9+IrQ
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA, fabio.estevam-3arQi8VN3Tc,
Spinrath, Christopher
Hi Igor,
On 06/15/2016 08:40 AM, Igor Grinberg wrote:
> Hi Christopher,
>
> On 06/13/2016 02:24 AM, christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org wrote:
>> From: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>>
>> The Utilite Pro has a mmc card slot connected to the usdhc3
>> controller. There is no card detection until hardware revision 1.3.
>>
>> Add support for it and signal the controller with the broken-cd
>> property that polling has to be used to detect a card.
>>
>> Signed-off-by: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>> Reviewed-by: Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>
>> ---
>>
>> Notes:
>> Changes since v2:
>> - add Fabio's Reviewed-By
>>
>> Changes since v1:
>> - enhance commit message to explain to the broken-cd property
>>
>> arch/arm/boot/dts/imx6q-utilite-pro.dts | 44 +++++++++++++++++++++++++++++++++
>> 1 file changed, 44 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/imx6q-utilite-pro.dts b/arch/arm/boot/dts/imx6q-utilite-pro.dts
>> index 7219745..6199063 100644
>> --- a/arch/arm/boot/dts/imx6q-utilite-pro.dts
>> +++ b/arch/arm/boot/dts/imx6q-utilite-pro.dts
>
> [...]
>
>> @@ -151,3 +184,14 @@
>> uart-has-rtscts;
>> status = "okay";
>> };
>> +
>> +&usdhc3 {
>> + pinctrl-names = "default", "state_100mhz", "state_200mhz";
>> + pinctrl-0 = <&pinctrl_usdhc3>;
>> + pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
>> + pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
>> + no-1-8-v;
>> + broken-cd;
>
> A wast majority of boards produced are of revision >=1.3.
> Can we please have the default as revision 1.3 with cd?
> And let the patch you have submitted to U-Boot do the job
> for older revisions?
>
Well, my board has revision 1.0. So I cannot test that and feel uneasy
to put my Signed-off-by under such a patch. IMHO the best solution would
be that someone with a revision >= 1.3 board sends a follow-up patch
adding the cd-gpios.
If I resend the patch with cd-gpios could you test it and provide a
Tested-By?
The other question is: should the dts provide a working/sane
configuration for all boards (which the broken-cd approach is)
independently of the bootloader? Is it ok to put both, the cd-gpios and
the broken-cd property into the dts and let the bootloader remove the
broken-cd property for revision >= 1.3 (due to the documentation it is
not but the driver favourites the broken cd property - hence, it would
work)?
Thanks,
Christopher
>> + keep-power-in-suspend;
>> + 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] 6+ messages in thread
* Re: [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support
[not found] ` <712bda47d095416f8b69f4eecbf979a5-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
@ 2016-06-15 10:33 ` Igor Grinberg
0 siblings, 0 replies; 6+ messages in thread
From: Igor Grinberg @ 2016-06-15 10:33 UTC (permalink / raw)
To: Christopher Spinrath, shawnguo-DgEjT+Ai2ygdnm+yROfE0A,
kernel-bIcnvbaLZ9MEGnE8C9+IrQ
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA, fabio.estevam-3arQi8VN3Tc
Hi Christopher,
On 06/15/2016 12:16 PM, Christopher Spinrath wrote:
> Hi Igor,
>
> On 06/15/2016 08:40 AM, Igor Grinberg wrote:
>> Hi Christopher,
>>
>> On 06/13/2016 02:24 AM, christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org wrote:
>>> From: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>>>
>>> The Utilite Pro has a mmc card slot connected to the usdhc3
>>> controller. There is no card detection until hardware revision 1.3.
>>>
>>> Add support for it and signal the controller with the broken-cd
>>> property that polling has to be used to detect a card.
>>>
>>> Signed-off-by: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>>> Reviewed-by: Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>
>>> ---
>>>
>>> Notes:
>>> Changes since v2:
>>> - add Fabio's Reviewed-By
>>>
>>> Changes since v1:
>>> - enhance commit message to explain to the broken-cd property
>>>
>>> arch/arm/boot/dts/imx6q-utilite-pro.dts | 44 +++++++++++++++++++++++++++++++++
>>> 1 file changed, 44 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/imx6q-utilite-pro.dts b/arch/arm/boot/dts/imx6q-utilite-pro.dts
>>> index 7219745..6199063 100644
>>> --- a/arch/arm/boot/dts/imx6q-utilite-pro.dts
>>> +++ b/arch/arm/boot/dts/imx6q-utilite-pro.dts
>>
>> [...]
>>
>>> @@ -151,3 +184,14 @@
>>> uart-has-rtscts;
>>> status = "okay";
>>> };
>>> +
>>> +&usdhc3 {
>>> + pinctrl-names = "default", "state_100mhz", "state_200mhz";
>>> + pinctrl-0 = <&pinctrl_usdhc3>;
>>> + pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
>>> + pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
>>> + no-1-8-v;
>>> + broken-cd;
>>
>> A wast majority of boards produced are of revision >=1.3.
>> Can we please have the default as revision 1.3 with cd?
>> And let the patch you have submitted to U-Boot do the job
>> for older revisions?
>>
>
> Well, my board has revision 1.0. So I cannot test that and feel uneasy
> to put my Signed-off-by under such a patch. IMHO the best solution would
> be that someone with a revision >= 1.3 board sends a follow-up patch
> adding the cd-gpios.
>
> If I resend the patch with cd-gpios could you test it and provide a
> Tested-By?
Yes. Absolutely.
>
> The other question is: should the dts provide a working/sane
> configuration for all boards (which the broken-cd approach is)
> independently of the bootloader?
Well, no, I wouldn't go that way, as "why wouldn't we just use the
broken-cd approach on all boards and not bother with gpio-cds"...
We do want the gpio-cd to work as expected where it can.
> Is it ok to put both, the cd-gpios and
> the broken-cd property into the dts and let the bootloader remove the
> broken-cd property for revision >= 1.3 (due to the documentation it is
> not but the driver favourites the broken cd property - hence, it would
> work)?
That sounds sane from my POV (unless I'm missing something), but
will have to have comments explaining what is going on.
There are two main reasons why I would not want to have the broken-cd
as a default:
1) It is not broken... It just does not exist prior to revision 1.3.
2) There are more boards of revision >=1.3 then older ones outside and
we would want them to work as expected even with older U-Boot versions.
--
Regards,
Igor.
--
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] 6+ messages in thread
* Re: [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support
[not found] ` <575994a0ec924e2db7282a2945d3355f-gtPewvpZjL+krInni0qv/15UTUQ924AY@public.gmane.org>
@ 2016-06-15 14:40 ` Christopher Spinrath
[not found] ` <2160fa6084b94c839b339a663334ae65-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
0 siblings, 1 reply; 6+ messages in thread
From: Christopher Spinrath @ 2016-06-15 14:40 UTC (permalink / raw)
To: Igor Grinberg, shawnguo-DgEjT+Ai2ygdnm+yROfE0A,
kernel-bIcnvbaLZ9MEGnE8C9+IrQ
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA, fabio.estevam-3arQi8VN3Tc,
Spinrath, Christopher
Hi Shawn,
On 06/15/2016 12:33 PM, Igor Grinberg wrote:
> Hi Christopher,
>
> On 06/15/2016 12:16 PM, Christopher Spinrath wrote:
>> Hi Igor,
>>
>> On 06/15/2016 08:40 AM, Igor Grinberg wrote:
>>> Hi Christopher,
>>>
>>> On 06/13/2016 02:24 AM, christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org wrote:
>>>> From: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>>>>
>>>> The Utilite Pro has a mmc card slot connected to the usdhc3
>>>> controller. There is no card detection until hardware revision 1.3.
>>>>
>>>> Add support for it and signal the controller with the broken-cd
>>>> property that polling has to be used to detect a card.
>>>>
>>>> Signed-off-by: Christopher Spinrath <christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
>>>> Reviewed-by: Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>
>>>> ---
>>>>
>>>> Notes:
>>>> Changes since v2:
>>>> - add Fabio's Reviewed-By
>>>>
>>>> Changes since v1:
>>>> - enhance commit message to explain to the broken-cd property
>>>>
>>>> arch/arm/boot/dts/imx6q-utilite-pro.dts | 44 +++++++++++++++++++++++++++++++++
>>>> 1 file changed, 44 insertions(+)
>>>>
>>>> diff --git a/arch/arm/boot/dts/imx6q-utilite-pro.dts b/arch/arm/boot/dts/imx6q-utilite-pro.dts
>>>> index 7219745..6199063 100644
>>>> --- a/arch/arm/boot/dts/imx6q-utilite-pro.dts
>>>> +++ b/arch/arm/boot/dts/imx6q-utilite-pro.dts
>>>
>>> [...]
>>>
>>>> @@ -151,3 +184,14 @@
>>>> uart-has-rtscts;
>>>> status = "okay";
>>>> };
>>>> +
>>>> +&usdhc3 {
>>>> + pinctrl-names = "default", "state_100mhz", "state_200mhz";
>>>> + pinctrl-0 = <&pinctrl_usdhc3>;
>>>> + pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
>>>> + pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
>>>> + no-1-8-v;
>>>> + broken-cd;
>>>
>>> A wast majority of boards produced are of revision >=1.3.
>>> Can we please have the default as revision 1.3 with cd?
>>> And let the patch you have submitted to U-Boot do the job
>>> for older revisions?
>>>
>>
>> Well, my board has revision 1.0. So I cannot test that and feel uneasy
>> to put my Signed-off-by under such a patch. IMHO the best solution would
>> be that someone with a revision >= 1.3 board sends a follow-up patch
>> adding the cd-gpios.
>>
>> If I resend the patch with cd-gpios could you test it and provide a
>> Tested-By?
>
> Yes. Absolutely.
>
Can we have your opinion on that matter?
>>
>> The other question is: should the dts provide a working/sane
>> configuration for all boards (which the broken-cd approach is)
>> independently of the bootloader?
>
> Well, no, I wouldn't go that way, as "why wouldn't we just use the
> broken-cd approach on all boards and not bother with gpio-cds"...
> We do want the gpio-cd to work as expected where it can.
>
>> Is it ok to put both, the cd-gpios and
>> the broken-cd property into the dts and let the bootloader remove the
>> broken-cd property for revision >= 1.3 (due to the documentation it is
>> not but the driver favourites the broken cd property - hence, it would
>> work)?
>
> That sounds sane from my POV (unless I'm missing something), but
> will have to have comments explaining what is going on.
>
> There are two main reasons why I would not want to have the broken-cd
> as a default:
> 1) It is not broken... It just does not exist prior to revision 1.3.
> 2) There are more boards of revision >=1.3 then older ones outside and
> we would want them to work as expected even with older U-Boot versions.
>
--
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] 6+ messages in thread
* Re: [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support
[not found] ` <2160fa6084b94c839b339a663334ae65-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
@ 2016-06-16 1:42 ` Shawn Guo
0 siblings, 0 replies; 6+ messages in thread
From: Shawn Guo @ 2016-06-16 1:42 UTC (permalink / raw)
To: Christopher Spinrath
Cc: Igor Grinberg, kernel-bIcnvbaLZ9MEGnE8C9+IrQ,
mark.rutland-5wv7dgnIgG8, devicetree-u79uwXL29TY76Z2rM5mHXA,
pawel.moll-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
linux-I+IVW8TIWO2tmTQ+vhA3Yw, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
galak-sgV2jX0FEOL9JmXXK+q4OQ, fabio.estevam-3arQi8VN3Tc,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Wed, Jun 15, 2016 at 04:40:26PM +0200, Christopher Spinrath wrote:
> >> Well, my board has revision 1.0. So I cannot test that and feel uneasy
> >> to put my Signed-off-by under such a patch. IMHO the best solution would
> >> be that someone with a revision >= 1.3 board sends a follow-up patch
> >> adding the cd-gpios.
This approach seems reasonable to me, so I just went ahead to apply the
patch, and wait a follow-up patch from someone who can test on 1.3 board.
> >>
> >> If I resend the patch with cd-gpios could you test it and provide a
> >> Tested-By?
> >
> > Yes. Absolutely.
> >
>
> Can we have your opinion on that matter?
Shawn
--
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] 6+ messages in thread
end of thread, other threads:[~2016-06-16 1:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20160612232456.1621-1-christopher.spinrath@rwth-aachen.de>
[not found] ` <20160612232456.1621-1-christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg@public.gmane.org>
2016-06-12 23:24 ` [PATCH v3 2/2] ARM: dts: utilite-pro: add mmc card slot support christopher.spinrath-vA1bhqPz9FBZXbeN9DUtxg
[not found] ` <c25d8e5d113240c5ab78771f301d901a-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
2016-06-15 6:40 ` Igor Grinberg
[not found] ` <089d827c9de9426eab79a907a2043828@rwthex-s1-a.rwth-ad.de>
[not found] ` <089d827c9de9426eab79a907a2043828-gtPewvpZjL/Ca9uivkkr415UTUQ924AY@public.gmane.org>
2016-06-15 9:16 ` Christopher Spinrath
[not found] ` <712bda47d095416f8b69f4eecbf979a5-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
2016-06-15 10:33 ` Igor Grinberg
[not found] ` <575994a0ec924e2db7282a2945d3355f@rwthex-s2-b.rwth-ad.de>
[not found] ` <575994a0ec924e2db7282a2945d3355f-gtPewvpZjL+krInni0qv/15UTUQ924AY@public.gmane.org>
2016-06-15 14:40 ` Christopher Spinrath
[not found] ` <2160fa6084b94c839b339a663334ae65-gtPewvpZjL8umhiu9RXYRl5UTUQ924AY@public.gmane.org>
2016-06-16 1:42 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).