On Fri, Mar 6, 2015 at 11:58 AM, Laurent Pinchart <[email protected]> wrote: > Hi Linus, > > On Friday 06 March 2015 11:43:45 Linus Walleij wrote: >> On Sat, Feb 28, 2015 at 12:39 AM, Sergei Shtylyov wrote: >> > The pin array handled by sh_pfc_map_pins() may contain holes representing >> > non- existing pins. We have to first count the valid pins in order to >> > calculate the size of the memory to be allocated, then to skip over the >> > non-existing pins when initializing the allocated arrays, and then to >> > return the number of valid pins from sh_pfc_map_pins() instead of 0 on >> > success. >> > >> > As we have to touch devm_kzalloc() calls anyway, use more fitting >> > devm_kcalloc() instead which additionally checks the array size. And >> > since PINMUX_TYPE_NONE is #define'd as 0, stop re-initializing already >> > zeroed out 'pmx->configs' array. >> > >> > Signed-off-by: Sergei Shtylyov <[email protected]> >> > >> > --- >> > The patch is against the 'devel' branch of Linus W.'s 'linux-pinctrl.git' >> > repo. This patch should be applied before my R8A7794 PFC support patch >> > and before Laurent's patches removing non-existent GPIOs for R8A779[01], >> > otherwise they would cause the kernel to hang while booting! >> >> OK not applying this until ACKed by Laurent, >> and in the meantime I'm taking the three applied R8A7794 patches >> out of my tree again. > > Could you also drop "pinctrl: sh-pfc: r8a7790: Remove non existing GPIO pins" > and "pinctrl: sh-pfc: r8a7791: Remove non existing GPIO pins" for the same > reason ? Sorry for the trouble.
OK dropped them (rebased .... argh hope noone will kick my ass) Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
