drm/msm/sde: add register name to logging

Use the symbol name as a best effort way to log the register
name along with the value and offset in the register logging.

Change-Id: I8befb674d9599725f66eadc4a293dcb24b96302f
Signed-off-by: Lloyd Atkinson <latkinso@codeaurora.org>

2 files changed, 14 insertions, 6 deletions

diff --git a/drivers/gpu/drm/msm/sde/sde_hw_util.c b/drivers/gpu/drm/msm/sde/sde_hw_util.c
index f87c0eed4cd1..51d8c2e50529 100644
--- a/drivers/gpu/drm/msm/sde/sde_hw_util.c
+++ b/drivers/gpu/drm/msm/sde/sde_hw_util.c
@@ -16,15 +16,18 @@
 /* using a file static variables for debugfs access */
 static u32 sde_hw_util_log_mask = SDE_DBG_MASK_NONE;
 
-void SDE_REG_WRITE(struct sde_hw_blk_reg_map *c, u32 reg_off, u32 val)
+void sde_reg_write(struct sde_hw_blk_reg_map *c,
+		u32 reg_off,
+		u32 val,
+		const char *name)
 {
 	/* don't need to mutex protect this */
 	if (c->log_mask & sde_hw_util_log_mask)
-		DBG("[0x%X] <= 0x%X", c->blk_off + reg_off, val);
+		DBG("[%s:0x%X] <= 0x%X", name, c->blk_off + reg_off, val);
 	writel_relaxed(val, c->base_off + c->blk_off + reg_off);
 }
 
-int SDE_REG_READ(struct sde_hw_blk_reg_map *c, u32 reg_off)
+int sde_reg_read(struct sde_hw_blk_reg_map *c, u32 reg_off)
 {
 	return readl_relaxed(c->base_off + c->blk_off + reg_off);
 }
diff --git a/drivers/gpu/drm/msm/sde/sde_hw_util.h b/drivers/gpu/drm/msm/sde/sde_hw_util.h
index 7866d18c3081..a4d8be9de907 100644
--- a/drivers/gpu/drm/msm/sde/sde_hw_util.h
+++ b/drivers/gpu/drm/msm/sde/sde_hw_util.h
@@ -36,9 +36,14 @@ struct sde_hw_blk_reg_map {
 
 u32 *sde_hw_util_get_log_mask_ptr(void);
 
-void SDE_REG_WRITE(struct sde_hw_blk_reg_map *c, u32 reg_off, u32 val);
-
-int SDE_REG_READ(struct sde_hw_blk_reg_map *c, u32 reg_off);
+void sde_reg_write(struct sde_hw_blk_reg_map *c,
+		u32 reg_off,
+		u32 val,
+		const char *name);
+int sde_reg_read(struct sde_hw_blk_reg_map *c, u32 reg_off);
+
+#define SDE_REG_WRITE(c, off, val) sde_reg_write(c, off, val, #off)
+#define SDE_REG_READ(c, off) sde_reg_read(c, off)
 
 void *sde_hw_util_get_dir(void);