mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [RFC] quirk_ich4_lpc_acpi() clarification
@ 2008-06-07 16:49 Stefan Bauer
  0 siblings, 0 replies; only message in thread
From: Stefan Bauer @ 2008-06-07 16:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: Nakajima, Jun, Linus Torvalds

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2008-06-07 17:21 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-06-07 16:49 [RFC] quirk_ich4_lpc_acpi() clarification Stefan Bauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®