On 3 December 2013 07:02, Peter Crosthwaite <peter.crosthwa...@xilinx.com> wrote: > Fix the CBAR initialisation by using the newly defined static property. > CBAR is now set before realization, so the intended value is now > actually used. > > So I have kind of tested this. I booted an ARM kernel on Highbank with > the stock Highbank DTB. It doesn't boot (and I will be doing something > wrong), but before this patch I got this:
Highbank insists on you specifying 4GB RAM to QEMU, IIRC; maybe it's that? > ------------[ cut here ]------------ > WARNING: CPU: 0 PID: 0 at > /workspaces/pcrost/public/linux2.git/arch/arm/mm/ioremap.c:301 > __arm_ioremap_pfn_caller+0x180/0x198() > CPU: 0 PID: 0 Comm: swapper/0 Tainted: G W > 3.13.0-rc1-next-20131126-dirty #2 > [<c0015164>] (unwind_backtrace) from [<c00118c0>] (show_stack+0x10/0x14) > [<c00118c0>] (show_stack) from [<c02bd5fc>] (dump_stack+0x78/0x90) > [<c02bd5fc>] (dump_stack) from [<c001f110>] (warn_slowpath_common+0x68/0x84) > [<c001f110>] (warn_slowpath_common) from [<c001f1f4>] > (warn_slowpath_null+0x1c/0x24) > [<c001f1f4>] (warn_slowpath_null) from [<c0017c6c>] > (__arm_ioremap_pfn_caller+0x180/0x198) > [<c0017c6c>] (__arm_ioremap_pfn_caller) from [<c0017cd8>] > (__arm_ioremap_caller+0x54/0x5c) > [<c0017cd8>] (__arm_ioremap_caller) from [<c0017d10>] > (__arm_ioremap+0x18/0x1c) > [<c0017d10>] (__arm_ioremap) from [<c03913c0>] (highbank_init_irq+0x34/0x8c) > [<c03913c0>] (highbank_init_irq) from [<c038c228>] (init_IRQ+0x28/0x2c) > [<c038c228>] (init_IRQ) from [<c03899ec>] (start_kernel+0x234/0x398) > [<c03899ec>] (start_kernel) from [<00008074>] (0x8074) > ---[ end trace 3406ff24bd97382f ]--- > > Which disappears with this patch. > > Signed-off-by: Peter Crosthwaite <peter.crosthwa...@xilinx.com> Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> -- PMM