[U-Boot] [PATCH 2/3] mmc: omap_mmc: Remove invert references with gpio xlate available
Jean-Jacques Hiblot
jjhiblot at ti.com
Wed Sep 5 12:34:15 UTC 2018
Adam,
On 05/09/2018 11:35, Adam Ford wrote:
> With omap_gpio now translating GPIO_ACTIVE_LOW, any boards
> using the 'invert' option will no longer need to do this. This
> patch removes the support for 'invert' from the MMC driver.
>
> Signed-off-by: Adam Ford <aford173 at gmail.com>
>
> diff --git a/drivers/mmc/omap_hsmmc.c b/drivers/mmc/omap_hsmmc.c
> index 8ab56d247d..e9786ec5bb 100644
> --- a/drivers/mmc/omap_hsmmc.c
> +++ b/drivers/mmc/omap_hsmmc.c
> @@ -83,7 +83,6 @@ struct omap_hsmmc_data {
> #if CONFIG_IS_ENABLED(DM_MMC)
> struct gpio_desc cd_gpio; /* Change Detect GPIO */
> struct gpio_desc wp_gpio; /* Write Protect GPIO */
> - bool cd_inverted;
> #else
> int cd_gpio;
> int wp_gpio;
> @@ -1377,8 +1376,6 @@ static int omap_hsmmc_getcd(struct udevice *dev)
> if (value < 0)
> return 1;
>
> - if (priv->cd_inverted)
> - return !value;
> return value;
> }
>
> @@ -1860,10 +1857,6 @@ static int omap_hsmmc_ofdata_to_platdata(struct udevice *dev)
> }
> #endif
>
> -#ifdef OMAP_HSMMC_USE_GPIO
> - plat->cd_inverted = fdtdec_get_bool(fdt, node, "cd-inverted");
> -#endif
> -
> return 0;
> }
> #endif
> @@ -1892,9 +1885,6 @@ static int omap_hsmmc_probe(struct udevice *dev)
> priv->base_addr = plat->base_addr;
> priv->controller_flags = plat->controller_flags;
> priv->hw_rev = plat->hw_rev;
> -#ifdef OMAP_HSMMC_USE_GPIO
> - priv->cd_inverted = plat->cd_inverted;
Could you remove it also from struct omap_hsmmc_plat since it is not
used anymore ?
JJ
> -#endif
>
> #ifdef CONFIG_BLK
> mmc = plat->mmc;
More information about the U-Boot
mailing list