Re: [PATCH] debugfs: allow to use regmap for print regs

From: Li Chen
Date: Wed Jan 11 2023 - 02:24:31 EST


Hi Greg,

> > void debugfs_print_regs32(struct seq_file *s, const struct debugfs_reg32 *regs,
> > -             int nregs, void __iomem *base, char *prefix)
> > +             int nregs, void __iomem *base, struct regmap *regmap,
> > +             char *prefix)
>
> This is confusing, please just provide a new function for regmap instead
> of trying to overload this function. You can use the same function
> inside the debugfs core like this, but never export such a complex set
> of parameters to actual users, that's just painful.

Thanks for your review, updated in v2.

Regards,
Li