linux-toradex.git/drivers/acpi/pci_link.c, branch v2.6.32.21 Linux kernel for Apalis and Colibri modules ACPI: Move definition of PREFIX from acpi_bus.h to internal..h 2009-08-28T23:57:27+00:00 Len Brown len.brown@intel.com 2009-07-28T20:45:54+00:00 a192a9580bcc41692be1f36b77c3b681827f566a Linux/ACPI core files using internal.h all PREFIX "ACPI: ", however, not all ACPI drivers use/want it -- and they should not have to #undef PREFIX to define their own. Add GPL commment to internal.h while we are there. This does not change any actual console output, asside from a whitespace fix. Signed-off-by: Len Brown <len.brown@intel.com> 
Linux/ACPI core files using internal.h all PREFIX "ACPI: ",
however, not all ACPI drivers use/want it -- and they
should not have to #undef PREFIX to define their own.

Add GPL commment to internal.h while we are there.

This does not change any actual console output,
asside from a whitespace fix.

Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: pci_link: simplify list of link devices 2009-03-17T05:53:29+00:00 Bjorn Helgaas bjorn.helgaas@hp.com 2009-02-17T21:00:55+00:00 5f0dccaa81e239477413d0def1133850530f1bbe We don't need a struct containing a count and a list_head; a simple list_head is sufficient. The list iterators handle empty lists fine. Furthermore, we don't need to check for null list entries because we only add non-null entries. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com> 
We don't need a struct containing a count and a list_head; a simple
list_head is sufficient.  The list iterators handle empty lists
fine.

Furthermore, we don't need to check for null list entries because we
only add non-null entries.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: pci_link: remove unnecessary null pointer checks 2009-03-17T05:52:40+00:00 Bjorn Helgaas bjorn.helgaas@hp.com 2009-02-17T21:00:50+00:00 6eca4b4ca168981d7648be371945c2a21f463a45 Better to oops and learn about a bug than to silently cover it up. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com> 
Better to oops and learn about a bug than to silently cover it up.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: pci_link: remove unnecessary casts and initializations 2009-03-17T05:52:18+00:00 Bjorn Helgaas bjorn.helgaas@hp.com 2009-02-17T21:00:45+00:00 c9d6244329c8149312dba27e78dc4a83b35a6ae5 Remove unnecessary casts and initializations. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com> 
Remove unnecessary casts and initializations.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: pci_link: clean up whitespace 2009-03-17T05:51:36+00:00 Bjorn Helgaas bjorn.helgaas@hp.com 2009-02-17T21:00:40+00:00 1c9ca3a7d41b5db884033900b539b9aeb61a399e This patch makes whitespace and indentation more consistent. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com> 
This patch makes whitespace and indentation more consistent.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: add missing KERN_* constants to printks 2009-02-07T05:29:32+00:00 Frank Seidel frank@f-seidel.de 2009-02-04T16:03:07+00:00 4d9391557b68475b118ec7626607c37b14ae8c16 According to kerneljanitors todo list all printk calls (beginning a new line) should have an according KERN_* constant. Those are the missing peaces here for the acpi subsystem. Signed-off-by: Frank Seidel <frank@f-seidel.de> Signed-off-by: Len Brown <len.brown@intel.com> 
According to kerneljanitors todo list all printk calls (beginning
a new line) should have an according KERN_* constant.
Those are the missing peaces here for the acpi subsystem.

Signed-off-by: Frank Seidel <frank@f-seidel.de>
Signed-off-by: Len Brown <len.brown@intel.com>
Merge branch 'misc' into release 2009-01-09T08:37:48+00:00 Len Brown len.brown@intel.com 2009-01-09T08:37:48+00:00 d0302bc62af7983040ea1cd1b5dd2b36e1a6a509 Conflicts: include/acpi/acpixf.h Signed-off-by: Len Brown <len.brown@intel.com> 
Conflicts:
	include/acpi/acpixf.h

Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: ec.c, pci_link.c, video_detec.c: static 2008-12-31T04:27:04+00:00 Roel Kluin roel.kluin@gmail.com 2008-12-09T19:45:30+00:00 8a383ef0be01e0e6e84c85f8bf35e4e6fcfb8981 Sparse asked whether these could be static. Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: Len Brown <len.brown@intel.com> 
Sparse asked whether these could be static.

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: Do not modify SCI_EN directly 2008-12-31T00:02:35+00:00 Rafael J. Wysocki rjw@sisk.pl 2008-12-29T18:19:07+00:00 11e93130c7ce5228d484fd5e86f3984835d4256b According to the ACPI specification the SCI_EN flag is controlled by the hardware, which sets this flag to inform the kernel that ACPI is enabled. For this reason, we shouldn't try to modify SCI_EN directly. Also, we don't need to do it in irqrouter_resume(), since lower-level resume code takes care of enabling ACPI in case it hasn't been enabled by the BIOS before passing control to the kernel (which by the way is against the ACPI specification). Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> Signed-off-by: Len Brown <len.brown@intel.com> 
According to the ACPI specification the SCI_EN flag is controlled by
the hardware, which sets this flag to inform the kernel that ACPI is
enabled.  For this reason, we shouldn't try to modify SCI_EN
directly.  Also, we don't need to do it in irqrouter_resume(), since
lower-level resume code takes care of enabling ACPI in case it hasn't
been enabled by the BIOS before passing control to the kernel (which
by the way is against the ACPI specification).

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Signed-off-by: Len Brown <len.brown@intel.com>
ACPI: pci_link: remove acpi_irq_balance_set() interface 2008-11-12T02:12:05+00:00 Bjorn Helgaas bjorn.helgaas@hp.com 2008-11-05T23:17:52+00:00 32836259ff25ce97010569706cd33ba94de81d62 This removes the acpi_irq_balance_set() interface from the PCI interrupt link driver. x86 used acpi_irq_balance_set() to tell the PCI interrupt link driver to configure links to minimize IRQ sharing. But the link driver can easily figure out whether to turn on IRQ balancing based on the IRQ model (PIC/IOAPIC/etc), so we can get rid of that external interface. It's better for the driver to figure this out at init-time. If we set it externally via the x86 code, the interface reduces modularity, and we depend on the fact that acpi_process_madt() happens before we process the kernel command line. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com> 
This removes the acpi_irq_balance_set() interface from the PCI
interrupt link driver.

x86 used acpi_irq_balance_set() to tell the PCI interrupt link
driver to configure links to minimize IRQ sharing.  But the link
driver can easily figure out whether to turn on IRQ balancing
based on the IRQ model (PIC/IOAPIC/etc), so we can get rid of
that external interface.

It's better for the driver to figure this out at init-time.  If
we set it externally via the x86 code, the interface reduces
modularity, and we depend on the fact that acpi_process_madt()
happens before we process the kernel command line.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>