From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ie0-f180.google.com (mail-ie0-f180.google.com [209.85.223.180]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 59777E014D0 for ; Wed, 10 Apr 2013 06:20:04 -0700 (PDT) Received: by mail-ie0-f180.google.com with SMTP id a11so477537iee.39 for ; Wed, 10 Apr 2013 06:20:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:cc:content-type; bh=yFSbOwmDwxsPtCoQ7zlJJeCGjbbtPUSnbnEt+37mF0k=; b=dW6aOoz91sPzM7o14SYtj6el7b+NvtqNbQHIDxLiHmqdOJs2zcVNSu8jK5QTVUBG5Z ZaY4lqyXnngaODu3X7EBWHcdOXfhJWI6p4s7NId4ZWm7e1QVvFzqeug8hcYBevaVblzW ErlYdy74UK66aC76WTIeMKQ8Ek3KhOZv0i+q8sEFSZegXDbvT1paJYhJPIjgcKMivfGV +TmYpHy9jFTDQAx6GBMAtvY9K/HoCWQivOBTWMN24jXn+XvgR9KSxhIvUhgTaZXaqeF4 K6fQDuk0S85rEHRr8zZeHZRBcNRtRD3Bm5b3Dj5zDJwX3wq80m0PJWtBTrqgc0nCm0ai HyEA== MIME-Version: 1.0 X-Received: by 10.42.231.195 with SMTP id jr3mr1214472icb.45.1365600003882; Wed, 10 Apr 2013 06:20:03 -0700 (PDT) Received: by 10.64.170.105 with HTTP; Wed, 10 Apr 2013 06:20:03 -0700 (PDT) In-Reply-To: <20130410150644.72e8765c@e6520eb> References: <1365535321-27350-1-git-send-email-otavio@ossystems.com.br> <1365535321-27350-8-git-send-email-otavio@ossystems.com.br> <5164D31A.7030108@gmail.com> <20130410141524.334be50f@e6520eb> <20130410150644.72e8765c@e6520eb> Date: Wed, 10 Apr 2013 08:20:03 -0500 Message-ID: From: John Weber To: =?ISO-8859-1?Q?Eric_B=E9nard?= Cc: meta-freescale Mailing List , Otavio Salvador Subject: Re: [meta-fsl-arm-extra][PATCH v2 6/8] u-boot-imx: Remove Wandboard patch as it is not supported by mainline 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, 10 Apr 2013 13:20:04 -0000 Content-Type: multipart/alternative; boundary=047d7bacbc7223053104da018798 --047d7bacbc7223053104da018798 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable I don't have a problem with removing the old code. Just that we need to do it in phases. Phase 1 is to add new code. Test that. Then get rid of the old code. If we do it all at once then I can't make a simple configuration change and compare code bases. On Wednesday, April 10, 2013, Eric B=E9nard wrote: > Le Wed, 10 Apr 2013 09:20:28 -0300, > Otavio Salvador > a =E9crit : > > > On Wed, Apr 10, 2013 at 9:15 AM, Eric B=E9nard > > wrote: > > > Hi Otavio, > > > > > > Le Wed, 10 Apr 2013 08:56:01 -0300, > > > Otavio Salvador > a =E9crit : > > > > > >> On Tue, Apr 9, 2013 at 11:48 PM, John Weber > > wrote: > > >> > Are we completely gutting the old u-boot for wandboard dual? This > patch > > >> > doesn't pertain to mainline but it does still work for the FSL > u-boot. > > >> > > >> I'd prefer to remove old and unused code. Except if you have a reaso= n > > >> to keep it? > > >> > > >> Both Dual and Solo variants are going to use U-Boot mainline and noo= ne > > >> will use this, except if explicitly set. So I think in the end it > > >> won't be used at all. Do you see a reason to keep it? > > > > > > maybe because you are migrating several boards to a new bootloader > which > > > is not even stable (u-boot 2013.04 is at rc2) and not present in the > > > official Freescale BSP ? > > > > Wandboard support is not present in official BSP either; the U-Boot is > > being daily tested by Fabio and me so it had a good coverage. > > > if two testers of a -rc2 is a good coverage to migrate several boards > (including some official Freescale evk) to this -rc2 a few days before > the release then everything is fine, sorry for the annoyance. > > > > Or maybe the policy has changed since last week and you consider > > > there is no risk to migrate to a -rc bootloader just before the > > > release ;-) > > > > The point is support. In case of maintain branch of kernel you won't > > be able to open SR issues about this kernel while U-Boot the support > > is already given by community. > > > > So the policy is to allow the change when we will have support; not > > the opposite. > > > OK fine. > > Eric > --047d7bacbc7223053104da018798 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable I don't have a problem with removing the old code. Just th= at we need to do it in phases. =A0Phase 1 is to add new code. Test that. Th= en get rid of the old code. =A0If we do it all at once then I can't mak= e a=A0simple configuration change and compare=A0code bases.=A0
=A0

On Wednesday, April 10, 2013, Eric B=E9nard wrote:
Le Wed, 10 Apr 2013 09:20:28 -0300,
Otavio Salvador <otavio@ossystems.com.br> = a =E9crit :

> On Wed, Apr 10, 2013 at 9:15 AM, Eric B=E9nard <eric= @eukrea.com> wrote:
> > Hi Otavio,
> >
> > Le Wed, 10 Apr 2013 08:56:01 -0300,
> > Otavio Salvador <otavio@ossystems.com.b= r> a =E9crit :
> >
> >> On Tue, Apr 9, 2013 at 11:48 PM, John Weber <rjohnweber@gmail.com> wrote:
> >> > Are we completely gutting the old u-boot for wandboard d= ual? =A0This patch
> >> > doesn't pertain to mainline but it does still work f= or the FSL u-boot.
> >>
> >> I'd prefer to remove old and unused code. Except if you h= ave a reason
> >> to keep it?
> >>
> >> Both Dual and Solo variants are going to use U-Boot mainline = and noone
> >> will use this, except if explicitly set. So I think in the en= d it
> >> won't be used at all. Do you see a reason to keep it?
> >
> > maybe because you are migrating several boards to a new bootloade= r which
> > is not even stable (u-boot 2013.04 is at rc2) and not present in = the
> > official Freescale BSP ?
>
> Wandboard support is not present in official BSP either; the U-Boot is=
> being daily tested by Fabio and me so it had a good coverage.
>
if two testers of a -rc2 is a good coverage to migrate several boards
(including some official Freescale evk) to this -rc2 a few days before
the release then everything is fine, sorry for the annoyance.

> > Or maybe the policy has changed since last week and you consider<= br> > > there is no risk to migrate to a -rc bootloader just before the > > release ;-)
>
> The point is support. In case of maintain branch of kernel you won'= ;t
> be able to open SR issues about this kernel while U-Boot the support > is already given by community.
>
> So the policy is to allow the change when we will have support; not > the opposite.
>
OK fine.

Eric
--047d7bacbc7223053104da018798--