[RFC] quirk_ich4_lpc_acpi() clarification

From: Stefan Bauer
Date: Sat Jun 07 2008 - 13:21:22 EST


Hi,


I was wondering why my quite ancient i815-based machine with ICH (not ICH4)
southbridge told me on startup:

pci 0000:00:1f.0: quirk: region 4000-407f claimed by ICH4 ACPI/GPIO/TCO
pci 0000:00:1f.0: quirk: region 4080-40bf claimed by ICH4 GPIO

Some search brought up that this quirk was originally registered for ICH4 and
ICH5 [1], but later extended to apply for ICH to ICH3, too [2].

The comment added by Linus in [2], telling "Intel LPC interface bridges all
have 128 bytes of magic ACPI/TCO regs and 64 bytes of GPIO", unfortunately
got lost with [3].

So in current state we have some uncertainty: Does "all" mean "all until now"
or "all ever using LPC"? In the last case not only comments, but also
DECLARE_PCI_FIXUP_HEADERs for the past-ICH5 chipsets would be missing.

I would like to fix this by...
- mentioning all affected ICHs in the comment
- probably give a better 'name' parameter then "ICH4 ..." to quirk_io_region()
- probably rename quirk_ich4_lpc_acpi()
- add DECLARE_PCI_FIXUP_HEADERs for eventually missing devices

But I need some information, especially if past-ICH5 southbridges have got
this quirk, too.


Regards, Stefan Bauer

[1] Add a quirk for the Intel ICH-[45] to add special ACPI regions.
http://git.kernel.org/?p=linux/kernel/git/torvalds/old-2.6-bkcvs.git;a=blobdiff;f=drivers/pci/quirks.c;h=0fe0f7656939ba8b2b55e55d2ac4fbe12dee59fa;hp=29313362a8b663ee85a43bcd7f70220bd64598e4;hb=0b7baee8e18d8d309b3918769fc4b60dc075baa5;hpb=7886b5151d14bde184e490201ad51a3f413df8d9

[2] All the Intel LPC bridges have the same PCI quirks.
http://git.kernel.org/?p=linux/kernel/git/torvalds/old-2.6-bkcvs.git;a=blobdiff;f=drivers/pci/quirks.c;h=b82a1c31e1df64a8aedb5c868d4e62979cc5fc19;hp=2a2a2f11f3d1a6732c4d119efc62cf74a174b4f8;hb=ca2a9e5a6a068b446ba0829385f9785695df9182;hpb=3b4871d830f2ace4c6dafe33272e0bb2ec8f83fb

[3] Merge http://nfsclient.bkbits.net/linux-2.6
http://git.kernel.org/?p=linux/kernel/git/torvalds/old-2.6-bkcvs.git;a=blobdiff;f=drivers/pci/quirks.c;h=164f738df6a7cb1c58f1033382530780a211e039;hp=6e7c350133becd05268d1cb6fa6073019c63457b;hb=ee66a188d8cd63c8baf60f7be2a24faf9d556bbd;hpb=9a1ad97be320d53eba3cdb6a69fbbea27a862461
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/