On 29.11.2023 17:06, Andy Shevchenko wrote:
> Make use of PINCTRL_GROUP_DESC() instead of open coding it.
>
> Signed-off-by: Andy Shevchenko<andriy.shevchenko@xxxxxxxxxxxxxxx>
This patch landed in linux-next as commit 1949e4630c3b ("pinctrl:
mediatek: Make use of PINCTRL_GROUP_DESC()"). Unfortunately it causes a
build break of ARM64 arch with standard defconfig.
> ---
> drivers/pinctrl/mediatek/pinctrl-moore.h | 7 +------
> drivers/pinctrl/mediatek/pinctrl-paris.h | 7 +------
> 2 files changed, 2 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/pinctrl/mediatek/pinctrl-moore.h b/drivers/pinctrl/mediatek/pinctrl-moore.h
> index e1b4b82b9d3d..22ef1ffbcdcb 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-moore.h
> +++ b/drivers/pinctrl/mediatek/pinctrl-moore.h
> @@ -38,12 +38,7 @@
> }
>
> #define PINCTRL_PIN_GROUP(name, id) \
> - { \
> - name, \
> - id##_pins, \
> - ARRAY_SIZE(id##_pins), \
> - id##_funcs, \
> - }
> + PINCTRL_GROUP_DESC(name, id##_pins, ARRAY_SIZE(id##_pins), id##_funcs)
> int mtk_moore_pinctrl_probe(struct platform_device *pdev,
> const struct mtk_pin_soc *soc);
> diff --git a/drivers/pinctrl/mediatek/pinctrl-paris.h b/drivers/pinctrl/mediatek/pinctrl-paris.h
> index 8762ac599329..f208a904c4a8 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-paris.h
> +++ b/drivers/pinctrl/mediatek/pinctrl-paris.h
> @@ -50,12 +50,7 @@
> }
>
> #define PINCTRL_PIN_GROUP(name, id) \
> - { \
> - name, \
> - id##_pins, \
> - ARRAY_SIZE(id##_pins), \
> - id##_funcs, \
> - }
> + PINCTRL_GROUP_DESC(name, id##_pins, ARRAY_SIZE(id##_pins), id##_funcs)
>
> int mtk_paris_pinctrl_probe(struct platform_device *pdev);
PINCTRL_GROUP_DESC() macro from drivers/pinctrl/core.h contains a cast
to (struct group_desc), what breaks users of the above macros.
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
