[U-Boot] mmc: disable UHS modes if Vcc cannot be switched on and off

[U-Boot] mmc: disable UHS modes if Vcc cannot be switched on and off

[Eugeniy Paltsev]

Hi Jean-Jacques,

after commit "mmc: disable UHS modes if Vcc cannot be switched on and off" (04a2ea248f)
we got MMC broken on several (at least two) platforms with DesignWare MMC controller.

The error log (with debug enabled in drivers/mmc/mmc.c and drivers/mmc/dw_mmc.c) is next:
--------------->8-------------
MMC:   Synopsys Mobile storage: 0
Buswidth = 0, clock: 400000
selecting mode MMC legacy (freq : 0 MHz)
Buswidth = 1, clock: 400000
Buswidth = 1, clock: 400000
Sending CMD0
dwmci_send_cmd: Timeout.
mmc_init: -110, time 19
** Bad device mmc 0 **
--------------->8-------------

Any ideas how to fix this?

Thanks.
-- 
 Eugeniy Paltsev

[U-Boot] mmc: disable UHS modes if Vcc cannot be switched on and off

[Jaehoon Chung]

On 01/23/2018 03:38 AM, Eugeniy Paltsev wrote:
> Hi Jean-Jacques,
> 
> after commit "mmc: disable UHS modes if Vcc cannot be switched on and off" (04a2ea248f)
> we got MMC broken on several (at least two) platforms with DesignWare MMC controller.
> 
> The error log (with debug enabled in drivers/mmc/mmc.c and drivers/mmc/dw_mmc.c) is next:
> --------------->8-------------
> MMC:   Synopsys Mobile storage: 0
> Buswidth = 0, clock: 400000
> selecting mode MMC legacy (freq : 0 MHz)
> Buswidth = 1, clock: 400000
> Buswidth = 1, clock: 400000
> Sending CMD0
> dwmci_send_cmd: Timeout.
> mmc_init: -110, time 19
> ** Bad device mmc 0 **
> --------------->8-------------
> 
> Any ideas how to fix this?

Could you share which git repository you use? 

Best Regards,
Jaehoon Chung

> 
> Thanks.
> 

[U-Boot] mmc: disable UHS modes if Vcc cannot be switched on and off

[Eugeniy Paltsev]

On Tue, 2018-01-23 at 10:44 +0900, Jaehoon Chung wrote:
> On 01/23/2018 03:38 AM, Eugeniy Paltsev wrote:
> > Hi Jean-Jacques,
> > 
> > after commit "mmc: disable UHS modes if Vcc cannot be switched on and off" (04a2ea248f)
> > we got MMC broken on several (at least two) platforms with DesignWare MMC controller.
> > 
> > The error log (with debug enabled in drivers/mmc/mmc.c and drivers/mmc/dw_mmc.c) is next:
> > --------------->8-------------
> > MMC:   Synopsys Mobile storage: 0
> > Buswidth = 0, clock: 400000
> > selecting mode MMC legacy (freq : 0 MHz)
> > Buswidth = 1, clock: 400000
> > Buswidth = 1, clock: 400000
> > Sending CMD0
> > dwmci_send_cmd: Timeout.
> > mmc_init: -110, time 19
> > ** Bad device mmc 0 **
> > --------------->8-------------
> > 
> > Any ideas how to fix this?
> 
> Could you share which git repository you use? 

I use u-boot master branch from git://git.denx.de/u-boot.git

last commits:
--------------->8-------------
485c13c Merge git://git.denx.de/u-boot-dm
b79221a lib: fdtdec: Fix some style violations
2e38662 lib: fdtdec: Fix whitespace style violations
29d11b8 core: Make device_is_compatible live-tree compatible
147c607 core: Add {ofnode, dev}_translate_address functions
--------------->8-------------

> Best Regards,
> Jaehoon Chung
> 
> > 
> > Thanks.
> > 
> 
> 
-- 
 Eugeniy Paltsev

[U-Boot] mmc: disable UHS modes if Vcc cannot be switched on and off

[Jaehoon Chung]

