Re: [PATCH 2/2] drm/msm/dsi: Remove dsi_phy_write_[un]delay()

2024-04-23 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 12:37:00AM +0200, Konrad Dybcio wrote:
> These are dummy wrappers that do literally nothing interesting.
> Remove them.
> 
> Signed-off-by: Konrad Dybcio 
> ---
>  drivers/gpu/drm/msm/dsi/phy/dsi_phy.h  |  3 --
>  drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c |  3 +-
>  drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c | 81 
> +++---
>  3 files changed, 54 insertions(+), 33 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov 

-- 
With best wishes
Dmitry


[PATCH 2/2] drm/msm/dsi: Remove dsi_phy_write_[un]delay()

2024-04-22 Thread Konrad Dybcio
These are dummy wrappers that do literally nothing interesting.
Remove them.

Signed-off-by: Konrad Dybcio 
---
 drivers/gpu/drm/msm/dsi/phy/dsi_phy.h  |  3 --
 drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c |  3 +-
 drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c | 81 +++---
 3 files changed, 54 insertions(+), 33 deletions(-)

diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
index 7df4d852e6fa..109d767a476b 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
@@ -12,9 +12,6 @@
 
 #include "dsi.h"
 
-#define dsi_phy_write_udelay(offset, data, delay_us) { writel((data), 
(offset)); udelay(delay_us); }
-#define dsi_phy_write_ndelay(offset, data, delay_ns) { writel((data), 
(offset)); ndelay(delay_ns); }
-
 struct msm_dsi_phy_ops {
int (*pll_init)(struct msm_dsi_phy *phy);
int (*enable)(struct msm_dsi_phy *phy,
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c
index b128c4acea23..1723f0e4faa4 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c
@@ -374,7 +374,8 @@ static void pll_14nm_software_reset(struct dsi_pll_14nm 
*pll_14nm)
writel(0, cmn_base + REG_DSI_14nm_PHY_CMN_PLL_CNTRL);
 
/* pll sw reset */
-   dsi_phy_write_udelay(cmn_base + REG_DSI_14nm_PHY_CMN_CTRL_1, 0x20, 10);
+   writel(0x20, cmn_base + REG_DSI_14nm_PHY_CMN_CTRL_1);
+   udelay(10);
wmb();  /* make sure register committed */
 
writel(0, cmn_base + REG_DSI_14nm_PHY_CMN_CTRL_1);
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c
index b3e914954f4a..db99526d11df 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c
@@ -104,9 +104,10 @@ static void pll_28nm_software_reset(struct dsi_pll_28nm 
*pll_28nm)
 * Add HW recommended delays after toggling the software
 * reset bit off and back on.
 */
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_TEST_CFG,
-DSI_28nm_PHY_PLL_TEST_CFG_PLL_SW_RESET, 1);
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_TEST_CFG, 0x00, 1);
+   writel(DSI_28nm_PHY_PLL_TEST_CFG_PLL_SW_RESET, base + 
REG_DSI_28nm_PHY_PLL_TEST_CFG);
+   udelay(1);
+   writel(0, base + REG_DSI_28nm_PHY_PLL_TEST_CFG);
+   udelay(1);
 }
 
 /*
@@ -303,21 +304,25 @@ static int _dsi_pll_28nm_vco_prepare_hpm(struct 
dsi_pll_28nm *pll_28nm)
 * Add necessary delays recommended by hardware.
 */
val = DSI_28nm_PHY_PLL_GLB_CFG_PLL_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 1);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(1);
 
val |= DSI_28nm_PHY_PLL_GLB_CFG_PLL_PWRGEN_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 200);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(200);
 
val |= DSI_28nm_PHY_PLL_GLB_CFG_PLL_LDO_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 500);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(500);
 
val |= DSI_28nm_PHY_PLL_GLB_CFG_PLL_ENABLE;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 600);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(600);
 
for (i = 0; i < 2; i++) {
/* DSI Uniphy lock detect setting */
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_LKDET_CFG2,
-0x0c, 100);
+   writel(0x0c, base + REG_DSI_28nm_PHY_PLL_LKDET_CFG2);
+   udelay(100);
writel(0x0d, base + REG_DSI_28nm_PHY_PLL_LKDET_CFG2);
 
/* poll for PLL ready status */
@@ -333,22 +338,28 @@ static int _dsi_pll_28nm_vco_prepare_hpm(struct 
dsi_pll_28nm *pll_28nm)
 * Add necessary delays recommended by hardware.
 */
val = DSI_28nm_PHY_PLL_GLB_CFG_PLL_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 
1);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(1);
 
val |= DSI_28nm_PHY_PLL_GLB_CFG_PLL_PWRGEN_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 
200);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(200);
 
val |= DSI_28nm_PHY_PLL_GLB_CFG_PLL_LDO_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG, val, 
250);
+   writel(val, base + REG_DSI_28nm_PHY_PLL_GLB_CFG);
+   udelay(250);
 
val &= ~DSI_28nm_PHY_PLL_GLB_CFG_PLL_LDO_PWRDN_B;
-   dsi_phy_write_udelay(base + REG_DSI_28nm_PHY_PLL_GLB_CFG,