<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 13, 2019, 7:21 PM Andre McCurdy <<a href="mailto:armccurdy@gmail.com" target="_blank" rel="noreferrer">armccurdy@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Dec 13, 2019 at 4:55 PM Joshua Watt <<a href="mailto:jpewhacker@gmail.com" rel="noreferrer noreferrer" target="_blank">jpewhacker@gmail.com</a>> wrote:<br>
><br>
> Adds tunes for Cortex-A72 Cortex-A53 big.LITTLE SoCs (with and without<br>
> crypto extensions), e.g. Rockchip RK3399<br>
><br>
> Signed-off-by: Joshua Watt <<a href="mailto:JPEWhacker@gmail.com" rel="noreferrer noreferrer" target="_blank">JPEWhacker@gmail.com</a>><br>
> ---<br>
> .../include/tune-cortexa72-cortexa53.inc | 24 +++++++++++++++++++<br>
> 1 file changed, 24 insertions(+)<br>
> create mode 100644 meta/conf/machine/include/tune-cortexa72-cortexa53.inc<br>
><br>
> diff --git a/meta/conf/machine/include/tune-cortexa72-cortexa53.inc b/meta/conf/machine/include/tune-cortexa72-cortexa53.inc<br>
> new file mode 100644<br>
> index 00000000000..6bf3b83abe9<br>
> --- /dev/null<br>
> +++ b/meta/conf/machine/include/tune-cortexa72-cortexa53.inc<br>
> @@ -0,0 +1,24 @@<br>
> +DEFAULTTUNE ?= "cortexa72-cortexa53"<br>
> +<br>
> +require conf/machine/include/arm/arch-armv8a.inc<br>
> +<br>
> +TUNEVALID[cortexa72-cortexa53] = "Enable big.LITTLE Cortex-A72.Cortex-A53 specific processor optimizations"<br>
> +TUNE_CCARGS .= "${@bb.utils.contains("TUNE_FEATURES", "aarch64", " -march=armv8-a", "" ,d)}"<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I also think this is redundant because arch-armv8a.inc does it already</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> +<br>
> +TUNECONFLICTS[aarch64] = "armv4 armv5 armv6 armv7 armv7a"<br>
> +<br>
> +MACHINEOVERRIDES =. "${@bb.utils.contains("TUNE_FEATURES", "cortexa72-cortexa53", "cortexa72-cortexa53:", "" ,d)}"<br>
> +<br>
> +TUNE_CCARGS .= "${@bb.utils.contains("TUNE_FEATURES", "cortexa72-cortexa53", " -mtune=cortex-a72.cortex-a53", "", d)}"<br>
> +<br>
> +# Little Endian base configs<br>
> +AVAILTUNES += "cortexa72-cortexa53 cortexa72-cortexa53-crypto"<br>
> +ARMPKGARCH_tune-cortexa72-cortexa53 = "cortexa72-cortexa53"<br>
> +ARMPKGARCH_tune-cortexa72-cortexa53-crypto = "cortexa72-cortexa53"<br>
> +TUNE_FEATURES_tune-cortexa72-cortexa53 = "${TUNE_FEATURES_tune-armv8a}"<br>
> +TUNE_FEATURES_tune-cortexa72-cortexa53-crypto = "${TUNE_FEATURES_tune-armv8a-crypto}"<br>
> +PACKAGE_EXTRA_ARCHS_tune-cortexa72-cortexa53 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} cortexa72-cortexa53"<br>
> +PACKAGE_EXTRA_ARCHS_tune-cortexa72-cortexa53-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crypto} cortexa72-cortexa53-crypto"<br>
<br>
Shouldn't cortexa72-cortexa53-crypto also be able to use<br>
cortexa72-cortexa53 packages?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yep. I'll send a v3</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +BASE_LIB_tune-cortexa72-cortexa53 = "lib64"<br>
> +BASE_LIB_tune-cortexa72-cortexa53-crypto = "lib64"<br>
> +<br>
> --<br>
> 2.23.0<br>
><br>
> --<br>
> _______________________________________________<br>
> Openembedded-core mailing list<br>
> <a href="mailto:Openembedded-core@lists.openembedded.org" rel="noreferrer noreferrer" target="_blank">Openembedded-core@lists.openembedded.org</a><br>
> <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
</blockquote></div></div></div>