Hi, On 01/17/2018 02:16 AM, Marek Vasut wrote: > Drop the ad-hoc DT caps parsing in favor of common framework function. > > Signed-off-by: Marek Vasut <marek.vasut+rene...@gmail.com> > Cc: Jaehoon Chung <jh80.ch...@samsung.com> > Cc: Masahiro Yamada <yamada.masah...@socionext.com>
If Masahiro is ok, i will pick this patch and others to u-boot-mmc. Best Regards, Jaehoon Chung > --- > drivers/mmc/uniphier-sd.c | 23 +++++++---------------- > 1 file changed, 7 insertions(+), 16 deletions(-) > > diff --git a/drivers/mmc/uniphier-sd.c b/drivers/mmc/uniphier-sd.c > index 741f9dfd9c..552e37d852 100644 > --- a/drivers/mmc/uniphier-sd.c > +++ b/drivers/mmc/uniphier-sd.c > @@ -799,24 +799,15 @@ static int uniphier_sd_probe(struct udevice *dev) > return ret; > } > > - plat->cfg.name = dev->name; > - plat->cfg.host_caps = MMC_MODE_HS_52MHz | MMC_MODE_HS; > - > - switch (fdtdec_get_int(gd->fdt_blob, dev_of_offset(dev), "bus-width", > - 1)) { > - case 8: > - plat->cfg.host_caps |= MMC_MODE_8BIT; > - break; > - case 4: > - plat->cfg.host_caps |= MMC_MODE_4BIT; > - break; > - case 1: > - break; > - default: > - dev_err(dev, "Invalid \"bus-width\" value\n"); > - return -EINVAL; > + ret = mmc_of_parse(dev, &plat->cfg); > + if (ret < 0) { > + dev_err(dev, "failed to parse host caps\n"); > + return ret; > } > > + plat->cfg.name = dev->name; > + plat->cfg.host_caps |= MMC_MODE_HS_52MHz | MMC_MODE_HS; > + > if (quirks) { > priv->caps = quirks; > } else { > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot