[U-Boot] [PATCH v2 08/13] drivers: mmc: add driver for Microchip PIC32 SDHCI controller.

[Purna Chandra Mandal]

On 01/06/2016 02:08 AM, Daniel Schwierzeck wrote:

> 2016-01-04 15:01 GMT+01:00 Purna Chandra Mandal <purna.mandal at microchip.com>:
>> PIC32 architecture has in-built SDHCI controller. This driver implements
>> platform specific glue to use common SDHCI functionality.
>>
>> Signed-off-by: Andrei Pistirica <andrei.pistirica at microchip.com>
>> Signed-off-by: Sandeep Sheriker Mallikarjun <sandeepsheriker.mallikarjun at microchip.com>
>> Signed-off-by: Purna Chandra Mandal <purna.mandal at microchip.com>
>>
>> ---
>>
>> Changes in v2:
>> - drop SDHCI shared bus configuration (for shared interrupt-and-clock pins)
>>
>>  drivers/mmc/Kconfig       |  6 +++++
>>  drivers/mmc/Makefile      |  2 +-
>>  drivers/mmc/pic32_sdhci.c | 63 +++++++++++++++++++++++++++++++++++++++++++++++
>>  drivers/mmc/sdhci.c       | 12 +++++++++
>>  4 files changed, 82 insertions(+), 1 deletion(-)
>>  create mode 100644 drivers/mmc/pic32_sdhci.c
>>
>> diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
>> index ceae7bc..0b6f54b 100644
>> --- a/drivers/mmc/Kconfig
>> +++ b/drivers/mmc/Kconfig
>> @@ -31,4 +31,10 @@ config SH_SDHI
>>         help
>>           Support for the on-chip SDHI host controller on SuperH/Renesas ARM SoCs platform
>>
>> +config PIC32_SDHCI
>> +       bool "Microchip PIC32 on-chip SDHCI support"
>> +       depends on DM_MMC && MACH_PIC32
>> +       help
>> +         Support for the on-chip SDHCI support on Microchip PIC32 platforms.
>> +
>>  endmenu
>> diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
>> index 5d35705..c9c3e3e 100644
>> --- a/drivers/mmc/Makefile
>> +++ b/drivers/mmc/Makefile
>> @@ -48,4 +48,4 @@ obj-$(CONFIG_SPL_MMC_BOOT) += fsl_esdhc_spl.o
>>  else
>>  obj-$(CONFIG_GENERIC_MMC) += mmc_write.o
>>  endif
>> -
>> +obj-$(CONFIG_PIC32_SDHCI) += pic32_sdhci.o
>> diff --git a/drivers/mmc/pic32_sdhci.c b/drivers/mmc/pic32_sdhci.c
>> new file mode 100644
>> index 0000000..9eb603e
>> --- /dev/null
>> +++ b/drivers/mmc/pic32_sdhci.c
>> @@ -0,0 +1,63 @@
>> +/*
>> + * Support of SDHCI for Microchip PIC32 SoC.
>> + *
>> + * Copyright (C) 2015 Microchip Technology Inc.
>> + * Andrei Pistirica <andrei.pistirica at microchip.com>
>> + *
>> + * SPDX-License-Identifier:    GPL-2.0+
>> + */
>> +
>> +#include <dm.h>
>> +#include <common.h>
>> +#include <sdhci.h>
>> +#include <asm/errno.h>
>> +#include <mach/pic32.h>
>> +
>> +DECLARE_GLOBAL_DATA_PTR;
>> +
>> +static int pic32_sdhci_probe(struct udevice *dev)
>> +{
>> +       struct sdhci_host *host = dev_get_priv(dev);
>> +       u32 f_min_max[2];
>> +       int ret;
>> +
>> +       ret = fdtdec_get_int_array(gd->fdt_blob, dev->of_offset,
>> +                                  "clock-freq-min-max", f_min_max, 2);
>> +       if (ret) {
>> +               printf("sdhci: clock-freq-min-max not found\n");
>> +               return ret;
>> +       }
>> +
>> +       return add_sdhci(host, f_min_max[1], f_min_max[0]);
>> +}
>> +
>> +static int pic32_sdhci_ofdata_to_platdata(struct udevice *dev)
>> +{
>> +       struct sdhci_host *host = dev_get_priv(dev);
>> +       fdt_addr_t addr;
>> +
>> +       addr = dev_get_addr(dev);
>> +       if (addr == FDT_ADDR_T_NONE)
>> +               return -EINVAL;
>> +
>> +       host->name      = (char *)dev->name;
>> +       host->ioaddr    = pic32_ioremap(addr);
>> +       host->quirks    = SDHCI_QUIRK_NO_HISPD_BIT;
>> +       host->bus_width = fdtdec_get_int(gd->fdt_blob, dev->of_offset,
>> +                                       "bus-width", 4);
>> +       return 0;
> all that code could be moved to pic32_sdhci_probe()

ack.

>> +}
>> +
>> +static const struct udevice_id pic32_sdhci_ids[] = {
>> +       { .compatible = "microchip,pic32mzda-sdhci" },
>> +       { }
>> +};
>> +
>> +U_BOOT_DRIVER(pic32_sdhci_drv) = {
>> +       .name                   = "pic32_sdhci",
>> +       .id                     = UCLASS_MMC,
>> +       .of_match               = pic32_sdhci_ids,
>> +       .probe                  = pic32_sdhci_probe,
>> +       .ofdata_to_platdata     = pic32_sdhci_ofdata_to_platdata,
>> +       .priv_auto_alloc_size   = sizeof(struct sdhci_host),
>> +};
>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>> index 02d71b9..f32fe67 100644
>> --- a/drivers/mmc/sdhci.c
>> +++ b/drivers/mmc/sdhci.c
>> @@ -424,6 +424,18 @@ static void sdhci_set_ios(struct mmc *mmc)
>>         if (host->quirks & SDHCI_QUIRK_NO_HISPD_BIT)
>>                 ctrl &= ~SDHCI_CTRL_HISPD;
>>
>> +#if defined(CONFIG_PIC32_SDHCI)
>> +       /*
>> +       * In PIC32MZ[DA] due to h/w bug SDHCI fails detecting card when JTAG
>> +       * is not connected.
>> +       * To work-around this problem:
>> +       *  - set Card_Detect_Signal_Selection bit in SDHCI_Host_Control register
>> +       *  - clear Card_Detect_Test_Level bit in SDHCI_Host_Control register
>> +       */
>> +       ctrl |= SDHCI_CTRL_CD_TEST;
>> +       ctrl &= ~SDHCI_CTRL_CD_TEST_INS;
>> +#endif
>> +
>>         sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL);
>>  }
>>
>> --
>> 1.8.3.1
>>
>>