On Aug 5, 2009, at 9:03 AM, Kumar Gala wrote: > If the size of a region equal to 4G it can't be represnted in a 32-bit > BAR so we should have marked that case as MEM64. > > Additionally bump the number of inbound windows up to 4 to handle the > fact that Freescale PPCs that have an implicit window for CCSRBAR. > > Signed-off-by: Kumar Gala <ga...@kernel.crashing.org> > --- > common/fdt_support.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-)
Can you look at picking this up as a fix for v2009.08. Its a pretty trivial change. - k > > diff --git a/common/fdt_support.c b/common/fdt_support.c > index fc077e8..89164a1 100644 > --- a/common/fdt_support.c > +++ b/common/fdt_support.c > @@ -625,7 +625,7 @@ int fdt_resize(void *blob) > } > > #ifdef CONFIG_PCI > -#define CONFIG_SYS_PCI_NR_INBOUND_WIN 3 > +#define CONFIG_SYS_PCI_NR_INBOUND_WIN 4 > > #define FDT_PCI_PREFETCH (0x40000000) > #define FDT_PCI_MEM32 (0x02000000) > @@ -655,7 +655,7 @@ int fdt_pci_dma_ranges(void *blob, int phb_off, > struct pci_controller *hose) { > size = (u64)hose->regions[r].size; > > dma_range[0] = 0; > - if (size > 0x100000000ull) > + if (size >= 0x100000000ull) > dma_range[0] |= FDT_PCI_MEM64; > else > dma_range[0] |= FDT_PCI_MEM32; > -- > 1.6.0.6 > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot