aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDeb McLemore <debmc@linux.ibm.com>2019-03-12 14:11:58 -0500
committerStewart Smith <stewart@linux.ibm.com>2019-03-15 14:31:30 +1100
commitc470806a2e5e25894aae772fc04069bbe35a21e4 (patch)
tree6894bb617aceec46885e46c62d66967154de4836
parent7cd8402c3bec9bcd99dfcc857a6b8ca12569a07b (diff)
downloadskiboot-c470806a2e5e25894aae772fc04069bbe35a21e4.zip
skiboot-c470806a2e5e25894aae772fc04069bbe35a21e4.tar.gz
skiboot-c470806a2e5e25894aae772fc04069bbe35a21e4.tar.bz2
p9dsu: Undo slot label name changes
During some code updates the slot labels were updated to reflect the phb layout, however expectations were that the slot labels be aligned with the riser card slots and not the system planar slots. [stewart: The tale of how we got here is long and varied and not at all clear. The first ESS systems went out with a skiboot v5.9.8 with additional SuperMicro patches. It was probably a slot table, but who knows, we don't have the code so can't check. It's possible it was all coming in through HDAT instead). The op-build tree (thus the exact patches) shipped on systems that work correct seems to not be around anywhere anymore (if it ever was). It was only in skiboot v6.0 that a slot table made it in, and, of course, only having remote machines in random configs, including possibly with riser cards from Briggs&Stratton rather than the ones destined for this system, doesn't make for verifying this at all. It also doesn't help that *consistently* there is *never* any review on slot tables, and we've had things be wrong in the past. Combine this with not upstream Hostboot patches.] Cc: skiboot-stable@lists.ozlabs.org Cc: Benjamin Mashak <mashak@us.ibm.com> Cc: Michael Lim <youhour@us.ibm.com> Fixes: 64a16ae05bb2 ("p9dsu: Fix slot labels for p9dsu2u") Fixes: 87517c8737b9 ("p9dsu: Fix p9dsu slot tables") Fixes: 31231ed300f2 ("p9dsu: Fix p9dsu default variant") Signed-off-by: Deb McLemore <debmc@linux.ibm.com> [stewart: added more detailed explanation, cc stable] Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
-rw-r--r--platforms/astbmc/p9dsu.c32
1 files changed, 16 insertions, 16 deletions
diff --git a/platforms/astbmc/p9dsu.c b/platforms/astbmc/p9dsu.c
index d28bfca..eeb8fd7 100644
--- a/platforms/astbmc/p9dsu.c
+++ b/platforms/astbmc/p9dsu.c
@@ -100,7 +100,7 @@ static const struct slot_table_entry p9dsu1u_phb8_1_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO-R Slot2",
+ .name = "WIO-R Slot",
.power_limit = 75,
},
{ .etype = st_end },
@@ -120,7 +120,7 @@ static const struct slot_table_entry p9dsu1u_phb8_3_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO Slot4",
+ .name = "WIO Slot2",
.power_limit = 75,
},
{ .etype = st_end },
@@ -251,7 +251,7 @@ static const struct slot_table_entry p9dsu2u_phb8_1_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO-R Slot2",
+ .name = "WIO-R Slot",
.power_limit = 75,
},
{ .etype = st_end },
@@ -271,7 +271,7 @@ static const struct slot_table_entry p9dsu2u_phb8_3_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO Slot4",
+ .name = "WIO Slot3",
.power_limit = 75,
},
{ .etype = st_end },
@@ -281,7 +281,7 @@ static const struct slot_table_entry p9dsu2u_phb8_4_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO Slot5",
+ .name = "WIO Slot2",
.power_limit = 75,
},
{ .etype = st_end },
@@ -357,13 +357,13 @@ static const struct slot_table_entry p9dsu2uess_uio_plx_down[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0x8,0),
- .name = "PLX SWITCH UIO Slot2",
+ .name = "PLX switch",
.power_limit = 75,
},
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0x9,0),
- .name = "PLX DOWN LAN UIO Slot2",
+ .name = "Onboard LAN",
},
{ .etype = st_end },
};
@@ -373,7 +373,7 @@ static const struct slot_table_entry p9dsu2uess_uio_plx_up[] = {
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
.children = p9dsu2uess_uio_plx_down,
- .name = "PLX UP UIO Slot2",
+ .name = "PLX up",
.power_limit = 75,
},
{ .etype = st_end },
@@ -383,19 +383,19 @@ static const struct slot_table_entry p9dsu2uess_wio_plx_down[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0x1,0),
- .name = "WIO Slot3",
+ .name = "WIO Slot1",
.power_limit = 75,
},
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0x8,0),
- .name = "PLX SWITCH WIO Slot3",
+ .name = "PLX switch",
.power_limit = 75,
},
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0x9,0),
- .name = "WIO Slot3",
+ .name = "WIO Slot2",
.power_limit = 75,
},
{ .etype = st_end },
@@ -406,7 +406,7 @@ static const struct slot_table_entry p9dsu2uess_wio_plx_up[] = {
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
.children = p9dsu2uess_wio_plx_down,
- .name = "PLX UP WIO Slot3",
+ .name = "PLX up",
.power_limit = 75,
},
{ .etype = st_end },
@@ -427,7 +427,7 @@ static const struct slot_table_entry p9dsu2uess_phb0_1_slot[] = {
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
.children = p9dsu2uess_uio_plx_up,
- .name = "PLX UIO Slot2",
+ .name = "PLX",
.power_limit = 75,
},
{ .etype = st_end },
@@ -474,7 +474,7 @@ static const struct slot_table_entry p9dsu2uess_phb8_0_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO Slot1",
+ .name = "WIO Slot3",
.power_limit = 75,
},
{ .etype = st_end },
@@ -484,7 +484,7 @@ static const struct slot_table_entry p9dsu2uess_phb8_1_slot[] = {
{
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
- .name = "WIO-R Slot2",
+ .name = "WIO-R Slot",
.power_limit = 75,
},
{ .etype = st_end },
@@ -495,7 +495,7 @@ static const struct slot_table_entry p9dsu2uess_phb8_2_slot[] = {
.etype = st_pluggable_slot,
.location = ST_LOC_DEVFN(0,0),
.children = p9dsu2uess_wio_plx_up,
- .name = "PLX WIO Slot3",
+ .name = "PLX",
.power_limit = 75,
},
{ .etype = st_end },