On Thu, Nov 01, 2018 at 11:09:56PM +0800, Peng Hao wrote:
> There is an extra semicolon in pvpanic_device_ids, remove it.
This is a comma, not a semicolon.
How about:
Since there should be nothing after the sentinel entry in an
acpi_device_id list, it doesn't make sense for it to have a trailing
comma.
This patch removes the redundant comma.
... with that commit message (and the title fixed to say 'comma' rather
than 'semicolon':
Acked-by: Mark Rutland <mark.rutland@xxxxxxx>
Either that, or simply drop this patch entirely.
Thanks,
Mark.
>
> Signed-off-by: Peng Hao <peng.hao2@xxxxxxxxxx>
> ---
> drivers/misc/pvpanic.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/misc/pvpanic.c b/drivers/misc/pvpanic.c
> index fd86dab..059005c 100644
> --- a/drivers/misc/pvpanic.c
> +++ b/drivers/misc/pvpanic.c
> @@ -35,7 +35,7 @@
>
> static const struct acpi_device_id pvpanic_device_ids[] = {
> { "QEMU0001", 0 },
> - { "", 0 },
> + { "", 0 }
> };
> MODULE_DEVICE_TABLE(acpi, pvpanic_device_ids);
>
> --
> 1.8.3.1
>
