On Wed, Aug 01, 2012 at 10:59:10PM +0200, Andreas Bießmann wrote:
> On 01.08.12 21:57, Markus Hubig wrote:
> > +Markus Hubig <mhu...@imko.de>
> > +
> > + STAMP9G20 ARM926EJS
> > +
> > Ilko Iliev <il...@ronetix.at>
>
> unfortunately Ilko Iliev is wrong here ... this is the PPC section (all
> these MPCxx). About line 566 begins the ARM section.
Moved both of us ...
> > + * RAM Memory address where to put the
> > + * Linux Kernel bevor starting.
> --------------------^
> before?
Ups ...
> > + "mtdparts=mtdparts=atmel_nand:128k(bootstrap)ro," \
> > + "256k(uboot)ro,128k(env1)ro," \
> > + "128k(env2)ro,12M(linux),-(root)rw\0" \
>
> your table in comment tells 6MiB for kernel, here you have 12MiB ;)
Grrr fixed! ;)
> > + "nfsboot=dhcp; setenv bootargs ${basicargs} ${mtdparts} " \
> > + "root=/dev/nfs ip=dhcp; bootm\0"
>
> dhcp command can use some env (I think bootfile) to directely boot. So
> setting this env, setting bootargs and then run dhcp would maybe better.
> Alternatively the nfsboot could imply a correctly set bootfile, this
> could be another env in your EXTRA_ENV_SETTINGS (another side note;).
Ahh it's autoboot=yes! ;)
Btw: I think it would be easier to handle if all the options in
env_common.h would be prefixed with CONFIG_ENV_* ...
> > +#endif /* __CONFIG_H */
> >
>
> We should think about the MACB ifdef stuff again. I learned that the
> stamp9g20 is a small device sitting on different base boards. In case of
> portux9g20 we will have ethernet. Therefore we should always enable MACB
> when we are configured as portux9g20. But if one uses the stamp on
> another base board he maybe want to disable ethernet support easily ...
> with this information I think it is good to have these ifdef blocks
> around all the ethernet stuff. Sorry, I understood this in your last
> mail and you sent this patch to fast to comment ...
Hmm ok good idea ... will include this in the next patch.
Cheers, Markus
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
