From: Esteban Urrutia <[email protected]>

Two versions of PLLs for v6 PHYs exist: v1 and v1.1.
These tables only cover v1.1 PLLs.
For serdes, the difference between v1 and v1.1 PLLs is the value written to
the BG_TIMER register.
This register is set to 0x0e for v1 PLLs and 0x0a for v1.1 PLLs.
As reference, SM8550 and SM8650 SoCs use v1.1 PLLs.
For RBR/HBR/HBR2/HBR3, most values change.
Make this distinction clear to avoid confusion when adding support for SoCs
making use of v1 PLLs.

Signed-off-by: Esteban Urrutia <[email protected]>
---
 drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 90 +++++++++++++++----------------
 1 file changed, 45 insertions(+), 45 deletions(-)

diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-combo.c 
b/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
index 9bd666ac2c49..36361ac02855 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-combo.c
@@ -1337,7 +1337,7 @@ static const struct qmp_phy_init_tbl 
qmp_v5_5nm_dp_tx_tbl[] = {
        QMP_PHY_INIT_CFG(QSERDES_V5_5NM_TX_TX_BAND, 0x01),
 };
 
-static const struct qmp_phy_init_tbl qmp_v6_dp_serdes_tbl[] = {
+static const struct qmp_phy_init_tbl qmp_v6_dp_v1_1_serdes_tbl[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_SVS_MODE_CLK_SEL, 0x15),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_SYSCLK_EN_SEL, 0x3b),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_SYS_CLK_CTRL, 0x02),
@@ -1471,7 +1471,7 @@ static const struct qmp_phy_init_tbl 
qmp_v8_n3p_dp_tx_tbl[] = {
        QMP_PHY_INIT_CFG(QSERDES_V8_LALB_TX1_RESTRIM_POST_CAL_OFFSET, 0x10),
 };
 
-static const struct qmp_phy_init_tbl qmp_v6_dp_serdes_tbl_rbr[] = {
+static const struct qmp_phy_init_tbl qmp_v6_dp_v1_1_serdes_tbl_rbr[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_HSCLK_SEL_1, 0x05),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DEC_START_MODE0, 0x34),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DIV_FRAC_START2_MODE0, 0xc0),
@@ -1483,7 +1483,7 @@ static const struct qmp_phy_init_tbl 
qmp_v6_dp_serdes_tbl_rbr[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_BIN_VCOCAL_CMP_CODE2_MODE0, 0x0c),
 };
 
-static const struct qmp_phy_init_tbl qmp_v6_dp_serdes_tbl_hbr[] = {
+static const struct qmp_phy_init_tbl qmp_v6_dp_v1_1_serdes_tbl_hbr[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_HSCLK_SEL_1, 0x03),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DEC_START_MODE0, 0x34),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DIV_FRAC_START2_MODE0, 0xc0),
@@ -1495,7 +1495,7 @@ static const struct qmp_phy_init_tbl 
qmp_v6_dp_serdes_tbl_hbr[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_BIN_VCOCAL_CMP_CODE2_MODE0, 0x0c),
 };
 
-static const struct qmp_phy_init_tbl qmp_v6_dp_serdes_tbl_hbr2[] = {
+static const struct qmp_phy_init_tbl qmp_v6_dp_v1_1_serdes_tbl_hbr2[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_HSCLK_SEL_1, 0x01),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DEC_START_MODE0, 0x46),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DIV_FRAC_START2_MODE0, 0x00),
@@ -1507,7 +1507,7 @@ static const struct qmp_phy_init_tbl 
qmp_v6_dp_serdes_tbl_hbr2[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_BIN_VCOCAL_CMP_CODE2_MODE0, 0x10),
 };
 
-static const struct qmp_phy_init_tbl qmp_v6_dp_serdes_tbl_hbr3[] = {
+static const struct qmp_phy_init_tbl qmp_v6_dp_v1_1_serdes_tbl_hbr3[] = {
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_HSCLK_SEL_1, 0x00),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DEC_START_MODE0, 0x34),
        QMP_PHY_INIT_CFG(QSERDES_V6_COM_DIV_FRAC_START2_MODE0, 0xc0),
