Re: [PATCH 16/16 v6] PCI: document the new PCI boot parameters

From: Bjorn Helgaas
Date: Wed Oct 22 2008 - 10:27:52 EST


On Wednesday 22 October 2008 02:45:31 am Yu Zhao wrote:
> Document the new PCI[x86] boot parameters.
>
> Cc: Alex Chiang <achiang@xxxxxx>
> Cc: Grant Grundler <grundler@xxxxxxxxxxxxxxxx>
> Cc: Greg KH <greg@xxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxx>
> Cc: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>
> Cc: Matthew Wilcox <matthew@xxxxxx>
> Cc: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
> Cc: Roland Dreier <rdreier@xxxxxxxxx>
> Signed-off-by: Yu Zhao <yu.zhao@xxxxxxxxx>
>
> ---
> Documentation/kernel-parameters.txt | 10 ++++++++++
> 1 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> index 53ba7c7..5482ae0 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -1677,6 +1677,16 @@ and is between 256 and 4096 characters. It is defined in the file
> cbmemsize=nn[KMG] The fixed amount of bus space which is
> reserved for the CardBus bridge's memory
> window. The default value is 64 megabytes.
> + assign-mmio=[dddd:]bb [X86] reassign memory resources of all
> + devices under bus [dddd:]bb (dddd is the domain
> + number and bb is the bus number).
> + assign-pio=[dddd:]bb [X86] reassign io port resources of all
> + devices under bus [dddd:]bb (dddd is the domain
> + number and bb is the bus number).
> + align-mmio=[dddd:]bb:dd.f [X86] relocate memory resources of a
> + device to minimum PAGE_SIZE alignment (dddd is
> + the domain number and bb, dd and f is the bus,
> + device and function number).
>
> pcie_aspm= [PCIE] Forcibly enable or disable PCIe Active State Power
> Management.

I think it's nicer to have the documentation change included in the
patch that implements the change. For example, I think this and
patch 9/16 "add boot option to align ..." should be folded into
a single patch. And similarly for the other documentation patches.

Bjorn

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/