The patch titled
Cardbus cards hidden, fixup parent subordinate# carefully
has been removed from the -mm tree. Its filename is
cardbus-cards-hidden-fixup-parent-subordinate-carefully.patch
This patch was probably dropped from -mm because
it has now been merged into a subsystem tree or
into Linus's tree, or because it was folded into
its parent patch in the -mm tree.
From: Bernhard Kaindl <bk@xxxxxxx>
There are currently two ways to fix the problem of hidden cardbus cards:
One is to use
setpci -s 0:a.0 SUBORDINATE_BUS=0x0A (or similar)
On Presario R3000/R4000 series and HP zv5000 series, this is the standard.
Using pci=assign-busses is the other, but as ACPI may contain references to
PCI bus numbers and as files like xorg.conf file can also contain
references to PCI devices, it can break installations:
https://bugzilla.novell.com/show_bug.cgi?id=146438
It would also be a big change to enable it by default on many machines.
The other possiblity is to fixup the parent subordinate numbers after we
completed the initial PCI scan, so that we can do checks to prevent
overlapping bus numbers when trying to increase the subordinate number.
I created a kernel module for testing code which does that and it was
tested on Presario R3000 laptops and a Samsung X20 notebook.
I then trimmed it down to call it from the CardBus probe function, before
the socket is registered and the bus(es) below it are scanned.
>From that location, we can directly check the parent of the CardBus bridge
and that simplifies the code dramatically, so I could merge it into one
small function.
It was tested on my X20 and (it seems) a R3000 series notebook:
https://bugzilla.novell.com/show_bug.cgi?id=146438#c31
Conclusion:
-----------
We apparently have a better solution, and since the full PCI renumbering
could cause problems which I mentioned above, I think we should revert my
previous patch which was committed to 2.6.17-rc1:
commit 8c4b2cf9af9b4ecc29d4f0ec4ecc8e94dc4432d7
Author: Bernhard Kaindl <bk@xxxxxxx>
Date: Sat Feb 18 01:36:55 2006 -0800
Regarding the waring message patched in that commit: It gives a few
(partly) false warnings, because it's not specific enough in its tests. So
that part should be reverted too and I'll propose a fix for this warning
check, preferably depending on this patch (we could skip cardbus there
then), in separate mail.
Short patch comment:
Fixup the subordinate number of the parent of CardBus bridges carefully
before the the slots are scanned by PCI, to make the CardBus cards and
child busses which may otherwise be hidden from PCI scans seen.
Affected systems (from an internet search), supposed to be fixed:
* ASUS Z71V and L3s
* Samsung X20 (fixed in latest BIOS, but older BIOSes are affected)
* Compaq R3140us and all Compaq R3000 series (AMD64-based Laptops),
also Compaq R4000 series
* HP zv5000z (AMD64 3700+, known that fixup_parent_subordinate_busnr fixes it)
* HP zv5200z
* HP SE2000L amd64 Turino laptop
* IBM ThinkPad 240
* An IBM ThinkPad (1.8 GHz Pentium M) debugged by Pavel Machek
gives the message which detects the problem.
* MSI S260 / Medion SIM 2100 MD 95600
Tested-by: Andreas Schneider <mail@xxxxxxxxxxxx>
Signed-off-by: Bernhard Kaindl <bk@xxxxxxx>
Signed-off-by: Dominik Brodowski <linux@xxxxxxxxxxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
drivers/pcmcia/yenta_socket.c | 74 ++++++++++++++++++++++++++++++++
1 files changed, 74 insertions(+)
diff -puN drivers/pcmcia/yenta_socket.c~cardbus-cards-hidden-fixup-parent-subordinate-carefully drivers/pcmcia/yenta_socket.c
--- devel/drivers/pcmcia/yenta_socket.c~cardbus-cards-hidden-fixup-parent-subordinate-carefully 2006-04-14 23:41:40.000000000 -0700
+++ devel-akpm/drivers/pcmcia/yenta_socket.c 2006-04-14 23:41:40.000000000 -0700
@@ -1076,6 +1076,78 @@ static void yenta_fixup_parent_bridge(st
}
}
+/**
+ * yenta_fixup_parent_subordinate - Fix subordinate bus# of the parent bridge
+ * @cardbus_bridge: The PCI bus which the CardBus bridge bridges to
+ *
+ * Checks if devices on the bus which the CardBus bridge bridges to would be
+ * invisible during PCI scans because of a misconfigured subordinate number
+ * of the parent brige - some BIOSes seem to be too lazy to set it right.
+ * Does the fixup carefully by checking how far it can go without conflicts.
+ * See http://bugzilla.kernel.org/show_bug.cgi?id=2944 for more information.
+ */
+static void yenta_fixup_parent_bridge(struct pci_bus *cardbus_bridge)
+{
+ struct list_head *tmp;
+ /* Our starting point is the max PCI bus number */
+ unsigned char upper_limit = 0xff;
+ /*
+ * We only check and fix the parent bridge: All systems which need
+ * this fixup that have been reviewed are laptops and the only bridge
+ * which needed fixing was the parent bridge of the CardBus bridge:
+ */
+ struct pci_bus *bridge_to_fix = cardbus_bridge->parent;
+
+ /* Check bus numbers are already set up correctly: */
+ if (bridge_to_fix->subordinate >= cardbus_bridge->subordinate)
+ return; /* The subordinate number is ok, nothing to do */
+
+ if (!bridge_to_fix->parent)
+ return; /* Root bridges are ok */
+
+ /* stay within the limits of the bus range of the parent: */
+ upper_limit = bridge_to_fix->parent->subordinate;
+
+ /* check the bus ranges of all silbling bridges to prevent overlap */
+ list_for_each(tmp, &bridge_to_fix->parent->children) {
+ struct pci_bus * silbling = pci_bus_b(tmp);
+ /*
+ * If the silbling has a higher secondary bus number
+ * and it's secondary is equal or smaller than our
+ * current upper limit, set the new upper limit to
+ * the bus number below the silbling's range:
+ */
+ if (silbling->secondary > bridge_to_fix->subordinate
+ && silbling->secondary <= upper_limit)
+ upper_limit = silbling->secondary - 1;
+ }
+
+ /* Show that the wanted subordinate number is not possible: */
+ if (cardbus_bridge->subordinate > upper_limit)
+ printk(KERN_WARNING "Yenta: Upper limit for fixing this "
+ "bridge's parent bridge: #%02x\n", upper_limit);
+
+ /* If we have room to increase the bridge's subordinate number, */
+ if (bridge_to_fix->subordinate < upper_limit) {
+
+ /* use the highest number of the hidden bus, within limits */
+ unsigned char subordinate_to_assign =
+ min(cardbus_bridge->subordinate, upper_limit);
+
+ printk(KERN_INFO "Yenta: Raising subordinate bus# of parent "
+ "bus (#%02x) from #%02x to #%02x\n",
+ bridge_to_fix->number,
+ bridge_to_fix->subordinate, subordinate_to_assign);
+
+ /* Save the new subordinate in the bus struct of the bridge */
+ bridge_to_fix->subordinate = subordinate_to_assign;
+
+ /* and update the PCI config space with the new subordinate */
+ pci_write_config_byte(bridge_to_fix->self,
+ PCI_SUBORDINATE_BUS, bridge_to_fix->subordinate);
+ }
+}
+
/*
* Initialize a cardbus controller. Make sure we have a usable
* interrupt, and that we can map the cardbus area. Fill in the
@@ -1193,6 +1265,8 @@ static int __devinit yenta_probe (struct
yenta_fixup_parent_bridge(dev->subordinate);
+ yenta_fixup_parent_bridge(dev->subordinate);
+
/* Register it with the pcmcia layer.. */
ret = pcmcia_register_socket(&socket->socket);
if (ret == 0) {
_
Patches currently in -mm which might be from bk@xxxxxxx are
git-pcmcia.patch
revert-pci-pci-cardbus-cards-hidden-needs-pci=assign-busses-to-fix.patch
cardbus-cards-hidden-fixup-parent-subordinate-carefully.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
