On 15/08/2019 21:29, Eddie James wrote:
> Add support for the Aspeed SD host controller engine.
> 
> Signed-off-by: Eddie James <eaja...@linux.ibm.com>


Reviewed-by: Cédric Le Goater <c...@kaod.org>

Thanks,

C.

> ---
>  arch/arm/include/asm/gpio.h |  3 +-
>  drivers/mmc/Kconfig         | 11 ++++++
>  drivers/mmc/Makefile        |  1 +
>  drivers/mmc/aspeed_sdhci.c  | 90 
> +++++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 104 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/mmc/aspeed_sdhci.c
> 
> diff --git a/arch/arm/include/asm/gpio.h b/arch/arm/include/asm/gpio.h
> index 370031f..38a5922 100644
> --- a/arch/arm/include/asm/gpio.h
> +++ b/arch/arm/include/asm/gpio.h
> @@ -1,6 +1,7 @@
>  #if !defined(CONFIG_ARCH_UNIPHIER) && !defined(CONFIG_ARCH_STI) && \
>       !defined(CONFIG_ARCH_K3) && !defined(CONFIG_ARCH_BCM6858) && \
> -     !defined(CONFIG_ARCH_BCM63158) && !defined(CONFIG_ARCH_ROCKCHIP)
> +     !defined(CONFIG_ARCH_BCM63158) && !defined(CONFIG_ARCH_ROCKCHIP) && \
> +     !defined(CONFIG_ARCH_ASPEED)
>  #include <asm/arch/gpio.h>
>  #endif
>  #include <asm-generic/gpio.h>
> diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
> index c6812f6..536f66a 100644
> --- a/drivers/mmc/Kconfig
> +++ b/drivers/mmc/Kconfig
> @@ -421,6 +421,17 @@ config SPL_MMC_SDHCI_ADMA
>         This enables support for the ADMA (Advanced DMA) defined
>         in the SD Host Controller Standard Specification Version 3.00 in SPL.
>  
> +config MMC_SDHCI_ASPEED
> +     bool "Aspeed SDHCI controller"
> +     depends on ARCH_ASPEED
> +     depends on DM_MMC
> +     depends on MMC_SDHCI
> +     help
> +       Enables support for the Aspeed SDHCI 2.0 controller present on Aspeed
> +       SoCs. This device is compatible with SD 3.0 and/or MMC 4.3
> +       specifications. On the AST2600, the device is also compatible with
> +       MMC 5.1 and eMMC 3.0.
> +
>  config MMC_SDHCI_ATMEL
>       bool "Atmel SDHCI controller support"
>       depends on ARCH_AT91
> diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
> index 6cc018b..5594195 100644
> --- a/drivers/mmc/Makefile
> +++ b/drivers/mmc/Makefile
> @@ -46,6 +46,7 @@ obj-$(CONFIG_JZ47XX_MMC) += jz_mmc.o
>  
>  # SDHCI
>  obj-$(CONFIG_MMC_SDHCI)                      += sdhci.o
> +obj-$(CONFIG_MMC_SDHCI_ASPEED)               += aspeed_sdhci.o
>  obj-$(CONFIG_MMC_SDHCI_ATMEL)                += atmel_sdhci.o
>  obj-$(CONFIG_MMC_SDHCI_BCM2835)              += bcm2835_sdhci.o
>  obj-$(CONFIG_MMC_SDHCI_BCMSTB)               += bcmstb_sdhci.o
> diff --git a/drivers/mmc/aspeed_sdhci.c b/drivers/mmc/aspeed_sdhci.c
> new file mode 100644
> index 0000000..1321ec3
> --- /dev/null
> +++ b/drivers/mmc/aspeed_sdhci.c
> @@ -0,0 +1,90 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright 2019 IBM Corp.
> + * Eddie James <eaja...@linux.ibm.com>
> + */
> +
> +#include <common.h>
> +#include <clk.h>
> +#include <dm.h>
> +#include <malloc.h>
> +#include <sdhci.h>
> +
> +struct aspeed_sdhci_plat {
> +     struct mmc_config cfg;
> +     struct mmc mmc;
> +};
> +
> +static int aspeed_sdhci_probe(struct udevice *dev)
> +{
> +     struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev);
> +     struct aspeed_sdhci_plat *plat = dev_get_platdata(dev);
> +     struct sdhci_host *host = dev_get_priv(dev);
> +     u32 max_clk;
> +     struct clk clk;
> +     int ret;
> +
> +     ret = clk_get_by_index(dev, 0, &clk);
> +     if (ret)
> +             return ret;
> +
> +     ret = clk_enable(&clk);
> +     if (ret)
> +             goto free;
> +
> +     host->name = dev->name;
> +     host->ioaddr = (void *)devfdt_get_addr(dev);
> +
> +     max_clk = clk_get_rate(&clk);
> +     if (IS_ERR_VALUE(max_clk)) {
> +             ret = max_clk;
> +             goto err;
> +     }
> +
> +     host->max_clk = max_clk;
> +     host->mmc = &plat->mmc;
> +     host->mmc->dev = dev;
> +     host->mmc->priv = host;
> +     upriv->mmc = host->mmc;
> +
> +     ret = sdhci_setup_cfg(&plat->cfg, host, 0, 0);
> +     if (ret)
> +             goto err;
> +
> +     ret = sdhci_probe(dev);
> +     if (ret)
> +             goto err;
> +
> +     return 0;
> +
> +err:
> +     clk_disable(&clk);
> +free:
> +     clk_free(&clk);
> +     return ret;
> +}
> +
> +static int aspeed_sdhci_bind(struct udevice *dev)
> +{
> +     struct aspeed_sdhci_plat *plat = dev_get_platdata(dev);
> +
> +     return sdhci_bind(dev, &plat->mmc, &plat->cfg);
> +}
> +
> +static const struct udevice_id aspeed_sdhci_ids[] = {
> +     { .compatible = "aspeed,ast2400-sdhci" },
> +     { .compatible = "aspeed,ast2500-sdhci" },
> +     { .compatible = "aspeed,ast2600-sdhci" },
> +     { }
> +};
> +
> +U_BOOT_DRIVER(aspeed_sdhci_drv) = {
> +     .name           = "aspeed_sdhci",
> +     .id             = UCLASS_MMC,
> +     .of_match       = aspeed_sdhci_ids,
> +     .ops            = &sdhci_ops,
> +     .bind           = aspeed_sdhci_bind,
> +     .probe          = aspeed_sdhci_probe,
> +     .priv_auto_alloc_size = sizeof(struct sdhci_host),
> +     .platdata_auto_alloc_size = sizeof(struct aspeed_sdhci_plat),
> +};
> 

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to