[PATCH v2 0/3] Get rid of big array from imx pinctrl driver

[Fabio Estevam]

On Sun, Apr 7, 2013 at 11:32 AM, Fabio Estevam <festevam at gmail.com> wrote:
> On Sun, Apr 7, 2013 at 11:26 AM, Shawn Guo <shawn.guo at linaro.org> wrote:
>
>> That's expected, as the pin ID gets re-numbered based on offset of
>> mux/conf register, so that we can determine the pin ID at runtime and
>> avoid encoding it in device tree.
>
> but debug messages seems buggy now:
>
> Prior to the patch we had:
>
> imx6q-pinctrl 20e0000.iomuxc: pinconf set pin MX6Q_PAD_EIM_D21
>
> and now we have:
>
> imx6q-pinctrl 20e0000.iomuxc: pinconf set pin MX6Q_PAD_EIM_A17
>
> MX6Q_PAD_EIM_A17 has nothing to do with I2C, so we should print
/s/should/should not/

> MX6Q_PAD_EIM_A17 when we are actually configuring MX6Q_PAD_EIM_D21.
>
> Also, where does the 19 offset in the  imx6q_pads enum come from?