[Skiboot] [PATCH] Revert "npu2: hw-procedures: Enable low power mode"

Reza Arbab arbab at linux.vnet.ibm.com
Wed Nov 15 04:28:42 AEDT 2017


As it turns out, low power mode is not yet ready for prime time. We
shouldn't write the low power config register until it is.

This reverts commit a05054c53a37850a2118d01fcf6669ebb10d1a33.

Signed-off-by: Reza Arbab <arbab at linux.vnet.ibm.com>
---
Sorry for the churn on this. We had gotten conflicting direction on 
exactly what the hardware is capable of at this time.

 hw/npu2-hw-procedures.c | 19 +------------------
 include/npu2-regs.h     |  6 ------
 2 files changed, 1 insertion(+), 24 deletions(-)

diff --git a/hw/npu2-hw-procedures.c b/hw/npu2-hw-procedures.c
index 85c2427..7a5d188 100644
--- a/hw/npu2-hw-procedures.c
+++ b/hw/npu2-hw-procedures.c
@@ -320,26 +320,9 @@ static uint32_t phy_reset_complete(struct npu2_dev *ndev)
 		phy_write_lane(ndev, &NPU2_PHY_RX_E_INTEG_COARSE_GAIN, lane, 11);
 	}
 
-	return PROCEDURE_NEXT;
-}
-
-/* Procedure 1.2.11 - Enable Low Power Mode */
-static uint32_t enable_low_power(struct npu2_dev *ndev)
-{
-	uint64_t val;
-
-	val = SETFIELD(NPU2_NTL_LOW_POWER_CFG_MODE_ENABLE, 0ull, 1);
-	val = SETFIELD(NPU2_NTL_LOW_POWER_CFG_ONLY_MODE, val, 1);
-	val = SETFIELD(NPU2_NTL_LOW_POWER_CFG_TIMER_TICK_CONFIG, val, 22);
-	val = SETFIELD(NPU2_NTL_LOW_POWER_CFG_MIN_CRED_THRESH, val, 68);
-	val = SETFIELD(NPU2_NTL_LOW_POWER_CFG_MAX_CRED_THRESH, val, 68);
-	val = SETFIELD(NPU2_NTL_LOW_POWER_CFG_CNT_THRESH, val, 477);
-	npu2_write(ndev->npu, NPU2_NTL_LOW_POWER_CFG(ndev), val);
-
 	return PROCEDURE_COMPLETE;
 }
-DEFINE_PROCEDURE(phy_reset, phy_reset_wait, phy_reset_complete,
-		 enable_low_power);
+DEFINE_PROCEDURE(phy_reset, phy_reset_wait, phy_reset_complete);
 
 /* Procedure 1.2.6 - I/O PHY Tx Impedance Calibration */
 static uint32_t phy_tx_zcal(struct npu2_dev *ndev)
diff --git a/include/npu2-regs.h b/include/npu2-regs.h
index 307e93b..759404c 100644
--- a/include/npu2-regs.h
+++ b/include/npu2-regs.h
@@ -248,12 +248,6 @@ void npu2_write_mask(struct npu2 *p, uint64_t reg, uint64_t val, uint64_t mask);
 #define NPU2_NTL_MISC_CFG1(ndev)		NPU2_NTLU_REG_OFFSET(ndev, 0x0C0)
 #define NPU2_NTL_SCRATCH1(ndev)			NPU2_NTLU_REG_OFFSET(ndev, 0x0D0)
 #define NPU2_NTL_LOW_POWER_CFG(ndev)		NPU2_NTLU_REG_OFFSET(ndev, 0x0E0)
-#define   NPU2_NTL_LOW_POWER_CFG_MODE_ENABLE		PPC_BIT(0)
-#define   NPU2_NTL_LOW_POWER_CFG_ONLY_MODE		PPC_BIT(1)
-#define   NPU2_NTL_LOW_POWER_CFG_TIMER_TICK_CONFIG	PPC_BITMASK(2,7)
-#define   NPU2_NTL_LOW_POWER_CFG_MIN_CRED_THRESH	PPC_BITMASK(8,19)
-#define   NPU2_NTL_LOW_POWER_CFG_MAX_CRED_THRESH	PPC_BITMASK(20,31)
-#define   NPU2_NTL_LOW_POWER_CFG_CNT_THRESH		PPC_BITMASK(32,43)
 #define NPU2_NTL_DBG_INHIBIT_CFG(ndev)		NPU2_NTL_REG_OFFSET(ndev, 0x220)
 #define NPU2_NTL_DISPLAY_CTL(ndev)		NPU2_NTL_REG_OFFSET(ndev, 0x280)
 #define NPU2_NTL_DISPLAY_DATA0(ndev)		NPU2_NTL_REG_OFFSET(ndev, 0x288)
-- 
1.8.3.1



More information about the Skiboot mailing list