From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pb0-f51.google.com (mail-pb0-f51.google.com [209.85.160.51]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 91D3DE0175F for ; Wed, 30 Oct 2013 09:16:27 -0700 (PDT) Received: by mail-pb0-f51.google.com with SMTP id wz7so1617202pbc.10 for ; Wed, 30 Oct 2013 09:16:27 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=SMuvCIcC2dVB7fX6+Sn55QzSunCyjvU0N7lJnj9QCMM=; b=Gu08GVXG9dopjq1uX00E4TWzmxv1sY14UDShT9srhDj587uMvK5ml4VrqwhzxBiRjG ix4Y+i8LbyuJewFbGdiVyMzC9bbUfdn0+y2CXiKzyINcAiRpZYj+cFcU3bDSFcq4BZkD 1iCVZpRokPmUvcXhivHdDyjP3bOyLQg55yvU9bmi5TwUjhEi2SEkvohZJskO7hBDaN0F ktGVHz6Q90eN//fX7/UbxfvN/nfPLom8RWJhi2ONuI+1um4AJPK2VYm6XrjqnYazDUpI k+dryJ95hXVrVCm7CZull9iFoT7X3cK2tOZqfkG494Rz9CLXelwm9Zt/h1MOyep1QZ6F 6Gfw== X-Gm-Message-State: ALoCoQnx0gyioiMsJGiKDAMpMpzBynfrbEva/Pavr2XE31vaoP+JFD36pzgBrdO2q1JM6vMejSpX X-Received: by 10.68.13.104 with SMTP id g8mr6046612pbc.33.1383149787233; Wed, 30 Oct 2013 09:16:27 -0700 (PDT) Received: from [192.168.0.53] (97-117-181-72.phnx.qwest.net. [97.117.181.72]) by mx.google.com with ESMTPSA id qz9sm42305766pbc.3.2013.10.30.09.16.25 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Wed, 30 Oct 2013 09:16:26 -0700 (PDT) Message-ID: <527130D8.3070102@boundarydevices.com> Date: Wed, 30 Oct 2013 09:16:24 -0700 From: Eric Nelson User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 MIME-Version: 1.0 To: Otavio Salvador , Gary Thomas References: <52710C89.7030903@mlbassoc.com> <527116CB.1050508@mlbassoc.com> <52711B37.6050900@mlbassoc.com> In-Reply-To: Cc: "meta-freescale@yoctoproject.org" Subject: Re: [meta-fsl-arm-extra][PATCH] nitrogen6x.conf: Allow kernel provider override X-BeenThere: meta-freescale@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-fsl-* layers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 30 Oct 2013 16:16:28 -0000 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 10/30/2013 09:01 AM, Otavio Salvador wrote: > On Wed, Oct 30, 2013 at 12:44 PM, Gary Thomas wrote: >> On 2013-10-30 08:33, Otavio Salvador wrote: >>> On Wed, Oct 30, 2013 at 12:25 PM, Gary Thomas wrote: >>>> On 2013-10-30 08:10, Otavio Salvador wrote: >>>>> On Wed, Oct 30, 2013 at 11:41 AM, Gary Thomas 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 >>>>> >>>>> 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 .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.