From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Fri, 09 Apr 2010 07:25:22 +0200 Subject: [U-Boot] [PATCH V5 1/3] Initial support for Marvell Orion5x SoC In-Reply-To: References: Message-ID: <4BBEBA42.7070507@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Prafulla Wadaskar a ?crit : > Dear Albert >> +++ b/cpu/arm926ejs/orion5x/Makefile >> @@ -0,0 +1,55 @@ >> +# >> +# Copyright (C) 2009 Albert ARIBAUD > > General comment for all files, > You should change copyright statement for year 2010 instead of 2009 Drat--didn't realize this. :o) > +#define PCIE_DEV_ID_OFF (ORION5X_REG_PCIE_BASE + 0x0000) > +#define PCIE_DEV_REV_OFF (ORION5X_REG_PCIE_BASE + 0x0008) > > These are already defined cpu.h, pls remove it Fixed >> +++ b/cpu/arm926ejs/orion5x/lowlevel_init.S >> @@ -0,0 +1,294 @@ >> +/************************************************************ >> ******************* >> +Copyright (c) Albert ARIBAUD >> +Redistributed under GPLv2, pursuant to the licensing terms >> +of the original Marvell U-boot code which is: >> +Copyright (C) Marvell International Ltd. and its affiliates >> +************************************************************* >> ******************* >> +Marvell GPL License Option > > You should put standard GPL statement here as you put for other files, Ok. > Also pls remove Marvell GPL statement I'd kept the notice as this file is a split from, and major part of, the original single lowlevel_init.S which is indeed (c) Marvell; if you feel I can remove the reference to Marvell, that's fine with me. But then, I should remove it also from board/LaCie/edminiv2/board_lowlevel_init.S, should I not? > Apart from above, the code looks okay to me. > Regards.. > Prafulla . . Thanks Prafulla; patch V6 comes in as soon as I get your confirmation on the copyright notice question. Amicalement, -- Albert.