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

[Manjukumar Harthikote Matha]

Hi Nathan,

On 07/31/2016 06:17 AM, Nathan Rossi wrote:
> 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.
>
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.

Thanks
Manju