On Thu, Jun 15, 2023 at 04:13:38PM +0200, Hans de Goede wrote: > Add an acpi_match_table now that all the other bits necessary for > ACPI support are in place. ... > +static const struct acpi_device_id ov2680_acpi_ids[] = { > + { "OVTI2680" }, > + { /* sentinel */ }, Comma is not needed for the terminator entry. > +}; -- With Best Regards, Andy Shevchenko