Message ID | 20161031110409.27300-1-tklauser@distanz.ch (mailing list archive) |
---|---|
State | New, archived |
Delegated to: | Bjorn Helgaas |
Headers | show |
> -----Original Message----- > From: Tobias Klauser [mailto:tklauser@distanz.ch] > Sent: Monday, October 31, 2016 4:04 AM > To: KY Srinivasan <kys@microsoft.com>; Haiyang Zhang > <haiyangz@microsoft.com> > Cc: Bjorn Helgaas <bhelgaas@google.com>; devel@linuxdriverproject.org; > linux-pci@vger.kernel.org > Subject: [PATCH] PCI: hv: Make unnecessarily global IRQ masking functions > static > > Make hv_irq_mask and hv_irq_unmask static as they are only used in > pci-hyperv.c > > This fixes a sparse warning. > > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> Thanks. Acked-by: K. Y. Srinivasan <kys@microsoft.com> > --- > drivers/pci/host/pci-hyperv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c > index 763ff8745828..06c98695c06c 100644 > --- a/drivers/pci/host/pci-hyperv.c > +++ b/drivers/pci/host/pci-hyperv.c > @@ -755,7 +755,7 @@ static int hv_set_affinity(struct irq_data *data, const > struct cpumask *dest, > return parent->chip->irq_set_affinity(parent, dest, force); > } > > -void hv_irq_mask(struct irq_data *data) > +static void hv_irq_mask(struct irq_data *data) > { > pci_msi_mask_irq(data); > } > @@ -770,7 +770,7 @@ void hv_irq_mask(struct irq_data *data) > * is built out of this PCI bus's instance GUID and the function > * number of the device. > */ > -void hv_irq_unmask(struct irq_data *data) > +static void hv_irq_unmask(struct irq_data *data) > { > struct msi_desc *msi_desc = irq_data_get_msi_desc(data); > struct irq_cfg *cfg = irqd_cfg(data); > -- > 2.9.0 > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Oct 31, 2016 at 12:04:09PM +0100, Tobias Klauser wrote: > Make hv_irq_mask and hv_irq_unmask static as they are only used in > pci-hyperv.c > > This fixes a sparse warning. > > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> Applied with KY's ack to pci/host-hv for v4.10, thanks! > --- > drivers/pci/host/pci-hyperv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c > index 763ff8745828..06c98695c06c 100644 > --- a/drivers/pci/host/pci-hyperv.c > +++ b/drivers/pci/host/pci-hyperv.c > @@ -755,7 +755,7 @@ static int hv_set_affinity(struct irq_data *data, const struct cpumask *dest, > return parent->chip->irq_set_affinity(parent, dest, force); > } > > -void hv_irq_mask(struct irq_data *data) > +static void hv_irq_mask(struct irq_data *data) > { > pci_msi_mask_irq(data); > } > @@ -770,7 +770,7 @@ void hv_irq_mask(struct irq_data *data) > * is built out of this PCI bus's instance GUID and the function > * number of the device. > */ > -void hv_irq_unmask(struct irq_data *data) > +static void hv_irq_unmask(struct irq_data *data) > { > struct msi_desc *msi_desc = irq_data_get_msi_desc(data); > struct irq_cfg *cfg = irqd_cfg(data); > -- > 2.9.0 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c index 763ff8745828..06c98695c06c 100644 --- a/drivers/pci/host/pci-hyperv.c +++ b/drivers/pci/host/pci-hyperv.c @@ -755,7 +755,7 @@ static int hv_set_affinity(struct irq_data *data, const struct cpumask *dest, return parent->chip->irq_set_affinity(parent, dest, force); } -void hv_irq_mask(struct irq_data *data) +static void hv_irq_mask(struct irq_data *data) { pci_msi_mask_irq(data); } @@ -770,7 +770,7 @@ void hv_irq_mask(struct irq_data *data) * is built out of this PCI bus's instance GUID and the function * number of the device. */ -void hv_irq_unmask(struct irq_data *data) +static void hv_irq_unmask(struct irq_data *data) { struct msi_desc *msi_desc = irq_data_get_msi_desc(data); struct irq_cfg *cfg = irqd_cfg(data);
Make hv_irq_mask and hv_irq_unmask static as they are only used in pci-hyperv.c This fixes a sparse warning. Signed-off-by: Tobias Klauser <tklauser@distanz.ch> --- drivers/pci/host/pci-hyperv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)