Dear Jean-Christophe PLAGNIOL-VILLARD, On Thu, 23 May 2013 13:58:45 +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > Or, alternatively, we can make the irqchip driver show using > > pr_debug() the value of IMR in its ->suspend() hook, the value of IMR & > > IPR in its ->resume() hook, and the pm.c code can show the PMC_PCSR on > > suspend. This way, you have all the informations you need for debugging. > > if you read the code as you said both of them are link PMC & AIC so you can > not hope to split it as both of the IP contain part of the information > > so no I want the information right where it's Sorry, but that seems like a completely unreasonable requirement, provided that it makes the PM code poke into a completely unrelated IP, just for the sake of one debug message. > > Having the PM driver and the irqchip driver step on each other register > > areas is anyway a bad idea, especially when the only reason to do so is > > for two debugging messages that can be implemented differently. > > > > > Notice that this solution can always be improved later on if needed, > > with further cleanups from you. > > > > But please stop rejecting valid patches from contributors, just because > > the code is not coming from you. What Boris is proposing is perfectly > so If I read you non clean code can come mainline, wow this not the way to > work. If for you do half clean is enough to go mainline. You really need to > change your way of working. Boris patches are far from being half-clean. To me, they look perfectly sane, and are going in the right direction. Again, please stop rejecting contributions from third-parties just because you're not the author of such contributions. It's you who said that they are not clean, but you still haven't given the slightest explanation to justify that. Until a proper justification is given, that points to specific, identifiable, and reasonable problems in the patches, there is absolutely no reason not to consider Boris patches as acceptable (once the header problem is fixed, of course). Best regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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/
