> > diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c > > index 33dda00002..8bbeb62f2e 100644 > > --- a/drivers/pci/pci-uclass.c > > +++ b/drivers/pci/pci-uclass.c > > @@ -1533,8 +1533,8 @@ static phys_addr_t dm_pci_map_ea_virt(struct udevice > > *dev, int ea_off, > > return addr; > > } > > > > -static void *dm_pci_map_ea_bar(struct udevice *dev, int bar, int flags, > > - int ea_off, struct pci_child_plat *pdata) > > +static void *dm_pci_map_ea_bar(struct udevice *dev, int bar, int ea_off, > > + struct pci_child_plat *pdata) > > { > > int ea_cnt, i, entry_size; > > int bar_id = (bar - PCI_BASE_ADDRESS_0) >> 2; > > @@ -1577,13 +1577,13 @@ static void *dm_pci_map_ea_bar(struct udevice *dev, > > int bar, int flags, > > addr += dm_pci_map_ea_virt(dev, ea_off, pdata); > > > > /* size ignored for now */ > > - return map_physmem(addr, 0, flags); > > + return map_physmem(addr, 0, MAP_NOCACHE); > > } > > > > return 0; > > } > > > > -void *dm_pci_map_bar(struct udevice *dev, int bar, int flags) > > +void *dm_pci_map_bar(struct udevice *dev, int bar, unsigned long flags) > > Why is this change (int => unsigned long) necessary?
The flags are used with pci_region::flags which is an unsigned long and it seemed sensible to use the same type, size and sign, to avoid anything unexpected.