[meta-xilinx] [PATCH v2] tune-zynqmp.inc: Add default tune for multilib lib32 builds

[Nathan Rossi]

On 7 November 2017 at 10:43, Manjukumar Matha
<manjukumar.harthikote-matha at xilinx.com> wrote:
> On ZU+ machines, we set the default tune for the lib32 multilib
> configuration.This allows users to select multilib by simply requiring
> conf/multilib.conf in their build setups.
>
> Signed-off-by: Cyril Chemparathy <cyril.chemparathy at xilinx.com>
> Tested-by: Bhargava Sreekantappa Gayathri <bhargava.sreekantappa-gayathri at xilinx.com>
> Signed-off-by: Manjukumar Matha <manjukumar.harthikote-matha at xilinx.com>
> ---
>  conf/machine/include/tune-zynqmp.inc | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/conf/machine/include/tune-zynqmp.inc b/conf/machine/include/tune-zynqmp.inc
> index 79c6296..c4e0853 100644
> --- a/conf/machine/include/tune-zynqmp.inc
> +++ b/conf/machine/include/tune-zynqmp.inc
> @@ -6,3 +6,6 @@ require conf/machine/include/soc-family.inc
>
>  # Linux Configuration
>  KERNEL_IMAGETYPE ?= "Image"
> +
> +# Support multilib on zynqmp
> +DEFAULTTUNE_virtclass-multilib-lib32_zynqmp ?= "armv7vethf-neon-vfpv4"

You should drop the "_zynqmp" override here, since this tune is only
included by zynqmp targets it is unnecessary. But more importantly
with the override it makes setting a non-default value for the
multilib defaulttune harder (since you would need to set it with the
override to effect the value).

Regards,
Nathan

> --
> 2.7.4
>
> --
> _______________________________________________
> meta-xilinx mailing list
> meta-xilinx at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-xilinx