On 10/11/23 20:00, Bin Meng wrote: > dm_pci_map_bar() return a value of (void *) already, hence no need > to cast it again before assigning to host->ioaddr. > > Signed-off-by: Bin Meng <bm...@tinylab.org> > Reviewed-by: Simon Glass <s...@chromium.org>
Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com> Best Regards, Jaehoon Chung > --- > > drivers/mmc/pci_mmc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/pci_mmc.c b/drivers/mmc/pci_mmc.c > index 9fb7044029..4d163ccba0 100644 > --- a/drivers/mmc/pci_mmc.c > +++ b/drivers/mmc/pci_mmc.c > @@ -50,8 +50,8 @@ static int pci_mmc_probe(struct udevice *dev) > desc = mmc_get_blk_desc(&plat->mmc); > desc->removable = !(plat->cfg.host_caps & MMC_CAP_NONREMOVABLE); > > - host->ioaddr = (void *)dm_pci_map_bar(dev, PCI_BASE_ADDRESS_0, 0, 0, > PCI_REGION_TYPE, > - PCI_REGION_MEM); > + host->ioaddr = dm_pci_map_bar(dev, PCI_BASE_ADDRESS_0, 0, 0, > + PCI_REGION_TYPE, PCI_REGION_MEM); > host->name = dev->name; > host->cd_gpio = priv->cd_gpio; > host->mmc = &plat->mmc;
