Re: [PATCH 2/3] PCI hotplug: create symlink to hotplug driver module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>:
> Create symbolic link to hotplug driver module in the PCI slot
> directory (/sys/bus/pci/slots/<SLOT#>). In the past, we need to load
> hotplug drivers one by one to identify the hotplug driver that handles
> the slot, and it was very inconvenient especially for trouble shooting.
> With this change, we can easily identify the hotplug driver.
> 
> Signed-off-by: Taku Izumi <izumi.taku@xxxxxxxxxxxxxx>
> Signed-off-by: Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>
> 
> ---
>  Documentation/ABI/testing/sysfs-bus-pci |    7 +++++

Hello Kenji-san,

> Index: 20090612/Documentation/ABI/testing/sysfs-bus-pci
> ===================================================================
> --- 20090612.orig/Documentation/ABI/testing/sysfs-bus-pci
> +++ 20090612/Documentation/ABI/testing/sysfs-bus-pci
> @@ -122,3 +122,10 @@ Description:
>  		This symbolic link appears when a device is a Virtual Function.
>  		The symbolic link points to the PCI device sysfs entry of the
>  		Physical Function this device associates with.
> +
> +What:		/sys/bus/pci/slots/.../module
> +Date:		June 2009
> +Contact:	Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>

Are you sure you want to put your contact information here?
Typically, I've used the linux-pci@xxxxxxxxxxxxxxx list as the
contact when documenting new files in Documentation/ABI/

Anyhow, if you want to keep it this way, I'm fine with it.

Reviewed-by: Alex Chiang <achiang@xxxxxx>

> +Description:
> +		This symbolic link points to the PCI hotplug controller driver
> +		module that manages the hotplug slot.
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux