aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2022-03-06 16:00:16 +0000
committerMauro Carvalho Chehab <mchehab@kernel.org>2022-06-27 08:38:36 +0100
commita124765f8781a75df6c491bd04a0389ce078454c (patch)
tree0fcf24843334b2dddc0dda4d5b342a02c8f05f84 /drivers
parent7d62f2f367f89db7ac9f03a4f664cc488ab764d8 (diff)
media: rkisp1: debug: Update max register name length
Update the maximum register name length to match all the registers being printed. This helps getting a consistent alignment of register dumps when concatenating multiple debugfs files. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Ricardo Ribalda <ribalda@chromium.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/media/platform/rockchip/rkisp1/rkisp1-debug.c12
1 files changed, 9 insertions, 3 deletions
diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-debug.c b/drivers/media/platform/rockchip/rkisp1/rkisp1-debug.c
index 2c226f20f525..e76dc2b164b6 100644
--- a/drivers/media/platform/rockchip/rkisp1/rkisp1-debug.c
+++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-debug.c
@@ -11,8 +11,10 @@
#include <linux/debugfs.h>
#include <linux/delay.h>
#include <linux/device.h>
+#include <linux/minmax.h>
#include <linux/pm_runtime.h>
#include <linux/seq_file.h>
+#include <linux/string.h>
#include "rkisp1-common.h"
#include "rkisp1-regs.h"
@@ -28,10 +30,14 @@ struct rkisp1_debug_register {
RKISP1_CIF_##name, RKISP1_CIF_##name##_SHD, #name \
}
+/* Keep this up-to-date when adding new registers. */
+#define RKISP1_MAX_REG_LENGTH 21
+
static int rkisp1_debug_dump_regs(struct rkisp1_device *rkisp1,
struct seq_file *m, unsigned int offset,
const struct rkisp1_debug_register *regs)
{
+ const int width = RKISP1_MAX_REG_LENGTH;
u32 val, shd;
int ret;
@@ -39,15 +45,15 @@ static int rkisp1_debug_dump_regs(struct rkisp1_device *rkisp1,
if (ret <= 0)
return ret ? : -ENODATA;
- for ( ; regs->name; ++regs) {
+ for (; regs->name; ++regs) {
val = rkisp1_read(rkisp1, offset + regs->reg);
if (regs->shd) {
shd = rkisp1_read(rkisp1, offset + regs->shd);
- seq_printf(m, "%14s: 0x%08x/0x%08x\n", regs->name,
+ seq_printf(m, "%*s: 0x%08x/0x%08x\n", width, regs->name,
val, shd);
} else {
- seq_printf(m, "%14s: 0x%08x\n", regs->name, val);
+ seq_printf(m, "%*s: 0x%08x\n", width, regs->name, val);
}
}

Privacy Policy