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

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

 



Hello Rajashekhara,

Rajashekhara, Sudhakar wrote:
> Hi,
> 
> On Sun, Dec 04, 2011 at 15:03:30, 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 tried to fix
>> in commit f23fe857bbea393b4b94fe2218c98d934bd3d4cf
>> from Ido Yariv, but missed a fix for the second MMC
>> controller on da850.
>>
>> Signed-off-by: Heiko Schocher <hs@xxxxxxx>
>> Signed-off-by: juha.kuikka@xxxxxxxxx
>> Reported-by: juha.kuikka@xxxxxxxxx
>> 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>
>> ---
>>  arch/arm/mach-davinci/dma.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/dma.c b/arch/arm/mach-davinci/dma.c
>> index da90103..e10a251 100644
>> --- a/arch/arm/mach-davinci/dma.c
>> +++ b/arch/arm/mach-davinci/dma.c
>> @@ -1513,7 +1513,7 @@ static int __init edma_probe(struct platform_device *pdev)
>>  		 * started by the codec engine will not cause audio defects.
>>  		 */
>>  		for (i = 0; i < edma_cc[j]->num_channels; i++)
>> -			map_dmach_queue(j, i, EVENTQ_1);
>> +			map_dmach_queue(j, i, info[j]->default_queue);
>>  
>>  		queue_tc_mapping = info[j]->queue_tc_mapping;
>>  		queue_priority_mapping = info[j]->queue_priority_mapping;
> 
> Acked-by: Rajashekhara, Sudhakar <sudhakar.raj@xxxxxx>

Thanks, but as Ido commented, this patch is no longer necessary, sorry
for the noise.

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