[Buildroot] [PATCH] qemu: Change CPU in qemu command for aarch64

Peter Korsgaard peter at korsgaard.com
Sun Jan 27 21:44:35 UTC 2019


>>>>> "Gerome" == Gerome Burlats <gerome.burlats at smile.fr> writes:

 > This commit changes to "cortex-a53" the CPU model in the qemu command
 > for aarch64 (since it is the default one for this architecture).

 > Signed-off-by: Gerome Burlats <gerome.burlats at smile.fr>
 > Cc: Romain Naour <romain.naour at smile.fr>
 > ---

 > cortex-a53 is the default cpu for aarch64, but some defconfigs, for
 > example raspberrypi3_64_defconfig, still have "BR2_cortex_a53=y"
 > (maybe at one point, this cpu was not the default).
 > Should this line stay in the defconfig for info, even if it is not used ?
 > If we generate the defconfig with "make savedefconfig", BR2_cortex_a53=y is indeed not added in the file.

Correct. We typically do add for clarity and robustness against changes
of the default value.

It is correct that savedefconfig will strip such redundant entries, but
as you can see from the comments, qemu_aarch64_virt_defconfig is edited
by hand.

Committed after extending the commit message and adding an explicit
BR2_cortex_a53=y to the defconfig, thanks.

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list