diff options
author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-03-22 10:13:57 -0700 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-03-22 22:14:07 -0700 |
commit | 1090f346713ae43319cfeb3eb65ca24f9864b628 (patch) | |
tree | 5ba34b11097a59b5f6e4446fe41b64681c820067 /platforms | |
parent | dcbf18c1f0e16111d5c86e845f37e8c2cad4f9ea (diff) | |
download | skiboot-1090f346713ae43319cfeb3eb65ca24f9864b628.zip skiboot-1090f346713ae43319cfeb3eb65ca24f9864b628.tar.gz skiboot-1090f346713ae43319cfeb3eb65ca24f9864b628.tar.bz2 |
Revert "platforms/astbmc/slots.c: Allow comparison of bus numbers when matching slots"
This reverts commit bda7cc4d0354eb3f66629d410b2afc08c79f795f.
Ben says:
It's on purpose that we do NOT compare the bus numbers,
they are always 0 in the slot table
we do a hierarchical walk of the tree, matching only the
devfn's along the way bcs the bus numbering isn't fixed
this breaks all slot naming etc... stuff on anything using
the "skiboot" slot tables (P8 opp typically)
Suggested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'platforms')
-rw-r--r-- | platforms/astbmc/slots.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/platforms/astbmc/slots.c b/platforms/astbmc/slots.c index a5bd50e..19b82fd 100644 --- a/platforms/astbmc/slots.c +++ b/platforms/astbmc/slots.c @@ -77,7 +77,7 @@ static const struct slot_table_entry *match_slot_dev_entry(struct phb *phb, if (ent->etype == st_npu_slot) bdfn = pd->bdfn & 0xf8; else - bdfn = pd->bdfn & 0xffff; + bdfn = pd->bdfn & 0xff; if (ent->location == bdfn) return ent; |