|From:||Alex Chiang <firstname.lastname@example.org>|
|To:||Gary Hade <email@example.com>, Matthew Wilcox <firstname.lastname@example.org>, email@example.com, firstname.lastname@example.org, email@example.com, firstname.lastname@example.org, email@example.com, firstname.lastname@example.org|
|Subject:||[PATCH 0/4, v3] Physical PCI slot objects|
|Date:||Sat, 17 Nov 2007 11:29:54 -0700|
Hi all, This is v3 of the pci_slot patch series. The major change is making the ACPI-PCI slot driver a Kconfig option, as per the recommendations of others (Gary, Kenji-san). In the process of doing so, it made sense to collapse the former 3/5 and 4/5 patches into a single 3/4 patch. There really wasn't a reason to introduce a pci_slot patch, and then immediately follow it with another patch modifying its interface; logically, the changes should have been in the same patch. Combining the patches also has the nice side benefit of keeping the tree fully buildable and bisectable at all stages of series. I have done quite a bit more testing, and verified that this series plays nicely with acpiphp during all stages of the series. Notably, you can modprobe/rmmod acpiphp repeatedly no matter where you are in the series, and no matter whether you have CONFIG_ACPI_PCI_SLOT turned on. The correct entries in /sys/bus/pci/slots/ will appear and disappear, and we correctly register/deregister ACPI slots with the pci_hp core. Of course, if you *do* have the ACPI-PCI slot driver configured, the slots/ entries in sysfs will be persistent. What you will see is the hotplug attributes appear/disappear, depending on whether you have acpiphp loaded or not. Thanks for your consideration and all the feedback comments. They're appreciated. /ac v2 -> v3: Patch 1/4 - no change Patch 2/4 - incorporate Eike's comments around snprintf Patch 3/4 - Separated slot creation and slot hotplug ability into two interfaces. Fixed bugs in pci_destroy_slot(), and now properly calling from pci_hp_deregister. Patch 4/4 - Add Kconfig option to driver, allowing users to [de]config this driver. If configured, take slightly different code paths in pci_hp_register and pci_hp_deregister. v1 -> v2: Patch 1/5 - reworked to fix stupid compile bug Patch 2/5 - incorporate Eike, Linas, and Willy's comments Patch 3/5 - no change Patch 4/5 - was acpi-pci-slot-driver patch, now modifies pci_add_hotplug(). I changed the ordering on this so the tree doesn't break at this point in the series Patch 5/5 - now is acpi-pci-slot-driver patch, cleaned up implementation so our slot detection is a little better
Copyright © 2007, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds