On Sat, Dec 19, 2015 at 01:19:26PM +0100, Robert Jarzmik wrote: > Brian Norris <computersforpe...@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.
Regmap is probably overkill, and wouldn't do a lot to satisfy Ezequiel's concern, I expect. > If we all agree on a path I could even make the final patch, whichever > solution > is chosen. I'm OK with this one, which still applies OK for me. But I do have one comment, which I'll post at the top-level. Brian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/