From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
Date: Wed, 24 Jan 2018 14:23:44 +0000
Subject: [U-Boot] mmc: disable UHS modes if Vcc cannot be switched on
	and off
In-Reply-To: <8d577459-aa3a-6b52-5e32-859c3caa722d@samsung.com>
References: <CGME20180122183900epcas2p2400da7fe3ed0e00a07272450aee92a17@epcas2p2.samsung.com>
 <1516646287.5399.67.camel@synopsys.com>
 <1471c6d2-c7f9-f9ea-2a3f-db853a133965@samsung.com>
 <1516709004.5399.69.camel@synopsys.com>
 <8d577459-aa3a-6b52-5e32-859c3caa722d@samsung.com>
Message-ID: <1516803823.5399.72.camel@synopsys.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

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