The driver uses the name elbi for application logic registers which is a wrong nomenclature. This patch fixes the same. This commit shouldn't be applied without changes "arm64: dts: exynos: Rename the term elbi to appl" and "dt-bindings: PCI: Rename the term elbi to appl" Signed-off-by: Shradha Todi <shradha.t@xxxxxxxxxxx> --- drivers/pci/controller/dwc/pci-samsung.c | 80 ++++++++++++------------ 1 file changed, 40 insertions(+), 40 deletions(-) diff --git a/drivers/pci/controller/dwc/pci-samsung.c b/drivers/pci/controller/dwc/pci-samsung.c index 6c07d3f151be..d5adf1017a05 100644 --- a/drivers/pci/controller/dwc/pci-samsung.c +++ b/drivers/pci/controller/dwc/pci-samsung.c @@ -25,7 +25,7 @@ #define to_exynos_pcie(x) dev_get_drvdata((x)->dev) -/* PCIe ELBI registers */ +/* PCIe APPL registers */ #define EXYNOS_PCIE_IRQ_PULSE 0x000 #define EXYNOS_IRQ_INTA_ASSERT BIT(0) #define EXYNOS_IRQ_INTB_ASSERT BIT(2) @@ -44,16 +44,16 @@ #define EXYNOS_PCIE_NONSTICKY_RESET 0x024 #define EXYNOS_PCIE_APP_INIT_RESET 0x028 #define EXYNOS_PCIE_APP_LTSSM_ENABLE 0x02c -#define EXYNOS_PCIE_ELBI_RDLH_LINKUP 0x074 -#define EXYNOS_PCIE_ELBI_XMLH_LINKUP BIT(4) -#define EXYNOS_PCIE_ELBI_LTSSM_ENABLE 0x1 -#define EXYNOS_PCIE_ELBI_SLV_AWMISC 0x11c -#define EXYNOS_PCIE_ELBI_SLV_ARMISC 0x120 -#define EXYNOS_PCIE_ELBI_SLV_DBI_ENABLE BIT(21) +#define EXYNOS_PCIE_APPL_RDLH_LINKUP 0x074 +#define EXYNOS_PCIE_APPL_XMLH_LINKUP BIT(4) +#define EXYNOS_PCIE_APPL_LTSSM_ENABLE 0x1 +#define EXYNOS_PCIE_APPL_SLV_AWMISC 0x11c +#define EXYNOS_PCIE_APPL_SLV_ARMISC 0x120 +#define EXYNOS_PCIE_APPL_SLV_DBI_ENABLE BIT(21) struct exynos_pcie { struct dw_pcie pci; - void __iomem *elbi_base; + void __iomem *appl_base; struct clk_bulk_data *clks; int clk_cnt; struct phy *phy; @@ -93,49 +93,49 @@ static void exynos_pcie_sideband_dbi_w_mode(struct exynos_pcie *ep, bool on) { u32 val; - val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_ELBI_SLV_AWMISC); + val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_APPL_SLV_AWMISC); if (on) - val |= EXYNOS_PCIE_ELBI_SLV_DBI_ENABLE; + val |= EXYNOS_PCIE_APPL_SLV_DBI_ENABLE; else - val &= ~EXYNOS_PCIE_ELBI_SLV_DBI_ENABLE; - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_ELBI_SLV_AWMISC); + val &= ~EXYNOS_PCIE_APPL_SLV_DBI_ENABLE; + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_APPL_SLV_AWMISC); } static void exynos_pcie_sideband_dbi_r_mode(struct exynos_pcie *ep, bool on) { u32 val; - val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_ELBI_SLV_ARMISC); + val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_APPL_SLV_ARMISC); if (on) - val |= EXYNOS_PCIE_ELBI_SLV_DBI_ENABLE; + val |= EXYNOS_PCIE_APPL_SLV_DBI_ENABLE; else - val &= ~EXYNOS_PCIE_ELBI_SLV_DBI_ENABLE; - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_ELBI_SLV_ARMISC); + val &= ~EXYNOS_PCIE_APPL_SLV_DBI_ENABLE; + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_APPL_SLV_ARMISC); } static void exynos_pcie_assert_core_reset(struct exynos_pcie *ep) { u32 val; - val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_CORE_RESET); + val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_CORE_RESET); val &= ~EXYNOS_PCIE_CORE_RESET_ENABLE; - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_CORE_RESET); - exynos_pcie_writel(ep->elbi_base, 0, EXYNOS_PCIE_STICKY_RESET); - exynos_pcie_writel(ep->elbi_base, 0, EXYNOS_PCIE_NONSTICKY_RESET); + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_CORE_RESET); + exynos_pcie_writel(ep->appl_base, 0, EXYNOS_PCIE_STICKY_RESET); + exynos_pcie_writel(ep->appl_base, 0, EXYNOS_PCIE_NONSTICKY_RESET); } static void exynos_pcie_deassert_core_reset(struct exynos_pcie *ep) { u32 val; - val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_CORE_RESET); + val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_CORE_RESET); val |= EXYNOS_PCIE_CORE_RESET_ENABLE; - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_CORE_RESET); - exynos_pcie_writel(ep->elbi_base, 1, EXYNOS_PCIE_STICKY_RESET); - exynos_pcie_writel(ep->elbi_base, 1, EXYNOS_PCIE_NONSTICKY_RESET); - exynos_pcie_writel(ep->elbi_base, 1, EXYNOS_PCIE_APP_INIT_RESET); - exynos_pcie_writel(ep->elbi_base, 0, EXYNOS_PCIE_APP_INIT_RESET); + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_CORE_RESET); + exynos_pcie_writel(ep->appl_base, 1, EXYNOS_PCIE_STICKY_RESET); + exynos_pcie_writel(ep->appl_base, 1, EXYNOS_PCIE_NONSTICKY_RESET); + exynos_pcie_writel(ep->appl_base, 1, EXYNOS_PCIE_APP_INIT_RESET); + exynos_pcie_writel(ep->appl_base, 0, EXYNOS_PCIE_APP_INIT_RESET); } static int exynos_pcie_start_link(struct dw_pcie *pci) @@ -143,21 +143,21 @@ static int exynos_pcie_start_link(struct dw_pcie *pci) struct exynos_pcie *ep = to_exynos_pcie(pci); u32 val; - val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_SW_WAKE); + val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_SW_WAKE); val &= ~EXYNOS_PCIE_BUS_EN; - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_SW_WAKE); + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_SW_WAKE); /* assert LTSSM enable */ - exynos_pcie_writel(ep->elbi_base, EXYNOS_PCIE_ELBI_LTSSM_ENABLE, + exynos_pcie_writel(ep->appl_base, EXYNOS_PCIE_APPL_LTSSM_ENABLE, EXYNOS_PCIE_APP_LTSSM_ENABLE); return 0; } static void exynos_pcie_clear_irq_pulse(struct exynos_pcie *ep) { - u32 val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_IRQ_PULSE); + u32 val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_IRQ_PULSE); - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_IRQ_PULSE); + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_IRQ_PULSE); } static irqreturn_t exynos_pcie_irq_handler(int irq, void *arg) @@ -173,9 +173,9 @@ static void exynos_pcie_enable_irq_pulse(struct exynos_pcie *ep) u32 val = EXYNOS_IRQ_INTA_ASSERT | EXYNOS_IRQ_INTB_ASSERT | EXYNOS_IRQ_INTC_ASSERT | EXYNOS_IRQ_INTD_ASSERT; - exynos_pcie_writel(ep->elbi_base, val, EXYNOS_PCIE_IRQ_EN_PULSE); - exynos_pcie_writel(ep->elbi_base, 0, EXYNOS_PCIE_IRQ_EN_LEVEL); - exynos_pcie_writel(ep->elbi_base, 0, EXYNOS_PCIE_IRQ_EN_SPECIAL); + exynos_pcie_writel(ep->appl_base, val, EXYNOS_PCIE_IRQ_EN_PULSE); + exynos_pcie_writel(ep->appl_base, 0, EXYNOS_PCIE_IRQ_EN_LEVEL); + exynos_pcie_writel(ep->appl_base, 0, EXYNOS_PCIE_IRQ_EN_SPECIAL); } static u32 exynos_pcie_read_dbi(struct dw_pcie *pci, void __iomem *base, @@ -232,9 +232,9 @@ static struct pci_ops exynos_pci_ops = { static int exynos_pcie_link_up(struct dw_pcie *pci) { struct exynos_pcie *ep = to_exynos_pcie(pci); - u32 val = exynos_pcie_readl(ep->elbi_base, EXYNOS_PCIE_ELBI_RDLH_LINKUP); + u32 val = exynos_pcie_readl(ep->appl_base, EXYNOS_PCIE_APPL_RDLH_LINKUP); - return (val & EXYNOS_PCIE_ELBI_XMLH_LINKUP); + return (val & EXYNOS_PCIE_APPL_XMLH_LINKUP); } static int exynos_pcie_host_init(struct dw_pcie_rp *pp) @@ -315,10 +315,10 @@ static int exynos_pcie_probe(struct platform_device *pdev) if (IS_ERR(ep->phy)) return PTR_ERR(ep->phy); - /* External Local Bus interface (ELBI) registers */ - ep->elbi_base = devm_platform_ioremap_resource_byname(pdev, "elbi"); - if (IS_ERR(ep->elbi_base)) - return PTR_ERR(ep->elbi_base); + /* Application logic registers */ + ep->appl_base = devm_platform_ioremap_resource_byname(pdev, "appl"); + if (IS_ERR(ep->appl_base)) + return PTR_ERR(ep->appl_base); ret = exynos_pcie_init_clk_resources(ep); if (ret < 0) -- 2.17.1