Hi Simon, On Thu, Aug 13, 2015 at 10:09 AM, Simon Glass <s...@chromium.org> wrote: > These functions are defined by macros so do not show up with grep. Add > a comment to help. > > Signed-off-by: Simon Glass <s...@chromium.org>
Reviewed-by: Bin Meng <bmeng...@gmail.com> One issue below: > --- > > include/pci.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/pci.h b/include/pci.h > index 488ff44..68df5cb 100644 > --- a/include/pci.h > +++ b/include/pci.h > @@ -653,6 +653,7 @@ extern pci_addr_t pci_hose_phys_to_bus(struct > pci_controller* hose, > #define pci_io_to_virt(dev, addr, len, map_flags) \ > pci_bus_to_virt((dev), (addr), PCI_REGION_IO, (len), (map_flags)) > > +/* For driver model these are defined in micros in pci_compat.c */ typo: micros > extern int pci_hose_read_config_byte(struct pci_controller *hose, > pci_dev_t dev, int where, u8 *val); > extern int pci_hose_read_config_word(struct pci_controller *hose, > -- Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot