Ben Warren had written, on 12/07/2009 03:09 PM, the following:
> Nishanth,
> 
> Nishanth Menon wrote:
>> This modification is NOT tested on any of the
>> platforms modified as I dont have them. please
>> help by testing+building+fixing
>>
>> Signed-off-by: Nishanth Menon <[email protected]>
>> Cc: Andrea Scian <[email protected]>
>> Cc: Ben Warren <[email protected]>
>> Cc: Dave Peverley <[email protected]>
>> Cc: George G. Davis <[email protected]>
>> Cc: Kyungmin Park <[email protected]>
>> Cc: Nishant Kamat <[email protected]>
>> Cc: Richard Woodruff <[email protected]>
>> Cc: Rishi Bhattacharya <[email protected]>
>> Cc: Sandeep Paulraj <[email protected]>
>> ---
>>  board/apollon/apollon.c       |   12 +++++++-----
>>  include/configs/B2.h          |    2 +-
>>  include/configs/apollon.h     |    3 ++-
>>  include/configs/assabet.h     |    3 ++-
>>  include/configs/gcplus.h      |    3 ++-
>>  include/configs/lubbock.h     |    3 ++-
>>  include/configs/omap1510inn.h |    3 ++-
>>  include/configs/omap1610h2.h  |    3 ++-
>>  include/configs/omap1610inn.h |    3 ++-
>>  include/configs/omap2420h4.h  |    3 ++-
>>  include/configs/omap5912osk.h |    3 ++-
>>  include/configs/omap730p2.h   |    3 ++-
>>  include/configs/pleb2.h       |    6 ++++--
>>  13 files changed, 32 insertions(+), 18 deletions(-)
>>   
> You've update the config files for a bunch of boards, but have not added 
> board_eth_init() functions for them.  Am I missing something?  If this 
> is indeed the case, I can't push these patches upstream from the net 
> tree because the associated boards will break.
I have added the init where they have been previously used for the old 
lan91c96 driver. I think the only one which was actively using it was 
apollon, which means the rest of the platforms probably dont use the old 
lan driver properly in the first place. I have maintained status quo 
there instead of introducing a change which may break them.

-- 
Regards,
Nishanth Menon
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to