Hi, On Thu, 15 Sep 2022, at 13:50, Andrei Gherzan wrote: > From: Andrei Gherzan <[email protected]> > > Grub tries to force the compilation in softfp mode for some known target > CPU. That is because it doesn't use floats or doubles and there are > known limitations of using a configuration with hardfp. There are though > target CPUs that grub won't know how to configure failing with something > similar to: > > | checking for options to get soft-float... no > | configure: error: could not force soft-float > > This change relaxes a bit the COMPATIBLE_HOST restriction to only apply > for hardfp configuration even for arm7ve/a. > > Signed-off-by: Andrei Gherzan <[email protected]> > --- > meta/recipes-bsp/grub/grub2.inc | 15 ++++++++++++--- > 1 file changed, 12 insertions(+), 3 deletions(-) > > diff --git a/meta/recipes-bsp/grub/grub2.inc b/meta/recipes-bsp/grub/grub2.inc > index 47ea561002..6c9952d2d7 100644 > --- a/meta/recipes-bsp/grub/grub2.inc > +++ b/meta/recipes-bsp/grub/grub2.inc > @@ -43,9 +43,18 @@ CVE_CHECK_IGNORE += "CVE-2021-46705" > > DEPENDS = "flex-native bison-native gettext-native" > > -COMPATIBLE_HOST = > '(x86_64.*|i.86.*|arm.*|aarch64.*|riscv.*)-(linux.*|freebsd.*)' > -COMPATIBLE_HOST:armv7a = 'null' > -COMPATIBLE_HOST:armv7ve = 'null' > +GRUB_COMPATIBLE_HOST = > '(x86_64.*|i.86.*|arm.*|aarch64.*|riscv.*)-(linux.*|freebsd.*)' > +COMPATIBLE_HOST = "${GRUB_COMPATIBLE_HOST}" > +# Grub doesn't support hard float toolchain and won't be able to > forcefully > +# disable it on some of the target CPUs. See 'configure.ac' for > +# supported/unsupported CPUs in hardfp. > +COMPATIBLE_HOST:armv7a = "${@'null' if d.getVar('TUNE_CCARGS_MFLOAT') > == 'hardfp' else d.getVar('GRUB_COMPATIBLE_HOST')}" > +COMPATIBLE_HOST:armv7ve = "${@'null' if d.getVar('TUNE_CCARGS_MFLOAT') > == 'hardfp' else d.getVar('GRUB_COMPATIBLE_HOST')}" > + > +# Grub doesn't use float or doubles and tries to disable forcefully > floating > +# point at configure time. That dynamic flags configuration doesn't > always work > +# (for example on arm 32bit) so force the compilation in softfp. > +TUNE_CCARGS_MFLOAT:class-target = "softfp"
Please ignore this v1 patch as the above was not supposed to be part of the patch. I've sent a cleaned up v2: https://lists.openembedded.org/g/openembedded-core/message/170695 Thanks, Andrei
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#170696): https://lists.openembedded.org/g/openembedded-core/message/170696 Mute This Topic: https://lists.openembedded.org/mt/93699086/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
