Re: [Qemu-devel] [PATCH v3 2/2] ppc: Add aCube Sam460ex board

[BALATON Zoltan]

On Tue, 20 Feb 2018, David Gibson wrote:
> On Mon, Feb 19, 2018 at 11:34:25AM +0100, BALATON Zoltan wrote:
> 1;5002;0c> Add emulation of aCube Sam460ex board based on AMCC 460EX embedded 
> SoC.
> > This is not a complete implementation yet with a lot of components
> > still missing but enough for the U-Boot firmware to start and to boot
> > a Linux kernel or AROS.
> >
> > Signed-off-by: François Revol <address@hidden>
> > Signed-off-by: BALATON Zoltan <address@hidden>
>
> I'm not thrilled with having a machine type that can't be used without
> extracting a bootloader image from somewhere, but from the other
> thread that's not easy to fix.

This is not meant to be like that and I'd like to also submit the u-boot 
image once we figure out how to do it. Especially because the boot loader 
one can extract from the updater is buggy but here's a fixed binary that 
works here: http://zero.eik.bme.hu/~balaton/qemu/

U-Boot is GPL so we would need to include the source as well (the README 
in the link above contain the source URL and the patches needed are there 
as well). If there's no easy way to host it in the QEMU repo I could put 
it in my own git repo and submit a patch to use that as a submodule if 
that's acceptable. It seems there are some firmware images that reference 
external git repos already but not sure what's the preferred way.

> Since we have dtc in a submodule, there's really no reason we should
> be including .dtb files as well as .dts files, we should build them
> ourselves.  But fixing that isn't really in scope for this patch.

I think there are already make rules to build a dtb from dts so it may be 
possible to build these if we can add them as a dependency somewhere. 
However a normal make does not build dtc for me from the submodule so I'm 
not sure it would work.

> So, applied.

Thank you,
BALATON Zoltan