On 09/19/2016 09:05 PM, Wenyou Yang wrote:
Add the driver model support for Atmel mci 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.


diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig

+config GENERIC_ATMEL_MCI
+       bool "Atmel Multimedia Card Interface support"
+       depends on DM_MMC && BLK && DM_MMC_OPS && ARCH_AT91
+       help
+         This enables support for Atmel High Speed Multimedia Card Interface
+         (HSMCI), which supports the MultiMedia Card (MMC) Specification V4.3,
+         the SD Memory Card Specification V2.0, the SDIO V2.0 specification
+         and CE-ATA V1.1.

This seems unrelated to the actual driver conversion. It's more like part of a convert-the-option-to-Kconfig patch. I suspect it should be removed for now?

I think you need to remove "#define GENERIC_ATMEL_MCI" from include/configs/*.h and add it to configs/*_defconfig as part of the patch that adds this Kconfig option.

+#ifndef CONFIG_DM_MMC
 /* Setup for MCI Clock and Block Size */
 static void mci_set_mode(struct mmc *mmc, u32 hz, u32 blklen)
 {
        struct atmel_mci_priv *priv = mmc->priv;
-       atmel_mci_t *mci = priv->mci;
        u32 bus_hz = get_mci_clk_rate();
+#else
+static void mci_set_mode(struct atmel_mci_priv *priv, u32 hz, u32 blklen)
+{
+       struct mmc *mmc = &priv->mmc;
+       u32 bus_hz = priv->bus_clk_rate;
+#endif

It's technically CONFIG_DM_MMC_OPS that changes that function signature, I believe. This code could allow someone to enable just CONFIG_DM_MMC but not CONFIG_DM_MMC_OPS and then see a build error. Once everything is converted to Kconfig that won't be possible due to the "depends on" you added above, but while the option is still selected by config.h files, this is possible.

Still, this is a minor issue so if you're going to do the config.h -> defconfig conversion soon, maybe just ignore this.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to