On Fri, 2009-05-15 at 15:33 -0700, David Daney wrote:
> Wu Zhangjin wrote:
> >>From 1e6360e89b239699ef1f5344e1d3a5c0b3c5bef1 Mon Sep 17 00:00:00 2001
> > From: Wu Zhangjin <wuzhangjin@gmail.com>
> > Date: Tue, 12 May 2009 10:33:37 +0800
> > Subject: [PATCH 01/30] Fix warning: incompatible argument type of
> > pci_fixup_irqs
> > MIME-Version: 1.0
> > Content-Type: text/plain; charset=utf-8
> > Content-Transfer-Encoding: 8bit
> >
> > arch/mips/pci/pci.c:160: warning: passing argument 2 of pci_fixup_irqs
> > from incompatible pointer type
> >
> > include/linux/pci.h:
> >
> > void pci_fixup_irqs(u8 (*)(struct pci_dev *, u8 *),
> > int (*)(struct pci_dev *, u8, u8));
> >
> > arch/mips/pci/pci.c:160:
> >
> > pci_fixup_irqs(pci_common_swizzle, pcibios_map_irq);
> >
> > arch/mips/include/asm/pci.h:
> >
> > extern int pcibios_map_irq(const struct pci_dev *dev, u8 slot, u8 pin);
> >
> > arch/mips/pci/fixup-malta.c
> >
> > int __init pcibios_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
> > ---
> > arch/mips/include/asm/pci.h | 2 +-
> > arch/mips/pci/fixup-au1000.c | 2 +-
> > arch/mips/pci/fixup-capcella.c | 2 +-
> > arch/mips/pci/fixup-cobalt.c | 2 +-
> > arch/mips/pci/fixup-emma2rh.c | 2 +-
> > arch/mips/pci/fixup-excite.c | 2 +-
> > arch/mips/pci/fixup-ip32.c | 2 +-
> > arch/mips/pci/fixup-lm2e.c | 2 +-
> > arch/mips/pci/fixup-malta.c | 2 +-
> > arch/mips/pci/fixup-mpc30x.c | 2 +-
> > arch/mips/pci/fixup-pmcmsp.c | 2 +-
> > arch/mips/pci/fixup-pnx8550.c | 2 +-
> > arch/mips/pci/fixup-rc32434.c | 2 +-
> > arch/mips/pci/fixup-sni.c | 2 +-
> > arch/mips/pci/fixup-tb0219.c | 2 +-
> > arch/mips/pci/fixup-tb0226.c | 2 +-
> > arch/mips/pci/fixup-tb0287.c | 2 +-
> > arch/mips/pci/fixup-wrppmc.c | 2 +-
> > arch/mips/pci/fixup-yosemite.c | 2 +-
> > arch/mips/pci/pci-bcm1480.c | 2 +-
> > arch/mips/pci/pci-bcm47xx.c | 2 +-
> > arch/mips/pci/pci-ip27.c | 2 +-
> > arch/mips/pci/pci-lasat.c | 2 +-
> > arch/mips/pci/pci-sb1250.c | 2 +-
> > arch/mips/txx9/generic/pci.c | 2 +-
> > 25 files changed, 25 insertions(+), 25 deletions(-)
> >
> [...]
>
> Did you intend that we should do something with all these patches?
>
> They cannot be merged without a Signed-off-by: tag. There may be other
> issues as well. Did you run checkpatch.pl on them? If not, I would
> recommend it. Try to fix all errors flagged by checkpatch.
sorry, forget to check it. I will do it later, thanks!
>
> This is what I get on this patch:
>
> ERROR: patch seems to be corrupt (line wrapped?)
> #130: FILE: arch/mips/include/asm/pci.h:55:
> pci_controller *hose);
>
> ERROR: Missing Signed-off-by: line(s)
>
> total: 2 errors, 0 warnings, 293 lines checked
>
> [PATCH 01_30] Fix warning: incompatible argument type of
> pci_fixup_irqs.eml has style problems, please review. If any of these
> errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
>
>
>
> David Daney
|