> -----Original Message-----
> From: Thomas Monjalon <tho...@monjalon.net>
> Sent: Monday, November 16, 2020 8:29 AM
> To: Juraj Linkeš <juraj.lin...@pantheon.tech>
> Cc: bruce.richard...@intel.com; ruifeng.w...@arm.com;
> honnappa.nagaraha...@arm.com; phil.y...@arm.com;
> vcchu...@amazon.com; dharmik.thak...@arm.com; jerinjac...@gmail.com;
> hemant.agra...@nxp.com; ajit.khapa...@broadcom.com;
> ferruh.yi...@intel.com; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v12 11/14] build: disable Arm drivers
>
> What do you mean by "disable Arm drivers"?
> What are "Arm drivers"
>
These are the drivers that we're disableing for Arm builds. I'll change it to
something more clearer like "disable drivers in Arm builds"
> 13/11/2020 15:31, Juraj Linkeš:
> > A few options that disabled drivers in the old makefiles were
> > improperly ported to the meson build system. Fix this by adding a to
> > the list of disabled drivers, similarly how the command line option
> > works. Remove unneeded driver options ported from the old makefile system.
> > Add support for removing drivers for cross builds.
> >
> > Signed-off-by: Juraj Linkeš <juraj.lin...@pantheon.tech>
> > Acked-by: Bruce Richardson <bruce.richard...@intel.com>
> > Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
> [...]
> > - ['RTE_NET_FM10K', false],
> > - ['RTE_NET_AVP', false],
>
> Isn't it enabling drivers?
>
These options don't do anything in the way the current meson build system is
set up, since they'll be overwritten in drivers/meson.build. The order is, from
meson.build (in the git root):
subdir('config') # here we're setting arm specific flags
...
subdir('drivers') # and here they'd be overwritten
> [...]
> > +if meson.is_cross_build()
> > + disabled_drivers += meson.get_cross_property('disabled_drivers', [])
> > +endif
>
> Why is it specific to cross build?
>
I don't understand the question, what is "it"? This is adding support for
disabling drivers in cross builds so I guess that part is specific to cross
builds?
