Russell, Thanks for your quick feedback! On Tue, 29 Jan 2013 15:15:01 +0000, Russell King - ARM Linux wrote: > Given the description, I'd feel much happier with this if we specified > the alignment numerically rather than allowing "some random code" to do > something with the passed values. So, you'd prefer to have two new members added in the hw_pci structure to give the alignment requirements for I/O regions and memory regions? Something like: struct hw_pci { [...] unsigned long io_align; unsigned long mem_align; }; If that's fine with you, I'll go ahead and change the implementation in this direction. As long as I can express my special 64 KB alignment requirement for I/O regions, I'm just fine :-) Thanks again, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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