[Fwd: Re: [PATCH] arm, da8xx, mmc: set second MMC controllers default queue]

[Heiko Schocher <hs@denx.de>]

Hello Sekhar,

here the forwarded comment from Ido Yariv to my patch. I tried
with current kernel MMC Controller 2 on my am1808 based board
without my fix, and it works fine, so no need for applying my
patch.

Thanks
bye,
Heiko

here the message from Ido:

Betreff: Re: [PATCH] arm, da8xx, mmc: set second MMC controllers default queue
Datum: Mon, 5 Dec 2011 10:35:10 +0200
Von: Ido Yariv <ido@wizery.com>
An: hs@denx.de
CC: juha.kuikka@gmail.com
Referenzen: <1322991210-20486-1-git-send-email-hs@denx.de>	<20111204102741.GL32400@WorkStation>	<4EDC7C00.30603@denx.de>

Hi Heiko,

On Dec 5, 2011 10:08 AM, "Heiko Schocher" <hs@denx.de> wro
> Ok ... patch not longer needed, but isn't it better to setup here
> immediately the right values? If so, I can sent a v2 with your
> suggested comment change.

These are just default values which will get overwritten later on, so im
not sure if it matters much. I guess it wouldn't hurt to use default_queue
instead of queue 1.

In any case, if you do chose to submit v2, the commit message should also
be changed if it doesnt really fix anything.

Thanks,
Ido.


-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany