From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [203.10.76.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mx.ozlabs.org", Issuer "CA Cert Signing Authority" (verified OK)) by bilbo.ozlabs.org (Postfix) with ESMTPS id 1ED81B70AA for ; Mon, 15 Jun 2009 19:43:18 +1000 (EST) Subject: Re: [PATCH][BUILD FAILURE 03/04] Next June 04:PPC64 randconfig [drivers/net/lance.o] From: Benjamin Herrenschmidt To: subrata@linux.vnet.ibm.com In-Reply-To: <1245050509.5239.11.camel@subratamodak.linux.ibm.com> References: <20090611035046.22125.26641.sendpatchset@subratamodak.linux.ibm.com> <1245048698.5239.9.camel@subratamodak.linux.ibm.com> <1245049899.19217.56.camel@pasglop> <1245050509.5239.11.camel@subratamodak.linux.ibm.com> Content-Type: text/plain Date: Mon, 15 Jun 2009 19:43:01 +1000 Message-Id: <1245058981.12400.3.camel@pasglop> Mime-Version: 1.0 Cc: Sachin P Sant , Stephen Rothwell , netdev@vger.kernel.org, Linux-Kernel , Linuxppc-dev , Linux-Next , Paul Mackerras , Balbir Singh List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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.