On 04-08-16 17:02, Nathan Rossi wrote:
On Tue, Aug 2, 2016 at 4:49 AM, Manjukumar Harthikote Matha
<[email protected]> wrote:
Hi Nathan,
On 07/31/2016 06:17 AM, Nathan Rossi wrote:
On Sat, Jul 30, 2016 at 4:46 AM, Manjukumar Harthikote Matha
<[email protected]> wrote:
Hi Mike,
On 07/26/2016 03:59 AM, Mike Looijmans wrote:
Not all zynqmp SoCs have a MALI GPU built in. Thus recommending
kernel-module-mali for all zynqmp machines is incorrect. Move
the recommendation to the board config.
This also makes for a nicer syntax, having less-specific platform
overrides for machine-specific variables just doesn't look right.
Signed-off-by: Mike Looijmans <[email protected]>
---
conf/machine/include/machine-xilinx-default.inc | 4 ----
conf/machine/zcu102-zynqmp.conf | 3 +++
2 files changed, 3 insertions(+), 4 deletions(-)
<...>
Yes this is true, after the road map has been made clear with CG parts,
however please see my previous comments on mali-modules, once resolved we
should move it to board-specific
It appears that the product table is public showing CG, EG and EV
parts respectively.
http://www.xilinx.com/products/silicon-devices/soc/zynq-ultrascale-mpsoc.html#productTable
Is it worth having three separate includes for the CG/EG/EV parts? So
that this sort of config can be easily included for machines/boards
that use the specified part? This would allow additional configuration
that would only be relevant to those variants as well.
Can we use MACHINE_FEATURES and enable appropriate configs? Do you think it
is a good idea? We can have one include instead of 3 or more.
That is a definitely a better solution, then all that is needed is the
two feature flags 'zynqmp-mali' (or just mali?) and 'zynqmp-venc' (or
something similar). And most of the conditional selection can be done
with bb.utils.contains.
For mali maybe you could use an existing machine feature like "opengl"?
--
Mike Looijmans
--
_______________________________________________
meta-xilinx mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-xilinx
