Hi Wenyou, On 08/11/2016 11:03 AM, Jaehoon Chung wrote: > Hi, > > On 08/10/2016 11:51 AM, Wenyou Yang wrote: >> Convert the driver to the driver model while retaining the existing >> legacy code. This allows the driver to support boards that have >> converted to driver model as well as those that have not. >> >> Signed-off-by: Wenyou Yang <wenyou.y...@atmel.com> >> Reviewed-by: Simon Glass <s...@chromium.org> >> Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com> >> Reviewed-by: Heiko Schocher <h...@denx.de> > > If others are ok, I will pick this patch to u-boot-mmc.
Applied on u-boot-mmc. Thanks! Best Regards, Jaehoon Chung > > Best Regards, > Jaehoon Chung > >> --- >> >> Changes in v11: >> - Due the removal of unnecessary arguments for sdhci_setup_cfg(), >> change accordingly. >> >> Changes in v10: >> - Add Reviewed-by tag. >> >> Changes in v9: >> - Add Reviewed-by tag. >> >> Changes in v8: >> - Make atmel_sdhci_get_clk() to get clock device. >> - Use ulong type for gck_rate. >> - Remove meaningless type casting before dev->name. >> >> Changes in v7: >> - Add support for using driver model for block devices and MMC operations. >> - Change clk_client.h -> clk.h to adapt to clk API conversion. >> >> Changes in v6: >> - Remove unnecessary white space. >> - Use sdhci_read(), instead of readl(). >> - Remove the local variables min_clk. >> >> Changes in v5: >> - Add Reviewed-by tag. >> >> Changes in v4: >> - Update the clk API based on [PATCH] clk: convert API to match >> reset/mailbox fstyle (http://patchwork.ozlabs.org/patch/625342/). >> - Remove check on dev_get_parent() return. >> - Fixed the return value, such as -ENODEV->-EINVAL. >> >> Changes in v3: >> - Remove the redundant log print. >> >> Changes in v2: >> - Add clock support, include enabling peripheral clock >> and generated clock. >> - Retain the existing legacy code to support boards which have not >> converted to driver model. >> >> drivers/mmc/Kconfig | 10 ++++ >> drivers/mmc/atmel_sdhci.c | 123 >> ++++++++++++++++++++++++++++++++++++++++++++++ >> include/sdhci.h | 2 + >> 3 files changed, 135 insertions(+) >> >> diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig >> index dc8f2b6..3616dee 100644 >> --- a/drivers/mmc/Kconfig >> +++ b/drivers/mmc/Kconfig >> @@ -34,6 +34,16 @@ config MSM_SDHCI >> SD 3.0 specifications. Both SD and eMMC devices are supported. >> Card-detect gpios are not supported. >> >> +config ATMEL_SDHCI >> + bool "Atmel SDHCI controller support" >> + depends on DM_MMC && BLK && DM_MMC_OPS && ARCH_AT91 >> + help >> + This enables support for the Atmel SDHCI controller, which supports >> + the embedded MultiMedia Card (e.MMC) Specification V4.51, the SD >> + Memory Card Specification V3.0, and the SDIO V3.0 specification. >> + It is compliant with the SD Host Controller Standard V3.0 >> + specification. >> + >> config ROCKCHIP_DWMMC >> bool "Rockchip SD/MMC controller support" >> depends on DM_MMC && OF_CONTROL >> diff --git a/drivers/mmc/atmel_sdhci.c b/drivers/mmc/atmel_sdhci.c >> index 24b68b6..dd6bd33 100644 >> --- a/drivers/mmc/atmel_sdhci.c >> +++ b/drivers/mmc/atmel_sdhci.c >> @@ -6,12 +6,15 @@ >> */ >> >> #include <common.h> >> +#include <clk.h> >> +#include <dm.h> >> #include <malloc.h> >> #include <sdhci.h> >> #include <asm/arch/clk.h> >> >> #define ATMEL_SDHC_MIN_FREQ 400000 >> >> +#ifndef CONFIG_DM_MMC >> int atmel_sdhci_init(void *regbase, u32 id) >> { >> struct sdhci_host *host; >> @@ -38,3 +41,123 @@ int atmel_sdhci_init(void *regbase, u32 id) >> >> return 0; >> } >> + >> +#else >> + >> +DECLARE_GLOBAL_DATA_PTR; >> + >> +struct atmel_sdhci_plat { >> + struct mmc_config cfg; >> + struct mmc mmc; >> +}; >> + >> +static int atmel_sdhci_get_clk(struct udevice *dev, int index, struct clk >> *clk) >> +{ >> + struct udevice *dev_clk; >> + int periph, ret; >> + >> + ret = clk_get_by_index(dev, index, clk); >> + if (ret) >> + return ret; >> + >> + periph = fdtdec_get_uint(gd->fdt_blob, clk->dev->of_offset, "reg", -1); >> + if (periph < 0) >> + return -EINVAL; >> + >> + dev_clk = dev_get_parent(clk->dev); >> + ret = clk_request(dev_clk, clk); >> + if (ret) >> + return ret; >> + >> + clk->id = periph; >> + >> + return 0; >> +} >> + >> +static int atmel_sdhci_probe(struct udevice *dev) >> +{ >> + struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev); >> + struct atmel_sdhci_plat *plat = dev_get_platdata(dev); >> + struct sdhci_host *host = dev_get_priv(dev); >> + u32 max_clk; >> + u32 caps, caps_1; >> + u32 clk_base, clk_mul; >> + ulong gck_rate; >> + struct clk clk; >> + int ret; >> + >> + ret = atmel_sdhci_get_clk(dev, 0, &clk); >> + if (ret) >> + return ret; >> + >> + ret = clk_enable(&clk); >> + if (ret) >> + return ret; >> + >> + host->name = dev->name; >> + host->ioaddr = (void *)dev_get_addr(dev); >> + >> + host->quirks = 0; >> + host->bus_width = fdtdec_get_int(gd->fdt_blob, dev->of_offset, >> + "bus-width", 4); >> + >> + caps = sdhci_readl(host, SDHCI_CAPABILITIES); >> + clk_base = (caps & SDHCI_CLOCK_V3_BASE_MASK) >> SDHCI_CLOCK_BASE_SHIFT; >> + caps_1 = sdhci_readl(host, SDHCI_CAPABILITIES_1); >> + clk_mul = (caps_1 & SDHCI_CLOCK_MUL_MASK) >> SDHCI_CLOCK_MUL_SHIFT; >> + gck_rate = clk_base * 1000000 * (clk_mul + 1); >> + >> + ret = atmel_sdhci_get_clk(dev, 1, &clk); >> + if (ret) >> + return ret; >> + >> + ret = clk_set_rate(&clk, gck_rate); >> + if (ret) >> + return ret; >> + >> + max_clk = clk_get_rate(&clk); >> + if (!max_clk) >> + return -EINVAL; >> + >> + ret = sdhci_setup_cfg(&plat->cfg, host, max_clk, ATMEL_SDHC_MIN_FREQ); >> + if (ret) >> + return ret; >> + >> + host->mmc = &plat->mmc; >> + host->mmc->dev = dev; >> + host->mmc->priv = host; >> + upriv->mmc = host->mmc; >> + >> + clk_free(&clk); >> + >> + return sdhci_probe(dev); >> +} >> + >> +static int atmel_sdhci_bind(struct udevice *dev) >> +{ >> + struct atmel_sdhci_plat *plat = dev_get_platdata(dev); >> + int ret; >> + >> + ret = sdhci_bind(dev, &plat->mmc, &plat->cfg); >> + if (ret) >> + return ret; >> + >> + return 0; >> +} >> + >> +static const struct udevice_id atmel_sdhci_ids[] = { >> + { .compatible = "atmel,sama5d2-sdhci" }, >> + { } >> +}; >> + >> +U_BOOT_DRIVER(atmel_sdhci_drv) = { >> + .name = "atmel_sdhci", >> + .id = UCLASS_MMC, >> + .of_match = atmel_sdhci_ids, >> + .ops = &sdhci_ops, >> + .bind = atmel_sdhci_bind, >> + .probe = atmel_sdhci_probe, >> + .priv_auto_alloc_size = sizeof(struct sdhci_host), >> + .platdata_auto_alloc_size = sizeof(struct atmel_sdhci_plat), >> +}; >> +#endif >> diff --git a/include/sdhci.h b/include/sdhci.h >> index fcef4dc..6844c73 100644 >> --- a/include/sdhci.h >> +++ b/include/sdhci.h >> @@ -166,6 +166,8 @@ >> #define SDHCI_CAN_64BIT 0x10000000 >> >> #define SDHCI_CAPABILITIES_1 0x44 >> +#define SDHCI_CLOCK_MUL_MASK 0x00FF0000 >> +#define SDHCI_CLOCK_MUL_SHIFT 16 >> >> #define SDHCI_MAX_CURRENT 0x48 >> >> > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
