On Wed, 2012-10-31 at 14:57 +0200, Vladimir Kondratiev wrote: > > You didn't also address my comments regarding WIL6210_ISR_COR #ifdef code. > > I added string with short explanation in Kconfig. > Longer explanation: while clear-on-read is good for production mode, it makes > debugging much harder - reading ISR registers clears interrupt, and one can no > more monitor ISR with debugfs. So, when debugging ISR flows - and they still > need some debugging - one have to use W1C mode. That's why it is still > present. Since I'm CC'ed on your patches (not sure why) I'll offer the opinion that you should add interrupt and IO access tracing rather than reading registers with some debug thing ;) johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
