On Thu, Feb 25, 2016 at 1:54 AM, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > > Usually, the patch title should be prefixed by the subsystem it > applies to so that maintainer and reviewers can spot it more > easily. In this case, it would be something like > > pinctrl: sunxi: Fix sun7i pin assignment for IRQ's > > On Mon, Feb 22, 2016 at 06:14:07PM +0100, Henry Paulissen wrote: >> After testing IRQ pins we found some bugs in the pinctrl declaration. >> Both PI* and PC* pins didn't work. PI* pins seemed to be connected >> to the wrong mux and PC* pins waren't working at all. >> >> Please note that the A20 soc manual is contradicting between version >> and even within the same document for both the PI and PC pins. > > Which sections are in contradiction? > >> Patch is based on testing with the hardware itself. > > How did you test it? Using the sysfs API, or did you have any hardware > connected to it? I'm taking this patch out of my tree now following Maxime's comments. I'm a bit annoyed that it is iterating too quickly as well, I will wait and see for a while. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html