On Thu, Jun 27, 2013 at 5:01 PM, Tom Rini <[email protected]> wrote:

> On Fri, Jun 21, 2013 at 07:36:09AM +0200, Wolfgang Denk wrote:
> > Dear [email protected],
> >
> > In message <[email protected]>
> 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, [email protected]
> > > - *
> > > - * 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 <[email protected]>
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
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to