On Friday 21 February 2014 10:31:05 Jason Gunthorpe wrote: > On Fri, Feb 21, 2014 at 06:05:08PM +0100, Thomas Petazzoni wrote: > > > Now I have another question: our mvebu_pcie_align_resource() function > > makes sure that the base address of the BAR is aligned on its size, > > because it is a requirement of MBus windows. However, if you later > > split the BAR into multiple windows, will this continue to work out? > > No, you must align to (1 << log2_round_down(size)) - that will always > be the largest mbus window created and thus the highest starting > alignment requirement. Unless you allow reordering the two windows. If you want a 96MB window, you only need 32MB alignment because you can either put the actual 64MB window first if you have 64MB alignment, or you put the 32MB window first if you don't and then the following 64MB will be aligned. It gets more complicated if you want to allow a 72MB window (16MB+64MB), as that could either be 64MB aligned or start 16MB before the next multiple of 64MB. I don't think there is any reason why code anywhere should align the window to a multiple of the size though if the size is not power-of-two, such as aligning to multiples of 96MB. That wouldn't help anyone. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