@@ -2442,19 +2442,19 @@ static const struct qmp_phy_cfg sar2130p_usb3dpphy_cfg 
= {
        .pcs_usb_tbl            = sm8550_usb3_pcs_usb_tbl,
        .pcs_usb_tbl_num        = ARRAY_SIZE(sm8550_usb3_pcs_usb_tbl),
 
-       .dp_serdes_tbl          = qmp_v6_dp_serdes_tbl,
-       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_serdes_tbl),
+       .dp_serdes_tbl          = qmp_v6_dp_v1_1_serdes_tbl,
+       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl),
        .dp_tx_tbl              = qmp_v6_dp_tx_tbl,
        .dp_tx_tbl_num          = ARRAY_SIZE(qmp_v6_dp_tx_tbl),
 
-       .serdes_tbl_rbr         = qmp_v6_dp_serdes_tbl_rbr,
-       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_rbr),
-       .serdes_tbl_hbr         = qmp_v6_dp_serdes_tbl_hbr,
-       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr),
-       .serdes_tbl_hbr2        = qmp_v6_dp_serdes_tbl_hbr2,
-       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr2),
-       .serdes_tbl_hbr3        = qmp_v6_dp_serdes_tbl_hbr3,
-       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr3),
+       .serdes_tbl_rbr         = qmp_v6_dp_v1_1_serdes_tbl_rbr,
+       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_rbr),
+       .serdes_tbl_hbr         = qmp_v6_dp_v1_1_serdes_tbl_hbr,
+       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr),
+       .serdes_tbl_hbr2        = qmp_v6_dp_v1_1_serdes_tbl_hbr2,
+       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr2),
+       .serdes_tbl_hbr3        = qmp_v6_dp_v1_1_serdes_tbl_hbr3,
+       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr3),
 
        .swing_hbr_rbr          = &qmp_dp_v5_voltage_swing_hbr_rbr,
        .pre_emphasis_hbr_rbr   = &qmp_dp_v6_pre_emphasis_hbr_rbr,
@@ -2852,19 +2852,19 @@ static const struct qmp_phy_cfg sm8550_usb3dpphy_cfg = {
        .pcs_usb_tbl            = sm8550_usb3_pcs_usb_tbl,
        .pcs_usb_tbl_num        = ARRAY_SIZE(sm8550_usb3_pcs_usb_tbl),
 
-       .dp_serdes_tbl          = qmp_v6_dp_serdes_tbl,
-       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_serdes_tbl),
+       .dp_serdes_tbl          = qmp_v6_dp_v1_1_serdes_tbl,
+       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl),
        .dp_tx_tbl              = qmp_v6_dp_tx_tbl,
        .dp_tx_tbl_num          = ARRAY_SIZE(qmp_v6_dp_tx_tbl),
 
-       .serdes_tbl_rbr         = qmp_v6_dp_serdes_tbl_rbr,
-       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_rbr),
-       .serdes_tbl_hbr         = qmp_v6_dp_serdes_tbl_hbr,
-       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr),
-       .serdes_tbl_hbr2        = qmp_v6_dp_serdes_tbl_hbr2,
-       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr2),
-       .serdes_tbl_hbr3        = qmp_v6_dp_serdes_tbl_hbr3,
-       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr3),
+       .serdes_tbl_rbr         = qmp_v6_dp_v1_1_serdes_tbl_rbr,
+       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_rbr),
+       .serdes_tbl_hbr         = qmp_v6_dp_v1_1_serdes_tbl_hbr,
+       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr),
+       .serdes_tbl_hbr2        = qmp_v6_dp_v1_1_serdes_tbl_hbr2,
+       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr2),
+       .serdes_tbl_hbr3        = qmp_v6_dp_v1_1_serdes_tbl_hbr3,
+       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr3),
 
        .swing_hbr_rbr          = &qmp_dp_v5_voltage_swing_hbr_rbr,
        .pre_emphasis_hbr_rbr   = &qmp_dp_v6_pre_emphasis_hbr_rbr,
