On Thu, Feb 15, 2018 at 1:19 PM, John Garry [off-list ref] 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