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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
An: hs@xxxxxxx
CC: juha.kuikka@xxxxxxxxx
Referenzen: <1322991210-20486-1-git-send-email-hs@xxxxxxx>	<20111204102741.GL32400@WorkStation>	<4EDC7C00.30603@xxxxxxx>

Hi Heiko,

On Dec 5, 2011 10:08 AM, "Heiko Schocher" <hs@xxxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux