On Tue, Sep 20, 2011 at 9:08 PM, Greg KH <greg@xxxxxxxxx> wrote: > 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() Andiry, see this patch: https://lkml.org/lkml/2011/4/19/145 regards Cliff > -- > 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 > -- 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