lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <dc520394-1777-4810-8f4f-2bc1e258fd1e@rock-chips.com>
Date: Mon, 16 Dec 2024 18:06:27 +0800
From: Damon Ding <damon.ding@...k-chips.com>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
Cc: heiko@...ech.de, robh@...nel.org, krzk+dt@...nel.org,
 conor+dt@...nel.org, rfoss@...nel.org, vkoul@...nel.org,
 sebastian.reichel@...labora.com, cristian.ciocaltea@...labora.com,
 l.stach@...gutronix.de, andy.yan@...k-chips.com, hjc@...k-chips.com,
 algea.cao@...k-chips.com, kever.yang@...k-chips.com,
 dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-rockchip@...ts.infradead.org,
 linux-kernel@...r.kernel.org, linux-phy@...ts.infradead.org
Subject: Re: [PATCH v2 04/11] phy: phy-rockchip-samsung-hdptx: Add support for
 eDP mode

Hi Dmitry,

On 2024/12/16 17:00, Dmitry Baryshkov wrote:
> On Mon, Dec 16, 2024 at 11:12:18AM +0800, Damon Ding wrote:
>> Add basic support for RBR/HBR/HBR2 link rates, and the voltage swing and
>> pre-emphasis configurations of each link rate have been verified according
>> to the eDP 1.3 requirements.
>>
>> Signed-off-by: Damon Ding <damon.ding@...k-chips.com>
>>
>> ---
>>
>> Changes in v2:
>> - Add the module author
>> ---
>>   .../phy/rockchip/phy-rockchip-samsung-hdptx.c | 937 +++++++++++++++++-
>>   1 file changed, 894 insertions(+), 43 deletions(-)
>>
>> diff --git a/drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c b/drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c
>> index 9f084697dd05..8dd0de267d10 100644
>> --- a/drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c
>> +++ b/drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c
>> @@ -25,6 +25,7 @@
>>   #define HDPTX_I_PLL_EN			BIT(7)
>>   #define HDPTX_I_BIAS_EN			BIT(6)
>>   #define HDPTX_I_BGR_EN			BIT(5)
>> +#define HDPTX_MODE_SEL			BIT(0)
>>   #define GRF_HDPTX_STATUS		0x80
>>   #define HDPTX_O_PLL_LOCK_DONE		BIT(3)
>>   #define HDPTX_O_PHY_CLK_RDY		BIT(2)
>> @@ -44,6 +45,7 @@
>>   #define LANE_REG(n)			HDTPX_REG(n, 0300, 062d)
>>   
>>   /* CMN_REG(0008) */
>> +#define OVRD_LCPLL_EN_MASK		BIT(7)
>>   #define LCPLL_EN_MASK			BIT(6)
>>   #define LCPLL_LCVCO_MODE_EN_MASK	BIT(4)
>>   /* CMN_REG(001e) */
>> @@ -61,49 +63,110 @@
>>   /* CMN_REG(002f) */
>>   #define LCPLL_SDC_DENOMINATOR_MASK	GENMASK(7, 2)
>>   #define LCPLL_SDC_NDIV_RSTN		BIT(0)
>> +/* CMN_REG(003c) */
>> +#define ANA_LCPLL_RESERVED7_MASK	BIT(7)
>>   /* CMN_REG(003d) */
>> +#define OVRD_ROPLL_EN_MASK		BIT(7)
>> +#define ROPLL_EN_MASK			BIT(6)
>>   #define ROPLL_LCVCO_EN			BIT(4)
>> +/* CMN_REG(0046) */
>> +#define ROPLL_ANA_CPP_CTRL_COARSE_MASK	GENMASK(7, 4)
>> +#define ROPLL_ANA_CPP_CTRL_FINE_MASK	GENMASK(3, 0)
>> +/* CMN_REG(0047) */
>> +#define ROPLL_ANA_LPF_C_SEL_COARSE_MASK	GENMASK(5, 3)
>> +#define ROPLL_ANA_LPF_C_SEL_FINE_MASK	GENMASK(2, 0)
>>   /* CMN_REG(004e) */
>>   #define ROPLL_PI_EN			BIT(5)
>> +/* CMN_REG(0051) */
>> +#define ROPLL_PMS_MDIV_MASK		GENMASK(7, 0)
>> +/* CMN_REG(0055) */
>> +#define ROPLL_PMS_MDIV_AFC_MASK		GENMASK(7, 0)
>> +/* CMN_REG(0059) */
>> +#define ANA_ROPLL_PMS_PDIV_MASK		GENMASK(7, 4)
>> +#define ANA_ROPLL_PMS_REFDIV_MASK	GENMASK(3, 0)
>> +/* CMN_REG(005a) */
>> +#define ROPLL_PMS_SDIV_RBR_MASK		GENMASK(7, 4)
>> +#define ROPLL_PMS_SDIV_HBR_MASK		GENMASK(3, 0)
>> +/* CMN_REG(005b) */
>> +#define ROPLL_PMS_SDIV_HBR2_MASK	GENMASK(7, 4)
>>   /* CMN_REG(005c) */
>>   #define ROPLL_PMS_IQDIV_RSTN		BIT(5)
>>   /* CMN_REG(005e) */
>>   #define ROPLL_SDM_EN_MASK		BIT(6)
>> -#define ROPLL_SDM_FRAC_EN_RBR		BIT(3)
>> -#define ROPLL_SDM_FRAC_EN_HBR		BIT(2)
>> -#define ROPLL_SDM_FRAC_EN_HBR2		BIT(1)
>> -#define ROPLL_SDM_FRAC_EN_HBR3		BIT(0)
> 
> You have mixed the redefinition of existing bitfields with addition of
> new ones. Please split them into separate patches: one for each of
> corrections (be sure to specify Fixes tag as required), one for generic
> renaming and one for the new platform and new registers / new bitfields)

Yes, it is better split this patch into two. I'll do it in the next version.

