[meta-xilinx] [PATCH] zynqmp: Make the MALI recommendation machine-specific

[Nathan Rossi]

On Sat, Jul 30, 2016 at 4:46 AM, Manjukumar Harthikote Matha
<manjukumar.harthikote-matha at xilinx.com> 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 <mike.looijmans at topic.nl>
>> ---
>>  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.

Regards,
Nathan

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