From: Abel Vesa <abel.vesa@nxp.com>
To: Jacky Bai <ping.bai@nxp.com>
Cc: "shawnguo@kernel.org" <shawnguo@kernel.org>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"sboyd@kernel.org" <sboyd@kernel.org>,
"s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
"kernel@pengutronix.de" <kernel@pengutronix.de>,
dl-linux-imx <linux-imx@nxp.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH v3 1/9] dt-bindings: clock: Add imx8ulp clock support
Date: Tue, 14 Sep 2021 18:30:47 +0300 [thread overview]
Message-ID: <YUDAJ18URdZRGIup@ryzen> (raw)
In-Reply-To: <DBBPR04MB7930561625265F08CD3B630887DA9@DBBPR04MB7930.eurprd04.prod.outlook.com>
On 21-09-14 16:11:13, Jacky Bai wrote:
> > Subject: Re: [PATCH v3 1/9] dt-bindings: clock: Add imx8ulp clock support
> >
> > On 21-09-14 14:52:00, Jacky Bai wrote:
> > > Add the clock dt-binding file for i.MX8ULP.
> > >
> > > For pcc node, it will also be used as a reset controller, so add the
> > > '#reset-cells' property description and add the pcc reset IDs.
> > >
> > > Signed-off-by: Jacky Bai <ping.bai@nxp.com>
> > > ---
> > > v3 changes:
> > > - split into two binding file, one for pcc, one for cgc
> > >
> > > v2 changes:
> > > - removed the redundant clocks & clock-names property
> > >
> > > v1 changes:
> > > - Move this patch from dts patchset into this patchset
> > > ---
> > > .../bindings/clock/imx8ulp-cgc-clock.yaml | 43 +++
> > > .../bindings/clock/imx8ulp-pcc-clock.yaml | 50 ++++
> > > include/dt-bindings/clock/imx8ulp-clock.h | 258
> > ++++++++++++++++++
> > > include/dt-bindings/reset/imx8ulp-pcc-reset.h | 59 ++++
> > > 4 files changed, 410 insertions(+)
> > > create mode 100644
> > > Documentation/devicetree/bindings/clock/imx8ulp-cgc-clock.yaml
> > > create mode 100644
> > > Documentation/devicetree/bindings/clock/imx8ulp-pcc-clock.yaml
> > > create mode 100644 include/dt-bindings/clock/imx8ulp-clock.h
> > > create mode 100644 include/dt-bindings/reset/imx8ulp-pcc-reset.h
> > >
> > > diff --git
> > > a/Documentation/devicetree/bindings/clock/imx8ulp-cgc-clock.yaml
> > > b/Documentation/devicetree/bindings/clock/imx8ulp-cgc-clock.yaml
> >
> > I'm not sure but I think the Documentation part is usually done in a separate
> > patch.
> >
>
> The clk-imx8ulp.c file highly depends on the clock/reset index in this dt-binding patch.
>
Yeah, I get that, but I was suggesting splitting the Documentation part
and the binding header part into two separate patches. It's up to you.
For everything else:
Reviewed-by: Abel Vesa <abel.vesa@nxp.com>
> BR
> Jacky Bai
>
> > > new file mode 100644
> > > index 000000000000..71f7186b135b
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/clock/imx8ulp-cgc-clock.yaml
> > > @@ -0,0 +1,43 @@
> > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > > +---
> > > +$id: http://devicetree.org/schemas/clock/imx8ulp-cgc-clock.yaml#
> > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > +
> > > +title: NXP i.MX8ULP Clock Generation & Control(CGC) Module Binding
> > > +
> > > +maintainers:
> > > + - Jacky Bai <ping.bai@nxp.com>
> > > +
> > > +description: |
> > > + On i.MX8ULP, The clock sources generation, distribution and
> > > +management is
> > > + under the control of several CGCs & PCCs modules. The CGC modules
> > > +generate
> > > + and distribute clocks on the device.
> > > +
> > > +properties:
> > > + compatible:
> > > + enum:
> > > + - fsl,imx8ulp-cgc1
> > > + - fsl,imx8ulp-cgc2
> > > +
> > > + reg:
> > > + maxItems: 1
> > > +
> > > + '#clock-cells':
> > > + const: 1
> > > +
> > > +required:
> > > + - compatible
> > > + - reg
> > > + - '#clock-cells'
> > > +
> > > +additionalProperties: false
> > > +
> > > +examples:
> > > + # Clock Generation & Control Module node:
> > > + - |
> > > + clock-controller@292c0000 {
> > > + compatible = "fsl,imx8ulp-cgc1";
> > > + reg = <0x292c0000 0x10000>;
> > > + #clock-cells = <1>;
> > > + };
> > > diff --git
> > > a/Documentation/devicetree/bindings/clock/imx8ulp-pcc-clock.yaml
> > > b/Documentation/devicetree/bindings/clock/imx8ulp-pcc-clock.yaml
> > > new file mode 100644
> > > index 000000000000..00612725bf8b
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/clock/imx8ulp-pcc-clock.yaml
> > > @@ -0,0 +1,50 @@
> > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > > +---
> > > +$id: http://devicetree.org/schemas/clock/imx8ulp-pcc-clock.yaml#
> > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > +
> > > +title: NXP i.MX8ULP Peripheral Clock Controller(PCC) Module Binding
> > > +
> > > +maintainers:
> > > + - Jacky Bai <ping.bai@nxp.com>
> > > +
> > > +description: |
> > > + On i.MX8ULP, The clock sources generation, distribution and
> > > +management is
> > > + under the control of several CGCs & PCCs modules. The PCC modules
> > > +control
> > > + software reset, clock selection, optional division and clock gating
> > > +mode
> > > + for peripherals.
> > > +
> > > +properties:
> > > + compatible:
> > > + enum:
> > > + - fsl,imx8ulp-pcc3
> > > + - fsl,imx8ulp-pcc4
> > > + - fsl,imx8ulp-pcc5
> > > +
> > > + reg:
> > > + maxItems: 1
> > > +
> > > + '#clock-cells':
> > > + const: 1
> > > +
> > > + '#reset-cells':
> > > + const: 1
> > > +
> > > +required:
> > > + - compatible
> > > + - reg
> > > + - '#clock-cells'
> > > + - '#reset-cells'
> > > +
> > > +additionalProperties: false
> > > +
> > > +examples:
> > > + # Peripheral Clock Control Module node:
> > > + - |
> > > + clock-controller@292d0000 {
> > > + compatible = "fsl,imx8ulp-pcc3";
> > > + reg = <0x292d0000 0x10000>;
> > > + #clock-cells = <1>;
> > > + #reset-cells = <1>;
> > > + };
> > > diff --git a/include/dt-bindings/clock/imx8ulp-clock.h
> > > b/include/dt-bindings/clock/imx8ulp-clock.h
> > > new file mode 100644
> > > index 000000000000..953ecfe8ebcc
> > > --- /dev/null
> > > +++ b/include/dt-bindings/clock/imx8ulp-clock.h
> > > @@ -0,0 +1,258 @@
> > > +/* SPDX-License-Identifier: GPL-2.0+ OR MIT */
> > > +/*
> > > + * Copyright 2021 NXP
> > > + */
> > > +
> > > +#ifndef __DT_BINDINGS_CLOCK_IMX8ULP_H #define
> > > +__DT_BINDINGS_CLOCK_IMX8ULP_H
> > > +
> > > +#define IMX8ULP_CLK_DUMMY 0
> > > +
> > > +/* CGC1 */
> > > +#define IMX8ULP_CLK_SPLL2 5
> > > +#define IMX8ULP_CLK_SPLL3 6
> > > +#define IMX8ULP_CLK_A35_SEL 7
> > > +#define IMX8ULP_CLK_A35_DIV 8
> > > +#define IMX8ULP_CLK_SPLL2_PRE_SEL 9
> > > +#define IMX8ULP_CLK_SPLL3_PRE_SEL 10
> > > +#define IMX8ULP_CLK_SPLL3_PFD0 11
> > > +#define IMX8ULP_CLK_SPLL3_PFD1 12
> > > +#define IMX8ULP_CLK_SPLL3_PFD2 13
> > > +#define IMX8ULP_CLK_SPLL3_PFD3 14
> > > +#define IMX8ULP_CLK_SPLL3_PFD0_DIV1 15
> > > +#define IMX8ULP_CLK_SPLL3_PFD0_DIV2 16
> > > +#define IMX8ULP_CLK_SPLL3_PFD1_DIV1 17
> > > +#define IMX8ULP_CLK_SPLL3_PFD1_DIV2 18
> > > +#define IMX8ULP_CLK_SPLL3_PFD2_DIV1 19
> > > +#define IMX8ULP_CLK_SPLL3_PFD2_DIV2 20
> > > +#define IMX8ULP_CLK_SPLL3_PFD3_DIV1 21
> > > +#define IMX8ULP_CLK_SPLL3_PFD3_DIV2 22
> > > +#define IMX8ULP_CLK_NIC_SEL 23
> > > +#define IMX8ULP_CLK_NIC_AD_DIVPLAT 24
> > > +#define IMX8ULP_CLK_NIC_PER_DIVPLAT 25
> > > +#define IMX8ULP_CLK_XBAR_SEL 26
> > > +#define IMX8ULP_CLK_XBAR_AD_DIVPLAT 27
> > > +#define IMX8ULP_CLK_XBAR_DIVBUS 28
> > > +#define IMX8ULP_CLK_XBAR_AD_SLOW 29
> > > +#define IMX8ULP_CLK_SOSC_DIV1 30
> > > +#define IMX8ULP_CLK_SOSC_DIV2 31
> > > +#define IMX8ULP_CLK_SOSC_DIV3 32
> > > +#define IMX8ULP_CLK_FROSC_DIV1 33
> > > +#define IMX8ULP_CLK_FROSC_DIV2 34
> > > +#define IMX8ULP_CLK_FROSC_DIV3 35
> > > +#define IMX8ULP_CLK_SPLL3_VCODIV 36
> > > +#define IMX8ULP_CLK_SPLL3_PFD0_DIV1_GATE 37
> > > +#define IMX8ULP_CLK_SPLL3_PFD0_DIV2_GATE 38
> > > +#define IMX8ULP_CLK_SPLL3_PFD1_DIV1_GATE 39
> > > +#define IMX8ULP_CLK_SPLL3_PFD1_DIV2_GATE 40
> > > +#define IMX8ULP_CLK_SPLL3_PFD2_DIV1_GATE 41
> > > +#define IMX8ULP_CLK_SPLL3_PFD2_DIV2_GATE 42
> > > +#define IMX8ULP_CLK_SPLL3_PFD3_DIV1_GATE 43
> > > +#define IMX8ULP_CLK_SPLL3_PFD3_DIV2_GATE 44
> > > +#define IMX8ULP_CLK_SOSC_DIV1_GATE 45
> > > +#define IMX8ULP_CLK_SOSC_DIV2_GATE 46
> > > +#define IMX8ULP_CLK_SOSC_DIV3_GATE 47
> > > +#define IMX8ULP_CLK_FROSC_DIV1_GATE 48
> > > +#define IMX8ULP_CLK_FROSC_DIV2_GATE 49
> > > +#define IMX8ULP_CLK_FROSC_DIV3_GATE 50
> > > +#define IMX8ULP_CLK_SAI4_SEL 51
> > > +#define IMX8ULP_CLK_SAI5_SEL 52
> > > +#define IMX8ULP_CLK_AUD_CLK1 53
> > > +#define IMX8ULP_CLK_ARM 54
> > > +#define IMX8ULP_CLK_ENET_TS_SEL 55
> > > +
> > > +#define IMX8ULP_CLK_CGC1_END 56
> > > +
> > > +/* CGC2 */
> > > +#define IMX8ULP_CLK_PLL4_PRE_SEL 0
> > > +#define IMX8ULP_CLK_PLL4 1
> > > +#define IMX8ULP_CLK_PLL4_VCODIV 2
> > > +#define IMX8ULP_CLK_DDR_SEL 3
> > > +#define IMX8ULP_CLK_DDR_DIV 4
> > > +#define IMX8ULP_CLK_LPAV_AXI_SEL 5
> > > +#define IMX8ULP_CLK_LPAV_AXI_DIV 6
> > > +#define IMX8ULP_CLK_LPAV_AHB_DIV 7
> > > +#define IMX8ULP_CLK_LPAV_BUS_DIV 8
> > > +#define IMX8ULP_CLK_PLL4_PFD0 9
> > > +#define IMX8ULP_CLK_PLL4_PFD1 10
> > > +#define IMX8ULP_CLK_PLL4_PFD2 11
> > > +#define IMX8ULP_CLK_PLL4_PFD3 12
> > > +#define IMX8ULP_CLK_PLL4_PFD0_DIV1_GATE 13
> > > +#define IMX8ULP_CLK_PLL4_PFD0_DIV2_GATE 14
> > > +#define IMX8ULP_CLK_PLL4_PFD1_DIV1_GATE 15
> > > +#define IMX8ULP_CLK_PLL4_PFD1_DIV2_GATE 16
> > > +#define IMX8ULP_CLK_PLL4_PFD2_DIV1_GATE 17
> > > +#define IMX8ULP_CLK_PLL4_PFD2_DIV2_GATE 18
> > > +#define IMX8ULP_CLK_PLL4_PFD3_DIV1_GATE 19
> > > +#define IMX8ULP_CLK_PLL4_PFD3_DIV2_GATE 20
> > > +#define IMX8ULP_CLK_PLL4_PFD0_DIV1 21
> > > +#define IMX8ULP_CLK_PLL4_PFD0_DIV2 22
> > > +#define IMX8ULP_CLK_PLL4_PFD1_DIV1 23
> > > +#define IMX8ULP_CLK_PLL4_PFD1_DIV2 24
> > > +#define IMX8ULP_CLK_PLL4_PFD2_DIV1 25
> > > +#define IMX8ULP_CLK_PLL4_PFD2_DIV2 26
> > > +#define IMX8ULP_CLK_PLL4_PFD3_DIV1 27
> > > +#define IMX8ULP_CLK_PLL4_PFD3_DIV2 28
> > > +#define IMX8ULP_CLK_CGC2_SOSC_DIV1_GATE 29
> > > +#define IMX8ULP_CLK_CGC2_SOSC_DIV2_GATE 30
> > > +#define IMX8ULP_CLK_CGC2_SOSC_DIV3_GATE 31
> > > +#define IMX8ULP_CLK_CGC2_SOSC_DIV1 32
> > > +#define IMX8ULP_CLK_CGC2_SOSC_DIV2 33
> > > +#define IMX8ULP_CLK_CGC2_SOSC_DIV3 34
> > > +#define IMX8ULP_CLK_CGC2_FROSC_DIV1_GATE 35
> > > +#define IMX8ULP_CLK_CGC2_FROSC_DIV2_GATE 36
> > > +#define IMX8ULP_CLK_CGC2_FROSC_DIV3_GATE 37
> > > +#define IMX8ULP_CLK_CGC2_FROSC_DIV1 38
> > > +#define IMX8ULP_CLK_CGC2_FROSC_DIV2 39
> > > +#define IMX8ULP_CLK_CGC2_FROSC_DIV3 40
> > > +#define IMX8ULP_CLK_AUD_CLK2 41
> > > +#define IMX8ULP_CLK_SAI6_SEL 42
> > > +#define IMX8ULP_CLK_SAI7_SEL 43
> > > +#define IMX8ULP_CLK_SPDIF_SEL 44
> > > +#define IMX8ULP_CLK_HIFI_SEL 45
> > > +#define IMX8ULP_CLK_HIFI_DIVCORE 46
> > > +#define IMX8ULP_CLK_HIFI_DIVPLAT 47
> > > +#define IMX8ULP_CLK_DSI_PHY_REF 48
> > > +
> > > +#define IMX8ULP_CLK_CGC2_END 49
> > > +
> > > +/* PCC3 */
> > > +#define IMX8ULP_CLK_WDOG3 0
> > > +#define IMX8ULP_CLK_WDOG4 1
> > > +#define IMX8ULP_CLK_LPIT1 2
> > > +#define IMX8ULP_CLK_TPM4 3
> > > +#define IMX8ULP_CLK_TPM5 4
> > > +#define IMX8ULP_CLK_FLEXIO1 5
> > > +#define IMX8ULP_CLK_I3C2 6
> > > +#define IMX8ULP_CLK_LPI2C4 7
> > > +#define IMX8ULP_CLK_LPI2C5 8
> > > +#define IMX8ULP_CLK_LPUART4 9
> > > +#define IMX8ULP_CLK_LPUART5 10
> > > +#define IMX8ULP_CLK_LPSPI4 11
> > > +#define IMX8ULP_CLK_LPSPI5 12
> > > +#define IMX8ULP_CLK_DMA1_MP 13
> > > +#define IMX8ULP_CLK_DMA1_CH0 14
> > > +#define IMX8ULP_CLK_DMA1_CH1 15
> > > +#define IMX8ULP_CLK_DMA1_CH2 16
> > > +#define IMX8ULP_CLK_DMA1_CH3 17
> > > +#define IMX8ULP_CLK_DMA1_CH4 18
> > > +#define IMX8ULP_CLK_DMA1_CH5 19
> > > +#define IMX8ULP_CLK_DMA1_CH6 20
> > > +#define IMX8ULP_CLK_DMA1_CH7 21
> > > +#define IMX8ULP_CLK_DMA1_CH8 22
> > > +#define IMX8ULP_CLK_DMA1_CH9 23
> > > +#define IMX8ULP_CLK_DMA1_CH10 24
> > > +#define IMX8ULP_CLK_DMA1_CH11 25
> > > +#define IMX8ULP_CLK_DMA1_CH12 26
> > > +#define IMX8ULP_CLK_DMA1_CH13 27
> > > +#define IMX8ULP_CLK_DMA1_CH14 28
> > > +#define IMX8ULP_CLK_DMA1_CH15 29
> > > +#define IMX8ULP_CLK_DMA1_CH16 30
> > > +#define IMX8ULP_CLK_DMA1_CH17 31
> > > +#define IMX8ULP_CLK_DMA1_CH18 32
> > > +#define IMX8ULP_CLK_DMA1_CH19 33
> > > +#define IMX8ULP_CLK_DMA1_CH20 34
> > > +#define IMX8ULP_CLK_DMA1_CH21 35
> > > +#define IMX8ULP_CLK_DMA1_CH22 36
> > > +#define IMX8ULP_CLK_DMA1_CH23 37
> > > +#define IMX8ULP_CLK_DMA1_CH24 38
> > > +#define IMX8ULP_CLK_DMA1_CH25 39
> > > +#define IMX8ULP_CLK_DMA1_CH26 40
> > > +#define IMX8ULP_CLK_DMA1_CH27 41
> > > +#define IMX8ULP_CLK_DMA1_CH28 42
> > > +#define IMX8ULP_CLK_DMA1_CH29 43
> > > +#define IMX8ULP_CLK_DMA1_CH30 44
> > > +#define IMX8ULP_CLK_DMA1_CH31 45
> > > +#define IMX8ULP_CLK_MU3_A 46
> > > +#define IMX8ULP_CLK_MU0_B 47
> > > +
> > > +#define IMX8ULP_CLK_PCC3_END 48
> > > +
> > > +/* PCC4 */
> > > +#define IMX8ULP_CLK_FLEXSPI2 0
> > > +#define IMX8ULP_CLK_TPM6 1
> > > +#define IMX8ULP_CLK_TPM7 2
> > > +#define IMX8ULP_CLK_LPI2C6 3
> > > +#define IMX8ULP_CLK_LPI2C7 4
> > > +#define IMX8ULP_CLK_LPUART6 5
> > > +#define IMX8ULP_CLK_LPUART7 6
> > > +#define IMX8ULP_CLK_SAI4 7
> > > +#define IMX8ULP_CLK_SAI5 8
> > > +#define IMX8ULP_CLK_PCTLE 9
> > > +#define IMX8ULP_CLK_PCTLF 10
> > > +#define IMX8ULP_CLK_USDHC0 11
> > > +#define IMX8ULP_CLK_USDHC1 12
> > > +#define IMX8ULP_CLK_USDHC2 13
> > > +#define IMX8ULP_CLK_USB0 14
> > > +#define IMX8ULP_CLK_USB0_PHY 15
> > > +#define IMX8ULP_CLK_USB1 16
> > > +#define IMX8ULP_CLK_USB1_PHY 17
> > > +#define IMX8ULP_CLK_USB_XBAR 18
> > > +#define IMX8ULP_CLK_ENET 19
> > > +#define IMX8ULP_CLK_SFA1 20
> > > +#define IMX8ULP_CLK_RGPIOE 21
> > > +#define IMX8ULP_CLK_RGPIOF 22
> > > +
> > > +#define IMX8ULP_CLK_PCC4_END 23
> > > +
> > > +/* PCC5 */
> > > +#define IMX8ULP_CLK_TPM8 0
> > > +#define IMX8ULP_CLK_SAI6 1
> > > +#define IMX8ULP_CLK_SAI7 2
> > > +#define IMX8ULP_CLK_SPDIF 3
> > > +#define IMX8ULP_CLK_ISI 4
> > > +#define IMX8ULP_CLK_CSI_REGS 5
> > > +#define IMX8ULP_CLK_PCTLD 6
> > > +#define IMX8ULP_CLK_CSI 7
> > > +#define IMX8ULP_CLK_DSI 8
> > > +#define IMX8ULP_CLK_WDOG5 9
> > > +#define IMX8ULP_CLK_EPDC 10
> > > +#define IMX8ULP_CLK_PXP 11
> > > +#define IMX8ULP_CLK_SFA2 12
> > > +#define IMX8ULP_CLK_GPU2D 13
> > > +#define IMX8ULP_CLK_GPU3D 14
> > > +#define IMX8ULP_CLK_DC_NANO 15
> > > +#define IMX8ULP_CLK_CSI_CLK_UI 16
> > > +#define IMX8ULP_CLK_CSI_CLK_ESC 17
> > > +#define IMX8ULP_CLK_RGPIOD 18
> > > +#define IMX8ULP_CLK_DMA2_MP 19
> > > +#define IMX8ULP_CLK_DMA2_CH0 20
> > > +#define IMX8ULP_CLK_DMA2_CH1 21
> > > +#define IMX8ULP_CLK_DMA2_CH2 22
> > > +#define IMX8ULP_CLK_DMA2_CH3 23
> > > +#define IMX8ULP_CLK_DMA2_CH4 24
> > > +#define IMX8ULP_CLK_DMA2_CH5 25
> > > +#define IMX8ULP_CLK_DMA2_CH6 26
> > > +#define IMX8ULP_CLK_DMA2_CH7 27
> > > +#define IMX8ULP_CLK_DMA2_CH8 28
> > > +#define IMX8ULP_CLK_DMA2_CH9 29
> > > +#define IMX8ULP_CLK_DMA2_CH10 30
> > > +#define IMX8ULP_CLK_DMA2_CH11 31
> > > +#define IMX8ULP_CLK_DMA2_CH12 32
> > > +#define IMX8ULP_CLK_DMA2_CH13 33
> > > +#define IMX8ULP_CLK_DMA2_CH14 34
> > > +#define IMX8ULP_CLK_DMA2_CH15 35
> > > +#define IMX8ULP_CLK_DMA2_CH16 36
> > > +#define IMX8ULP_CLK_DMA2_CH17 37
> > > +#define IMX8ULP_CLK_DMA2_CH18 38
> > > +#define IMX8ULP_CLK_DMA2_CH19 39
> > > +#define IMX8ULP_CLK_DMA2_CH20 40
> > > +#define IMX8ULP_CLK_DMA2_CH21 41
> > > +#define IMX8ULP_CLK_DMA2_CH22 42
> > > +#define IMX8ULP_CLK_DMA2_CH23 43
> > > +#define IMX8ULP_CLK_DMA2_CH24 44
> > > +#define IMX8ULP_CLK_DMA2_CH25 45
> > > +#define IMX8ULP_CLK_DMA2_CH26 46
> > > +#define IMX8ULP_CLK_DMA2_CH27 47
> > > +#define IMX8ULP_CLK_DMA2_CH28 48
> > > +#define IMX8ULP_CLK_DMA2_CH29 49
> > > +#define IMX8ULP_CLK_DMA2_CH30 50
> > > +#define IMX8ULP_CLK_DMA2_CH31 51
> > > +#define IMX8ULP_CLK_MU2_B 52
> > > +#define IMX8ULP_CLK_MU3_B 53
> > > +#define IMX8ULP_CLK_AVD_SIM 54
> > > +#define IMX8ULP_CLK_DSI_TX_ESC 55
> > > +
> > > +#define IMX8ULP_CLK_PCC5_END 56
> > > +
> > > +#endif
> > > diff --git a/include/dt-bindings/reset/imx8ulp-pcc-reset.h
> > > b/include/dt-bindings/reset/imx8ulp-pcc-reset.h
> > > new file mode 100644
> > > index 000000000000..e99a4735c3c4
> > > --- /dev/null
> > > +++ b/include/dt-bindings/reset/imx8ulp-pcc-reset.h
> > > @@ -0,0 +1,59 @@
> > > +/* SPDX-License-Identifier: GPL-2.0-only */
> > > +/*
> > > + * Copyright 2021 NXP
> > > + */
> > > +
> > > +#ifndef DT_BINDING_PCC_RESET_IMX8ULP_H #define
> > > +DT_BINDING_PCC_RESET_IMX8ULP_H
> > > +
> > > +/* PCC3 */
> > > +#define PCC3_WDOG3_SWRST 0
> > > +#define PCC3_WDOG4_SWRST 1
> > > +#define PCC3_LPIT1_SWRST 2
> > > +#define PCC3_TPM4_SWRST 3
> > > +#define PCC3_TPM5_SWRST 4
> > > +#define PCC3_FLEXIO1_SWRST 5
> > > +#define PCC3_I3C2_SWRST 6
> > > +#define PCC3_LPI2C4_SWRST 7
> > > +#define PCC3_LPI2C5_SWRST 8
> > > +#define PCC3_LPUART4_SWRST 9
> > > +#define PCC3_LPUART5_SWRST 10
> > > +#define PCC3_LPSPI4_SWRST 11
> > > +#define PCC3_LPSPI5_SWRST 12
> > > +
> > > +/* PCC4 */
> > > +#define PCC4_FLEXSPI2_SWRST 0
> > > +#define PCC4_TPM6_SWRST 1
> > > +#define PCC4_TPM7_SWRST 2
> > > +#define PCC4_LPI2C6_SWRST 3
> > > +#define PCC4_LPI2C7_SWRST 4
> > > +#define PCC4_LPUART6_SWRST 5
> > > +#define PCC4_LPUART7_SWRST 6
> > > +#define PCC4_SAI4_SWRST 7
> > > +#define PCC4_SAI5_SWRST 8
> > > +#define PCC4_USDHC0_SWRST 9
> > > +#define PCC4_USDHC1_SWRST 10
> > > +#define PCC4_USDHC2_SWRST 11
> > > +#define PCC4_USB0_SWRST 12
> > > +#define PCC4_USB0_PHY_SWRST 13
> > > +#define PCC4_USB1_SWRST 14
> > > +#define PCC4_USB1_PHY_SWRST 15
> > > +#define PCC4_ENET_SWRST 16
> > > +
> > > +/* PCC5 */
> > > +#define PCC5_TPM8_SWRST 0
> > > +#define PCC5_SAI6_SWRST 1
> > > +#define PCC5_SAI7_SWRST 2
> > > +#define PCC5_SPDIF_SWRST 3
> > > +#define PCC5_ISI_SWRST 4
> > > +#define PCC5_CSI_REGS_SWRST 5
> > > +#define PCC5_CSI_SWRST 6
> > > +#define PCC5_DSI_SWRST 7
> > > +#define PCC5_WDOG5_SWRST 8
> > > +#define PCC5_EPDC_SWRST 9
> > > +#define PCC5_PXP_SWRST 10
> > > +#define PCC5_GPU2D_SWRST 11
> > > +#define PCC5_GPU3D_SWRST 12
> > > +#define PCC5_DC_NANO_SWRST 13
> > > +
> > > +#endif /*DT_BINDING_RESET_IMX8ULP_H */
> > > --
> > > 2.26.2
> > >
next prev parent reply other threads:[~2021-09-14 15:30 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-14 6:51 [PATCH v3 0/9] Add imx8ulp clock & reset driver support Jacky Bai
2021-09-14 6:52 ` [PATCH v3 1/9] dt-bindings: clock: Add imx8ulp clock support Jacky Bai
2021-09-14 11:25 ` Abel Vesa
2021-09-14 13:11 ` Jacky Bai
2021-09-14 15:30 ` Abel Vesa [this message]
2021-09-20 21:44 ` Rob Herring
2021-09-14 6:52 ` [PATCH v3 2/9] clk: imx: Update the pllv4 to support imx8ulp Jacky Bai
2021-09-14 11:29 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 3/9] clk: imx: Update the compsite driver " Jacky Bai
2021-09-14 11:30 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 4/9] clk: imx: disable i.mx7ulp composite clock during initialization Jacky Bai
2021-09-14 11:32 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 5/9] clk: imx: Add 'CLK_SET_RATE_NO_REPARENT' for composite-7ulp Jacky Bai
2021-09-14 11:33 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 6/9] clk: imx: disable the pfd when set pfdv2 clock rate Jacky Bai
2021-09-14 11:34 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 7/9] clk: imx: Update the pfdv2 for 8ulp specific support Jacky Bai
2021-09-14 11:35 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 8/9] clk: imx: Add clock driver for imx8ulp Jacky Bai
2021-09-14 11:55 ` Abel Vesa
2021-09-14 6:52 ` [PATCH v3 9/9] clk: imx: Add the pcc reset controller support on imx8ulp Jacky Bai
2021-09-14 12:09 ` Abel Vesa
2021-09-14 13:07 ` Jacky Bai
2021-09-14 15:19 ` Abel Vesa
2021-09-14 11:17 ` [PATCH v3 0/9] Add imx8ulp clock & reset driver support Abel Vesa
2021-09-16 6:46 ` Abel Vesa
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YUDAJ18URdZRGIup@ryzen \
--to=abel.vesa@nxp.com \
--cc=devicetree@vger.kernel.org \
--cc=kernel@pengutronix.de \
--cc=linux-imx@nxp.com \
--cc=p.zabel@pengutronix.de \
--cc=ping.bai@nxp.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=sboyd@kernel.org \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).