[U-Boot] [PATCH] drivers: mmc: Change buffer type in ALLOC_CACHE_ALIGN_BUFFER macro

Suniel Mahesh sunil.m at techveda.org
Fri Aug 4 09:33:40 UTC 2017


On Monday 17 July 2017 04:38 PM, Jaehoon Chung wrote:
> On 06/19/2017 04:33 PM, sunil.m at techveda.org wrote:
>> From: Suniel Mahesh <sunil.m at techveda.org>
>>
>> __be32_to_cpu() accepts argument of type __be32. This patch changes type of
>> the buffer in ALLOC_CACHE_ALIGN_BUFFER macro to __be32, which is then passed
>> to __be32_to_cpu().
>> This prevents sparse build warnings.
>> drivers/mmc/mmc.c: warning: cast to restricted __be32
>>
>> Signed-off-by: Suniel Mahesh <sunil.m at techveda.org>
>> Signed-off-by: Karthik Tummala <karthik at techveda.org>
> 
> Applied to u-boot-mmc.

Hi, It says the patch has been applied to u-boot-mmc git tree,
but I couldn't find it there applied.
http://git.denx.de/?p=u-boot/u-boot-mmc.git;a=summary

are there any issues with the patch or am I looking at the wrong place ? 

Thanks
suniel
> 
> Best Regards,
> Jaehoon Chung
> 
>> ---
>> Note:
>> - Build was carried out with the above changes, no build errors
>>   reported.
>> ---
>>  drivers/mmc/mmc.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
>> index 3cdf6a4..3d4da4c 100644
>> --- a/drivers/mmc/mmc.c
>> +++ b/drivers/mmc/mmc.c
>> @@ -882,8 +882,8 @@ static int sd_change_freq(struct mmc *mmc)
>>  {
>>  	int err;
>>  	struct mmc_cmd cmd;
>> -	ALLOC_CACHE_ALIGN_BUFFER(uint, scr, 2);
>> -	ALLOC_CACHE_ALIGN_BUFFER(uint, switch_status, 16);
>> +	ALLOC_CACHE_ALIGN_BUFFER(__be32, scr, 2);
>> +	ALLOC_CACHE_ALIGN_BUFFER(__be32, switch_status, 16);
>>  	struct mmc_data data;
>>  	int timeout;
>>  
>>
> 



More information about the U-Boot mailing list