On Tue, Jun 15, 2021 at 7:11 PM Wei Yongjun <weiyongjun1@xxxxxxxxxx> wrote: > > The sparse tool complains as follows: > > drivers/acpi/prmt.c:53:1: warning: > symbol 'prm_module_list' was not declared. Should it be static? > > This symbol is not used outside of prmt.c, so marks it static. > > Fixes: cefc7ca46235 ("ACPI: PRM: implement OperationRegion handler for the PlatformRtMechanism subtype") > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> > --- > drivers/acpi/prmt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/prmt.c b/drivers/acpi/prmt.c > index 33c274698d07..31cf9aee5edd 100644 > --- a/drivers/acpi/prmt.c > +++ b/drivers/acpi/prmt.c > @@ -50,7 +50,7 @@ struct prm_context_buffer { > #pragma pack() > > > -LIST_HEAD(prm_module_list); > +static LIST_HEAD(prm_module_list); > > struct prm_handler_info { > guid_t guid; > Applied, thanks!