Hi,

On 03/21/2014 08:39 PM, Olliver Schinagl wrote:
> David,
> 
> can you confirm/deny this change makes sense? If so, can you give mnemoc the 
> ok to go ahead with this fix?
> 
> Olliver
> 
> On 03/10/2014 02:58 AM, Wills Wang wrote:
>> ---
>>   drivers/mmc/host/sunxi-mci.h | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/mmc/host/sunxi-mci.h b/drivers/mmc/host/sunxi-mci.h
>> index 1447257..b885c32 100644
>> --- a/drivers/mmc/host/sunxi-mci.h
>> +++ b/drivers/mmc/host/sunxi-mci.h
>> @@ -35,7 +35,7 @@
>>   #define MMC_MODCLK_PREFIX        "sdc"
>>   #define MMC3_DMA_TL                (0x20070008)
>>   #define MMC_MAX_DMA_DES_BIT        13
>> -#define MMC_DMA_DES_BIT_LEFT    5
>> +#define MMC_DMA_DES_BIT_LEFT    6
>>   #endif
>>
>>   /*---------- for sun5i ----------*/

The above chunk is correct, it will not matter functionally but the correct
value for MMC_DMA_DES_BIT_LEFT on sun4i is 6 not 5.

>> @@ -250,7 +250,7 @@
>>   #define SDXC_IDMA_ERR (SDXC_IDMACFatalBusErr|SDXC_IDMACDesInvalid \
>>               |SDXC_IDMACCardErrSum|SDXC_IDMACAbnormalIntSum)
>>
>> -#define SDXC_DES_NUM_SHIFT    (MMC_MAX_DMA_DES_BIT)
>> +#define SDXC_DES_NUM_SHIFT    (MMC_MAX_DMA_DES_BIT - 1)
>>   #define SDXC_DES_BUFFER_MAX_LEN    (1U << SDXC_DES_NUM_SHIFT)
>>   struct sunxi_mmc_idma_des {
>>       u32    config;

This chunk is non-sense and should not I repeat NOT be applied.

Regards,

Hans

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to