These are a few cleanups for pciehp to make it easier to read. There shouldn't be any change in behavior except a dmesg change or two. Rajat, I know these might conflict with what you're doing, but it shouldn't be major. I can fix up any conflicts for you, or if it makes your life easier, you can rebase on top of my pci/pciehp branch. --- Bjorn Helgaas (8): PCI: pciehp: Drop pciehp_readw()/pciehp_writew() wrappers PCI: pciehp: Remove error checks when accessing PCIe Capability PCI: pciehp: Make various functions void since they can't fail PCI: pciehp: Announce slot capabilities (slot #, button, LEDs, etc) PCI: pciehp: Simplify "Power Fault Detected" checking/clearing PCI: pciehp: Use symbolic constants, not hard-coded bitmask PCI: pciehp: Use symbolic constants for Slot Control fields PCI: pciehp: Move Attention & Power Indicator support tests to accessors drivers/pci/hotplug/pciehp.h | 14 + drivers/pci/hotplug/pciehp_core.c | 15 + drivers/pci/hotplug/pciehp_ctrl.c | 90 +++------ drivers/pci/hotplug/pciehp_hpc.c | 380 ++++++++++++------------------------- drivers/pci/hotplug/pciehp_pci.c | 6 - include/uapi/linux/pci_regs.h | 8 + 6 files changed, 185 insertions(+), 328 deletions(-) -- 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