From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Date: Fri, 06 Mar 2015 12:42:05 +0000 Subject: Re: [PATCH] sh-pfc: handle pin array holes in sh_pfc_map_pins() Message-Id: <54F9A09D.7040909@cogentembedded.com> List-Id: References: <1528110.Mb5xiNU0rc@wasted.cogentembedded.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Linus Walleij Cc: "linux-sh@vger.kernel.org" , Laurent Pinchart , "linux-gpio@vger.kernel.org" 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 >> --- >> 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