From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Subject: Re: [PATCH v2 1/7] PCI: dwc: designware: Add new *ops* for cpu addr fixup To: Kishon Vijay Abraham I , Bjorn Helgaas , Joao Pinto , , , , , , References: <1488880372-7390-1-git-send-email-kishon@ti.com> <1488880372-7390-2-git-send-email-kishon@ti.com> From: Joao Pinto Message-ID: <2372241a-759c-e521-6d65-5a762ab159c4@synopsys.com> Date: Tue, 7 Mar 2017 11:11:38 +0000 MIME-Version: 1.0 In-Reply-To: <1488880372-7390-2-git-send-email-kishon@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: nsekhar@ti.com Content-Type: text/plain; charset="windows-1252" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+bjorn=helgaas.com@lists.infradead.org List-ID: =C0s 9:52 AM de 3/7/2017, Kishon Vijay Abraham I escreveu: > Some platforms (like dra7xx) require only the least 28 bits of the > corresponding 32 bit CPU address to be programmed in the address > translation unit. This modified address is stored in io_base/mem_base/ > cfg0_base/cfg1_base in dra7xx_pcie_host_init. While this is okay for > host mode where the address range is fixed, device mode requires > different addresses to be programmed based on the host buffer address. > Add a new ops to get the least 28 bits of the corresponding 32 bit > CPU address and invoke it before programming the address translation > unit. > = > Signed-off-by: Kishon Vijay Abraham I > --- > drivers/pci/dwc/pcie-designware.c | 3 +++ > drivers/pci/dwc/pcie-designware.h | 1 + > 2 files changed, 4 insertions(+) > = > diff --git a/drivers/pci/dwc/pcie-designware.c b/drivers/pci/dwc/pcie-des= ignware.c > index 7e1fb7d..14ee7a3 100644 > --- a/drivers/pci/dwc/pcie-designware.c > +++ b/drivers/pci/dwc/pcie-designware.c > @@ -97,6 +97,9 @@ void dw_pcie_prog_outbound_atu(struct dw_pcie *pci, int= index, int type, > { > u32 retries, val; > = > + if (pp->ops->cpu_addr_fixup) > + cpu_addr =3D pp->ops->cpu_addr_fixup(cpu_addr); > + > if (pci->iatu_unroll_enabled) { > dw_pcie_writel_unroll(pci, index, PCIE_ATU_UNR_LOWER_BASE, > lower_32_bits(cpu_addr)); > diff --git a/drivers/pci/dwc/pcie-designware.h b/drivers/pci/dwc/pcie-des= ignware.h > index cd3b871..8f3dcb2 100644 > --- a/drivers/pci/dwc/pcie-designware.h > +++ b/drivers/pci/dwc/pcie-designware.h > @@ -143,6 +143,7 @@ struct pcie_port { > }; > = > struct dw_pcie_ops { > + u64 (*cpu_addr_fixup)(u64 cpu_addr); > u32 (*readl_dbi)(struct dw_pcie *pcie, u32 reg); > void (*writel_dbi)(struct dw_pcie *pcie, u32 reg, u32 val); > int (*link_up)(struct dw_pcie *pcie); > = Looks good. Acked-by: Joao Pinto _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel