All of lore.kernel.org
 help / color / mirror / Atom feed
From: Otavio Salvador <otavio@ossystems.com.br>
To: Nikolay Dimitrov <picmaster@mail.bg>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: [meta-fsl-demos][PATCH] riotboard: Fix broken image builds against linux-fslc
Date: Wed, 13 May 2015 08:06:48 -0300	[thread overview]
Message-ID: <CAP9ODKpveWxE-m=YQzewkDhE3zwDywuvUp7zFPc=W5_uM+AKZg@mail.gmail.com> (raw)
In-Reply-To: <5552CE65.8030507@mail.bg>

On Wed, May 13, 2015 at 1:09 AM, Nikolay Dimitrov <picmaster@mail.bg> wrote:
> On 05/12/2015 03:41 PM, Otavio Salvador wrote:
>> I reproduced it here. It fails only with X11 and my test was using
>> Framebuffer. I have sent a patch for review.
>
> The patch looks ok to me. I suppose that after this fix, the RDEPENDS
> hack for imx6sl is not needed anymore?

It is as imx6sl does not provide the headers[1] and gst-plugins-gl
fail to build.

1. http://git.yoctoproject.org/cgit/cgit.cgi/meta-fsl-arm/tree/recipes-graphics/imx-gpu-viv/imx-gpu-viv.inc#n220

...
>> The biggest challenge here is how to express KERNEL_FEATURES which are
>> recipe/provider specific.
>>
>> We have some providers:
>>
>>   linux-fslc
>>   linux-imx
>>   linux-<vendor>
>>
>> and each can provide different kernel features. I agree with the long
>> term plan but I didn't yet came up with something which scales and
>> work.
>
>
> I'll reiterate my thoughts - it's not only the kernel features. All the
> parts in the system need to consider a feature, before it's built:
> - hardware
> - kernel
> - userspace
>
> A feature shouldn't be compiled and/or deployed in the image unless all
> the system parts confirm its support, otherwise we'll have a failed
> build or runtime issues.
>
> With my current limited knowledge I see 2 basic paths for implementing
> this:
> - 1 shared variable per feature, which is mangled across the
> recipes/layers (the usual enable/disable dance);
> - mechanism similar to COMBINED_FEATURES, which needs to "AND" 2 or
> more sets of variables across the system. This is already done
> implicitly in lots of places with heavy usage of $base_contains, but
> it's not easy to read and explicit.
>
> // board.conf
> MACHINE_FEATURES += "opengl"
>
> // kernel provider
> KERNEL_FEATURES += "opengl"
>
> // image or recipe should check features indirectly
> XXXXXXX_append = "${base_contains('COMBINED_FEATURES', 'opengl',
> 'gst-plugins-gl', '', d)}
>
> Something like this...

The biggest challenge here is how to express KERNEL_FEATURES which are
recipe/provider specific.

We have some providers:

  linux-fslc
  linux-imx
  linux-<vendor>

and each can provide different kernel features. We will need to come
up with a solution to export the KERNEL_FEATURES outside of recipe
namespace and make it in a clean and easy to maintain way. This is the
part of puzzle still not in place.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


  reply	other threads:[~2015-05-13 11:06 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-30 15:49 [meta-fsl-arm-extra][PATCH v3] riotboard: Fix broken image builds against linux-fslc Nikolay Dimitrov
2015-04-30 15:49 ` [meta-fsl-demos][PATCH] " Nikolay Dimitrov
2015-05-04 12:41   ` Fabio Estevam
2015-05-04 13:33     ` Nikolay Dimitrov
2015-05-04 17:06       ` Daiane Angolini
2015-05-04 17:12         ` Gary Thomas
2015-05-04 19:38         ` Nikolay Dimitrov
2015-05-11 16:43   ` Nikolay Dimitrov
2015-05-11 17:09     ` Otavio Salvador
2015-05-11 17:24       ` Nikolay Dimitrov
2015-05-11 17:46         ` Otavio Salvador
2015-05-11 23:42           ` Nikolay Dimitrov
2015-05-12 12:41             ` Otavio Salvador
2015-05-13  4:09               ` Nikolay Dimitrov
2015-05-13 11:06                 ` Otavio Salvador [this message]
2015-05-11 16:42 ` [meta-fsl-arm-extra][PATCH v3] " Nikolay Dimitrov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAP9ODKpveWxE-m=YQzewkDhE3zwDywuvUp7zFPc=W5_uM+AKZg@mail.gmail.com' \
    --to=otavio@ossystems.com.br \
    --cc=meta-freescale@yoctoproject.org \
    --cc=picmaster@mail.bg \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.