[U-Boot] [PATCH 01/10 v5] powerpc/mpc85xx: support application without resetvec segment in the linker script

Andy Fleming afleming at gmail.com
Fri Jun 28 00:16:30 CEST 2013


On Thu, Jun 27, 2013 at 5:01 PM, Tom Rini <trini at ti.com> wrote:

> On Fri, Jun 21, 2013 at 07:36:09AM +0200, Wolfgang Denk wrote:
> > Dear ying.zhang at freescale.com,
> >
> > In message <1371715468-21120-1-git-send-email-ying.zhang at freescale.com>
> you wrote:
> > >
> > > --- a/arch/powerpc/cpu/mpc85xx/u-boot-spl.lds
> > > +++ b/arch/powerpc/cpu/mpc85xx/u-boot-spl.lds
> > > @@ -1,8 +1,5 @@
> > >  /*
> > > - * (C) Copyright 2006
> > > - * Wolfgang Denk, DENX Software Engineering, wd at denx.de
> > > - *
> > > - * Copyright 2009 Freescale Semiconductor, Inc.
> > > + * Copyright 2013 Freescale Semiconductor, Inc.
> >
> > Please never, ever modify existing copyright headers in any such way!!
> >
> > You MUST keeep existing entries (unless they are clearly wrong).
> >
> >
> >
> > NAK!
>
> Please address this NAK and make sure it's not also happening in the
> rest of the series.  Thanks!
>


Unfortunately, this went in with the last batch:

commit 5df572f0131cf5e0abd8ce4e8f57841b790c40d4
Author: Ying Zhang <b40530 at freescale.com>
Date:   Mon May 20 14:07:23 2013 +0800

    powerpc/mpc85xx: support application without resetvec segment in the
linker


Fortunately, while the change history on this patch says "No change", the
copyright screwup WAS a change, and does NOT exist in the committed version
of the patch.

Sometimes two wrongs do make a right. :)

Andy


More information about the U-Boot mailing list