Matthew, On Tue, Apr 08, 2014 at 01:31:48PM +0100, Matthew Minter wrote: > To add to my previous point, I would consider releasing these patches > as soon as possible as it seems that it can easily cause panics on a > range of boards. Failing that perhaps it is at least worth releasing a > hotfix which will cause PCI probing to fail should the window not be a > power of two? Thus it will fail early with a warning instead of > potentially causing a kernel crash. Have you tested the whole patch or just the one enforcing the power of 2 ? It would be interesting to know if the one with SZ_4M is needed for you or not (I guess not). I suspect that just this single-liner will work as well, as it does for me. Thanks, Willy diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c index 0e79665..552ab73 100644 --- a/drivers/pci/host/pci-mvebu.c +++ b/drivers/pci/host/pci-mvebu.c @@ -363,6 +363,7 @@ static void mvebu_pcie_handle_membase_change(struct mvebu_pcie_port *port) port->memwin_size = (((port->bridge.memlimit & 0xFFF0) << 16) | 0xFFFFF) - port->memwin_base; + port->memwin_size = roundup_pow_of_two(port->memwin_size); mvebu_mbus_add_window_by_id(port->mem_target, port->mem_attr, port->memwin_base, port->memwin_size); _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-arm-kernel