Re: [PATCH 1/1] pinctrl: imx: fix NO_PAD_CTL setting for MMIO pads

[Kevin Hilman <khilman@xxxxxxxxxxxx>]

"A.s. Dong" <aisheng.dong@xxxxxxx> writes:

> After patch b96eea718bf6 ("pinctrl: fsl: add scu based pinctrl support"),
> NO_PAD_CTL pads map are not skipped anymore which results in
> a possible memory corruption. As we actually only need to create config
> maps for SCU pads and MMIO pads which are not using the default config
> (a.k.a IMX_NO_PAD_CTL), so let's add a proper check before creating
> the config maps. And during MMIO pads parsing, we also need update the
> list_p point as SCU case to ensure the pin data next parsed is correct.
>
> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Fixes: b96eea718bf6 ("pinctrl: fsl: add scu based pinctrl support")
> Reported-by: Martin Kaiser <martin@xxxxxxxxx>
> Suggested-by: Leonard Crestez <leonard.crestez@xxxxxxx>
> Signed-off-by: Dong Aisheng <aisheng.dong@xxxxxxx>

After kernelCI found a similar failure, I verified that this patch gets
things booting again in today's linux-next.

Tested-by: Kevin Hilman <khilman@xxxxxxxxxxxx>

Kevin