Messages in this thread | | | Date | Thu, 3 Dec 2015 13:58:40 +0100 | From | LABBE Corentin <> | Subject | Re: [PATCH v3 1/1] atm: solos-pci: Replace simple_strtol by kstrtoint |
| |
On Thu, Dec 03, 2015 at 06:26:31AM -0500, Charles (Chas) Williams wrote: > On Thu, 2015-12-03 at 09:06 +0100, LABBE Corentin wrote: > > @@ -357,11 +357,11 @@ static int process_status(struct solos_card *card, int port, struct sk_buff *skb > > if (!str) > > return -EIO; > > > > - ver = simple_strtol(str, NULL, 10); > > - if (ver < 1) { > > + err = kstrtoint(str, 10, &ver); > > + if (err || ver < 1) { > > dev_warn(&card->dev->dev, "Unexpected status interrupt version %d\n", > > ver); > > - return -EIO; > > + return err; > > > If ver < 1 then you might return a 0 here. Always returning -EIO is > probably just fine. >
Hello
I think the best solution is to split the test, since returning error code from kstrtoint was asked by David Miller. if (err) return err; if (ver < 1) return -EIO; Thanks Regards
| |