From mboxrd@z Thu Jan  1 00:00:00 1970
From: Zang Roy-r61911 <tie-fei.zang@freescale.com>
Date: 01 Dec 2006 19:23:00 +0800
Subject: [U-Boot-Users] [PATCH 00/07 v2]: Add mpc7448hpc2 (Taiga)
	board	support
In-Reply-To: <20061201085938.5E2AB353B0E@atlas.denx.de>
References: <20061201085938.5E2AB353B0E@atlas.denx.de>
Message-ID: <1164972180.7260.8.camel@localhost.localdomain>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Wolfgang

On Fri, 2006-12-01 at 16:59, Wolfgang Denk wrote:
> In message <1164940274.5921.26.camel@localhost.localdomain> you wrote:

> 
> > > 5) board/mpc7448hpc2/mpc7448hpc2.c contains yet another memory
> test.
> > >   
> As far as I  can  tell  your  code  is  mostly  a  verbatim  copy  of
> post/memory.c  - if you need a good memory test then please configure
> POST on your system and use the POST code instead of copying it.
The copy is not necessary. I will remove them. Thanks.
> 
> > > 9) The login here looks weird to me - is this correct?
> > >    cpu/74xx_7xx/speed.c:
> > >         ...
> 
> But why do we need both CFG_CONFIG_BUS_CLK and CFG_BUS_CLK ?
No! I will remove one!
> 
> > > 11) Please don't define CONFIG_ETHADDR / CONFIG_ETH1ADDR in your
> board
> > >     config file. It is really evil when all boards have the same
> MAC
> > >     addresses. Also, are the addresses you used officially
> assigned
> > >     ones?
> > 00:06:D2 is officially assigned to Tundra :-).
> 
> Anyway: please don;t define MAC addresses in the board config file. It
> will only cause harm.
> 
> > >     Same is for CONFIG_IPADDR, CONFIG_SERVERIP, CONFIG_NETMASK,
> > >     CONFIG_GATEWAYIP - it may save some time to have these set
> during
> > >     development, but for a public source version I don't ever want
> to
> > >     see these.
> > Can you make sure all of these define are not necessary? I can see
> them
> > in many other board.
> 
> There may be a *few* boards that do this, but in general it's a  very
> bad  idea:  it  works  fine  for the guy who is eorking on the U-Boot
> port, because he usually uses just a single board. But as soon as you
> have a second board in the net it becomes a major PITA. Please  don't
> do it.
> 
> If needed, you can set up a valid per-board network setup as part of
> the software initialization - a simple expect script can do magic.
I will remove them.
> 
> > > 12) In lib_ppc/extable.c you add code with a "#ifdef
> > >     CFG_EXCEPTION_AFTER_RELOCATE; there is absolutely no
> explanation
> > >     nor comment anywhere why you think this is necessary.
> > I need to deal with exception after the code relocation. I need add
> the
> 
> Why do you need to do this?
> 
Because my exception occurs in RAM, after the u-boot relocation.
> > gd->reloc_off to search the exception table. 
> > If I do not find better method for this, I will add a detailed
> comment
> > here. 
> 
> I think this part of the code is pretty  generic.  I  would  like  to
> understand  why on your board such a change is necessary which is not
> needed on any other system.
I have explain the detail reason in my previous mail and provide a draft
patch for list review.
I hope I can present the issue here and have a discussion.
Thanks.
Roy