[lkml]   [2009]   [Jun]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[PATCH v2 00/11] Dynamic ACPI-PCI binding
Hi Len,

I hope this isn't too late for the .31 merge window. If so, no big deal;
this patchset isn't urgent, it's just a nice cleanup that was inspired
from investigating the recent bug in acpi_pci_bind() (fixed by dacd254).

This patch series eliminates static boot-time binding of ACPI
and PCI devices, and introduces an API to perform this lookup
during runtime.

This change has the following advantages:

- eliminates struct acpi_device vs struct pci_dev lifetime issues
- lays groundwork for eliminating .bind/.unbind from acpi_device_ops
- lays more groundwork for eliminating .start from acpi_device_ops
and thus simplifying ACPI drivers
- whacks out a lot of code

This patchset is based on lenb/test (66c74fa1d4), and has been boot
tested on ia64 and x86. I also performed physical PCI hotplug tests
using acpiphp on ia64. I do not have an acpiphp-capable x86 platform.

I didn't test the changes in the video driver either, as I don't have
the hardware.

v1 -> v2
- rearrange series into a more logical order
- much simpler acpi_is_root_bridge() implementation
- no longer export acpi_pci_find_root()
- no longer leak memory in acpi_get_pci_dev()
- no longer leak references in acpi_pci_unbind/acpi_pci_bind
- convert video driver to use acpi_get_pci_dev()
- kill off acpi_get_physical_pci_device()
- incorporate Bjorn's other comments. :)


Alex Chiang (11):
ACPI: kill acpi_get_physical_pci_device()
ACPI: video: convert to acpi_get_pci_dev
ACPI: kill acpi_get_pci_id
PCI Hotplug: acpiphp: convert to acpi_get_pci_dev
ACPI: acpi_pci_unbind should clean up properly after acpi_pci_bind
ACPI: simplify acpi_pci_irq_del_prt() API
ACPI: simplify acpi_pci_irq_add_prt() API
ACPI: eviscerate pci_bind.c
ACPI: Introduce acpi_get_pci_dev()
ACPI: Introduce acpi_is_root_bridge()
ACPI: make acpi_pci_bind() static

drivers/acpi/glue.c | 40 -----
drivers/acpi/pci_bind.c | 315 +++++-------------------------------
drivers/acpi/pci_irq.c | 17 +-
drivers/acpi/pci_root.c | 112 ++++++++++++-
drivers/acpi/video.c | 6 -
drivers/acpi/video_detect.c | 9 +
drivers/pci/hotplug/acpi_pcihp.c | 40 -----
drivers/pci/hotplug/acpiphp_glue.c | 27 +--
include/acpi/acpi_bus.h | 2
include/acpi/acpi_drivers.h | 10 -
include/linux/pci_hotplug.h | 1
11 files changed, 179 insertions(+), 400 deletions(-)

 \ /
  Last update: 2009-06-04 08:01    [W:0.094 / U:25.940 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site