On 03/10/2017 12:36 PM, Kishon Vijay Abraham I wrote:

Hi,

On Thursday 09 March 2017 08:35 PM, Niklas Cassel wrote:

quoted

On 03/09/2017 03:48 PM, Niklas Cassel wrote:

quoted

On 03/09/2017 07:39 AM, Kishon Vijay Abraham I wrote:

quoted

dwc has 2 dbi address space labeled dbics and dbics2. The existing
helper to access dbi address space can access only dbics. However
dbics2 has to be accessed for programming the BAR registers in the
case of EP mode. This is in preparation for adding EP mode support
to dwc driver.

Hello Kishon

I don't really like the idea of adding an extra argument to every existing read/write.
Will not a read/write using dbi2 be quite uncommon compared to a read/write
using dbi?

How about something like this:

void __dw_pcie_writel(struct dw_pcie *pci, void __iomem *base, u32 reg, u32 val)
{
    if (pci->ops->writel_dbi)
        pci->ops->writel_dbi(pci, base, reg, val);x?W?R