pinctrl: npcm8xx: Fix incorrect struct npcm8xx_pincfg assignment
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 18 Mar 2025 10:57:14 +0000 (12:57 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 20 Mar 2025 08:14:09 +0000 (09:14 +0100)
commit113ec87b0f26a17b02c58aa2714a9b8f1020eed9
treeb203e94ad9e672b9a76695ecc3db02e68428184d
parent5a062c3c3b82004766bc3ece82b594d337076152
pinctrl: npcm8xx: Fix incorrect struct npcm8xx_pincfg assignment

Sparse is not happy about implementation of the NPCM8XX_PINCFG()

 pinctrl-npcm8xx.c:1314:9: warning: obsolete array initializer, use C99 syntax
 pinctrl-npcm8xx.c:1315:9: warning: obsolete array initializer, use C99 syntax
 ...
 pinctrl-npcm8xx.c:1412:9: warning: obsolete array initializer, use C99 syntax
 pinctrl-npcm8xx.c:1413:9: warning: too many warnings

which uses index-based assignment in a wrong way, i.e. it missed
the equal sign and hence the index is simply ignored, while the
entries are indexed naturally. This is not a problem as the pin
numbering repeats the natural order, but it might be in case of
shuffling the entries. Fix this by adding missed equal sign and
reformat a bit for better readability.

Fixes: acf4884a5717 ("pinctrl: nuvoton: add NPCM8XX pinctrl and GPIO driver")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/20250318105932.2090926-2-andriy.shevchenko@linux.intel.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/nuvoton/pinctrl-npcm8xx.c