diff mbox

config/raspberrypi3: use correct CPU

Message ID 20170501091104.3183-1-yann.morin.1998@free.fr
State Accepted
Headers show

Commit Message

Yann E. MORIN May 1, 2017, 9:11 a.m. UTC
We so far were using a Cortex-A7 as the CPU for the rpi3, as this was
the closest we had available, in 32-bit mode, to the real CPU used in
the rpi3.

But now we can also use the Cortex-A53 in 32-bit mode, so we use that,
which makes for an optimised build (A53 has an improved instruction set,
as compared to the A7).

Also drop the DTB overlay option, it defaults to 'y'.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Martin Bark <martin@barkynet.com>
Cc: Peter Seiderer <ps.report@gmx.net>

---
Martin, Peter, you are the last ones to have touched this defconfig, and
I lack a rpi3. Would you be kind enough to test this on your hw, please?
---
 configs/raspberrypi3_defconfig | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Ricardo Martincoski May 1, 2017, 10:52 p.m. UTC | #1
Hello,

On Mon, May 01, 2017 at 06:11 AM, Yann E. MORIN wrote:

> We so far were using a Cortex-A7 as the CPU for the rpi3, as this was
> the closest we had available, in 32-bit mode, to the real CPU used in
> the rpi3.
> 
> But now we can also use the Cortex-A53 in 32-bit mode, so we use that,
> which makes for an optimised build (A53 has an improved instruction set,
> as compared to the A7).
> 
> Also drop the DTB overlay option, it defaults to 'y'.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Tested-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
[boot-tested: I can login using the HDMI console]

but see comment below...

[snip]
> +++ b/configs/raspberrypi3_defconfig
> @@ -1,6 +1,5 @@
>  BR2_arm=y
> -BR2_cortex_a7=y
> -BR2_ARM_EABIHF=y
> +BR2_cortex_a53=y

This patch is very similar to:
http://patchwork.ozlabs.org/patch/751142/

I applied 751142 and 757022 to separate git trees above 289137ed61.
The resulting .config are the same (except for the sha1, of course).

Regards,
Ricardo
Thomas Petazzoni May 2, 2017, 8:40 p.m. UTC | #2
Hello,

On Mon,  1 May 2017 11:11:04 +0200, Yann E. MORIN wrote:
> We so far were using a Cortex-A7 as the CPU for the rpi3, as this was
> the closest we had available, in 32-bit mode, to the real CPU used in
> the rpi3.
> 
> But now we can also use the Cortex-A53 in 32-bit mode, so we use that,
> which makes for an optimised build (A53 has an improved instruction set,
> as compared to the A7).
> 
> Also drop the DTB overlay option, it defaults to 'y'.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Martin Bark <martin@barkynet.com>
> Cc: Peter Seiderer <ps.report@gmx.net>
> 
> ---
> Martin, Peter, you are the last ones to have touched this defconfig, and
> I lack a rpi3. Would you be kind enough to test this on your hw, please?
> ---
>  configs/raspberrypi3_defconfig | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)

Applied to master, thanks.

Thomas
diff mbox

Patch

diff --git a/configs/raspberrypi3_defconfig b/configs/raspberrypi3_defconfig
index 1b2134e0bb..f5bfebcb83 100644
--- a/configs/raspberrypi3_defconfig
+++ b/configs/raspberrypi3_defconfig
@@ -1,6 +1,5 @@ 
 BR2_arm=y
-BR2_cortex_a7=y
-BR2_ARM_EABIHF=y
+BR2_cortex_a53=y
 BR2_ARM_FPU_NEON_VFPV4=y
 
 BR2_TOOLCHAIN_BUILDROOT_CXX=y
@@ -21,7 +20,6 @@  BR2_LINUX_KERNEL_DTS_SUPPORT=y
 BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2710-rpi-3-b bcm2710-rpi-cm3"
 
 BR2_PACKAGE_RPI_FIRMWARE=y
-BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTB_OVERLAYS=y
 
 # Required tools to create the SD image
 BR2_PACKAGE_HOST_DOSFSTOOLS=y