On Thu, Feb 15, 2018 at 1:19 PM, John Garry <john.ga...@huawei.com> 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

Reply via email to