[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] hw/ppc/ppc400_bamboo: Set dcr-base correctly when creating U

From: BALATON Zoltan
Subject: Re: [PATCH] hw/ppc/ppc400_bamboo: Set dcr-base correctly when creating UIC
Date: Mon, 11 Jan 2021 20:19:25 +0100 (CET)
User-agent: Alpine 2.03 (LMD 1266 2009-07-14)

On Mon, 11 Jan 2021, Peter Maydell wrote:
In commit 0270d74ef8862350 we switched from ppcuic_init() to directly
creating the UIC device, but I missed that the Bamboo's UIC has a
non-standard DCR base register value (0xc0 rather than the default
of 0x30). This made Linux panic early in the boot process.

Specify the correct dcr-base property value when creating the UIC.

Fixes: 0270d74ef8862350
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


With this fix Nathan's test case goes on to eventually hit
a QEMU assert:
qemu-system-ppc: ../../hw/pci/pci.c:255: pci_bus_change_irq_level: Assertion 
`irq_num >= 0' failed.
but it was doing that on 5.2 as well.
hw/ppc/ppc440_bamboo.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/hw/ppc/ppc440_bamboo.c b/hw/ppc/ppc440_bamboo.c
index b156bcb9990..2c7a578ba73 100644
--- a/hw/ppc/ppc440_bamboo.c
+++ b/hw/ppc/ppc440_bamboo.c
@@ -198,6 +198,7 @@ static void bamboo_init(MachineState *machine)
    uicdev = qdev_new(TYPE_PPC_UIC);
    uicsbd = SYS_BUS_DEVICE(uicdev);

+    qdev_prop_set_uint32(uicdev, "dcr-base", 0xc0);

This fixes Bamboo but not virtex and 405 which seem to have same problem as I've just shown in replies to those patches. So maybe this is better fixed by changing default value in ppc-uic.c to 0xc0 then. You say in commit message that 0xc0 is non-standard but most boards seem to use that than the default you have now. I don't know if there's a standard by the way or every CPU implementation just puts DCRs where they want.


    object_property_set_link(OBJECT(uicdev), "cpu", OBJECT(cpu),
    sysbus_realize_and_unref(uicsbd, &error_fatal);

reply via email to

[Prev in Thread] Current Thread [Next in Thread]