[Buildroot] [PATCH v3] board: Add PowerPC mac99

[Joel Stanley]

On Thu, 1 Aug 2019 at 15:54, Peter Korsgaard <peter at korsgaard.com> wrote:
>
> >>>>> "Joel" == Joel Stanley <joel at jms.id.au> writes:
>
>  > PowerPC kernel developers were after a userspace for testing 32-bit
>  > powerpc kernels. This machine both suits that requirement and has
>  > support in qemu. It's also a fairly common piece of 32-bit ppc hardware.
>
>  > Signed-off-by: Joel Stanley <joel at jms.id.au>
>  > Tested-by: Daniel Axtens <dja at axtens.net>
>  > ---
>  > v3: Add ROOTFS_CPIO output as well as ext
>  > v2: Use default output directory for qemu command line in readme
>
>  > +++ b/configs/qemu_ppc_mac99_defconfig
>  > @@ -0,0 +1,11 @@
>  > +BR2_powerpc=y
>  > +BR2_powerpc_7400=y
>  > +BR2_TARGET_GENERIC_GETTY_PORT="ttyS0"
>  > +BR2_SYSTEM_DHCP="eth0"
>  > +BR2_LINUX_KERNEL=y
>  > +BR2_LINUX_KERNEL_DEFCONFIG="pmac32"
>  > +BR2_LINUX_KERNEL_CONFIG_FRAGMENT_FILES="board/qemu/ppc-mac99/linux.fragment"
>  > +BR2_LINUX_KERNEL_VMLINUX=y
>
> You forgot to specify the version of the Linux kernel /
> kernel-headers. Committed with those set to 5.2.4, thanks.

I didn't forget, I omitted them so the config would use the latest default.

Is there a reason we specify a fixed version?

> With that said, we now have 4 32bit ppc qemu defconfigs which sounds
> like a lot to me. Are they all still interesting / useful?

I am not familiar with them. Michael?

They all target different CPUs which means there is no overlap at least:

$ grep -r BR2_powerpc_ configs/qemu_ppc_*
configs/qemu_ppc_g3beige_defconfig:BR2_powerpc_750=y
configs/qemu_ppc_mac99_defconfig:BR2_powerpc_7400=y
configs/qemu_ppc_mpc8544ds_defconfig:BR2_powerpc_8548=y
configs/qemu_ppc_virtex_ml507_defconfig:BR2_powerpc_440=y

Cheers,

Joel