phy: qcom-qmp-ufs: rework regs layout arrays

Use symbolic names for the values inside reg layout arrays. New register
names are added following the PCS register layout that is used by the
particular PHY.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Link: https://lore.kernel.org/r/20221110192248.873973-8-dmitry.baryshkov@linaro.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>

diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v2.h b/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v2.h
index af87066..a0803a8 100644 (file)
--- a/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v2.h
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v2.h
@@ -6,6 +6,9 @@
 #ifndef QCOM_PHY_QMP_PCS_UFS_V2_H_
 #define QCOM_PHY_QMP_PCS_UFS_V2_H_
 
+#define QPHY_V2_PCS_UFS_PHY_START                      0x000
+#define QPHY_V2_PCS_UFS_POWER_DOWN_CONTROL             0x004
+
 #define QPHY_V2_PCS_UFS_TX_LARGE_AMP_DRV_LVL           0x034
 #define QPHY_V2_PCS_UFS_TX_LARGE_AMP_POST_EMP_LVL      0x038
 #define QPHY_V2_PCS_UFS_TX_SMALL_AMP_DRV_LVL           0x03c
@@ -17,4 +20,6 @@
 #define QPHY_V2_PCS_UFS_RX_SIGDET_CTRL2                        0x148
 #define QPHY_V2_PCS_UFS_RX_PWM_GEAR_BAND                       0x154
 
+#define QPHY_V2_PCS_UFS_READY_STATUS                   0x168
+
 #endif

diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v3.h b/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v3.h
index ba1ea29..adea13c 100644 (file)
--- a/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v3.h
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-pcs-ufs-v3.h
@@ -6,12 +6,15 @@
 #ifndef QCOM_PHY_QMP_PCS_UFS_V3_H_
 #define QCOM_PHY_QMP_PCS_UFS_V3_H_
 
+#define QPHY_V3_PCS_UFS_PHY_START                      0x000
+#define QPHY_V3_PCS_UFS_POWER_DOWN_CONTROL             0x004
 #define QPHY_V3_PCS_UFS_TX_LARGE_AMP_DRV_LVL           0x02c
 #define QPHY_V3_PCS_UFS_TX_SMALL_AMP_DRV_LVL           0x034
 #define QPHY_V3_PCS_UFS_RX_SYM_RESYNC_CTRL             0x134
 #define QPHY_V3_PCS_UFS_RX_MIN_HIBERN8_TIME            0x138
 #define QPHY_V3_PCS_UFS_RX_SIGDET_CTRL1                        0x13c
 #define QPHY_V3_PCS_UFS_RX_SIGDET_CTRL2                        0x140
+#define QPHY_V3_PCS_UFS_READY_STATUS                   0x160
 #define QPHY_V3_PCS_UFS_TX_MID_TERM_CTRL1              0x1bc
 #define QPHY_V3_PCS_UFS_MULTI_LANE_CTRL1               0x1c4
 

diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c b/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
index b352d38..7ccd7f0 100644 (file)
--- a/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-ufs.c
@@ -70,21 +70,21 @@ enum qphy_reg_layout {
 };
 
 static const unsigned int msm8996_ufsphy_regs_layout[QPHY_LAYOUT_SIZE] = {
-       [QPHY_START_CTRL]               = 0x00,
-       [QPHY_PCS_READY_STATUS]         = 0x168,
-       [QPHY_PCS_POWER_DOWN_CONTROL]   = 0x04,
+       [QPHY_START_CTRL]               = QPHY_V2_PCS_UFS_PHY_START,
+       [QPHY_PCS_READY_STATUS]         = QPHY_V2_PCS_UFS_READY_STATUS,
+       [QPHY_PCS_POWER_DOWN_CONTROL]   = QPHY_V2_PCS_UFS_POWER_DOWN_CONTROL,
 };
 
 static const unsigned int sdm845_ufsphy_regs_layout[QPHY_LAYOUT_SIZE] = {
-       [QPHY_START_CTRL]               = 0x00,
-       [QPHY_PCS_READY_STATUS]         = 0x160,
-       [QPHY_PCS_POWER_DOWN_CONTROL]   = 0x04,
+       [QPHY_START_CTRL]               = QPHY_V3_PCS_UFS_PHY_START,
+       [QPHY_PCS_READY_STATUS]         = QPHY_V3_PCS_UFS_READY_STATUS,
+       [QPHY_PCS_POWER_DOWN_CONTROL]   = QPHY_V3_PCS_UFS_POWER_DOWN_CONTROL,
 };
 
 static const unsigned int sm6115_ufsphy_regs_layout[QPHY_LAYOUT_SIZE] = {
-       [QPHY_START_CTRL]               = 0x00,
-       [QPHY_PCS_READY_STATUS]         = 0x168,
-       [QPHY_PCS_POWER_DOWN_CONTROL]   = 0x04,
+       [QPHY_START_CTRL]               = QPHY_V2_PCS_UFS_PHY_START,
+       [QPHY_PCS_READY_STATUS]         = QPHY_V2_PCS_UFS_READY_STATUS,
+       [QPHY_PCS_POWER_DOWN_CONTROL]   = QPHY_V2_PCS_UFS_POWER_DOWN_CONTROL,
 };
 
 static const unsigned int sm8150_ufsphy_regs_layout[QPHY_LAYOUT_SIZE] = {