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

Ben Warren bwarren at qstreams.com
Thu Dec 27 17:27:50 CET 2007


Hi Rafal,

Rafal Jaworowski wrote:
> 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.
>
>   
When you send the re-spun patch I'll be sure to pull it in.

regards,
Ben




More information about the U-Boot mailing list