Re: [PATCH 5/7] PCI: Make pci bridge reallocating enabled/disabled
From: Jesse Barnes
Date: Thu Feb 23 2012 - 14:57:52 EST
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