On Thu, Feb 15, 2018 at 1:19 PM, John Garry <john.garry@xxxxxxxxxx> wrote: > +static const struct acpi_device_id indirect_io_hosts[] = { > + {"HISI0191", 0}, /* HiSilicon LPC host */ > + {}, Just a nit. It seems lately this happens more often than usual, I mean a comma in the terminator line. If we remove it. we terminate not only at runtime, but at compile time, which is slightly better. > +}; -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html