@@ -2897,19 +2897,19 @@ static const struct qmp_phy_cfg sm8650_usb3dpphy_cfg = {
        .pcs_usb_tbl            = sm8550_usb3_pcs_usb_tbl,
        .pcs_usb_tbl_num        = ARRAY_SIZE(sm8550_usb3_pcs_usb_tbl),
 
-       .dp_serdes_tbl          = qmp_v6_dp_serdes_tbl,
-       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_serdes_tbl),
+       .dp_serdes_tbl          = qmp_v6_dp_v1_1_serdes_tbl,
+       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl),
        .dp_tx_tbl              = qmp_v6_dp_tx_tbl,
        .dp_tx_tbl_num          = ARRAY_SIZE(qmp_v6_dp_tx_tbl),
 
-       .serdes_tbl_rbr         = qmp_v6_dp_serdes_tbl_rbr,
-       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_rbr),
-       .serdes_tbl_hbr         = qmp_v6_dp_serdes_tbl_hbr,
-       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr),
-       .serdes_tbl_hbr2        = qmp_v6_dp_serdes_tbl_hbr2,
-       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr2),
-       .serdes_tbl_hbr3        = qmp_v6_dp_serdes_tbl_hbr3,
-       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr3),
+       .serdes_tbl_rbr         = qmp_v6_dp_v1_1_serdes_tbl_rbr,
+       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_rbr),
+       .serdes_tbl_hbr         = qmp_v6_dp_v1_1_serdes_tbl_hbr,
+       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr),
+       .serdes_tbl_hbr2        = qmp_v6_dp_v1_1_serdes_tbl_hbr2,
+       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr2),
+       .serdes_tbl_hbr3        = qmp_v6_dp_v1_1_serdes_tbl_hbr3,
+       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr3),
 
        .swing_hbr_rbr          = &qmp_dp_v6_voltage_swing_hbr_rbr,
        .pre_emphasis_hbr_rbr   = &qmp_dp_v6_pre_emphasis_hbr_rbr,
@@ -2942,19 +2942,19 @@ static const struct qmp_phy_cfg sm8750_usb3dpphy_cfg = {
        .pcs_usb_tbl            = sm8750_usb3_pcs_usb_tbl,
        .pcs_usb_tbl_num        = ARRAY_SIZE(sm8750_usb3_pcs_usb_tbl),
 
-       .dp_serdes_tbl          = qmp_v6_dp_serdes_tbl,
-       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_serdes_tbl),
+       .dp_serdes_tbl          = qmp_v6_dp_v1_1_serdes_tbl,
+       .dp_serdes_tbl_num      = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl),
        .dp_tx_tbl              = qmp_v6_dp_tx_tbl,
        .dp_tx_tbl_num          = ARRAY_SIZE(qmp_v6_dp_tx_tbl),
 
-       .serdes_tbl_rbr         = qmp_v6_dp_serdes_tbl_rbr,
-       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_rbr),
-       .serdes_tbl_hbr         = qmp_v6_dp_serdes_tbl_hbr,
-       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr),
-       .serdes_tbl_hbr2        = qmp_v6_dp_serdes_tbl_hbr2,
-       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr2),
-       .serdes_tbl_hbr3        = qmp_v6_dp_serdes_tbl_hbr3,
-       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_serdes_tbl_hbr3),
+       .serdes_tbl_rbr         = qmp_v6_dp_v1_1_serdes_tbl_rbr,
+       .serdes_tbl_rbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_rbr),
+       .serdes_tbl_hbr         = qmp_v6_dp_v1_1_serdes_tbl_hbr,
+       .serdes_tbl_hbr_num     = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr),
+       .serdes_tbl_hbr2        = qmp_v6_dp_v1_1_serdes_tbl_hbr2,
+       .serdes_tbl_hbr2_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr2),
+       .serdes_tbl_hbr3        = qmp_v6_dp_v1_1_serdes_tbl_hbr3,
+       .serdes_tbl_hbr3_num    = ARRAY_SIZE(qmp_v6_dp_v1_1_serdes_tbl_hbr3),
 
        .swing_hbr_rbr          = &qmp_dp_v6_voltage_swing_hbr_rbr,
        .pre_emphasis_hbr_rbr   = &qmp_dp_v6_pre_emphasis_hbr_rbr,

-- 
2.55.0



Reply via email to