* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-02-25 14:47 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-02-25 14:47 UTC (permalink / raw) To: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones Cc: linux-pci, linux-arm-kernel v3: * adapt devicetree bindings documentation fsl,imx6q-pcie.txt v2: * leave out regulator handling from driver, will be done later in PM subsystem * use BIT() macro for MFD Christoph Fritz (2): MFD: imx6sx: Add PCIe register definitions for iomuxc gpr PCI: imx6: add initial imx6sx support .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + 3 files changed, 101 insertions(+), 40 deletions(-) -- 2.1.4 ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-02-25 14:47 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-02-25 14:47 UTC (permalink / raw) To: linux-arm-kernel v3: * adapt devicetree bindings documentation fsl,imx6q-pcie.txt v2: * leave out regulator handling from driver, will be done later in PM subsystem * use BIT() macro for MFD Christoph Fritz (2): MFD: imx6sx: Add PCIe register definitions for iomuxc gpr PCI: imx6: add initial imx6sx support .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + 3 files changed, 101 insertions(+), 40 deletions(-) -- 2.1.4 ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 1/2] MFD: imx6sx: Add PCIe register definitions for iomuxc gpr 2016-02-25 14:47 ` Christoph Fritz @ 2016-02-25 14:47 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-02-25 14:47 UTC (permalink / raw) To: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones Cc: linux-pci, linux-arm-kernel This patch adds macros to define masks and bits for imx6sx PCIe registers. This is based on a patch by Richard Zhu. Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> --- include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h index 558a485..238c8db 100644 --- a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h +++ b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h @@ -422,6 +422,7 @@ #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_MASK (0x1 << 26) #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_ENABLE (0x1 << 26) #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_DISABLE (0x0 << 26) +#define IMX6SX_GPR5_PCIE_BTNRST_RESET BIT(19) #define IMX6SX_GPR5_CSI1_MUX_CTRL_MASK (0x3 << 4) #define IMX6SX_GPR5_CSI1_MUX_CTRL_EXT_PIN (0x0 << 4) #define IMX6SX_GPR5_CSI1_MUX_CTRL_CVD (0x1 << 4) @@ -435,6 +436,10 @@ #define IMX6SX_GPR5_DISP_MUX_DCIC1_LVDS (0x1 << 1) #define IMX6SX_GPR5_DISP_MUX_DCIC1_MASK (0x1 << 1) +#define IMX6SX_GPR12_PCIE_TEST_POWERDOWN BIT(30) +#define IMX6SX_GPR12_PCIE_RX_EQ_MASK (0x7 << 0) +#define IMX6SX_GPR12_PCIE_RX_EQ_2 (0x2 << 0) + /* For imx6ul iomux gpr register field define */ #define IMX6UL_GPR1_ENET1_CLK_DIR (0x1 << 17) #define IMX6UL_GPR1_ENET2_CLK_DIR (0x1 << 18) -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* [PATCH v3 1/2] MFD: imx6sx: Add PCIe register definitions for iomuxc gpr @ 2016-02-25 14:47 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-02-25 14:47 UTC (permalink / raw) To: linux-arm-kernel This patch adds macros to define masks and bits for imx6sx PCIe registers. This is based on a patch by Richard Zhu. Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> --- include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h index 558a485..238c8db 100644 --- a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h +++ b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h @@ -422,6 +422,7 @@ #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_MASK (0x1 << 26) #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_ENABLE (0x1 << 26) #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_DISABLE (0x0 << 26) +#define IMX6SX_GPR5_PCIE_BTNRST_RESET BIT(19) #define IMX6SX_GPR5_CSI1_MUX_CTRL_MASK (0x3 << 4) #define IMX6SX_GPR5_CSI1_MUX_CTRL_EXT_PIN (0x0 << 4) #define IMX6SX_GPR5_CSI1_MUX_CTRL_CVD (0x1 << 4) @@ -435,6 +436,10 @@ #define IMX6SX_GPR5_DISP_MUX_DCIC1_LVDS (0x1 << 1) #define IMX6SX_GPR5_DISP_MUX_DCIC1_MASK (0x1 << 1) +#define IMX6SX_GPR12_PCIE_TEST_POWERDOWN BIT(30) +#define IMX6SX_GPR12_PCIE_RX_EQ_MASK (0x7 << 0) +#define IMX6SX_GPR12_PCIE_RX_EQ_2 (0x2 << 0) + /* For imx6ul iomux gpr register field define */ #define IMX6UL_GPR1_ENET1_CLK_DIR (0x1 << 17) #define IMX6UL_GPR1_ENET2_CLK_DIR (0x1 << 18) -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH v3 1/2] MFD: imx6sx: Add PCIe register definitions for iomuxc gpr 2016-02-25 14:47 ` Christoph Fritz @ 2016-03-08 4:32 ` Lee Jones -1 siblings, 0 replies; 35+ messages in thread From: Lee Jones @ 2016-03-08 4:32 UTC (permalink / raw) To: Christoph Fritz Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, linux-pci, linux-arm-kernel On Thu, 25 Feb 2016, Christoph Fritz wrote: > This patch adds macros to define masks and bits for imx6sx > PCIe registers. This is based on a patch by Richard Zhu. > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > --- > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 +++++ > 1 file changed, 5 insertions(+) I will change the non-conformant $SUBJECT line for you this time, but please be more vigilant in the future. Patch applied. > diff --git a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > index 558a485..238c8db 100644 > --- a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > +++ b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > @@ -422,6 +422,7 @@ > #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_MASK (0x1 << 26) > #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_ENABLE (0x1 << 26) > #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_DISABLE (0x0 << 26) > +#define IMX6SX_GPR5_PCIE_BTNRST_RESET BIT(19) > #define IMX6SX_GPR5_CSI1_MUX_CTRL_MASK (0x3 << 4) > #define IMX6SX_GPR5_CSI1_MUX_CTRL_EXT_PIN (0x0 << 4) > #define IMX6SX_GPR5_CSI1_MUX_CTRL_CVD (0x1 << 4) > @@ -435,6 +436,10 @@ > #define IMX6SX_GPR5_DISP_MUX_DCIC1_LVDS (0x1 << 1) > #define IMX6SX_GPR5_DISP_MUX_DCIC1_MASK (0x1 << 1) > > +#define IMX6SX_GPR12_PCIE_TEST_POWERDOWN BIT(30) > +#define IMX6SX_GPR12_PCIE_RX_EQ_MASK (0x7 << 0) > +#define IMX6SX_GPR12_PCIE_RX_EQ_2 (0x2 << 0) > + > /* For imx6ul iomux gpr register field define */ > #define IMX6UL_GPR1_ENET1_CLK_DIR (0x1 << 17) > #define IMX6UL_GPR1_ENET2_CLK_DIR (0x1 << 18) -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 1/2] MFD: imx6sx: Add PCIe register definitions for iomuxc gpr @ 2016-03-08 4:32 ` Lee Jones 0 siblings, 0 replies; 35+ messages in thread From: Lee Jones @ 2016-03-08 4:32 UTC (permalink / raw) To: linux-arm-kernel On Thu, 25 Feb 2016, Christoph Fritz wrote: > This patch adds macros to define masks and bits for imx6sx > PCIe registers. This is based on a patch by Richard Zhu. > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > --- > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 +++++ > 1 file changed, 5 insertions(+) I will change the non-conformant $SUBJECT line for you this time, but please be more vigilant in the future. Patch applied. > diff --git a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > index 558a485..238c8db 100644 > --- a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > +++ b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > @@ -422,6 +422,7 @@ > #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_MASK (0x1 << 26) > #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_ENABLE (0x1 << 26) > #define IMX6SX_GPR5_VADC_TO_CSI_CAPTURE_EN_DISABLE (0x0 << 26) > +#define IMX6SX_GPR5_PCIE_BTNRST_RESET BIT(19) > #define IMX6SX_GPR5_CSI1_MUX_CTRL_MASK (0x3 << 4) > #define IMX6SX_GPR5_CSI1_MUX_CTRL_EXT_PIN (0x0 << 4) > #define IMX6SX_GPR5_CSI1_MUX_CTRL_CVD (0x1 << 4) > @@ -435,6 +436,10 @@ > #define IMX6SX_GPR5_DISP_MUX_DCIC1_LVDS (0x1 << 1) > #define IMX6SX_GPR5_DISP_MUX_DCIC1_MASK (0x1 << 1) > > +#define IMX6SX_GPR12_PCIE_TEST_POWERDOWN BIT(30) > +#define IMX6SX_GPR12_PCIE_RX_EQ_MASK (0x7 << 0) > +#define IMX6SX_GPR12_PCIE_RX_EQ_2 (0x2 << 0) > + > /* For imx6ul iomux gpr register field define */ > #define IMX6UL_GPR1_ENET1_CLK_DIR (0x1 << 17) > #define IMX6UL_GPR1_ENET2_CLK_DIR (0x1 << 18) -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 2/2] PCI: imx6: add initial imx6sx support 2016-02-25 14:47 ` Christoph Fritz @ 2016-02-25 14:47 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-02-25 14:47 UTC (permalink / raw) To: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones Cc: linux-pci, linux-arm-kernel This patch adds initial PCIe support for the imx6 SoC derivate imx6sx. PCI_MSI support is untested as its necessary suspend/resume quirk is left out of this patch. This patch is heavily based on patches by Richard Zhu. Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> --- .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- 2 files changed, 96 insertions(+), 40 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt index 6fbba53..c4323be 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt @@ -4,7 +4,7 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP and thus inherits all the common properties defined in designware-pcie.txt. Required properties: -- compatible: "fsl,imx6q-pcie" +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" - reg: base addresse and length of the pcie controller - interrupts: A list of interrupt outputs of the controller. Must contain an entry for each entry in the interrupt-names property. @@ -13,6 +13,10 @@ Required properties: - clock-names: Must include the following additional entries: - "pcie_phy" +Additional required properties for imx6sx-pcie: +- clock names: Must include the following additional entries: + - "pcie_inbound_axi" + Example: pcie@0x01000000 { diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index fe60096..7e634a6 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -35,9 +35,11 @@ struct imx6_pcie { struct gpio_desc *reset_gpio; struct clk *pcie_bus; struct clk *pcie_phy; + struct clk *pcie_inbound_axi; struct clk *pcie; struct pcie_port pp; struct regmap *iomuxc_gpr; + bool is_imx6sx; void __iomem *mem_base; }; @@ -214,35 +216,46 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); u32 val, gpr1, gpr12; - /* - * If the bootloader already enabled the link we need some special - * handling to get the core back into a state where it is safe to - * touch it for configuration. As there is no dedicated reset signal - * wired up for MX6QDL, we need to manually force LTSSM into "detect" - * state before completely disabling LTSSM, which is a prerequisite - * for core configuration. - * - * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong - * indication that the bootloader activated the link. - */ - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); - - if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && - (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { - val = readl(pp->dbi_base + PCIE_PL_PFLR); - val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; - val |= PCIE_PL_PFLR_FORCE_LINK; - writel(val, pp->dbi_base + PCIE_PL_PFLR); - + if (imx6_pcie->is_imx6sx) { regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, - IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); - } + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); + /* Force PCIe PHY reset */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, + IMX6SX_GPR5_PCIE_BTNRST_RESET); + } else { + /* + * If the bootloader already enabled the link we need some + * special handling to get the core back into a state where + * it is safe to touch it for configuration. As there is no + * dedicated reset signal to manually force LTSSM into "detect" + * state before completely disabling LTSSM, which is a + * prerequisite for core configuration. + * + * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have + * a strong indication that the bootloader activated the link. + */ + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); + + if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && + (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { + val = readl(pp->dbi_base + PCIE_PL_PFLR); + val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; + val |= PCIE_PL_PFLR_FORCE_LINK; + writel(val, pp->dbi_base + PCIE_PL_PFLR); + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6Q_GPR12_PCIE_CTL_2, 0); + } - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 1 << 18); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 0 << 16); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, + IMX6Q_GPR1_PCIE_TEST_PD); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, 0); + } return 0; } @@ -270,18 +283,30 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) goto err_pcie; } - /* power up core phy and enable ref clock */ - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); - /* - * the async reset input need ref clock to sync internally, - * when the ref clock comes after reset, internal synced - * reset time is too short, cannot meet the requirement. - * add one ~10us delay here. - */ - udelay(10); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + if (imx6_pcie->is_imx6sx) { + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); + if (ret) { + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); + goto err_inbound_axi; + } + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); + } else { + /* power up core phy and enable ref clock */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, 0); + /* + * the async reset input need ref clock to sync internally, + * when the ref clock comes after reset, internal synced + * reset time is too short , cannot meet the requirement. + * add one ~10us delay here. + */ + udelay(10); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, + IMX6Q_GPR1_PCIE_REF_CLK_EN); + } /* allow the clocks to stabilize */ usleep_range(200, 500); @@ -292,8 +317,15 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) msleep(100); gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); } + + if (imx6_pcie->is_imx6sx) + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); + return 0; +err_inbound_axi: + clk_disable_unprepare(imx6_pcie->pcie); err_pcie: clk_disable_unprepare(imx6_pcie->pcie_bus); err_pcie_bus: @@ -307,6 +339,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_RX_EQ_MASK, + IMX6SX_GPR12_PCIE_RX_EQ_2); + } + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); @@ -571,6 +609,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) pp = &imx6_pcie->pp; pp->dev = &pdev->dev; + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, + "fsl,imx6sx-pcie"); + /* Added for PCI abort handling */ hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, "imprecise external abort"); @@ -606,6 +647,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) return PTR_ERR(imx6_pcie->pcie); } + if (imx6_pcie->is_imx6sx) { + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, + "pcie_inbound_axi"); + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { + dev_err(&pdev->dev, + "pcie_incbound_axi clock missing or invalid\n"); + return PTR_ERR(imx6_pcie->pcie_inbound_axi); + } + } + /* Grab GPR config register range */ imx6_pcie->iomuxc_gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); @@ -632,6 +683,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) static const struct of_device_id imx6_pcie_of_match[] = { { .compatible = "fsl,imx6q-pcie", }, + { .compatible = "fsl,imx6sx-pcie", }, {}, }; MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* [PATCH v3 2/2] PCI: imx6: add initial imx6sx support @ 2016-02-25 14:47 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-02-25 14:47 UTC (permalink / raw) To: linux-arm-kernel This patch adds initial PCIe support for the imx6 SoC derivate imx6sx. PCI_MSI support is untested as its necessary suspend/resume quirk is left out of this patch. This patch is heavily based on patches by Richard Zhu. Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> --- .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- 2 files changed, 96 insertions(+), 40 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt index 6fbba53..c4323be 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt @@ -4,7 +4,7 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP and thus inherits all the common properties defined in designware-pcie.txt. Required properties: -- compatible: "fsl,imx6q-pcie" +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" - reg: base addresse and length of the pcie controller - interrupts: A list of interrupt outputs of the controller. Must contain an entry for each entry in the interrupt-names property. @@ -13,6 +13,10 @@ Required properties: - clock-names: Must include the following additional entries: - "pcie_phy" +Additional required properties for imx6sx-pcie: +- clock names: Must include the following additional entries: + - "pcie_inbound_axi" + Example: pcie at 0x01000000 { diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index fe60096..7e634a6 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -35,9 +35,11 @@ struct imx6_pcie { struct gpio_desc *reset_gpio; struct clk *pcie_bus; struct clk *pcie_phy; + struct clk *pcie_inbound_axi; struct clk *pcie; struct pcie_port pp; struct regmap *iomuxc_gpr; + bool is_imx6sx; void __iomem *mem_base; }; @@ -214,35 +216,46 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); u32 val, gpr1, gpr12; - /* - * If the bootloader already enabled the link we need some special - * handling to get the core back into a state where it is safe to - * touch it for configuration. As there is no dedicated reset signal - * wired up for MX6QDL, we need to manually force LTSSM into "detect" - * state before completely disabling LTSSM, which is a prerequisite - * for core configuration. - * - * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong - * indication that the bootloader activated the link. - */ - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); - - if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && - (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { - val = readl(pp->dbi_base + PCIE_PL_PFLR); - val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; - val |= PCIE_PL_PFLR_FORCE_LINK; - writel(val, pp->dbi_base + PCIE_PL_PFLR); - + if (imx6_pcie->is_imx6sx) { regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, - IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); - } + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); + /* Force PCIe PHY reset */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, + IMX6SX_GPR5_PCIE_BTNRST_RESET); + } else { + /* + * If the bootloader already enabled the link we need some + * special handling to get the core back into a state where + * it is safe to touch it for configuration. As there is no + * dedicated reset signal to manually force LTSSM into "detect" + * state before completely disabling LTSSM, which is a + * prerequisite for core configuration. + * + * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have + * a strong indication that the bootloader activated the link. + */ + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); + + if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && + (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { + val = readl(pp->dbi_base + PCIE_PL_PFLR); + val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; + val |= PCIE_PL_PFLR_FORCE_LINK; + writel(val, pp->dbi_base + PCIE_PL_PFLR); + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6Q_GPR12_PCIE_CTL_2, 0); + } - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 1 << 18); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 0 << 16); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, + IMX6Q_GPR1_PCIE_TEST_PD); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, 0); + } return 0; } @@ -270,18 +283,30 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) goto err_pcie; } - /* power up core phy and enable ref clock */ - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); - /* - * the async reset input need ref clock to sync internally, - * when the ref clock comes after reset, internal synced - * reset time is too short, cannot meet the requirement. - * add one ~10us delay here. - */ - udelay(10); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + if (imx6_pcie->is_imx6sx) { + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); + if (ret) { + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); + goto err_inbound_axi; + } + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); + } else { + /* power up core phy and enable ref clock */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, 0); + /* + * the async reset input need ref clock to sync internally, + * when the ref clock comes after reset, internal synced + * reset time is too short , cannot meet the requirement. + * add one ~10us delay here. + */ + udelay(10); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, + IMX6Q_GPR1_PCIE_REF_CLK_EN); + } /* allow the clocks to stabilize */ usleep_range(200, 500); @@ -292,8 +317,15 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) msleep(100); gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); } + + if (imx6_pcie->is_imx6sx) + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); + return 0; +err_inbound_axi: + clk_disable_unprepare(imx6_pcie->pcie); err_pcie: clk_disable_unprepare(imx6_pcie->pcie_bus); err_pcie_bus: @@ -307,6 +339,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_RX_EQ_MASK, + IMX6SX_GPR12_PCIE_RX_EQ_2); + } + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); @@ -571,6 +609,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) pp = &imx6_pcie->pp; pp->dev = &pdev->dev; + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, + "fsl,imx6sx-pcie"); + /* Added for PCI abort handling */ hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, "imprecise external abort"); @@ -606,6 +647,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) return PTR_ERR(imx6_pcie->pcie); } + if (imx6_pcie->is_imx6sx) { + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, + "pcie_inbound_axi"); + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { + dev_err(&pdev->dev, + "pcie_incbound_axi clock missing or invalid\n"); + return PTR_ERR(imx6_pcie->pcie_inbound_axi); + } + } + /* Grab GPR config register range */ imx6_pcie->iomuxc_gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); @@ -632,6 +683,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) static const struct of_device_id imx6_pcie_of_match[] = { { .compatible = "fsl,imx6q-pcie", }, + { .compatible = "fsl,imx6sx-pcie", }, {}, }; MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH v3 2/2] PCI: imx6: add initial imx6sx support 2016-02-25 14:47 ` Christoph Fritz @ 2016-03-11 17:58 ` Bjorn Helgaas -1 siblings, 0 replies; 35+ messages in thread From: Bjorn Helgaas @ 2016-03-11 17:58 UTC (permalink / raw) To: Christoph Fritz Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel Hi Christoph (and Lee; there's a question for you below, too), On Thu, Feb 25, 2016 at 03:47:49PM +0100, Christoph Fritz wrote: > This patch adds initial PCIe support for the imx6 SoC derivate imx6sx. > PCI_MSI support is untested as its necessary suspend/resume quirk is > left out of this patch. > This patch is heavily based on patches by Richard Zhu. > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > 2 files changed, 96 insertions(+), 40 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > index 6fbba53..c4323be 100644 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > @@ -4,7 +4,7 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > and thus inherits all the common properties defined in designware-pcie.txt. > > Required properties: > -- compatible: "fsl,imx6q-pcie" > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > - reg: base addresse and length of the pcie controller I folded in a typo fix for "addresse". > - interrupts: A list of interrupt outputs of the controller. Must contain an > entry for each entry in the interrupt-names property. > @@ -13,6 +13,10 @@ Required properties: > - clock-names: Must include the following additional entries: > - "pcie_phy" > > +Additional required properties for imx6sx-pcie: > +- clock names: Must include the following additional entries: > + - "pcie_inbound_axi" > + > Example: > > pcie@0x01000000 { > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index fe60096..7e634a6 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; > + struct clk *pcie_inbound_axi; > struct clk *pcie; > struct pcie_port pp; > struct regmap *iomuxc_gpr; > + bool is_imx6sx; > void __iomem *mem_base; > }; > > @@ -214,35 +216,46 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > u32 val, gpr1, gpr12; > > - /* > - * If the bootloader already enabled the link we need some special > - * handling to get the core back into a state where it is safe to > - * touch it for configuration. As there is no dedicated reset signal > - * wired up for MX6QDL, we need to manually force LTSSM into "detect" > - * state before completely disabling LTSSM, which is a prerequisite > - * for core configuration. > - * > - * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong > - * indication that the bootloader activated the link. > - */ > - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); > - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); > - > - if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && > - (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { > - val = readl(pp->dbi_base + PCIE_PL_PFLR); > - val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; > - val |= PCIE_PL_PFLR_FORCE_LINK; > - writel(val, pp->dbi_base + PCIE_PL_PFLR); > - > + if (imx6_pcie->is_imx6sx) { > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > - IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > - } > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); > + /* Force PCIe PHY reset */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, > + IMX6SX_GPR5_PCIE_BTNRST_RESET); Since there's nothing after the "else" clause, I think it's better to return immediately here. Then we don't have extra diffs that merely indent the original function. I did make this change (I'll attach my resulting patch), and that made some other changes to the indented code more visible. > + } else { > + /* > + * If the bootloader already enabled the link we need some > + * special handling to get the core back into a state where > + * it is safe to touch it for configuration. As there is no > + * dedicated reset signal to manually force LTSSM into "detect" > + * state before completely disabling LTSSM, which is a > + * prerequisite for core configuration. For example, you changed this comment, and now the last sentence is no longer a sentence. I don't know if this was an editing mistake or if this comment really does need to change. If it does need to change, it needs to stay a sentence. > + * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have > + * a strong indication that the bootloader activated the link. > + */ > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); > + > + if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && > + (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { > + val = readl(pp->dbi_base + PCIE_PL_PFLR); > + val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; > + val |= PCIE_PL_PFLR_FORCE_LINK; > + writel(val, pp->dbi_base + PCIE_PL_PFLR); > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6Q_GPR12_PCIE_CTL_2, 0); > + } > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 1 << 18); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 0 << 16); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, > + IMX6Q_GPR1_PCIE_TEST_PD); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 0); And there are subtle changes here (changing "1 << 18" to IMX6Q_GPR1_PCIE_TEST_PD, etc.) These are fine, but should be in a separate patch because (a) they're not related to imx6sx and (b) it makes them obvious and easy to review. As it is, they just get snuck in under the radar. > + } > > return 0; > } > @@ -270,18 +283,30 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > goto err_pcie; > } > > - /* power up core phy and enable ref clock */ > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > - /* > - * the async reset input need ref clock to sync internally, > - * when the ref clock comes after reset, internal synced > - * reset time is too short, cannot meet the requirement. > - * add one ~10us delay here. > - */ > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + goto err_inbound_axi; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + } else { > + /* power up core phy and enable ref clock */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, 0); > + /* > + * the async reset input need ref clock to sync internally, > + * when the ref clock comes after reset, internal synced > + * reset time is too short , cannot meet the requirement. > + * add one ~10us delay here. > + */ > + udelay(10); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, > + IMX6Q_GPR1_PCIE_REF_CLK_EN); > + } Here again the diff is bigger than necessary because it re-indents the original code (and makes another unrelated "1 << 16" to IMX6Q_GPR1_PCIE_REF_CLK_EN change). I'll show a different proposal in my patch below. > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -292,8 +317,15 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > msleep(100); > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > } > + > + if (imx6_pcie->is_imx6sx) > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > + > return 0; > > +err_inbound_axi: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: > @@ -307,6 +339,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > + IMX6SX_GPR12_PCIE_RX_EQ_2); I see that you added definitions for IMX6SX_GPR12_PCIE_RX_EQ_MASK and IMX6SX_GPR12_PCIE_RX_EQ_2 in the first patch of this series, but doing that separately creates a merge problem. I can't put this change in my tree by itself because it won't build. If Lee applied the first patch to a git branch, I guess I could pull that into my tree. But it would be far simpler to just include those new definitions in this patch. > + } > + > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > > @@ -571,6 +609,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > pp = &imx6_pcie->pp; > pp->dev = &pdev->dev; > > + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, > + "fsl,imx6sx-pcie"); > + > /* Added for PCI abort handling */ > hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, > "imprecise external abort"); > @@ -606,6 +647,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > return PTR_ERR(imx6_pcie->pcie); > } > > + if (imx6_pcie->is_imx6sx) { > + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, > + "pcie_inbound_axi"); > + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { > + dev_err(&pdev->dev, > + "pcie_incbound_axi clock missing or invalid\n"); > + return PTR_ERR(imx6_pcie->pcie_inbound_axi); > + } > + } > + > /* Grab GPR config register range */ > imx6_pcie->iomuxc_gpr = > syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); > @@ -632,6 +683,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) > > static const struct of_device_id imx6_pcie_of_match[] = { > { .compatible = "fsl,imx6q-pcie", }, > + { .compatible = "fsl,imx6sx-pcie", }, > {}, > }; > MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); The patches below are what I currently have in my tree, but they're not ready for v4.6 (yet). Note that the second one is very clearly imx6sx-only. It's obvious there are no tweaks to the existing imx6 code, which makes it easier to review, backport, bisect, revert, etc. Obviously, I can't apply these directly. First we have to: - Fix the LTSSM comment - Figure out how to get the IMX6SX_GPR12_PCIE_RX_EQ_MASK definitions commit 78abf60b815a8f28207d029d61f7809647faa43a Author: Bjorn Helgaas <bhelgaas@google.com> Date: Fri Mar 11 11:15:36 2016 -0600 PCI: imx6: Factor out ref clock enable Factor out ref clock enable to make it cleaner to add imx6sx support. No functional change intended. Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index 8c9b389..ecc8537 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -269,6 +269,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) return 0; } +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) +{ + /* power up core phy and enable ref clock */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); + /* + * the async reset input need ref clock to sync internally, + * when the ref clock comes after reset, internal synced + * reset time is too short, cannot meet the requirement. + * add one ~10us delay here. + */ + udelay(10); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + return 0; +} + static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); @@ -292,18 +309,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) goto err_pcie; } - /* power up core phy and enable ref clock */ - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); - /* - * the async reset input need ref clock to sync internally, - * when the ref clock comes after reset, internal synced - * reset time is too short, cannot meet the requirement. - * add one ~10us delay here. - */ - udelay(10); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + ret = imx6_pcie_enable_ref_clk(imx6_pcie); + if (ret) + dev_err(pp->dev, "unable to enable pcie ref clock\n"); + goto err_ref_clk; + } /* allow the clocks to stabilize */ usleep_range(200, 500); @@ -316,6 +326,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) } return 0; +err_ref_clk: + clk_disable_unprepare(imx6_pcie->pcie); err_pcie: clk_disable_unprepare(imx6_pcie->pcie_bus); err_pcie_bus: commit d061033570b7fd22a3ed8c029d4793e5df7324e1 Author: Christoph Fritz <chf.fritz@googlemail.com> Date: Thu Mar 10 15:04:25 2016 -0600 PCI: imx6: Add initial imx6sx support Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support is untested as the necessary suspend/resume quirk is not included in this patch. This patch is heavily based on patches by Richard Zhu. [bhelgaas: factor out refclk enable, fix adjacent typos in fsl,imx6q-pcie.txt] Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> Acked-by: Richard Zhu <Richard.Zhu@freescale.com> Acked-by: Lucas Stach <l.stach@pengutronix.de> diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt index 3be80c6..0561e88 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP and thus inherits all the common properties defined in designware-pcie.txt. Required properties: -- compatible: "fsl,imx6q-pcie" -- reg: base addresse and length of the pcie controller +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" +- reg: base address and length of the PCIe controller - interrupts: A list of interrupt outputs of the controller. Must contain an entry for each entry in the interrupt-names property. - interrupt-names: Must include the following entries: @@ -20,6 +20,10 @@ Optional properties: - fsl,tx-swing-full: Gen2 TX SWING FULL value. Default: 127 - fsl,tx-swing-low: TX launch amplitude swing_low value. Default: 127 +Additional required properties for imx6sx-pcie: +- clock names: Must include the following additional entries: + - "pcie_inbound_axi" + Example: pcie@0x01000000 { diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index ecc8537..4d6af21 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -35,9 +35,11 @@ struct imx6_pcie { struct gpio_desc *reset_gpio; struct clk *pcie_bus; struct clk *pcie_phy; + struct clk *pcie_inbound_axi; struct clk *pcie; struct pcie_port pp; struct regmap *iomuxc_gpr; + bool is_imx6sx; void __iomem *mem_base; u32 tx_deemph_gen1; u32 tx_deemph_gen2_3p5db; @@ -236,13 +238,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); u32 val, gpr1, gpr12; + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); + /* Force PCIe PHY reset */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, + IMX6SX_GPR5_PCIE_BTNRST_RESET); + return 0; + } + /* * If the bootloader already enabled the link we need some special * handling to get the core back into a state where it is safe to * touch it for configuration. As there is no dedicated reset signal - * wired up for MX6QDL, we need to manually force LTSSM into "detect" - * state before completely disabling LTSSM, which is a prerequisite - * for core configuration. + * to manually force LTSSM into "detect" state before completely + * disabling LTSSM, which is a prerequisite for core configuration. * * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong * indication that the bootloader activated the link. @@ -271,6 +283,18 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) { + if (imx6_pcie->is_imx6sx) { + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); + if (ret) { + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); + return ret; + } + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); + return 0; + } + /* power up core phy and enable ref clock */ regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); @@ -324,6 +348,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) msleep(100); gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); } + + if (imx6_pcie->is_imx6sx) + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); + return 0; err_ref_clk: @@ -341,6 +370,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_RX_EQ_MASK, + IMX6SX_GPR12_PCIE_RX_EQ_2); + } + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); @@ -565,6 +600,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) pp = &imx6_pcie->pp; pp->dev = &pdev->dev; + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, + "fsl,imx6sx-pcie"); + /* Added for PCI abort handling */ hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, "imprecise external abort"); @@ -600,6 +638,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) return PTR_ERR(imx6_pcie->pcie); } + if (imx6_pcie->is_imx6sx) { + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, + "pcie_inbound_axi"); + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { + dev_err(&pdev->dev, + "pcie_incbound_axi clock missing or invalid\n"); + return PTR_ERR(imx6_pcie->pcie_inbound_axi); + } + } + /* Grab GPR config register range */ imx6_pcie->iomuxc_gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); @@ -647,6 +695,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) static const struct of_device_id imx6_pcie_of_match[] = { { .compatible = "fsl,imx6q-pcie", }, + { .compatible = "fsl,imx6sx-pcie", }, {}, }; MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); ^ permalink raw reply related [flat|nested] 35+ messages in thread
* [PATCH v3 2/2] PCI: imx6: add initial imx6sx support @ 2016-03-11 17:58 ` Bjorn Helgaas 0 siblings, 0 replies; 35+ messages in thread From: Bjorn Helgaas @ 2016-03-11 17:58 UTC (permalink / raw) To: linux-arm-kernel Hi Christoph (and Lee; there's a question for you below, too), On Thu, Feb 25, 2016 at 03:47:49PM +0100, Christoph Fritz wrote: > This patch adds initial PCIe support for the imx6 SoC derivate imx6sx. > PCI_MSI support is untested as its necessary suspend/resume quirk is > left out of this patch. > This patch is heavily based on patches by Richard Zhu. > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > 2 files changed, 96 insertions(+), 40 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > index 6fbba53..c4323be 100644 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > @@ -4,7 +4,7 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > and thus inherits all the common properties defined in designware-pcie.txt. > > Required properties: > -- compatible: "fsl,imx6q-pcie" > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > - reg: base addresse and length of the pcie controller I folded in a typo fix for "addresse". > - interrupts: A list of interrupt outputs of the controller. Must contain an > entry for each entry in the interrupt-names property. > @@ -13,6 +13,10 @@ Required properties: > - clock-names: Must include the following additional entries: > - "pcie_phy" > > +Additional required properties for imx6sx-pcie: > +- clock names: Must include the following additional entries: > + - "pcie_inbound_axi" > + > Example: > > pcie at 0x01000000 { > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index fe60096..7e634a6 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; > + struct clk *pcie_inbound_axi; > struct clk *pcie; > struct pcie_port pp; > struct regmap *iomuxc_gpr; > + bool is_imx6sx; > void __iomem *mem_base; > }; > > @@ -214,35 +216,46 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > u32 val, gpr1, gpr12; > > - /* > - * If the bootloader already enabled the link we need some special > - * handling to get the core back into a state where it is safe to > - * touch it for configuration. As there is no dedicated reset signal > - * wired up for MX6QDL, we need to manually force LTSSM into "detect" > - * state before completely disabling LTSSM, which is a prerequisite > - * for core configuration. > - * > - * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong > - * indication that the bootloader activated the link. > - */ > - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); > - regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); > - > - if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && > - (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { > - val = readl(pp->dbi_base + PCIE_PL_PFLR); > - val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; > - val |= PCIE_PL_PFLR_FORCE_LINK; > - writel(val, pp->dbi_base + PCIE_PL_PFLR); > - > + if (imx6_pcie->is_imx6sx) { > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > - IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > - } > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); > + /* Force PCIe PHY reset */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, > + IMX6SX_GPR5_PCIE_BTNRST_RESET); Since there's nothing after the "else" clause, I think it's better to return immediately here. Then we don't have extra diffs that merely indent the original function. I did make this change (I'll attach my resulting patch), and that made some other changes to the indented code more visible. > + } else { > + /* > + * If the bootloader already enabled the link we need some > + * special handling to get the core back into a state where > + * it is safe to touch it for configuration. As there is no > + * dedicated reset signal to manually force LTSSM into "detect" > + * state before completely disabling LTSSM, which is a > + * prerequisite for core configuration. For example, you changed this comment, and now the last sentence is no longer a sentence. I don't know if this was an editing mistake or if this comment really does need to change. If it does need to change, it needs to stay a sentence. > + * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have > + * a strong indication that the bootloader activated the link. > + */ > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); > + > + if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && > + (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { > + val = readl(pp->dbi_base + PCIE_PL_PFLR); > + val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; > + val |= PCIE_PL_PFLR_FORCE_LINK; > + writel(val, pp->dbi_base + PCIE_PL_PFLR); > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6Q_GPR12_PCIE_CTL_2, 0); > + } > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 1 << 18); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 0 << 16); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, > + IMX6Q_GPR1_PCIE_TEST_PD); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 0); And there are subtle changes here (changing "1 << 18" to IMX6Q_GPR1_PCIE_TEST_PD, etc.) These are fine, but should be in a separate patch because (a) they're not related to imx6sx and (b) it makes them obvious and easy to review. As it is, they just get snuck in under the radar. > + } > > return 0; > } > @@ -270,18 +283,30 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > goto err_pcie; > } > > - /* power up core phy and enable ref clock */ > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > - /* > - * the async reset input need ref clock to sync internally, > - * when the ref clock comes after reset, internal synced > - * reset time is too short, cannot meet the requirement. > - * add one ~10us delay here. > - */ > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + goto err_inbound_axi; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + } else { > + /* power up core phy and enable ref clock */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, 0); > + /* > + * the async reset input need ref clock to sync internally, > + * when the ref clock comes after reset, internal synced > + * reset time is too short , cannot meet the requirement. > + * add one ~10us delay here. > + */ > + udelay(10); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, > + IMX6Q_GPR1_PCIE_REF_CLK_EN); > + } Here again the diff is bigger than necessary because it re-indents the original code (and makes another unrelated "1 << 16" to IMX6Q_GPR1_PCIE_REF_CLK_EN change). I'll show a different proposal in my patch below. > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -292,8 +317,15 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > msleep(100); > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > } > + > + if (imx6_pcie->is_imx6sx) > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > + > return 0; > > +err_inbound_axi: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: > @@ -307,6 +339,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > + IMX6SX_GPR12_PCIE_RX_EQ_2); I see that you added definitions for IMX6SX_GPR12_PCIE_RX_EQ_MASK and IMX6SX_GPR12_PCIE_RX_EQ_2 in the first patch of this series, but doing that separately creates a merge problem. I can't put this change in my tree by itself because it won't build. If Lee applied the first patch to a git branch, I guess I could pull that into my tree. But it would be far simpler to just include those new definitions in this patch. > + } > + > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > > @@ -571,6 +609,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > pp = &imx6_pcie->pp; > pp->dev = &pdev->dev; > > + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, > + "fsl,imx6sx-pcie"); > + > /* Added for PCI abort handling */ > hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, > "imprecise external abort"); > @@ -606,6 +647,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > return PTR_ERR(imx6_pcie->pcie); > } > > + if (imx6_pcie->is_imx6sx) { > + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, > + "pcie_inbound_axi"); > + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { > + dev_err(&pdev->dev, > + "pcie_incbound_axi clock missing or invalid\n"); > + return PTR_ERR(imx6_pcie->pcie_inbound_axi); > + } > + } > + > /* Grab GPR config register range */ > imx6_pcie->iomuxc_gpr = > syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); > @@ -632,6 +683,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) > > static const struct of_device_id imx6_pcie_of_match[] = { > { .compatible = "fsl,imx6q-pcie", }, > + { .compatible = "fsl,imx6sx-pcie", }, > {}, > }; > MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); The patches below are what I currently have in my tree, but they're not ready for v4.6 (yet). Note that the second one is very clearly imx6sx-only. It's obvious there are no tweaks to the existing imx6 code, which makes it easier to review, backport, bisect, revert, etc. Obviously, I can't apply these directly. First we have to: - Fix the LTSSM comment - Figure out how to get the IMX6SX_GPR12_PCIE_RX_EQ_MASK definitions commit 78abf60b815a8f28207d029d61f7809647faa43a Author: Bjorn Helgaas <bhelgaas@google.com> Date: Fri Mar 11 11:15:36 2016 -0600 PCI: imx6: Factor out ref clock enable Factor out ref clock enable to make it cleaner to add imx6sx support. No functional change intended. Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index 8c9b389..ecc8537 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -269,6 +269,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) return 0; } +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) +{ + /* power up core phy and enable ref clock */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); + /* + * the async reset input need ref clock to sync internally, + * when the ref clock comes after reset, internal synced + * reset time is too short, cannot meet the requirement. + * add one ~10us delay here. + */ + udelay(10); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + return 0; +} + static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); @@ -292,18 +309,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) goto err_pcie; } - /* power up core phy and enable ref clock */ - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); - /* - * the async reset input need ref clock to sync internally, - * when the ref clock comes after reset, internal synced - * reset time is too short, cannot meet the requirement. - * add one ~10us delay here. - */ - udelay(10); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + ret = imx6_pcie_enable_ref_clk(imx6_pcie); + if (ret) + dev_err(pp->dev, "unable to enable pcie ref clock\n"); + goto err_ref_clk; + } /* allow the clocks to stabilize */ usleep_range(200, 500); @@ -316,6 +326,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) } return 0; +err_ref_clk: + clk_disable_unprepare(imx6_pcie->pcie); err_pcie: clk_disable_unprepare(imx6_pcie->pcie_bus); err_pcie_bus: commit d061033570b7fd22a3ed8c029d4793e5df7324e1 Author: Christoph Fritz <chf.fritz@googlemail.com> Date: Thu Mar 10 15:04:25 2016 -0600 PCI: imx6: Add initial imx6sx support Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support is untested as the necessary suspend/resume quirk is not included in this patch. This patch is heavily based on patches by Richard Zhu. [bhelgaas: factor out refclk enable, fix adjacent typos in fsl,imx6q-pcie.txt] Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> Acked-by: Richard Zhu <Richard.Zhu@freescale.com> Acked-by: Lucas Stach <l.stach@pengutronix.de> diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt index 3be80c6..0561e88 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP and thus inherits all the common properties defined in designware-pcie.txt. Required properties: -- compatible: "fsl,imx6q-pcie" -- reg: base addresse and length of the pcie controller +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" +- reg: base address and length of the PCIe controller - interrupts: A list of interrupt outputs of the controller. Must contain an entry for each entry in the interrupt-names property. - interrupt-names: Must include the following entries: @@ -20,6 +20,10 @@ Optional properties: - fsl,tx-swing-full: Gen2 TX SWING FULL value. Default: 127 - fsl,tx-swing-low: TX launch amplitude swing_low value. Default: 127 +Additional required properties for imx6sx-pcie: +- clock names: Must include the following additional entries: + - "pcie_inbound_axi" + Example: pcie at 0x01000000 { diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index ecc8537..4d6af21 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -35,9 +35,11 @@ struct imx6_pcie { struct gpio_desc *reset_gpio; struct clk *pcie_bus; struct clk *pcie_phy; + struct clk *pcie_inbound_axi; struct clk *pcie; struct pcie_port pp; struct regmap *iomuxc_gpr; + bool is_imx6sx; void __iomem *mem_base; u32 tx_deemph_gen1; u32 tx_deemph_gen2_3p5db; @@ -236,13 +238,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); u32 val, gpr1, gpr12; + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); + /* Force PCIe PHY reset */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, + IMX6SX_GPR5_PCIE_BTNRST_RESET); + return 0; + } + /* * If the bootloader already enabled the link we need some special * handling to get the core back into a state where it is safe to * touch it for configuration. As there is no dedicated reset signal - * wired up for MX6QDL, we need to manually force LTSSM into "detect" - * state before completely disabling LTSSM, which is a prerequisite - * for core configuration. + * to manually force LTSSM into "detect" state before completely + * disabling LTSSM, which is a prerequisite for core configuration. * * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong * indication that the bootloader activated the link. @@ -271,6 +283,18 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) { + if (imx6_pcie->is_imx6sx) { + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); + if (ret) { + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); + return ret; + } + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); + return 0; + } + /* power up core phy and enable ref clock */ regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); @@ -324,6 +348,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) msleep(100); gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); } + + if (imx6_pcie->is_imx6sx) + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); + return 0; err_ref_clk: @@ -341,6 +370,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_RX_EQ_MASK, + IMX6SX_GPR12_PCIE_RX_EQ_2); + } + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); @@ -565,6 +600,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) pp = &imx6_pcie->pp; pp->dev = &pdev->dev; + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, + "fsl,imx6sx-pcie"); + /* Added for PCI abort handling */ hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, "imprecise external abort"); @@ -600,6 +638,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) return PTR_ERR(imx6_pcie->pcie); } + if (imx6_pcie->is_imx6sx) { + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, + "pcie_inbound_axi"); + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { + dev_err(&pdev->dev, + "pcie_incbound_axi clock missing or invalid\n"); + return PTR_ERR(imx6_pcie->pcie_inbound_axi); + } + } + /* Grab GPR config register range */ imx6_pcie->iomuxc_gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); @@ -647,6 +695,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) static const struct of_device_id imx6_pcie_of_match[] = { { .compatible = "fsl,imx6q-pcie", }, + { .compatible = "fsl,imx6sx-pcie", }, {}, }; MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); ^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH v3 2/2] PCI: imx6: add initial imx6sx support 2016-03-11 17:58 ` Bjorn Helgaas @ 2016-03-13 23:26 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-13 23:26 UTC (permalink / raw) To: Bjorn Helgaas Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel Hi Bjorn On Fri, 2016-03-11 at 11:58 -0600, Bjorn Helgaas wrote: > Hi Christoph (and Lee; there's a question for you below, too), Thanks for your review, please see my answers and comments below. > On Thu, Feb 25, 2016 at 03:47:49PM +0100, Christoph Fritz wrote: > > This patch adds initial PCIe support for the imx6 SoC derivate imx6sx. > > PCI_MSI support is untested as its necessary suspend/resume quirk is > > left out of this patch. > > This patch is heavily based on patches by Richard Zhu. > > > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > > --- > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > 2 files changed, 96 insertions(+), 40 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > index 6fbba53..c4323be 100644 > > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > @@ -4,7 +4,7 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > > and thus inherits all the common properties defined in designware-pcie.txt. > > > > Required properties: > > -- compatible: "fsl,imx6q-pcie" > > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > > - reg: base addresse and length of the pcie controller > > I folded in a typo fix for "addresse". > > > - interrupts: A list of interrupt outputs of the controller. Must contain an > > entry for each entry in the interrupt-names property. > > @@ -13,6 +13,10 @@ Required properties: > > - clock-names: Must include the following additional entries: > > - "pcie_phy" > > > > +Additional required properties for imx6sx-pcie: > > +- clock names: Must include the following additional entries: > > + - "pcie_inbound_axi" > > + > > Example: > > > > pcie@0x01000000 { > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > > index fe60096..7e634a6 100644 > > --- a/drivers/pci/host/pci-imx6.c > > +++ b/drivers/pci/host/pci-imx6.c > > @@ -35,9 +35,11 @@ struct imx6_pcie { > > struct gpio_desc *reset_gpio; > > struct clk *pcie_bus; > > struct clk *pcie_phy; > > + struct clk *pcie_inbound_axi; > > struct clk *pcie; > > struct pcie_port pp; > > struct regmap *iomuxc_gpr; > > + bool is_imx6sx; > > void __iomem *mem_base; > > }; > > > > @@ -214,35 +216,46 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > u32 val, gpr1, gpr12; > > > > - /* > > - * If the bootloader already enabled the link we need some special > > - * handling to get the core back into a state where it is safe to > > - * touch it for configuration. As there is no dedicated reset signal > > - * wired up for MX6QDL, we need to manually force LTSSM into "detect" > > - * state before completely disabling LTSSM, which is a prerequisite > > - * for core configuration. <snip> > > + } else { > > + /* > > + * If the bootloader already enabled the link we need some > > + * special handling to get the core back into a state where > > + * it is safe to touch it for configuration. As there is no > > + * dedicated reset signal to manually force LTSSM into "detect" > > + * state before completely disabling LTSSM, which is a > > + * prerequisite for core configuration. > > For example, you changed this comment, and now the last sentence is no > longer a sentence. I don't know if this was an editing mistake or if > this comment really does need to change. If it does need to change, > it needs to stay a sentence. I suppose Richard's intend was to get rid of the explicit mentioning of "MX6QDL". Since I don't touch this code I'll drop this change and leave this comment as it is. > > > + * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have > > + * a strong indication that the bootloader activated the link. > > + */ > > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); > > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); > > + > > + if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && > > + (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { > > + val = readl(pp->dbi_base + PCIE_PL_PFLR); > > + val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; > > + val |= PCIE_PL_PFLR_FORCE_LINK; > > + writel(val, pp->dbi_base + PCIE_PL_PFLR); > > + > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > > + IMX6Q_GPR12_PCIE_CTL_2, 0); > > + } > > > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_TEST_PD, 1 << 18); > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 0 << 16); > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_TEST_PD, > > + IMX6Q_GPR1_PCIE_TEST_PD); > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 0); > > And there are subtle changes here (changing "1 << 18" to > IMX6Q_GPR1_PCIE_TEST_PD, etc.) These are fine, but should be in a > separate patch because (a) they're not related to imx6sx and (b) it > makes them obvious and easy to review. As it is, they just get snuck > in under the radar. OK > > > + } > > > > return 0; > > } > > @@ -270,18 +283,30 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > > goto err_pcie; > > } > > > > - /* power up core phy and enable ref clock */ > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > > - /* > > - * the async reset input need ref clock to sync internally, > > - * when the ref clock comes after reset, internal synced > > - * reset time is too short, cannot meet the requirement. > > - * add one ~10us delay here. > > - */ > > - udelay(10); > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > > + if (imx6_pcie->is_imx6sx) { > > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > > + if (ret) { > > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > > + goto err_inbound_axi; > > + } > > + > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > > + } else { > > + /* power up core phy and enable ref clock */ > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_TEST_PD, 0); > > + /* > > + * the async reset input need ref clock to sync internally, > > + * when the ref clock comes after reset, internal synced > > + * reset time is too short , cannot meet the requirement. > > + * add one ~10us delay here. > > + */ > > + udelay(10); > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_REF_CLK_EN, > > + IMX6Q_GPR1_PCIE_REF_CLK_EN); > > + } > > Here again the diff is bigger than necessary because it re-indents the > original code (and makes another unrelated "1 << 16" to > IMX6Q_GPR1_PCIE_REF_CLK_EN change). I'll show a different proposal in > my patch below. OK > > > /* allow the clocks to stabilize */ > > usleep_range(200, 500); > > @@ -292,8 +317,15 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > > msleep(100); > > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > > } > > + > > + if (imx6_pcie->is_imx6sx) > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > > + > > return 0; > > > > +err_inbound_axi: > > + clk_disable_unprepare(imx6_pcie->pcie); > > err_pcie: > > clk_disable_unprepare(imx6_pcie->pcie_bus); > > err_pcie_bus: > > @@ -307,6 +339,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > > { > > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > > > + if (imx6_pcie->is_imx6sx) { > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > > + IMX6SX_GPR12_PCIE_RX_EQ_2); > > I see that you added definitions for IMX6SX_GPR12_PCIE_RX_EQ_MASK and > IMX6SX_GPR12_PCIE_RX_EQ_2 in the first patch of this series, but doing > that separately creates a merge problem. I can't put this change in > my tree by itself because it won't build. If Lee applied the first > patch to a git branch, I guess I could pull that into my tree. But it > would be far simpler to just include those new definitions in this > patch. Sorry, Lee Jones already applied this patch to his public git repository: git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git Commit id: 987480efd6d39447981e96b438cf5a781b756712 currently in branch for-mfd-next Is it okay for you to pull this commit? It's also already in linux-next. <snip> > The patches below are what I currently have in my tree, but they're > not ready for v4.6 (yet). > > Note that the second one is very clearly imx6sx-only. It's obvious > there are no tweaks to the existing imx6 code, which makes it easier > to review, backport, bisect, revert, etc. > > Obviously, I can't apply these directly. First we have to: > > - Fix the LTSSM comment > - Figure out how to get the IMX6SX_GPR12_PCIE_RX_EQ_MASK definitions > > > commit 78abf60b815a8f28207d029d61f7809647faa43a > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri Mar 11 11:15:36 2016 -0600 > > PCI: imx6: Factor out ref clock enable > > Factor out ref clock enable to make it cleaner to add imx6sx support. No > functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 8c9b389..ecc8537 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -269,6 +269,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) <snip> > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + ret = imx6_pcie_enable_ref_clk(imx6_pcie); > + if (ret) a '{ ' is missing > + dev_err(pp->dev, "unable to enable pcie ref clock\n"); > + goto err_ref_clk; > + } > > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -316,6 +326,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > } > return 0; > > +err_ref_clk: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: > > commit d061033570b7fd22a3ed8c029d4793e5df7324e1 > Author: Christoph Fritz <chf.fritz@googlemail.com> > Date: Thu Mar 10 15:04:25 2016 -0600 > > PCI: imx6: Add initial imx6sx support > > Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support > is untested as the necessary suspend/resume quirk is not included in this > patch. > > This patch is heavily based on patches by Richard Zhu. > > [bhelgaas: factor out refclk enable, fix adjacent typos in fsl,imx6q-pcie.txt] > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Acked-by: Lucas Stach <l.stach@pengutronix.de> > <snip> > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index ecc8537..4d6af21 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; <snip> > /* > * If the bootloader already enabled the link we need some special > * handling to get the core back into a state where it is safe to > * touch it for configuration. As there is no dedicated reset signal > - * wired up for MX6QDL, we need to manually force LTSSM into "detect" > - * state before completely disabling LTSSM, which is a prerequisite > - * for core configuration. > + * to manually force LTSSM into "detect" state before completely > + * disabling LTSSM, which is a prerequisite for core configuration. > * > * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong > * indication that the bootloader activated the link. > @@ -271,6 +283,18 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > { this is missing: struct pcie_port *pp = &imx6_pcie->pp; int ret; > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + return ret; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + return 0; then we can use return ret; instead > + } > + > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, <snip> I'll fix these two patches and send them as follow up to this mail. Thanks -- Christoph ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 2/2] PCI: imx6: add initial imx6sx support @ 2016-03-13 23:26 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-13 23:26 UTC (permalink / raw) To: linux-arm-kernel Hi Bjorn On Fri, 2016-03-11 at 11:58 -0600, Bjorn Helgaas wrote: > Hi Christoph (and Lee; there's a question for you below, too), Thanks for your review, please see my answers and comments below. > On Thu, Feb 25, 2016 at 03:47:49PM +0100, Christoph Fritz wrote: > > This patch adds initial PCIe support for the imx6 SoC derivate imx6sx. > > PCI_MSI support is untested as its necessary suspend/resume quirk is > > left out of this patch. > > This patch is heavily based on patches by Richard Zhu. > > > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > > --- > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > 2 files changed, 96 insertions(+), 40 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > index 6fbba53..c4323be 100644 > > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > @@ -4,7 +4,7 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > > and thus inherits all the common properties defined in designware-pcie.txt. > > > > Required properties: > > -- compatible: "fsl,imx6q-pcie" > > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > > - reg: base addresse and length of the pcie controller > > I folded in a typo fix for "addresse". > > > - interrupts: A list of interrupt outputs of the controller. Must contain an > > entry for each entry in the interrupt-names property. > > @@ -13,6 +13,10 @@ Required properties: > > - clock-names: Must include the following additional entries: > > - "pcie_phy" > > > > +Additional required properties for imx6sx-pcie: > > +- clock names: Must include the following additional entries: > > + - "pcie_inbound_axi" > > + > > Example: > > > > pcie at 0x01000000 { > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > > index fe60096..7e634a6 100644 > > --- a/drivers/pci/host/pci-imx6.c > > +++ b/drivers/pci/host/pci-imx6.c > > @@ -35,9 +35,11 @@ struct imx6_pcie { > > struct gpio_desc *reset_gpio; > > struct clk *pcie_bus; > > struct clk *pcie_phy; > > + struct clk *pcie_inbound_axi; > > struct clk *pcie; > > struct pcie_port pp; > > struct regmap *iomuxc_gpr; > > + bool is_imx6sx; > > void __iomem *mem_base; > > }; > > > > @@ -214,35 +216,46 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > u32 val, gpr1, gpr12; > > > > - /* > > - * If the bootloader already enabled the link we need some special > > - * handling to get the core back into a state where it is safe to > > - * touch it for configuration. As there is no dedicated reset signal > > - * wired up for MX6QDL, we need to manually force LTSSM into "detect" > > - * state before completely disabling LTSSM, which is a prerequisite > > - * for core configuration. <snip> > > + } else { > > + /* > > + * If the bootloader already enabled the link we need some > > + * special handling to get the core back into a state where > > + * it is safe to touch it for configuration. As there is no > > + * dedicated reset signal to manually force LTSSM into "detect" > > + * state before completely disabling LTSSM, which is a > > + * prerequisite for core configuration. > > For example, you changed this comment, and now the last sentence is no > longer a sentence. I don't know if this was an editing mistake or if > this comment really does need to change. If it does need to change, > it needs to stay a sentence. I suppose Richard's intend was to get rid of the explicit mentioning of "MX6QDL". Since I don't touch this code I'll drop this change and leave this comment as it is. > > > + * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have > > + * a strong indication that the bootloader activated the link. > > + */ > > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, &gpr1); > > + regmap_read(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, &gpr12); > > + > > + if ((gpr1 & IMX6Q_GPR1_PCIE_REF_CLK_EN) && > > + (gpr12 & IMX6Q_GPR12_PCIE_CTL_2)) { > > + val = readl(pp->dbi_base + PCIE_PL_PFLR); > > + val &= ~PCIE_PL_PFLR_LINK_STATE_MASK; > > + val |= PCIE_PL_PFLR_FORCE_LINK; > > + writel(val, pp->dbi_base + PCIE_PL_PFLR); > > + > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > > + IMX6Q_GPR12_PCIE_CTL_2, 0); > > + } > > > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_TEST_PD, 1 << 18); > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 0 << 16); > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_TEST_PD, > > + IMX6Q_GPR1_PCIE_TEST_PD); > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 0); > > And there are subtle changes here (changing "1 << 18" to > IMX6Q_GPR1_PCIE_TEST_PD, etc.) These are fine, but should be in a > separate patch because (a) they're not related to imx6sx and (b) it > makes them obvious and easy to review. As it is, they just get snuck > in under the radar. OK > > > + } > > > > return 0; > > } > > @@ -270,18 +283,30 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > > goto err_pcie; > > } > > > > - /* power up core phy and enable ref clock */ > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > > - /* > > - * the async reset input need ref clock to sync internally, > > - * when the ref clock comes after reset, internal synced > > - * reset time is too short, cannot meet the requirement. > > - * add one ~10us delay here. > > - */ > > - udelay(10); > > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > > + if (imx6_pcie->is_imx6sx) { > > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > > + if (ret) { > > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > > + goto err_inbound_axi; > > + } > > + > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > > + } else { > > + /* power up core phy and enable ref clock */ > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_TEST_PD, 0); > > + /* > > + * the async reset input need ref clock to sync internally, > > + * when the ref clock comes after reset, internal synced > > + * reset time is too short , cannot meet the requirement. > > + * add one ~10us delay here. > > + */ > > + udelay(10); > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > > + IMX6Q_GPR1_PCIE_REF_CLK_EN, > > + IMX6Q_GPR1_PCIE_REF_CLK_EN); > > + } > > Here again the diff is bigger than necessary because it re-indents the > original code (and makes another unrelated "1 << 16" to > IMX6Q_GPR1_PCIE_REF_CLK_EN change). I'll show a different proposal in > my patch below. OK > > > /* allow the clocks to stabilize */ > > usleep_range(200, 500); > > @@ -292,8 +317,15 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > > msleep(100); > > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > > } > > + > > + if (imx6_pcie->is_imx6sx) > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > > + > > return 0; > > > > +err_inbound_axi: > > + clk_disable_unprepare(imx6_pcie->pcie); > > err_pcie: > > clk_disable_unprepare(imx6_pcie->pcie_bus); > > err_pcie_bus: > > @@ -307,6 +339,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > > { > > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > > > + if (imx6_pcie->is_imx6sx) { > > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > > + IMX6SX_GPR12_PCIE_RX_EQ_2); > > I see that you added definitions for IMX6SX_GPR12_PCIE_RX_EQ_MASK and > IMX6SX_GPR12_PCIE_RX_EQ_2 in the first patch of this series, but doing > that separately creates a merge problem. I can't put this change in > my tree by itself because it won't build. If Lee applied the first > patch to a git branch, I guess I could pull that into my tree. But it > would be far simpler to just include those new definitions in this > patch. Sorry, Lee Jones already applied this patch to his public git repository: git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git Commit id: 987480efd6d39447981e96b438cf5a781b756712 currently in branch for-mfd-next Is it okay for you to pull this commit? It's also already in linux-next. <snip> > The patches below are what I currently have in my tree, but they're > not ready for v4.6 (yet). > > Note that the second one is very clearly imx6sx-only. It's obvious > there are no tweaks to the existing imx6 code, which makes it easier > to review, backport, bisect, revert, etc. > > Obviously, I can't apply these directly. First we have to: > > - Fix the LTSSM comment > - Figure out how to get the IMX6SX_GPR12_PCIE_RX_EQ_MASK definitions > > > commit 78abf60b815a8f28207d029d61f7809647faa43a > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri Mar 11 11:15:36 2016 -0600 > > PCI: imx6: Factor out ref clock enable > > Factor out ref clock enable to make it cleaner to add imx6sx support. No > functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 8c9b389..ecc8537 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -269,6 +269,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) <snip> > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + ret = imx6_pcie_enable_ref_clk(imx6_pcie); > + if (ret) a '{ ' is missing > + dev_err(pp->dev, "unable to enable pcie ref clock\n"); > + goto err_ref_clk; > + } > > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -316,6 +326,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > } > return 0; > > +err_ref_clk: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: > > commit d061033570b7fd22a3ed8c029d4793e5df7324e1 > Author: Christoph Fritz <chf.fritz@googlemail.com> > Date: Thu Mar 10 15:04:25 2016 -0600 > > PCI: imx6: Add initial imx6sx support > > Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support > is untested as the necessary suspend/resume quirk is not included in this > patch. > > This patch is heavily based on patches by Richard Zhu. > > [bhelgaas: factor out refclk enable, fix adjacent typos in fsl,imx6q-pcie.txt] > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Acked-by: Lucas Stach <l.stach@pengutronix.de> > <snip> > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index ecc8537..4d6af21 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; <snip> > /* > * If the bootloader already enabled the link we need some special > * handling to get the core back into a state where it is safe to > * touch it for configuration. As there is no dedicated reset signal > - * wired up for MX6QDL, we need to manually force LTSSM into "detect" > - * state before completely disabling LTSSM, which is a prerequisite > - * for core configuration. > + * to manually force LTSSM into "detect" state before completely > + * disabling LTSSM, which is a prerequisite for core configuration. > * > * If both LTSSM_ENABLE and REF_SSP_ENABLE are active we have a strong > * indication that the bootloader activated the link. > @@ -271,6 +283,18 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > { this is missing: struct pcie_port *pp = &imx6_pcie->pp; int ret; > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + return ret; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + return 0; then we can use return ret; instead > + } > + > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, <snip> I'll fix these two patches and send them as follow up to this mail. Thanks -- Christoph ^ permalink raw reply [flat|nested] 35+ messages in thread
* PCI: imx6: Factor out ref clock enable 2016-03-13 23:26 ` Christoph Fritz (?) @ 2016-03-13 23:30 ` Christoph Fritz 2016-03-20 7:29 ` Christoph Fritz 2016-04-05 21:56 ` Bjorn Helgaas -1 siblings, 2 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-13 23:30 UTC (permalink / raw) To: Bjorn Helgaas Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel commit dfcc1a16e6954e8ca6cadfc9dd309db6b6ef46b2 Author: Bjorn Helgaas <bhelgaas@google.com> Date: Fri, 11 Mar 2016 11:15:36 -0600 Factor out ref clock enable to make it cleaner to add imx6sx support. No functional change intended. Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> Tested-by: Christoph Fritz <chf.fritz@googlemail.com> --- drivers/pci/host/pci-imx6.c | 36 ++++++++++++++++++++++++------------ 1 file changed, 24 insertions(+), 12 deletions(-) diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index bd3f7d0..4e0e47f 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -264,6 +264,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) return 0; } +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) +{ + /* power up core phy and enable ref clock */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); + /* + * the async reset input need ref clock to sync internally, + * when the ref clock comes after reset, internal synced + * reset time is too short, cannot meet the requirement. + * add one ~10us delay here. + */ + udelay(10); + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + return 0; +} + static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); @@ -287,18 +304,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) goto err_pcie; } - /* power up core phy and enable ref clock */ - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); - /* - * the async reset input need ref clock to sync internally, - * when the ref clock comes after reset, internal synced - * reset time is too short, cannot meet the requirement. - * add one ~10us delay here. - */ - udelay(10); - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); + ret = imx6_pcie_enable_ref_clk(imx6_pcie); + if (ret) { + dev_err(pp->dev, "unable to enable pcie ref clock\n"); + goto err_ref_clk; + } /* allow the clocks to stabilize */ usleep_range(200, 500); @@ -311,6 +321,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) } return 0; +err_ref_clk: + clk_disable_unprepare(imx6_pcie->pcie); err_pcie: clk_disable_unprepare(imx6_pcie->pcie_bus); err_pcie_bus: -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: PCI: imx6: Factor out ref clock enable 2016-03-13 23:30 ` PCI: imx6: Factor out ref clock enable Christoph Fritz @ 2016-03-20 7:29 ` Christoph Fritz 2016-04-05 21:56 ` Bjorn Helgaas 1 sibling, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-20 7:29 UTC (permalink / raw) To: Bjorn Helgaas Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel *ping* On Mon, 2016-03-14 at 00:30 +0100, Christoph Fritz wrote: > commit dfcc1a16e6954e8ca6cadfc9dd309db6b6ef46b2 > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri, 11 Mar 2016 11:15:36 -0600 > > Factor out ref clock enable to make it cleaner to add imx6sx support. No > functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > Tested-by: Christoph Fritz <chf.fritz@googlemail.com> > --- > drivers/pci/host/pci-imx6.c | 36 ++++++++++++++++++++++++------------ > 1 file changed, 24 insertions(+), 12 deletions(-) > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index bd3f7d0..4e0e47f 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -264,6 +264,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > return 0; > } > > +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > +{ > + /* power up core phy and enable ref clock */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > + /* > + * the async reset input need ref clock to sync internally, > + * when the ref clock comes after reset, internal synced > + * reset time is too short, cannot meet the requirement. > + * add one ~10us delay here. > + */ > + udelay(10); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + return 0; > +} > + > static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > @@ -287,18 +304,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > goto err_pcie; > } > > - /* power up core phy and enable ref clock */ > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > - /* > - * the async reset input need ref clock to sync internally, > - * when the ref clock comes after reset, internal synced > - * reset time is too short, cannot meet the requirement. > - * add one ~10us delay here. > - */ > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + ret = imx6_pcie_enable_ref_clk(imx6_pcie); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie ref clock\n"); > + goto err_ref_clk; > + } > > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -311,6 +321,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > } > return 0; > > +err_ref_clk: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: ^ permalink raw reply [flat|nested] 35+ messages in thread
* PCI: imx6: Factor out ref clock enable @ 2016-03-20 7:29 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-20 7:29 UTC (permalink / raw) To: linux-arm-kernel *ping* On Mon, 2016-03-14 at 00:30 +0100, Christoph Fritz wrote: > commit dfcc1a16e6954e8ca6cadfc9dd309db6b6ef46b2 > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri, 11 Mar 2016 11:15:36 -0600 > > Factor out ref clock enable to make it cleaner to add imx6sx support. No > functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > Tested-by: Christoph Fritz <chf.fritz@googlemail.com> > --- > drivers/pci/host/pci-imx6.c | 36 ++++++++++++++++++++++++------------ > 1 file changed, 24 insertions(+), 12 deletions(-) > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index bd3f7d0..4e0e47f 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -264,6 +264,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > return 0; > } > > +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > +{ > + /* power up core phy and enable ref clock */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > + /* > + * the async reset input need ref clock to sync internally, > + * when the ref clock comes after reset, internal synced > + * reset time is too short, cannot meet the requirement. > + * add one ~10us delay here. > + */ > + udelay(10); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + return 0; > +} > + > static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > @@ -287,18 +304,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > goto err_pcie; > } > > - /* power up core phy and enable ref clock */ > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > - /* > - * the async reset input need ref clock to sync internally, > - * when the ref clock comes after reset, internal synced > - * reset time is too short, cannot meet the requirement. > - * add one ~10us delay here. > - */ > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + ret = imx6_pcie_enable_ref_clk(imx6_pcie); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie ref clock\n"); > + goto err_ref_clk; > + } > > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -311,6 +321,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > } > return 0; > > +err_ref_clk: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: ^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: PCI: imx6: Factor out ref clock enable 2016-03-13 23:30 ` PCI: imx6: Factor out ref clock enable Christoph Fritz @ 2016-04-05 21:56 ` Bjorn Helgaas 2016-04-05 21:56 ` Bjorn Helgaas 1 sibling, 0 replies; 35+ messages in thread From: Bjorn Helgaas @ 2016-04-05 21:56 UTC (permalink / raw) To: Christoph Fritz Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel Hi Christoph, On Mon, Mar 14, 2016 at 12:30:55AM +0100, Christoph Fritz wrote: > commit dfcc1a16e6954e8ca6cadfc9dd309db6b6ef46b2 > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri, 11 Mar 2016 11:15:36 -0600 > > Factor out ref clock enable to make it cleaner to add imx6sx support. No > functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > Tested-by: Christoph Fritz <chf.fritz@googlemail.com> Applied to pci/host-imx6 for v4.7, thanks! > --- > drivers/pci/host/pci-imx6.c | 36 ++++++++++++++++++++++++------------ > 1 file changed, 24 insertions(+), 12 deletions(-) > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index bd3f7d0..4e0e47f 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -264,6 +264,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > return 0; > } > > +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > +{ > + /* power up core phy and enable ref clock */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > + /* > + * the async reset input need ref clock to sync internally, > + * when the ref clock comes after reset, internal synced > + * reset time is too short, cannot meet the requirement. > + * add one ~10us delay here. > + */ > + udelay(10); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + return 0; > +} > + > static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > @@ -287,18 +304,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > goto err_pcie; > } > > - /* power up core phy and enable ref clock */ > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > - /* > - * the async reset input need ref clock to sync internally, > - * when the ref clock comes after reset, internal synced > - * reset time is too short, cannot meet the requirement. > - * add one ~10us delay here. > - */ > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + ret = imx6_pcie_enable_ref_clk(imx6_pcie); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie ref clock\n"); > + goto err_ref_clk; > + } > > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -311,6 +321,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > } > return 0; > > +err_ref_clk: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: > -- > 2.1.4 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 35+ messages in thread
* PCI: imx6: Factor out ref clock enable @ 2016-04-05 21:56 ` Bjorn Helgaas 0 siblings, 0 replies; 35+ messages in thread From: Bjorn Helgaas @ 2016-04-05 21:56 UTC (permalink / raw) To: linux-arm-kernel Hi Christoph, On Mon, Mar 14, 2016 at 12:30:55AM +0100, Christoph Fritz wrote: > commit dfcc1a16e6954e8ca6cadfc9dd309db6b6ef46b2 > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri, 11 Mar 2016 11:15:36 -0600 > > Factor out ref clock enable to make it cleaner to add imx6sx support. No > functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > Tested-by: Christoph Fritz <chf.fritz@googlemail.com> Applied to pci/host-imx6 for v4.7, thanks! > --- > drivers/pci/host/pci-imx6.c | 36 ++++++++++++++++++++++++------------ > 1 file changed, 24 insertions(+), 12 deletions(-) > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index bd3f7d0..4e0e47f 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -264,6 +264,23 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > return 0; > } > > +static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > +{ > + /* power up core phy and enable ref clock */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > + /* > + * the async reset input need ref clock to sync internally, > + * when the ref clock comes after reset, internal synced > + * reset time is too short, cannot meet the requirement. > + * add one ~10us delay here. > + */ > + udelay(10); > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > + IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + return 0; > +} > + > static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > @@ -287,18 +304,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > goto err_pcie; > } > > - /* power up core phy and enable ref clock */ > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > - /* > - * the async reset input need ref clock to sync internally, > - * when the ref clock comes after reset, internal synced > - * reset time is too short, cannot meet the requirement. > - * add one ~10us delay here. > - */ > - udelay(10); > - regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > - IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16); > + ret = imx6_pcie_enable_ref_clk(imx6_pcie); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie ref clock\n"); > + goto err_ref_clk; > + } > > /* allow the clocks to stabilize */ > usleep_range(200, 500); > @@ -311,6 +321,8 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > } > return 0; > > +err_ref_clk: > + clk_disable_unprepare(imx6_pcie->pcie); > err_pcie: > clk_disable_unprepare(imx6_pcie->pcie_bus); > err_pcie_bus: > -- > 2.1.4 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v4] PCI: imx6: Add initial imx6sx support 2016-03-13 23:26 ` Christoph Fritz @ 2016-03-13 23:31 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-13 23:31 UTC (permalink / raw) To: Bjorn Helgaas Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support is untested as the necessary suspend/resume quirk is not included in this patch. This patch is heavily based on patches by Richard Zhu. [bhelgaas: factor out refclk enable, fix adjacent typos in imx6q-pcie.txt] Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> Acked-by: Richard Zhu <Richard.Zhu@freescale.com> Acked-by: Lucas Stach <l.stach@pengutronix.de> --- .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 8 +++- drivers/pci/host/pci-imx6.c | 53 ++++++++++++++++++++++ 2 files changed, 59 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt index 6fbba53..6ee4439 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP and thus inherits all the common properties defined in designware-pcie.txt. Required properties: -- compatible: "fsl,imx6q-pcie" -- reg: base addresse and length of the pcie controller +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" +- reg: base address and length of the PCIe controller - interrupts: A list of interrupt outputs of the controller. Must contain an entry for each entry in the interrupt-names property. - interrupt-names: Must include the following entries: @@ -13,6 +13,10 @@ Required properties: - clock-names: Must include the following additional entries: - "pcie_phy" +Additional required properties for imx6sx-pcie: +- clock names: Must include the following additional entries: + - "pcie_inbound_axi" + Example: pcie@0x01000000 { diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index 4e0e47f..fa3a544 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -35,9 +35,11 @@ struct imx6_pcie { struct gpio_desc *reset_gpio; struct clk *pcie_bus; struct clk *pcie_phy; + struct clk *pcie_inbound_axi; struct clk *pcie; struct pcie_port pp; struct regmap *iomuxc_gpr; + bool is_imx6sx; void __iomem *mem_base; }; @@ -231,6 +233,17 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); u32 val, gpr1, gpr12; + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); + /* Force PCIe PHY reset */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, + IMX6SX_GPR5_PCIE_BTNRST_RESET); + return 0; + } + /* * If the bootloader already enabled the link we need some special * handling to get the core back into a state where it is safe to @@ -266,6 +279,21 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) { + struct pcie_port *pp = &imx6_pcie->pp; + int ret; + + if (imx6_pcie->is_imx6sx) { + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); + if (ret) { + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); + return ret; + } + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); + return ret; + } + /* power up core phy and enable ref clock */ regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); @@ -319,6 +347,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) msleep(100); gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); } + + if (imx6_pcie->is_imx6sx) + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); + return 0; err_ref_clk: @@ -336,6 +369,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_RX_EQ_MASK, + IMX6SX_GPR12_PCIE_RX_EQ_2); + } + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); @@ -554,6 +593,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) pp = &imx6_pcie->pp; pp->dev = &pdev->dev; + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, + "fsl,imx6sx-pcie"); + /* Added for PCI abort handling */ hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, "imprecise external abort"); @@ -589,6 +631,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) return PTR_ERR(imx6_pcie->pcie); } + if (imx6_pcie->is_imx6sx) { + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, + "pcie_inbound_axi"); + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { + dev_err(&pdev->dev, + "pcie_incbound_axi clock missing or invalid\n"); + return PTR_ERR(imx6_pcie->pcie_inbound_axi); + } + } + /* Grab GPR config register range */ imx6_pcie->iomuxc_gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); @@ -615,6 +667,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) static const struct of_device_id imx6_pcie_of_match[] = { { .compatible = "fsl,imx6q-pcie", }, + { .compatible = "fsl,imx6sx-pcie", }, {}, }; MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* [PATCH v4] PCI: imx6: Add initial imx6sx support @ 2016-03-13 23:31 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-13 23:31 UTC (permalink / raw) To: linux-arm-kernel Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support is untested as the necessary suspend/resume quirk is not included in this patch. This patch is heavily based on patches by Richard Zhu. [bhelgaas: factor out refclk enable, fix adjacent typos in imx6q-pcie.txt] Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> Acked-by: Richard Zhu <Richard.Zhu@freescale.com> Acked-by: Lucas Stach <l.stach@pengutronix.de> --- .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 8 +++- drivers/pci/host/pci-imx6.c | 53 ++++++++++++++++++++++ 2 files changed, 59 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt index 6fbba53..6ee4439 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP and thus inherits all the common properties defined in designware-pcie.txt. Required properties: -- compatible: "fsl,imx6q-pcie" -- reg: base addresse and length of the pcie controller +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" +- reg: base address and length of the PCIe controller - interrupts: A list of interrupt outputs of the controller. Must contain an entry for each entry in the interrupt-names property. - interrupt-names: Must include the following entries: @@ -13,6 +13,10 @@ Required properties: - clock-names: Must include the following additional entries: - "pcie_phy" +Additional required properties for imx6sx-pcie: +- clock names: Must include the following additional entries: + - "pcie_inbound_axi" + Example: pcie at 0x01000000 { diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c index 4e0e47f..fa3a544 100644 --- a/drivers/pci/host/pci-imx6.c +++ b/drivers/pci/host/pci-imx6.c @@ -35,9 +35,11 @@ struct imx6_pcie { struct gpio_desc *reset_gpio; struct clk *pcie_bus; struct clk *pcie_phy; + struct clk *pcie_inbound_axi; struct clk *pcie; struct pcie_port pp; struct regmap *iomuxc_gpr; + bool is_imx6sx; void __iomem *mem_base; }; @@ -231,6 +233,17 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); u32 val, gpr1, gpr12; + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); + /* Force PCIe PHY reset */ + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, + IMX6SX_GPR5_PCIE_BTNRST_RESET); + return 0; + } + /* * If the bootloader already enabled the link we need some special * handling to get the core back into a state where it is safe to @@ -266,6 +279,21 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) { + struct pcie_port *pp = &imx6_pcie->pp; + int ret; + + if (imx6_pcie->is_imx6sx) { + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); + if (ret) { + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); + return ret; + } + + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); + return ret; + } + /* power up core phy and enable ref clock */ regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); @@ -319,6 +347,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) msleep(100); gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); } + + if (imx6_pcie->is_imx6sx) + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); + return 0; err_ref_clk: @@ -336,6 +369,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) { struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); + if (imx6_pcie->is_imx6sx) { + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, + IMX6SX_GPR12_PCIE_RX_EQ_MASK, + IMX6SX_GPR12_PCIE_RX_EQ_2); + } + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); @@ -554,6 +593,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) pp = &imx6_pcie->pp; pp->dev = &pdev->dev; + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, + "fsl,imx6sx-pcie"); + /* Added for PCI abort handling */ hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, "imprecise external abort"); @@ -589,6 +631,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) return PTR_ERR(imx6_pcie->pcie); } + if (imx6_pcie->is_imx6sx) { + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, + "pcie_inbound_axi"); + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { + dev_err(&pdev->dev, + "pcie_incbound_axi clock missing or invalid\n"); + return PTR_ERR(imx6_pcie->pcie_inbound_axi); + } + } + /* Grab GPR config register range */ imx6_pcie->iomuxc_gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); @@ -615,6 +667,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) static const struct of_device_id imx6_pcie_of_match[] = { { .compatible = "fsl,imx6q-pcie", }, + { .compatible = "fsl,imx6sx-pcie", }, {}, }; MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); -- 2.1.4 ^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH v4] PCI: imx6: Add initial imx6sx support 2016-03-13 23:31 ` Christoph Fritz @ 2016-03-20 7:30 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-20 7:30 UTC (permalink / raw) To: Bjorn Helgaas Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel *ping* On Mon, 2016-03-14 at 00:31 +0100, Christoph Fritz wrote: > Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support > is untested as the necessary suspend/resume quirk is not included in this > patch. > > This patch is heavily based on patches by Richard Zhu. > > [bhelgaas: factor out refclk enable, fix adjacent typos in imx6q-pcie.txt] > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Acked-by: Lucas Stach <l.stach@pengutronix.de> > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 8 +++- > drivers/pci/host/pci-imx6.c | 53 ++++++++++++++++++++++ > 2 files changed, 59 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > index 6fbba53..6ee4439 100644 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > and thus inherits all the common properties defined in designware-pcie.txt. > > Required properties: > -- compatible: "fsl,imx6q-pcie" > -- reg: base addresse and length of the pcie controller > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > +- reg: base address and length of the PCIe controller > - interrupts: A list of interrupt outputs of the controller. Must contain an > entry for each entry in the interrupt-names property. > - interrupt-names: Must include the following entries: > @@ -13,6 +13,10 @@ Required properties: > - clock-names: Must include the following additional entries: > - "pcie_phy" > > +Additional required properties for imx6sx-pcie: > +- clock names: Must include the following additional entries: > + - "pcie_inbound_axi" > + > Example: > > pcie@0x01000000 { > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 4e0e47f..fa3a544 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; > + struct clk *pcie_inbound_axi; > struct clk *pcie; > struct pcie_port pp; > struct regmap *iomuxc_gpr; > + bool is_imx6sx; > void __iomem *mem_base; > }; > > @@ -231,6 +233,17 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > u32 val, gpr1, gpr12; > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); > + /* Force PCIe PHY reset */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, > + IMX6SX_GPR5_PCIE_BTNRST_RESET); > + return 0; > + } > + > /* > * If the bootloader already enabled the link we need some special > * handling to get the core back into a state where it is safe to > @@ -266,6 +279,21 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > { > + struct pcie_port *pp = &imx6_pcie->pp; > + int ret; > + > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + return ret; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + return ret; > + } > + > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > @@ -319,6 +347,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > msleep(100); > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > } > + > + if (imx6_pcie->is_imx6sx) > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > + > return 0; > > err_ref_clk: > @@ -336,6 +369,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > + IMX6SX_GPR12_PCIE_RX_EQ_2); > + } > + > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > > @@ -554,6 +593,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > pp = &imx6_pcie->pp; > pp->dev = &pdev->dev; > > + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, > + "fsl,imx6sx-pcie"); > + > /* Added for PCI abort handling */ > hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, > "imprecise external abort"); > @@ -589,6 +631,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > return PTR_ERR(imx6_pcie->pcie); > } > > + if (imx6_pcie->is_imx6sx) { > + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, > + "pcie_inbound_axi"); > + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { > + dev_err(&pdev->dev, > + "pcie_incbound_axi clock missing or invalid\n"); > + return PTR_ERR(imx6_pcie->pcie_inbound_axi); > + } > + } > + > /* Grab GPR config register range */ > imx6_pcie->iomuxc_gpr = > syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); > @@ -615,6 +667,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) > > static const struct of_device_id imx6_pcie_of_match[] = { > { .compatible = "fsl,imx6q-pcie", }, > + { .compatible = "fsl,imx6sx-pcie", }, > {}, > }; > MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v4] PCI: imx6: Add initial imx6sx support @ 2016-03-20 7:30 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-20 7:30 UTC (permalink / raw) To: linux-arm-kernel *ping* On Mon, 2016-03-14 at 00:31 +0100, Christoph Fritz wrote: > Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support > is untested as the necessary suspend/resume quirk is not included in this > patch. > > This patch is heavily based on patches by Richard Zhu. > > [bhelgaas: factor out refclk enable, fix adjacent typos in imx6q-pcie.txt] > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Acked-by: Lucas Stach <l.stach@pengutronix.de> > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 8 +++- > drivers/pci/host/pci-imx6.c | 53 ++++++++++++++++++++++ > 2 files changed, 59 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > index 6fbba53..6ee4439 100644 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > and thus inherits all the common properties defined in designware-pcie.txt. > > Required properties: > -- compatible: "fsl,imx6q-pcie" > -- reg: base addresse and length of the pcie controller > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > +- reg: base address and length of the PCIe controller > - interrupts: A list of interrupt outputs of the controller. Must contain an > entry for each entry in the interrupt-names property. > - interrupt-names: Must include the following entries: > @@ -13,6 +13,10 @@ Required properties: > - clock-names: Must include the following additional entries: > - "pcie_phy" > > +Additional required properties for imx6sx-pcie: > +- clock names: Must include the following additional entries: > + - "pcie_inbound_axi" > + > Example: > > pcie at 0x01000000 { > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 4e0e47f..fa3a544 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; > + struct clk *pcie_inbound_axi; > struct clk *pcie; > struct pcie_port pp; > struct regmap *iomuxc_gpr; > + bool is_imx6sx; > void __iomem *mem_base; > }; > > @@ -231,6 +233,17 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > u32 val, gpr1, gpr12; > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); > + /* Force PCIe PHY reset */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, > + IMX6SX_GPR5_PCIE_BTNRST_RESET); > + return 0; > + } > + > /* > * If the bootloader already enabled the link we need some special > * handling to get the core back into a state where it is safe to > @@ -266,6 +279,21 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > { > + struct pcie_port *pp = &imx6_pcie->pp; > + int ret; > + > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + return ret; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + return ret; > + } > + > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > @@ -319,6 +347,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > msleep(100); > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > } > + > + if (imx6_pcie->is_imx6sx) > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > + > return 0; > > err_ref_clk: > @@ -336,6 +369,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > + IMX6SX_GPR12_PCIE_RX_EQ_2); > + } > + > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > > @@ -554,6 +593,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > pp = &imx6_pcie->pp; > pp->dev = &pdev->dev; > > + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, > + "fsl,imx6sx-pcie"); > + > /* Added for PCI abort handling */ > hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, > "imprecise external abort"); > @@ -589,6 +631,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > return PTR_ERR(imx6_pcie->pcie); > } > > + if (imx6_pcie->is_imx6sx) { > + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, > + "pcie_inbound_axi"); > + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { > + dev_err(&pdev->dev, > + "pcie_incbound_axi clock missing or invalid\n"); > + return PTR_ERR(imx6_pcie->pcie_inbound_axi); > + } > + } > + > /* Grab GPR config register range */ > imx6_pcie->iomuxc_gpr = > syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); > @@ -615,6 +667,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) > > static const struct of_device_id imx6_pcie_of_match[] = { > { .compatible = "fsl,imx6q-pcie", }, > + { .compatible = "fsl,imx6sx-pcie", }, > {}, > }; > MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); ^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [PATCH v4] PCI: imx6: Add initial imx6sx support 2016-03-13 23:31 ` Christoph Fritz @ 2016-04-05 21:56 ` Bjorn Helgaas -1 siblings, 0 replies; 35+ messages in thread From: Bjorn Helgaas @ 2016-04-05 21:56 UTC (permalink / raw) To: Christoph Fritz Cc: Richard Zhu, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel On Mon, Mar 14, 2016 at 12:31:10AM +0100, Christoph Fritz wrote: > Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support > is untested as the necessary suspend/resume quirk is not included in this > patch. > > This patch is heavily based on patches by Richard Zhu. > > [bhelgaas: factor out refclk enable, fix adjacent typos in imx6q-pcie.txt] > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Acked-by: Lucas Stach <l.stach@pengutronix.de> Applied to pci/host-imx6 for v4.7, thanks! > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 8 +++- > drivers/pci/host/pci-imx6.c | 53 ++++++++++++++++++++++ > 2 files changed, 59 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > index 6fbba53..6ee4439 100644 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > and thus inherits all the common properties defined in designware-pcie.txt. > > Required properties: > -- compatible: "fsl,imx6q-pcie" > -- reg: base addresse and length of the pcie controller > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > +- reg: base address and length of the PCIe controller > - interrupts: A list of interrupt outputs of the controller. Must contain an > entry for each entry in the interrupt-names property. > - interrupt-names: Must include the following entries: > @@ -13,6 +13,10 @@ Required properties: > - clock-names: Must include the following additional entries: > - "pcie_phy" > > +Additional required properties for imx6sx-pcie: > +- clock names: Must include the following additional entries: > + - "pcie_inbound_axi" > + > Example: > > pcie@0x01000000 { > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 4e0e47f..fa3a544 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; > + struct clk *pcie_inbound_axi; > struct clk *pcie; > struct pcie_port pp; > struct regmap *iomuxc_gpr; > + bool is_imx6sx; > void __iomem *mem_base; > }; > > @@ -231,6 +233,17 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > u32 val, gpr1, gpr12; > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); > + /* Force PCIe PHY reset */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, > + IMX6SX_GPR5_PCIE_BTNRST_RESET); > + return 0; > + } > + > /* > * If the bootloader already enabled the link we need some special > * handling to get the core back into a state where it is safe to > @@ -266,6 +279,21 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > { > + struct pcie_port *pp = &imx6_pcie->pp; > + int ret; > + > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + return ret; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + return ret; > + } > + > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > @@ -319,6 +347,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > msleep(100); > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > } > + > + if (imx6_pcie->is_imx6sx) > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > + > return 0; > > err_ref_clk: > @@ -336,6 +369,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > + IMX6SX_GPR12_PCIE_RX_EQ_2); > + } > + > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > > @@ -554,6 +593,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > pp = &imx6_pcie->pp; > pp->dev = &pdev->dev; > > + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, > + "fsl,imx6sx-pcie"); > + > /* Added for PCI abort handling */ > hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, > "imprecise external abort"); > @@ -589,6 +631,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > return PTR_ERR(imx6_pcie->pcie); > } > > + if (imx6_pcie->is_imx6sx) { > + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, > + "pcie_inbound_axi"); > + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { > + dev_err(&pdev->dev, > + "pcie_incbound_axi clock missing or invalid\n"); > + return PTR_ERR(imx6_pcie->pcie_inbound_axi); > + } > + } > + > /* Grab GPR config register range */ > imx6_pcie->iomuxc_gpr = > syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); > @@ -615,6 +667,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) > > static const struct of_device_id imx6_pcie_of_match[] = { > { .compatible = "fsl,imx6q-pcie", }, > + { .compatible = "fsl,imx6sx-pcie", }, > {}, > }; > MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); > -- > 2.1.4 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v4] PCI: imx6: Add initial imx6sx support @ 2016-04-05 21:56 ` Bjorn Helgaas 0 siblings, 0 replies; 35+ messages in thread From: Bjorn Helgaas @ 2016-04-05 21:56 UTC (permalink / raw) To: linux-arm-kernel On Mon, Mar 14, 2016 at 12:31:10AM +0100, Christoph Fritz wrote: > Add initial PCIe support for the imx6 SoC derivate imx6sx. PCI MSI support > is untested as the necessary suspend/resume quirk is not included in this > patch. > > This patch is heavily based on patches by Richard Zhu. > > [bhelgaas: factor out refclk enable, fix adjacent typos in imx6q-pcie.txt] > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> > Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Acked-by: Lucas Stach <l.stach@pengutronix.de> Applied to pci/host-imx6 for v4.7, thanks! > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 8 +++- > drivers/pci/host/pci-imx6.c | 53 ++++++++++++++++++++++ > 2 files changed, 59 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > index 6fbba53..6ee4439 100644 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > @@ -4,8 +4,8 @@ This PCIe host controller is based on the Synopsis Designware PCIe IP > and thus inherits all the common properties defined in designware-pcie.txt. > > Required properties: > -- compatible: "fsl,imx6q-pcie" > -- reg: base addresse and length of the pcie controller > +- compatible: "fsl,imx6q-pcie", "fsl,imx6sx-pcie" > +- reg: base address and length of the PCIe controller > - interrupts: A list of interrupt outputs of the controller. Must contain an > entry for each entry in the interrupt-names property. > - interrupt-names: Must include the following entries: > @@ -13,6 +13,10 @@ Required properties: > - clock-names: Must include the following additional entries: > - "pcie_phy" > > +Additional required properties for imx6sx-pcie: > +- clock names: Must include the following additional entries: > + - "pcie_inbound_axi" > + > Example: > > pcie at 0x01000000 { > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 4e0e47f..fa3a544 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -35,9 +35,11 @@ struct imx6_pcie { > struct gpio_desc *reset_gpio; > struct clk *pcie_bus; > struct clk *pcie_phy; > + struct clk *pcie_inbound_axi; > struct clk *pcie; > struct pcie_port pp; > struct regmap *iomuxc_gpr; > + bool is_imx6sx; > void __iomem *mem_base; > }; > > @@ -231,6 +233,17 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > u32 val, gpr1, gpr12; > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN); > + /* Force PCIe PHY reset */ > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, > + IMX6SX_GPR5_PCIE_BTNRST_RESET); > + return 0; > + } > + > /* > * If the bootloader already enabled the link we need some special > * handling to get the core back into a state where it is safe to > @@ -266,6 +279,21 @@ static int imx6_pcie_assert_core_reset(struct pcie_port *pp) > > static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > { > + struct pcie_port *pp = &imx6_pcie->pp; > + int ret; > + > + if (imx6_pcie->is_imx6sx) { > + ret = clk_prepare_enable(imx6_pcie->pcie_inbound_axi); > + if (ret) { > + dev_err(pp->dev, "unable to enable pcie_axi clock\n"); > + return ret; > + } > + > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_TEST_POWERDOWN, 0); > + return ret; > + } > + > /* power up core phy and enable ref clock */ > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1, > IMX6Q_GPR1_PCIE_TEST_PD, 0 << 18); > @@ -319,6 +347,11 @@ static int imx6_pcie_deassert_core_reset(struct pcie_port *pp) > msleep(100); > gpiod_set_value_cansleep(imx6_pcie->reset_gpio, 1); > } > + > + if (imx6_pcie->is_imx6sx) > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR5, > + IMX6SX_GPR5_PCIE_BTNRST_RESET, 0); > + > return 0; > > err_ref_clk: > @@ -336,6 +369,12 @@ static void imx6_pcie_init_phy(struct pcie_port *pp) > { > struct imx6_pcie *imx6_pcie = to_imx6_pcie(pp); > > + if (imx6_pcie->is_imx6sx) { > + regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > + IMX6SX_GPR12_PCIE_RX_EQ_MASK, > + IMX6SX_GPR12_PCIE_RX_EQ_2); > + } > + > regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, > IMX6Q_GPR12_PCIE_CTL_2, 0 << 10); > > @@ -554,6 +593,9 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > pp = &imx6_pcie->pp; > pp->dev = &pdev->dev; > > + imx6_pcie->is_imx6sx = of_device_is_compatible(pp->dev->of_node, > + "fsl,imx6sx-pcie"); > + > /* Added for PCI abort handling */ > hook_fault_code(16 + 6, imx6q_pcie_abort_handler, SIGBUS, 0, > "imprecise external abort"); > @@ -589,6 +631,16 @@ static int __init imx6_pcie_probe(struct platform_device *pdev) > return PTR_ERR(imx6_pcie->pcie); > } > > + if (imx6_pcie->is_imx6sx) { > + imx6_pcie->pcie_inbound_axi = devm_clk_get(&pdev->dev, > + "pcie_inbound_axi"); > + if (IS_ERR(imx6_pcie->pcie_inbound_axi)) { > + dev_err(&pdev->dev, > + "pcie_incbound_axi clock missing or invalid\n"); > + return PTR_ERR(imx6_pcie->pcie_inbound_axi); > + } > + } > + > /* Grab GPR config register range */ > imx6_pcie->iomuxc_gpr = > syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr"); > @@ -615,6 +667,7 @@ static void imx6_pcie_shutdown(struct platform_device *pdev) > > static const struct of_device_id imx6_pcie_of_match[] = { > { .compatible = "fsl,imx6q-pcie", }, > + { .compatible = "fsl,imx6sx-pcie", }, > {}, > }; > MODULE_DEVICE_TABLE(of, imx6_pcie_of_match); > -- > 2.1.4 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [PATCH v3 0/2] Add PCIe driver support for imx6sx 2016-02-25 14:47 ` Christoph Fritz @ 2016-03-03 1:41 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-03 1:41 UTC (permalink / raw) To: Richard Zhu, Shawn Guo, Lucas Stach Cc: Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel *ping* On Thu, 2016-02-25 at 15:47 +0100, Christoph Fritz wrote: > v3: > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > v2: > * leave out regulator handling from driver, will be done later > in PM subsystem > * use BIT() macro for MFD > > Christoph Fritz (2): > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > PCI: imx6: add initial imx6sx support > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > 3 files changed, 101 insertions(+), 40 deletions(-) > ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-03-03 1:41 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-03 1:41 UTC (permalink / raw) To: linux-arm-kernel *ping* On Thu, 2016-02-25 at 15:47 +0100, Christoph Fritz wrote: > v3: > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > v2: > * leave out regulator handling from driver, will be done later > in PM subsystem > * use BIT() macro for MFD > > Christoph Fritz (2): > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > PCI: imx6: add initial imx6sx support > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > 3 files changed, 101 insertions(+), 40 deletions(-) > ^ permalink raw reply [flat|nested] 35+ messages in thread
* RE: [PATCH v3 0/2] Add PCIe driver support for imx6sx 2016-03-03 1:41 ` Christoph Fritz @ 2016-03-03 1:46 ` Richard Zhu -1 siblings, 0 replies; 35+ messages in thread From: Richard Zhu @ 2016-03-03 1:46 UTC (permalink / raw) To: chf.fritz, Shawn Guo, Lucas Stach Cc: Fabio Estevam, Bjorn Helgaas, Lee Jones, linux-pci, linux-arm-kernel PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBDaHJpc3RvcGggRnJpdHogW21h aWx0bzpjaGYuZnJpdHpAZ29vZ2xlbWFpbC5jb21dDQo+IFNlbnQ6IFRodXJzZGF5LCBNYXJjaCAw MywgMjAxNiA5OjQxIEFNDQo+IFRvOiBSaWNoYXJkIFpodTsgU2hhd24gR3VvOyBMdWNhcyBTdGFj aA0KPiBDYzogRmFiaW8gRXN0ZXZhbTsgQmpvcm4gSGVsZ2FhczsgTGVlIEpvbmVzOyBsaW51eC1w Y2lAdmdlci5rZXJuZWwub3JnOyBsaW51eC0NCj4gYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQu b3JnDQo+IFN1YmplY3Q6IFJlOiBbUEFUQ0ggdjMgMC8yXSBBZGQgUENJZSBkcml2ZXIgc3VwcG9y dCBmb3IgaW14NnN4DQo+IA0KPiAqcGluZyoNCj4gDQo+IE9uIFRodSwgMjAxNi0wMi0yNSBhdCAx NTo0NyArMDEwMCwgQ2hyaXN0b3BoIEZyaXR6IHdyb3RlOg0KPiA+IHYzOg0KPiA+ICAqIGFkYXB0 IGRldmljZXRyZWUgYmluZGluZ3MgZG9jdW1lbnRhdGlvbiBmc2wsaW14NnEtcGNpZS50eHQNCj4g PiB2MjoNCj4gPiAgKiBsZWF2ZSBvdXQgcmVndWxhdG9yIGhhbmRsaW5nIGZyb20gZHJpdmVyLCB3 aWxsIGJlIGRvbmUgbGF0ZXINCj4gPiAgICBpbiBQTSBzdWJzeXN0ZW0NCj4gPiAgKiB1c2UgQklU KCkgbWFjcm8gZm9yIE1GRA0KPiA+DQo+ID4gQ2hyaXN0b3BoIEZyaXR6ICgyKToNCj4gPiAgIE1G RDogaW14NnN4OiBBZGQgUENJZSByZWdpc3RlciBkZWZpbml0aW9ucyBmb3IgaW9tdXhjIGdwcg0K PiA+ICAgUENJOiBpbXg2OiBhZGQgaW5pdGlhbCBpbXg2c3ggc3VwcG9ydA0KPiA+DQo+ID4gIC4u Li9kZXZpY2V0cmVlL2JpbmRpbmdzL3BjaS9mc2wsaW14NnEtcGNpZS50eHQgICAgIHwgICA2ICst DQo+ID4gIGRyaXZlcnMvcGNpL2hvc3QvcGNpLWlteDYuYyAgICAgICAgICAgICAgICAgICAgICAg IHwgMTMwICsrKysrKysrKysrKysrLS0tLS0tLQ0KPiA+ICBpbmNsdWRlL2xpbnV4L21mZC9zeXNj b24vaW14NnEtaW9tdXhjLWdwci5oICAgICAgICB8ICAgNSArDQo+ID4gIDMgZmlsZXMgY2hhbmdl ZCwgMTAxIGluc2VydGlvbnMoKyksIDQwIGRlbGV0aW9ucygtKQ0KPiA+DQo+IA0KPiANCltaaHUg aG9uZ3hpbmddIEhpIENocmlzdG9waDoNCkknbSBva2F5IHdpdGggdGhpcyBwYXRjaC1zZXQuIEhv cGUgdGhlIHJlZ3VsYXRvciBoYW5kaW5nIHBhdGNoIG1hbmRhdG9yeSByZXF1aXJlZCBieSBpbXg2 c3ggcGNpZQ0KaXMgbWVyZ2VkIGluIFBNIHN5c3RlbSBhc2FwLiA6KS4NClRoYW5rcy4NClJpY2hh cmQNCg== ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-03-03 1:46 ` Richard Zhu 0 siblings, 0 replies; 35+ messages in thread From: Richard Zhu @ 2016-03-03 1:46 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Christoph Fritz [mailto:chf.fritz at googlemail.com] > Sent: Thursday, March 03, 2016 9:41 AM > To: Richard Zhu; Shawn Guo; Lucas Stach > Cc: Fabio Estevam; Bjorn Helgaas; Lee Jones; linux-pci at vger.kernel.org; linux- > arm-kernel at lists.infradead.org > Subject: Re: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > *ping* > > On Thu, 2016-02-25 at 15:47 +0100, Christoph Fritz wrote: > > v3: > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > v2: > > * leave out regulator handling from driver, will be done later > > in PM subsystem > > * use BIT() macro for MFD > > > > Christoph Fritz (2): > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > PCI: imx6: add initial imx6sx support > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > [Zhu hongxing] Hi Christoph: I'm okay with this patch-set. Hope the regulator handing patch mandatory required by imx6sx pcie is merged in PM system asap. :). Thanks. Richard ^ permalink raw reply [flat|nested] 35+ messages in thread
* RE: [PATCH v3 0/2] Add PCIe driver support for imx6sx 2016-02-25 14:47 ` Christoph Fritz @ 2016-03-03 1:48 ` Richard Zhu -1 siblings, 0 replies; 35+ messages in thread From: Richard Zhu @ 2016-03-03 1:48 UTC (permalink / raw) To: Christoph Fritz, Lucas Stach, Shawn Guo, Fabio Estevam, Bjorn Helgaas, Lee Jones Cc: linux-pci, linux-arm-kernel > -----Original Message----- > From: Christoph Fritz [mailto:chf.fritz@googlemail.com] > Sent: Thursday, February 25, 2016 10:48 PM > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > Jones > Cc: linux-pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > v3: > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > v2: > * leave out regulator handling from driver, will be done later > in PM subsystem > * use BIT() macro for MFD > > Christoph Fritz (2): > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > PCI: imx6: add initial imx6sx support > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > 3 files changed, 101 insertions(+), 40 deletions(-) > > -- > 2.1.4 [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> Richard Best Regards ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-03-03 1:48 ` Richard Zhu 0 siblings, 0 replies; 35+ messages in thread From: Richard Zhu @ 2016-03-03 1:48 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Christoph Fritz [mailto:chf.fritz at googlemail.com] > Sent: Thursday, February 25, 2016 10:48 PM > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > Jones > Cc: linux-pci at vger.kernel.org; linux-arm-kernel at lists.infradead.org > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > v3: > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > v2: > * leave out regulator handling from driver, will be done later > in PM subsystem > * use BIT() macro for MFD > > Christoph Fritz (2): > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > PCI: imx6: add initial imx6sx support > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > 3 files changed, 101 insertions(+), 40 deletions(-) > > -- > 2.1.4 [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> Richard Best Regards ^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [PATCH v3 0/2] Add PCIe driver support for imx6sx 2016-03-03 1:48 ` Richard Zhu @ 2016-03-07 8:20 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-07 8:20 UTC (permalink / raw) To: Bjorn Helgaas, Lee Jones Cc: Lucas Stach, Richard Zhu, Shawn Guo, Fabio Estevam, linux-pci, linux-arm-kernel On Thu, 2016-03-03 at 01:48 +0000, Richard Zhu wrote: > > -----Original Message----- > > From: Christoph Fritz [mailto:chf.fritz@googlemail.com] > > Sent: Thursday, February 25, 2016 10:48 PM > > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > > Jones > > Cc: linux-pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > > > v3: > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > v2: > > * leave out regulator handling from driver, will be done later > > in PM subsystem > > * use BIT() macro for MFD > > > > Christoph Fritz (2): > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > PCI: imx6: add initial imx6sx support > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > -- > > 2.1.4 > > [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Richard > Best Regards @Bjorn: I suppose the PCI patch need to go through your tree. Would you pick up this patch? @Lee: What about the MFD patch, does this need to go through your tree? Thanks -- Christoph ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-03-07 8:20 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-07 8:20 UTC (permalink / raw) To: linux-arm-kernel On Thu, 2016-03-03 at 01:48 +0000, Richard Zhu wrote: > > -----Original Message----- > > From: Christoph Fritz [mailto:chf.fritz at googlemail.com] > > Sent: Thursday, February 25, 2016 10:48 PM > > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > > Jones > > Cc: linux-pci at vger.kernel.org; linux-arm-kernel at lists.infradead.org > > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > > > v3: > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > v2: > > * leave out regulator handling from driver, will be done later > > in PM subsystem > > * use BIT() macro for MFD > > > > Christoph Fritz (2): > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > PCI: imx6: add initial imx6sx support > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > -- > > 2.1.4 > > [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > Richard > Best Regards @Bjorn: I suppose the PCI patch need to go through your tree. Would you pick up this patch? @Lee: What about the MFD patch, does this need to go through your tree? Thanks -- Christoph ^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [PATCH v3 0/2] Add PCIe driver support for imx6sx 2016-03-07 8:20 ` Christoph Fritz @ 2016-03-09 15:39 ` Christoph Fritz -1 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-09 15:39 UTC (permalink / raw) To: Bjorn Helgaas Cc: Lee Jones, Lucas Stach, Richard Zhu, Shawn Guo, Fabio Estevam, linux-pci, linux-arm-kernel On Mon, 2016-03-07 at 09:20 +0100, Christoph Fritz wrote: > On Thu, 2016-03-03 at 01:48 +0000, Richard Zhu wrote: > > > -----Original Message----- > > > From: Christoph Fritz [mailto:chf.fritz@googlemail.com] > > > Sent: Thursday, February 25, 2016 10:48 PM > > > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > > > Jones > > > Cc: linux-pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > > > > > v3: > > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > > v2: > > > * leave out regulator handling from driver, will be done later > > > in PM subsystem > > > * use BIT() macro for MFD > > > > > > Christoph Fritz (2): > > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > > PCI: imx6: add initial imx6sx support > > > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > > > -- > > > 2.1.4 > > > > [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > > Richard > > Best Regards > > @Bjorn: I suppose the PCI patch need to go through your tree. Would you > pick up this patch? *ping* Do you want me to rebase my patch to your pci/host-imx6 branch? Thanks -- Christoph ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-03-09 15:39 ` Christoph Fritz 0 siblings, 0 replies; 35+ messages in thread From: Christoph Fritz @ 2016-03-09 15:39 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2016-03-07 at 09:20 +0100, Christoph Fritz wrote: > On Thu, 2016-03-03 at 01:48 +0000, Richard Zhu wrote: > > > -----Original Message----- > > > From: Christoph Fritz [mailto:chf.fritz at googlemail.com] > > > Sent: Thursday, February 25, 2016 10:48 PM > > > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > > > Jones > > > Cc: linux-pci at vger.kernel.org; linux-arm-kernel at lists.infradead.org > > > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > > > > > v3: > > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > > v2: > > > * leave out regulator handling from driver, will be done later > > > in PM subsystem > > > * use BIT() macro for MFD > > > > > > Christoph Fritz (2): > > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > > PCI: imx6: add initial imx6sx support > > > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > > > -- > > > 2.1.4 > > > > [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > > Richard > > Best Regards > > @Bjorn: I suppose the PCI patch need to go through your tree. Would you > pick up this patch? *ping* Do you want me to rebase my patch to your pci/host-imx6 branch? Thanks -- Christoph ^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [PATCH v3 0/2] Add PCIe driver support for imx6sx 2016-03-09 15:39 ` Christoph Fritz @ 2016-03-09 15:53 ` Lucas Stach -1 siblings, 0 replies; 35+ messages in thread From: Lucas Stach @ 2016-03-09 15:53 UTC (permalink / raw) To: chf.fritz Cc: Bjorn Helgaas, Lee Jones, Richard Zhu, Shawn Guo, Fabio Estevam, linux-pci, linux-arm-kernel Am Mittwoch, den 09.03.2016, 16:39 +0100 schrieb Christoph Fritz: > On Mon, 2016-03-07 at 09:20 +0100, Christoph Fritz wrote: > > On Thu, 2016-03-03 at 01:48 +0000, Richard Zhu wrote: > > > > -----Original Message----- > > > > From: Christoph Fritz [mailto:chf.fritz@googlemail.com] > > > > Sent: Thursday, February 25, 2016 10:48 PM > > > > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > > > > Jones > > > > Cc: linux-pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > > > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > > > > > > > v3: > > > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > > > v2: > > > > * leave out regulator handling from driver, will be done later > > > > in PM subsystem > > > > * use BIT() macro for MFD > > > > > > > > Christoph Fritz (2): > > > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > > > PCI: imx6: add initial imx6sx support > > > > > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > > > > > -- > > > > 2.1.4 > > > > > > [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > > > Richard > > > Best Regards > > > > @Bjorn: I suppose the PCI patch need to go through your tree. Would you > > pick up this patch? > > *ping* > Do you want me to rebase my patch to your pci/host-imx6 branch? > I was unsure if we are going to break DT stability at a later point if we pull this series in without the GPC changes. But after thinking a bit more about this it should be fine. So series is: Acked-by: Lucas Stach <l.stach@pengutronix.de> ^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH v3 0/2] Add PCIe driver support for imx6sx @ 2016-03-09 15:53 ` Lucas Stach 0 siblings, 0 replies; 35+ messages in thread From: Lucas Stach @ 2016-03-09 15:53 UTC (permalink / raw) To: linux-arm-kernel Am Mittwoch, den 09.03.2016, 16:39 +0100 schrieb Christoph Fritz: > On Mon, 2016-03-07 at 09:20 +0100, Christoph Fritz wrote: > > On Thu, 2016-03-03 at 01:48 +0000, Richard Zhu wrote: > > > > -----Original Message----- > > > > From: Christoph Fritz [mailto:chf.fritz at googlemail.com] > > > > Sent: Thursday, February 25, 2016 10:48 PM > > > > To: Richard Zhu; Lucas Stach; Shawn Guo; Fabio Estevam; Bjorn Helgaas; Lee > > > > Jones > > > > Cc: linux-pci at vger.kernel.org; linux-arm-kernel at lists.infradead.org > > > > Subject: [PATCH v3 0/2] Add PCIe driver support for imx6sx > > > > > > > > v3: > > > > * adapt devicetree bindings documentation fsl,imx6q-pcie.txt > > > > v2: > > > > * leave out regulator handling from driver, will be done later > > > > in PM subsystem > > > > * use BIT() macro for MFD > > > > > > > > Christoph Fritz (2): > > > > MFD: imx6sx: Add PCIe register definitions for iomuxc gpr > > > > PCI: imx6: add initial imx6sx support > > > > > > > > .../devicetree/bindings/pci/fsl,imx6q-pcie.txt | 6 +- > > > > drivers/pci/host/pci-imx6.c | 130 ++++++++++++++------- > > > > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 5 + > > > > 3 files changed, 101 insertions(+), 40 deletions(-) > > > > > > > > -- > > > > 2.1.4 > > > > > > [Zhu hongxing] Acked-by: Richard Zhu <Richard.Zhu@freescale.com> > > > Richard > > > Best Regards > > > > @Bjorn: I suppose the PCI patch need to go through your tree. Would you > > pick up this patch? > > *ping* > Do you want me to rebase my patch to your pci/host-imx6 branch? > I was unsure if we are going to break DT stability at a later point if we pull this series in without the GPC changes. But after thinking a bit more about this it should be fine. So series is: Acked-by: Lucas Stach <l.stach@pengutronix.de> ^ permalink raw reply [flat|nested] 35+ messages in thread
end of thread, other threads:[~2016-04-05 21:56 UTC | newest] Thread overview: 35+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-02-25 14:47 [PATCH v3 0/2] Add PCIe driver support for imx6sx Christoph Fritz 2016-02-25 14:47 ` Christoph Fritz 2016-02-25 14:47 ` [PATCH v3 1/2] MFD: imx6sx: Add PCIe register definitions for iomuxc gpr Christoph Fritz 2016-02-25 14:47 ` Christoph Fritz 2016-03-08 4:32 ` Lee Jones 2016-03-08 4:32 ` Lee Jones 2016-02-25 14:47 ` [PATCH v3 2/2] PCI: imx6: add initial imx6sx support Christoph Fritz 2016-02-25 14:47 ` Christoph Fritz 2016-03-11 17:58 ` Bjorn Helgaas 2016-03-11 17:58 ` Bjorn Helgaas 2016-03-13 23:26 ` Christoph Fritz 2016-03-13 23:26 ` Christoph Fritz 2016-03-13 23:30 ` PCI: imx6: Factor out ref clock enable Christoph Fritz 2016-03-20 7:29 ` Christoph Fritz 2016-03-20 7:29 ` Christoph Fritz 2016-04-05 21:56 ` Bjorn Helgaas 2016-04-05 21:56 ` Bjorn Helgaas 2016-03-13 23:31 ` [PATCH v4] PCI: imx6: Add initial imx6sx support Christoph Fritz 2016-03-13 23:31 ` Christoph Fritz 2016-03-20 7:30 ` Christoph Fritz 2016-03-20 7:30 ` Christoph Fritz 2016-04-05 21:56 ` Bjorn Helgaas 2016-04-05 21:56 ` Bjorn Helgaas 2016-03-03 1:41 ` [PATCH v3 0/2] Add PCIe driver support for imx6sx Christoph Fritz 2016-03-03 1:41 ` Christoph Fritz 2016-03-03 1:46 ` Richard Zhu 2016-03-03 1:46 ` Richard Zhu 2016-03-03 1:48 ` Richard Zhu 2016-03-03 1:48 ` Richard Zhu 2016-03-07 8:20 ` Christoph Fritz 2016-03-07 8:20 ` Christoph Fritz 2016-03-09 15:39 ` Christoph Fritz 2016-03-09 15:39 ` Christoph Fritz 2016-03-09 15:53 ` Lucas Stach 2016-03-09 15:53 ` Lucas Stach
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.