[Skiboot] [PATCH 5/7] npu2: Add checks to npu2-only codepaths
Reza Arbab
arbab at linux.ibm.com
Thu Jun 13 07:08:55 AEST 2019
To prepare for npu3, add a few checks in codepaths that are only for
npu2. Compare against PVR_TYPE_P9, as npu3 will be in systems of
PVR_TYPE_P9P (or greater). Alternatively, check for dt compatibility
with "ibm,power9-npu" because npu3 will use "ibm,power9-npu3".
Signed-off-by: Reza Arbab <arbab at linux.ibm.com>
---
core/hmi.c | 4 ++++
hdata/spira.c | 6 +++---
hw/npu2-common.c | 4 ++++
3 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/core/hmi.c b/core/hmi.c
index 5cde67138109..2202bfba9ca2 100644
--- a/core/hmi.c
+++ b/core/hmi.c
@@ -636,6 +636,10 @@ static void find_npu2_checkstop_reason(int flat_chip_id,
int total_errors = 0;
const char *loc;
+ /* NPU2 only */
+ if (PVR_TYPE(mfspr(SPR_PVR)) != PVR_TYPE_P9)
+ return;
+
/* Find the NPU on the chip associated with the HMI. */
for_each_phb(phb) {
/* NOTE: if a chip ever has >1 NPU this will need adjusting */
diff --git a/hdata/spira.c b/hdata/spira.c
index 6891a9c71bdf..804d90da34a1 100644
--- a/hdata/spira.c
+++ b/hdata/spira.c
@@ -1506,7 +1506,8 @@ static void add_npus(void)
int phb_index = 7; /* Start counting from 7, for no reason */
int npu_index = 0;
- if (proc_gen < proc_gen_p9)
+ /* Only consult HDAT for npu2 */
+ if (cpu_type != PVR_TYPE_P9)
return;
dt_for_each_compatible(dt_root, xscom, "ibm,xscom") {
@@ -1633,8 +1634,7 @@ int parse_hdat(bool is_opal)
io_parse();
/* Add NPU nodes */
- if (proc_gen >= proc_gen_p9)
- add_npus();
+ add_npus();
/* Parse VPD */
vpd_parse();
diff --git a/hw/npu2-common.c b/hw/npu2-common.c
index f3f2f45a10d4..776030759693 100644
--- a/hw/npu2-common.c
+++ b/hw/npu2-common.c
@@ -654,6 +654,10 @@ void probe_npu2(void)
struct dt_node *np;
const char *zcal;
+ /* npu2 only */
+ if (!dt_find_compatible_node(dt_root, NULL, "ibm,power9-npu"))
+ return;
+
/* Abort if we're running on POWER9C DD1 (P9N DD1 is not supported) */
if (chip &&
chip->type == PROC_CHIP_P9_CUMULUS &&
--
1.8.3.1
More information about the Skiboot
mailing list