From 0d81bda68f2b7adcecf268522ab9149b91115739 Mon Sep 17 00:00:00 2001 From: Benjamin Herrenschmidt Date: Fri, 13 Jan 2017 12:03:58 -0600 Subject: [PATCH] hdat: Fix XSCOM nodes for P9 The address mapping for multiple chips is different Signed-off-by: Benjamin Herrenschmidt Signed-off-by: Stewart Smith --- hdata/spira.c | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/hdata/spira.c b/hdata/spira.c index 987955d5a3ce..18a0c6935012 100644 --- a/hdata/spira.c +++ b/hdata/spira.c @@ -233,7 +233,21 @@ static struct dt_node *add_xscom_node(uint64_t base, uint32_t hw_id, uint64_t addr, size; uint64_t freq; - addr = base | ((uint64_t)hw_id << PPC_BITLSHIFT(28)); + switch (proc_gen) { + case proc_gen_p7: + case proc_gen_p8: + /* On P7 and P8 all the chip SCOMs share single region */ + addr = base | ((uint64_t)hw_id << PPC_BITLSHIFT(28)); + break; + case proc_gen_p9: + default: + /* On P9 we need to put the chip ID in the natural powerbus + * position. + */ + addr = base | (((uint64_t)hw_id) << 42); + break; + }; + size = (u64)1 << PPC_BITLSHIFT(28); prlog(PR_INFO, "XSCOM: Found HW ID 0x%x (PCID 0x%x) @ 0x%llx\n", @@ -262,6 +276,10 @@ static struct dt_node *add_xscom_node(uint64_t base, uint32_t hw_id, dt_add_property_strings(node, "compatible", "ibm,xscom", "ibm,power9-xscom"); break; + case proc_gen_p9: + dt_add_property_strings(node, "compatible", + "ibm,xscom", "ibm,power9-xscom"); + break; default: dt_add_property_strings(node, "compatible", "ibm,xscom"); }