[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