Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753222AbaBCTSn (ORCPT ); Mon, 3 Feb 2014 14:18:43 -0500 Received: from service87.mimecast.com ([91.220.42.44]:45929 "EHLO service87.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752559AbaBCTSl convert rfc822-to-8bit (ORCPT ); Mon, 3 Feb 2014 14:18:41 -0500 Date: Mon, 3 Feb 2014 19:18:38 +0000 From: Liviu Dudau To: Arnd Bergmann Cc: linux-pci , Bjorn Helgaas , Catalin Marinas , Will Deacon , LKML , "devicetree@vger.kernel.org" , LAKML , linaro-kernel Subject: Re: [PATCH] arm64: Add architecture support for PCI Message-ID: <20140203191837.GC4889@e106497-lin.cambridge.arm.com> Mail-Followup-To: Arnd Bergmann , linux-pci , Bjorn Helgaas , Catalin Marinas , Will Deacon , LKML , "devicetree@vger.kernel.org" , LAKML , linaro-kernel References: <1391453028-23191-1-git-send-email-Liviu.Dudau@arm.com> <1391453028-23191-2-git-send-email-Liviu.Dudau@arm.com> <21596846.kVTqp7roW4@wuerfel> MIME-Version: 1.0 In-Reply-To: <21596846.kVTqp7roW4@wuerfel> User-Agent: Mutt/1.5.22 (2013-10-16) X-OriginalArrivalTime: 03 Feb 2014 19:18:38.0331 (UTC) FILETIME=[BDBBFCB0:01CF2114] X-MC-Unique: 114020319183800301 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 03, 2014 at 06:58:56PM +0000, Arnd Bergmann wrote: > On Monday 03 February 2014 18:43:48 Liviu Dudau wrote: > > diff --git a/arch/arm64/include/asm/io.h b/arch/arm64/include/asm/io.h > > index 4cc813e..ce5bad2 100644 > > --- a/arch/arm64/include/asm/io.h > > +++ b/arch/arm64/include/asm/io.h > > @@ -120,9 +120,13 @@ static inline u64 __raw_readq(const volatile void __iomem *addr) > > /* > > * I/O port access primitives. > > */ > > +#define arch_has_dev_port() (0) > > Why not? Maybe I got it the wrong way around, but the comment in include/linux/io.h says: /* * Some systems do not have legacy ISA devices. * /dev/port is not a valid interface on these systems. * So for those archs, should define the following symbol. */ So ... defining it should mean no legacy ISA devices, right? > > > #define IO_SPACE_LIMIT 0xffff > > You probably want to increase this a bit, to allow multiple host bridges > to have their own I/O space. OK, but to what size? > > > #define PCI_IOBASE ((void __iomem *)(MODULES_VADDR - SZ_2M)) > > And modify this location: There is no particular reason to have the I/O space > mapped exactly 2MB below the loadable modules, as virtual address space is > essentially free. Will talk with Catalin about where to place this. > > > +#define ioport_map(port, nr) (PCI_IOBASE + ((port) & IO_SPACE_LIMIT)) > > +#define ioport_unmap(addr) > > inline functions? Will do, thanks! > > > static inline u8 inb(unsigned long addr) > > { > > return readb(addr + PCI_IOBASE); > > diff --git a/arch/arm64/include/asm/pci.h b/arch/arm64/include/asm/pci.h > > new file mode 100644 > > index 0000000..dd084a3 > > --- /dev/null > > +++ b/arch/arm64/include/asm/pci.h > > @@ -0,0 +1,35 @@ > > +#ifndef __ASM_PCI_H > > +#define __ASM_PCI_H > > +#ifdef __KERNEL__ > > + > > +#include > > +#include > > +#include > > + > > +#include > > +#include > > +#include > > + > > +#define PCIBIOS_MIN_IO 0 > > +#define PCIBIOS_MIN_MEM 0 > > PCIBIOS_MIN_IO is normally set to 0x1000, to stay out of the ISA range. :) No ISA support! (Die ISA, die!!) > > > diff --git a/arch/arm64/kernel/pci.c b/arch/arm64/kernel/pci.c > > new file mode 100644 > > index 0000000..7b652cf > > --- /dev/null > > +++ b/arch/arm64/kernel/pci.c > > @@ -0,0 +1,112 @@ > > None of this looks really arm64 specific, nor should it be. I think > we should try a little harder to move this as a default implementation > into common code, even if we start out by having all architectures > override it. Agree. This is the RFC version. I didn't dare to post a patch with fixes for all architectures. :) > > > +int pci_ioremap_io(unsigned int offset, phys_addr_t phys_addr) > > +{ > > + BUG_ON(offset + SZ_64K - 1 > IO_SPACE_LIMIT); > > + > > + return ioremap_page_range((unsigned long)PCI_IOBASE + offset, > > + (unsigned long)PCI_IOBASE + offset + SZ_64K, > > + phys_addr, > > + __pgprot(PROT_DEVICE_nGnRE)); > > +} > > Not sure if we want to treat this one as architecture specific though. > It certainly won't be portable to x86, but it could be shared with > a couple of others. We may also want to redesign the interface. > I've been thinking we could make this function allocate space in the > Linux virtual I/O space aperture, and pass two resources into it > (physical I/O aperture and bus I/O range), and get the actual > io_offset as the return value, or a negative error number. Not sure I completely follow your idea. > > That way, you could have an arbitrary number of host bridges in the > system and each one gets a share of the virtual aperture until > it's full. One still needs to fix the pci_request_region use that checks against ioport_resource. But it is an interesting idea. > > Arnd > > Thanks for reviewing this patch! Liviu -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ -- 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/