diff mbox

[U-Boot] fsl_esdhc: fix warning if CONFIG_DM_REGULATOR is not set

Message ID 1502882178-31172-1-git-send-email-sbabic@denx.de
State Rejected
Delegated to: Stefano Babic
Headers show

Commit Message

Stefano Babic Aug. 16, 2017, 11:16 a.m. UTC
Warning appears when i.MX6 (icore) boards are built.

Signed-off-by: Stefano Babic <sbabic@denx.de>
---
 drivers/mmc/fsl_esdhc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jaehoon Chung Aug. 18, 2017, 7:06 a.m. UTC | #1
Hi Stefano.

On 08/16/2017 08:16 PM, Stefano Babic wrote:
> Warning appears when i.MX6 (icore) boards are built.

ommit 9bb272e90acf7182311dc524df69b125eac92a53
Author: York Sun <york.sun@nxp.com>
Date:   Tue Aug 8 15:45:13 2017 -0700

    driver: mmc: fsl_esdhc: Fix compiling warning

    Commit 4483b7eb added variable vqmmc_dev but only uses it under
    CONFIG_DM_REGULATOR. Add the same macro to variable declaration to
    get rid of compiling warning.

    Signed-off-by: York Sun <york.sun@nxp.com>

It seems that already fixed this issue.

Best Regards,
Jaehoon Chung

> 
> Signed-off-by: Stefano Babic <sbabic@denx.de>
> ---
>  drivers/mmc/fsl_esdhc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c
> index 3abd2d3..b98da69 100644
> --- a/drivers/mmc/fsl_esdhc.c
> +++ b/drivers/mmc/fsl_esdhc.c
> @@ -968,7 +968,6 @@ static int fsl_esdhc_probe(struct udevice *dev)
>  	struct fsl_esdhc_priv *priv = dev_get_priv(dev);
>  	const void *fdt = gd->fdt_blob;
>  	int node = dev_of_offset(dev);
> -	struct udevice *vqmmc_dev;
>  	fdt_addr_t addr;
>  	unsigned int val;
>  	int ret;
> @@ -1014,6 +1013,7 @@ static int fsl_esdhc_probe(struct udevice *dev)
>  	 * If emmc I/O has a fixed voltage at 1.8V, this must be provided,
>  	 * otherwise, emmc will work abnormally.
>  	 */
> +	struct udevice *vqmmc_dev;
>  	ret = device_get_supply_regulator(dev, "vqmmc-supply", &vqmmc_dev);
>  	if (ret) {
>  		dev_dbg(dev, "no vqmmc-supply\n");
>
diff mbox

Patch

diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c
index 3abd2d3..b98da69 100644
--- a/drivers/mmc/fsl_esdhc.c
+++ b/drivers/mmc/fsl_esdhc.c
@@ -968,7 +968,6 @@  static int fsl_esdhc_probe(struct udevice *dev)
 	struct fsl_esdhc_priv *priv = dev_get_priv(dev);
 	const void *fdt = gd->fdt_blob;
 	int node = dev_of_offset(dev);
-	struct udevice *vqmmc_dev;
 	fdt_addr_t addr;
 	unsigned int val;
 	int ret;
@@ -1014,6 +1013,7 @@  static int fsl_esdhc_probe(struct udevice *dev)
 	 * If emmc I/O has a fixed voltage at 1.8V, this must be provided,
 	 * otherwise, emmc will work abnormally.
 	 */
+	struct udevice *vqmmc_dev;
 	ret = device_get_supply_regulator(dev, "vqmmc-supply", &vqmmc_dev);
 	if (ret) {
 		dev_dbg(dev, "no vqmmc-supply\n");