Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1232027 > unrolled thread
| Started by | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| First post | 2015-09-24 12:20 +0200 |
| Last post | 2015-09-25 21:40 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] net: via/Kconfig: GENERIC_PCI_IOMAP required if PCI not selected Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-24 12:20 +0200
Re: [PATCH] net: via/Kconfig: GENERIC_PCI_IOMAP required if PCI not selected Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> - 2015-09-25 20:10 +0200
Re: [PATCH] net: via/Kconfig: GENERIC_PCI_IOMAP required if PCI not selected David Miller <davem@davemloft.net> - 2015-09-25 21:40 +0200
| From | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| Date | 2015-09-24 12:20 +0200 |
| Subject | [PATCH] net: via/Kconfig: GENERIC_PCI_IOMAP required if PCI not selected |
| Message-ID | <qcbN8-8v1-15@gated-at.bofh.it> |
The builds of allmodconfig of avr32 is failing with: drivers/net/ethernet/via/via-rhine.c:1098:2: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration] drivers/net/ethernet/via/via-rhine.c:1119:2: error: implicit declaration of function 'pci_iounmap' [-Werror=implicit-function-declaration] The generic empty pci_iomap and pci_iounmap is used only if CONFIG_PCI is not defined and CONFIG_GENERIC_PCI_IOMAP is defined. Add GENERIC_PCI_IOMAP in the dependency list for VIA_RHINE as we are getting build failure when CONFIG_PCI and CONFIG_GENERIC_PCI_IOMAP both are not defined. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> --- Failed build log is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/81127188 An exactly similar patch has been posted earlier and relevant discussion is at: https://patchwork.ozlabs.org/patch/498722/ I posted a patch to include empty pci_iomap, pci_iounmap in avr32. That discussion is at: https://lkml.org/lkml/2015/9/19/100 drivers/net/ethernet/via/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/via/Kconfig b/drivers/net/ethernet/via/Kconfig index 2f1264b..17831e5 100644 --- a/drivers/net/ethernet/via/Kconfig +++ b/drivers/net/ethernet/via/Kconfig @@ -17,7 +17,7 @@ if NET_VENDOR_VIA config VIA_RHINE tristate "VIA Rhine support" - depends on (PCI || OF_IRQ) + depends on (PCI || (OF_IRQ && GENERIC_PCI_IOMAP)) depends on HAS_DMA select CRC32 select MII -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> |
|---|---|
| Date | 2015-09-25 20:10 +0200 |
| Subject | Re: [PATCH] net: via/Kconfig: GENERIC_PCI_IOMAP required if PCI not selected |
| Message-ID | <qcFBw-zc-9@gated-at.bofh.it> |
| In reply to | #1232027 |
Hello. On 9/24/2015 1:16 PM, Sudip Mukherjee wrote: > The builds of allmodconfig of avr32 is failing with: > > drivers/net/ethernet/via/via-rhine.c:1098:2: error: implicit declaration > of function 'pci_iomap' [-Werror=implicit-function-declaration] > drivers/net/ethernet/via/via-rhine.c:1119:2: error: implicit declaration > of function 'pci_iounmap' [-Werror=implicit-function-declaration] > > The generic empty pci_iomap and pci_iounmap is used only if CONFIG_PCI > is not defined and CONFIG_GENERIC_PCI_IOMAP is defined. > > Add GENERIC_PCI_IOMAP in the dependency list for VIA_RHINE as we are > getting build failure when CONFIG_PCI and CONFIG_GENERIC_PCI_IOMAP both > are not defined. > > Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> > --- > > Failed build log is at: > https://travis-ci.org/sudipm-mukherjee/parport/jobs/81127188 > > An exactly similar patch has been posted earlier and relevant discussion > is at: > https://patchwork.ozlabs.org/patch/498722/ > > I posted a patch to include empty pci_iomap, pci_iounmap in avr32. That > discussion is at: > https://lkml.org/lkml/2015/9/19/100 > > drivers/net/ethernet/via/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/via/Kconfig b/drivers/net/ethernet/via/Kconfig > index 2f1264b..17831e5 100644 > --- a/drivers/net/ethernet/via/Kconfig > +++ b/drivers/net/ethernet/via/Kconfig > @@ -17,7 +17,7 @@ if NET_VENDOR_VIA > > config VIA_RHINE > tristate "VIA Rhine support" > - depends on (PCI || OF_IRQ) > + depends on (PCI || (OF_IRQ && GENERIC_PCI_IOMAP)) Could you remove outer braces, while at it? [...] MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | David Miller <davem@davemloft.net> |
|---|---|
| Date | 2015-09-25 21:40 +0200 |
| Subject | Re: [PATCH] net: via/Kconfig: GENERIC_PCI_IOMAP required if PCI not selected |
| Message-ID | <qcH0C-2u4-27@gated-at.bofh.it> |
| In reply to | #1232993 |
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Date: Fri, 25 Sep 2015 21:03:52 +0300 > Hello. > > On 9/24/2015 1:16 PM, Sudip Mukherjee wrote: > >> The builds of allmodconfig of avr32 is failing with: >> >> drivers/net/ethernet/via/via-rhine.c:1098:2: error: implicit >> declaration >> of function 'pci_iomap' [-Werror=implicit-function-declaration] >> drivers/net/ethernet/via/via-rhine.c:1119:2: error: implicit >> declaration >> of function 'pci_iounmap' [-Werror=implicit-function-declaration] >> >> The generic empty pci_iomap and pci_iounmap is used only if CONFIG_PCI >> is not defined and CONFIG_GENERIC_PCI_IOMAP is defined. >> >> Add GENERIC_PCI_IOMAP in the dependency list for VIA_RHINE as we are >> getting build failure when CONFIG_PCI and CONFIG_GENERIC_PCI_IOMAP >> both >> are not defined. >> >> Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> ... >> @@ -17,7 +17,7 @@ if NET_VENDOR_VIA >> >> config VIA_RHINE >> tristate "VIA Rhine support" >> - depends on (PCI || OF_IRQ) >> + depends on (PCI || (OF_IRQ && GENERIC_PCI_IOMAP)) > > Could you remove outer braces, while at it? I took care of this while applying this patch, thanks everyone. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web