[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120223115747.4b85fdb2@jbarnes-desktop>
Date: Thu, 23 Feb 2012 11:57:47 -0800
From: Jesse Barnes <jbarnes@...tuousgeek.org>
To: Yinghai Lu <yinghai@...nel.org>
Cc: Ram Pai <linuxram@...ibm.com>,
Dominik Brodowski <linux@...inikbrodowski.net>,
Linus Torvalds <torvalds@...ux-foundation.org>,
linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 5/7] PCI: Make pci bridge reallocating enabled/disabled
On Fri, 10 Feb 2012 15:33:50 -0800
Yinghai Lu <yinghai@...nel.org> 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
Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)
Powered by blists - more mailing lists