On 2/15/21 3:51 AM, Peter Maydell wrote:
@@ -544,7 +546,7 @@ static void mps2tz_common_init(MachineState *machine)
* + wire up the PPC's control lines to the IoTKit object
*/
- const PPCInfo ppcs[] = { {
+ const PPCInfo an505_ppcs[] = { {
.name = "apb_ppcexp0",
.ports = {
{ "ssram-0", make_mpc, &mms->ssram_mpc[0], 0x58007000, 0x1000
},
Existing nit, but about to be exacerbated by another array. I'm not a fan of
large, initialized, on-stack arrays. Do you really need the pointer into the
runtime variable mms? Perhaps arrange for it to be an offsetof? Then the
whole thing can be static const.
That said, the change in this patch is ok.
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
r~