| Message ID | 20210421140334.3847155-1-arnd@kernel.org |
|---|---|
| State | New |
| Headers | show |
| Series | PCI/VPD: fix unused pci_vpd_set_size function warning | expand |
Hi Arnd, > The only user of this function is now in an #ifdef, causing > a warning when that symbol is not defined: > > drivers/pci/vpd.c:289:13: error: 'pci_vpd_set_size' defined but not used [-Werror=unused-function] > 289 | static void pci_vpd_set_size(struct pci_dev *dev, size_t len) > > Move the function into that #ifdef block. [...] Thank you! Reviewed-by: Krzysztof Wilczyński <kw@linux.com> Krzysztof
On Wed, Apr 21, 2021 at 8:56 PM Arnd Bergmann <arnd@kernel.org> wrote: > From: Arnd Bergmann <arnd@arndb.de> > > The only user of this function is now in an #ifdef, causing > a warning when that symbol is not defined: > > drivers/pci/vpd.c:289:13: error: 'pci_vpd_set_size' defined but not used [-Werror=unused-function] > 289 | static void pci_vpd_set_size(struct pci_dev *dev, size_t len) > > Move the function into that #ifdef block. > > Fixes: f349223f076e ("PCI/VPD: Remove pci_set_vpd_size()") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert
diff --git a/drivers/pci/vpd.c b/drivers/pci/vpd.c index 6909253bb13c..ee8c41a88548 100644 --- a/drivers/pci/vpd.c +++ b/drivers/pci/vpd.c @@ -286,17 +286,6 @@ static ssize_t pci_vpd_write(struct pci_dev *dev, loff_t pos, size_t count, return ret ? ret : count; } -static void pci_vpd_set_size(struct pci_dev *dev, size_t len) -{ - struct pci_vpd *vpd = dev->vpd; - - if (!vpd || len == 0 || len > PCI_VPD_MAX_SIZE) - return; - - vpd->valid = 1; - vpd->len = len; -} - static const struct pci_vpd_ops pci_vpd_ops = { .read = pci_vpd_read, .write = pci_vpd_write, @@ -482,6 +471,17 @@ DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_QLOGIC, 0x2261, quirk_blacklist_vpd); DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_AMAZON_ANNAPURNA_LABS, 0x0031, PCI_CLASS_BRIDGE_PCI, 8, quirk_blacklist_vpd); +static void pci_vpd_set_size(struct pci_dev *dev, size_t len) +{ + struct pci_vpd *vpd = dev->vpd; + + if (!vpd || len == 0 || len > PCI_VPD_MAX_SIZE) + return; + + vpd->valid = 1; + vpd->len = len; +} + static void quirk_chelsio_extend_vpd(struct pci_dev *dev) { int chip = (dev->device & 0xf000) >> 12;