From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from moutng.kundenserver.de ([212.227.17.10]:56912 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932447Ab2JaH1j (ORCPT ); Wed, 31 Oct 2012 03:27:39 -0400 From: Arnd Bergmann To: Pratyush Anand Subject: Re: [PATCH 03/15] SPEAr13xx: Add mach/io.h Date: Tue, 30 Oct 2012 21:45:46 +0000 Cc: viresh kumar , Shiraz HASHIM , "spear-devel" , "linux-pci@vger.kernel.org" , "devicetree-discuss@lists.ozlabs.org" , "olof@lixom.net" References: <508E3A2E.3000204@st.com> In-Reply-To: <508E3A2E.3000204@st.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Message-Id: <201210302145.47011.arnd@arndb.de> Sender: linux-pci-owner@vger.kernel.org List-ID: On Monday 29 October 2012, Pratyush Anand wrote: > Any transaction between 0x80000000 to 0x8FFFFFFF can only be used for > any type(IO/MEM/CFG0/CFG1) of PCIe transaction. Therefore , we need to > have this mapping in io.h. No you don't. The I/O space window will only be in one place on a given system, so you should register that window using pci_ioremap_io() as all the other modern platforms. There is no excuse for having a per-platform mach/io.h any more and that would prevent you from enabling spear in a multiplatform configuration. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 03/15] SPEAr13xx: Add mach/io.h Date: Tue, 30 Oct 2012 21:45:46 +0000 Message-ID: <201210302145.47011.arnd@arndb.de> References: <508E3A2E.3000204@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <508E3A2E.3000204-qxv4g6HH51o@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Pratyush Anand Cc: viresh kumar , "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" , spear-devel , Shiraz HASHIM , "linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On Monday 29 October 2012, Pratyush Anand wrote: > Any transaction between 0x80000000 to 0x8FFFFFFF can only be used for > any type(IO/MEM/CFG0/CFG1) of PCIe transaction. Therefore , we need to > have this mapping in io.h. No you don't. The I/O space window will only be in one place on a given system, so you should register that window using pci_ioremap_io() as all the other modern platforms. There is no excuse for having a per-platform mach/io.h any more and that would prevent you from enabling spear in a multiplatform configuration. Arnd