Re: [PATCH v4 1/6] pinctrl: Add pingroup and define PINCTRL_PINGROUP

[Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>]

On Wed, Jun 01, 2022 at 03:32:09PM +0530, Basavaraj Natikar wrote:
> Add 'struct pingroup' to represent pingroup and 'PINCTRL_PINGROUP'
> macro for inline use. Both are used to manage and represent
> larger number of pingroups.

...

> +/* Convenience macro to define a single named or anonymous pingroup */
> +#define PINCTRL_PINGROUP(_name, _pins, _npins)	\
> +((struct pingroup){				\
> +	.name = _name,				\
> +	.pins = _pins,				\
> +	.npins = _npins,			\
> +})

Still the same Q: Why do you need GCC expression (besides the checkpatch,
likely false positive, warning)?

Note, we have many compound literals in the kernel: first comes to my mind
is property.h.

-- 
With Best Regards,
Andy Shevchenko