On Tue, Dec 7, 2021 at 9:00 AM Jon Mason <[email protected]> wrote: > > From: Jagadeesh Krishnanjanappa <[email protected]> > > The cryptographic unit is optional for the Cortex-A72, but it was > included by default previously. This breaks building systems that > lack this functionality when using tune-cortexa72.inc. > > To correct this, add a crypto entry in the tune file. Since CRC is > optional for ARMv8.0, do the same thing while we're at it. > > For platforms that had been happily using tune-cortexa72.inc, a slight > degradation of performance will occur using the default. To correct > this, simply add: > DEFAULTTUNE = "cortexa72-crc-crypto"
sounds good to me. meta-rockpi might need to pay attention here once merged. > > Signed-off-by: Jagadeesh Krishnanjanappa <[email protected]> > Signed-off-by: Jon Mason <[email protected]> (rewording commit message) > --- > .../include/arm/armv8a/tune-cortexa72.inc | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) > > diff --git a/meta/conf/machine/include/arm/armv8a/tune-cortexa72.inc > b/meta/conf/machine/include/arm/armv8a/tune-cortexa72.inc > index 2a510bd45bdf..3651cf6e802b 100644 > --- a/meta/conf/machine/include/arm/armv8a/tune-cortexa72.inc > +++ b/meta/conf/machine/include/arm/armv8a/tune-cortexa72.inc > @@ -6,8 +6,16 @@ TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', > 'cortexa72', ' -mcpu=corte > require conf/machine/include/arm/arch-armv8a.inc > > # Little Endian base configs > -AVAILTUNES += "cortexa72" > +AVAILTUNES += "cortexa72 cortexa72-crc cortexa72-crc-crypto" > ARMPKGARCH:tune-cortexa72 = "cortexa72" > -TUNE_FEATURES:tune-cortexa72 = > "${TUNE_FEATURES:tune-armv8a-crc-crypto} cortexa72" > -PACKAGE_EXTRA_ARCHS:tune-cortexa72 = > "${PACKAGE_EXTRA_ARCHS:tune-armv8a-crc-crypto} cortexa72" > -BASE_LIB:tune-cortexa72 = "lib64" > +ARMPKGARCH:tune-cortexa72-crc = "cortexa72" > +ARMPKGARCH:tune-cortexa72-crc-crypto = "cortexa72" > +TUNE_FEATURES:tune-cortexa72 = "${TUNE_FEATURES:tune-armv8a} > cortexa72" > +TUNE_FEATURES:tune-cortexa72-crc = "${TUNE_FEATURES:tune-cortexa72} crc" > +TUNE_FEATURES:tune-cortexa72-crc-crypto = "${TUNE_FEATURES:tune-cortexa72} > crc crypto" > +PACKAGE_EXTRA_ARCHS:tune-cortexa72 = > "${PACKAGE_EXTRA_ARCHS:tune-armv8} cortexa72" > +PACKAGE_EXTRA_ARCHS:tune-cortexa72-crc = > "${PACKAGE_EXTRA_ARCHS:tune-armv8a-crc} cortexa72 cortexa72-crc" > +PACKAGE_EXTRA_ARCHS:tune-cortexa72-crc-crypto = > "${PACKAGE_EXTRA_ARCHS:tune-armv8a-crc-crypto} cortexa72 cortexa72-crc > cortexa72-crc-crypto" > +BASE_LIB:tune-cortexa72 = "lib64" > +BASE_LIB:tune-cortexa72-crc = "lib64" > +BASE_LIB:tune-cortexa72-crc-crypto = "lib64" > -- > 2.20.1 > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#159325): https://lists.openembedded.org/g/openembedded-core/message/159325 Mute This Topic: https://lists.openembedded.org/mt/87569173/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
