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 <[email protected]>
An: [email protected]
CC: [email protected]
Referenzen: <[email protected]>      
<20111204102741.GL32400@WorkStation>    <[email protected]>

Hi Heiko,

On Dec 5, 2011 10:08 AM, "Heiko Schocher" <[email protected]> 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
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to