[U-Boot] [PATCH] smc911x: make smc911x_initialize return correct value (Was: Re: [PATCH 1/5] smc911x: return -1 when initialization fails)

[Mike Rapoport]

Mike Frysinger wrote:
> On Wednesday 11 November 2009 03:03:00 Mike Rapoport wrote:
>> --- a/drivers/net/smc911x.c
>> +++ b/drivers/net/smc911x.c
>> @@ -243,7 +243,7 @@
>>  	dev = malloc(sizeof(*dev));
>>  	if (!dev) {
>>  		free(dev);
>> -		return 0;
>> +		return -1;
>>  	}
> 
> this is correct as this is an error
> 
>> @@ -252,7 +252,7 @@
>>  	/* Try to detect chip. Will fail if not present. */
>>  	if (smc911x_detect_chip(dev)) {
>>  		free(dev);
>> -		return 0;
>> +		return -1;
>>  	}
> 
> this is not -- we want it to return 0 if no parts are found.  see recent net 
> doc updates and discussions.
> -mike

Hope this one is better: