[U-Boot-Users] Pull request: u-boot-freebsd
Wolfgang Denk
wd at denx.de
Thu Dec 27 01:05:34 CET 2007
Dear Rafal,
in message <47666652.30600 at semihalf.com> you wrote:
>
> This is a re-spin on the already requested pull for the initial implementation
> of the extended API for standalone apps, that is used for FreeBSD bootstrapping.
I'm afraid I have to reject this pull request.
First, I think it breaks Linux booting support for a couple of boards
(all 4xx ones, that is).
> The following changes since commit 41be969f4957115ed7b1fe8b890bfaee99d7a7a2:
> Wolfgang Denk (1):
> Release v1.3.1
>
> are available in the git repository at:
...
> [POWERPC] Simplify bd_info struct
This is the culprit.
With your patch, you *always* include an entry "unsigned long
bi_bar;" in the bd_infor structure - even for processors which didn't
have one before (like 4xx boards). Thus you change the layout of the
bd_info structure for such systems, which causes an incompatibility
with the respective structure used by the Linux kernel.
Also, I have to admit that I dislike this type of #ifdef based
"simplification". I don't think the resulting code becomes more
readable.
Please fix and resubmit.
Then, we have this part:
> [Net] Introduce standalone eth_receive() routine
This adds a lot of code to the networking code which is not neede dby
most of the boards. Please make this an optional feature that get's
only compiled in for boards that explicitely request it. Then run this
patch separately through the network custodian.
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Success in marriage is not so much finding the right person as it is
being the right person.
More information about the U-Boot
mailing list