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 | |
| parent | dcbf18c1f0e16111d5c86e845f37e8c2cad4f9ea (diff) | |
| download | talos-skiboot-1090f346713ae43319cfeb3eb65ca24f9864b628.tar.gz talos-skiboot-1090f346713ae43319cfeb3eb65ca24f9864b628.zip | |
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>
| -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 a5bd50ee..19b82fdd 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; |
