Re: [PATCH v1 2/2] phy: qcom-qmp: Add support for SM6115 UFS phy
From: Iskren Chernev
Date: Fri Aug 06 2021 - 12:09:24 EST
On 8/6/21 3:57 PM, Vinod Koul wrote:
> On 05-08-21, 12:17, Iskren Chernev wrote:
>
>> +/* Only for QMP V3 660 PHY - QSERDES COM registers*/
>> +#define QSERDES_V3_660_COM_ATB_SEL1 0x000
>> +#define QSERDES_V3_660_COM_ATB_SEL2 0x004
>> +#define QSERDES_V3_660_COM_FREQ_UPDATE 0x008
>> +#define QSERDES_V3_660_COM_BG_TIMER 0x00c
>> +#define QSERDES_V3_660_COM_SSC_EN_CENTER 0x010
>> +#define QSERDES_V3_660_COM_SSC_ADJ_PER1 0x014
>> +#define QSERDES_V3_660_COM_SSC_ADJ_PER2 0x018
>> +#define QSERDES_V3_660_COM_SSC_PER1 0x01c
>> +#define QSERDES_V3_660_COM_SSC_PER2 0x020
>> +#define QSERDES_V3_660_COM_SSC_STEP_SIZE1 0x024
>> +#define QSERDES_V3_660_COM_SSC_STEP_SIZE2 0x028
>> +#define QSERDES_V3_660_COM_POST_DIV 0x02c
>> +#define QSERDES_V3_660_COM_POST_DIV_MUX 0x030
>> +#define QSERDES_V3_660_COM_BIAS_EN_CLKBUFLR_EN 0x034
>> +#define QSERDES_V3_660_COM_CLK_ENABLE1 0x038
>> +#define QSERDES_V3_660_COM_SYS_CLK_CTRL 0x03c
>> +#define QSERDES_V3_660_COM_SYSCLK_BUF_ENABLE 0x040
>> +#define QSERDES_V3_660_COM_PLL_EN 0x044
>> +#define QSERDES_V3_660_COM_PLL_IVCO 0x048
>> +#define QSERDES_V3_660_COM_LOCK_CMP1_MODE0 0x04c
>> +#define QSERDES_V3_660_COM_LOCK_CMP2_MODE0 0x050
>> +#define QSERDES_V3_660_COM_LOCK_CMP3_MODE0 0x054
>> +#define QSERDES_V3_660_COM_LOCK_CMP1_MODE1 0x058
>> +#define QSERDES_V3_660_COM_LOCK_CMP2_MODE1 0x05c
>> +#define QSERDES_V3_660_COM_LOCK_CMP3_MODE1 0x060
>> +#define QSERDES_V3_660_COM_CMD_RSVD0 0x064
>> +#define QSERDES_V3_660_COM_EP_CLOCK_DETECT_CTRL 0x068
>> +#define QSERDES_V3_660_COM_SYSCLK_DET_COMP_STATUS 0x06c
>> +#define QSERDES_V3_660_COM_BG_TRIM 0x070
>> +#define QSERDES_V3_660_COM_CLK_EP_DIV 0x074
>> +#define QSERDES_V3_660_COM_CP_CTRL_MODE0 0x078
>> +#define QSERDES_V3_660_COM_CP_CTRL_MODE1 0x07c
>> +#define QSERDES_V3_660_COM_CMN_RSVD1 0x080
>> +#define QSERDES_V3_660_COM_PLL_RCTRL_MODE0 0x084
>> +#define QSERDES_V3_660_COM_PLL_RCTRL_MODE1 0x088
>> +#define QSERDES_V3_660_COM_CMN_RSVD2 0x08c
>> +#define QSERDES_V3_660_COM_PLL_CCTRL_MODE0 0x090
>> +#define QSERDES_V3_660_COM_PLL_CCTRL_MODE1 0x094
>> +#define QSERDES_V3_660_COM_CMN_RSVD3 0x098
>> +#define QSERDES_V3_660_COM_PLL_CNTRL 0x09c
>> +#define QSERDES_V3_660_COM_PHASE_SEL_CTRL 0x0a0
>> +#define QSERDES_V3_660_COM_PHASE_SEL_DC 0x0a4
>> +#define QSERDES_V3_660_COM_BIAS_EN_CTRL_BY_PSM 0x0a8
>> +#define QSERDES_V3_660_COM_SYSCLK_EN_SEL 0x0ac
>> +#define QSERDES_V3_660_COM_CML_SYSCLK_SEL 0x0b0
>> +#define QSERDES_V3_660_COM_RESETSM_CNTRL 0x0b4
>> +#define QSERDES_V3_660_COM_RESETSM_CNTRL2 0x0b8
>> +#define QSERDES_V3_660_COM_RESTRIM_CTRL 0x0bc
>> +#define QSERDES_V3_660_COM_RESTRIM_CTRL2 0x0c0
>> +#define QSERDES_V3_660_COM_LOCK_CMP_EN 0x0c8
>> +#define QSERDES_V3_660_COM_LOCK_CMP_CFG 0x0cc
>> +#define QSERDES_V3_660_COM_DEC_START_MODE0 0x0d0
>> +#define QSERDES_V3_660_COM_DEC_START_MODE1 0x0d4
>> +#define QSERDES_V3_660_COM_VCOCAL_DEADMAN_CTRL 0x0d8
>> +#define QSERDES_V3_660_COM_DIV_FRAC_START1_MODE0 0x0dc
>> +#define QSERDES_V3_660_COM_DIV_FRAC_START2_MODE0 0x0e0
>> +#define QSERDES_V3_660_COM_DIV_FRAC_START3_MODE0 0x0e4
>> +#define QSERDES_V3_660_COM_DIV_FRAC_START1_MODE1 0x0e8
>> +#define QSERDES_V3_660_COM_DIV_FRAC_START2_MODE1 0x0ec
>> +#define QSERDES_V3_660_COM_DIV_FRAC_START3_MODE1 0x0f0
>> +#define QSERDES_V3_660_COM_VCO_TUNE_MINVAL1 0x0f4
>> +#define QSERDES_V3_660_COM_VCO_TUNE_MINVAL2 0x0f8
>> +#define QSERDES_V3_660_COM_CMN_RSVD4 0x0fc
>> +#define QSERDES_V3_660_COM_INTEGLOOP_INITVAL 0x100
>> +#define QSERDES_V3_660_COM_INTEGLOOP_EN 0x104
>> +#define QSERDES_V3_660_COM_INTEGLOOP_GAIN0_MODE0 0x108
>> +#define QSERDES_V3_660_COM_INTEGLOOP_GAIN1_MODE0 0x10c
>> +#define QSERDES_V3_660_COM_INTEGLOOP_GAIN0_MODE1 0x110
>> +#define QSERDES_V3_660_COM_INTEGLOOP_GAIN1_MODE1 0x114
>> +#define QSERDES_V3_660_COM_VCO_TUNE_MAXVAL1 0x118
>> +#define QSERDES_V3_660_COM_VCO_TUNE_MAXVAL2 0x11c
>> +#define QSERDES_V3_660_COM_RES_TRIM_CONTROL2 0x120
>> +#define QSERDES_V3_660_COM_VCO_TUNE_CTRL 0x124
>> +#define QSERDES_V3_660_COM_VCO_TUNE_MAP 0x128
>> +#define QSERDES_V3_660_COM_VCO_TUNE1_MODE0 0x12c
>> +#define QSERDES_V3_660_COM_VCO_TUNE2_MODE0 0x130
>> +#define QSERDES_V3_660_COM_VCO_TUNE1_MODE1 0x134
>> +#define QSERDES_V3_660_COM_VCO_TUNE2_MODE1 0x138
>> +#define QSERDES_V3_660_COM_VCO_TUNE_INITVAL1 0x13c
>> +#define QSERDES_V3_660_COM_VCO_TUNE_INITVAL2 0x140
>> +#define QSERDES_V3_660_COM_VCO_TUNE_TIMER1 0x144
>> +#define QSERDES_V3_660_COM_VCO_TUNE_TIMER2 0x148
>> +#define QSERDES_V3_660_COM_SAR 0x14c
>> +#define QSERDES_V3_660_COM_SAR_CLK 0x150
>> +#define QSERDES_V3_660_COM_SAR_CODE_OUT_STATUS 0x154
>> +#define QSERDES_V3_660_COM_SAR_CODE_READY_STATUS 0x158
>> +#define QSERDES_V3_660_COM_CMN_STATUS 0x15c
>> +#define QSERDES_V3_660_COM_RESET_SM_STATUS 0x160
>> +#define QSERDES_V3_660_COM_RESTRIM_CODE_STATUS 0x164
>> +#define QSERDES_V3_660_COM_PLLCAL_CODE1_STATUS 0x168
>> +#define QSERDES_V3_660_COM_PLLCAL_CODE2_STATUS 0x16c
>> +#define QSERDES_V3_660_COM_BG_CTRL 0x170
>> +#define QSERDES_V3_660_COM_CLK_SELECT 0x174
>> +#define QSERDES_V3_660_COM_HSCLK_SEL 0x178
>> +#define QSERDES_V3_660_COM_INTEGLOOP_BINCODE_STATUS 0x17c
>> +#define QSERDES_V3_660_COM_PLL_ANALOG 0x180
>> +#define QSERDES_V3_660_COM_CORECLK_DIV 0x184
>> +#define QSERDES_V3_660_COM_SW_RESET 0x188
>> +#define QSERDES_V3_660_COM_CORE_CLK_EN 0x18c
>> +#define QSERDES_V3_660_COM_C_READY_STATUS 0x190
>> +#define QSERDES_V3_660_COM_CMN_CONFIG 0x194
>> +#define QSERDES_V3_660_COM_CMN_RATE_OVERRIDE 0x198
>> +#define QSERDES_V3_660_COM_SVS_MODE_CLK_SEL 0x19c
>> +#define QSERDES_V3_660_COM_DEBUG_BUS0 0x1a0
>> +#define QSERDES_V3_660_COM_DEBUG_BUS1 0x1a4
>> +#define QSERDES_V3_660_COM_DEBUG_BUS2 0x1a8
>> +#define QSERDES_V3_660_COM_DEBUG_BUS3 0x1ac
>> +#define QSERDES_V3_660_COM_DEBUG_BUS_SEL 0x1b0
>> +#define QSERDES_V3_660_COM_CMN_MISC1 0x1b4
>> +#define QSERDES_V3_660_COM_CORECLK_DIV_MODE1 0x1bc
>> +#define QSERDES_V3_660_COM_CMN_RSVD5 0x1c0
>
> why are these registers 660 specific and not V3 like we have for
> other generations..? Quick looks with V3 regs tells me that seem
> similar....
>
Well, I looked at them for a long time and the only thing that is similar
are the first few QSERDES registers from COM_ATB_SEL1=0x000 to
COM_BIAS_EN_CLKBUFLR_EN=0x034. Everything else is different. So I can re-use
the V3 for the similar regs, but they are 10%, so I figured instead of combing
through all of the sequences to save 10% of the constants and make it slightly
less readable just use new constants.
Let me know if I should reuse the 14 regs from V3.
Regards,
Iskren