On Saturday 21 February 2009 09:28:31 pm Len Brown wrote:
> I like this patch, but not wild about the name "pci.h"
>
> there are already about 20 pci.h files in the tree.
> sure, they are context specific, but still i don't like it.
>
> (and I don't like having 11 acpi.h files in the tree either...)
>
> how about acpi_pci.h?
>
> yeah, i know it would be redundant, as it lives in drivers/acpi/
> but as basically everything it the file starts with "acpi_pci..."
> maybe a little redundancy can be seen as consistent?
Personally, I *like* the fact that we have a hierarchy where the
context provides useful information.
"ls drivers/acpi" already looks goofy because most of the names are
nice and simple, but we have the "acpi_memhotplug.c" weird thing.
I think it would be better to see "pci.h" next to the PCI-related
drivers and "processor.h" next to the processor-related drivers
rather than having "acpi_pci.h" and "acpi_processor.h" off by
themselves.
But I want the stuff out of include/acpi/acpi_drivers.h more than
I care about the new names, so if these names:
drivers/acpi/acpi_pci.h
drivers/acpi/acpi_processor.h
drivers/acpi/internal.h
work for you, I'll update the patches.
Bjorn
> On Thu, 19 Feb 2009, Bjorn Helgaas wrote:
>
> > The PCI-related declarations in acpi_drivers.h are used
> > only by the PCI root and related drivers. This patch moves
> > those declarations to a new drivers/acpi/pci.h file.
> >
> > Signed-off-by: Bjorn Helgaas <bjorn.helgaas@xxxxxx>
> > ---
> > drivers/acpi/pci.h | 18 ++++++++++++++++++
> > drivers/acpi/pci_bind.c | 2 ++
> > drivers/acpi/pci_irq.c | 2 ++
> > drivers/acpi/pci_root.c | 2 ++
> > include/acpi/acpi_drivers.h | 13 -------------
> > 5 files changed, 24 insertions(+), 13 deletions(-)
> > create mode 100644 drivers/acpi/pci.h
> >
> > diff --git a/drivers/acpi/pci.h b/drivers/acpi/pci.h
> > new file mode 100644
> > index 0000000..b3512cf
> > --- /dev/null
> > +++ b/drivers/acpi/pci.h
> > @@ -0,0 +1,18 @@
> > +/* ACPI PCI Interrupt Link (pci_link.c) */
> > +
> > +int acpi_pci_link_allocate_irq(acpi_handle handle, int index, int *triggering,
> > + int *polarity, char **name);
> > +int acpi_pci_link_free_irq(acpi_handle handle);
> > +
> > +/* ACPI PCI Interrupt Routing (pci_irq.c) */
> > +
> > +int acpi_pci_irq_add_prt(acpi_handle handle, int segment, int bus);
> > +void acpi_pci_irq_del_prt(int segment, int bus);
> > +
> > +/* ACPI PCI Device Binding (pci_bind.c) */
> > +
> > +struct pci_bus;
> > +
> > +int acpi_pci_bind(struct acpi_device *device);
> > +int acpi_pci_bind_root(struct acpi_device *device, struct acpi_pci_id *id,
> > + struct pci_bus *bus);
> > diff --git a/drivers/acpi/pci_bind.c b/drivers/acpi/pci_bind.c
> > index 95650f8..5504c9d 100644
> > --- a/drivers/acpi/pci_bind.c
> > +++ b/drivers/acpi/pci_bind.c
> > @@ -35,6 +35,8 @@
> > #include <acpi/acpi_bus.h>
> > #include <acpi/acpi_drivers.h>
> >
> > +#include "pci.h"
> > +
> > #define _COMPONENT ACPI_PCI_COMPONENT
> > ACPI_MODULE_NAME("pci_bind");
> >
> > diff --git a/drivers/acpi/pci_irq.c b/drivers/acpi/pci_irq.c
> > index be6b909..c20c848 100644
> > --- a/drivers/acpi/pci_irq.c
> > +++ b/drivers/acpi/pci_irq.c
> > @@ -40,6 +40,8 @@
> > #include <acpi/acpi_bus.h>
> > #include <acpi/acpi_drivers.h>
> >
> > +#include "pci.h"
> > +
> > #define _COMPONENT ACPI_PCI_COMPONENT
> > ACPI_MODULE_NAME("pci_irq");
> >
> > diff --git a/drivers/acpi/pci_root.c b/drivers/acpi/pci_root.c
> > index 5b38a02..61e8979 100644
> > --- a/drivers/acpi/pci_root.c
> > +++ b/drivers/acpi/pci_root.c
> > @@ -36,6 +36,8 @@
> > #include <acpi/acpi_bus.h>
> > #include <acpi/acpi_drivers.h>
> >
> > +#include "pci.h"
> > +
> > #define _COMPONENT ACPI_PCI_COMPONENT
> > ACPI_MODULE_NAME("pci_root");
> > #define ACPI_PCI_ROOT_CLASS "pci_bridge"
> > diff --git a/include/acpi/acpi_drivers.h b/include/acpi/acpi_drivers.h
> > index 5fc1bb0..9ea2684 100644
> > --- a/include/acpi/acpi_drivers.h
> > +++ b/include/acpi/acpi_drivers.h
> > @@ -75,23 +75,10 @@
> > /* ACPI PCI Interrupt Link (pci_link.c) */
> >
> > int acpi_irq_penalty_init(void);
> > -int acpi_pci_link_allocate_irq(acpi_handle handle, int index, int *triggering,
> > - int *polarity, char **name);
> > -int acpi_pci_link_free_irq(acpi_handle handle);
> > -
> > -/* ACPI PCI Interrupt Routing (pci_irq.c) */
> > -
> > -int acpi_pci_irq_add_prt(acpi_handle handle, int segment, int bus);
> > -void acpi_pci_irq_del_prt(int segment, int bus);
> >
> > /* ACPI PCI Device Binding (pci_bind.c) */
> >
> > -struct pci_bus;
> > -
> > acpi_status acpi_get_pci_id(acpi_handle handle, struct acpi_pci_id *id);
> > -int acpi_pci_bind(struct acpi_device *device);
> > -int acpi_pci_bind_root(struct acpi_device *device, struct acpi_pci_id *id,
> > - struct pci_bus *bus);
> >
> > /* Arch-defined function to add a bus to the system */
> >
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
