From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v7 1/6] pci: Introduce pci_register_io_range() helper function. Date: Mon, 07 Apr 2014 13:16:56 +0200 Message-ID: <4164782.o9VMXFGud6@wuerfel> References: <1394811272-1547-1-git-send-email-Liviu.Dudau@arm.com> <20140407083550.GF17163@e106497-lin.cambridge.arm.com> <1396862008.3671.39.camel@pasglop> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1396862008.3671.39.camel@pasglop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: linux-arm-kernel@lists.infradead.org Cc: "devicetree@vger.kernel.org" , linaro-kernel , Catalin Marinas , linux-pci , Liviu Dudau , LKML , Will Deacon , Grant Likely , Tanmay Inamdar , Benjamin Herrenschmidt , Bjorn Helgaas List-Id: devicetree@vger.kernel.org On Monday 07 April 2014 19:13:28 Benjamin Herrenschmidt wrote: > On Mon, 2014-04-07 at 09:35 +0100, Liviu Dudau wrote: > > Thanks for the summary, is really useful as I was recently looking > > into code in that > > area. One thing I was trying to understand is why ppc needs _IO_BASE > > at all rather > > than using the generic PCI_IOBASE? > > Perhaps because our code predates it ? I haven't looked much into > the semantics of PCI_IOBASE yet... Yes, I'm pretty sure that's all there is to it. PCI_IOBASE just happened to be an identifier we picked for asm-generic, but the use on PowerPC is much older than the generic file. Arnd