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

[Manjukumar Harthikote Matha]

> -----Original Message-----
> From: Nathan Rossi [mailto:nathan at nathanrossi.com]
> Sent: Monday, October 23, 2017 11:16 PM
> To: Manjukumar Harthikote Matha <MANJUKUM at xilinx.com>
> Cc: meta-xilinx at lists.yoctoproject.org
> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Add default tune for
> multilib lib32 builds
> 
> On 24 October 2017 at 10:05, 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/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 137f49e..38b7dd3 100644
> > --- a/conf/machine/include/machine-xilinx-default.inc
> > +++ b/conf/machine/include/machine-xilinx-default.inc
> > @@ -41,3 +41,6 @@ XSERVER ?= " \
> >
> >  XSERVER_EXT ?= ""
> >  XSERVER_EXT_zynqmp ?= "xf86-video-armsoc"
> > +
> > +# Support multilib on zynqmp
> > +DEFAULTTUNE_virtclass-multilib-lib32_zynqmp = "armv7vethf-neon-vfpv4"
> 
> So you had sent a very similar patch a few months back, I had commented then that
> it would be a good idea to have the
> DEFAULTTUNE_virtclass-multilib-lib32 set in tune-zynqmp.inc. Is there an issue with
> setting the DEFAULTTUNE that way?
> 
> https://lists.yoctoproject.org/pipermail/meta-xilinx/2017-June/002985.html
>
I don’t think there is an issue with that, we tested against thumb setting but did not move the patch to tune-zynqmp.inc, will send a V2

Thanks,
Manju