Re: [Qemu-arm] [PATCH 0/3] aspeed: cleanups and extenstions

[Peter Maydell]

On Mon, 29 Apr 2019 at 13:07, Peter Maydell <address@hidden> wrote:
>
> On Thu, 11 Apr 2019 at 17:10, Cédric Le Goater <address@hidden> wrote:
> >
> > Hello,
> >
> > Here is a series adding a couple of cleanups to the Aspeed SoCs to
> > prepare ground for extensions and new SoCs.
> >
> > Thanks,
> >
> > C.
> >
> > Cédric Le Goater (3):
> >   aspeed: add a per SoC mapping for the interrupt space
> >   aspeed: add a per SoC mapping for the memory space
> >   aspeed: use sysbus_init_child_obj() to initialize children
> >
>
>
>
> Applied to target-arm.next, thanks.

...spoke too soon. Clang doesn't like this:

/home/petmay01/linaro/qemu-from-laptop/qemu/hw/arm/aspeed_soc.c:155:25:
error: initializer element is not a
      compile-time constant
        .sdram_base   = aspeed_soc_ast2400_memmap[ASPEED_SDRAM],
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
/home/petmay01/linaro/qemu-from-laptop/qemu/rules.mak:69: recipe for
target 'hw/arm/aspeed_soc.o' failed

thanks
-- PMM