On 27 May 2017 at 08:18, Manjukumar Matha
<[email protected]> 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 <[email protected]>
> Signed-off-by: Manjukumar Matha <[email protected]>
> ---
>  conf/machine/include/machine-xilinx-default.inc | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/conf/machine/include/machine-xilinx-default.inc 
> b/conf/machine/include/machine-xilinx-default.inc
> index 24d78bc..7b7a549 100644
> --- a/conf/machine/include/machine-xilinx-default.inc
> +++ b/conf/machine/include/machine-xilinx-default.inc
> @@ -42,3 +42,6 @@ XSERVER ?= " \
>
>  XSERVER_EXT ?= ""
>  XSERVER_EXT_zynqmp ?= "xf86-video-armsoc"
> +
> +# Support multilib on zynqmp
> +DEFAULTTUNE_virtclass-multilib-lib32_zynqmp = "armv7vehf-neon-vfpv4"

I think this would be better set in tune-zynqmp.inc (alongside the
DEFAULTTUNE) as a ?=. That way it is not forced with an override and
can be easily modified since MULTILIB tunes are really distro level
config.

Also since the A53 does T32 instructions you should allow thumb with
the armv7vethf-... variant.

Regards,
Nathan
-- 
_______________________________________________
meta-xilinx mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-xilinx

Reply via email to