> -----Original Message----- > From: Andrey Smirnov [mailto:andrew.smirnov@xxxxxxxxx] > Sent: Monday, December 17, 2018 10:38 AM > To: linux-kernel@xxxxxxxxxxxxxxx > Cc: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>; p.zabel@xxxxxxxxxxxxxx; > Fabio Estevam <fabio.estevam@xxxxxxx>; cphealy@xxxxxxxxx; > l.stach@xxxxxxxxxxxxxx; Leonard Crestez <leonard.crestez@xxxxxxx>; Aisheng > Dong <aisheng.dong@xxxxxxx>; Richard Zhu <hongxing.zhu@xxxxxxx>; Rob > Herring <robh@xxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; dl-linux-imx > <linux-imx@xxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > Subject: [PATCH v3 3/3] reset: imx7: Add support for i.MX8MQ IP block variant > > Add bits and pieces needed to support IP block variant found on i.MX8MQ > SoCs. > > Cc: p.zabel@xxxxxxxxxxxxxx > Cc: Fabio Estevam <fabio.estevam@xxxxxxx> > Cc: cphealy@xxxxxxxxx > Cc: l.stach@xxxxxxxxxxxxxx > Cc: Leonard Crestez <leonard.crestez@xxxxxxx> > Cc: "A.s. Dong" <aisheng.dong@xxxxxxx> > Cc: Richard Zhu <hongxing.zhu@xxxxxxx> > Cc: Rob Herring <robh@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > Cc: linux-imx@xxxxxxx > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > Cc: linux-kernel@xxxxxxxxxxxxxxx > Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> > --- > drivers/reset/Kconfig | 2 +- > drivers/reset/reset-imx7.c | 106 > +++++++++++++++++++++++++++++++++++++ > 2 files changed, 107 insertions(+), 1 deletion(-) > > diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig index > c21da9fe51ec..4909aab7401b 100644 > --- a/drivers/reset/Kconfig > +++ b/drivers/reset/Kconfig > @@ -50,7 +50,7 @@ config RESET_HSDK > config RESET_IMX7 > bool "i.MX7 Reset Driver" if COMPILE_TEST > depends on HAS_IOMEM > - default SOC_IMX7D > + default SOC_IMX7D || SOC_IMX8MQ SOC_IMX8MQ has been removed in Shawn's tree. I'd suggest simply using ARCH_MXC. Regards Dong Aisheng > select MFD_SYSCON > help > This enables the reset controller driver for i.MX7 SoCs. > diff --git a/drivers/reset/reset-imx7.c b/drivers/reset/reset-imx7.c index > 3a36d5863891..bb826935db6d 100644 > --- a/drivers/reset/reset-imx7.c > +++ b/drivers/reset/reset-imx7.c > @@ -22,6 +22,7 @@ > #include <linux/reset-controller.h> > #include <linux/regmap.h> > #include <dt-bindings/reset/imx7-reset.h> > +#include <dt-bindings/reset/imx8mq-reset.h> > > struct imx7_src_signal { > unsigned int offset, bit; > @@ -113,6 +114,110 @@ static const struct imx7_src_variant variant_imx7 = > { > .prepare = imx7_src_prepare, > }; > > +enum imx8mq_src_registers { > + SRC_A53RCR0 = 0x0004, > + SRC_HDMI_RCR = 0x0030, > + SRC_DISP_RCR = 0x0034, > + SRC_GPU_RCR = 0x0040, > + SRC_VPU_RCR = 0x0044, > + SRC_PCIE2_RCR = 0x0048, > + SRC_MIPIPHY1_RCR = 0x004c, > + SRC_MIPIPHY2_RCR = 0x0050, > + SRC_DDRC2_RCR = 0x1004, > +}; > + > +static const struct imx7_src_signal > imx8mq_src_signals[IMX8MQ_RESET_NUM] = { > + [IMX8MQ_RESET_A53_CORE_POR_RESET0] = { SRC_A53RCR0, BIT(0) }, > + [IMX8MQ_RESET_A53_CORE_POR_RESET1] = { SRC_A53RCR0, BIT(1) }, > + [IMX8MQ_RESET_A53_CORE_POR_RESET2] = { SRC_A53RCR0, BIT(2) }, > + [IMX8MQ_RESET_A53_CORE_POR_RESET3] = { SRC_A53RCR0, BIT(3) }, > + [IMX8MQ_RESET_A53_CORE_RESET0] = { SRC_A53RCR0, BIT(4) }, > + [IMX8MQ_RESET_A53_CORE_RESET1] = { SRC_A53RCR0, BIT(5) }, > + [IMX8MQ_RESET_A53_CORE_RESET2] = { SRC_A53RCR0, BIT(6) }, > + [IMX8MQ_RESET_A53_CORE_RESET3] = { SRC_A53RCR0, BIT(7) }, > + [IMX8MQ_RESET_A53_DBG_RESET0] = { SRC_A53RCR0, BIT(8) }, > + [IMX8MQ_RESET_A53_DBG_RESET1] = { SRC_A53RCR0, BIT(9) }, > + [IMX8MQ_RESET_A53_DBG_RESET2] = { SRC_A53RCR0, BIT(10) }, > + [IMX8MQ_RESET_A53_DBG_RESET3] = { SRC_A53RCR0, BIT(11) }, > + [IMX8MQ_RESET_A53_ETM_RESET0] = { SRC_A53RCR0, BIT(12) }, > + [IMX8MQ_RESET_A53_ETM_RESET1] = { SRC_A53RCR0, BIT(13) }, > + [IMX8MQ_RESET_A53_ETM_RESET2] = { SRC_A53RCR0, BIT(14) }, > + [IMX8MQ_RESET_A53_ETM_RESET3] = { SRC_A53RCR0, BIT(15) }, > + [IMX8MQ_RESET_A53_SOC_DBG_RESET] = { SRC_A53RCR0, BIT(20) }, > + [IMX8MQ_RESET_A53_L2RESET] = { SRC_A53RCR0, BIT(21) }, > + [IMX8MQ_RESET_SW_NON_SCLR_M4C_RST] = { SRC_M4RCR, BIT(0) }, > + [IMX8MQ_RESET_OTG1_PHY_RESET] = { SRC_USBOPHY1_RCR, > BIT(0) }, > + [IMX8MQ_RESET_OTG2_PHY_RESET] = { SRC_USBOPHY2_RCR, > BIT(0) }, > + [IMX8MQ_RESET_MIPI_DSI_RESET_BYTE_N] = { SRC_MIPIPHY_RCR, > BIT(1) }, > + [IMX8MQ_RESET_MIPI_DSI_RESET_N] = { SRC_MIPIPHY_RCR, > BIT(2) }, > + [IMX8MQ_RESET_MIPI_DIS_DPI_RESET_N] = { SRC_MIPIPHY_RCR, > BIT(3) }, > + [IMX8MQ_RESET_MIPI_DIS_ESC_RESET_N] = { SRC_MIPIPHY_RCR, > BIT(4) }, > + [IMX8MQ_RESET_MIPI_DIS_PCLK_RESET_N] = { SRC_MIPIPHY_RCR, > BIT(5) }, > + [IMX8MQ_RESET_PCIEPHY] = { SRC_PCIEPHY_RCR, > + BIT(2) | BIT(1) }, > + [IMX8MQ_RESET_PCIEPHY_PERST] = { SRC_PCIEPHY_RCR, > BIT(3) }, > + [IMX8MQ_RESET_PCIE_CTRL_APPS_EN] = { SRC_PCIEPHY_RCR, > BIT(6) }, > + [IMX8MQ_RESET_PCIE_CTRL_APPS_TURNOFF] = { SRC_PCIEPHY_RCR, > BIT(11) }, > + [IMX8MQ_RESET_HDMI_PHY_APB_RESET] = { SRC_HDMI_RCR, BIT(0) }, > + [IMX8MQ_RESET_DISP_RESET] = { SRC_DISP_RCR, BIT(0) }, > + [IMX8MQ_RESET_GPU_RESET] = { SRC_GPU_RCR, BIT(0) }, > + [IMX8MQ_RESET_VPU_RESET] = { SRC_VPU_RCR, BIT(0) }, > + [IMX8MQ_RESET_PCIEPHY2] = { SRC_PCIE2_RCR, > + BIT(2) | BIT(1) }, > + [IMX8MQ_RESET_PCIEPHY2_PERST] = { SRC_PCIE2_RCR, BIT(3) }, > + [IMX8MQ_RESET_PCIE2_CTRL_APPS_EN] = { SRC_PCIE2_RCR, BIT(6) }, > + [IMX8MQ_RESET_PCIE2_CTRL_APPS_TURNOFF] = { SRC_PCIE2_RCR, > BIT(11) }, > + [IMX8MQ_RESET_MIPI_CSI1_CORE_RESET] = { SRC_MIPIPHY1_RCR, > BIT(0) }, > + [IMX8MQ_RESET_MIPI_CSI1_PHY_REF_RESET] = { SRC_MIPIPHY1_RCR, > BIT(1) }, > + [IMX8MQ_RESET_MIPI_CSI1_ESC_RESET] = { SRC_MIPIPHY1_RCR, > BIT(2) }, > + [IMX8MQ_RESET_MIPI_CSI2_CORE_RESET] = { SRC_MIPIPHY2_RCR, > BIT(0) }, > + [IMX8MQ_RESET_MIPI_CSI2_PHY_REF_RESET] = { SRC_MIPIPHY2_RCR, > BIT(1) }, > + [IMX8MQ_RESET_MIPI_CSI2_ESC_RESET] = { SRC_MIPIPHY2_RCR, > BIT(2) }, > + [IMX8MQ_RESET_DDRC1_PRST] = { SRC_DDRC_RCR, BIT(0) }, > + [IMX8MQ_RESET_DDRC1_CORE_RESET] = { SRC_DDRC_RCR, > BIT(1) }, > + [IMX8MQ_RESET_DDRC1_PHY_RESET] = { SRC_DDRC_RCR, BIT(2) }, > + [IMX8MQ_RESET_DDRC2_PHY_RESET] = { SRC_DDRC2_RCR, BIT(0) }, > + [IMX8MQ_RESET_DDRC2_CORE_RESET] = { SRC_DDRC2_RCR, > BIT(1) }, > + [IMX8MQ_RESET_DDRC2_PRST] = { SRC_DDRC2_RCR, BIT(2) }, > +}; > + > +static unsigned int > +imx8mq_src_prepare(struct imx7_src *imx7src, unsigned long id, bool > +assert) { > + const unsigned int bit = imx7src->variant->signals[id].bit; > + unsigned int value = assert ? bit : 0; > + > + switch (id) { > + case IMX8MQ_RESET_PCIEPHY: > + case IMX8MQ_RESET_PCIEPHY2: /* fallthrough */ > + /* > + * wait for more than 10us to release phy g_rst and > + * btnrst > + */ > + if (!assert) > + udelay(10); > + break; > + > + case IMX8MQ_RESET_PCIE_CTRL_APPS_EN: > + case IMX8MQ_RESET_PCIE2_CTRL_APPS_EN: /* fallthrough */ > + case IMX8MQ_RESET_MIPI_DIS_PCLK_RESET_N: /* fallthrough */ > + case IMX8MQ_RESET_MIPI_DIS_ESC_RESET_N: /* fallthrough */ > + case IMX8MQ_RESET_MIPI_DIS_DPI_RESET_N: /* fallthrough */ > + case IMX8MQ_RESET_MIPI_DSI_RESET_N: /* fallthrough */ > + case IMX8MQ_RESET_MIPI_DSI_RESET_BYTE_N: /* fallthrough */ > + value = assert ? 0 : bit; > + break; > + } > + > + return value; > +} > + > +static const struct imx7_src_variant variant_imx8mq = { > + .signals = imx8mq_src_signals, > + .signals_num = ARRAY_SIZE(imx8mq_src_signals), > + .prepare = imx8mq_src_prepare, > +}; > + > static struct imx7_src *to_imx7_src(struct reset_controller_dev *rcdev) { > return container_of(rcdev, struct imx7_src, rcdev); @@ -175,6 +280,7 > @@ static int imx7_reset_probe(struct platform_device *pdev) > > static const struct of_device_id imx7_reset_dt_ids[] = { > { .compatible = "fsl,imx7d-src", .data = &variant_imx7 }, > + { .compatible = "fsl,imx8mq-src", .data = &variant_imx8mq }, > { /* sentinel */ }, > }; > > -- > 2.19.1