On 7/6/21 11:54 PM, Stephan Gerhold wrote: > Simplify the code a bit by using dev_read_addr_ptr() instead of > dev_read_addr(). This avoids having to cast explicitly to void*. > > Signed-off-by: Stephan Gerhold <[email protected]>
Reviewed-by: Jaehoon Chung <[email protected]> Best Regards, Jaehoon Chung > --- > > drivers/mmc/arm_pl180_mmci.c | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/drivers/mmc/arm_pl180_mmci.c b/drivers/mmc/arm_pl180_mmci.c > index 5d1ee64356..e632eed03f 100644 > --- a/drivers/mmc/arm_pl180_mmci.c > +++ b/drivers/mmc/arm_pl180_mmci.c > @@ -526,14 +526,11 @@ static const struct dm_mmc_ops arm_pl180_dm_mmc_ops = { > static int arm_pl180_mmc_of_to_plat(struct udevice *dev) > { > struct pl180_mmc_host *host = dev_get_priv(dev); > - fdt_addr_t addr; > > - addr = dev_read_addr(dev); > - if (addr == FDT_ADDR_T_NONE) > + host->base = dev_read_addr_ptr(dev); > + if (!host->base) > return -EINVAL; > > - host->base = (void *)addr; > - > return 0; > } > >
