[PATCH v1 2/3] mfd: Initialize spi_device_id arrays using member names
Charles Keepax
ckeepax at opensource.cirrus.com
Mon Jul 6 01:29:39 PDT 2026
On Thu, Jul 02, 2026 at 04:53:40PM +0200, Uwe Kleine-König (The Capable Hub) wrote:
> While being less compact, using named initializers allows to more easily
> see which members of the structs are assigned which value without having
> to lookup the declaration of the struct. And it's also more robust
> against changes to the struct definition.
>
> The mentioned robustness is relevant for a planned change to struct
> spi_device_id that replaces .driver_data by an anonymous union.
>
> Also adapt spacing and usage of commas to the most common style.
>
> This patch doesn't modify the compiled array, only its representation
> in source form benefits.
>
> Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig at baylibre.com>
> ---
Cirrus/Wolfson bits all look good to me:
Reviewed-by: Charles Keepax <ckeepax at opensource.cirrus.com>
Thanks,
Charles
More information about the linux-arm-kernel
mailing list