[meta-xilinx] [RFC] machine-xilinx-default.inc: Include kernel modules

[Nathan Rossi]

On Wed, May 4, 2016 at 4:18 PM, Mike Looijmans <mike.looijmans at topic.nl> wrote:
> That looks like too much of a simplistic approach, as this will dump each
> and every built module into any image.
>
> The proper solution would be to explicitly list the modules that are really
> required.

I agree that it would be best to only add the specific modules of interest.

After a quick look it does appear that a common setup (in a variety of
layers/machines) is to just add all modules to
MACHINE_EXTRA_RRECOMMENDS (as opposed to ESSENTIAL_EXTRA). So in that
case Manju, if you are not sure of the exact modules then setting this
via MACHINE_EXTRA_RRECOMMENDS instead should work out better, since
for mali related stuff it is very likely to be using a graphics
enabled image (aka. one that includes packagegroup-base, like
core-image-sato).

Regards,
Nathan

>
>
> On 04-05-16 08:04, Manjukumar Matha wrote:
>>
>> Include kernel modules for ZynqMP. Kernel modules will be needed when
>> including
>> mali modules.
>>
>> Signed-off-by: Manjukumar Matha <manjukumar.harthikote-matha at xilinx.com>
>> ---
>>   conf/machine/include/machine-xilinx-default.inc | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/conf/machine/include/machine-xilinx-default.inc
>> b/conf/machine/include/machine-xilinx-default.inc
>> index 2df8f4d..ad5465d 100644
>> --- a/conf/machine/include/machine-xilinx-default.inc
>> +++ b/conf/machine/include/machine-xilinx-default.inc
>> @@ -32,3 +32,4 @@ UBOOT_BINARY ?= "u-boot${UBOOT_OFEMBED}.${UBOOT_SUFFIX}"
>>   UBOOT_ELF ?= "u-boot"
>>   UBOOT_ELF_aarch64 ?= "u-boot.elf"
>>
>> +MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS_zynqmp = "kernel-modules"
>>
>
>
>
> Kind regards,
>
> Mike Looijmans
> System Expert
>
> TOPIC Embedded Products
> Eindhovenseweg 32-C, NL-5683 KH Best
> Postbus 440, NL-5680 AK Best
> Telefoon: +31 (0) 499 33 69 79
> E-mail: mike.looijmans at topicproducts.com
> Website: www.topicproducts.com
>
> Please consider the environment before printing this e-mail
>
>
>
>
>
>
> --
> _______________________________________________
> meta-xilinx mailing list
> meta-xilinx at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-xilinx