[U-Boot-Users] [PATCH] Enable the IceCube/lite5200 variants to pass a device tree to Linux.

Grant Likely grant.likely at secretlab.ca
Mon Oct 30 06:49:10 CET 2006


On 10/29/06, Wolfgang Denk <wd at denx.de> wrote:
> In message <11621624513123-git-send-email-grant.likely at secretlab.ca> you wrote:
> > This patch adds the code and configuration necessary to boot with an
> > arch/powerpc Linux kernel.
> ...
> > --- a/board/icecube/icecube.c
> > +++ b/board/icecube/icecube.c
> ...
> > @@ -334,3 +338,11 @@ void ide_set_reset (int idereset)
> >       }
> >  }
> >  #endif /* defined (CFG_CMD_IDE) && defined (CONFIG_IDE_RESET) */
> > +
> > +#if defined(CONFIG_OF_FLAT_TREE) && defined(CONFIG_OF_BOARD_SETUP)
> > +void
> > +ft_board_setup(void *blob, bd_t *bd)
> > +{
> > +     ft_cpu_setup(blob, bd);
> > +}
> > +#endif
>
> How much of this is really board dependent? How likely  is  it,  then
> all other 5200 boards will just copy & paste these few lines of code?
> Maybe we can make this common code, and just allow for board-specific
> overwrites when really needed?

Oh, probably.  :)

In this case I was following the convention already established w/ the
other boards.  I can take another look and see about making a
recommendation for cleaning up this construct.

Cheers,
g.

>
> Best regards,
>
> Wolfgang Denk
>
> --
> Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> "The bad reputation UNIX has gotten is totally undeserved, laid on by
> people who don't understand, who have not gotten in there  and  tried
> anything."          -- Jim Joyce, owner of Jim Joyce's UNIX Bookstore
>


-- 
Grant Likely, B.Sc. P.Eng.
Secret Lab Technologies Ltd.
grant.likely at secretlab.ca
(403) 399-0195




More information about the U-Boot mailing list