On Thursday 12 November 2009 08:35:08 Mike Rapoport wrote:
> 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.
> 
> Hope this one is better:
> 
> From 4a9420704dd81a08f950017d365e0826880536ed Mon Sep 17 00:00:00 2001
> From: Mike Rapoport <m...@compulab.co.il>
> Date: Tue, 10 Nov 2009 15:31:46 +0200
> Subject: [PATCH] smc911x: make smc911x_initialize return correct value
> 
> Make smc911x_initialize return -1 on error and number of interfaces
> detected otherwise.

Acked-by: Mike Frysinger <vap...@gentoo.org>
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to