[U-Boot-Users] [PATCH 00/07 v2]: Add mpc7448hpc2 (Taiga) board support
tie-fei.zang at freescale.com
Fri Dec 1 12:23:00 CET 2006
On Fri, 2006-12-01 at 16:59, Wolfgang Denk wrote:
> In message <1164940274.5921.26.camel at localhost.localdomain> you wrote:
> > > 5) board/mpc7448hpc2/mpc7448hpc2.c contains yet another memory
> > >
> 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
> > > config file. It is really evil when all boards have the same
> > > addresses. Also, are the addresses you used officially
> > > 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
> > > development, but for a public source version I don't ever want
> > > see these.
> > Can you make sure all of these define are not necessary? I can see
> > 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
> > > nor comment anywhere why you think this is necessary.
> > I need to deal with exception after the code relocation. I need add
> 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
> > 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.
More information about the U-Boot