[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Pali Rohár]

Hi,

this commit is breaking MMC support on Nokia N900:
a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be clear

I'm geting only error message "timedout waiting on cmd inhibit to clear" and I 
cannot read files from eMMC and SD card. When I reverted commit that commit I 
all is OK.

-- 
Pali Roh?r
pali.rohar at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120304/ce599501/attachment.pgp>

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Tom Rini]

On Sun, Mar 4, 2012 at 1:06 PM, Pali Roh?r <pali.rohar@gmail.com> wrote:
> Hi,
>
> this commit is breaking MMC support on Nokia N900:
> a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be clear
>
> I'm geting only error message "timedout waiting on cmd inhibit to clear" and I
> cannot read files from eMMC and SD card. When I reverted commit that commit I
> all is OK.

OK, we need to debug what the n900 is doing (do you see this in qemu,
real HW or both?) since clearing the bit fixes certain SD cards on
other boards (beagleboard and gumstix iirc, but it was a generic
issue).  Thanks!

-- 
Tom

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Pali Rohár]

On Sunday 04 March 2012 13:23:44 Tom Rini wrote:
> On Sun, Mar 4, 2012 at 1:06 PM, Pali Roh?r <pali.rohar@gmail.com> wrote:
> > Hi,
> > 
> > this commit is breaking MMC support on Nokia N900:
> > a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be
> > clear
> > 
> > I'm geting only error message "timedout waiting on cmd inhibit to clear"
> > and I cannot read files from eMMC and SD card. When I reverted commit
> > that commit I all is OK.
> 
> OK, we need to debug what the n900 is doing (do you see this in qemu,
> real HW or both?) since clearing the bit fixes certain SD cards on
> other boards (beagleboard and gumstix iirc, but it was a generic
> issue).  Thanks!

RX-51 build of U-Boot in qemu cannot read mmc. It report error about bad 
voltage. This was problem also before that commit. RX-51 Linux kernel does not 
have any problem with mmc, special prepaired Maemo 5 image (with some SW on 
eMMC) from Nokia working fine in qemu.

I ignored above problem in qemu, because mmc worked fine on real n900. But 
after that commit mmc does not working on real n900 too.

-- 
Pali Roh?r
pali.rohar at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120304/d00f07af/attachment.pgp>

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Tom Rini]

On 03/04/2012 01:32 PM, Pali Roh?r wrote:
> On Sunday 04 March 2012 13:23:44 Tom Rini wrote:
>> On Sun, Mar 4, 2012 at 1:06 PM, Pali Roh?r<pali.rohar@gmail.com>  wrote:
>>> Hi,
>>>
>>> this commit is breaking MMC support on Nokia N900:
>>> a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be
>>> clear
>>>
>>> I'm geting only error message "timedout waiting on cmd inhibit to clear"
>>> and I cannot read files from eMMC and SD card. When I reverted commit
>>> that commit I all is OK.
>>
>> OK, we need to debug what the n900 is doing (do you see this in qemu,
>> real HW or both?) since clearing the bit fixes certain SD cards on
>> other boards (beagleboard and gumstix iirc, but it was a generic
>> issue).  Thanks!
>
> RX-51 build of U-Boot in qemu cannot read mmc. It report error about bad
> voltage. This was problem also before that commit. RX-51 Linux kernel does not
> have any problem with mmc, special prepaired Maemo 5 image (with some SW on
> eMMC) from Nokia working fine in qemu.
>
> I ignored above problem in qemu, because mmc worked fine on real n900. But
> after that commit mmc does not working on real n900 too.

First, if you change SD cards, does the problem go away?  Second, my 
expectation is that we will make further changes, probably not in time 
for v2012.03, to the omap_hsmmc.c driver to bring it more in-line with 
how the kernel driver works.  Thanks!

-- 
Tom

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Pali Rohár]

