On Tue, Sep 20, 2011 at 04:18:04PM +0800, Andiry Xu wrote: > On Fri, 2011-09-09 at 09:11 -0700, Greg KH wrote: > > On Fri, Sep 09, 2011 at 05:57:38PM +0800, Andiry Xu wrote: > > > +static const char enable_string[] = "enable"; > > > +static const char disable_string[] = "disable"; > > > > I think we have a "generic" way of doing "yes/no enable/disable 0/1 T/F" > > now with sysfs, so please use that and not these strings. > > > > I wonder what the "generic" way is? I can see driver/base/power/sysfs.c > using enabled/disabled strings, and some other sysfs files using 0/1 > directly. Can you specify the generic way you mentioned or point to me > some code I can refer to? Thanks. strtobool() -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html