[U-Boot] [PATCH] mmc: xenon_sdhci: Add missing host->max_clk to Xenon SDHCI driver

Jaehoon Chung jh80.chung at samsung.com
Tue Mar 21 12:07:14 UTC 2017


Hi Stefan,

On 03/21/2017 01:00 AM, Stefan Roese wrote:
> The Xenon SDHCI driver just missed the integration of this patch:
> 
> git ID 6d0e34bf
> mmc: sdhci: Distinguish between base clock and maximum peripheral frequency
> 
> With this patch applied, the SDHCI subsystem complains now with this warning
> while probing:
> 
> sdhci_setup_cfg: Hardware doesn't specify base clock frequency
> 
> This patch fixes this issue, by providing the missing host->max_clk
> variable to the SDHCI subsystem.
> 
> Signed-off-by: Stefan Roese <sr at denx.de>
> Cc: Hu Ziji <huziji at marvell.com>
> Cc: Victor Gu <xigu at marvell.com>
> Cc: Konstantin Porotchkin <kostap at marvell.com>
> Cc: Nadav Haklai <nadavh at marvell.com>
> Cc: Stefan Herbrechtsmeier <stefan.herbrechtsmeier at weidmueller.com>
> Cc: Jaehoon Chung <jh80.chung at samsung.com>

Applied on u-boot-mmc. Thanks!

Best Regards,
Jaehoon Chung

> ---
>  drivers/mmc/xenon_sdhci.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/xenon_sdhci.c b/drivers/mmc/xenon_sdhci.c
> index f678110239..2a0d8b46c6 100644
> --- a/drivers/mmc/xenon_sdhci.c
> +++ b/drivers/mmc/xenon_sdhci.c
> @@ -422,7 +422,8 @@ static int xenon_sdhci_probe(struct udevice *dev)
>  
>  	host->ops = &xenon_sdhci_ops;
>  
> -	ret = sdhci_setup_cfg(&plat->cfg, host, XENON_MMC_MAX_CLK, 0);
> +	host->max_clk = XENON_MMC_MAX_CLK;
> +	ret = sdhci_setup_cfg(&plat->cfg, host, 0, 0);
>  	if (ret)
>  		return ret;
>  
> 



More information about the U-Boot mailing list