Re: [PATCH v13 7/9] ACPI: Translate the I/O range of non-MMIO devices before scanning

From: Andy Shevchenko
Date: Thu Feb 15 2018 - 07:23:07 EST


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