The sysfs enabled value is a boolean, so kstrtobool() is a better fit
for parsing the input string since it does the range checking for us.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Julian Wiedmann <jwi@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
                struct device_attribute *attr, const char *buf, size_t count)
 {
        struct qeth_card *card = dev_get_drvdata(dev);
-       int rc = 0;
-       int enable;
+       bool enable;
+       int rc;
 
        if (!card)
                return -EINVAL;
 
-       if (sysfs_streq(buf, "0"))
-               enable = 0;
-       else if (sysfs_streq(buf, "1"))
-               enable = 1;
-       else
-               return -EINVAL;
+       rc = kstrtobool(buf, &enable);
+       if (rc)
+               return rc;
 
        mutex_lock(&card->conf_mutex);