On Monday 05 March 2012 07:58:16 Tom Rini wrote:
> On 03/04/2012 01:32 PM, Pali Roh?r wrote:
> > On Sunday 04 March 2012 13:23:44 Tom Rini wrote:
> >> On Sun, Mar 4, 2012 at 1:06 PM, Pali Roh?r<pali.rohar@gmail.com>  wrote:
> >>> Hi,
> >>> 
> >>> this commit is breaking MMC support on Nokia N900:
> >>> a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be
> >>> clear
> >>> 
> >>> I'm geting only error message "timedout waiting on cmd inhibit to clear"
> >>> and I cannot read files from eMMC and SD card. When I reverted commit
> >>> that commit I all is OK.
> >> 
> >> OK, we need to debug what the n900 is doing (do you see this in qemu,
> >> real HW or both?) since clearing the bit fixes certain SD cards on
> >> other boards (beagleboard and gumstix iirc, but it was a generic
> >> issue).  Thanks!
> > 
> > RX-51 build of U-Boot in qemu cannot read mmc. It report error about bad
> > voltage. This was problem also before that commit. RX-51 Linux kernel does
> > not have any problem with mmc, special prepaired Maemo 5 image (with some
> > SW on eMMC) from Nokia working fine in qemu.
> > 
> > I ignored above problem in qemu, because mmc worked fine on real n900. But
> > after that commit mmc does not working on real n900 too.
> 
> First, if you change SD cards, does the problem go away?  Second, my
> expectation is that we will make further changes, probably not in time
> for v2012.03, to the omap_hsmmc.c driver to bring it more in-line with
> how the kernel driver works.  Thanks!

Problem is with internal eMMC (which cannot be changed) and with some external 
Samsung microSD class 10. I do not have other microSD cards for testing now.

-- 
Pali Roh?r
pali.rohar at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120305/6d183cd8/attachment.pgp>

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Tom Rini]

On Mon, Mar 05, 2012 at 11:03:35PM +0100, Pali Roh?r wrote:
> On Monday 05 March 2012 07:58:16 Tom Rini wrote:
> > On 03/04/2012 01:32 PM, Pali Roh?r wrote:
> > > On Sunday 04 March 2012 13:23:44 Tom Rini wrote:
> > >> On Sun, Mar 4, 2012 at 1:06 PM, Pali Roh?r<pali.rohar@gmail.com>  wrote:
> > >>> Hi,
> > >>> 
> > >>> this commit is breaking MMC support on Nokia N900:
> > >>> a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be
> > >>> clear
> > >>> 
> > >>> I'm geting only error message "timedout waiting on cmd inhibit to clear"
> > >>> and I cannot read files from eMMC and SD card. When I reverted commit
> > >>> that commit I all is OK.
> > >> 
> > >> OK, we need to debug what the n900 is doing (do you see this in qemu,
> > >> real HW or both?) since clearing the bit fixes certain SD cards on
> > >> other boards (beagleboard and gumstix iirc, but it was a generic
> > >> issue).  Thanks!
> > > 
> > > RX-51 build of U-Boot in qemu cannot read mmc. It report error about bad
> > > voltage. This was problem also before that commit. RX-51 Linux kernel does
> > > not have any problem with mmc, special prepaired Maemo 5 image (with some
> > > SW on eMMC) from Nokia working fine in qemu.
> > > 
> > > I ignored above problem in qemu, because mmc worked fine on real n900. But
> > > after that commit mmc does not working on real n900 too.
> > 
> > First, if you change SD cards, does the problem go away?  Second, my
> > expectation is that we will make further changes, probably not in time
> > for v2012.03, to the omap_hsmmc.c driver to bring it more in-line with
> > how the kernel driver works.  Thanks!
> 
> Problem is with internal eMMC (which cannot be changed) and with some external 
> Samsung microSD class 10. I do not have other microSD cards for testing now.

Is there anyone you can borrow another uSD card from?

-- 
Tom

[U-Boot] MMC broken after a7778f8fbee098c78b1fa1e1331313a7e217fb30

[Grazvydas Ignotas]

On Sun, Mar 4, 2012 at 10:06 PM, Pali Roh?r <pali.rohar@gmail.com> wrote:
> Hi,
>
> this commit is breaking MMC support on Nokia N900:
> a7778f8fbee098c78b1fa1e1331313a7e217fb30 omap_hsmmc: Wait for CMDI to be clear
>
> I'm geting only error message "timedout waiting on cmd inhibit to clear" and I
> cannot read files from eMMC and SD card. When I reverted commit that commit I
> all is OK.

Same problem here on pandora board, I have old 512MB SanDisk, 4GB SDHC
A-data, 64 SDXC Transcend and only A-data one works with this patch,
after revert all of them work fine.



-- 
Gra?vydas