[Skiboot] [PATCH 2/9] SLW: Split init functions

Akshay Adiga akshay.adiga at linux.vnet.ibm.com
Thu Jan 4 22:27:59 AEDT 2018


This patch seperates code which deals with wakeup_engine from one which
doesn't. Init functions for power8 and power9 are split  into chip_init
and late_init. slw_late_init_p?() contains wakeup_engine related code.

Signed-off-by: Akshay Adiga <akshay.adiga at linux.vnet.ibm.com>
---
 hw/slw.c | 75 +++++++++++++++++++++++++++++++++++++---------------------------
 1 file changed, 43 insertions(+), 32 deletions(-)

diff --git a/hw/slw.c b/hw/slw.c
index d6d140f8..2fdfa4e8 100644
--- a/hw/slw.c
+++ b/hw/slw.c
@@ -803,6 +803,36 @@ static struct cpu_idle_states power9_ndd1_cpu_idle_states[] = {
 				 | OPAL_PM_PSSCR_EC,
 		.pm_ctrl_reg_mask = OPAL_PM_PSSCR_MASK }
 };
+static void slw_late_init_p9(struct proc_chip *chip)
+{
+	struct cpu_thread *c;
+	int rc;
+
+	if (!chip->homer_base) {
+		log_simple_error(&e_info(OPAL_RC_SLW_REG),
+				 "SLW: HOMER base not set %x\n",
+				 chip->id);
+		return;
+	}
+	prlog(PR_NOTICE, "SLW: Configuring self-restore for HRMOR\n");
+	for_each_available_cpu(c) {
+		if (c->chip_id != chip->id)
+			continue;
+		/*
+		 * Clear HRMOR. Need to update only for thread
+		 * 0 of each core. Doing it anyway for all threads
+		 */
+		rc =  p9_stop_save_cpureg((void *)chip->homer_base,
+						P9_STOP_SPR_HRMOR, 0,
+						c->pir);
+		if (rc) {
+			log_simple_error(&e_info(OPAL_RC_SLW_REG),
+			"SLW: Failed to set HRMOR for CPU %x,RC=0x%x\n",
+			c->pir, rc);
+			prlog(PR_ERR, "Disabling deep stop states\n");
+		}
+	}
+}
 
 /* Add device tree properties to describe idle states */
 void add_cpu_idle_state_properties(void)
@@ -1265,7 +1295,6 @@ static void slw_patch_regs(struct proc_chip *chip)
 static void slw_init_chip_p9(struct proc_chip *chip)
 {
 	struct cpu_thread *c;
-	int rc;
 
 	prlog(PR_DEBUG, "SLW: Init chip 0x%x\n", chip->id);
 
@@ -1273,38 +1302,11 @@ static void slw_init_chip_p9(struct proc_chip *chip)
 	for_each_available_core_in_chip(c, chip->id)
 		slw_set_overrides_p9(chip, c);
 
-	if (!chip->homer_base) {
-		log_simple_error(&e_info(OPAL_RC_SLW_REG),
-				 "SLW: HOMER base not set %x\n",
-				 chip->id);
-		return;
-	}
-
-	prlog(PR_NOTICE, "SLW: Configuring self-restore for HRMOR\n");
-
-	/* Should this be for_each_present_cpu() ? */
-	for_each_available_cpu(c) {
-		if (c->chip_id != chip->id)
-			continue;
 
-		/*
-		 * Clear HRMOR. Need to update only for thread
-		 * 0 of each core. Doing it anyway for all threads
-		 */
-		rc =  p9_stop_save_cpureg((void *)chip->homer_base,
-						P9_STOP_SPR_HRMOR, 0,
-					       c->pir);
-		if (rc) {
-			log_simple_error(&e_info(OPAL_RC_SLW_REG),
-				 "SLW: Failed to set HRMOR for CPU %x,RC=0x%x\n",
-				 c->pir, rc);
-		}
-	}
 }
-static void slw_init_chip(struct proc_chip *chip)
+static void slw_late_init_p8(struct proc_chip *chip)
 {
 	int64_t rc;
-	struct cpu_thread *c;
 
 	prlog(PR_DEBUG, "SLW: Init chip 0x%x\n", chip->id);
 
@@ -1337,6 +1339,11 @@ static void slw_init_chip(struct proc_chip *chip)
 	/* Patch SLW image */
         slw_patch_regs(chip);
 
+}
+static void slw_init_chip_p8(struct proc_chip *chip)
+{
+	struct cpu_thread *c;
+
 	/* At power ON setup inits for fast-sleep */
 	for_each_available_core_in_chip(c, chip->id) {
 		idle_prepare_core(chip, c);
@@ -1682,12 +1689,16 @@ void slw_init(void)
 	struct proc_chip *chip;
 
 	if (proc_gen == proc_gen_p8) {
-		for_each_chip(chip)
-			slw_init_chip(chip);
+		for_each_chip(chip) {
+			slw_init_chip_p8(chip);
+			slw_late_init_p8(chip);
+		}
 		slw_init_timer();
 	} else if (proc_gen == proc_gen_p9) {
-		for_each_chip(chip)
+		for_each_chip(chip) {
 			slw_init_chip_p9(chip);
+			slw_late_init_p9(chip);
+		}
 	}
 	add_cpu_idle_state_properties();
 }
-- 
2.14.3



More information about the Skiboot mailing list