[U-Boot] [PATCH] drivers/mmc/mv_sdhci.c: Fix build warning

Lei Wen adrian.wenl at gmail.com
Mon Dec 12 15:33:56 CET 2011


Hi Anatolij,

On Thu, Dec 8, 2011 at 5:47 AM, Anatolij Gustschin <agust at denx.de> wrote:
> Fix:
> mv_sdhci.c: In function 'mv_sdh_init':
> mv_sdhci.c:47:22: warning: the comparison will always
> evaluate as 'true' for the address of 'mv_sdhci_writeb'
> will never be NULL [-Waddress]
>
> Signed-off-by: Anatolij Gustschin <agust at denx.de>
> Cc: Lei Wen <leiwen at marvell.com>
> Cc: Andy Fleming <afleming at freescale.com>
> ---
>  drivers/mmc/mv_sdhci.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/mv_sdhci.c b/drivers/mmc/mv_sdhci.c
> index 1501974..2fe34b6 100644
> --- a/drivers/mmc/mv_sdhci.c
> +++ b/drivers/mmc/mv_sdhci.c
> @@ -44,8 +44,7 @@ int mv_sdh_init(u32 regbase, u32 max_clk, u32 min_clk, u32 quirks)
>        host->quirks = quirks;
>  #ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
>        memset(&mv_ops, 0, sizeof(struct sdhci_ops));
> -       if (mv_sdhci_writeb != NULL)
> -               mv_ops.write_b = mv_sdhci_writeb;
> +       mv_ops.write_b = mv_sdhci_writeb;
>        host->ops = &mv_ops;
>  #endif
>        if (quirks & SDHCI_QUIRK_REG32_RW)
> --
> 1.7.5.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

Thanks for pointing it out!

Acked-by: Lei Wen <leiwen at marvell.com>


More information about the U-Boot mailing list