Message ID | 20090611035046.22125.26641.sendpatchset@subratamodak.linux.ibm.com |
---|---|
State | Not Applicable, archived |
Delegated to: | David Miller |
Headers | show |
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 > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 > > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 > > > > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- 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