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 <n...@ti.com>
>> Cc: Andrea Scian <andrea.sc...@dave-tech.it>
>> Cc: Ben Warren <biggerbadder...@gmail.com>
>> Cc: Dave Peverley <dpever...@mpc-data.co.uk>
>> Cc: George G. Davis <gda...@mvista.com>
>> Cc: Kyungmin Park <kyungmin.p...@samsung.com>
>> Cc: Nishant Kamat <nska...@ti.com>
>> Cc: Richard Woodruff <r-woodru...@ti.com>
>> Cc: Rishi Bhattacharya <ri...@ti.com>
>> Cc: Sandeep Paulraj <s-paul...@ti.com>
>> ---
>>  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
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to