Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-10 Thread Lorenzo Pieralisi
On Thu, Apr 06, 2017 at 11:47:37AM +0100, Russell King - ARM Linux wrote: > On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > > On Wed, Apr 05, 2017 at 01:38:12PM +0100, Lorenzo Pieralisi wrote: > > > eg: openrisc > > > > > > >> include/asm-generic/io.h:922:9: error: implicit de

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Russell King - ARM Linux
On Thu, Apr 06, 2017 at 06:09:51PM +0100, Lorenzo Pieralisi wrote: > On Thu, Apr 06, 2017 at 05:40:16PM +0100, Russell King - ARM Linux wrote: > > On Thu, Apr 06, 2017 at 05:21:56PM +0100, Lorenzo Pieralisi wrote: > > > Ok, so: > > > > > > (1) I can do asm-generic/ioremap-nopost.h, which I assume

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Lorenzo Pieralisi
On Thu, Apr 06, 2017 at 05:40:16PM +0100, Russell King - ARM Linux wrote: > On Thu, Apr 06, 2017 at 05:21:56PM +0100, Lorenzo Pieralisi wrote: > > Ok, so: > > > > (1) I can do asm-generic/ioremap-nopost.h, which I assume you want to > > contain something like > > > > static inline void __iome

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Russell King - ARM Linux
On Thu, Apr 06, 2017 at 05:21:56PM +0100, Lorenzo Pieralisi wrote: > Ok, so: > > (1) I can do asm-generic/ioremap-nopost.h, which I assume you want to > contain something like > > static inline void __iomem *ioremap_nopost(phys_addr_t offset, size_t size) > { > return ioremap_nocache(of

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Lorenzo Pieralisi
On Thu, Apr 06, 2017 at 02:07:27PM +0100, Russell King - ARM Linux wrote: > On Thu, Apr 06, 2017 at 01:59:07PM +0200, Luis R. Rodriguez wrote: > > On Thu, Apr 06, 2017 at 12:38:45PM +0100, Lorenzo Pieralisi wrote: > > > Ok but where ? linux/io.h or asm-generic/io.h ? As I replied to Russell, > > >

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Russell King - ARM Linux
On Thu, Apr 06, 2017 at 01:59:07PM +0200, Luis R. Rodriguez wrote: > On Thu, Apr 06, 2017 at 12:38:45PM +0100, Lorenzo Pieralisi wrote: > > Ok but where ? linux/io.h or asm-generic/io.h ? As I replied to Russell, > > having it in linux/io.h is a bit odd given that it would be the only > > ioremap_*

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Luis R. Rodriguez
On Thu, Apr 06, 2017 at 01:11:57PM +0100, Russell King - ARM Linux wrote: > On Thu, Apr 06, 2017 at 12:53:12PM +0200, Luis R. Rodriguez wrote: > > On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > > > Indeed, the static inline ioremap_nocache() fallback does not work > > > on all

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Russell King - ARM Linux
On Thu, Apr 06, 2017 at 12:53:12PM +0200, Luis R. Rodriguez wrote: > On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > > Indeed, the static inline ioremap_nocache() fallback does not work > > on all arches (whether I add the fallback in linux/io.h or > > asm-generic/io.h is irrel

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Luis R. Rodriguez
On Thu, Apr 06, 2017 at 12:38:45PM +0100, Lorenzo Pieralisi wrote: > On Thu, Apr 06, 2017 at 12:53:12PM +0200, Luis R. Rodriguez wrote: > > On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > > > On Wed, Apr 05, 2017 at 01:38:12PM +0100, Lorenzo Pieralisi wrote: > > > > On Wed, Apr

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Lorenzo Pieralisi
On Thu, Apr 06, 2017 at 12:53:12PM +0200, Luis R. Rodriguez wrote: > On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > > On Wed, Apr 05, 2017 at 01:38:12PM +0100, Lorenzo Pieralisi wrote: > > > On Wed, Apr 05, 2017 at 11:58:41AM +0100, Russell King - ARM Linux wrote: > > > > On T

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Luis R. Rodriguez
On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > On Wed, Apr 05, 2017 at 01:38:12PM +0100, Lorenzo Pieralisi wrote: > > On Wed, Apr 05, 2017 at 11:58:41AM +0100, Russell King - ARM Linux wrote: > > > On Tue, Mar 28, 2017 at 03:45:43PM +0100, Lorenzo Pieralisi wrote: > > > > On M

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Russell King - ARM Linux
On Thu, Apr 06, 2017 at 11:26:36AM +0100, Lorenzo Pieralisi wrote: > On Wed, Apr 05, 2017 at 01:38:12PM +0100, Lorenzo Pieralisi wrote: > > eg: openrisc > > > > >> include/asm-generic/io.h:922:9: error: implicit declaration of > > >> function 'ioremap_nocache' [-Werror=implicit-function-declaratio

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-06 Thread Lorenzo Pieralisi
On Wed, Apr 05, 2017 at 01:38:12PM +0100, Lorenzo Pieralisi wrote: > On Wed, Apr 05, 2017 at 11:58:41AM +0100, Russell King - ARM Linux wrote: > > On Tue, Mar 28, 2017 at 03:45:43PM +0100, Lorenzo Pieralisi wrote: > > > On Mon, Mar 27, 2017 at 08:41:10PM -0500, Bjorn Helgaas wrote: > > > > On Mon,

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-05 Thread Lorenzo Pieralisi
On Wed, Apr 05, 2017 at 11:58:41AM +0100, Russell King - ARM Linux wrote: > On Tue, Mar 28, 2017 at 03:45:43PM +0100, Lorenzo Pieralisi wrote: > > On Mon, Mar 27, 2017 at 08:41:10PM -0500, Bjorn Helgaas wrote: > > > On Mon, Mar 27, 2017 at 10:49:30AM +0100, Lorenzo Pieralisi wrote: > > > > diff --g

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-04-05 Thread Russell King - ARM Linux
On Tue, Mar 28, 2017 at 03:45:43PM +0100, Lorenzo Pieralisi wrote: > On Mon, Mar 27, 2017 at 08:41:10PM -0500, Bjorn Helgaas wrote: > > On Mon, Mar 27, 2017 at 10:49:30AM +0100, Lorenzo Pieralisi wrote: > > > diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h > > > index 7afb0e2..50

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-03-30 Thread Bjorn Helgaas
On Tue, Mar 28, 2017 at 03:45:43PM +0100, Lorenzo Pieralisi wrote: > On Mon, Mar 27, 2017 at 08:41:10PM -0500, Bjorn Helgaas wrote: > > On Mon, Mar 27, 2017 at 10:49:30AM +0100, Lorenzo Pieralisi wrote: > > > #define ioremap_uc ioremap_uc > > > +#define ioremap_nopost ioremap_nocache > > > > The

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-03-28 Thread Lorenzo Pieralisi
On Mon, Mar 27, 2017 at 08:41:10PM -0500, Bjorn Helgaas wrote: > On Mon, Mar 27, 2017 at 10:49:30AM +0100, Lorenzo Pieralisi wrote: > > The PCI specifications (Rev 3.0, 3.2.5 "Transaction Ordering and > > Posting") mandate non-posted configuration transactions. As further > > highlighted in the PCI

Re: [PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-03-27 Thread Bjorn Helgaas
On Mon, Mar 27, 2017 at 10:49:30AM +0100, Lorenzo Pieralisi wrote: > The PCI specifications (Rev 3.0, 3.2.5 "Transaction Ordering and > Posting") mandate non-posted configuration transactions. As further > highlighted in the PCIe specifications (4.0 - Rev0.3, "Ordering > Considerations for the Enha

[PATCH v2 02/22] asm-generic/io.h: add ioremap_nopost remap interface

2017-03-27 Thread Lorenzo Pieralisi
The PCI specifications (Rev 3.0, 3.2.5 "Transaction Ordering and Posting") mandate non-posted configuration transactions. As further highlighted in the PCIe specifications (4.0 - Rev0.3, "Ordering Considerations for the Enhanced Configuration Access Mechanism"), through ECAM and ECAM-derivative con