[meta-freescale] [meta-fsl-arm-extra][PATCH] nitrogen6x.conf: Allow kernel provider override

Otavio Salvador otavio at ossystems.com.br
Wed Oct 30 07:33:50 PDT 2013


On Wed, Oct 30, 2013 at 12:25 PM, Gary Thomas <gary at mlbassoc.com> wrote:
> On 2013-10-30 08:10, Otavio Salvador wrote:
>>
>> On Wed, Oct 30, 2013 at 11:41 AM, Gary Thomas <gary at mlbassoc.com> wrote:
>>>
>>> This change lets the user override the choice of kernel in local.conf
>>> Without it, there is no way to build any kernel, e.g. linux-imx, other
>>> than the linux-boundary version.
>>>
>>> Signed-off-by: Gary Thomas <gary at mlbassoc.com>
>>
>>
>> I really dislike this.
>>
>> I understand your need, and it is a valid one, but it'd be better you
>> to make a new machine (which includes this one) and override it there.
>
> Even if I did that (and I have), if I include this config file, I can
> never override this setting in a soft (user settable) way.

Then you can force it in your machine.

>> If we start allow all kind of override in machine configuration it
>> loses its meaning and complicates the support.
>>
>> Eric? comments?
>>
>
> Eric has already approved this change.

This is not up to me but I'd like to discuss it a little bit further
before merging it...

My main concern about this is the support. As I said it is a valid
use-case but /most/ people will end with a .conf for the custom
settings in the end with their internal fork or so. Do it in
local.conf or environment is not advised as it is easy to be forgotten
or do wrong so for every product it is better to have a .conf in case
it needs specific kernel/u-boot patches.

-- 
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



More information about the meta-freescale mailing list