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

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

 



Hello,

Heiko Schocher wrote:
> The MMC driver allocates channels with EVENTQ_DEFAULT, they
> get put into EVENTQ_1, which the second EDMA controller does
> not have and hence transfers stall. This is fixed
> in commit f23fe857bbea393b4b94fe2218c98d934bd3d4cf
> "ARM: davinci: Explicitly set channel controllers' default queues"
> from Ido Yariv. This patch sets immediately in edma_probe()
> the proper default_queue, so this patch does not really fix
> something, it is more a cosmetic change.
> 
> Signed-off-by: Heiko Schocher <hs@xxxxxxx>
> Signed-off-by: juha.kuikka@xxxxxxxxx
> Reported-by: juha.kuikka@xxxxxxxxx
> Acked-by: Rajashekhara, Sudhakar <sudhakar.raj@xxxxxx>
> Cc: linux-mmc@xxxxxxxxxxxxxxx
> Cc: davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx
> Cc: Rajashekhara, Sudhakar <sudhakar.raj@xxxxxx>
> Cc: Ido Yariv <ido@xxxxxxxxxx>
> Cc: Sekhar Nori <nsekhar@xxxxxx>
> Cc: Wolfgang Denk <wd@xxxxxxx>
> Cc: Sergei Shtylyov <sshtylyov@xxxxxxxxxx>
> 
> ---
> - changes for v2:
>   - add comment from Sergei Shtylyov
>     add in commit message the commit's summary in parens.
>   - add comment from Ido Yariv:
>     changed comment and commit message
>   - added Acked-by from Rajashekhara, Sudhakar
> 
>  arch/arm/mach-davinci/dma.c |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)

Ping. Can this patch go into mainline, or are there some more objections?

Thanks
bye,
Heiko
-- 
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