> 
>> +#define OVRD_ROPLL_SDM_RSTN_MASK	BIT(5)
>> +#define ROPLL_SDM_RSTN_MASK		BIT(4)
>> +#define ROPLL_SDC_FRAC_EN_RBR_MASK	BIT(3)
>> +#define ROPLL_SDC_FRAC_EN_HBR_MASK	BIT(2)
>> +#define ROPLL_SDC_FRAC_EN_HBR2_MASK	BIT(1)
>> +/* CMN_REG(005f) */
>> +#define OVRD_ROPLL_SDC_RSTN_MASK	BIT(5)
>> +#define ROPLL_SDC_RSTN_MASK		BIT(4)
>> +/* CMN_REG(0060)  */
>> +#define ROPLL_SDM_DENOMINATOR_MASK	GENMASK(7, 0)
>>   /* CMN_REG(0064) */
>>   #define ROPLL_SDM_NUM_SIGN_RBR_MASK	BIT(3)
>> +#define ROPLL_SDM_NUM_SIGN_HBR_MASK	BIT(2)
>> +#define ROPLL_SDM_NUM_SIGN_HBR2_MASK	BIT(1)
>> +/* CMN_REG(0065) */
>> +#define ROPLL_SDM_NUM_MASK		GENMASK(7, 0)
>>   /* CMN_REG(0069) */
>>   #define ROPLL_SDC_N_RBR_MASK		GENMASK(2, 0)
>> +/* CMN_REG(006a) */
>> +#define ROPLL_SDC_N_HBR_MASK		GENMASK(5, 3)
>> +#define ROPLL_SDC_N_HBR2_MASK		GENMASK(2, 0)
>> +/* CMN_REG(006b) */
>> +#define ROPLL_SDC_N_HBR3_MASK		GENMASK(3, 1)
>> +/* CMN_REG(006c) */
>> +#define ROPLL_SDC_NUM_MASK		GENMASK(5, 0)
>> +/* cmn_reg0070 */
>> +#define ROPLL_SDC_DENO_MASK		GENMASK(5, 0)
>>   /* CMN_REG(0074) */
>> -#define ROPLL_SDC_NDIV_RSTN		BIT(2)
>> -#define ROPLL_SSC_EN			BIT(0)
>> +#define OVRD_ROPLL_SDC_NDIV_RSTN_MASK	BIT(3)
>> +#define ROPLL_SDC_NDIV_RSTN_MASK	BIT(2)
>> +#define OVRD_ROPLL_SSC_EN_MASK		BIT(1)
>> +#define ROPLL_SSC_EN_MASK		BIT(0)
>> +/* CMN_REG(0075) */
>> +#define ANA_ROPLL_SSC_FM_DEVIATION_MASK	GENMASK(5, 0)
>> +/* CMN_REG(0076) */
>> +#define ANA_ROPLL_SSC_FM_FREQ_MASK	GENMASK(6, 2)
>> +/* CMN_REG(0077) */
>> +#define ANA_ROPLL_SSC_CLK_DIV_SEL_MASK	GENMASK(6, 3)
>>   /* CMN_REG(0081) */
>> -#define OVRD_PLL_CD_CLK_EN		BIT(8)
>> -#define PLL_CD_HSCLK_EAST_EN		BIT(0)
>> +#define ANA_PLL_CD_TX_SER_RATE_SEL_MASK	BIT(3)
>> +#define ANA_PLL_CD_HSCLK_WEST_EN_MASK	BIT(1)
>> +#define ANA_PLL_CD_HSCLK_EAST_EN_MASK	BIT(0)
>> +/* CMN_REG(0082) */
>> +#define ANA_PLL_CD_VREG_GAIN_CTRL_MASK	GENMASK(3, 0)
>> +/* CMN_REG(0083) */
>> +#define ANA_PLL_CD_VREG_ICTRL_MASK	GENMASK(6, 5)
>> +/* CMN_REG(0084) */
>> +#define PLL_LCRO_CLK_SEL_MASK		BIT(5)
>> +/* CMN_REG(0085) */
>> +#define ANA_PLL_SYNC_LOSS_DET_MODE_MASK	GENMASK(1, 0)
>>   /* CMN_REG(0086) */
>>   #define PLL_PCG_POSTDIV_SEL_MASK	GENMASK(7, 4)
>>   #define PLL_PCG_CLK_SEL_MASK		GENMASK(3, 1)
>>   #define PLL_PCG_CLK_EN			BIT(0)
>>   /* CMN_REG(0087) */
>> -#define PLL_FRL_MODE_EN			BIT(3)
>> -#define PLL_TX_HS_CLK_EN		BIT(2)
>> +#define ANA_PLL_FRL_MODE_EN_MASK	BIT(3)
>> +#define ANA_PLL_TX_HS_CLK_EN_MASK	BIT(2)
>>   /* CMN_REG(0089) */
>>   #define LCPLL_ALONE_MODE		BIT(1)
>> +/* CMN_REG(0095) */
>> +#define DP_TX_LINK_BW_MASK		GENMASK(1, 0)
>>   /* CMN_REG(0097) */
>> -#define DIG_CLK_SEL			BIT(1)
>> -#define ROPLL_REF			BIT(1)
>> -#define LCPLL_REF			0
>> +#define DIG_CLK_SEL_MASK		BIT(1)
>> +#define LCPLL_REF			BIT(1)
>> +#define ROPLL_REF			0
>>   /* CMN_REG(0099) */
>> +#define SSC_EN_MASK			GENMASK(7, 6)
>>   #define CMN_ROPLL_ALONE_MODE		BIT(2)
>>   #define ROPLL_ALONE_MODE		BIT(2)
>>   /* CMN_REG(009a) */
>> -#define HS_SPEED_SEL			BIT(0)
>> +#define HS_SPEED_SEL_MASK		BIT(0)
>>   #define DIV_10_CLOCK			BIT(0)
>>   /* CMN_REG(009b) */
>> -#define IS_SPEED_SEL			BIT(4)
>> +#define LS_SPEED_SEL_MASK		BIT(4)
>>   #define LINK_SYMBOL_CLOCK		BIT(4)
>>   #define LINK_SYMBOL_CLOCK1_2		0
>>   
>> @@ -118,6 +181,8 @@
>>   /* SB_REG(0104) */
>>   #define OVRD_SB_EN_MASK			BIT(5)
>>   #define SB_EN_MASK			BIT(4)
>> +#define OVRD_SB_AUX_EN_MASK		BIT(1)
>> +#define SB_AUX_EN_MASK			BIT(0)
>>   /* SB_REG(0105) */
>>   #define OVRD_SB_EARC_CMDC_EN_MASK	BIT(6)
>>   #define SB_EARC_CMDC_EN_MASK		BIT(5)
>> @@ -126,6 +191,8 @@
>>   #define ANA_SB_TX_LLVL_PROG_MASK	GENMASK(6, 4)
>>   /* SB_REG(0109) */
>>   #define ANA_SB_DMRX_AFC_DIV_RATIO_MASK	GENMASK(2, 0)
>> +/* SB_REG(010d) */
>> +#define ANA_SB_DMRX_LPBK_DATA_MASK	BIT(4)
>>   /* SB_REG(010f) */
>>   #define OVRD_SB_VREG_EN_MASK		BIT(7)
>>   #define SB_VREG_EN_MASK			BIT(6)
>> @@ -133,6 +200,7 @@
>>   #define SB_VREG_LPF_BYPASS_MASK		BIT(4)
>>   #define ANA_SB_VREG_GAIN_CTRL_MASK	GENMASK(3, 0)
>>   /* SB_REG(0110) */
>> +#define ANA_SB_VREG_OUT_SEL_MASK	BIT(1)
>>   #define ANA_SB_VREG_REF_SEL_MASK	BIT(0)
>>   /* SB_REG(0113) */
>>   #define SB_RX_RCAL_OPT_CODE_MASK	GENMASK(5, 4)
>> @@ -147,13 +215,24 @@
>>   #define AFC_RSTN_DELAY_TIME_MASK	GENMASK(6, 4)
>>   /* SB_REG(0117) */
>>   #define FAST_PULSE_TIME_MASK		GENMASK(3, 0)
>> +/* SB_REG(0118) */
>> +#define SB_TG_EARC_DMRX_RECVRD_CLK_CNT_MASK	GENMASK(7, 0)
>> +/* SB_REG(011a) */
>> +#define SB_TG_CNT_RUN_NO_7_0_MASK	GENMASK(7, 0)
>>   /* SB_REG(011b) */
>>   #define SB_EARC_SIG_DET_BYPASS_MASK	BIT(4)
>>   #define SB_AFC_TOL_MASK			GENMASK(3, 0)
>> +/* SB_REG(011c) */
>> +#define SB_AFC_STB_NUM_MASK		GENMASK(3, 0)
>> +/* SB_REG(011d) */
>> +#define SB_TG_OSC_CNT_MIN_MASK		GENMASK(7, 0)
>> +/* SB_REG(011e) */
>> +#define SB_TG_OSC_CNT_MAX_MASK		GENMASK(7, 0)
>>   /* SB_REG(011f) */
>>   #define SB_PWM_AFC_CTRL_MASK		GENMASK(7, 2)
>>   #define SB_RCAL_RSTN_MASK		BIT(1)
>>   /* SB_REG(0120) */
>> +#define SB_AUX_EN_IN_MASK		BIT(7)
>>   #define SB_EARC_EN_MASK			BIT(1)
>>   #define SB_EARC_AFC_EN_MASK		BIT(2)
>>   /* SB_REG(0123) */
>> @@ -165,35 +244,65 @@
>>   #define HDMI_MODE			BIT(2)
>>   #define HDMI_TMDS_FRL_SEL		BIT(1)
>>   /* LNTOP_REG(0206) */
>> -#define DATA_BUS_SEL			BIT(0)
>> +#define DATA_BUS_WIDTH_MASK		GENMASK(2, 1)
>> +#define DATA_BUS_WIDTH_SEL_MASK		BIT(0)
>>   #define DATA_BUS_36_40			BIT(0)
>>   /* LNTOP_REG(0207) */
>>   #define LANE_EN				0xf
>>   #define ALL_LANE_EN			0xf
>>   
>> -/* LANE_REG(0312) */
>> -#define LN0_TX_SER_RATE_SEL_RBR		BIT(5)
>> -#define LN0_TX_SER_RATE_SEL_HBR		BIT(4)
>> -#define LN0_TX_SER_RATE_SEL_HBR2	BIT(3)
>> -#define LN0_TX_SER_RATE_SEL_HBR3	BIT(2)
>> -/* LANE_REG(0412) */
>> -#define LN1_TX_SER_RATE_SEL_RBR		BIT(5)
>> -#define LN1_TX_SER_RATE_SEL_HBR		BIT(4)
>> -#define LN1_TX_SER_RATE_SEL_HBR2	BIT(3)
>> -#define LN1_TX_SER_RATE_SEL_HBR3	BIT(2)
>> -/* LANE_REG(0512) */
>> -#define LN2_TX_SER_RATE_SEL_RBR		BIT(5)
>> -#define LN2_TX_SER_RATE_SEL_HBR		BIT(4)
>> -#define LN2_TX_SER_RATE_SEL_HBR2	BIT(3)
>> -#define LN2_TX_SER_RATE_SEL_HBR3	BIT(2)
>> -/* LANE_REG(0612) */
>> -#define LN3_TX_SER_RATE_SEL_RBR		BIT(5)
>> -#define LN3_TX_SER_RATE_SEL_HBR		BIT(4)
>> -#define LN3_TX_SER_RATE_SEL_HBR2	BIT(3)
>> -#define LN3_TX_SER_RATE_SEL_HBR3	BIT(2)
>> +/* LANE_REG(0301) */
>> +#define OVRD_LN_TX_DRV_EI_EN_MASK	BIT(7)
>> +#define LN_TX_DRV_EI_EN_MASK		BIT(6)
>> +/* LANE_REG(0303) */
>> +#define OVRD_LN_TX_DRV_LVL_CTRL_MASK	BIT(5)
>> +#define LN_TX_DRV_LVL_CTRL_MASK		GENMASK(4, 0)
>> +/* LANE_REG(0304)  */
>> +#define OVRD_LN_TX_DRV_POST_LVL_CTRL_MASK	BIT(4)
>> +#define LN_TX_DRV_POST_LVL_CTRL_MASK	GENMASK(3, 0)
>> +/* LANE_REG(0305) */
>> +#define OVRD_LN_TX_DRV_PRE_LVL_CTRL_MASK	BIT(6)
>> +#define LN_TX_DRV_PRE_LVL_CTRL_MASK	GENMASK(5, 2)
>> +/* LANE_REG(0306) */
>> +#define LN_ANA_TX_DRV_IDRV_IDN_CTRL_MASK	GENMASK(7, 5)
>> +#define LN_ANA_TX_DRV_IDRV_IUP_CTRL_MASK	GENMASK(4, 2)
>> +#define LN_ANA_TX_DRV_ACCDRV_EN_MASK	BIT(0)
>> +/* LANE_REG(0307) */
>> +#define LN_ANA_TX_DRV_ACCDRV_POL_SEL_MASK	BIT(6)
>> +#define LN_ANA_TX_DRV_ACCDRV_CTRL_MASK	GENMASK(5, 3)
>> +/* LANE_REG(030a) */
>> +#define LN_ANA_TX_JEQ_EN_MASK		BIT(4)
>> +#define LN_TX_JEQ_EVEN_CTRL_RBR_MASK	GENMASK(3, 0)
>> +/* LANE_REG(030b) */
>> +#define LN_TX_JEQ_EVEN_CTRL_HBR_MASK	GENMASK(7, 4)
>> +#define LN_TX_JEQ_EVEN_CTRL_HBR2_MASK	GENMASK(3, 0)
>> +/* LANE_REG(030c) */
>> +#define LN_TX_JEQ_ODD_CTRL_RBR_MASK	GENMASK(3, 0)
>> +/* LANE_REG(030d) */
>> +#define LN_TX_JEQ_ODD_CTRL_HBR_MASK	GENMASK(7, 4)
>> +#define LN_TX_JEQ_ODD_CTRL_HBR2_MASK	GENMASK(3, 0)
>> +/* LANE_REG(0310) */
>> +#define LN_ANA_TX_SYNC_LOSS_DET_MODE_MASK	GENMASK(1, 0)
>> +/* LANE_REG(0311) */
>> +#define LN_TX_SER_40BIT_EN_RBR_MASK	BIT(3)
>> +#define LN_TX_SER_40BIT_EN_HBR_MASK	BIT(2)
>> +#define LN_TX_SER_40BIT_EN_HBR2_MASK	BIT(1)
>> +/* LANE_REG(0316) */
>> +#define LN_ANA_TX_SER_VREG_GAIN_CTRL_MASK	GENMASK(3, 0)
>> +/* LANE_REG(031B) */
>> +#define LN_ANA_TX_RESERVED_MASK		GENMASK(7, 0)
>> +/* LANE_REG(031e) */
>> +#define LN_POLARITY_INV_MASK		BIT(2)
>> +#define LN_LANE_MODE_MASK		BIT(1)
>>   
>>   #define HDMI20_MAX_RATE			600000000
>>   
>> +enum dp_link_rate {
>> +	DP_BW_RBR,
>> +	DP_BW_HBR,
>> +	DP_BW_HBR2,
>> +};
>> +
>>   struct lcpll_config {
>>   	u32 bit_rate;
>>   	u8 lcvco_mode_en;
>> @@ -255,6 +364,19 @@ struct ropll_config {
>>   	u8 cd_tx_ser_rate_sel;
>>   };
>>   
>> +struct tx_drv_ctrl {
>> +	u8 tx_drv_lvl_ctrl;
>> +	u8 tx_drv_post_lvl_ctrl;
>> +	u8 ana_tx_drv_idrv_idn_ctrl;
>> +	u8 ana_tx_drv_idrv_iup_ctrl;
>> +	u8 ana_tx_drv_accdrv_en;
>> +	u8 ana_tx_drv_accdrv_ctrl;
>> +	u8 tx_drv_pre_lvl_ctrl;
>> +	u8 ana_tx_jeq_en;
>> +	u8 tx_jeq_even_ctrl;
>> +	u8 tx_jeq_odd_ctrl;
>> +};
>> +
>>   enum rk_hdptx_reset {
>>   	RST_PHY = 0,
>>   	RST_APB,
>> @@ -560,6 +682,90 @@ static const struct reg_sequence rk_hdtpx_tmds_lane_init_seq[] = {
>>   	REG_SEQ0(LANE_REG(0606), 0x1c),
>>   };
>>   
>> +static struct tx_drv_ctrl tx_drv_ctrl_rbr[4][4] = {
>> +	/* voltage swing 0, pre-emphasis 0->3 */
>> +	{
>> +		{ 0x2, 0x0, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0x4, 0x3, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0x7, 0x6, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0xd, 0xc, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 1, pre-emphasis 0->2 */
>> +	{
>> +		{ 0x4, 0x0, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0x9, 0x5, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0xc, 0x8, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 2, pre-emphasis 0->1 */
>> +	{
>> +		{ 0x8, 0x0, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0xc, 0x5, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 3, pre-emphasis 0 */
>> +	{
>> +		{ 0xb, 0x0, 0x7, 0x7, 0x1, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +	}
>> +};
>> +
>> +static struct tx_drv_ctrl tx_drv_ctrl_hbr[4][4] = {
>> +	/* voltage swing 0, pre-emphasis 0->3 */
>> +	{
>> +		{ 0x2, 0x0, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0x5, 0x4, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0x9, 0x8, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0xd, 0xc, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 1, pre-emphasis 0->2 */
>> +	{
>> +		{ 0x6, 0x1, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0xa, 0x6, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0xc, 0x8, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 2, pre-emphasis 0->1 */
>> +	{
>> +		{ 0x9, 0x1, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0xd, 0x6, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 3, pre-emphasis 0 */
>> +	{
>> +		{ 0xc, 0x1, 0x7, 0x7, 0x1, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +	}
>> +};
>> +
>> +static struct tx_drv_ctrl tx_drv_ctrl_hbr2[4][4] = {
>> +	/* voltage swing 0, pre-emphasis 0->3 */
>> +	{
>> +		{ 0x2, 0x1, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0x5, 0x4, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0x9, 0x8, 0x4, 0x6, 0x1, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0xd, 0xc, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 1, pre-emphasis 0->2 */
>> +	{
>> +		{ 0x6, 0x1, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0xb, 0x7, 0x4, 0x6, 0x0, 0x4, 0x0, 0x1, 0x7, 0x7 },
>> +		{ 0xd, 0x9, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 2, pre-emphasis 0->1 */
>> +	{
>> +		{ 0x8, 0x1, 0x4, 0x6, 0x0, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +		{ 0xc, 0x6, 0x7, 0x7, 0x1, 0x7, 0x0, 0x1, 0x7, 0x7 },
>> +	},
>> +
>> +	/* voltage swing 3, pre-emphasis 0 */
>> +	{
>> +		{ 0xb, 0x0, 0x7, 0x7, 0x1, 0x4, 0x1, 0x1, 0x7, 0x7 },
>> +	}
>> +};
>> +
>>   static bool rk_hdptx_phy_is_rw_reg(struct device *dev, unsigned int reg)
>>   {
>>   	switch (reg) {
>> @@ -911,11 +1117,297 @@ static int rk_hdptx_phy_consumer_put(struct rk_hdptx_phy *hdptx, bool force)
>>   	return ret;
>>   }
>>   
>> +static void rk_hdptx_dp_reset(struct rk_hdptx_phy *hdptx)
>> +{
>> +	reset_control_assert(hdptx->rsts[RST_LANE].rstc);
>> +	reset_control_assert(hdptx->rsts[RST_CMN].rstc);
>> +	reset_control_assert(hdptx->rsts[RST_INIT].rstc);
>> +
>> +	reset_control_assert(hdptx->rsts[RST_APB].rstc);
>> +	udelay(10);
>> +	reset_control_deassert(hdptx->rsts[RST_APB].rstc);
>> +
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0301),
>> +			   OVRD_LN_TX_DRV_EI_EN_MASK | LN_TX_DRV_EI_EN_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_EI_EN_MASK, 1) |
>> +			   FIELD_PREP(LN_TX_DRV_EI_EN_MASK, 0));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0401),
>> +			   OVRD_LN_TX_DRV_EI_EN_MASK | LN_TX_DRV_EI_EN_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_EI_EN_MASK, 1) |
>> +			   FIELD_PREP(LN_TX_DRV_EI_EN_MASK, 0));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0501),
>> +			   OVRD_LN_TX_DRV_EI_EN_MASK | LN_TX_DRV_EI_EN_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_EI_EN_MASK, 1) |
>> +			   FIELD_PREP(LN_TX_DRV_EI_EN_MASK, 0));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0601),
>> +			   OVRD_LN_TX_DRV_EI_EN_MASK | LN_TX_DRV_EI_EN_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_EI_EN_MASK, 1) |
>> +			   FIELD_PREP(LN_TX_DRV_EI_EN_MASK, 0));
>> +
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_PLL_EN << 16 | FIELD_PREP(HDPTX_I_PLL_EN, 0x0));
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_BIAS_EN << 16 | FIELD_PREP(HDPTX_I_BIAS_EN, 0x0));
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_BGR_EN << 16 | FIELD_PREP(HDPTX_I_BGR_EN, 0x0));
>> +}
>> +
>> +static void rk_hdptx_dp_pll_init(struct rk_hdptx_phy *hdptx)
>> +{
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(003c), ANA_LCPLL_RESERVED7_MASK,
>> +			   FIELD_PREP(ANA_LCPLL_RESERVED7_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0046),
>> +			   ROPLL_ANA_CPP_CTRL_COARSE_MASK | ROPLL_ANA_CPP_CTRL_FINE_MASK,
>> +			   FIELD_PREP(ROPLL_ANA_CPP_CTRL_COARSE_MASK, 0xe) |
>> +			   FIELD_PREP(ROPLL_ANA_CPP_CTRL_FINE_MASK, 0xe));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0047),
>> +			   ROPLL_ANA_LPF_C_SEL_COARSE_MASK |
>> +			   ROPLL_ANA_LPF_C_SEL_FINE_MASK,
>> +			   FIELD_PREP(ROPLL_ANA_LPF_C_SEL_COARSE_MASK, 0x4) |
>> +			   FIELD_PREP(ROPLL_ANA_LPF_C_SEL_FINE_MASK, 0x4));
>> +
>> +	regmap_write(hdptx->regmap, CMN_REG(0051), FIELD_PREP(ROPLL_PMS_MDIV_MASK, 0x87));
>> +	regmap_write(hdptx->regmap, CMN_REG(0052), FIELD_PREP(ROPLL_PMS_MDIV_MASK, 0x71));
>> +	regmap_write(hdptx->regmap, CMN_REG(0053), FIELD_PREP(ROPLL_PMS_MDIV_MASK, 0x71));
>> +
>> +	regmap_write(hdptx->regmap, CMN_REG(0055),
>> +		     FIELD_PREP(ROPLL_PMS_MDIV_AFC_MASK, 0x87));
>> +	regmap_write(hdptx->regmap, CMN_REG(0056),
>> +		     FIELD_PREP(ROPLL_PMS_MDIV_AFC_MASK, 0x71));
>> +	regmap_write(hdptx->regmap, CMN_REG(0057),
>> +		     FIELD_PREP(ROPLL_PMS_MDIV_AFC_MASK, 0x71));
>> +
>> +	regmap_write(hdptx->regmap, CMN_REG(0059),
>> +		     FIELD_PREP(ANA_ROPLL_PMS_PDIV_MASK, 0x1) |
>> +		     FIELD_PREP(ANA_ROPLL_PMS_REFDIV_MASK, 0x1));
>> +	regmap_write(hdptx->regmap, CMN_REG(005a),
>> +		     FIELD_PREP(ROPLL_PMS_SDIV_RBR_MASK, 0x3) |
>> +		     FIELD_PREP(ROPLL_PMS_SDIV_HBR_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005b), ROPLL_PMS_SDIV_HBR2_MASK,
>> +			   FIELD_PREP(ROPLL_PMS_SDIV_HBR2_MASK, 0x0));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005e), ROPLL_SDM_EN_MASK,
>> +			   FIELD_PREP(ROPLL_SDM_EN_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005e),
>> +			   OVRD_ROPLL_SDM_RSTN_MASK | ROPLL_SDM_RSTN_MASK,
>> +			   FIELD_PREP(OVRD_ROPLL_SDM_RSTN_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_SDM_RSTN_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005e), ROPLL_SDC_FRAC_EN_RBR_MASK,
>> +			   FIELD_PREP(ROPLL_SDC_FRAC_EN_RBR_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005e), ROPLL_SDC_FRAC_EN_HBR_MASK,
>> +			   FIELD_PREP(ROPLL_SDC_FRAC_EN_HBR_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005e), ROPLL_SDC_FRAC_EN_HBR2_MASK,
>> +			   FIELD_PREP(ROPLL_SDC_FRAC_EN_HBR2_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(005f),
>> +			   OVRD_ROPLL_SDC_RSTN_MASK | ROPLL_SDC_RSTN_MASK,
>> +			   FIELD_PREP(OVRD_ROPLL_SDC_RSTN_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_SDC_RSTN_MASK, 0x1));
>> +	regmap_write(hdptx->regmap, CMN_REG(0060),
>> +		     FIELD_PREP(ROPLL_SDM_DENOMINATOR_MASK, 0x21));
>> +	regmap_write(hdptx->regmap, CMN_REG(0061),
>> +		     FIELD_PREP(ROPLL_SDM_DENOMINATOR_MASK, 0x27));
>> +	regmap_write(hdptx->regmap, CMN_REG(0062),
>> +		     FIELD_PREP(ROPLL_SDM_DENOMINATOR_MASK, 0x27));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0064),
>> +			   ROPLL_SDM_NUM_SIGN_RBR_MASK |
>> +			   ROPLL_SDM_NUM_SIGN_HBR_MASK |
>> +			   ROPLL_SDM_NUM_SIGN_HBR2_MASK,
>> +			   FIELD_PREP(ROPLL_SDM_NUM_SIGN_RBR_MASK, 0x0) |
>> +			   FIELD_PREP(ROPLL_SDM_NUM_SIGN_HBR_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_SDM_NUM_SIGN_HBR2_MASK, 0x1));
>> +	regmap_write(hdptx->regmap, CMN_REG(0065),
>> +		     FIELD_PREP(ROPLL_SDM_NUM_MASK, 0x0));
>> +	regmap_write(hdptx->regmap, CMN_REG(0066),
>> +		     FIELD_PREP(ROPLL_SDM_NUM_MASK, 0xd));
>> +	regmap_write(hdptx->regmap, CMN_REG(0067),
>> +		     FIELD_PREP(ROPLL_SDM_NUM_MASK, 0xd));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0069), ROPLL_SDC_N_RBR_MASK,
>> +			   FIELD_PREP(ROPLL_SDC_N_RBR_MASK, 0x2));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(006a),
>> +			   ROPLL_SDC_N_HBR_MASK | ROPLL_SDC_N_HBR2_MASK,
>> +			   FIELD_PREP(ROPLL_SDC_N_HBR_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_SDC_N_HBR2_MASK, 0x1));
>> +
>> +	regmap_write(hdptx->regmap, CMN_REG(006c),
>> +		     FIELD_PREP(ROPLL_SDC_NUM_MASK, 0x3));
>> +	regmap_write(hdptx->regmap, CMN_REG(006d),
>> +		     FIELD_PREP(ROPLL_SDC_NUM_MASK, 0x7));
>> +	regmap_write(hdptx->regmap, CMN_REG(006e),
>> +		     FIELD_PREP(ROPLL_SDC_NUM_MASK, 0x7));
>> +
>> +	regmap_write(hdptx->regmap, CMN_REG(0070),
>> +		     FIELD_PREP(ROPLL_SDC_DENO_MASK, 0x8));
>> +	regmap_write(hdptx->regmap, CMN_REG(0071),
>> +		     FIELD_PREP(ROPLL_SDC_DENO_MASK, 0x18));
>> +	regmap_write(hdptx->regmap, CMN_REG(0072),
>> +		     FIELD_PREP(ROPLL_SDC_DENO_MASK, 0x18));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0074),
>> +			   OVRD_ROPLL_SDC_NDIV_RSTN_MASK | ROPLL_SDC_NDIV_RSTN_MASK,
>> +			   FIELD_PREP(OVRD_ROPLL_SDC_NDIV_RSTN_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_SDC_NDIV_RSTN_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0077), ANA_ROPLL_SSC_CLK_DIV_SEL_MASK,
>> +			   FIELD_PREP(ANA_ROPLL_SSC_CLK_DIV_SEL_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0081), ANA_PLL_CD_TX_SER_RATE_SEL_MASK,
>> +			   FIELD_PREP(ANA_PLL_CD_TX_SER_RATE_SEL_MASK, 0x0));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0081),
>> +			   ANA_PLL_CD_HSCLK_EAST_EN_MASK | ANA_PLL_CD_HSCLK_WEST_EN_MASK,
>> +			   FIELD_PREP(ANA_PLL_CD_HSCLK_EAST_EN_MASK, 0x1) |
>> +			   FIELD_PREP(ANA_PLL_CD_HSCLK_WEST_EN_MASK, 0x0));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0082), ANA_PLL_CD_VREG_GAIN_CTRL_MASK,
>> +			   FIELD_PREP(ANA_PLL_CD_VREG_GAIN_CTRL_MASK, 0x4));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0083), ANA_PLL_CD_VREG_ICTRL_MASK,
>> +			   FIELD_PREP(ANA_PLL_CD_VREG_ICTRL_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0084), PLL_LCRO_CLK_SEL_MASK,
>> +			   FIELD_PREP(PLL_LCRO_CLK_SEL_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0085), ANA_PLL_SYNC_LOSS_DET_MODE_MASK,
>> +			   FIELD_PREP(ANA_PLL_SYNC_LOSS_DET_MODE_MASK, 0x3));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0087), ANA_PLL_TX_HS_CLK_EN_MASK,
>> +			   FIELD_PREP(ANA_PLL_TX_HS_CLK_EN_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0097), DIG_CLK_SEL_MASK,
>> +			   FIELD_PREP(DIG_CLK_SEL_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0099), CMN_ROPLL_ALONE_MODE,
>> +			   FIELD_PREP(CMN_ROPLL_ALONE_MODE, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(009a), HS_SPEED_SEL_MASK,
>> +			   FIELD_PREP(HS_SPEED_SEL_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(009b), LS_SPEED_SEL_MASK,
>> +			   FIELD_PREP(LS_SPEED_SEL_MASK, 0x1));
>> +}
>> +
>> +static int rk_hdptx_dp_aux_init(struct rk_hdptx_phy *hdptx)
>> +{
>> +	u32 status;
>> +	int ret;
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0102), ANA_SB_RXTERM_OFFSP_MASK,
>> +			   FIELD_PREP(ANA_SB_RXTERM_OFFSP_MASK, 0x3));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0103), ANA_SB_RXTERM_OFFSN_MASK,
>> +			   FIELD_PREP(ANA_SB_RXTERM_OFFSN_MASK, 0x3));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0104), SB_AUX_EN_MASK,
>> +			   FIELD_PREP(SB_AUX_EN_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0105), ANA_SB_TX_HLVL_PROG_MASK,
>> +			   FIELD_PREP(ANA_SB_TX_HLVL_PROG_MASK, 0x7));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0106), ANA_SB_TX_LLVL_PROG_MASK,
>> +			   FIELD_PREP(ANA_SB_TX_LLVL_PROG_MASK, 0x7));
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(010d), ANA_SB_DMRX_LPBK_DATA_MASK,
>> +			   FIELD_PREP(ANA_SB_DMRX_LPBK_DATA_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(010f), ANA_SB_VREG_GAIN_CTRL_MASK,
>> +			   FIELD_PREP(ANA_SB_VREG_GAIN_CTRL_MASK, 0x0));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0110),
>> +			   ANA_SB_VREG_OUT_SEL_MASK | ANA_SB_VREG_REF_SEL_MASK,
>> +			   FIELD_PREP(ANA_SB_VREG_OUT_SEL_MASK, 0x1) |
>> +			   FIELD_PREP(ANA_SB_VREG_REF_SEL_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0113),
>> +			   SB_RX_RCAL_OPT_CODE_MASK | SB_RX_RTERM_CTRL_MASK,
>> +			   FIELD_PREP(SB_RX_RCAL_OPT_CODE_MASK, 0x1) |
>> +			   FIELD_PREP(SB_RX_RTERM_CTRL_MASK, 0x3));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0114),
>> +			   SB_TG_SB_EN_DELAY_TIME_MASK | SB_TG_RXTERM_EN_DELAY_TIME_MASK,
>> +			   FIELD_PREP(SB_TG_SB_EN_DELAY_TIME_MASK, 0x2) |
>> +			   FIELD_PREP(SB_TG_RXTERM_EN_DELAY_TIME_MASK, 0x2));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0115),
>> +			   SB_READY_DELAY_TIME_MASK | SB_TG_OSC_EN_DELAY_TIME_MASK,
>> +			   FIELD_PREP(SB_READY_DELAY_TIME_MASK, 0x2) |
>> +			   FIELD_PREP(SB_TG_OSC_EN_DELAY_TIME_MASK, 0x2));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0116),
>> +			   AFC_RSTN_DELAY_TIME_MASK,
>> +			   FIELD_PREP(AFC_RSTN_DELAY_TIME_MASK, 0x2));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0117),
>> +			   FAST_PULSE_TIME_MASK,
>> +			   FIELD_PREP(FAST_PULSE_TIME_MASK, 0x4));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0118),
>> +			   SB_TG_EARC_DMRX_RECVRD_CLK_CNT_MASK,
>> +			   FIELD_PREP(SB_TG_EARC_DMRX_RECVRD_CLK_CNT_MASK, 0xa));
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011a), SB_TG_CNT_RUN_NO_7_0_MASK,
>> +			   FIELD_PREP(SB_TG_CNT_RUN_NO_7_0_MASK, 0x3));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011b),
>> +			   SB_EARC_SIG_DET_BYPASS_MASK | SB_AFC_TOL_MASK,
>> +			   FIELD_PREP(SB_EARC_SIG_DET_BYPASS_MASK, 0x1) |
>> +			   FIELD_PREP(SB_AFC_TOL_MASK, 0x3));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011c), SB_AFC_STB_NUM_MASK,
>> +			   FIELD_PREP(SB_AFC_STB_NUM_MASK, 0x4));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011d), SB_TG_OSC_CNT_MIN_MASK,
>> +			   FIELD_PREP(SB_TG_OSC_CNT_MIN_MASK, 0x67));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011e), SB_TG_OSC_CNT_MAX_MASK,
>> +			   FIELD_PREP(SB_TG_OSC_CNT_MAX_MASK, 0x6a));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011f), SB_PWM_AFC_CTRL_MASK,
>> +			   FIELD_PREP(SB_PWM_AFC_CTRL_MASK, 0x5));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(011f), SB_RCAL_RSTN_MASK,
>> +			   FIELD_PREP(SB_RCAL_RSTN_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0120), SB_AUX_EN_IN_MASK,
>> +			   FIELD_PREP(SB_AUX_EN_IN_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0102), OVRD_SB_RXTERM_EN_MASK,
>> +			   FIELD_PREP(OVRD_SB_RXTERM_EN_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0103), OVRD_SB_RX_RESCAL_DONE_MASK,
>> +			   FIELD_PREP(OVRD_SB_RX_RESCAL_DONE_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0104), OVRD_SB_EN_MASK,
>> +			   FIELD_PREP(OVRD_SB_EN_MASK, 0x1));
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0104), OVRD_SB_AUX_EN_MASK,
>> +			   FIELD_PREP(OVRD_SB_AUX_EN_MASK, 0x1));
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(010f), OVRD_SB_VREG_EN_MASK,
>> +			   FIELD_PREP(OVRD_SB_VREG_EN_MASK, 0x1));
>> +
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_BGR_EN << 16 | FIELD_PREP(HDPTX_I_BGR_EN, 0x1));
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_BIAS_EN << 16 | FIELD_PREP(HDPTX_I_BIAS_EN, 0x1));
>> +	usleep_range(20, 25);
>> +
>> +	reset_control_deassert(hdptx->rsts[RST_INIT].rstc);
>> +	usleep_range(20, 25);
>> +	reset_control_deassert(hdptx->rsts[RST_CMN].rstc);
>> +	usleep_range(20, 25);
>> +
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0103), OVRD_SB_RX_RESCAL_DONE_MASK,
>> +			   FIELD_PREP(OVRD_SB_RX_RESCAL_DONE_MASK, 0x1));
>> +	usleep_range(100, 110);
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0104), SB_EN_MASK,
>> +			   FIELD_PREP(SB_EN_MASK, 0x1));
>> +	usleep_range(100, 110);
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0102), SB_RXTERM_EN_MASK,
>> +			   FIELD_PREP(SB_RXTERM_EN_MASK, 0x1));
>> +	usleep_range(20, 25);
>> +	regmap_update_bits(hdptx->regmap, SB_REG(010f), SB_VREG_EN_MASK,
>> +			   FIELD_PREP(SB_VREG_EN_MASK, 0x1));
>> +	usleep_range(20, 25);
>> +	regmap_update_bits(hdptx->regmap, SB_REG(0104), SB_AUX_EN_MASK,
>> +			   FIELD_PREP(SB_AUX_EN_MASK, 0x1));
>> +	usleep_range(100, 110);
>> +
>> +	ret = regmap_read_poll_timeout(hdptx->grf, GRF_HDPTX_STATUS,
>> +				       status, FIELD_GET(HDPTX_O_SB_RDY, status),
>> +				       50, 1000);
>> +	if (ret) {
>> +		dev_err(hdptx->dev, "Failed to get phy sb ready: %d\n", ret);
>> +		return ret;
>> +	}
>> +
>> +	return 0;
>> +}
>> +
>>   static int rk_hdptx_phy_power_on(struct phy *phy)
>>   {
>>   	struct rk_hdptx_phy *hdptx = phy_get_drvdata(phy);
>>   	int bus_width = phy_get_bus_width(hdptx->phy);
>> -	int ret;
>> +	enum phy_mode mode = phy_get_mode(phy);
>> +	int ret, lane;
>>   
>>   	/*
>>   	 * FIXME: Temporary workaround to pass pixel_clk_rate
>> @@ -927,13 +1419,43 @@ static int rk_hdptx_phy_power_on(struct phy *phy)
>>   	dev_dbg(hdptx->dev, "%s bus_width=%x rate=%u\n",
>>   		__func__, bus_width, rate);
>>   
>> -	ret = rk_hdptx_phy_consumer_get(hdptx, rate);
>> -	if (ret)
>> -		return ret;
>> +	if (mode == PHY_MODE_DP) {
>> +		rk_hdptx_dp_reset(hdptx);
>>   
>> -	ret = rk_hdptx_ropll_tmds_mode_config(hdptx, rate);
>> -	if (ret)
>> -		rk_hdptx_phy_consumer_put(hdptx, true);
>> +		for (lane = 0; lane < 4; lane++) {
>> +			regmap_update_bits(hdptx->regmap, LANE_REG(031e) + 0x400 * lane,
>> +					   LN_POLARITY_INV_MASK | LN_LANE_MODE_MASK,
>> +					   FIELD_PREP(LN_POLARITY_INV_MASK, 0) |
>> +					   FIELD_PREP(LN_LANE_MODE_MASK, 1));
>> +		}
>> +
>> +		regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +			     HDPTX_MODE_SEL << 16 | FIELD_PREP(HDPTX_MODE_SEL, 0x1));
>> +
>> +		regmap_update_bits(hdptx->regmap, LNTOP_REG(0200), PROTOCOL_SEL,
>> +				   FIELD_PREP(PROTOCOL_SEL, 0x0));
>> +		regmap_update_bits(hdptx->regmap, LNTOP_REG(0206), DATA_BUS_WIDTH_MASK,
>> +				   FIELD_PREP(DATA_BUS_WIDTH_MASK, 0x1));
>> +		regmap_update_bits(hdptx->regmap, LNTOP_REG(0206), DATA_BUS_WIDTH_SEL_MASK,
>> +				   FIELD_PREP(DATA_BUS_WIDTH_SEL_MASK, 0x0));
>> +
>> +		rk_hdptx_dp_pll_init(hdptx);
>> +
>> +		ret = rk_hdptx_dp_aux_init(hdptx);
>> +		if (ret)
>> +			pm_runtime_put(hdptx->dev);
>> +	} else {
>> +		regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +			     HDPTX_MODE_SEL << 16 | FIELD_PREP(HDPTX_MODE_SEL, 0x0));
>> +
>> +		ret = rk_hdptx_phy_consumer_get(hdptx, rate);
>> +		if (ret)
>> +			return ret;
>> +
>> +		ret = rk_hdptx_ropll_tmds_mode_config(hdptx, rate);
>> +		if (ret)
>> +			rk_hdptx_phy_consumer_put(hdptx, true);
>> +	}
>>   
>>   	return ret;
>>   }
>> @@ -945,9 +1467,337 @@ static int rk_hdptx_phy_power_off(struct phy *phy)
>>   	return rk_hdptx_phy_consumer_put(hdptx, false);
>>   }
>>   
>> +static int rk_hdptx_phy_set_mode(struct phy *phy, enum phy_mode mode,
>> +				 int submode)
>> +{
>> +	return 0;
>> +}
>> +
>> +static int rk_hdptx_phy_verify_config(struct rk_hdptx_phy *hdptx,
>> +				      struct phy_configure_opts_dp *dp)
>> +{
>> +	int i;
>> +
>> +	if (dp->set_rate) {
>> +		switch (dp->link_rate) {
>> +		case 1620:
>> +		case 2700:
>> +		case 5400:
>> +			break;
>> +		default:
>> +			return -EINVAL;
>> +		}
>> +	}
>> +
>> +	if (dp->set_lanes) {
>> +		switch (dp->lanes) {
>> +		case 0:
>> +		case 1:
>> +		case 2:
>> +		case 4:
>> +			break;
>> +		default:
>> +			return -EINVAL;
>> +		}
>> +	}
>> +
>> +	if (dp->set_voltages) {
>> +		for (i = 0; i < dp->lanes; i++) {
>> +			if (dp->voltage[i] > 3 || dp->pre[i] > 3)
>> +				return -EINVAL;
>> +
>> +			if (dp->voltage[i] + dp->pre[i] > 3)
>> +				return -EINVAL;
>> +		}
>> +	}
>> +
>> +	return 0;
>> +}
>> +
>> +static int rk_hdptx_phy_set_rate(struct rk_hdptx_phy *hdptx,
>> +				 struct phy_configure_opts_dp *dp)
>> +{
>> +	u32 bw, status;
>> +	int ret;
>> +
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_PLL_EN << 16 | FIELD_PREP(HDPTX_I_PLL_EN, 0x0));
>> +
>> +	switch (dp->link_rate) {
>> +	case 1620:
>> +		bw = DP_BW_RBR;
>> +		break;
>> +	case 2700:
>> +		bw = DP_BW_HBR;
>> +		break;
>> +	case 5400:
>> +		bw = DP_BW_HBR2;
>> +		break;
>> +	default:
>> +		return -EINVAL;
>> +	}
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0008), OVRD_LCPLL_EN_MASK | LCPLL_EN_MASK,
>> +			   FIELD_PREP(OVRD_LCPLL_EN_MASK, 0x1) |
>> +			   FIELD_PREP(LCPLL_EN_MASK, 0x0));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(003d), OVRD_ROPLL_EN_MASK | ROPLL_EN_MASK,
>> +			   FIELD_PREP(OVRD_ROPLL_EN_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_EN_MASK, 0x1));
>> +
>> +	if (dp->ssc) {
>> +		regmap_update_bits(hdptx->regmap, CMN_REG(0074),
>> +				   OVRD_ROPLL_SSC_EN_MASK | ROPLL_SSC_EN_MASK,
>> +				   FIELD_PREP(OVRD_ROPLL_SSC_EN_MASK, 0x1) |
>> +				   FIELD_PREP(ROPLL_SSC_EN_MASK, 0x1));
>> +		regmap_write(hdptx->regmap, CMN_REG(0075),
>> +			     FIELD_PREP(ANA_ROPLL_SSC_FM_DEVIATION_MASK, 0xc));
>> +		regmap_update_bits(hdptx->regmap, CMN_REG(0076),
>> +				   ANA_ROPLL_SSC_FM_FREQ_MASK,
>> +				   FIELD_PREP(ANA_ROPLL_SSC_FM_FREQ_MASK, 0x1f));
>> +
>> +		regmap_update_bits(hdptx->regmap, CMN_REG(0099), SSC_EN_MASK,
>> +				   FIELD_PREP(SSC_EN_MASK, 0x2));
>> +	} else {
>> +		regmap_update_bits(hdptx->regmap, CMN_REG(0074),
>> +				   OVRD_ROPLL_SSC_EN_MASK | ROPLL_SSC_EN_MASK,
>> +				   FIELD_PREP(OVRD_ROPLL_SSC_EN_MASK, 0x1) |
>> +				   FIELD_PREP(ROPLL_SSC_EN_MASK, 0x0));
>> +		regmap_write(hdptx->regmap, CMN_REG(0075),
>> +			     FIELD_PREP(ANA_ROPLL_SSC_FM_DEVIATION_MASK, 0x20));
>> +		regmap_update_bits(hdptx->regmap, CMN_REG(0076),
>> +				   ANA_ROPLL_SSC_FM_FREQ_MASK,
>> +				   FIELD_PREP(ANA_ROPLL_SSC_FM_FREQ_MASK, 0xc));
>> +
>> +		regmap_update_bits(hdptx->regmap, CMN_REG(0099), SSC_EN_MASK,
>> +				   FIELD_PREP(SSC_EN_MASK, 0x0));
>> +	}
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0095), DP_TX_LINK_BW_MASK,
>> +			   FIELD_PREP(DP_TX_LINK_BW_MASK, bw));
>> +
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_PLL_EN << 16 | FIELD_PREP(HDPTX_I_PLL_EN, 0x1));
>> +
>> +	ret = regmap_read_poll_timeout(hdptx->grf, GRF_HDPTX_STATUS,
>> +				       status, FIELD_GET(HDPTX_O_PLL_LOCK_DONE, status),
>> +				       50, 1000);
>> +	if (ret) {
>> +		dev_err(hdptx->dev, "Failed to get phy pll lock: %d\n", ret);
>> +		return ret;
>> +	}
>> +
>> +	return 0;
>> +}
>> +
>> +static void rk_hdptx_phy_lane_disable(struct rk_hdptx_phy *hdptx)
>> +{
>> +	reset_control_assert(hdptx->rsts[RST_LANE].rstc);
>> +
>> +	regmap_update_bits(hdptx->regmap, LNTOP_REG(0207), LANE_EN,
>> +			   FIELD_PREP(LANE_EN, 0x0));
>> +
>> +	regmap_write(hdptx->grf, GRF_HDPTX_CON0,
>> +		     HDPTX_I_PLL_EN << 16 | FIELD_PREP(HDPTX_I_PLL_EN, 0x0));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(0008), OVRD_LCPLL_EN_MASK | LCPLL_EN_MASK,
>> +			   FIELD_PREP(OVRD_LCPLL_EN_MASK, 0x1) |
>> +			   FIELD_PREP(LCPLL_EN_MASK, 0x0));
>> +
>> +	regmap_update_bits(hdptx->regmap, CMN_REG(003d), OVRD_ROPLL_EN_MASK | ROPLL_EN_MASK,
>> +			   FIELD_PREP(OVRD_ROPLL_EN_MASK, 0x1) |
>> +			   FIELD_PREP(ROPLL_EN_MASK, 0x0));
>> +}
>> +
>> +static int rk_hdptx_phy_set_lanes(struct rk_hdptx_phy *hdptx,
>> +				  struct phy_configure_opts_dp *dp)
>> +{
>> +	if (!dp->lanes) {
>> +		rk_hdptx_phy_lane_disable(hdptx);
>> +		return 0;
>> +	}
>> +
>> +	regmap_update_bits(hdptx->regmap, LNTOP_REG(0207), LANE_EN,
>> +			   FIELD_PREP(LANE_EN, GENMASK(dp->lanes - 1, 0)));
>> +
>> +	return 0;
>> +}
>> +
>> +static void rk_hdptx_phy_set_voltage(struct rk_hdptx_phy *hdptx,
>> +				     struct phy_configure_opts_dp *dp,
>> +				     u8 lane)
>> +{
>> +	const struct tx_drv_ctrl *ctrl;
>> +	u32 offset = lane * 0x400;
>> +
>> +	switch (dp->link_rate) {
>> +	case 1620:
>> +		ctrl = &tx_drv_ctrl_rbr[dp->voltage[lane]][dp->pre[lane]];
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(030a) + offset,
>> +				   LN_TX_JEQ_EVEN_CTRL_RBR_MASK,
>> +				   FIELD_PREP(LN_TX_JEQ_EVEN_CTRL_RBR_MASK,
>> +				   ctrl->tx_jeq_even_ctrl));
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(030c) + offset,
>> +				   LN_TX_JEQ_ODD_CTRL_RBR_MASK,
>> +				   FIELD_PREP(LN_TX_JEQ_ODD_CTRL_RBR_MASK,
>> +				   ctrl->tx_jeq_odd_ctrl));
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(0311) + offset,
>> +				   LN_TX_SER_40BIT_EN_RBR_MASK,
>> +				   FIELD_PREP(LN_TX_SER_40BIT_EN_RBR_MASK, 0x1));
>> +		break;
>> +	case 2700:
>> +		ctrl = &tx_drv_ctrl_hbr[dp->voltage[lane]][dp->pre[lane]];
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(030b) + offset,
>> +				   LN_TX_JEQ_EVEN_CTRL_HBR_MASK,
>> +				   FIELD_PREP(LN_TX_JEQ_EVEN_CTRL_HBR_MASK,
>> +				   ctrl->tx_jeq_even_ctrl));
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(030d) + offset,
>> +				   LN_TX_JEQ_ODD_CTRL_HBR_MASK,
>> +				   FIELD_PREP(LN_TX_JEQ_ODD_CTRL_HBR_MASK,
>> +				   ctrl->tx_jeq_odd_ctrl));
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(0311) + offset,
>> +				   LN_TX_SER_40BIT_EN_HBR_MASK,
>> +				   FIELD_PREP(LN_TX_SER_40BIT_EN_HBR_MASK, 0x1));
>> +		break;
>> +	case 5400:
>> +	default:
>> +		ctrl = &tx_drv_ctrl_hbr2[dp->voltage[lane]][dp->pre[lane]];
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(030b) + offset,
>> +				   LN_TX_JEQ_EVEN_CTRL_HBR2_MASK,
>> +				   FIELD_PREP(LN_TX_JEQ_EVEN_CTRL_HBR2_MASK,
>> +				   ctrl->tx_jeq_even_ctrl));
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(030d) + offset,
>> +				   LN_TX_JEQ_ODD_CTRL_HBR2_MASK,
>> +				   FIELD_PREP(LN_TX_JEQ_ODD_CTRL_HBR2_MASK,
>> +				   ctrl->tx_jeq_odd_ctrl));
>> +		regmap_update_bits(hdptx->regmap, LANE_REG(0311) + offset,
>> +				   LN_TX_SER_40BIT_EN_HBR2_MASK,
>> +				   FIELD_PREP(LN_TX_SER_40BIT_EN_HBR2_MASK, 0x1));
>> +		break;
>> +	}
>> +
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0303) + offset,
>> +			   OVRD_LN_TX_DRV_LVL_CTRL_MASK | LN_TX_DRV_LVL_CTRL_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_LVL_CTRL_MASK, 0x1) |
>> +			   FIELD_PREP(LN_TX_DRV_LVL_CTRL_MASK,
>> +				      ctrl->tx_drv_lvl_ctrl));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0304) + offset,
>> +			   OVRD_LN_TX_DRV_POST_LVL_CTRL_MASK |
>> +			   LN_TX_DRV_POST_LVL_CTRL_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_POST_LVL_CTRL_MASK, 0x1) |
>> +			   FIELD_PREP(LN_TX_DRV_POST_LVL_CTRL_MASK,
>> +				      ctrl->tx_drv_post_lvl_ctrl));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0305) + offset,
>> +			   OVRD_LN_TX_DRV_PRE_LVL_CTRL_MASK |
>> +			   LN_TX_DRV_PRE_LVL_CTRL_MASK,
>> +			   FIELD_PREP(OVRD_LN_TX_DRV_PRE_LVL_CTRL_MASK, 0x1) |
>> +			   FIELD_PREP(LN_TX_DRV_PRE_LVL_CTRL_MASK,
>> +				      ctrl->tx_drv_pre_lvl_ctrl));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0306) + offset,
>> +			   LN_ANA_TX_DRV_IDRV_IDN_CTRL_MASK |
>> +			   LN_ANA_TX_DRV_IDRV_IUP_CTRL_MASK |
>> +			   LN_ANA_TX_DRV_ACCDRV_EN_MASK,
>> +			   FIELD_PREP(LN_ANA_TX_DRV_IDRV_IDN_CTRL_MASK,
>> +				      ctrl->ana_tx_drv_idrv_idn_ctrl) |
>> +			   FIELD_PREP(LN_ANA_TX_DRV_IDRV_IUP_CTRL_MASK,
>> +				      ctrl->ana_tx_drv_idrv_iup_ctrl) |
>> +			   FIELD_PREP(LN_ANA_TX_DRV_ACCDRV_EN_MASK,
>> +				      ctrl->ana_tx_drv_accdrv_en));
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0307) + offset,
>> +			   LN_ANA_TX_DRV_ACCDRV_POL_SEL_MASK |
>> +			   LN_ANA_TX_DRV_ACCDRV_CTRL_MASK,
>> +			   FIELD_PREP(LN_ANA_TX_DRV_ACCDRV_POL_SEL_MASK, 0x1) |
>> +			   FIELD_PREP(LN_ANA_TX_DRV_ACCDRV_CTRL_MASK,
>> +				      ctrl->ana_tx_drv_accdrv_ctrl));
>> +
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(030a) + offset,
>> +			   LN_ANA_TX_JEQ_EN_MASK,
>> +			   FIELD_PREP(LN_ANA_TX_JEQ_EN_MASK, ctrl->ana_tx_jeq_en));
>> +
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0310) + offset,
>> +			   LN_ANA_TX_SYNC_LOSS_DET_MODE_MASK,
>> +			   FIELD_PREP(LN_ANA_TX_SYNC_LOSS_DET_MODE_MASK, 0x3));
>> +
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(0316) + offset,
>> +			   LN_ANA_TX_SER_VREG_GAIN_CTRL_MASK,
>> +			   FIELD_PREP(LN_ANA_TX_SER_VREG_GAIN_CTRL_MASK, 0x2));
>> +
>> +	regmap_update_bits(hdptx->regmap, LANE_REG(031b) + offset,
>> +			   LN_ANA_TX_RESERVED_MASK,
>> +			   FIELD_PREP(LN_ANA_TX_RESERVED_MASK, 0x1));
>> +}
>> +
>> +static int rk_hdptx_phy_set_voltages(struct rk_hdptx_phy *hdptx,
>> +				     struct phy_configure_opts_dp *dp)
>> +{
>> +	u8 lane;
>> +	u32 status;
>> +	int ret;
>> +
>> +	for (lane = 0; lane < dp->lanes; lane++)
>> +		rk_hdptx_phy_set_voltage(hdptx, dp, lane);
>> +
>> +	reset_control_deassert(hdptx->rsts[RST_LANE].rstc);
>> +
>> +	ret = regmap_read_poll_timeout(hdptx->grf, GRF_HDPTX_STATUS,
>> +				       status, FIELD_GET(HDPTX_O_PHY_RDY, status),
>> +				       50, 5000);
>> +	if (ret) {
>> +		dev_err(hdptx->dev, "Failed to get phy ready: %d\n", ret);
>> +		return ret;
>> +	}
>> +
>> +	return 0;
>> +}
>> +
>> +static int rk_hdptx_phy_configure(struct phy *phy, union phy_configure_opts *opts)
>> +{
>> +	struct rk_hdptx_phy *hdptx = phy_get_drvdata(phy);
>> +	enum phy_mode mode = phy_get_mode(phy);
>> +	int ret;
>> +
>> +	if (mode != PHY_MODE_DP)
>> +		return -EINVAL;
>> +
>> +	ret = rk_hdptx_phy_verify_config(hdptx, &opts->dp);
>> +	if (ret) {
>> +		dev_err(hdptx->dev, "invalid params for phy configure\n");
>> +		return ret;
>> +	}
>> +
>> +	if (opts->dp.set_rate) {
>> +		ret = rk_hdptx_phy_set_rate(hdptx, &opts->dp);
>> +		if (ret) {
>> +			dev_err(hdptx->dev, "failed to set rate: %d\n", ret);
>> +			return ret;
>> +		}
>> +	}
>> +
>> +	if (opts->dp.set_lanes) {
>> +		ret = rk_hdptx_phy_set_lanes(hdptx, &opts->dp);
>> +		if (ret) {
>> +			dev_err(hdptx->dev, "failed to set lanes: %d\n", ret);
>> +			return ret;
>> +		}
>> +	}
>> +
>> +	if (opts->dp.set_voltages) {
>> +		ret = rk_hdptx_phy_set_voltages(hdptx, &opts->dp);
>> +		if (ret) {
>> +			dev_err(hdptx->dev, "failed to set voltages: %d\n",
>> +				ret);
>> +			return ret;
>> +		}
>> +	}
>> +
>> +	return 0;
>> +}
>> +
>>   static const struct phy_ops rk_hdptx_phy_ops = {
>>   	.power_on  = rk_hdptx_phy_power_on,
>>   	.power_off = rk_hdptx_phy_power_off,
>> +	.set_mode  = rk_hdptx_phy_set_mode,
>> +	.configure = rk_hdptx_phy_configure,
>>   	.owner	   = THIS_MODULE,
>>   };
>>   
>> @@ -1164,5 +2014,6 @@ module_platform_driver(rk_hdptx_phy_driver);
>>   
>>   MODULE_AUTHOR("Algea Cao <algea.cao@...k-chips.com>");
>>   MODULE_AUTHOR("Cristian Ciocaltea <cristian.ciocaltea@...labora.com>");
>> +MODULE_AUTHOR("Damon Ding <damon.ding@...k-chips.com>");
>>   MODULE_DESCRIPTION("Samsung HDMI/eDP Transmitter Combo PHY Driver");
>>   MODULE_LICENSE("GPL");
>> -- 
>> 2.34.1
>>
> 

Best regards,
Damon

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