[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] PPC: E500: Implement reboot controller
From: |
Scott Wood |
Subject: |
Re: [Qemu-devel] [PATCH v2] PPC: E500: Implement reboot controller |
Date: |
Fri, 3 Jun 2011 18:42:52 -0500 |
On Thu, 2 Jun 2011 20:37:30 +0200
Alexander Graf <address@hidden> wrote:
> + case MPC8544_GUTS_ADDR_PVR:
> + value = env->spr[SPR_PVR];
> + break;
> + case MPC8544_GUTS_ADDR_SVR:
> + value = env->spr[SPR_E500_SVR];
> + break;
Heh, I didn't even realize these were in there -- I was just thinking of
the guest using the SPR version of SVR to decide how to program devices
like guts or tsec, and the conflicting demands that might place on that
value if you have both directly-assigned devices and emulated devices that
target a specific chip.
I can see where that produced confusion. :-)
> + default:
> + fprintf(stderr, "Unknown register read: %x = %x\n", (int)addr,
> value);
> + break;
> + }
Should say something like "guts" or __func__ in there somewhere.
Especially if we're going to throw away the upper 4 bits of the address. :-)
-Scott