mmc: mmc_legacy: fix the compiler error with disabled CONFIG_DM_MMC_OPS
authorJaehoon Chung <jh80.chung@samsung.com>
Fri, 12 Aug 2016 02:39:05 +0000 (11:39 +0900)
committerJaehoon Chung <jh80.chung@samsung.com>
Tue, 16 Aug 2016 01:27:24 +0000 (10:27 +0900)
To prevent the compiler error, split the checking condition whether
cfg->ops is NULL or not.
It's more clearly, because it's not included in mmc_config structure
when CONFIG_DM_MMC_OPS is disabled.

drivers/mmc/mmc_legacy.c: In function ‘mmc_create’:
drivers/mmc/mmc_legacy.c:118:31: error: ‘const struct mmc_config’ has no member named ‘ops’
drivers/mmc/mmc_legacy.c:118:58: error: ‘const struct mmc_config’ has no member named ‘ops’
make[1]: *** [drivers/mmc/mmc_legacy.o] Error 1

Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/mmc/mmc_legacy.c

index 040728b45d3c9f415b31bdb330f506319457e45a..25361d10f60cfa04bd53724161819ca7e120c62f 100644 (file)
@@ -115,10 +115,15 @@ struct mmc *mmc_create(const struct mmc_config *cfg, void *priv)
        struct mmc *mmc;
 
        /* quick validation */
-       if (cfg == NULL || cfg->ops == NULL || cfg->ops->send_cmd == NULL ||
-           cfg->f_min == 0 || cfg->f_max == 0 || cfg->b_max == 0)
+       if (cfg == NULL || cfg->f_min == 0 ||
+           cfg->f_max == 0 || cfg->b_max == 0)
                return NULL;
 
+#ifndef CONFIG_DM_MMC_OPS
+       if (cfg->ops == NULL || cfg->ops->send_cmd == NULL)
+               return NULL;
+#endif
+
        mmc = calloc(1, sizeof(*mmc));
        if (mmc == NULL)
                return NULL;