* [PATCH] [V2] PCI: sysfs: remove pci_bridge_groups and pcie_dev_groups
@ 2019-10-16 8:03 Ben Dooks (Codethink)
2019-10-16 22:17 ` Bjorn Helgaas
0 siblings, 1 reply; 2+ messages in thread
From: Ben Dooks (Codethink) @ 2019-10-16 8:03 UTC (permalink / raw)
To: linux-kernel; +Cc: Ben Dooks, Bjorn Helgaas, linux-pci, linux-kernel
From: Ben Dooks <ben.dooks@codethink.co.uk>
The pci_bridge_groups and pcie_dev_groups objects are
not exported and not used at-all, so remove them to
fix the following warnings from sparse:
drivers/pci/pci-sysfs.c:1546:30: warning: symbol 'pci_bridge_groups' was not declared. Should it be static?
drivers/pci/pci-sysfs.c:1555:30: warning: symbol 'pcie_dev_groups' was not declared. Should it be static?
Also remove the unused pci_bridge_group and pcie_dev_group
as they are not used any more.
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
Cc: Bjorn Helgaas <bhelgaas@google.com>
Cc: linux-pci@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
fixup - more unused pci bits
---
drivers/pci/pci-sysfs.c | 18 ------------------
1 file changed, 18 deletions(-)
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
index 793412954529..eaffb477c5bf 100644
--- a/drivers/pci/pci-sysfs.c
+++ b/drivers/pci/pci-sysfs.c
@@ -1539,24 +1539,6 @@ const struct attribute_group *pci_dev_groups[] = {
NULL,
};
-static const struct attribute_group pci_bridge_group = {
- .attrs = pci_bridge_attrs,
-};
-
-const struct attribute_group *pci_bridge_groups[] = {
- &pci_bridge_group,
- NULL,
-};
-
-static const struct attribute_group pcie_dev_group = {
- .attrs = pcie_dev_attrs,
-};
-
-const struct attribute_group *pcie_dev_groups[] = {
- &pcie_dev_group,
- NULL,
-};
-
static const struct attribute_group pci_dev_hp_attr_group = {
.attrs = pci_dev_hp_attrs,
.is_visible = pci_dev_hp_attrs_are_visible,
--
2.23.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] [V2] PCI: sysfs: remove pci_bridge_groups and pcie_dev_groups
2019-10-16 8:03 [PATCH] [V2] PCI: sysfs: remove pci_bridge_groups and pcie_dev_groups Ben Dooks (Codethink)
@ 2019-10-16 22:17 ` Bjorn Helgaas
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2019-10-16 22:17 UTC (permalink / raw)
To: Ben Dooks (Codethink); +Cc: linux-kernel, linux-pci, linux-kernel
On Wed, Oct 16, 2019 at 09:03:24AM +0100, Ben Dooks (Codethink) wrote:
> From: Ben Dooks <ben.dooks@codethink.co.uk>
>
> The pci_bridge_groups and pcie_dev_groups objects are
> not exported and not used at-all, so remove them to
> fix the following warnings from sparse:
>
> drivers/pci/pci-sysfs.c:1546:30: warning: symbol 'pci_bridge_groups' was not declared. Should it be static?
> drivers/pci/pci-sysfs.c:1555:30: warning: symbol 'pcie_dev_groups' was not declared. Should it be static?
>
> Also remove the unused pci_bridge_group and pcie_dev_group
> as they are not used any more.
>
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Applied to pci/misc for v5.5, thanks a lot, Ben!
> ---
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: linux-pci@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
>
> fixup - more unused pci bits
> ---
> drivers/pci/pci-sysfs.c | 18 ------------------
> 1 file changed, 18 deletions(-)
>
> diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
> index 793412954529..eaffb477c5bf 100644
> --- a/drivers/pci/pci-sysfs.c
> +++ b/drivers/pci/pci-sysfs.c
> @@ -1539,24 +1539,6 @@ const struct attribute_group *pci_dev_groups[] = {
> NULL,
> };
>
> -static const struct attribute_group pci_bridge_group = {
> - .attrs = pci_bridge_attrs,
> -};
> -
> -const struct attribute_group *pci_bridge_groups[] = {
> - &pci_bridge_group,
> - NULL,
> -};
> -
> -static const struct attribute_group pcie_dev_group = {
> - .attrs = pcie_dev_attrs,
> -};
> -
> -const struct attribute_group *pcie_dev_groups[] = {
> - &pcie_dev_group,
> - NULL,
> -};
> -
> static const struct attribute_group pci_dev_hp_attr_group = {
> .attrs = pci_dev_hp_attrs,
> .is_visible = pci_dev_hp_attrs_are_visible,
> --
> 2.23.0
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-10-16 22:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-16 8:03 [PATCH] [V2] PCI: sysfs: remove pci_bridge_groups and pcie_dev_groups Ben Dooks (Codethink)
2019-10-16 22:17 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).