[4/4,v3] package/kvm-unit-test: available for big.LITTLE arm cores too

Message ID e9a53f9e66ea0d124e8c4ab028f01b41d694910d.1499592591.git.yann.morin.1998@free.fr
State Accepted
Headers show

Commit Message

Yann E. MORIN July 9, 2017, 9:30 a.m.
Since it supports each variants individually, it should also supports
the big.LITTLE configurations.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Cyril Bur <cyrilbur@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/kvm-unit-tests/Config.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Thomas Petazzoni July 22, 2017, 9:29 p.m. | #1
Hello,

On Sun,  9 Jul 2017 11:30:01 +0200, Yann E. MORIN wrote:
> Since it supports each variants individually, it should also supports
> the big.LITTLE configurations.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Cyril Bur <cyrilbur@gmail.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/kvm-unit-tests/Config.in | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Applied to master, thanks.

Thomas
Yann E. MORIN July 22, 2017, 9:50 p.m. | #2
Thomas, All,

On 2017-07-22 23:29 +0200, Thomas Petazzoni spake thusly:
> On Sun,  9 Jul 2017 11:30:01 +0200, Yann E. MORIN wrote:
> > Since it supports each variants individually, it should also supports
> > the big.LITTLE configurations.
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Cyril Bur <cyrilbur@gmail.com>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > ---
> >  package/kvm-unit-tests/Config.in | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> Applied to master, thanks.

Alas, this is also causing kconfig issues:

    package/kvm-unit-tests/Config.in:5: syntax error
    package/kvm-unit-tests/Config.in:4: invalid option

I've sent a fix:
    https://patchwork.ozlabs.org/patch/792481/

Sorry about that. There realyl is something that I miossed in that
series... :-/

Regards,
Yann E. MORIN.

Patch

diff --git a/package/kvm-unit-tests/Config.in b/package/kvm-unit-tests/Config.in
index 3db10fc820..21eb2d51ff 100644
--- a/package/kvm-unit-tests/Config.in
+++ b/package/kvm-unit-tests/Config.in
@@ -2,7 +2,8 @@  config BR2_PACKAGE_KVM_UNIT_TESTS_ARCH_SUPPORTS
 	bool
 	# On ARM, it uses virtualization extensions
 	default y if BR2_cortex_a7 || BR2_cortex_a12 || \
-		BR2_cortex_a15 || BR2_cortex_a17
+		BR2_cortex_a15 || BR2_cortex_a15_a7 \
+		BR2_cortex_a17 || BR2_cortex_a17_a7
 	default y if BR2_i386 || BR2_x86_64
 	default y if BR2_powerpc64 || BR2_powerpc64le