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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
