On Thu, Jun 27, 2013 at 5:01 PM, Tom Rini trini@ti.com wrote:
On Fri, Jun 21, 2013 at 07:36:09AM +0200, Wolfgang Denk wrote:
Dear ying.zhang@freescale.com,
In message 1371715468-21120-1-git-send-email-ying.zhang@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@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@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