Re: [PATCH] sh-pfc: handle pin array holes in sh_pfc_map_pins()

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

On 3/6/2015 1:43 PM, Linus Walleij 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 <sergei.shtylyov@cogentembedded.com>

>> ---
>> 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.

    Thanks!

> Can you please send these depending patches as a series with this
> patch as 1/4 and the three R8A7794 patches as 2,3,4/4? Thanks.

    OK, will try to remember. But what about Lauren't 2 patches also dependent 
on this one?

> Yours,
> Linus Walleij

WBR, Sergei