[Skiboot] [RFC PATCH 2/3] Write boot progress to LPC port 80h
Stewart Smith
stewart at linux.ibm.com
Thu Dec 13 17:07:04 AEDT 2018
This is an adaptation of what we currently do for op_display() on FSP
machines, inventing an encoding for what we can write into the single
byte at LPC port 80h.
Port 80h is often used on x86 systems to indicate boot progress/status
and dates back a decent amount of time. Since a byte isn't exactly very
expressive for everything that can go on (and wrong) during boot, it's
all about compromise.
Some systems (such as Zaius/Barreleye G2) have a physical dual 7 segment
display that display these codes. So far, this has only been driven by
hostboot (see hostboot commit 90ec2e65314c).
Signed-off-by: Stewart Smith <stewart at linux.ibm.com>
---
hw/Makefile.inc | 1 +
hw/fsp/fsp-op-panel.c | 4 +-
hw/lpc-port80h.c | 99 ++++++++++++++++++++++++++++++++++++++++++
hw/test/Makefile.check | 13 +++++-
hw/test/run-port80h.c | 79 +++++++++++++++++++++++++++++++++
include/op-panel.h | 2 +-
6 files changed, 195 insertions(+), 3 deletions(-)
create mode 100644 hw/lpc-port80h.c
create mode 100644 hw/test/run-port80h.c
diff --git a/hw/Makefile.inc b/hw/Makefile.inc
index 005772a5f5d2..f42d4817522a 100644
--- a/hw/Makefile.inc
+++ b/hw/Makefile.inc
@@ -10,6 +10,7 @@ HW_OBJS += fake-nvram.o lpc-mbox.o npu2.o npu2-hw-procedures.o
HW_OBJS += npu2-common.o phys-map.o sbe-p9.o capp.o occ-sensor.o vas.o
HW_OBJS += npu2-common.o npu2-opencapi.o phys-map.o sbe-p9.o capp.o occ-sensor.o
HW_OBJS += vas.o sbe-p8.o
+HW_OBJS += lpc-port80h.o
HW=hw/built-in.a
include $(SRC)/hw/fsp/Makefile.inc
diff --git a/hw/fsp/fsp-op-panel.c b/hw/fsp/fsp-op-panel.c
index 7063cbb3a335..6477e73c1c4d 100644
--- a/hw/fsp/fsp-op-panel.c
+++ b/hw/fsp/fsp-op-panel.c
@@ -55,8 +55,10 @@ void op_display(enum op_severity sev, enum op_module mod, uint16_t code)
uint32_t w0;
uint32_t w1;
- if (!fsp_present())
+ if (!fsp_present()) {
+ op_display_lpc(sev, mod, code);
return;
+ }
w0 = sev << 16 | mod;
diff --git a/hw/lpc-port80h.c b/hw/lpc-port80h.c
new file mode 100644
index 000000000000..72ac8b24b59d
--- /dev/null
+++ b/hw/lpc-port80h.c
@@ -0,0 +1,99 @@
+/* Copyright 2018 IBM Corp.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+ * implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#define pr_fmt(fmt) "Port80h: " fmt
+
+#include <lpc.h>
+#include <op-panel.h>
+
+/*
+ * Convert our detailed op_display() call into 1 byte for LPC port 80h
+ *
+ * Our layout looks like this:
+ * MSB (bit 7): 1 = Comes from OPAL
+ * bit 6 : 0 = OP_MOD_INIT (the main one), 1 = (see bit 5)
+ * bit 5432 : (if bit 6=0, low nibble of op-panel code)
+ * bit 5432 : (if bit 6=1, other OP_MOD_ values in bits 54:
+ * 00b=OP_MOD_CPU, 01b=OP_MOD_LOCK,
+ * 10b=OP_MOD_MEM, 11b=OP_MOD_CHIPTOD
+ * bits 0,1 from code in bits 32)
+ *
+ * bit 1,0: 00b=OP_LOG, 10b=OP_WARN, 01b=OP_ERROR, 11b=OP_FATAL
+ * i.e. bit 0 indicates ERROR or FATAL.
+ *
+ * If port 80h number has the MSB and LSB set, then you died in OPAL.
+ * Any *odd* number with the MSB set (i.e. > 0x80) indicates error.
+ */
+static inline uint8_t op_display_to_port80(uint8_t last_value, enum op_severity s, enum op_module m, uint16_t c)
+{
+ uint8_t r = 0x80; /* Start with top bit set indicating in OPAL */
+
+ switch(m) {
+ case OP_MOD_INIT:
+ /* bit 6 is zero */
+ /* bits 5432 have low nibble of c */
+ r |= (c & 0x0f) << 2;
+ break;
+ case OP_MOD_CPU:
+ r |= 0x40 | (c & 0x03) << 2;
+ break;
+ case OP_MOD_LOCK:
+ r |= 0x50 | (c & 0x03) << 2;
+ break;
+ case OP_MOD_MEM:
+ r |= 0x60 | (c & 0x03) << 2;
+ break;
+ case OP_MOD_CHIPTOD:
+ r |= 0x70 | (c & 0x03) << 2;
+ break;
+ case OP_MOD_CORE:
+ /*
+ * Only current OP_MOD_CORE is where we're OP_FATAL,
+ * So let's go for the last value set and tweak the
+ * bits for OP_FATAL.
+ */
+ r = last_value & 0xFC;
+ break;
+ case OP_MOD_FSP:
+ case OP_MOD_FSPCON:
+ /* Should never be hit, port80h only used on non-FSP! */
+ break;
+ }
+
+ switch(s) {
+ case OP_LOG:
+ break;
+ case OP_WARN:
+ r |= 0x02;
+ break;
+ case OP_ERROR:
+ r |= 0x01;
+ break;
+ case OP_FATAL:
+ r |= 0x03;
+ }
+
+ return r;
+}
+
+void op_display_lpc(enum op_severity s, enum op_module m, uint16_t c)
+{
+ static uint8_t port80_val = 0x80;
+
+ port80_val = op_display_to_port80(port80_val, s, m, c);
+ lpc_outb(port80_val, 0x80);
+}
+
diff --git a/hw/test/Makefile.check b/hw/test/Makefile.check
index f722ff6e6bcf..1681027a3106 100644
--- a/hw/test/Makefile.check
+++ b/hw/test/Makefile.check
@@ -1,10 +1,18 @@
# -*-Makefile-*-
-HW_TEST := hw/test/phys-map-test
+SUBDIRS += hw/test/
+HW_TEST := hw/test/phys-map-test hw/test/run-port80h
.PHONY : hw-check
hw-check: $(HW_TEST:%=%-check)
+.PHONY : hw-coverage
+hw-coverage: $(HW_TEST:%=%-gcov-run)
+
check: hw-check
+coverage: hw-coverage
+
+$(HW_TEST:%=%-gcov-run) : %-run: %
+ $(call QTEST, TEST-COVERAGE ,$< , $<)
$(HW_TEST:%=%-check) : %-check: %
$(call Q, RUN-TEST ,$(VALGRIND) $<, $<)
@@ -12,6 +20,9 @@ $(HW_TEST:%=%-check) : %-check: %
$(HW_TEST) : % : %.c hw/phys-map.o
$(call Q, HOSTCC ,$(HOSTCC) $(HOSTCFLAGS) -O0 -g -I include -I . -o $@ $<, $<)
+$(HW_TEST:%=%-gcov): %-gcov : %.c %
+ $(call Q, HOSTCC ,$(HOSTCC) $(HOSTCFLAGS) $(HOSTGCOVCFLAGS) -I include -I . -lgcov -o $@ $<, $<)
+
clean: hw-clean
hw-clean:
diff --git a/hw/test/run-port80h.c b/hw/test/run-port80h.c
new file mode 100644
index 000000000000..60f69864e8bb
--- /dev/null
+++ b/hw/test/run-port80h.c
@@ -0,0 +1,79 @@
+/* Copyright 2018 IBM Corp.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+ * implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <stdio.h>
+#include <stdarg.h>
+#include <stdint.h>
+#include <assert.h>
+
+#define __LPC_H
+
+uint8_t port80;
+
+static inline void lpc_outb(uint8_t data, uint32_t addr)
+{
+ assert(addr == 0x80);
+ port80 = data;
+}
+
+#include "op-panel.h"
+
+void op_display_lpc(enum op_severity s, enum op_module m, uint16_t c);
+
+#include "../lpc-port80h.c"
+#include "../../core/test/stubs.c"
+
+int main(void)
+{
+ op_display_lpc(OP_LOG, OP_MOD_INIT, 0x00);
+ assert(port80 == 0x80);
+ op_display_lpc(OP_WARN, OP_MOD_INIT, 0x00);
+ assert(port80 == 0x82);
+ op_display_lpc(OP_ERROR, OP_MOD_INIT, 0x00);
+ assert(port80 == 0x81);
+ op_display_lpc(OP_FATAL, OP_MOD_INIT, 0x00);
+ assert(port80 == 0x83);
+ op_display_lpc(OP_FATAL, OP_MOD_INIT, 0x0f);
+ assert(port80 == 0xBF);
+ op_display_lpc(OP_LOG, OP_MOD_INIT, 0x0f);
+ assert(port80 == 0xBC);
+ op_display_lpc(OP_FATAL, OP_MOD_CORE, 0x6666);
+ assert(port80 == 0xBF);
+ op_display_lpc(OP_LOG, OP_MOD_INIT, 0x01);
+ assert(port80 == 0x84);
+ op_display_lpc(OP_LOG, OP_MOD_CPU, 0x05);
+ assert(port80 == 0xC4);
+ op_display_lpc(OP_LOG, OP_MOD_LOCK, 0x07);
+ assert(port80 == 0xDC);
+ op_display_lpc(OP_FATAL, OP_MOD_LOCK, 0x07);
+ assert(port80 == 0xDF);
+ op_display_lpc(OP_FATAL, OP_MOD_MEM, 0x07);
+ assert(port80 == 0xEF);
+ op_display_lpc(OP_WARN, OP_MOD_MEM, 0x02);
+ assert(port80 == 0xEA);
+ op_display_lpc(OP_WARN, OP_MOD_CHIPTOD, 0x02);
+ assert(port80 == 0xFA);
+
+ /*
+ * We can't assert that OP_MOD_FSP is invalid as we'd end up
+ * trying to set port80 in the assert parth
+ */
+ op_display_lpc(OP_LOG, OP_MOD_FSP, 0x00);
+ assert(port80 == 0x80);
+ op_display_lpc(OP_LOG, OP_MOD_FSPCON, 0x00);
+ assert(port80 == 0x80);
+ return 0;
+}
diff --git a/include/op-panel.h b/include/op-panel.h
index 6a935b800a1b..557e750dd54e 100644
--- a/include/op-panel.h
+++ b/include/op-panel.h
@@ -41,7 +41,6 @@ enum op_module {
OP_MOD_CHIPTOD = 0x3035, /* '05' - ChipTOP */
OP_MOD_CPU = 0x3036, /* '06' - CPU bringup */
OP_MOD_MEM = 0x3037, /* '07' - Memory */
- OP_MOD_XSCOM = 0x3038, /* '08' - XSCOM */
};
/* Common codes:
@@ -63,6 +62,7 @@ enum op_module {
*/
extern void op_display(enum op_severity, enum op_module, uint16_t code);
+extern void op_display_lpc(enum op_severity s, enum op_module m, uint16_t c);
extern void op_panel_disable_src_echo(void);
extern void op_panel_clear_src(void);
--
2.20.0
More information about the Skiboot
mailing list