All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Nelson <eric.nelson@boundarydevices.com>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: [meta-fsl-arm-extra][PATCH] nitrogen6x.conf: Allow kernel provider override
Date: Wed, 30 Oct 2013 07:42:07 -0700	[thread overview]
Message-ID: <52711ABF.5050308@boundarydevices.com> (raw)
In-Reply-To: <CAP9ODKpM4qyyUycBDkF-ag6RpBFbGWqzbrYOLNSWmgtgWWEn=A@mail.gmail.com>

Hi Otavio,

On 10/30/2013 07:28 AM, Otavio Salvador wrote:
> Hello Eric,
>
> On Wed, Oct 30, 2013 at 12:25 PM, Eric Nelson
> <eric.nelson@boundarydevices.com> wrote:
>> On 10/30/2013 07:10 AM, 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.
>>>
>>
>> I'm not sure I understand why. It seems perfectly reasonable to me
>> to build a new kernel recipe and use it, while retaining other
>> information from the machine configuration.
>>
>> I'm thinking specifically of a user who may have a custom kernel
>> tree with pin-muxing based on their usage.
>
> I see the point and I do have this case internally here; the point is
> if we have it 'soft' and someone reports:
>
> Nitrogen6X is not working. Next question needs to be, are you using
> linux-boundary or another?
>

I understand your point, but it's going to happen anyway. The beauty
(and curse) of flexible hardware is that it's flexible ;)

I think the best we can hope for in terms of support is to always
ask if a failure occurs with a "stock" build... Fresh "repo sync",
specified U-Boot, et cetera.

I don't know if hard-coding this value will meaningfully change
things.

>>> If we start allow all kind of override in machine configuration it
>>> loses its meaning and complicates the support.
>>>
>>> Eric? comments?
>>>
>>
>> I'm not sure I understand the concern. This seems pretty
>> straightforward. The default is there, but a user can
>> over-ride it.
>
> If user forks the kernel, adding a machine .conf  file is the easiest
> part of it. I'd to avoid support uncertainty regarding settings.
>

That's true, too (machine configurations aren't very complicated), but
my understanding of that is recent.

Regards,


Eric



  reply	other threads:[~2013-10-30 14:42 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-30 13:41 [meta-fsl-arm-extra][PATCH] nitrogen6x.conf: Allow kernel provider override Gary Thomas
2013-10-30 14:06 ` Eric Nelson
2013-10-30 14:10 ` Otavio Salvador
2013-10-30 14:25   ` Eric Nelson
2013-10-30 14:28     ` Otavio Salvador
2013-10-30 14:42       ` Eric Nelson [this message]
2013-10-30 14:25   ` Gary Thomas
2013-10-30 14:33     ` Otavio Salvador
2013-10-30 14:44       ` Gary Thomas
2013-10-30 16:01         ` Otavio Salvador
2013-10-30 16:16           ` Eric Nelson
2013-10-30 16:56           ` Daiane Angolini
2013-10-30 17:29             ` Otavio Salvador
2013-10-30 17:46               ` Eric Nelson
2013-10-30 17:51                 ` Otavio Salvador
2013-10-30 18:04                 ` Gary Thomas

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=52711ABF.5050308@boundarydevices.com \
    --to=eric.nelson@boundarydevices.com \
    --cc=meta-freescale@yoctoproject.org \
    --cc=otavio@ossystems.com.br \
    /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.