Re: [Qemu-devel] [PATCH v2] PPC: E500: Implement reboot controller

[Scott Wood <scottwood@freescale.com>]

On Thu, 2 Jun 2011 20:37:30 +0200
Alexander Graf <agraf@suse.de> 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