Hi, On 6/15/23 16:41, Andy Shevchenko wrote: > 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. Ack, thank you. Fixed for the next version. Regards, Hans > >> +}; >