On Wed, Apr 15, 2020 at 10:23 AM Jason Yan <yanaijie@xxxxxxxxxx> wrote: > > Fix the following sparse warning: > > drivers/acpi/acpica/exfield.c:25:10: warning: symbol > 'acpi_protocol_lengths' was not declared. Should it be static? > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Jason Yan <yanaijie@xxxxxxxxxx> Erik, Bob, this one is yours. :-) > --- > drivers/acpi/acpica/exfield.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/acpica/exfield.c b/drivers/acpi/acpica/exfield.c > index e85eb31e5075..3323a2ba6a31 100644 > --- a/drivers/acpi/acpica/exfield.c > +++ b/drivers/acpi/acpica/exfield.c > @@ -22,7 +22,7 @@ ACPI_MODULE_NAME("exfield") > */ > #define ACPI_INVALID_PROTOCOL_ID 0x80 > #define ACPI_MAX_PROTOCOL_ID 0x0F > -const u8 acpi_protocol_lengths[] = { > +static const u8 acpi_protocol_lengths[] = { > ACPI_INVALID_PROTOCOL_ID, /* 0 - reserved */ > ACPI_INVALID_PROTOCOL_ID, /* 1 - reserved */ > 0x00, /* 2 - ATTRIB_QUICK */ > -- > 2.21.1 >