* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
@ 2017-05-17 11:55 Jean-Jacques Hiblot
2017-05-17 13:04 ` Tom Rini
0 siblings, 1 reply; 8+ messages in thread
From: Jean-Jacques Hiblot @ 2017-05-17 11:55 UTC (permalink / raw)
To: u-boot
SDIO is not supported in u-boot, there is no point in enabling mmc3.
Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
---
Tom,
this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
when mmc3's registers are accessed for the first time.
Jean-Jacques
arch/arm/dts/am335x-evm.dts | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
index a6f20af..e94fcad 100644
--- a/arch/arm/dts/am335x-evm.dts
+++ b/arch/arm/dts/am335x-evm.dts
@@ -721,12 +721,14 @@
};
&mmc3 {
+ /* disable MMC3 as SDIO is not supported in U-Boot */
+ status = "disabled";
+
/* these are on the crossbar and are outlined in the
xbar-event-map element */
dmas = <&edma 12
&edma 13>;
dma-names = "tx", "rx";
- status = "okay";
vmmc-supply = <&wlan_en_reg>;
bus-width = <4>;
pinctrl-names = "default";
--
1.9.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 11:55 [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3 Jean-Jacques Hiblot
@ 2017-05-17 13:04 ` Tom Rini
2017-05-17 14:46 ` Jean-Jacques Hiblot
0 siblings, 1 reply; 8+ messages in thread
From: Tom Rini @ 2017-05-17 13:04 UTC (permalink / raw)
To: u-boot
On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
> SDIO is not supported in u-boot, there is no point in enabling mmc3.
>
> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> ---
>
> Tom,
>
> this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
> on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
> enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
> when mmc3's registers are accessed for the first time.
>
> Jean-Jacques
>
>
> arch/arm/dts/am335x-evm.dts | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
> index a6f20af..e94fcad 100644
> --- a/arch/arm/dts/am335x-evm.dts
> +++ b/arch/arm/dts/am335x-evm.dts
> @@ -721,12 +721,14 @@
> };
>
> &mmc3 {
> + /* disable MMC3 as SDIO is not supported in U-Boot */
> + status = "disabled";
> +
> /* these are on the crossbar and are outlined in the
> xbar-event-map element */
> dmas = <&edma 12
> &edma 13>;
> dma-names = "tx", "rx";
> - status = "okay";
> vmmc-supply = <&wlan_en_reg>;
> bus-width = <4>;
> pinctrl-names = "default";
Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170517/95726ee1/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 13:04 ` Tom Rini
@ 2017-05-17 14:46 ` Jean-Jacques Hiblot
2017-05-17 15:00 ` Tom Rini
2017-05-17 16:01 ` Tom Rini
0 siblings, 2 replies; 8+ messages in thread
From: Jean-Jacques Hiblot @ 2017-05-17 14:46 UTC (permalink / raw)
To: u-boot
On 17/05/2017 15:04, Tom Rini wrote:
> On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
>
>> SDIO is not supported in u-boot, there is no point in enabling mmc3.
>>
>> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
>> ---
>>
>> Tom,
>>
>> this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
>> on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
>> enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
>> when mmc3's registers are accessed for the first time.
>>
>> Jean-Jacques
>>
>>
>> arch/arm/dts/am335x-evm.dts | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
>> index a6f20af..e94fcad 100644
>> --- a/arch/arm/dts/am335x-evm.dts
>> +++ b/arch/arm/dts/am335x-evm.dts
>> @@ -721,12 +721,14 @@
>> };
>>
>> &mmc3 {
>> + /* disable MMC3 as SDIO is not supported in U-Boot */
>> + status = "disabled";
>> +
>> /* these are on the crossbar and are outlined in the
>> xbar-event-map element */
>> dmas = <&edma 12
>> &edma 13>;
>> dma-names = "tx", "rx";
>> - status = "okay";
>> vmmc-supply = <&wlan_en_reg>;
>> bus-width = <4>;
>> pinctrl-names = "default";
> Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
I tried and it doesn't work. The XXX-u-boot.dtsi being included at the
beginning of the DTS, it cannot override the
'status = "okay"' present in am335x-evm.dts.
> thanks!
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 14:46 ` Jean-Jacques Hiblot
@ 2017-05-17 15:00 ` Tom Rini
2017-05-17 15:09 ` Jean-Jacques Hiblot
2017-05-17 16:01 ` Tom Rini
1 sibling, 1 reply; 8+ messages in thread
From: Tom Rini @ 2017-05-17 15:00 UTC (permalink / raw)
To: u-boot
On Wed, May 17, 2017 at 04:46:29PM +0200, Jean-Jacques Hiblot wrote:
>
>
> On 17/05/2017 15:04, Tom Rini wrote:
> >On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
> >
> >>SDIO is not supported in u-boot, there is no point in enabling mmc3.
> >>
> >>Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> >>---
> >>
> >>Tom,
> >>
> >>this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
> >>on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
> >>enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
> >>when mmc3's registers are accessed for the first time.
> >>
> >>Jean-Jacques
> >>
> >>
> >> arch/arm/dts/am335x-evm.dts | 4 +++-
> >> 1 file changed, 3 insertions(+), 1 deletion(-)
> >>
> >>diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
> >>index a6f20af..e94fcad 100644
> >>--- a/arch/arm/dts/am335x-evm.dts
> >>+++ b/arch/arm/dts/am335x-evm.dts
> >>@@ -721,12 +721,14 @@
> >> };
> >> &mmc3 {
> >>+ /* disable MMC3 as SDIO is not supported in U-Boot */
> >>+ status = "disabled";
> >>+
> >> /* these are on the crossbar and are outlined in the
> >> xbar-event-map element */
> >> dmas = <&edma 12
> >> &edma 13>;
> >> dma-names = "tx", "rx";
> >>- status = "okay";
> >> vmmc-supply = <&wlan_en_reg>;
> >> bus-width = <4>;
> >> pinctrl-names = "default";
> >Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
> I tried and it doesn't work. The XXX-u-boot.dtsi being included at
> the beginning of the DTS, it cannot override the
> 'status = "okay"' present in am335x-evm.dts.
Ug, that's a problem. I'm working on it then, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170517/c0f9b6d3/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 15:00 ` Tom Rini
@ 2017-05-17 15:09 ` Jean-Jacques Hiblot
0 siblings, 0 replies; 8+ messages in thread
From: Jean-Jacques Hiblot @ 2017-05-17 15:09 UTC (permalink / raw)
To: u-boot
On 17/05/2017 17:00, Tom Rini wrote:
> On Wed, May 17, 2017 at 04:46:29PM +0200, Jean-Jacques Hiblot wrote:
>>
>> On 17/05/2017 15:04, Tom Rini wrote:
>>> On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
>>>
>>>> SDIO is not supported in u-boot, there is no point in enabling mmc3.
>>>>
>>>> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
>>>> ---
>>>>
>>>> Tom,
>>>>
>>>> this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
>>>> on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
>>>> enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
>>>> when mmc3's registers are accessed for the first time.
>>>>
>>>> Jean-Jacques
>>>>
>>>>
>>>> arch/arm/dts/am335x-evm.dts | 4 +++-
>>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
>>>> index a6f20af..e94fcad 100644
>>>> --- a/arch/arm/dts/am335x-evm.dts
>>>> +++ b/arch/arm/dts/am335x-evm.dts
>>>> @@ -721,12 +721,14 @@
>>>> };
>>>> &mmc3 {
>>>> + /* disable MMC3 as SDIO is not supported in U-Boot */
>>>> + status = "disabled";
>>>> +
>>>> /* these are on the crossbar and are outlined in the
>>>> xbar-event-map element */
>>>> dmas = <&edma 12
>>>> &edma 13>;
>>>> dma-names = "tx", "rx";
>>>> - status = "okay";
>>>> vmmc-supply = <&wlan_en_reg>;
>>>> bus-width = <4>;
>>>> pinctrl-names = "default";
>>> Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
>> I tried and it doesn't work. The XXX-u-boot.dtsi being included at
>> the beginning of the DTS, it cannot override the
>> 'status = "okay"' present in am335x-evm.dts.
> Ug, that's a problem. I'm working on it then, thanks!
>
We could include it at the end of the DTS. Since the goal is to be able
to amend the DTS it makes more sense.
Also it will greatly simplify this part of Makefile.lib.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 14:46 ` Jean-Jacques Hiblot
2017-05-17 15:00 ` Tom Rini
@ 2017-05-17 16:01 ` Tom Rini
2017-05-17 16:14 ` Jean-Jacques Hiblot
1 sibling, 1 reply; 8+ messages in thread
From: Tom Rini @ 2017-05-17 16:01 UTC (permalink / raw)
To: u-boot
On Wed, May 17, 2017 at 04:46:29PM +0200, Jean-Jacques Hiblot wrote:
>
>
> On 17/05/2017 15:04, Tom Rini wrote:
> >On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
> >
> >>SDIO is not supported in u-boot, there is no point in enabling mmc3.
> >>
> >>Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> >>---
> >>
> >>Tom,
> >>
> >>this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
> >>on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
> >>enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
> >>when mmc3's registers are accessed for the first time.
> >>
> >>Jean-Jacques
> >>
> >>
> >> arch/arm/dts/am335x-evm.dts | 4 +++-
> >> 1 file changed, 3 insertions(+), 1 deletion(-)
> >>
> >>diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
> >>index a6f20af..e94fcad 100644
> >>--- a/arch/arm/dts/am335x-evm.dts
> >>+++ b/arch/arm/dts/am335x-evm.dts
> >>@@ -721,12 +721,14 @@
> >> };
> >> &mmc3 {
> >>+ /* disable MMC3 as SDIO is not supported in U-Boot */
> >>+ status = "disabled";
> >>+
> >> /* these are on the crossbar and are outlined in the
> >> xbar-event-map element */
> >> dmas = <&edma 12
> >> &edma 13>;
> >> dma-names = "tx", "rx";
> >>- status = "okay";
> >> vmmc-supply = <&wlan_en_reg>;
> >> bus-width = <4>;
> >> pinctrl-names = "default";
> >Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
> I tried and it doesn't work. The XXX-u-boot.dtsi being included at
> the beginning of the DTS, it cannot override the
> 'status = "okay"' present in am335x-evm.dts.
OK, I've manually confirmed that
https://patchwork.ozlabs.org/patch/763654/ does what I expect (and you
were suggesting too), can you please test that with a patch that adds
am335x-evm-u-boot.dtsi? Thanks again!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170517/6764c446/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 16:01 ` Tom Rini
@ 2017-05-17 16:14 ` Jean-Jacques Hiblot
2017-05-17 17:46 ` Tom Rini
0 siblings, 1 reply; 8+ messages in thread
From: Jean-Jacques Hiblot @ 2017-05-17 16:14 UTC (permalink / raw)
To: u-boot
On 17/05/2017 18:01, Tom Rini wrote:
> On Wed, May 17, 2017 at 04:46:29PM +0200, Jean-Jacques Hiblot wrote:
>>
>> On 17/05/2017 15:04, Tom Rini wrote:
>>> On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
>>>
>>>> SDIO is not supported in u-boot, there is no point in enabling mmc3.
>>>>
>>>> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
>>>> ---
>>>>
>>>> Tom,
>>>>
>>>> this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
>>>> on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
>>>> enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
>>>> when mmc3's registers are accessed for the first time.
>>>>
>>>> Jean-Jacques
>>>>
>>>>
>>>> arch/arm/dts/am335x-evm.dts | 4 +++-
>>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
>>>> index a6f20af..e94fcad 100644
>>>> --- a/arch/arm/dts/am335x-evm.dts
>>>> +++ b/arch/arm/dts/am335x-evm.dts
>>>> @@ -721,12 +721,14 @@
>>>> };
>>>> &mmc3 {
>>>> + /* disable MMC3 as SDIO is not supported in U-Boot */
>>>> + status = "disabled";
>>>> +
>>>> /* these are on the crossbar and are outlined in the
>>>> xbar-event-map element */
>>>> dmas = <&edma 12
>>>> &edma 13>;
>>>> dma-names = "tx", "rx";
>>>> - status = "okay";
>>>> vmmc-supply = <&wlan_en_reg>;
>>>> bus-width = <4>;
>>>> pinctrl-names = "default";
>>> Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
>> I tried and it doesn't work. The XXX-u-boot.dtsi being included at
>> the beginning of the DTS, it cannot override the
>> 'status = "okay"' present in am335x-evm.dts.
> OK, I've manually confirmed that
> https://patchwork.ozlabs.org/patch/763654/ does what I expect (and you
> were suggesting too), can you please test that with a patch that adds
> am335x-evm-u-boot.dtsi? Thanks again!
The dependency on the *-u-boot.dtsi is broken. including is better than
concatenating in that sense.
Other than that, it does the job.
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3
2017-05-17 16:14 ` Jean-Jacques Hiblot
@ 2017-05-17 17:46 ` Tom Rini
0 siblings, 0 replies; 8+ messages in thread
From: Tom Rini @ 2017-05-17 17:46 UTC (permalink / raw)
To: u-boot
On Wed, May 17, 2017 at 06:14:53PM +0200, Jean-Jacques Hiblot wrote:
>
>
> On 17/05/2017 18:01, Tom Rini wrote:
> >On Wed, May 17, 2017 at 04:46:29PM +0200, Jean-Jacques Hiblot wrote:
> >>
> >>On 17/05/2017 15:04, Tom Rini wrote:
> >>>On Wed, May 17, 2017 at 01:55:56PM +0200, Jean-Jacques Hiblot wrote:
> >>>
> >>>>SDIO is not supported in u-boot, there is no point in enabling mmc3.
> >>>>
> >>>>Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> >>>>---
> >>>>
> >>>>Tom,
> >>>>
> >>>>this will fix the problem youreported with commit b5511d6cb8dc601efc14b8cf60755
> >>>>on am335x-evm. For the record, the problem is that the clock of mmc 3 is not
> >>>>enbled by the arch code. With DM_MMC enabled, mmc3 gets probed and u-boot crashes
> >>>>when mmc3's registers are accessed for the first time.
> >>>>
> >>>>Jean-Jacques
> >>>>
> >>>>
> >>>> arch/arm/dts/am335x-evm.dts | 4 +++-
> >>>> 1 file changed, 3 insertions(+), 1 deletion(-)
> >>>>
> >>>>diff --git a/arch/arm/dts/am335x-evm.dts b/arch/arm/dts/am335x-evm.dts
> >>>>index a6f20af..e94fcad 100644
> >>>>--- a/arch/arm/dts/am335x-evm.dts
> >>>>+++ b/arch/arm/dts/am335x-evm.dts
> >>>>@@ -721,12 +721,14 @@
> >>>> };
> >>>> &mmc3 {
> >>>>+ /* disable MMC3 as SDIO is not supported in U-Boot */
> >>>>+ status = "disabled";
> >>>>+
> >>>> /* these are on the crossbar and are outlined in the
> >>>> xbar-event-map element */
> >>>> dmas = <&edma 12
> >>>> &edma 13>;
> >>>> dma-names = "tx", "rx";
> >>>>- status = "okay";
> >>>> vmmc-supply = <&wlan_en_reg>;
> >>>> bus-width = <4>;
> >>>> pinctrl-names = "default";
> >>>Please put this into arch/arm/dts/am335x-evm-u-boot.dtsi instead,
> >>I tried and it doesn't work. The XXX-u-boot.dtsi being included at
> >>the beginning of the DTS, it cannot override the
> >>'status = "okay"' present in am335x-evm.dts.
> >OK, I've manually confirmed that
> >https://patchwork.ozlabs.org/patch/763654/ does what I expect (and you
> >were suggesting too), can you please test that with a patch that adds
> >am335x-evm-u-boot.dtsi? Thanks again!
> The dependency on the *-u-boot.dtsi is broken. including is better
> than concatenating in that sense.
> Other than that, it does the job.
Ah, OK, let me do a v2 that keeps depends but doesn't clean up the
Makefile logic really.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170517/d7fdd129/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-05-17 17:46 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-17 11:55 [U-Boot] [PATCH] ARM: dts: am335x: disable mmc3 Jean-Jacques Hiblot
2017-05-17 13:04 ` Tom Rini
2017-05-17 14:46 ` Jean-Jacques Hiblot
2017-05-17 15:00 ` Tom Rini
2017-05-17 15:09 ` Jean-Jacques Hiblot
2017-05-17 16:01 ` Tom Rini
2017-05-17 16:14 ` Jean-Jacques Hiblot
2017-05-17 17:46 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.