Re: [PATCH 1/3] mtd: nand: pxa3xx_nand: add register access debug

From	Robert Jarzmik <robert.jarzmik@free.fr>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/3] mtd: nand: pxa3xx_nand: add register access debug
Date	2015-12-19 13:20 +0100
Message-ID	<qHoEp-33W-11@gated-at.bofh.it> (permalink)
References	(1 earlier) <pYGH7-3Jf-1@gated-at.bofh.it> <pYTXI-61Z-13@gated-at.bofh.it> <q0IYa-1r5-17@gated-at.bofh.it> <q10in-T2-31@gated-at.bofh.it> <qHdSF-4si-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Brian Norris <computersforpeace@gmail.com> writes:

> I don't have very strong opinions on this. It's kind of annoying to have
> this sort of stuff duplicated for every driver, if it's really needed.
> But I'll admit this kind of infrastructure is sometimes useful.
>
> Anecdote: I recently found the regmap trace event infrastructure pretty
> nice for debugging some other drivers. This would only require you to
> have tracing enabled, and then no recompiles are necessary at all. Just
> cmdline changes.
>
> So, I could go with this patch, if Robert still desires it. Or you could
> convert to using regmap for MMIO :)
I'm as you, I don't feel strong opinion about it, I'd like to have a debug
tracing tool, be that this patch or regmap MMIO.

If we all agree on a path I could even make the final patch, whichever solution
is chosen.

Cheers.

-- 
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 1/3] mtd: nand: pxa3xx_nand: add register access debug Brian Norris <computersforpeace@gmail.com> - 2015-12-19 01:50 +0100
  Re: [PATCH 1/3] mtd: nand: pxa3xx_nand: add register access debug Robert Jarzmik <robert.jarzmik@free.fr> - 2015-12-19 13:20 +0100

csiph-web