Re: [PATCH v3 1/2] iio: imu: st_lsm6dsx: Drop trailing comma in ACPI match table entries

[Andy Shevchenko]

On Sun, Sep 27, 2026 at 05:05:58PM -0500, Jared Wuerzburger wrote:
> The trailing comma after the last member of an initializer is only
> useful when more members may follow, which is never the case for an
> acpi_device_id entry. Drop it, as suggested during review of an earlier
> version of the next patch.

> Link: https://lore.kernel.org/linux-iio/aVFmec8JA6ZX18il@xxxxxxxxxxxxxxxxxx/
> Suggested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
> Assisted-by: Claude:claude-opus-5-5

Assisted-by: LLM

> Signed-off-by: Jared Wuerzburger <jwuerz@xxxxxxxxx>

...

>  static const struct acpi_device_id st_lsm6dsx_i2c_acpi_match[] = {
> -	{ "SMO8B30", ST_LSM6DS3TRC_ID, },
> -	{ "SMOCF00", ST_LSM6DSO_ID, },
> +	{ "SMO8B30", ST_LSM6DS3TRC_ID },
> +	{ "SMOCF00", ST_LSM6DSO_ID },
>  	{ }
>  };

Thanks, but while at it, also switch to C99 initialisers to avoid an additional
churn (there is ongoing activity to switch all ID tables to use C99 initialisers).

-- 
With Best Regards,
Andy Shevchenko