diff mbox series

PCI: microchip: Make the struct event_descs static

Message ID 20210509041932.560340-1-kw@linux.com (mailing list archive)
State Accepted
Delegated to: Lorenzo Pieralisi
Headers show
Series PCI: microchip: Make the struct event_descs static | expand

Commit Message

Krzysztof Wilczyński May 9, 2021, 4:19 a.m. UTC
The struct event_descs does not have any users outside the
pcie-microchip-host.c file, and has no previous declaration,
thus it can be made static.

This resolves the following sparse warning:

  drivers/pci/controller/pcie-microchip-host.c:352:3: warning: symbol 'event_descs' was not declared. Should it be static?

Signed-off-by: Krzysztof Wilczyński <kw@linux.com>
---
 drivers/pci/controller/pcie-microchip-host.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Lorenzo Pieralisi June 3, 2021, 4:12 p.m. UTC | #1
On Sun, 9 May 2021 04:19:32 +0000, Krzysztof Wilczyński wrote:
> The struct event_descs does not have any users outside the
> pcie-microchip-host.c file, and has no previous declaration,
> thus it can be made static.
> 
> This resolves the following sparse warning:
> 
>   drivers/pci/controller/pcie-microchip-host.c:352:3: warning: symbol 'event_descs' was not declared. Should it be static?

Applied to pci/microchip, thanks!

[1/1] PCI: microchip: Make the struct event_descs static
      https://git.kernel.org/lpieralisi/pci/c/1243106474

Thanks,
Lorenzo
diff mbox series

Patch

diff --git a/drivers/pci/controller/pcie-microchip-host.c b/drivers/pci/controller/pcie-microchip-host.c
index 89c68c56d93b..fdab8202ae5d 100644
--- a/drivers/pci/controller/pcie-microchip-host.c
+++ b/drivers/pci/controller/pcie-microchip-host.c
@@ -341,7 +341,7 @@  static struct event_map local_status_to_event[] = {
 	LOCAL_STATUS_TO_EVENT_MAP(PM_MSI_INT_SYS_ERR),
 };
 
-struct {
+static struct {
 	u32 base;
 	u32 offset;
 	u32 mask;