* [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o]
@ 2009-06-11 3:50 Subrata Modak
2009-06-15 6:51 ` Subrata Modak
0 siblings, 1 reply; 5+ messages in thread
From: Subrata Modak @ 2009-06-11 3:50 UTC (permalink / raw)
To: netdev, Paul Mackerras, Benjamin Herrenschmidt, Linuxppc-dev
Cc: Sachin P Sant, Stephen Rothwell, Linux-Kernel, Linux-Next,
Subrata Modak, Balbir Singh
Hi Benjamin/Paul,
>On Thu, 2009-06-04 at 19:02 +0530, Subrata Modak wrote:
>CC drivers/net/lance.o
> drivers/net/lance.c: In function 'lance_probe1':
> drivers/net/lance.c:575: error: implicit declaration of function 'isa_virt_to_bus'
> drivers/net/lance.c: In function 'lance_rx':
> drivers/net/lance.c:1197: error: implicit declaration of function 'isa_bus_to_virt'
> make[2]: *** [drivers/net/lance.o] Error 1
> make[1]: *** [drivers/net] Error 2
> make: *** [drivers] Error 2
Reference: http://lkml.org/lkml/2009/6/4/240,
To fix the following build error:
drivers/net/lance.c: In function 'lance_probe1':
drivers/net/lance.c:575: error: implicit declaration of function 'isa_virt_to_bus'
drivers/net/lance.c: In function 'lance_rx':
drivers/net/lance.c:1197: error: implicit declaration of function 'isa_bus_to_virt'
make[2]: *** [drivers/net/lance.o] Error 1
make[1]: *** [drivers/net] Error 2
make: *** [drivers] Error 2
I would like to propose the following patch. The prototypes for the functions:
'isa_virt_to_bus' & 'isa_virt_to_bus' are existing for some archs like the
mips, x86, parisc, arm & alpha, but, is missing for powerpc. Is it safe to
introduce the following soultion for powerpc ? It fixes the build problem
i reported earlier.
Signed-off-by: Subrata Modak <subrata@linux.vnet.ibm.com>
---
--- linux-2.6.30-rc8/arch/powerpc/include/asm/io.h.orig 2009-06-10 21:56:49.000000000 -0500
+++ linux-2.6.30-rc8/arch/powerpc/include/asm/io.h 2009-06-10 22:21:35.000000000 -0500
@@ -680,6 +680,9 @@ extern void __iounmap_at(void *ea, unsig
#define mmio_outsw(addr, src, count) writesw(addr, src, count)
#define mmio_outsl(addr, src, count) writesl(addr, src, count)
+#define isa_virt_to_bus virt_to_phys
+#define isa_bus_to_virt phys_to_virt
+
/**
* virt_to_phys - map virtual addresses to physical
* @address: address to remap
---
Regards--
Subrata
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o]
2009-06-11 3:50 [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o] Subrata Modak
@ 2009-06-15 6:51 ` Subrata Modak
2009-06-15 7:11 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 5+ messages in thread
From: Subrata Modak @ 2009-06-15 6:51 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Sachin P Sant, Stephen Rothwell, netdev, Linux-Kernel,
Linuxppc-dev, Linux-Next, Paul Mackerras, Balbir Singh
On Thu, 2009-06-11 at 09:20 +0530, Subrata Modak wrote:
> Hi Benjamin/Paul,
>
> >On Thu, 2009-06-04 at 19:02 +0530, Subrata Modak wrote:
> >CC drivers/net/lance.o
> > drivers/net/lance.c: In function 'lance_probe1':
> > drivers/net/lance.c:575: error: implicit declaration of function 'isa_virt_to_bus'
> > drivers/net/lance.c: In function 'lance_rx':
> > drivers/net/lance.c:1197: error: implicit declaration of function 'isa_bus_to_virt'
> > make[2]: *** [drivers/net/lance.o] Error 1
> > make[1]: *** [drivers/net] Error 2
> > make: *** [drivers] Error 2
>
> Reference: http://lkml.org/lkml/2009/6/4/240,
> To fix the following build error:
>
> drivers/net/lance.c: In function 'lance_probe1':
> drivers/net/lance.c:575: error: implicit declaration of function 'isa_virt_to_bus'
> drivers/net/lance.c: In function 'lance_rx':
> drivers/net/lance.c:1197: error: implicit declaration of function 'isa_bus_to_virt'
> make[2]: *** [drivers/net/lance.o] Error 1
> make[1]: *** [drivers/net] Error 2
> make: *** [drivers] Error 2
>
> I would like to propose the following patch. The prototypes for the functions:
> 'isa_virt_to_bus' & 'isa_virt_to_bus' are existing for some archs like the
> mips, x86, parisc, arm & alpha, but, is missing for powerpc. Is it safe to
> introduce the following soultion for powerpc ? It fixes the build problem
> i reported earlier.
>
> Signed-off-by: Subrata Modak <subrata@linux.vnet.ibm.com>
> ---
Benjamin,
I am not sure whether you liked the following patch to solve the above
problem. Do, you want me address some other issue(s)/fixes for this ?
Regards--
Subrata
>
> --- linux-2.6.30-rc8/arch/powerpc/include/asm/io.h.orig 2009-06-10 21:56:49.000000000 -0500
> +++ linux-2.6.30-rc8/arch/powerpc/include/asm/io.h 2009-06-10 22:21:35.000000000 -0500
> @@ -680,6 +680,9 @@ extern void __iounmap_at(void *ea, unsig
> #define mmio_outsw(addr, src, count) writesw(addr, src, count)
> #define mmio_outsl(addr, src, count) writesl(addr, src, count)
>
> +#define isa_virt_to_bus virt_to_phys
> +#define isa_bus_to_virt phys_to_virt
> +
> /**
> * virt_to_phys - map virtual addresses to physical
> * @address: address to remap
>
> ---
> Regards--
> Subrata
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o]
2009-06-15 6:51 ` Subrata Modak
@ 2009-06-15 7:11 ` Benjamin Herrenschmidt
2009-06-15 7:21 ` Subrata Modak
0 siblings, 1 reply; 5+ messages in thread
From: Benjamin Herrenschmidt @ 2009-06-15 7:11 UTC (permalink / raw)
To: subrata
Cc: Sachin P Sant, Stephen Rothwell, netdev, Linux-Kernel,
Linuxppc-dev, Linux-Next, Paul Mackerras, Balbir Singh
On Mon, 2009-06-15 at 12:21 +0530, Subrata Modak wrote:
> Benjamin,
>
> I am not sure whether you liked the following patch to solve the above
> problem. Do, you want me address some other issue(s)/fixes for this ?
I suppose ... in fact some old PReP machines can even have ISA slots
iirc, but it's not going to do good with machines that have an iommu...
IE. Nobody should use that crap, period.
Cheers,
Ben.
Cheers,
Ben.
> Regards--
> Subrata
>
> >
> > --- linux-2.6.30-rc8/arch/powerpc/include/asm/io.h.orig 2009-06-10 21:56:49.000000000 -0500
> > +++ linux-2.6.30-rc8/arch/powerpc/include/asm/io.h 2009-06-10 22:21:35.000000000 -0500
> > @@ -680,6 +680,9 @@ extern void __iounmap_at(void *ea, unsig
> > #define mmio_outsw(addr, src, count) writesw(addr, src, count)
> > #define mmio_outsl(addr, src, count) writesl(addr, src, count)
> >
> > +#define isa_virt_to_bus virt_to_phys
> > +#define isa_bus_to_virt phys_to_virt
> > +
> > /**
> > * virt_to_phys - map virtual addresses to physical
> > * @address: address to remap
> >
> > ---
> > Regards--
> > Subrata
> >
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o]
2009-06-15 7:11 ` Benjamin Herrenschmidt
@ 2009-06-15 7:21 ` Subrata Modak
2009-06-15 9:43 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 5+ messages in thread
From: Subrata Modak @ 2009-06-15 7:21 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Sachin P Sant, Stephen Rothwell, netdev, Linux-Kernel,
Linuxppc-dev, Linux-Next, Paul Mackerras, Balbir Singh
On Mon, 2009-06-15 at 17:11 +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2009-06-15 at 12:21 +0530, Subrata Modak wrote:
>
> > Benjamin,
> >
> > I am not sure whether you liked the following patch to solve the above
> > problem. Do, you want me address some other issue(s)/fixes for this ?
>
> I suppose ... in fact some old PReP machines can even have ISA slots
> iirc, but it's not going to do good with machines that have an iommu...
>
> IE. Nobody should use that crap, period.
Great,
Can you please make some changes at drivers/net/Kconfig so that this guy
does not build for the PPC64 systems ?
Regards--
Subrata
>
> Cheers,
> Ben.
>
> Cheers,
> Ben.
>
> > Regards--
> > Subrata
> >
> > >
> > > --- linux-2.6.30-rc8/arch/powerpc/include/asm/io.h.orig 2009-06-10 21:56:49.000000000 -0500
> > > +++ linux-2.6.30-rc8/arch/powerpc/include/asm/io.h 2009-06-10 22:21:35.000000000 -0500
> > > @@ -680,6 +680,9 @@ extern void __iounmap_at(void *ea, unsig
> > > #define mmio_outsw(addr, src, count) writesw(addr, src, count)
> > > #define mmio_outsl(addr, src, count) writesl(addr, src, count)
> > >
> > > +#define isa_virt_to_bus virt_to_phys
> > > +#define isa_bus_to_virt phys_to_virt
> > > +
> > > /**
> > > * virt_to_phys - map virtual addresses to physical
> > > * @address: address to remap
> > >
> > > ---
> > > Regards--
> > > Subrata
> > >
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o]
2009-06-15 7:21 ` Subrata Modak
@ 2009-06-15 9:43 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 5+ messages in thread
From: Benjamin Herrenschmidt @ 2009-06-15 9:43 UTC (permalink / raw)
To: subrata
Cc: Sachin P Sant, Stephen Rothwell, netdev, Linux-Kernel,
Linuxppc-dev, Linux-Next, Paul Mackerras, Balbir Singh
On Mon, 2009-06-15 at 12:51 +0530, Subrata Modak wrote:
> On Mon, 2009-06-15 at 17:11 +1000, Benjamin Herrenschmidt wrote:
> > On Mon, 2009-06-15 at 12:21 +0530, Subrata Modak wrote:
> >
> > > Benjamin,
> > >
> > > I am not sure whether you liked the following patch to solve the above
> > > problem. Do, you want me address some other issue(s)/fixes for this ?
> >
> > I suppose ... in fact some old PReP machines can even have ISA slots
> > iirc, but it's not going to do good with machines that have an iommu...
> >
> > IE. Nobody should use that crap, period.
>
> Great,
>
> Can you please make some changes at drivers/net/Kconfig so that this guy
> does not build for the PPC64 systems ?
When I said "that crap" I was talking about those interfaces which I
-hope- are deprecated :-)
I suspect we never really sorted out the issue of DMA mapping for old
legacy ISA stuff... pci_map_* should work to some extent, though you
have to pass it the ISA bridge pci_dev.
Maybe it's time for somebody to start exposing a -proper- abstraction
accross all platforms for that :-)
Ben.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-06-15 9:43 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-06-11 3:50 [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o] Subrata Modak
2009-06-15 6:51 ` Subrata Modak
2009-06-15 7:11 ` Benjamin Herrenschmidt
2009-06-15 7:21 ` Subrata Modak
2009-06-15 9:43 ` Benjamin Herrenschmidt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).