[U-Boot-Users] Pull request: u-boot-freebsd

Rafal Jaworowski raj at semihalf.com
Thu Dec 27 12:56:59 CET 2007


Dear Wolfgang,
you wrote:
> 
> 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.
> 

Mhm, right. Maybe it was too much of an improvement :) Given all the legacy
dependencies between bd_info and Linux it's probably better to just leave it
as is and deal with those #idef'ed fields locally (where they are accessed).

> Please fix and resubmit.
> 

Will do.

> 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.
> 

I already sent this patch to Ben and the list some time ago and got an initial
ACK, but I'll re-spin with.

kind regards,
Rafal




More information about the U-Boot mailing list