| Message ID | 1383815155-5318-1-git-send-email-jszhang@marvell.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
Dear Jisheng Zhang, On Thu, 7 Nov 2013 17:05:55 +0800, Jisheng Zhang wrote: > This patch fixes the following sparse warning: > drivers/pci/host/pci-mvebu.c:676:17: warning: symbol > 'mvebu_pcie_align_resource' was not declared. Should it be static? > > Signed-off-by: Jisheng Zhang <jszhang@marvell.com> Thanks, but a similar patch was already merged: https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/drivers/pci/host/pci-mvebu.c?id=f5072dfbac053200c8865c4fb15e4f020b7b5d1d Thanks! Thomas
diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c index 729d5a1..bf4ff23 100644 --- a/drivers/pci/host/pci-mvebu.c +++ b/drivers/pci/host/pci-mvebu.c @@ -673,11 +673,11 @@ static struct pci_bus *mvebu_pcie_scan_bus(int nr, struct pci_sys_data *sys) return bus; } -resource_size_t mvebu_pcie_align_resource(struct pci_dev *dev, - const struct resource *res, - resource_size_t start, - resource_size_t size, - resource_size_t align) +static resource_size_t mvebu_pcie_align_resource(struct pci_dev *dev, + const struct resource *res, + resource_size_t start, + resource_size_t size, + resource_size_t align) { if (dev->bus->number != 0) return start;
This patch fixes the following sparse warning: drivers/pci/host/pci-mvebu.c:676:17: warning: symbol 'mvebu_pcie_align_resource' was not declared. Should it be static? Signed-off-by: Jisheng Zhang <jszhang@marvell.com> --- drivers/pci/host/pci-mvebu.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)