On Wed, Jan 12, 2022 at 04:14:36PM -0800, Khem Raj wrote: > [Please note: This e-mail is from an EXTERNAL e-mail address] > > > > On Wed, Jan 12, 2022 at 1:58 PM Richard Purdie < > [email protected]> wrote: > > On Wed, 2022-01-12 at 14:23 -0500, Jon Mason wrote: > > On Wed, Jan 12, 2022 at 4:11 AM Kevin Hao <[email protected]> > wrote: > > > > > > The crc extension is optional for the ARMv8.0 but is mandatory for the > > > cortexa72, so there is no reason not to enable it for the cortexa72 > > > tune. With this change, the cortexa72-crc seems redundant. But we > > > had better to keep it to be compatible with the BSP which already used > > > that tune. > > > > > > Signed-off-by: Kevin Hao <[email protected]> > > > > Acked-by: Jon Mason <[email protected]> > > > > From the GCC sources, it's hard coded for A34, A35, A53, A57, A72, > > A73, and almost all of the others listed. I can patch the rest. > > I did wonder if we should just drop the duplicate tune too. It will cause > an > error but it is an error that should be quickly/easily fixed... > > > I am good with that problem is only for bsp layers which support multiple > releases with a single branch but I guess that’s something less of a concern > for me
OK, I will send a add-on patch to drop the cortexa72-crc tune. Thanks, Kevin > > > > Cheers, > > Richard > > > > >
signature.asc
Description: PGP signature
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#160509): https://lists.openembedded.org/g/openembedded-core/message/160509 Mute This Topic: https://lists.openembedded.org/mt/88369540/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
