* [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
@ 2021-12-08 18:01 ` Dmitry Baryshkov
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2021-12-08 18:01 UTC (permalink / raw)
To: Bjorn Andersson, Rob Clark, Sean Paul, Abhinav Kumar
Cc: Stephen Boyd, linux-arm-msm, dri-devel, David Airlie, freedreno
msm_ioremap() functions take additional argument dbgname used to output
single debug line telling IO range. Drop that extra argument, use
resource name instead.
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
drivers/gpu/drm/msm/msm_gpu.c | 2 +-
15 files changed, 34 insertions(+), 40 deletions(-)
diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
index 51b83776951b..5750a3af126d 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
+++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
@@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
if (a6xx_gpu->have_mmu500)
a6xx_gpu->llc_mmio = NULL;
else
- a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
+ a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
index 2bd258eaf334..f86419cb2b4b 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
@@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
atomic_set(&dpu_kms->bandwidth_ref, 0);
- dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
+ dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
if (IS_ERR(dpu_kms->mmio)) {
rc = PTR_ERR(dpu_kms->mmio);
DPU_ERROR("mdp register memory map failed: %d\n", rc);
@@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
}
DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
- dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
+ dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
DPU_ERROR("vbif register memory map failed: %d\n", rc);
dpu_kms->vbif[VBIF_RT] = NULL;
goto error;
}
- dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
+ dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
dpu_kms->vbif[VBIF_NRT] = NULL;
DPU_DEBUG("VBIF NRT is not defined");
}
- dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
+ dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
if (IS_ERR(dpu_kms->reg_dma)) {
dpu_kms->reg_dma = NULL;
DPU_DEBUG("REG_DMA is not defined");
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
index 131c1f1a869c..5bdb1504035c 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
@@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
if (!dpu_mdss)
return -ENOMEM;
- dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
+ dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
if (IS_ERR(dpu_mdss->mmio))
return PTR_ERR(dpu_mdss->mmio);
diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
index 5a33bb148e9e..3cf476c55158 100644
--- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
+++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
@@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
mdp4_kms->dev = dev;
- mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
+ mdp4_kms->mmio = msm_ioremap(pdev, NULL);
if (IS_ERR(mdp4_kms->mmio)) {
ret = PTR_ERR(mdp4_kms->mmio);
goto fail;
diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
index 12a5f81e402b..81bd434980cf 100644
--- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
+++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
@@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
if (ret)
goto fail;
- mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
+ mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
if (IS_ERR(mdp5_kms->mmio)) {
ret = PTR_ERR(mdp5_kms->mmio);
goto fail;
diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
index b3f79c2277e9..049c6784a531 100644
--- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
+++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
@@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
mdp5_mdss->base.dev = &pdev->dev;
- mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
+ mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
if (IS_ERR(mdp5_mdss->mmio)) {
ret = PTR_ERR(mdp5_mdss->mmio);
goto fail;
}
- mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
+ mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
if (IS_ERR(mdp5_mdss->vbif)) {
ret = PTR_ERR(mdp5_mdss->vbif);
goto fail;
diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
index 5b4bb722f750..4e2dfc53ca28 100644
--- a/drivers/gpu/drm/msm/dsi/dsi_host.c
+++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
@@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
goto fail;
}
- msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
+ msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
if (IS_ERR(msm_host->ctrl_base)) {
pr_err("%s: unable to map Dsi ctrl base\n", __func__);
ret = PTR_ERR(msm_host->ctrl_base);
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
index 0b2ae5c15240..6425b6c34f4c 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
@@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
- phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
+ phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
if (IS_ERR(phy->base)) {
DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
ret = -ENOMEM;
goto fail;
}
- phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
+ phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
if (IS_ERR(phy->pll_base)) {
DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
ret = -ENOMEM;
@@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
}
if (phy->cfg->has_phy_lane) {
- phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
+ phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
if (IS_ERR(phy->lane_base)) {
DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
ret = -ENOMEM;
@@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
}
if (phy->cfg->has_phy_regulator) {
- phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
+ phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
if (IS_ERR(phy->reg_base)) {
DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
ret = -ENOMEM;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
index 3acdeae25caf..3eb45cde0330 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
@@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
hdmi->config = config;
spin_lock_init(&hdmi->reg_lock);
- hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
+ hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
if (IS_ERR(hdmi->mmio)) {
ret = PTR_ERR(hdmi->mmio);
goto fail;
@@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
config->mmio_name);
hdmi->mmio_phy_addr = res->start;
- hdmi->qfprom_mmio = msm_ioremap(pdev,
- config->qfprom_mmio_name, "HDMI_QFPROM");
+ hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
if (IS_ERR(hdmi->qfprom_mmio)) {
DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
hdmi->qfprom_mmio = NULL;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
index 16b0e8836d27..9780107e1cc9 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
@@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
if (!phy->cfg)
return -ENODEV;
- phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
+ phy->mmio = msm_ioremap(pdev, "hdmi_phy");
if (IS_ERR(phy->mmio)) {
DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
return -ENOMEM;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
index 99c7853353fd..b06d9d25a189 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
@@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
pll->pdev = pdev;
- pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
+ pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
if (IS_ERR(pll->mmio_qserdes_com)) {
DRM_DEV_ERROR(dev, "failed to map pll base\n");
return -ENOMEM;
}
for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
- char name[32], label[32];
+ char name[32];
snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
- snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
- pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
+ pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
if (IS_ERR(pll->mmio_qserdes_tx[i])) {
DRM_DEV_ERROR(dev, "failed to map pll base\n");
return -ENOMEM;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
index 562dfac67792..be4b0b67e797 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
@@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
if (!pll)
return -ENOMEM;
- pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
+ pll->mmio = msm_ioremap(pdev, "hdmi_pll");
if (IS_ERR(pll->mmio)) {
DRM_DEV_ERROR(dev, "failed to map pll base\n");
return -ENOMEM;
diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index ad35a5d94053..d593c5d7dbc7 100644
--- a/drivers/gpu/drm/msm/msm_drv.c
+++ b/drivers/gpu/drm/msm/msm_drv.c
@@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
}
static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
- const char *dbgname, bool quiet, phys_addr_t *psize)
+ bool quiet, phys_addr_t *psize)
{
struct resource *res;
unsigned long size;
@@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
}
if (reglog)
- printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
+ printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
if (psize)
*psize = size;
@@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
return ptr;
}
-void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
- const char *dbgname)
+void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
{
- return _msm_ioremap(pdev, name, dbgname, false, NULL);
+ return _msm_ioremap(pdev, name, false, NULL);
}
-void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
- const char *dbgname)
+void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
{
- return _msm_ioremap(pdev, name, dbgname, true, NULL);
+ return _msm_ioremap(pdev, name, true, NULL);
}
void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
- const char *dbgname, phys_addr_t *psize)
+ phys_addr_t *psize)
{
- return _msm_ioremap(pdev, name, dbgname, false, psize);
+ return _msm_ioremap(pdev, name, false, psize);
}
void msm_writel(u32 data, void __iomem *addr)
diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
index d7574e6bd4e4..56d1242efcc1 100644
--- a/drivers/gpu/drm/msm/msm_drv.h
+++ b/drivers/gpu/drm/msm/msm_drv.h
@@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
const char *name);
-void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
- const char *dbgname);
+void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
- const char *dbgname, phys_addr_t *size);
-void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
- const char *dbgname);
+ phys_addr_t *size);
+void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
void msm_writel(u32 data, void __iomem *addr);
u32 msm_readl(const void __iomem *addr);
void msm_rmw(void __iomem *addr, u32 mask, u32 or);
diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
index 0f78c2615272..c4debc8ea3ba 100644
--- a/drivers/gpu/drm/msm/msm_gpu.c
+++ b/drivers/gpu/drm/msm/msm_gpu.c
@@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
/* Map registers: */
- gpu->mmio = msm_ioremap(pdev, config->ioname, name);
+ gpu->mmio = msm_ioremap(pdev, config->ioname);
if (IS_ERR(gpu->mmio)) {
ret = PTR_ERR(gpu->mmio);
goto fail;
--
2.33.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
@ 2021-12-08 18:01 ` Dmitry Baryshkov
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2021-12-08 18:01 UTC (permalink / raw)
To: Bjorn Andersson, Rob Clark, Sean Paul, Abhinav Kumar
Cc: Stephen Boyd, David Airlie, Daniel Vetter, linux-arm-msm,
dri-devel, freedreno
msm_ioremap() functions take additional argument dbgname used to output
single debug line telling IO range. Drop that extra argument, use
resource name instead.
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
drivers/gpu/drm/msm/msm_gpu.c | 2 +-
15 files changed, 34 insertions(+), 40 deletions(-)
diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
index 51b83776951b..5750a3af126d 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
+++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
@@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
if (a6xx_gpu->have_mmu500)
a6xx_gpu->llc_mmio = NULL;
else
- a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
+ a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
index 2bd258eaf334..f86419cb2b4b 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
@@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
atomic_set(&dpu_kms->bandwidth_ref, 0);
- dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
+ dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
if (IS_ERR(dpu_kms->mmio)) {
rc = PTR_ERR(dpu_kms->mmio);
DPU_ERROR("mdp register memory map failed: %d\n", rc);
@@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
}
DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
- dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
+ dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
DPU_ERROR("vbif register memory map failed: %d\n", rc);
dpu_kms->vbif[VBIF_RT] = NULL;
goto error;
}
- dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
+ dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
dpu_kms->vbif[VBIF_NRT] = NULL;
DPU_DEBUG("VBIF NRT is not defined");
}
- dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
+ dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
if (IS_ERR(dpu_kms->reg_dma)) {
dpu_kms->reg_dma = NULL;
DPU_DEBUG("REG_DMA is not defined");
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
index 131c1f1a869c..5bdb1504035c 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
@@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
if (!dpu_mdss)
return -ENOMEM;
- dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
+ dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
if (IS_ERR(dpu_mdss->mmio))
return PTR_ERR(dpu_mdss->mmio);
diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
index 5a33bb148e9e..3cf476c55158 100644
--- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
+++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
@@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
mdp4_kms->dev = dev;
- mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
+ mdp4_kms->mmio = msm_ioremap(pdev, NULL);
if (IS_ERR(mdp4_kms->mmio)) {
ret = PTR_ERR(mdp4_kms->mmio);
goto fail;
diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
index 12a5f81e402b..81bd434980cf 100644
--- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
+++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
@@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
if (ret)
goto fail;
- mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
+ mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
if (IS_ERR(mdp5_kms->mmio)) {
ret = PTR_ERR(mdp5_kms->mmio);
goto fail;
diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
index b3f79c2277e9..049c6784a531 100644
--- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
+++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
@@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
mdp5_mdss->base.dev = &pdev->dev;
- mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
+ mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
if (IS_ERR(mdp5_mdss->mmio)) {
ret = PTR_ERR(mdp5_mdss->mmio);
goto fail;
}
- mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
+ mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
if (IS_ERR(mdp5_mdss->vbif)) {
ret = PTR_ERR(mdp5_mdss->vbif);
goto fail;
diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
index 5b4bb722f750..4e2dfc53ca28 100644
--- a/drivers/gpu/drm/msm/dsi/dsi_host.c
+++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
@@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
goto fail;
}
- msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
+ msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
if (IS_ERR(msm_host->ctrl_base)) {
pr_err("%s: unable to map Dsi ctrl base\n", __func__);
ret = PTR_ERR(msm_host->ctrl_base);
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
index 0b2ae5c15240..6425b6c34f4c 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
@@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
- phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
+ phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
if (IS_ERR(phy->base)) {
DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
ret = -ENOMEM;
goto fail;
}
- phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
+ phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
if (IS_ERR(phy->pll_base)) {
DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
ret = -ENOMEM;
@@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
}
if (phy->cfg->has_phy_lane) {
- phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
+ phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
if (IS_ERR(phy->lane_base)) {
DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
ret = -ENOMEM;
@@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
}
if (phy->cfg->has_phy_regulator) {
- phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
+ phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
if (IS_ERR(phy->reg_base)) {
DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
ret = -ENOMEM;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
index 3acdeae25caf..3eb45cde0330 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
@@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
hdmi->config = config;
spin_lock_init(&hdmi->reg_lock);
- hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
+ hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
if (IS_ERR(hdmi->mmio)) {
ret = PTR_ERR(hdmi->mmio);
goto fail;
@@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
config->mmio_name);
hdmi->mmio_phy_addr = res->start;
- hdmi->qfprom_mmio = msm_ioremap(pdev,
- config->qfprom_mmio_name, "HDMI_QFPROM");
+ hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
if (IS_ERR(hdmi->qfprom_mmio)) {
DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
hdmi->qfprom_mmio = NULL;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
index 16b0e8836d27..9780107e1cc9 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
@@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
if (!phy->cfg)
return -ENODEV;
- phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
+ phy->mmio = msm_ioremap(pdev, "hdmi_phy");
if (IS_ERR(phy->mmio)) {
DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
return -ENOMEM;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
index 99c7853353fd..b06d9d25a189 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
@@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
pll->pdev = pdev;
- pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
+ pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
if (IS_ERR(pll->mmio_qserdes_com)) {
DRM_DEV_ERROR(dev, "failed to map pll base\n");
return -ENOMEM;
}
for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
- char name[32], label[32];
+ char name[32];
snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
- snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
- pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
+ pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
if (IS_ERR(pll->mmio_qserdes_tx[i])) {
DRM_DEV_ERROR(dev, "failed to map pll base\n");
return -ENOMEM;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
index 562dfac67792..be4b0b67e797 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
@@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
if (!pll)
return -ENOMEM;
- pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
+ pll->mmio = msm_ioremap(pdev, "hdmi_pll");
if (IS_ERR(pll->mmio)) {
DRM_DEV_ERROR(dev, "failed to map pll base\n");
return -ENOMEM;
diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index ad35a5d94053..d593c5d7dbc7 100644
--- a/drivers/gpu/drm/msm/msm_drv.c
+++ b/drivers/gpu/drm/msm/msm_drv.c
@@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
}
static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
- const char *dbgname, bool quiet, phys_addr_t *psize)
+ bool quiet, phys_addr_t *psize)
{
struct resource *res;
unsigned long size;
@@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
}
if (reglog)
- printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
+ printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
if (psize)
*psize = size;
@@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
return ptr;
}
-void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
- const char *dbgname)
+void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
{
- return _msm_ioremap(pdev, name, dbgname, false, NULL);
+ return _msm_ioremap(pdev, name, false, NULL);
}
-void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
- const char *dbgname)
+void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
{
- return _msm_ioremap(pdev, name, dbgname, true, NULL);
+ return _msm_ioremap(pdev, name, true, NULL);
}
void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
- const char *dbgname, phys_addr_t *psize)
+ phys_addr_t *psize)
{
- return _msm_ioremap(pdev, name, dbgname, false, psize);
+ return _msm_ioremap(pdev, name, false, psize);
}
void msm_writel(u32 data, void __iomem *addr)
diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
index d7574e6bd4e4..56d1242efcc1 100644
--- a/drivers/gpu/drm/msm/msm_drv.h
+++ b/drivers/gpu/drm/msm/msm_drv.h
@@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
const char *name);
-void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
- const char *dbgname);
+void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
- const char *dbgname, phys_addr_t *size);
-void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
- const char *dbgname);
+ phys_addr_t *size);
+void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
void msm_writel(u32 data, void __iomem *addr);
u32 msm_readl(const void __iomem *addr);
void msm_rmw(void __iomem *addr, u32 mask, u32 or);
diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
index 0f78c2615272..c4debc8ea3ba 100644
--- a/drivers/gpu/drm/msm/msm_gpu.c
+++ b/drivers/gpu/drm/msm/msm_gpu.c
@@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
/* Map registers: */
- gpu->mmio = msm_ioremap(pdev, config->ioname, name);
+ gpu->mmio = msm_ioremap(pdev, config->ioname);
if (IS_ERR(gpu->mmio)) {
ret = PTR_ERR(gpu->mmio);
goto fail;
--
2.33.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
2021-12-08 18:01 ` Dmitry Baryshkov
@ 2021-12-08 21:23 ` Rob Clark
-1 siblings, 0 replies; 8+ messages in thread
From: Rob Clark @ 2021-12-08 21:23 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Bjorn Andersson, Sean Paul, Abhinav Kumar, Stephen Boyd,
David Airlie, Daniel Vetter, linux-arm-msm, dri-devel, freedreno
On Wed, Dec 8, 2021 at 10:01 AM Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> msm_ioremap() functions take additional argument dbgname used to output
> single debug line telling IO range. Drop that extra argument, use
> resource name instead.
For some history, the dbgname traces were used by an older tool that
could decode the register read/write reglog traces based on the xml
register definitions.. so the dbgname matched up to domain names in
the xml. I had hacked up the downstream fbdev(ish) driver to generate
similar traces, so I could compare sequences of programming and values
(which was kind of easier that finding my way around the downstream
code).
That said, the tooling was never updated to deal with mdp5 and later,
with various different sub-blocks having their own base addresses.
And it looks like I dropped the demsm.c tool when importing the other
freedreno tools + xml into mesa. So probably no great loss to drop
this and the reglog stuff.
BR,
-R
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
> drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
> drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
> drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
> drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
> drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
> drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
> drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
> drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
> drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
> drivers/gpu/drm/msm/msm_gpu.c | 2 +-
> 15 files changed, 34 insertions(+), 40 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> index 51b83776951b..5750a3af126d 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> @@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
> if (a6xx_gpu->have_mmu500)
> a6xx_gpu->llc_mmio = NULL;
> else
> - a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
> + a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
>
> a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
> a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> index 2bd258eaf334..f86419cb2b4b 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> @@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>
> atomic_set(&dpu_kms->bandwidth_ref, 0);
>
> - dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
> + dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
> if (IS_ERR(dpu_kms->mmio)) {
> rc = PTR_ERR(dpu_kms->mmio);
> DPU_ERROR("mdp register memory map failed: %d\n", rc);
> @@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
> }
> DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
>
> - dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
> + dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
> if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
> rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
> DPU_ERROR("vbif register memory map failed: %d\n", rc);
> dpu_kms->vbif[VBIF_RT] = NULL;
> goto error;
> }
> - dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
> + dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
> if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
> dpu_kms->vbif[VBIF_NRT] = NULL;
> DPU_DEBUG("VBIF NRT is not defined");
> }
>
> - dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
> + dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
> if (IS_ERR(dpu_kms->reg_dma)) {
> dpu_kms->reg_dma = NULL;
> DPU_DEBUG("REG_DMA is not defined");
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> index 131c1f1a869c..5bdb1504035c 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> @@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
> if (!dpu_mdss)
> return -ENOMEM;
>
> - dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
> + dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
> if (IS_ERR(dpu_mdss->mmio))
> return PTR_ERR(dpu_mdss->mmio);
>
> diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> index 5a33bb148e9e..3cf476c55158 100644
> --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> @@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
>
> mdp4_kms->dev = dev;
>
> - mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
> + mdp4_kms->mmio = msm_ioremap(pdev, NULL);
> if (IS_ERR(mdp4_kms->mmio)) {
> ret = PTR_ERR(mdp4_kms->mmio);
> goto fail;
> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> index 12a5f81e402b..81bd434980cf 100644
> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> @@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
> if (ret)
> goto fail;
>
> - mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
> + mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
> if (IS_ERR(mdp5_kms->mmio)) {
> ret = PTR_ERR(mdp5_kms->mmio);
> goto fail;
> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> index b3f79c2277e9..049c6784a531 100644
> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> @@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
>
> mdp5_mdss->base.dev = &pdev->dev;
>
> - mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
> + mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
> if (IS_ERR(mdp5_mdss->mmio)) {
> ret = PTR_ERR(mdp5_mdss->mmio);
> goto fail;
> }
>
> - mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
> + mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
> if (IS_ERR(mdp5_mdss->vbif)) {
> ret = PTR_ERR(mdp5_mdss->vbif);
> goto fail;
> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
> index 5b4bb722f750..4e2dfc53ca28 100644
> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
> @@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
> goto fail;
> }
>
> - msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
> + msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
> if (IS_ERR(msm_host->ctrl_base)) {
> pr_err("%s: unable to map Dsi ctrl base\n", __func__);
> ret = PTR_ERR(msm_host->ctrl_base);
> diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> index 0b2ae5c15240..6425b6c34f4c 100644
> --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> @@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
> phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
>
> - phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
> + phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
> if (IS_ERR(phy->base)) {
> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> ret = -ENOMEM;
> goto fail;
> }
>
> - phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
> + phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
> if (IS_ERR(phy->pll_base)) {
> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
> ret = -ENOMEM;
> @@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> }
>
> if (phy->cfg->has_phy_lane) {
> - phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
> + phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
> if (IS_ERR(phy->lane_base)) {
> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
> ret = -ENOMEM;
> @@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> }
>
> if (phy->cfg->has_phy_regulator) {
> - phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
> + phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
> if (IS_ERR(phy->reg_base)) {
> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
> ret = -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
> index 3acdeae25caf..3eb45cde0330 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
> @@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> hdmi->config = config;
> spin_lock_init(&hdmi->reg_lock);
>
> - hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
> + hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
> if (IS_ERR(hdmi->mmio)) {
> ret = PTR_ERR(hdmi->mmio);
> goto fail;
> @@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> config->mmio_name);
> hdmi->mmio_phy_addr = res->start;
>
> - hdmi->qfprom_mmio = msm_ioremap(pdev,
> - config->qfprom_mmio_name, "HDMI_QFPROM");
> + hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
> if (IS_ERR(hdmi->qfprom_mmio)) {
> DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
> hdmi->qfprom_mmio = NULL;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> index 16b0e8836d27..9780107e1cc9 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> @@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
> if (!phy->cfg)
> return -ENODEV;
>
> - phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
> + phy->mmio = msm_ioremap(pdev, "hdmi_phy");
> if (IS_ERR(phy->mmio)) {
> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> return -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> index 99c7853353fd..b06d9d25a189 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> @@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
>
> pll->pdev = pdev;
>
> - pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> + pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
> if (IS_ERR(pll->mmio_qserdes_com)) {
> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> return -ENOMEM;
> }
>
> for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
> - char name[32], label[32];
> + char name[32];
>
> snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
> - snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
>
> - pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
> + pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
> if (IS_ERR(pll->mmio_qserdes_tx[i])) {
> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> return -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> index 562dfac67792..be4b0b67e797 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> @@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
> if (!pll)
> return -ENOMEM;
>
> - pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> + pll->mmio = msm_ioremap(pdev, "hdmi_pll");
> if (IS_ERR(pll->mmio)) {
> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> return -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
> index ad35a5d94053..d593c5d7dbc7 100644
> --- a/drivers/gpu/drm/msm/msm_drv.c
> +++ b/drivers/gpu/drm/msm/msm_drv.c
> @@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
> }
>
> static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
> - const char *dbgname, bool quiet, phys_addr_t *psize)
> + bool quiet, phys_addr_t *psize)
> {
> struct resource *res;
> unsigned long size;
> @@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> }
>
> if (reglog)
> - printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
> + printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
>
> if (psize)
> *psize = size;
> @@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> return ptr;
> }
>
> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> - const char *dbgname)
> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
> {
> - return _msm_ioremap(pdev, name, dbgname, false, NULL);
> + return _msm_ioremap(pdev, name, false, NULL);
> }
>
> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> - const char *dbgname)
> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
> {
> - return _msm_ioremap(pdev, name, dbgname, true, NULL);
> + return _msm_ioremap(pdev, name, true, NULL);
> }
>
> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> - const char *dbgname, phys_addr_t *psize)
> + phys_addr_t *psize)
> {
> - return _msm_ioremap(pdev, name, dbgname, false, psize);
> + return _msm_ioremap(pdev, name, false, psize);
> }
>
> void msm_writel(u32 data, void __iomem *addr)
> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
> index d7574e6bd4e4..56d1242efcc1 100644
> --- a/drivers/gpu/drm/msm/msm_drv.h
> +++ b/drivers/gpu/drm/msm/msm_drv.h
> @@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
>
> struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
> const char *name);
> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> - const char *dbgname);
> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> - const char *dbgname, phys_addr_t *size);
> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> - const char *dbgname);
> + phys_addr_t *size);
> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
> void msm_writel(u32 data, void __iomem *addr);
> u32 msm_readl(const void __iomem *addr);
> void msm_rmw(void __iomem *addr, u32 mask, u32 or);
> diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
> index 0f78c2615272..c4debc8ea3ba 100644
> --- a/drivers/gpu/drm/msm/msm_gpu.c
> +++ b/drivers/gpu/drm/msm/msm_gpu.c
> @@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
>
>
> /* Map registers: */
> - gpu->mmio = msm_ioremap(pdev, config->ioname, name);
> + gpu->mmio = msm_ioremap(pdev, config->ioname);
> if (IS_ERR(gpu->mmio)) {
> ret = PTR_ERR(gpu->mmio);
> goto fail;
> --
> 2.33.0
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
@ 2021-12-08 21:23 ` Rob Clark
0 siblings, 0 replies; 8+ messages in thread
From: Rob Clark @ 2021-12-08 21:23 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: freedreno, Stephen Boyd, linux-arm-msm, Abhinav Kumar,
Bjorn Andersson, David Airlie, dri-devel, Sean Paul
On Wed, Dec 8, 2021 at 10:01 AM Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> msm_ioremap() functions take additional argument dbgname used to output
> single debug line telling IO range. Drop that extra argument, use
> resource name instead.
For some history, the dbgname traces were used by an older tool that
could decode the register read/write reglog traces based on the xml
register definitions.. so the dbgname matched up to domain names in
the xml. I had hacked up the downstream fbdev(ish) driver to generate
similar traces, so I could compare sequences of programming and values
(which was kind of easier that finding my way around the downstream
code).
That said, the tooling was never updated to deal with mdp5 and later,
with various different sub-blocks having their own base addresses.
And it looks like I dropped the demsm.c tool when importing the other
freedreno tools + xml into mesa. So probably no great loss to drop
this and the reglog stuff.
BR,
-R
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
> drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
> drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
> drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
> drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
> drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
> drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
> drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
> drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
> drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
> drivers/gpu/drm/msm/msm_gpu.c | 2 +-
> 15 files changed, 34 insertions(+), 40 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> index 51b83776951b..5750a3af126d 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> @@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
> if (a6xx_gpu->have_mmu500)
> a6xx_gpu->llc_mmio = NULL;
> else
> - a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
> + a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
>
> a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
> a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> index 2bd258eaf334..f86419cb2b4b 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> @@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>
> atomic_set(&dpu_kms->bandwidth_ref, 0);
>
> - dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
> + dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
> if (IS_ERR(dpu_kms->mmio)) {
> rc = PTR_ERR(dpu_kms->mmio);
> DPU_ERROR("mdp register memory map failed: %d\n", rc);
> @@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
> }
> DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
>
> - dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
> + dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
> if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
> rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
> DPU_ERROR("vbif register memory map failed: %d\n", rc);
> dpu_kms->vbif[VBIF_RT] = NULL;
> goto error;
> }
> - dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
> + dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
> if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
> dpu_kms->vbif[VBIF_NRT] = NULL;
> DPU_DEBUG("VBIF NRT is not defined");
> }
>
> - dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
> + dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
> if (IS_ERR(dpu_kms->reg_dma)) {
> dpu_kms->reg_dma = NULL;
> DPU_DEBUG("REG_DMA is not defined");
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> index 131c1f1a869c..5bdb1504035c 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> @@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
> if (!dpu_mdss)
> return -ENOMEM;
>
> - dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
> + dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
> if (IS_ERR(dpu_mdss->mmio))
> return PTR_ERR(dpu_mdss->mmio);
>
> diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> index 5a33bb148e9e..3cf476c55158 100644
> --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> @@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
>
> mdp4_kms->dev = dev;
>
> - mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
> + mdp4_kms->mmio = msm_ioremap(pdev, NULL);
> if (IS_ERR(mdp4_kms->mmio)) {
> ret = PTR_ERR(mdp4_kms->mmio);
> goto fail;
> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> index 12a5f81e402b..81bd434980cf 100644
> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> @@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
> if (ret)
> goto fail;
>
> - mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
> + mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
> if (IS_ERR(mdp5_kms->mmio)) {
> ret = PTR_ERR(mdp5_kms->mmio);
> goto fail;
> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> index b3f79c2277e9..049c6784a531 100644
> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> @@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
>
> mdp5_mdss->base.dev = &pdev->dev;
>
> - mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
> + mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
> if (IS_ERR(mdp5_mdss->mmio)) {
> ret = PTR_ERR(mdp5_mdss->mmio);
> goto fail;
> }
>
> - mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
> + mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
> if (IS_ERR(mdp5_mdss->vbif)) {
> ret = PTR_ERR(mdp5_mdss->vbif);
> goto fail;
> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
> index 5b4bb722f750..4e2dfc53ca28 100644
> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
> @@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
> goto fail;
> }
>
> - msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
> + msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
> if (IS_ERR(msm_host->ctrl_base)) {
> pr_err("%s: unable to map Dsi ctrl base\n", __func__);
> ret = PTR_ERR(msm_host->ctrl_base);
> diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> index 0b2ae5c15240..6425b6c34f4c 100644
> --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> @@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
> phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
>
> - phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
> + phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
> if (IS_ERR(phy->base)) {
> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> ret = -ENOMEM;
> goto fail;
> }
>
> - phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
> + phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
> if (IS_ERR(phy->pll_base)) {
> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
> ret = -ENOMEM;
> @@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> }
>
> if (phy->cfg->has_phy_lane) {
> - phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
> + phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
> if (IS_ERR(phy->lane_base)) {
> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
> ret = -ENOMEM;
> @@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> }
>
> if (phy->cfg->has_phy_regulator) {
> - phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
> + phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
> if (IS_ERR(phy->reg_base)) {
> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
> ret = -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
> index 3acdeae25caf..3eb45cde0330 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
> @@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> hdmi->config = config;
> spin_lock_init(&hdmi->reg_lock);
>
> - hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
> + hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
> if (IS_ERR(hdmi->mmio)) {
> ret = PTR_ERR(hdmi->mmio);
> goto fail;
> @@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> config->mmio_name);
> hdmi->mmio_phy_addr = res->start;
>
> - hdmi->qfprom_mmio = msm_ioremap(pdev,
> - config->qfprom_mmio_name, "HDMI_QFPROM");
> + hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
> if (IS_ERR(hdmi->qfprom_mmio)) {
> DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
> hdmi->qfprom_mmio = NULL;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> index 16b0e8836d27..9780107e1cc9 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> @@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
> if (!phy->cfg)
> return -ENODEV;
>
> - phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
> + phy->mmio = msm_ioremap(pdev, "hdmi_phy");
> if (IS_ERR(phy->mmio)) {
> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> return -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> index 99c7853353fd..b06d9d25a189 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> @@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
>
> pll->pdev = pdev;
>
> - pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> + pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
> if (IS_ERR(pll->mmio_qserdes_com)) {
> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> return -ENOMEM;
> }
>
> for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
> - char name[32], label[32];
> + char name[32];
>
> snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
> - snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
>
> - pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
> + pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
> if (IS_ERR(pll->mmio_qserdes_tx[i])) {
> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> return -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> index 562dfac67792..be4b0b67e797 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> @@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
> if (!pll)
> return -ENOMEM;
>
> - pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> + pll->mmio = msm_ioremap(pdev, "hdmi_pll");
> if (IS_ERR(pll->mmio)) {
> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> return -ENOMEM;
> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
> index ad35a5d94053..d593c5d7dbc7 100644
> --- a/drivers/gpu/drm/msm/msm_drv.c
> +++ b/drivers/gpu/drm/msm/msm_drv.c
> @@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
> }
>
> static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
> - const char *dbgname, bool quiet, phys_addr_t *psize)
> + bool quiet, phys_addr_t *psize)
> {
> struct resource *res;
> unsigned long size;
> @@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> }
>
> if (reglog)
> - printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
> + printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
>
> if (psize)
> *psize = size;
> @@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> return ptr;
> }
>
> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> - const char *dbgname)
> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
> {
> - return _msm_ioremap(pdev, name, dbgname, false, NULL);
> + return _msm_ioremap(pdev, name, false, NULL);
> }
>
> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> - const char *dbgname)
> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
> {
> - return _msm_ioremap(pdev, name, dbgname, true, NULL);
> + return _msm_ioremap(pdev, name, true, NULL);
> }
>
> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> - const char *dbgname, phys_addr_t *psize)
> + phys_addr_t *psize)
> {
> - return _msm_ioremap(pdev, name, dbgname, false, psize);
> + return _msm_ioremap(pdev, name, false, psize);
> }
>
> void msm_writel(u32 data, void __iomem *addr)
> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
> index d7574e6bd4e4..56d1242efcc1 100644
> --- a/drivers/gpu/drm/msm/msm_drv.h
> +++ b/drivers/gpu/drm/msm/msm_drv.h
> @@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
>
> struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
> const char *name);
> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> - const char *dbgname);
> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> - const char *dbgname, phys_addr_t *size);
> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> - const char *dbgname);
> + phys_addr_t *size);
> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
> void msm_writel(u32 data, void __iomem *addr);
> u32 msm_readl(const void __iomem *addr);
> void msm_rmw(void __iomem *addr, u32 mask, u32 or);
> diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
> index 0f78c2615272..c4debc8ea3ba 100644
> --- a/drivers/gpu/drm/msm/msm_gpu.c
> +++ b/drivers/gpu/drm/msm/msm_gpu.c
> @@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
>
>
> /* Map registers: */
> - gpu->mmio = msm_ioremap(pdev, config->ioname, name);
> + gpu->mmio = msm_ioremap(pdev, config->ioname);
> if (IS_ERR(gpu->mmio)) {
> ret = PTR_ERR(gpu->mmio);
> goto fail;
> --
> 2.33.0
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
2021-12-08 21:23 ` Rob Clark
@ 2021-12-15 16:45 ` Dmitry Baryshkov
-1 siblings, 0 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2021-12-15 16:45 UTC (permalink / raw)
To: Rob Clark
Cc: Bjorn Andersson, Sean Paul, Abhinav Kumar, Stephen Boyd,
David Airlie, Daniel Vetter, linux-arm-msm, dri-devel, freedreno
Hi,
On 09/12/2021 00:23, Rob Clark wrote:
> On Wed, Dec 8, 2021 at 10:01 AM Dmitry Baryshkov
> <dmitry.baryshkov@linaro.org> wrote:
>>
>> msm_ioremap() functions take additional argument dbgname used to output
>> single debug line telling IO range. Drop that extra argument, use
>> resource name instead.
>
> For some history, the dbgname traces were used by an older tool that
> could decode the register read/write reglog traces based on the xml
> register definitions.. so the dbgname matched up to domain names in
> the xml. I had hacked up the downstream fbdev(ish) driver to generate
> similar traces, so I could compare sequences of programming and values
> (which was kind of easier that finding my way around the downstream
> code).
I suspected something like this, as the dbgname is only used for the reglog.
>
> That said, the tooling was never updated to deal with mdp5 and later,
> with various different sub-blocks having their own base addresses.
> And it looks like I dropped the demsm.c tool when importing the other
> freedreno tools + xml into mesa. So probably no great loss to drop
> this and the reglog stuff.
Is this an ack? Or would you prefer to drop reglog altogether? This
would allow us to move msm_readl/writel/rmw into msm_drv.h as static
inlines.
>
> BR,
> -R
>
>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
>> drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
>> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
>> drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
>> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
>> drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
>> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
>> drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
>> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
>> drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
>> drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
>> drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
>> drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
>> drivers/gpu/drm/msm/msm_gpu.c | 2 +-
>> 15 files changed, 34 insertions(+), 40 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
>> index 51b83776951b..5750a3af126d 100644
>> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
>> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
>> @@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
>> if (a6xx_gpu->have_mmu500)
>> a6xx_gpu->llc_mmio = NULL;
>> else
>> - a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
>> + a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
>>
>> a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
>> a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
>> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
>> index 2bd258eaf334..f86419cb2b4b 100644
>> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
>> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
>> @@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>>
>> atomic_set(&dpu_kms->bandwidth_ref, 0);
>>
>> - dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
>> + dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
>> if (IS_ERR(dpu_kms->mmio)) {
>> rc = PTR_ERR(dpu_kms->mmio);
>> DPU_ERROR("mdp register memory map failed: %d\n", rc);
>> @@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>> }
>> DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
>>
>> - dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
>> + dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
>> if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
>> rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
>> DPU_ERROR("vbif register memory map failed: %d\n", rc);
>> dpu_kms->vbif[VBIF_RT] = NULL;
>> goto error;
>> }
>> - dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
>> + dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
>> if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
>> dpu_kms->vbif[VBIF_NRT] = NULL;
>> DPU_DEBUG("VBIF NRT is not defined");
>> }
>>
>> - dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
>> + dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
>> if (IS_ERR(dpu_kms->reg_dma)) {
>> dpu_kms->reg_dma = NULL;
>> DPU_DEBUG("REG_DMA is not defined");
>> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
>> index 131c1f1a869c..5bdb1504035c 100644
>> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
>> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
>> @@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
>> if (!dpu_mdss)
>> return -ENOMEM;
>>
>> - dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
>> + dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
>> if (IS_ERR(dpu_mdss->mmio))
>> return PTR_ERR(dpu_mdss->mmio);
>>
>> diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
>> index 5a33bb148e9e..3cf476c55158 100644
>> --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
>> +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
>> @@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
>>
>> mdp4_kms->dev = dev;
>>
>> - mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
>> + mdp4_kms->mmio = msm_ioremap(pdev, NULL);
>> if (IS_ERR(mdp4_kms->mmio)) {
>> ret = PTR_ERR(mdp4_kms->mmio);
>> goto fail;
>> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
>> index 12a5f81e402b..81bd434980cf 100644
>> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
>> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
>> @@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
>> if (ret)
>> goto fail;
>>
>> - mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
>> + mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
>> if (IS_ERR(mdp5_kms->mmio)) {
>> ret = PTR_ERR(mdp5_kms->mmio);
>> goto fail;
>> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
>> index b3f79c2277e9..049c6784a531 100644
>> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
>> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
>> @@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
>>
>> mdp5_mdss->base.dev = &pdev->dev;
>>
>> - mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
>> + mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
>> if (IS_ERR(mdp5_mdss->mmio)) {
>> ret = PTR_ERR(mdp5_mdss->mmio);
>> goto fail;
>> }
>>
>> - mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
>> + mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
>> if (IS_ERR(mdp5_mdss->vbif)) {
>> ret = PTR_ERR(mdp5_mdss->vbif);
>> goto fail;
>> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
>> index 5b4bb722f750..4e2dfc53ca28 100644
>> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
>> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
>> @@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
>> goto fail;
>> }
>>
>> - msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
>> + msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
>> if (IS_ERR(msm_host->ctrl_base)) {
>> pr_err("%s: unable to map Dsi ctrl base\n", __func__);
>> ret = PTR_ERR(msm_host->ctrl_base);
>> diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
>> index 0b2ae5c15240..6425b6c34f4c 100644
>> --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
>> +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
>> @@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
>> if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
>> phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
>>
>> - phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
>> + phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
>> if (IS_ERR(phy->base)) {
>> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
>> ret = -ENOMEM;
>> goto fail;
>> }
>>
>> - phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
>> + phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
>> if (IS_ERR(phy->pll_base)) {
>> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
>> ret = -ENOMEM;
>> @@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
>> }
>>
>> if (phy->cfg->has_phy_lane) {
>> - phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
>> + phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
>> if (IS_ERR(phy->lane_base)) {
>> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
>> ret = -ENOMEM;
>> @@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
>> }
>>
>> if (phy->cfg->has_phy_regulator) {
>> - phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
>> + phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
>> if (IS_ERR(phy->reg_base)) {
>> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
>> ret = -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
>> index 3acdeae25caf..3eb45cde0330 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
>> @@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
>> hdmi->config = config;
>> spin_lock_init(&hdmi->reg_lock);
>>
>> - hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
>> + hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
>> if (IS_ERR(hdmi->mmio)) {
>> ret = PTR_ERR(hdmi->mmio);
>> goto fail;
>> @@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
>> config->mmio_name);
>> hdmi->mmio_phy_addr = res->start;
>>
>> - hdmi->qfprom_mmio = msm_ioremap(pdev,
>> - config->qfprom_mmio_name, "HDMI_QFPROM");
>> + hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
>> if (IS_ERR(hdmi->qfprom_mmio)) {
>> DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
>> hdmi->qfprom_mmio = NULL;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
>> index 16b0e8836d27..9780107e1cc9 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
>> @@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
>> if (!phy->cfg)
>> return -ENODEV;
>>
>> - phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
>> + phy->mmio = msm_ioremap(pdev, "hdmi_phy");
>> if (IS_ERR(phy->mmio)) {
>> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
>> return -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
>> index 99c7853353fd..b06d9d25a189 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
>> @@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
>>
>> pll->pdev = pdev;
>>
>> - pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
>> + pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
>> if (IS_ERR(pll->mmio_qserdes_com)) {
>> DRM_DEV_ERROR(dev, "failed to map pll base\n");
>> return -ENOMEM;
>> }
>>
>> for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
>> - char name[32], label[32];
>> + char name[32];
>>
>> snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
>> - snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
>>
>> - pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
>> + pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
>> if (IS_ERR(pll->mmio_qserdes_tx[i])) {
>> DRM_DEV_ERROR(dev, "failed to map pll base\n");
>> return -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
>> index 562dfac67792..be4b0b67e797 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
>> @@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
>> if (!pll)
>> return -ENOMEM;
>>
>> - pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
>> + pll->mmio = msm_ioremap(pdev, "hdmi_pll");
>> if (IS_ERR(pll->mmio)) {
>> DRM_DEV_ERROR(dev, "failed to map pll base\n");
>> return -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
>> index ad35a5d94053..d593c5d7dbc7 100644
>> --- a/drivers/gpu/drm/msm/msm_drv.c
>> +++ b/drivers/gpu/drm/msm/msm_drv.c
>> @@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
>> }
>>
>> static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
>> - const char *dbgname, bool quiet, phys_addr_t *psize)
>> + bool quiet, phys_addr_t *psize)
>> {
>> struct resource *res;
>> unsigned long size;
>> @@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
>> }
>>
>> if (reglog)
>> - printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
>> + printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
>>
>> if (psize)
>> *psize = size;
>> @@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
>> return ptr;
>> }
>>
>> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
>> - const char *dbgname)
>> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
>> {
>> - return _msm_ioremap(pdev, name, dbgname, false, NULL);
>> + return _msm_ioremap(pdev, name, false, NULL);
>> }
>>
>> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
>> - const char *dbgname)
>> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
>> {
>> - return _msm_ioremap(pdev, name, dbgname, true, NULL);
>> + return _msm_ioremap(pdev, name, true, NULL);
>> }
>>
>> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
>> - const char *dbgname, phys_addr_t *psize)
>> + phys_addr_t *psize)
>> {
>> - return _msm_ioremap(pdev, name, dbgname, false, psize);
>> + return _msm_ioremap(pdev, name, false, psize);
>> }
>>
>> void msm_writel(u32 data, void __iomem *addr)
>> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
>> index d7574e6bd4e4..56d1242efcc1 100644
>> --- a/drivers/gpu/drm/msm/msm_drv.h
>> +++ b/drivers/gpu/drm/msm/msm_drv.h
>> @@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
>>
>> struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
>> const char *name);
>> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
>> - const char *dbgname);
>> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
>> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
>> - const char *dbgname, phys_addr_t *size);
>> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
>> - const char *dbgname);
>> + phys_addr_t *size);
>> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
>> void msm_writel(u32 data, void __iomem *addr);
>> u32 msm_readl(const void __iomem *addr);
>> void msm_rmw(void __iomem *addr, u32 mask, u32 or);
>> diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
>> index 0f78c2615272..c4debc8ea3ba 100644
>> --- a/drivers/gpu/drm/msm/msm_gpu.c
>> +++ b/drivers/gpu/drm/msm/msm_gpu.c
>> @@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
>>
>>
>> /* Map registers: */
>> - gpu->mmio = msm_ioremap(pdev, config->ioname, name);
>> + gpu->mmio = msm_ioremap(pdev, config->ioname);
>> if (IS_ERR(gpu->mmio)) {
>> ret = PTR_ERR(gpu->mmio);
>> goto fail;
>> --
>> 2.33.0
>>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
@ 2021-12-15 16:45 ` Dmitry Baryshkov
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2021-12-15 16:45 UTC (permalink / raw)
To: Rob Clark
Cc: freedreno, Stephen Boyd, linux-arm-msm, Abhinav Kumar,
Bjorn Andersson, David Airlie, dri-devel, Sean Paul
Hi,
On 09/12/2021 00:23, Rob Clark wrote:
> On Wed, Dec 8, 2021 at 10:01 AM Dmitry Baryshkov
> <dmitry.baryshkov@linaro.org> wrote:
>>
>> msm_ioremap() functions take additional argument dbgname used to output
>> single debug line telling IO range. Drop that extra argument, use
>> resource name instead.
>
> For some history, the dbgname traces were used by an older tool that
> could decode the register read/write reglog traces based on the xml
> register definitions.. so the dbgname matched up to domain names in
> the xml. I had hacked up the downstream fbdev(ish) driver to generate
> similar traces, so I could compare sequences of programming and values
> (which was kind of easier that finding my way around the downstream
> code).
I suspected something like this, as the dbgname is only used for the reglog.
>
> That said, the tooling was never updated to deal with mdp5 and later,
> with various different sub-blocks having their own base addresses.
> And it looks like I dropped the demsm.c tool when importing the other
> freedreno tools + xml into mesa. So probably no great loss to drop
> this and the reglog stuff.
Is this an ack? Or would you prefer to drop reglog altogether? This
would allow us to move msm_readl/writel/rmw into msm_drv.h as static
inlines.
>
> BR,
> -R
>
>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
>> drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
>> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
>> drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
>> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
>> drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
>> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
>> drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
>> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
>> drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
>> drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
>> drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
>> drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
>> drivers/gpu/drm/msm/msm_gpu.c | 2 +-
>> 15 files changed, 34 insertions(+), 40 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
>> index 51b83776951b..5750a3af126d 100644
>> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
>> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
>> @@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
>> if (a6xx_gpu->have_mmu500)
>> a6xx_gpu->llc_mmio = NULL;
>> else
>> - a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
>> + a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
>>
>> a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
>> a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
>> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
>> index 2bd258eaf334..f86419cb2b4b 100644
>> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
>> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
>> @@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>>
>> atomic_set(&dpu_kms->bandwidth_ref, 0);
>>
>> - dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
>> + dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
>> if (IS_ERR(dpu_kms->mmio)) {
>> rc = PTR_ERR(dpu_kms->mmio);
>> DPU_ERROR("mdp register memory map failed: %d\n", rc);
>> @@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>> }
>> DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
>>
>> - dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
>> + dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
>> if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
>> rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
>> DPU_ERROR("vbif register memory map failed: %d\n", rc);
>> dpu_kms->vbif[VBIF_RT] = NULL;
>> goto error;
>> }
>> - dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
>> + dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
>> if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
>> dpu_kms->vbif[VBIF_NRT] = NULL;
>> DPU_DEBUG("VBIF NRT is not defined");
>> }
>>
>> - dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
>> + dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
>> if (IS_ERR(dpu_kms->reg_dma)) {
>> dpu_kms->reg_dma = NULL;
>> DPU_DEBUG("REG_DMA is not defined");
>> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
>> index 131c1f1a869c..5bdb1504035c 100644
>> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
>> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
>> @@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
>> if (!dpu_mdss)
>> return -ENOMEM;
>>
>> - dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
>> + dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
>> if (IS_ERR(dpu_mdss->mmio))
>> return PTR_ERR(dpu_mdss->mmio);
>>
>> diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
>> index 5a33bb148e9e..3cf476c55158 100644
>> --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
>> +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
>> @@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
>>
>> mdp4_kms->dev = dev;
>>
>> - mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
>> + mdp4_kms->mmio = msm_ioremap(pdev, NULL);
>> if (IS_ERR(mdp4_kms->mmio)) {
>> ret = PTR_ERR(mdp4_kms->mmio);
>> goto fail;
>> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
>> index 12a5f81e402b..81bd434980cf 100644
>> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
>> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
>> @@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
>> if (ret)
>> goto fail;
>>
>> - mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
>> + mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
>> if (IS_ERR(mdp5_kms->mmio)) {
>> ret = PTR_ERR(mdp5_kms->mmio);
>> goto fail;
>> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
>> index b3f79c2277e9..049c6784a531 100644
>> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
>> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
>> @@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
>>
>> mdp5_mdss->base.dev = &pdev->dev;
>>
>> - mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
>> + mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
>> if (IS_ERR(mdp5_mdss->mmio)) {
>> ret = PTR_ERR(mdp5_mdss->mmio);
>> goto fail;
>> }
>>
>> - mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
>> + mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
>> if (IS_ERR(mdp5_mdss->vbif)) {
>> ret = PTR_ERR(mdp5_mdss->vbif);
>> goto fail;
>> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
>> index 5b4bb722f750..4e2dfc53ca28 100644
>> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
>> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
>> @@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
>> goto fail;
>> }
>>
>> - msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
>> + msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
>> if (IS_ERR(msm_host->ctrl_base)) {
>> pr_err("%s: unable to map Dsi ctrl base\n", __func__);
>> ret = PTR_ERR(msm_host->ctrl_base);
>> diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
>> index 0b2ae5c15240..6425b6c34f4c 100644
>> --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
>> +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
>> @@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
>> if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
>> phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
>>
>> - phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
>> + phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
>> if (IS_ERR(phy->base)) {
>> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
>> ret = -ENOMEM;
>> goto fail;
>> }
>>
>> - phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
>> + phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
>> if (IS_ERR(phy->pll_base)) {
>> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
>> ret = -ENOMEM;
>> @@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
>> }
>>
>> if (phy->cfg->has_phy_lane) {
>> - phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
>> + phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
>> if (IS_ERR(phy->lane_base)) {
>> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
>> ret = -ENOMEM;
>> @@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
>> }
>>
>> if (phy->cfg->has_phy_regulator) {
>> - phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
>> + phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
>> if (IS_ERR(phy->reg_base)) {
>> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
>> ret = -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
>> index 3acdeae25caf..3eb45cde0330 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
>> @@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
>> hdmi->config = config;
>> spin_lock_init(&hdmi->reg_lock);
>>
>> - hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
>> + hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
>> if (IS_ERR(hdmi->mmio)) {
>> ret = PTR_ERR(hdmi->mmio);
>> goto fail;
>> @@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
>> config->mmio_name);
>> hdmi->mmio_phy_addr = res->start;
>>
>> - hdmi->qfprom_mmio = msm_ioremap(pdev,
>> - config->qfprom_mmio_name, "HDMI_QFPROM");
>> + hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
>> if (IS_ERR(hdmi->qfprom_mmio)) {
>> DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
>> hdmi->qfprom_mmio = NULL;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
>> index 16b0e8836d27..9780107e1cc9 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
>> @@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
>> if (!phy->cfg)
>> return -ENODEV;
>>
>> - phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
>> + phy->mmio = msm_ioremap(pdev, "hdmi_phy");
>> if (IS_ERR(phy->mmio)) {
>> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
>> return -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
>> index 99c7853353fd..b06d9d25a189 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
>> @@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
>>
>> pll->pdev = pdev;
>>
>> - pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
>> + pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
>> if (IS_ERR(pll->mmio_qserdes_com)) {
>> DRM_DEV_ERROR(dev, "failed to map pll base\n");
>> return -ENOMEM;
>> }
>>
>> for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
>> - char name[32], label[32];
>> + char name[32];
>>
>> snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
>> - snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
>>
>> - pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
>> + pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
>> if (IS_ERR(pll->mmio_qserdes_tx[i])) {
>> DRM_DEV_ERROR(dev, "failed to map pll base\n");
>> return -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
>> index 562dfac67792..be4b0b67e797 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
>> @@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
>> if (!pll)
>> return -ENOMEM;
>>
>> - pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
>> + pll->mmio = msm_ioremap(pdev, "hdmi_pll");
>> if (IS_ERR(pll->mmio)) {
>> DRM_DEV_ERROR(dev, "failed to map pll base\n");
>> return -ENOMEM;
>> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
>> index ad35a5d94053..d593c5d7dbc7 100644
>> --- a/drivers/gpu/drm/msm/msm_drv.c
>> +++ b/drivers/gpu/drm/msm/msm_drv.c
>> @@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
>> }
>>
>> static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
>> - const char *dbgname, bool quiet, phys_addr_t *psize)
>> + bool quiet, phys_addr_t *psize)
>> {
>> struct resource *res;
>> unsigned long size;
>> @@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
>> }
>>
>> if (reglog)
>> - printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
>> + printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
>>
>> if (psize)
>> *psize = size;
>> @@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
>> return ptr;
>> }
>>
>> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
>> - const char *dbgname)
>> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
>> {
>> - return _msm_ioremap(pdev, name, dbgname, false, NULL);
>> + return _msm_ioremap(pdev, name, false, NULL);
>> }
>>
>> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
>> - const char *dbgname)
>> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
>> {
>> - return _msm_ioremap(pdev, name, dbgname, true, NULL);
>> + return _msm_ioremap(pdev, name, true, NULL);
>> }
>>
>> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
>> - const char *dbgname, phys_addr_t *psize)
>> + phys_addr_t *psize)
>> {
>> - return _msm_ioremap(pdev, name, dbgname, false, psize);
>> + return _msm_ioremap(pdev, name, false, psize);
>> }
>>
>> void msm_writel(u32 data, void __iomem *addr)
>> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
>> index d7574e6bd4e4..56d1242efcc1 100644
>> --- a/drivers/gpu/drm/msm/msm_drv.h
>> +++ b/drivers/gpu/drm/msm/msm_drv.h
>> @@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
>>
>> struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
>> const char *name);
>> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
>> - const char *dbgname);
>> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
>> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
>> - const char *dbgname, phys_addr_t *size);
>> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
>> - const char *dbgname);
>> + phys_addr_t *size);
>> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
>> void msm_writel(u32 data, void __iomem *addr);
>> u32 msm_readl(const void __iomem *addr);
>> void msm_rmw(void __iomem *addr, u32 mask, u32 or);
>> diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
>> index 0f78c2615272..c4debc8ea3ba 100644
>> --- a/drivers/gpu/drm/msm/msm_gpu.c
>> +++ b/drivers/gpu/drm/msm/msm_gpu.c
>> @@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
>>
>>
>> /* Map registers: */
>> - gpu->mmio = msm_ioremap(pdev, config->ioname, name);
>> + gpu->mmio = msm_ioremap(pdev, config->ioname);
>> if (IS_ERR(gpu->mmio)) {
>> ret = PTR_ERR(gpu->mmio);
>> goto fail;
>> --
>> 2.33.0
>>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
2021-12-15 16:45 ` Dmitry Baryshkov
@ 2021-12-15 20:16 ` Rob Clark
-1 siblings, 0 replies; 8+ messages in thread
From: Rob Clark @ 2021-12-15 20:16 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Bjorn Andersson, Sean Paul, Abhinav Kumar, Stephen Boyd,
David Airlie, Daniel Vetter, linux-arm-msm, dri-devel, freedreno
On Wed, Dec 15, 2021 at 8:45 AM Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> Hi,
>
> On 09/12/2021 00:23, Rob Clark wrote:
> > On Wed, Dec 8, 2021 at 10:01 AM Dmitry Baryshkov
> > <dmitry.baryshkov@linaro.org> wrote:
> >>
> >> msm_ioremap() functions take additional argument dbgname used to output
> >> single debug line telling IO range. Drop that extra argument, use
> >> resource name instead.
> >
> > For some history, the dbgname traces were used by an older tool that
> > could decode the register read/write reglog traces based on the xml
> > register definitions.. so the dbgname matched up to domain names in
> > the xml. I had hacked up the downstream fbdev(ish) driver to generate
> > similar traces, so I could compare sequences of programming and values
> > (which was kind of easier that finding my way around the downstream
> > code).
>
> I suspected something like this, as the dbgname is only used for the reglog.
>
> >
> > That said, the tooling was never updated to deal with mdp5 and later,
> > with various different sub-blocks having their own base addresses.
> > And it looks like I dropped the demsm.c tool when importing the other
> > freedreno tools + xml into mesa. So probably no great loss to drop
> > this and the reglog stuff.
>
> Is this an ack? Or would you prefer to drop reglog altogether? This
> would allow us to move msm_readl/writel/rmw into msm_drv.h as static
> inlines.
yeah, ack.. we should probably drop reglog altogether, but that can be
it's own patch
BR,
-R
> >
> > BR,
> > -R
> >
> >> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> >> ---
> >> drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
> >> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
> >> drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
> >> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
> >> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
> >> drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
> >> drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
> >> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
> >> drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
> >> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
> >> drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
> >> drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
> >> drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
> >> drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
> >> drivers/gpu/drm/msm/msm_gpu.c | 2 +-
> >> 15 files changed, 34 insertions(+), 40 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> >> index 51b83776951b..5750a3af126d 100644
> >> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> >> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> >> @@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
> >> if (a6xx_gpu->have_mmu500)
> >> a6xx_gpu->llc_mmio = NULL;
> >> else
> >> - a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
> >> + a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
> >>
> >> a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
> >> a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
> >> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> >> index 2bd258eaf334..f86419cb2b4b 100644
> >> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> >> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> >> @@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
> >>
> >> atomic_set(&dpu_kms->bandwidth_ref, 0);
> >>
> >> - dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
> >> + dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
> >> if (IS_ERR(dpu_kms->mmio)) {
> >> rc = PTR_ERR(dpu_kms->mmio);
> >> DPU_ERROR("mdp register memory map failed: %d\n", rc);
> >> @@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
> >> }
> >> DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
> >>
> >> - dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
> >> + dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
> >> if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
> >> rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
> >> DPU_ERROR("vbif register memory map failed: %d\n", rc);
> >> dpu_kms->vbif[VBIF_RT] = NULL;
> >> goto error;
> >> }
> >> - dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
> >> + dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
> >> if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
> >> dpu_kms->vbif[VBIF_NRT] = NULL;
> >> DPU_DEBUG("VBIF NRT is not defined");
> >> }
> >>
> >> - dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
> >> + dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
> >> if (IS_ERR(dpu_kms->reg_dma)) {
> >> dpu_kms->reg_dma = NULL;
> >> DPU_DEBUG("REG_DMA is not defined");
> >> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> >> index 131c1f1a869c..5bdb1504035c 100644
> >> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> >> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> >> @@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
> >> if (!dpu_mdss)
> >> return -ENOMEM;
> >>
> >> - dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
> >> + dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
> >> if (IS_ERR(dpu_mdss->mmio))
> >> return PTR_ERR(dpu_mdss->mmio);
> >>
> >> diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> >> index 5a33bb148e9e..3cf476c55158 100644
> >> --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> >> +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> >> @@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
> >>
> >> mdp4_kms->dev = dev;
> >>
> >> - mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
> >> + mdp4_kms->mmio = msm_ioremap(pdev, NULL);
> >> if (IS_ERR(mdp4_kms->mmio)) {
> >> ret = PTR_ERR(mdp4_kms->mmio);
> >> goto fail;
> >> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> >> index 12a5f81e402b..81bd434980cf 100644
> >> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> >> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> >> @@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
> >> if (ret)
> >> goto fail;
> >>
> >> - mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
> >> + mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
> >> if (IS_ERR(mdp5_kms->mmio)) {
> >> ret = PTR_ERR(mdp5_kms->mmio);
> >> goto fail;
> >> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> >> index b3f79c2277e9..049c6784a531 100644
> >> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> >> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> >> @@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
> >>
> >> mdp5_mdss->base.dev = &pdev->dev;
> >>
> >> - mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
> >> + mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
> >> if (IS_ERR(mdp5_mdss->mmio)) {
> >> ret = PTR_ERR(mdp5_mdss->mmio);
> >> goto fail;
> >> }
> >>
> >> - mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
> >> + mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
> >> if (IS_ERR(mdp5_mdss->vbif)) {
> >> ret = PTR_ERR(mdp5_mdss->vbif);
> >> goto fail;
> >> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
> >> index 5b4bb722f750..4e2dfc53ca28 100644
> >> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
> >> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
> >> @@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
> >> goto fail;
> >> }
> >>
> >> - msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
> >> + msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
> >> if (IS_ERR(msm_host->ctrl_base)) {
> >> pr_err("%s: unable to map Dsi ctrl base\n", __func__);
> >> ret = PTR_ERR(msm_host->ctrl_base);
> >> diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> >> index 0b2ae5c15240..6425b6c34f4c 100644
> >> --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> >> +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> >> @@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> >> if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
> >> phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
> >>
> >> - phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
> >> + phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
> >> if (IS_ERR(phy->base)) {
> >> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> >> ret = -ENOMEM;
> >> goto fail;
> >> }
> >>
> >> - phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
> >> + phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
> >> if (IS_ERR(phy->pll_base)) {
> >> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
> >> ret = -ENOMEM;
> >> @@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> >> }
> >>
> >> if (phy->cfg->has_phy_lane) {
> >> - phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
> >> + phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
> >> if (IS_ERR(phy->lane_base)) {
> >> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
> >> ret = -ENOMEM;
> >> @@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> >> }
> >>
> >> if (phy->cfg->has_phy_regulator) {
> >> - phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
> >> + phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
> >> if (IS_ERR(phy->reg_base)) {
> >> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
> >> ret = -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
> >> index 3acdeae25caf..3eb45cde0330 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
> >> @@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> >> hdmi->config = config;
> >> spin_lock_init(&hdmi->reg_lock);
> >>
> >> - hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
> >> + hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
> >> if (IS_ERR(hdmi->mmio)) {
> >> ret = PTR_ERR(hdmi->mmio);
> >> goto fail;
> >> @@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> >> config->mmio_name);
> >> hdmi->mmio_phy_addr = res->start;
> >>
> >> - hdmi->qfprom_mmio = msm_ioremap(pdev,
> >> - config->qfprom_mmio_name, "HDMI_QFPROM");
> >> + hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
> >> if (IS_ERR(hdmi->qfprom_mmio)) {
> >> DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
> >> hdmi->qfprom_mmio = NULL;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> >> index 16b0e8836d27..9780107e1cc9 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> >> @@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
> >> if (!phy->cfg)
> >> return -ENODEV;
> >>
> >> - phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
> >> + phy->mmio = msm_ioremap(pdev, "hdmi_phy");
> >> if (IS_ERR(phy->mmio)) {
> >> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> >> return -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> >> index 99c7853353fd..b06d9d25a189 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> >> @@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
> >>
> >> pll->pdev = pdev;
> >>
> >> - pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> >> + pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
> >> if (IS_ERR(pll->mmio_qserdes_com)) {
> >> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> >> return -ENOMEM;
> >> }
> >>
> >> for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
> >> - char name[32], label[32];
> >> + char name[32];
> >>
> >> snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
> >> - snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
> >>
> >> - pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
> >> + pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
> >> if (IS_ERR(pll->mmio_qserdes_tx[i])) {
> >> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> >> return -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> >> index 562dfac67792..be4b0b67e797 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> >> @@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
> >> if (!pll)
> >> return -ENOMEM;
> >>
> >> - pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> >> + pll->mmio = msm_ioremap(pdev, "hdmi_pll");
> >> if (IS_ERR(pll->mmio)) {
> >> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> >> return -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
> >> index ad35a5d94053..d593c5d7dbc7 100644
> >> --- a/drivers/gpu/drm/msm/msm_drv.c
> >> +++ b/drivers/gpu/drm/msm/msm_drv.c
> >> @@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
> >> }
> >>
> >> static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
> >> - const char *dbgname, bool quiet, phys_addr_t *psize)
> >> + bool quiet, phys_addr_t *psize)
> >> {
> >> struct resource *res;
> >> unsigned long size;
> >> @@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> >> }
> >>
> >> if (reglog)
> >> - printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
> >> + printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
> >>
> >> if (psize)
> >> *psize = size;
> >> @@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> >> return ptr;
> >> }
> >>
> >> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> >> - const char *dbgname)
> >> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
> >> {
> >> - return _msm_ioremap(pdev, name, dbgname, false, NULL);
> >> + return _msm_ioremap(pdev, name, false, NULL);
> >> }
> >>
> >> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> >> - const char *dbgname)
> >> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
> >> {
> >> - return _msm_ioremap(pdev, name, dbgname, true, NULL);
> >> + return _msm_ioremap(pdev, name, true, NULL);
> >> }
> >>
> >> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> >> - const char *dbgname, phys_addr_t *psize)
> >> + phys_addr_t *psize)
> >> {
> >> - return _msm_ioremap(pdev, name, dbgname, false, psize);
> >> + return _msm_ioremap(pdev, name, false, psize);
> >> }
> >>
> >> void msm_writel(u32 data, void __iomem *addr)
> >> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
> >> index d7574e6bd4e4..56d1242efcc1 100644
> >> --- a/drivers/gpu/drm/msm/msm_drv.h
> >> +++ b/drivers/gpu/drm/msm/msm_drv.h
> >> @@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
> >>
> >> struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
> >> const char *name);
> >> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> >> - const char *dbgname);
> >> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
> >> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> >> - const char *dbgname, phys_addr_t *size);
> >> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> >> - const char *dbgname);
> >> + phys_addr_t *size);
> >> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
> >> void msm_writel(u32 data, void __iomem *addr);
> >> u32 msm_readl(const void __iomem *addr);
> >> void msm_rmw(void __iomem *addr, u32 mask, u32 or);
> >> diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
> >> index 0f78c2615272..c4debc8ea3ba 100644
> >> --- a/drivers/gpu/drm/msm/msm_gpu.c
> >> +++ b/drivers/gpu/drm/msm/msm_gpu.c
> >> @@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
> >>
> >>
> >> /* Map registers: */
> >> - gpu->mmio = msm_ioremap(pdev, config->ioname, name);
> >> + gpu->mmio = msm_ioremap(pdev, config->ioname);
> >> if (IS_ERR(gpu->mmio)) {
> >> ret = PTR_ERR(gpu->mmio);
> >> goto fail;
> >> --
> >> 2.33.0
> >>
>
>
> --
> With best wishes
> Dmitry
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/msm: drop dbgname argument from msm_ioremap*()
@ 2021-12-15 20:16 ` Rob Clark
0 siblings, 0 replies; 8+ messages in thread
From: Rob Clark @ 2021-12-15 20:16 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: freedreno, Stephen Boyd, linux-arm-msm, Abhinav Kumar,
Bjorn Andersson, David Airlie, dri-devel, Sean Paul
On Wed, Dec 15, 2021 at 8:45 AM Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> Hi,
>
> On 09/12/2021 00:23, Rob Clark wrote:
> > On Wed, Dec 8, 2021 at 10:01 AM Dmitry Baryshkov
> > <dmitry.baryshkov@linaro.org> wrote:
> >>
> >> msm_ioremap() functions take additional argument dbgname used to output
> >> single debug line telling IO range. Drop that extra argument, use
> >> resource name instead.
> >
> > For some history, the dbgname traces were used by an older tool that
> > could decode the register read/write reglog traces based on the xml
> > register definitions.. so the dbgname matched up to domain names in
> > the xml. I had hacked up the downstream fbdev(ish) driver to generate
> > similar traces, so I could compare sequences of programming and values
> > (which was kind of easier that finding my way around the downstream
> > code).
>
> I suspected something like this, as the dbgname is only used for the reglog.
>
> >
> > That said, the tooling was never updated to deal with mdp5 and later,
> > with various different sub-blocks having their own base addresses.
> > And it looks like I dropped the demsm.c tool when importing the other
> > freedreno tools + xml into mesa. So probably no great loss to drop
> > this and the reglog stuff.
>
> Is this an ack? Or would you prefer to drop reglog altogether? This
> would allow us to move msm_readl/writel/rmw into msm_drv.h as static
> inlines.
yeah, ack.. we should probably drop reglog altogether, but that can be
it's own patch
BR,
-R
> >
> > BR,
> > -R
> >
> >> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> >> ---
> >> drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +-
> >> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 ++++----
> >> drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 2 +-
> >> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 2 +-
> >> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
> >> drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c | 4 ++--
> >> drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
> >> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 8 ++++----
> >> drivers/gpu/drm/msm/hdmi/hdmi.c | 5 ++---
> >> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 2 +-
> >> drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c | 7 +++----
> >> drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c | 2 +-
> >> drivers/gpu/drm/msm/msm_drv.c | 18 ++++++++----------
> >> drivers/gpu/drm/msm/msm_drv.h | 8 +++-----
> >> drivers/gpu/drm/msm/msm_gpu.c | 2 +-
> >> 15 files changed, 34 insertions(+), 40 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> >> index 51b83776951b..5750a3af126d 100644
> >> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> >> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> >> @@ -1506,7 +1506,7 @@ static void a6xx_llc_slices_init(struct platform_device *pdev,
> >> if (a6xx_gpu->have_mmu500)
> >> a6xx_gpu->llc_mmio = NULL;
> >> else
> >> - a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem", "gpu_cx");
> >> + a6xx_gpu->llc_mmio = msm_ioremap(pdev, "cx_mem");
> >>
> >> a6xx_gpu->llc_slice = llcc_slice_getd(LLCC_GPU);
> >> a6xx_gpu->htw_llc_slice = llcc_slice_getd(LLCC_GPUHTW);
> >> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> >> index 2bd258eaf334..f86419cb2b4b 100644
> >> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> >> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> >> @@ -959,7 +959,7 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
> >>
> >> atomic_set(&dpu_kms->bandwidth_ref, 0);
> >>
> >> - dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp", "mdp");
> >> + dpu_kms->mmio = msm_ioremap(dpu_kms->pdev, "mdp");
> >> if (IS_ERR(dpu_kms->mmio)) {
> >> rc = PTR_ERR(dpu_kms->mmio);
> >> DPU_ERROR("mdp register memory map failed: %d\n", rc);
> >> @@ -968,20 +968,20 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
> >> }
> >> DRM_DEBUG("mapped dpu address space @%pK\n", dpu_kms->mmio);
> >>
> >> - dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif", "vbif");
> >> + dpu_kms->vbif[VBIF_RT] = msm_ioremap(dpu_kms->pdev, "vbif");
> >> if (IS_ERR(dpu_kms->vbif[VBIF_RT])) {
> >> rc = PTR_ERR(dpu_kms->vbif[VBIF_RT]);
> >> DPU_ERROR("vbif register memory map failed: %d\n", rc);
> >> dpu_kms->vbif[VBIF_RT] = NULL;
> >> goto error;
> >> }
> >> - dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt", "vbif_nrt");
> >> + dpu_kms->vbif[VBIF_NRT] = msm_ioremap_quiet(dpu_kms->pdev, "vbif_nrt");
> >> if (IS_ERR(dpu_kms->vbif[VBIF_NRT])) {
> >> dpu_kms->vbif[VBIF_NRT] = NULL;
> >> DPU_DEBUG("VBIF NRT is not defined");
> >> }
> >>
> >> - dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma", "regdma");
> >> + dpu_kms->reg_dma = msm_ioremap_quiet(dpu_kms->pdev, "regdma");
> >> if (IS_ERR(dpu_kms->reg_dma)) {
> >> dpu_kms->reg_dma = NULL;
> >> DPU_DEBUG("REG_DMA is not defined");
> >> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> >> index 131c1f1a869c..5bdb1504035c 100644
> >> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> >> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c
> >> @@ -222,7 +222,7 @@ int dpu_mdss_init(struct platform_device *pdev)
> >> if (!dpu_mdss)
> >> return -ENOMEM;
> >>
> >> - dpu_mdss->mmio = msm_ioremap(pdev, "mdss", "mdss");
> >> + dpu_mdss->mmio = msm_ioremap(pdev, "mdss");
> >> if (IS_ERR(dpu_mdss->mmio))
> >> return PTR_ERR(dpu_mdss->mmio);
> >>
> >> diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> >> index 5a33bb148e9e..3cf476c55158 100644
> >> --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> >> +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c
> >> @@ -418,7 +418,7 @@ struct msm_kms *mdp4_kms_init(struct drm_device *dev)
> >>
> >> mdp4_kms->dev = dev;
> >>
> >> - mdp4_kms->mmio = msm_ioremap(pdev, NULL, "MDP4");
> >> + mdp4_kms->mmio = msm_ioremap(pdev, NULL);
> >> if (IS_ERR(mdp4_kms->mmio)) {
> >> ret = PTR_ERR(mdp4_kms->mmio);
> >> goto fail;
> >> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> >> index 12a5f81e402b..81bd434980cf 100644
> >> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> >> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> >> @@ -827,7 +827,7 @@ static int mdp5_init(struct platform_device *pdev, struct drm_device *dev)
> >> if (ret)
> >> goto fail;
> >>
> >> - mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys", "MDP5");
> >> + mdp5_kms->mmio = msm_ioremap(pdev, "mdp_phys");
> >> if (IS_ERR(mdp5_kms->mmio)) {
> >> ret = PTR_ERR(mdp5_kms->mmio);
> >> goto fail;
> >> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> >> index b3f79c2277e9..049c6784a531 100644
> >> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> >> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_mdss.c
> >> @@ -210,13 +210,13 @@ int mdp5_mdss_init(struct platform_device *pdev)
> >>
> >> mdp5_mdss->base.dev = &pdev->dev;
> >>
> >> - mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys", "MDSS");
> >> + mdp5_mdss->mmio = msm_ioremap(pdev, "mdss_phys");
> >> if (IS_ERR(mdp5_mdss->mmio)) {
> >> ret = PTR_ERR(mdp5_mdss->mmio);
> >> goto fail;
> >> }
> >>
> >> - mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys", "VBIF");
> >> + mdp5_mdss->vbif = msm_ioremap(pdev, "vbif_phys");
> >> if (IS_ERR(mdp5_mdss->vbif)) {
> >> ret = PTR_ERR(mdp5_mdss->vbif);
> >> goto fail;
> >> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
> >> index 5b4bb722f750..4e2dfc53ca28 100644
> >> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
> >> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
> >> @@ -1813,7 +1813,7 @@ int msm_dsi_host_init(struct msm_dsi *msm_dsi)
> >> goto fail;
> >> }
> >>
> >> - msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", "DSI CTRL", &msm_host->ctrl_size);
> >> + msm_host->ctrl_base = msm_ioremap_size(pdev, "dsi_ctrl", &msm_host->ctrl_size);
> >> if (IS_ERR(msm_host->ctrl_base)) {
> >> pr_err("%s: unable to map Dsi ctrl base\n", __func__);
> >> ret = PTR_ERR(msm_host->ctrl_base);
> >> diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> >> index 0b2ae5c15240..6425b6c34f4c 100644
> >> --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> >> +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
> >> @@ -709,14 +709,14 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> >> if (!of_property_read_u32(dev->of_node, "phy-type", &phy_type))
> >> phy->cphy_mode = (phy_type == PHY_TYPE_CPHY);
> >>
> >> - phy->base = msm_ioremap_size(pdev, "dsi_phy", "DSI_PHY", &phy->base_size);
> >> + phy->base = msm_ioremap_size(pdev, "dsi_phy", &phy->base_size);
> >> if (IS_ERR(phy->base)) {
> >> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> >> ret = -ENOMEM;
> >> goto fail;
> >> }
> >>
> >> - phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", "DSI_PLL", &phy->pll_size);
> >> + phy->pll_base = msm_ioremap_size(pdev, "dsi_pll", &phy->pll_size);
> >> if (IS_ERR(phy->pll_base)) {
> >> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map pll base\n", __func__);
> >> ret = -ENOMEM;
> >> @@ -724,7 +724,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> >> }
> >>
> >> if (phy->cfg->has_phy_lane) {
> >> - phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", "DSI_PHY_LANE", &phy->lane_size);
> >> + phy->lane_base = msm_ioremap_size(pdev, "dsi_phy_lane", &phy->lane_size);
> >> if (IS_ERR(phy->lane_base)) {
> >> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy lane base\n", __func__);
> >> ret = -ENOMEM;
> >> @@ -733,7 +733,7 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
> >> }
> >>
> >> if (phy->cfg->has_phy_regulator) {
> >> - phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", "DSI_PHY_REG", &phy->reg_size);
> >> + phy->reg_base = msm_ioremap_size(pdev, "dsi_phy_regulator", &phy->reg_size);
> >> if (IS_ERR(phy->reg_base)) {
> >> DRM_DEV_ERROR(&pdev->dev, "%s: failed to map phy regulator base\n", __func__);
> >> ret = -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
> >> index 3acdeae25caf..3eb45cde0330 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
> >> @@ -128,7 +128,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> >> hdmi->config = config;
> >> spin_lock_init(&hdmi->reg_lock);
> >>
> >> - hdmi->mmio = msm_ioremap(pdev, config->mmio_name, "HDMI");
> >> + hdmi->mmio = msm_ioremap(pdev, config->mmio_name);
> >> if (IS_ERR(hdmi->mmio)) {
> >> ret = PTR_ERR(hdmi->mmio);
> >> goto fail;
> >> @@ -139,8 +139,7 @@ static struct hdmi *msm_hdmi_init(struct platform_device *pdev)
> >> config->mmio_name);
> >> hdmi->mmio_phy_addr = res->start;
> >>
> >> - hdmi->qfprom_mmio = msm_ioremap(pdev,
> >> - config->qfprom_mmio_name, "HDMI_QFPROM");
> >> + hdmi->qfprom_mmio = msm_ioremap(pdev, config->qfprom_mmio_name);
> >> if (IS_ERR(hdmi->qfprom_mmio)) {
> >> DRM_DEV_INFO(&pdev->dev, "can't find qfprom resource\n");
> >> hdmi->qfprom_mmio = NULL;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> >> index 16b0e8836d27..9780107e1cc9 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy.c
> >> @@ -144,7 +144,7 @@ static int msm_hdmi_phy_probe(struct platform_device *pdev)
> >> if (!phy->cfg)
> >> return -ENODEV;
> >>
> >> - phy->mmio = msm_ioremap(pdev, "hdmi_phy", "HDMI_PHY");
> >> + phy->mmio = msm_ioremap(pdev, "hdmi_phy");
> >> if (IS_ERR(phy->mmio)) {
> >> DRM_DEV_ERROR(dev, "%s: failed to map phy base\n", __func__);
> >> return -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> >> index 99c7853353fd..b06d9d25a189 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_phy_8996.c
> >> @@ -716,19 +716,18 @@ int msm_hdmi_pll_8996_init(struct platform_device *pdev)
> >>
> >> pll->pdev = pdev;
> >>
> >> - pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> >> + pll->mmio_qserdes_com = msm_ioremap(pdev, "hdmi_pll");
> >> if (IS_ERR(pll->mmio_qserdes_com)) {
> >> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> >> return -ENOMEM;
> >> }
> >>
> >> for (i = 0; i < HDMI_NUM_TX_CHANNEL; i++) {
> >> - char name[32], label[32];
> >> + char name[32];
> >>
> >> snprintf(name, sizeof(name), "hdmi_tx_l%d", i);
> >> - snprintf(label, sizeof(label), "HDMI_TX_L%d", i);
> >>
> >> - pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name, label);
> >> + pll->mmio_qserdes_tx[i] = msm_ioremap(pdev, name);
> >> if (IS_ERR(pll->mmio_qserdes_tx[i])) {
> >> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> >> return -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> >> index 562dfac67792..be4b0b67e797 100644
> >> --- a/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> >> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_pll_8960.c
> >> @@ -434,7 +434,7 @@ int msm_hdmi_pll_8960_init(struct platform_device *pdev)
> >> if (!pll)
> >> return -ENOMEM;
> >>
> >> - pll->mmio = msm_ioremap(pdev, "hdmi_pll", "HDMI_PLL");
> >> + pll->mmio = msm_ioremap(pdev, "hdmi_pll");
> >> if (IS_ERR(pll->mmio)) {
> >> DRM_DEV_ERROR(dev, "failed to map pll base\n");
> >> return -ENOMEM;
> >> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
> >> index ad35a5d94053..d593c5d7dbc7 100644
> >> --- a/drivers/gpu/drm/msm/msm_drv.c
> >> +++ b/drivers/gpu/drm/msm/msm_drv.c
> >> @@ -124,7 +124,7 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name)
> >> }
> >>
> >> static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name,
> >> - const char *dbgname, bool quiet, phys_addr_t *psize)
> >> + bool quiet, phys_addr_t *psize)
> >> {
> >> struct resource *res;
> >> unsigned long size;
> >> @@ -151,7 +151,7 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> >> }
> >>
> >> if (reglog)
> >> - printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
> >> + printk(KERN_DEBUG "IO:region %s %p %08lx\n", name, ptr, size);
> >>
> >> if (psize)
> >> *psize = size;
> >> @@ -159,22 +159,20 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
> >> return ptr;
> >> }
> >>
> >> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> >> - const char *dbgname)
> >> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name)
> >> {
> >> - return _msm_ioremap(pdev, name, dbgname, false, NULL);
> >> + return _msm_ioremap(pdev, name, false, NULL);
> >> }
> >>
> >> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> >> - const char *dbgname)
> >> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name)
> >> {
> >> - return _msm_ioremap(pdev, name, dbgname, true, NULL);
> >> + return _msm_ioremap(pdev, name, true, NULL);
> >> }
> >>
> >> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> >> - const char *dbgname, phys_addr_t *psize)
> >> + phys_addr_t *psize)
> >> {
> >> - return _msm_ioremap(pdev, name, dbgname, false, psize);
> >> + return _msm_ioremap(pdev, name, false, psize);
> >> }
> >>
> >> void msm_writel(u32 data, void __iomem *addr)
> >> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
> >> index d7574e6bd4e4..56d1242efcc1 100644
> >> --- a/drivers/gpu/drm/msm/msm_drv.h
> >> +++ b/drivers/gpu/drm/msm/msm_drv.h
> >> @@ -480,12 +480,10 @@ struct clk *msm_clk_get(struct platform_device *pdev, const char *name);
> >>
> >> struct clk *msm_clk_bulk_get_clock(struct clk_bulk_data *bulk, int count,
> >> const char *name);
> >> -void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
> >> - const char *dbgname);
> >> +void __iomem *msm_ioremap(struct platform_device *pdev, const char *name);
> >> void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
> >> - const char *dbgname, phys_addr_t *size);
> >> -void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name,
> >> - const char *dbgname);
> >> + phys_addr_t *size);
> >> +void __iomem *msm_ioremap_quiet(struct platform_device *pdev, const char *name);
> >> void msm_writel(u32 data, void __iomem *addr);
> >> u32 msm_readl(const void __iomem *addr);
> >> void msm_rmw(void __iomem *addr, u32 mask, u32 or);
> >> diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
> >> index 0f78c2615272..c4debc8ea3ba 100644
> >> --- a/drivers/gpu/drm/msm/msm_gpu.c
> >> +++ b/drivers/gpu/drm/msm/msm_gpu.c
> >> @@ -858,7 +858,7 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev,
> >>
> >>
> >> /* Map registers: */
> >> - gpu->mmio = msm_ioremap(pdev, config->ioname, name);
> >> + gpu->mmio = msm_ioremap(pdev, config->ioname);
> >> if (IS_ERR(gpu->mmio)) {
> >> ret = PTR_ERR(gpu->mmio);
> >> goto fail;
> >> --
> >> 2.33.0
> >>
>
>
> --
> With best wishes
> Dmitry
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-12-15 20:11 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-08 18:01 [PATCH] drm/msm: drop dbgname argument from msm_ioremap*() Dmitry Baryshkov
2021-12-08 18:01 ` Dmitry Baryshkov
2021-12-08 21:23 ` Rob Clark
2021-12-08 21:23 ` Rob Clark
2021-12-15 16:45 ` Dmitry Baryshkov
2021-12-15 16:45 ` Dmitry Baryshkov
2021-12-15 20:16 ` Rob Clark
2021-12-15 20:16 ` Rob Clark
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.