[U-Boot] [PATCH] KS8695: convert KS8695 eth driver to CONFIG_MULTI_ETH

Greg Ungerer greg.ungerer at opengear.com
Mon Sep 12 14:39:44 CEST 2011


On 09/12/2011 05:15 AM, Mike Frysinger wrote:
> On Saturday, September 10, 2011 04:40:34 Greg Ungerer wrote:
>> --- a/include/netdev.h
>> +++ b/include/netdev.h
>>
>> +int ks8695_eth_initialize(bd_t *bis);
>
> so you say the func needs bd_t* ...
>
>> --- a/board/cm4008/cm4008.c
>> +++ b/board/cm4008/cm4008.c
>> --- a/board/cm41xx/cm41xx.c
>> +++ b/board/cm41xx/cm41xx.c
>>
>> +int board_eth_init(bd_t *bis)
>> +{
>> +	return ks8695_eth_initialize();
>> +}
>
> but you dont pass it in the board funcs ...
>
>> --- a/drivers/net/ks8695eth.c
>> +++ b/drivers/net/ks8695eth.c
>>
>> +int ks8695_eth_initialize(void)
>
> nor does the func itself even take it ...
>
> not sure how this compiles ?  do none of these files include netdev.h ?  if
> they don't, i don't see how it compiles cleanly as that means you'd get
> warnings about implicit func prototypes ...

It compiled with a warning. Wolfgang fixed it in a follow-up patch.

Regards
Greg




More information about the U-Boot mailing list