On Fri, 10 Feb 2012 15:33:50 -0800 Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > +enum enable_type { > + enable_not_set = -1, > + enable_no_user, > + enable_no_detected, > + enable_yes_user, > + enable_yes_detected, > +}; How about: user_enabled user_disabled auto_enabled auto_disabled ? enable_no_* seems a bit confusing... also the help text in kernel-parameters.txt should mention the config option and describe default behavior with and without that config option enabled. Thanks, -- Jesse Barnes, Intel Open Source Technology Center
Attachment:
signature.asc
Description: PGP signature