On 01/23/2018 09:03 PM, Eugeniy Paltsev wrote:
> On Tue, 2018-01-23 at 10:44 +0900, Jaehoon Chung wrote:
>> On 01/23/2018 03:38 AM, Eugeniy Paltsev wrote:
>>> Hi Jean-Jacques,
>>>
>>> after commit "mmc: disable UHS modes if Vcc cannot be switched on and off" (04a2ea248f)
>>> we got MMC broken on several (at least two) platforms with DesignWare MMC controller.
>>>
>>> The error log (with debug enabled in drivers/mmc/mmc.c and drivers/mmc/dw_mmc.c) is next:
>>> --------------->8-------------
>>> MMC:   Synopsys Mobile storage: 0
>>> Buswidth = 0, clock: 400000
>>> selecting mode MMC legacy (freq : 0 MHz)
>>> Buswidth = 1, clock: 400000
>>> Buswidth = 1, clock: 400000
>>> Sending CMD0
>>> dwmci_send_cmd: Timeout.
>>> mmc_init: -110, time 19
>>> ** Bad device mmc 0 **
>>> --------------->8-------------
>>>
>>> Any ideas how to fix this?
>>
>> Could you share which git repository you use? 
> 
> I use u-boot master branch from git://git.denx.de/u-boot.git
> 
> last commits:
> --------------->8-------------
> 485c13c Merge git://git.denx.de/u-boot-dm
> b79221a lib: fdtdec: Fix some style violations
> 2e38662 lib: fdtdec: Fix whitespace style violations
> 29d11b8 core: Make device_is_compatible live-tree compatible
> 147c607 core: Add {ofnode, dev}_translate_address functions
> --------------->8-------------
> 

I guess that this issue has already been fixed.

http://patchwork.ozlabs.org/patch/862160/
http://patchwork.ozlabs.org/patch/864607/

Best Regards,
Jaehoon Chung

>> Best Regards,
>> Jaehoon Chung
>>
>>>
>>> Thanks.
>>>
>>
>>

[U-Boot] mmc: disable UHS modes if Vcc cannot be switched on and off

[Eugeniy Paltsev]

On Wed, 2018-01-24 at 17:28 +0900, Jaehoon Chung wrote:
> On 01/23/2018 09:03 PM, Eugeniy Paltsev wrote:
> > On Tue, 2018-01-23 at 10:44 +0900, Jaehoon Chung wrote:
> > > On 01/23/2018 03:38 AM, Eugeniy Paltsev wrote:
> > > > Hi Jean-Jacques,
> > > > 
> > > > after commit "mmc: disable UHS modes if Vcc cannot be switched on and off" (04a2ea248f)
> > > > we got MMC broken on several (at least two) platforms with DesignWare MMC controller.
> > > > 
> > > > The error log (with debug enabled in drivers/mmc/mmc.c and drivers/mmc/dw_mmc.c) is next:
> > > > --------------->8-------------
> > > > MMC:   Synopsys Mobile storage: 0
> > > > Buswidth = 0, clock: 400000
> > > > selecting mode MMC legacy (freq : 0 MHz)
> > > > Buswidth = 1, clock: 400000
> > > > Buswidth = 1, clock: 400000
> > > > Sending CMD0
> > > > dwmci_send_cmd: Timeout.
> > > > mmc_init: -110, time 19
> > > > ** Bad device mmc 0 **
> > > > --------------->8-------------
> > > > 
> > > > Any ideas how to fix this?
> > > 
> > > Could you share which git repository you use? 
> > 
> > I use u-boot master branch from git://git.denx.de/u-boot.git
> > 
> > last commits:
> > --------------->8-------------
> > 485c13c Merge git://git.denx.de/u-boot-dm
> > b79221a lib: fdtdec: Fix some style violations
> > 2e38662 lib: fdtdec: Fix whitespace style violations
> > 29d11b8 core: Make device_is_compatible live-tree compatible
> > 147c607 core: Add {ofnode, dev}_translate_address functions
> > --------------->8-------------
> > 
> 
> I guess that this issue has already been fixed.
> 
> http://patchwork.ozlabs.org/patch/862160/
> 

Yep, this patch solves the problem.

Tested-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>

Thanks.

> Best Regards,
> Jaehoon Chung
> 
> > > Best Regards,
> > > Jaehoon Chung
> > > 
> > > > 
> > > > Thanks.
> > > > 
> > > 
> > > 
> 
> 
-- 
 Eugeniy Paltsev