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

[Eric Nelson <eric.nelson@boundarydevices.com>]

On 10/30/2013 09:01 AM, Otavio Salvador wrote:
> On Wed, Oct 30, 2013 at 12:44 PM, Gary Thomas <gary@mlbassoc.com> wrote:
>> On 2013-10-30 08:33, Otavio Salvador wrote:
>>> On Wed, Oct 30, 2013 at 12:25 PM, Gary Thomas <gary@mlbassoc.com> wrote:
>>>> On 2013-10-30 08:10, Otavio Salvador wrote:
>>>>> On Wed, Oct 30, 2013 at 11:41 AM, Gary Thomas <gary@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@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.
>>
>> A reasonable concern.  I do note that having this be soft in <MACHINE>.conf
>> seems to be common practice in the rest of the Yocto world.
>
> Ok; you guys won :-)
>
> I applied it to dora and master (I manually applied it as it has not
> applied cleanly).
>
> Eric, please, if you have time do the same change in nitrogen6x-lite
> so it is kept in sync.
>

Will do.