drm/msm: drop register logging support

Register logging was used during early stages of msm driver development
to compare upstream and downstream register traces. However the tool was
never updated to work with mdp5 hardware. Later it was dropped
completely when Rob imported freedreno tools into mesa. All this makes
DRM_MSM_REGISTER_LOGGING irrelevant now, so it can be dropped.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Link: https://lore.kernel.org/r/20220105232700.444170-1-dmitry.baryshkov@linaro.org
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

diff --git a/drivers/gpu/drm/msm/Kconfig b/drivers/gpu/drm/msm/Kconfig
index 1eae5a9645f41dc134dc559eddf889d7e8c181f9..9b019598e042e26aed3c043ba34fcb31f5d01eca 100644 (file)
--- a/drivers/gpu/drm/msm/Kconfig
+++ b/drivers/gpu/drm/msm/Kconfig
@@ -33,15 +33,6 @@ config DRM_MSM_GPU_STATE
        depends on DRM_MSM && (DEBUG_FS || DEV_COREDUMP)
        default y
 
-config DRM_MSM_REGISTER_LOGGING
-       bool "MSM DRM register logging"
-       depends on DRM_MSM
-       default n
-       help
-         Compile in support for logging register reads/writes in a format
-         that can be parsed by envytools demsm tool.  If enabled, register
-         logging can be switched on via msm.reglog=y module param.
-
 config DRM_MSM_GPU_SUDO
        bool "Enable SUDO flag on submits"
        depends on DRM_MSM && EXPERT

diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index 555666e3f9603a2305a44c84453a56ceda2b9abf..1145e06513df57c285677fa6a41c611c647b2d68 100644 (file)
--- a/drivers/gpu/drm/msm/msm_drv.c
+++ b/drivers/gpu/drm/msm/msm_drv.c
@@ -57,14 +57,6 @@ static const struct drm_mode_config_helper_funcs mode_config_helper_funcs = {
        .atomic_commit_tail = msm_atomic_commit_tail,
 };
 
-#ifdef CONFIG_DRM_MSM_REGISTER_LOGGING
-static bool reglog;
-MODULE_PARM_DESC(reglog, "Enable register read/write logging");
-module_param(reglog, bool, 0600);
-#else
-#define reglog 0
-#endif
-
 #ifdef CONFIG_DRM_FBDEV_EMULATION
 static bool fbdev = true;
 MODULE_PARM_DESC(fbdev, "Enable fbdev compat layer");
@@ -150,9 +142,6 @@ static void __iomem *_msm_ioremap(struct platform_device *pdev, const char *name
                return ERR_PTR(-ENOMEM);
        }
 
-       if (reglog)
-               printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
-
        if (psize)
                *psize = size;
 
@@ -179,16 +168,13 @@ void __iomem *msm_ioremap_size(struct platform_device *pdev, const char *name,
 
 void msm_writel(u32 data, void __iomem *addr)
 {
-       if (reglog)
-               printk(KERN_DEBUG "IO:W %p %08x\n", addr, data);
        writel(data, addr);
 }
 
 u32 msm_readl(const void __iomem *addr)
 {
        u32 val = readl(addr);
-       if (reglog)
-               pr_err("IO:R %p %08x\n", addr, val);
+
        return val;
 }