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

Jaehoon Chung jh80.chung at samsung.com
Mon Jul 17 11:08:38 UTC 2017


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.

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