On Wed, May 2, 2012 at 1:01 AM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > On Tue, May 1, 2012 at 10:19 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: >> On Tue, May 1, 2012 at 4:57 PM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote: >>> I spent most of the day merging the patches up to this point, and they >>> mostly make sense, but this one and the following ones are beyond my >>> ken, so I gave up. >> >> ok, let me check if i could simplify that code more. I suggested that you need a simpler *interface*, not just smaller patches and more comments. If the interfaces make sense and the code is good, you don't need many comments. > Split this one into two patches, and add more comments. > > Please check if it is readable. Nope, sorry, I'm not putting my name on that. If you still think that's the only reasonable way to do it, you might be able to find somebody else to merge the resource.c changes, and I'll just worry about the drivers/pci parts. Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html