* [PATCH v1 0/3] dt-bindings: PCI: imx6: convert the imx pcie
@ 2021-08-19 7:14 Richard Zhu
2021-08-19 7:14 ` [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema Richard Zhu
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Richard Zhu @ 2021-08-19 7:14 UTC (permalink / raw)
To: robh, l.stach, galak, shawnguo
Cc: devicetree, linux-arm-kernel, linux-kernel, linux-imx, kernel
And fix according errors on imx6qdl.dtsi, imx6qp.dtsi, imx6sx.dtsi,
imx7d.dtsi and imx8mq.dtsi found by dt_binding_check and dtbs_check.
Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt | 100 ----------------------------------------
Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml | 207 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
MAINTAINERS | 2 +-
arch/arm/boot/dts/imx6qdl.dtsi | 6 +--
arch/arm/boot/dts/imx6qp.dtsi | 2 +-
arch/arm/boot/dts/imx6sx.dtsi | 7 +--
arch/arm/boot/dts/imx7d.dtsi | 6 +--
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 8 ++--
8 files changed, 223 insertions(+), 115 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt
create mode 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
[PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie
[PATCH v1 2/3] arm: dts: imx: fix the schema check errors
[PATCH v1 3/3] arm64: dts: imx8mq: fix the schema check errors
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema 2021-08-19 7:14 [PATCH v1 0/3] dt-bindings: PCI: imx6: convert the imx pcie Richard Zhu @ 2021-08-19 7:14 ` Richard Zhu 2021-08-23 18:34 ` Rob Herring 2021-08-19 7:14 ` [PATCH v1 2/3] arm: dts: imx: fix the schema check errors Richard Zhu 2021-08-19 7:14 ` [PATCH v1 3/3] arm64: dts: imx8mq: " Richard Zhu 2 siblings, 1 reply; 9+ messages in thread From: Richard Zhu @ 2021-08-19 7:14 UTC (permalink / raw) To: robh, l.stach, galak, shawnguo Cc: devicetree, linux-arm-kernel, linux-kernel, linux-imx, kernel, Richard Zhu Convert the fsl,imx6q-pcie.txt into a schema. - the ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# - only one propert is allowed in the compatible string, remove "snps,dw-pcie". Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> --- .../bindings/pci/fsl,imx6q-pcie.txt | 100 --------- .../bindings/pci/fsl,imx6q-pcie.yaml | 207 ++++++++++++++++++ MAINTAINERS | 2 +- 3 files changed, 208 insertions(+), 101 deletions(-) delete mode 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt create mode 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt deleted file mode 100644 index d8971ab99274..000000000000 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +++ /dev/null @@ -1,100 +0,0 @@ -* Freescale i.MX6 PCIe interface - -This PCIe host controller is based on the Synopsys DesignWare PCIe IP -and thus inherits all the common properties defined in designware-pcie.txt. - -Required properties: -- compatible: - - "fsl,imx6q-pcie" - - "fsl,imx6sx-pcie", - - "fsl,imx6qp-pcie" - - "fsl,imx7d-pcie" - - "fsl,imx8mq-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: - - "msi": The interrupt that is asserted when an MSI is received -- clock-names: Must include the following additional entries: - - "pcie_phy" - -Optional properties: -- fsl,tx-deemph-gen1: Gen1 De-emphasis value. Default: 0 -- fsl,tx-deemph-gen2-3p5db: Gen2 (3.5db) De-emphasis value. Default: 0 -- fsl,tx-deemph-gen2-6db: Gen2 (6db) De-emphasis value. Default: 20 -- fsl,tx-swing-full: Gen2 TX SWING FULL value. Default: 127 -- fsl,tx-swing-low: TX launch amplitude swing_low value. Default: 127 -- fsl,max-link-speed: Specify PCI gen for link capability. Must be '2' for - gen2, otherwise will default to gen1. Note that the IMX6 LVDS clock outputs - do not meet gen2 jitter requirements and thus for gen2 capability a gen2 - compliant clock generator should be used and configured. -- reset-gpio: Should specify the GPIO for controlling the PCI bus device reset - signal. It's not polarity aware and defaults to active-low reset sequence - (L=reset state, H=operation state). -- reset-gpio-active-high: If present then the reset sequence using the GPIO - specified in the "reset-gpio" property is reversed (H=reset state, - L=operation state). -- vpcie-supply: Should specify the regulator in charge of PCIe port power. - The regulator will be enabled when initializing the PCIe host and - disabled either as part of the init process or when shutting down the - host. -- vph-supply: Should specify the regulator in charge of VPH one of the three - PCIe PHY powers. This regulator can be supplied by both 1.8v and 3.3v voltage - supplies. - -Additional required properties for imx6sx-pcie: -- clock names: Must include the following additional entries: - - "pcie_inbound_axi" -- power-domains: Must be set to phandles pointing to the DISPLAY and - PCIE_PHY power domains -- power-domain-names: Must be "pcie", "pcie_phy" - -Additional required properties for imx7d-pcie and imx8mq-pcie: -- power-domains: Must be set to a phandle pointing to PCIE_PHY power domain -- resets: Must contain phandles to PCIe-related reset lines exposed by SRC - IP block -- reset-names: Must contain the following entries: - - "pciephy" - - "apps" - - "turnoff" -- fsl,imx7d-pcie-phy: A phandle to an fsl,imx7d-pcie-phy node. - -Additional required properties for imx8mq-pcie: -- clock-names: Must include the following additional entries: - - "pcie_aux" - -Example: - - pcie@01000000 { - compatible = "fsl,imx6q-pcie", "snps,dw-pcie"; - reg = <0x01ffc000 0x04000>, - <0x01f00000 0x80000>; - reg-names = "dbi", "config"; - #address-cells = <3>; - #size-cells = <2>; - device_type = "pci"; - ranges = <0x00000800 0 0x01f00000 0x01f00000 0 0x00080000 - 0x81000000 0 0 0x01f80000 0 0x00010000 - 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; - num-lanes = <1>; - interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; - interrupt-names = "msi"; - #interrupt-cells = <1>; - interrupt-map-mask = <0 0 0 0x7>; - interrupt-map = <0 0 0 1 &intc GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>, - <0 0 0 2 &intc GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>, - <0 0 0 3 &intc GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>, - <0 0 0 4 &intc GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; - clocks = <&clks 144>, <&clks 206>, <&clks 189>; - clock-names = "pcie", "pcie_bus", "pcie_phy"; - }; - -* Freescale i.MX7d PCIe PHY - -This is the PHY associated with the IMX7d PCIe controller. It's used by the -PCI-e controller via the fsl,imx7d-pcie-phy phandle. - -Required properties: -- compatible: - - "fsl,imx7d-pcie-phy" -- reg: base address and length of the PCIe PHY controller diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml new file mode 100644 index 000000000000..3503ce3de05d --- /dev/null +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml @@ -0,0 +1,207 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Freescale i.MX6 PCIe host controller + +maintainers: + - Lucas Stach <l.stach@pengutronix.de> + - Richard Zhu <hongxing.zhu@nxp.com> + +description: |+ + This PCIe host controller is based on the Synopsys DesignWare PCIe IP + and thus inherits all the common properties defined in snps,dw-pcie.yaml. + +allOf: + - $ref: /schemas/pci/pci-bus.yaml# + +properties: + compatible: + enum: + - fsl,imx6q-pcie + - fsl,imx6sx-pcie + - fsl,imx6qp-pcie + - fsl,imx7d-pcie + - fsl,imx8mq-pcie + + reg: + items: + - description: Data Bus Interface (DBI) registers. + - description: PCIe configuration space region. + + reg-names: + items: + - const: dbi + - const: config + + interrupts: + minItems: 1 + items: + - description: builtin MSI controller. + + interrupt-names: + minItems: 1 + items: + - const: msi + + clocks: + minItems: 3 + maxItems: 4 + items: + - description: PCIe bridge clock. + - description: PCIe bus clock. + - description: PCIe PHY clock. + - description: Additional required clock entry for imx6sx-pcie, + imx8mq-pcie. + + clock-names: + minItems: 3 + maxItems: 4 + items: + - const: pcie + - const: pcie_bus + - const: pcie_phy + - const: pcie_inbound_axi for imx6sx-pcie, pcie_aux for imx8mq-pcie + + num-lanes: + const: 1 + + num-viewport: + const: 4 + + fsl,imx7d-pcie-phy: + $ref: /schemas/types.yaml#/definitions/phandle + description: A phandle to an fsl,imx7d-pcie-phy node. Additional + required properties for imx7d-pcie and imx8mq-pcie. + + power-domains: + maxItems: 2 + description: The first phandle pointing to the DISPLAY domain for + imx6sx-pcie, to PCIE_PHY power domain for imx7d-pcie and imx8mq-pcie. + The second phandle pointing to the PCIE_PHY power domains for imx6sx-pcie. + + power-domain-names: + maxItems: 2 + + resets: + maxItems: 3 + description: Phandles to PCIe-related reset lines exposed by SRC + IP block. Additional required by imx7d-pcie and imx8mq-pcie. + + reset-names: + items: + - const: pciephy + - const: apps + - const: turnoff + + fsl,tx-deemph-gen1: + description: Gen1 De-emphasis value (optional required). + $ref: /schemas/types.yaml#/definitions/uint32 + default: 0 + + fsl,tx-deemph-gen2-3p5db: + description: Gen2 (3.5db) De-emphasis value (optional required). + $ref: /schemas/types.yaml#/definitions/uint32 + default: 0 + + fsl,tx-deemph-gen2-6db: + description: Gen2 (6db) De-emphasis value (optional required). + $ref: /schemas/types.yaml#/definitions/uint32 + default: 20 + + fsl,tx-swing-full: + description: Gen2 TX SWING FULL value (optional required). + $ref: /schemas/types.yaml#/definitions/uint32 + default: 127 + + fsl,tx-swing-low: + description: TX launch amplitude swing_low value (optional required). + $ref: /schemas/types.yaml#/definitions/uint32 + default: 127 + + fsl,max-link-speed: + description: Specify PCI Gen for link capability (optional required). + Note that the IMX6 LVDS clock outputs do not meet gen2 jitter + requirements and thus for gen2 capability a gen2 compliant clock + generator should be used and configured. + $ref: /schemas/types.yaml#/definitions/uint32 + enum: [1, 2, 3, 4] + default: 1 + + reset-gpio: + description: Should specify the GPIO for controlling the PCI bus device + reset signal. It's not polarity aware and defaults to active-low reset + sequence (L=reset state, H=operation state) (optional required). + + reset-gpio-active-high: + description: If present then the reset sequence using the GPIO + specified in the "reset-gpio" property is reversed (H=reset state, + L=operation state) (optional required). + + vpcie-supply: + description: Should specify the regulator in charge of PCIe port power. + The regulator will be enabled when initializing the PCIe host and + disabled either as part of the init process or when shutting down + the host (optional required). + + vph-supply: + description: Should specify the regulator in charge of VPH one of + the three PCIe PHY powers. This regulator can be supplied by both + 1.8v and 3.3v voltage supplies (optional required). + +required: + - compatible + - reg + - reg-names + - "#address-cells" + - "#size-cells" + - device_type + - bus-range + - ranges + - num-lanes + - num-viewport + - interrupts + - interrupt-names + - "#interrupt-cells" + - interrupt-map-mask + - interrupt-map + - clocks + - clock-names + +unevaluatedProperties: false + +examples: + - | + #include <dt-bindings/clock/imx6qdl-clock.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + + pcie: pcie@1ffc000 { + compatible = "fsl,imx6q-pcie"; + reg = <0x01ffc000 0x04000>, + <0x01f00000 0x80000>; + reg-names = "dbi", "config"; + #address-cells = <3>; + #size-cells = <2>; + device_type = "pci"; + bus-range = <0x00 0xff>; + ranges = <0x81000000 0 0 0x01f80000 0 0x00010000>, + <0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; + num-lanes = <1>; + num-viewport = <4>; + interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "msi"; + #interrupt-cells = <1>; + interrupt-map-mask = <0 0 0 0x7>; + interrupt-map = <0 0 0 1 &gpc GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>, + <0 0 0 2 &gpc GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>, + <0 0 0 3 &gpc GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>, + <0 0 0 4 &gpc GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&clks IMX6QDL_CLK_PCIE_AXI>, + <&clks IMX6QDL_CLK_LVDS1_GATE>, + <&clks IMX6QDL_CLK_PCIE_REF_125M>; + clock-names = "pcie", "pcie_bus", "pcie_phy"; + status = "disabled"; + }; +... diff --git a/MAINTAINERS b/MAINTAINERS index f73fef615702..a1e2cc0be9e8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -14187,7 +14187,7 @@ M: Lucas Stach <l.stach@pengutronix.de> L: linux-pci@vger.kernel.org L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) S: Maintained -F: Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt +F: Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml F: drivers/pci/controller/dwc/*imx6* PCI DRIVER FOR FU740 -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema 2021-08-19 7:14 ` [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema Richard Zhu @ 2021-08-23 18:34 ` Rob Herring 2021-08-24 6:04 ` Richard Zhu 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2021-08-23 18:34 UTC (permalink / raw) To: Richard Zhu Cc: l.stach, galak, shawnguo, devicetree, linux-arm-kernel, linux-kernel, linux-imx, kernel On Thu, Aug 19, 2021 at 03:14:06PM +0800, Richard Zhu wrote: > Convert the fsl,imx6q-pcie.txt into a schema. > - the ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# > - only one propert is allowed in the compatible string, remove > "snps,dw-pcie". Didn't I already see a v1? > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> > --- > .../bindings/pci/fsl,imx6q-pcie.txt | 100 --------- > .../bindings/pci/fsl,imx6q-pcie.yaml | 207 ++++++++++++++++++ > MAINTAINERS | 2 +- > 3 files changed, 208 insertions(+), 101 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > create mode 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > deleted file mode 100644 > index d8971ab99274..000000000000 > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +++ /dev/null > @@ -1,100 +0,0 @@ > -* Freescale i.MX6 PCIe interface > - > -This PCIe host controller is based on the Synopsys DesignWare PCIe IP > -and thus inherits all the common properties defined in designware-pcie.txt. > - > -Required properties: > -- compatible: > - - "fsl,imx6q-pcie" > - - "fsl,imx6sx-pcie", > - - "fsl,imx6qp-pcie" > - - "fsl,imx7d-pcie" > - - "fsl,imx8mq-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: > - - "msi": The interrupt that is asserted when an MSI is received > -- clock-names: Must include the following additional entries: > - - "pcie_phy" > - > -Optional properties: > -- fsl,tx-deemph-gen1: Gen1 De-emphasis value. Default: 0 > -- fsl,tx-deemph-gen2-3p5db: Gen2 (3.5db) De-emphasis value. Default: 0 > -- fsl,tx-deemph-gen2-6db: Gen2 (6db) De-emphasis value. Default: 20 > -- fsl,tx-swing-full: Gen2 TX SWING FULL value. Default: 127 > -- fsl,tx-swing-low: TX launch amplitude swing_low value. Default: 127 > -- fsl,max-link-speed: Specify PCI gen for link capability. Must be '2' for > - gen2, otherwise will default to gen1. Note that the IMX6 LVDS clock outputs > - do not meet gen2 jitter requirements and thus for gen2 capability a gen2 > - compliant clock generator should be used and configured. > -- reset-gpio: Should specify the GPIO for controlling the PCI bus device reset > - signal. It's not polarity aware and defaults to active-low reset sequence > - (L=reset state, H=operation state). > -- reset-gpio-active-high: If present then the reset sequence using the GPIO > - specified in the "reset-gpio" property is reversed (H=reset state, > - L=operation state). > -- vpcie-supply: Should specify the regulator in charge of PCIe port power. > - The regulator will be enabled when initializing the PCIe host and > - disabled either as part of the init process or when shutting down the > - host. > -- vph-supply: Should specify the regulator in charge of VPH one of the three > - PCIe PHY powers. This regulator can be supplied by both 1.8v and 3.3v voltage > - supplies. > - > -Additional required properties for imx6sx-pcie: > -- clock names: Must include the following additional entries: > - - "pcie_inbound_axi" > -- power-domains: Must be set to phandles pointing to the DISPLAY and > - PCIE_PHY power domains > -- power-domain-names: Must be "pcie", "pcie_phy" > - > -Additional required properties for imx7d-pcie and imx8mq-pcie: > -- power-domains: Must be set to a phandle pointing to PCIE_PHY power domain > -- resets: Must contain phandles to PCIe-related reset lines exposed by SRC > - IP block > -- reset-names: Must contain the following entries: > - - "pciephy" > - - "apps" > - - "turnoff" > -- fsl,imx7d-pcie-phy: A phandle to an fsl,imx7d-pcie-phy node. > - > -Additional required properties for imx8mq-pcie: > -- clock-names: Must include the following additional entries: > - - "pcie_aux" > - > -Example: > - > - pcie@01000000 { > - compatible = "fsl,imx6q-pcie", "snps,dw-pcie"; > - reg = <0x01ffc000 0x04000>, > - <0x01f00000 0x80000>; > - reg-names = "dbi", "config"; > - #address-cells = <3>; > - #size-cells = <2>; > - device_type = "pci"; > - ranges = <0x00000800 0 0x01f00000 0x01f00000 0 0x00080000 > - 0x81000000 0 0 0x01f80000 0 0x00010000 > - 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; > - num-lanes = <1>; > - interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > - interrupt-names = "msi"; > - #interrupt-cells = <1>; > - interrupt-map-mask = <0 0 0 0x7>; > - interrupt-map = <0 0 0 1 &intc GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>, > - <0 0 0 2 &intc GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>, > - <0 0 0 3 &intc GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>, > - <0 0 0 4 &intc GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > - clocks = <&clks 144>, <&clks 206>, <&clks 189>; > - clock-names = "pcie", "pcie_bus", "pcie_phy"; > - }; > - > -* Freescale i.MX7d PCIe PHY > - > -This is the PHY associated with the IMX7d PCIe controller. It's used by the > -PCI-e controller via the fsl,imx7d-pcie-phy phandle. > - > -Required properties: > -- compatible: > - - "fsl,imx7d-pcie-phy" > -- reg: base address and length of the PCIe PHY controller > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > new file mode 100644 > index 000000000000..3503ce3de05d > --- /dev/null > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > @@ -0,0 +1,207 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Freescale i.MX6 PCIe host controller > + > +maintainers: > + - Lucas Stach <l.stach@pengutronix.de> > + - Richard Zhu <hongxing.zhu@nxp.com> > + > +description: |+ > + This PCIe host controller is based on the Synopsys DesignWare PCIe IP > + and thus inherits all the common properties defined in snps,dw-pcie.yaml. Don't need a freeform text reference, but... > + > +allOf: > + - $ref: /schemas/pci/pci-bus.yaml# ...this needs to reference the snps,dw-pcie.yaml DWC schema. > + > +properties: > + compatible: > + enum: > + - fsl,imx6q-pcie > + - fsl,imx6sx-pcie > + - fsl,imx6qp-pcie > + - fsl,imx7d-pcie > + - fsl,imx8mq-pcie > + > + reg: > + items: > + - description: Data Bus Interface (DBI) registers. > + - description: PCIe configuration space region. > + > + reg-names: > + items: > + - const: dbi > + - const: config > + > + interrupts: > + minItems: 1 Don't need 'minItems: 1' when there's only 1 possible item: > + items: > + - description: builtin MSI controller. > + > + interrupt-names: > + minItems: 1 > + items: > + - const: msi > + > + clocks: > + minItems: 3 > + maxItems: 4 Don't need maxItems here. > + items: > + - description: PCIe bridge clock. > + - description: PCIe bus clock. > + - description: PCIe PHY clock. > + - description: Additional required clock entry for imx6sx-pcie, > + imx8mq-pcie. > + > + clock-names: > + minItems: 3 > + maxItems: 4 And here. > + items: > + - const: pcie > + - const: pcie_bus > + - const: pcie_phy > + - const: pcie_inbound_axi for imx6sx-pcie, pcie_aux for imx8mq-pcie > + > + num-lanes: > + const: 1 > + > + num-viewport: > + const: 4 This is deprecated. The driver detects this now. > + > + fsl,imx7d-pcie-phy: > + $ref: /schemas/types.yaml#/definitions/phandle > + description: A phandle to an fsl,imx7d-pcie-phy node. Additional > + required properties for imx7d-pcie and imx8mq-pcie. > + > + power-domains: > + maxItems: 2 > + description: The first phandle pointing to the DISPLAY domain for > + imx6sx-pcie, to PCIE_PHY power domain for imx7d-pcie and imx8mq-pcie. > + The second phandle pointing to the PCIE_PHY power domains for imx6sx-pcie. Split the description: items: - description: ... - description: ... > + > + power-domain-names: > + maxItems: 2 You have to define the names. > + > + resets: > + maxItems: 3 > + description: Phandles to PCIe-related reset lines exposed by SRC > + IP block. Additional required by imx7d-pcie and imx8mq-pcie. > + > + reset-names: > + items: > + - const: pciephy > + - const: apps > + - const: turnoff > + > + fsl,tx-deemph-gen1: > + description: Gen1 De-emphasis value (optional required). optional or required? > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 0 > + > + fsl,tx-deemph-gen2-3p5db: > + description: Gen2 (3.5db) De-emphasis value (optional required). > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 0 > + > + fsl,tx-deemph-gen2-6db: > + description: Gen2 (6db) De-emphasis value (optional required). > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 20 > + > + fsl,tx-swing-full: > + description: Gen2 TX SWING FULL value (optional required). > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 127 > + > + fsl,tx-swing-low: > + description: TX launch amplitude swing_low value (optional required). > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 127 > + > + fsl,max-link-speed: > + description: Specify PCI Gen for link capability (optional required). > + Note that the IMX6 LVDS clock outputs do not meet gen2 jitter > + requirements and thus for gen2 capability a gen2 compliant clock > + generator should be used and configured. > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [1, 2, 3, 4] > + default: 1 > + > + reset-gpio: Deprecated. > + description: Should specify the GPIO for controlling the PCI bus device > + reset signal. It's not polarity aware and defaults to active-low reset > + sequence (L=reset state, H=operation state) (optional required). > + > + reset-gpio-active-high: Deprecated. This should only be used on older compatibles I hope... > + description: If present then the reset sequence using the GPIO > + specified in the "reset-gpio" property is reversed (H=reset state, > + L=operation state) (optional required). > + > + vpcie-supply: > + description: Should specify the regulator in charge of PCIe port power. > + The regulator will be enabled when initializing the PCIe host and > + disabled either as part of the init process or when shutting down > + the host (optional required). > + > + vph-supply: > + description: Should specify the regulator in charge of VPH one of > + the three PCIe PHY powers. This regulator can be supplied by both > + 1.8v and 3.3v voltage supplies (optional required). > + > +required: > + - compatible > + - reg > + - reg-names > + - "#address-cells" > + - "#size-cells" > + - device_type > + - bus-range > + - ranges > + - num-lanes > + - num-viewport > + - interrupts > + - interrupt-names > + - "#interrupt-cells" > + - interrupt-map-mask > + - interrupt-map > + - clocks > + - clock-names > + > +unevaluatedProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/imx6qdl-clock.h> > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + > + pcie: pcie@1ffc000 { > + compatible = "fsl,imx6q-pcie"; > + reg = <0x01ffc000 0x04000>, > + <0x01f00000 0x80000>; > + reg-names = "dbi", "config"; > + #address-cells = <3>; > + #size-cells = <2>; > + device_type = "pci"; > + bus-range = <0x00 0xff>; > + ranges = <0x81000000 0 0 0x01f80000 0 0x00010000>, > + <0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; > + num-lanes = <1>; > + num-viewport = <4>; > + interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-names = "msi"; > + #interrupt-cells = <1>; > + interrupt-map-mask = <0 0 0 0x7>; > + interrupt-map = <0 0 0 1 &gpc GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>, > + <0 0 0 2 &gpc GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>, > + <0 0 0 3 &gpc GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>, > + <0 0 0 4 &gpc GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&clks IMX6QDL_CLK_PCIE_AXI>, > + <&clks IMX6QDL_CLK_LVDS1_GATE>, > + <&clks IMX6QDL_CLK_PCIE_REF_125M>; > + clock-names = "pcie", "pcie_bus", "pcie_phy"; > + status = "disabled"; Why are you disabling your example? > + }; > +... > diff --git a/MAINTAINERS b/MAINTAINERS > index f73fef615702..a1e2cc0be9e8 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -14187,7 +14187,7 @@ M: Lucas Stach <l.stach@pengutronix.de> > L: linux-pci@vger.kernel.org > L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) > S: Maintained > -F: Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > +F: Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > F: drivers/pci/controller/dwc/*imx6* > > PCI DRIVER FOR FU740 > -- > 2.25.1 > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema 2021-08-23 18:34 ` Rob Herring @ 2021-08-24 6:04 ` Richard Zhu 0 siblings, 0 replies; 9+ messages in thread From: Richard Zhu @ 2021-08-24 6:04 UTC (permalink / raw) To: Rob Herring Cc: l.stach, galak, shawnguo, devicetree, linux-arm-kernel, linux-kernel, dl-linux-imx, kernel > -----Original Message----- > From: Rob Herring <robh@kernel.org> > Sent: Tuesday, August 24, 2021 2:34 AM > To: Richard Zhu <hongxing.zhu@nxp.com> > Cc: l.stach@pengutronix.de; galak@kernel.crashing.org; > shawnguo@kernel.org; devicetree@vger.kernel.org; > linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; > dl-linux-imx <linux-imx@nxp.com>; kernel@pengutronix.de > Subject: Re: [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie > controller to dtschema > > On Thu, Aug 19, 2021 at 03:14:06PM +0800, Richard Zhu wrote: > > Convert the fsl,imx6q-pcie.txt into a schema. > > - the ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# > > - only one propert is allowed in the compatible string, remove > > "snps,dw-pcie". > > Didn't I already see a v1? [Richard Zhu] I assume that the RFC version as the v0, not formal v1. Would pay attention to that. > > > > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> > > --- > > .../bindings/pci/fsl,imx6q-pcie.txt | 100 --------- > > .../bindings/pci/fsl,imx6q-pcie.yaml | 207 > ++++++++++++++++++ > > MAINTAINERS | 2 +- > > 3 files changed, 208 insertions(+), 101 deletions(-) delete mode > > 100644 Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > create mode 100644 > > Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > > > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > deleted file mode 100644 > > index d8971ab99274..000000000000 > > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > +++ /dev/null > > @@ -1,100 +0,0 @@ > > -* Freescale i.MX6 PCIe interface > > - > > -This PCIe host controller is based on the Synopsys DesignWare PCIe IP > > -and thus inherits all the common properties defined in designware-pcie.txt. > > - > > -Required properties: > > -- compatible: > > - - "fsl,imx6q-pcie" > > - - "fsl,imx6sx-pcie", > > - - "fsl,imx6qp-pcie" > > - - "fsl,imx7d-pcie" > > - - "fsl,imx8mq-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: > > - - "msi": The interrupt that is asserted when an MSI is received > > -- clock-names: Must include the following additional entries: > > - - "pcie_phy" > > - > > -Optional properties: > > -- fsl,tx-deemph-gen1: Gen1 De-emphasis value. Default: 0 > > -- fsl,tx-deemph-gen2-3p5db: Gen2 (3.5db) De-emphasis value. Default: > > 0 > > -- fsl,tx-deemph-gen2-6db: Gen2 (6db) De-emphasis value. Default: 20 > > -- fsl,tx-swing-full: Gen2 TX SWING FULL value. Default: 127 > > -- fsl,tx-swing-low: TX launch amplitude swing_low value. Default: 127 > > -- fsl,max-link-speed: Specify PCI gen for link capability. Must be > > '2' for > > - gen2, otherwise will default to gen1. Note that the IMX6 LVDS clock > > outputs > > - do not meet gen2 jitter requirements and thus for gen2 capability a > > gen2 > > - compliant clock generator should be used and configured. > > -- reset-gpio: Should specify the GPIO for controlling the PCI bus > > device reset > > - signal. It's not polarity aware and defaults to active-low reset > > sequence > > - (L=reset state, H=operation state). > > -- reset-gpio-active-high: If present then the reset sequence using > > the GPIO > > - specified in the "reset-gpio" property is reversed (H=reset state, > > - L=operation state). > > -- vpcie-supply: Should specify the regulator in charge of PCIe port power. > > - The regulator will be enabled when initializing the PCIe host and > > - disabled either as part of the init process or when shutting down > > the > > - host. > > -- vph-supply: Should specify the regulator in charge of VPH one of > > the three > > - PCIe PHY powers. This regulator can be supplied by both 1.8v and > > 3.3v voltage > > - supplies. > > - > > -Additional required properties for imx6sx-pcie: > > -- clock names: Must include the following additional entries: > > - - "pcie_inbound_axi" > > -- power-domains: Must be set to phandles pointing to the DISPLAY and > > - PCIE_PHY power domains > > -- power-domain-names: Must be "pcie", "pcie_phy" > > - > > -Additional required properties for imx7d-pcie and imx8mq-pcie: > > -- power-domains: Must be set to a phandle pointing to PCIE_PHY power > > domain > > -- resets: Must contain phandles to PCIe-related reset lines exposed > > by SRC > > - IP block > > -- reset-names: Must contain the following entries: > > - - "pciephy" > > - - "apps" > > - - "turnoff" > > -- fsl,imx7d-pcie-phy: A phandle to an fsl,imx7d-pcie-phy node. > > - > > -Additional required properties for imx8mq-pcie: > > -- clock-names: Must include the following additional entries: > > - - "pcie_aux" > > - > > -Example: > > - > > - pcie@01000000 { > > - compatible = "fsl,imx6q-pcie", "snps,dw-pcie"; > > - reg = <0x01ffc000 0x04000>, > > - <0x01f00000 0x80000>; > > - reg-names = "dbi", "config"; > > - #address-cells = <3>; > > - #size-cells = <2>; > > - device_type = "pci"; > > - ranges = <0x00000800 0 0x01f00000 0x01f00000 0 0x00080000 > > - 0x81000000 0 0 0x01f80000 0 0x00010000 > > - 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; > > - num-lanes = <1>; > > - interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > > - interrupt-names = "msi"; > > - #interrupt-cells = <1>; > > - interrupt-map-mask = <0 0 0 0x7>; > > - interrupt-map = <0 0 0 1 &intc GIC_SPI 123 > IRQ_TYPE_LEVEL_HIGH>, > > - <0 0 0 2 &intc GIC_SPI 122 > IRQ_TYPE_LEVEL_HIGH>, > > - <0 0 0 3 &intc GIC_SPI 121 > IRQ_TYPE_LEVEL_HIGH>, > > - <0 0 0 4 &intc GIC_SPI 120 > IRQ_TYPE_LEVEL_HIGH>; > > - clocks = <&clks 144>, <&clks 206>, <&clks 189>; > > - clock-names = "pcie", "pcie_bus", "pcie_phy"; > > - }; > > - > > -* Freescale i.MX7d PCIe PHY > > - > > -This is the PHY associated with the IMX7d PCIe controller. It's used > > by the -PCI-e controller via the fsl,imx7d-pcie-phy phandle. > > - > > -Required properties: > > -- compatible: > > - - "fsl,imx7d-pcie-phy" > > -- reg: base address and length of the PCIe PHY controller diff --git > > a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > > b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > > new file mode 100644 > > index 000000000000..3503ce3de05d > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > > @@ -0,0 +1,207 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2 > > +--- > > +$id: > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi > > > +cetree.org%2Fschemas%2Fpci%2Ffsl%2Cimx6q-pcie.yaml%23&data=0 > 4%7C0 > > > +1%7Chongxing.zhu%40nxp.com%7Ca0e50943284145edfdc808d966649a08 > %7C686ea > > > +1d3bc2b4c6fa92cd99c5c301635%7C0%7C1%7C637653404543147546%7C > Unknown%7C > > > +TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwi > LCJXV > > > +CI6Mn0%3D%7C3000&sdata=mW%2FTR8dffuxQDk%2Fm1eiUd8IIRBjE > mJMhkxQZPo > > +GYfHM%3D&reserved=0 > > +$schema: > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi > > > +cetree.org%2Fmeta-schemas%2Fcore.yaml%23&data=04%7C01%7Cho > ngxing. > > > +zhu%40nxp.com%7Ca0e50943284145edfdc808d966649a08%7C686ea1d3b > c2b4c6fa9 > > > +2cd99c5c301635%7C0%7C1%7C637653404543147546%7CUnknown%7CT > WFpbGZsb3d8e > > > +yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3 > D%7C3 > > > +000&sdata=Gh1dUyXO80b2aKvWKe0Q2GIN4w%2BCit1oj06AvToamuE > %3D&re > > +served=0 > > + > > +title: Freescale i.MX6 PCIe host controller > > + > > +maintainers: > > + - Lucas Stach <l.stach@pengutronix.de> > > + - Richard Zhu <hongxing.zhu@nxp.com> > > + > > +description: |+ > > + This PCIe host controller is based on the Synopsys DesignWare PCIe > > +IP > > + and thus inherits all the common properties defined in > snps,dw-pcie.yaml. > > Don't need a freeform text reference, but... [Richard Zhu] This part is copied from the description of previous the binding document. Should I remove them totally? > > + > > +allOf: > > + - $ref: /schemas/pci/pci-bus.yaml# > > ...this needs to reference the snps,dw-pcie.yaml DWC schema. > [Richard Zhu] Got that, would be change in next version patch. > > + > > +properties: > > + compatible: > > + enum: > > + - fsl,imx6q-pcie > > + - fsl,imx6sx-pcie > > + - fsl,imx6qp-pcie > > + - fsl,imx7d-pcie > > + - fsl,imx8mq-pcie > > + > > + reg: > > + items: > > + - description: Data Bus Interface (DBI) registers. > > + - description: PCIe configuration space region. > > + > > + reg-names: > > + items: > > + - const: dbi > > + - const: config > > + > > + interrupts: > > + minItems: 1 > > Don't need 'minItems: 1' when there's only 1 possible item: [Richard Zhu] Okay, would be removed. > > > + items: > > + - description: builtin MSI controller. > > + > > + interrupt-names: > > + minItems: 1 > > + items: > > + - const: msi > > + > > + clocks: > > + minItems: 3 > > + maxItems: 4 > > Don't need maxItems here. > [Richard Zhu] Okay, would be removed. > > + items: > > + - description: PCIe bridge clock. > > + - description: PCIe bus clock. > > + - description: PCIe PHY clock. > > + - description: Additional required clock entry for imx6sx-pcie, > > + imx8mq-pcie. > > + > > + clock-names: > > + minItems: 3 > > + maxItems: 4 > > And here. > [Richard Zhu] Okay, would be removed. > > + items: > > + - const: pcie > > + - const: pcie_bus > > + - const: pcie_phy > > + - const: pcie_inbound_axi for imx6sx-pcie, pcie_aux for > > + imx8mq-pcie > > + > > + num-lanes: > > + const: 1 > > + > > + num-viewport: > > + const: 4 > > This is deprecated. The driver detects this now. [Richard Zhu] Got that, thanks. Would delete this property. > > > + > > + fsl,imx7d-pcie-phy: > > + $ref: /schemas/types.yaml#/definitions/phandle > > + description: A phandle to an fsl,imx7d-pcie-phy node. Additional > > + required properties for imx7d-pcie and imx8mq-pcie. > > + > > + power-domains: > > + maxItems: 2 > > + description: The first phandle pointing to the DISPLAY domain for > > + imx6sx-pcie, to PCIE_PHY power domain for imx7d-pcie and > imx8mq-pcie. > > + The second phandle pointing to the PCIE_PHY power domains for > imx6sx-pcie. > > Split the description: > > items: > - description: ... > - description: ... > [Richard Zhu] Got that, thanks. > > + > > + power-domain-names: > > + maxItems: 2 > > You have to define the names. > [Richard Zhu] Okay, would be added in next version patch. > > + > > + resets: > > + maxItems: 3 > > + description: Phandles to PCIe-related reset lines exposed by SRC > > + IP block. Additional required by imx7d-pcie and imx8mq-pcie. > > + > > + reset-names: > > + items: > > + - const: pciephy > > + - const: apps > > + - const: turnoff > > + > > + fsl,tx-deemph-gen1: > > + description: Gen1 De-emphasis value (optional required). > > optional or required? [Richard Zhu] Yes, it's optional required by i.MX6Q/DL PCIe. > > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + default: 0 > > + > > + fsl,tx-deemph-gen2-3p5db: > > + description: Gen2 (3.5db) De-emphasis value (optional required). > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + default: 0 > > + > > + fsl,tx-deemph-gen2-6db: > > + description: Gen2 (6db) De-emphasis value (optional required). > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + default: 20 > > + > > + fsl,tx-swing-full: > > + description: Gen2 TX SWING FULL value (optional required). > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + default: 127 > > + > > + fsl,tx-swing-low: > > + description: TX launch amplitude swing_low value (optional required). > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + default: 127 > > + > > + fsl,max-link-speed: > > + description: Specify PCI Gen for link capability (optional required). > > + Note that the IMX6 LVDS clock outputs do not meet gen2 jitter > > + requirements and thus for gen2 capability a gen2 compliant clock > > + generator should be used and configured. > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + enum: [1, 2, 3, 4] > > + default: 1 > > + > > + reset-gpio: > > Deprecated. [Richard Zhu] The reset-gpio is designed to connect the PERST# signal on the slot. And it is derived from the previous binding document. > > > + description: Should specify the GPIO for controlling the PCI bus device > > + reset signal. It's not polarity aware and defaults to active-low reset > > + sequence (L=reset state, H=operation state) (optional required). > > + > > + reset-gpio-active-high: > > Deprecated. This should only be used on older compatibles I hope... [Richard Zhu] Because that the polarity of the PERST-active is reversed on some i.MX6 customer's hardware board designs. This property is derived from the previous binding document, and only be used on older compatibles. So, this is optional required. > > > + description: If present then the reset sequence using the GPIO > > + specified in the "reset-gpio" property is reversed (H=reset state, > > + L=operation state) (optional required). > > + > > + vpcie-supply: > > + description: Should specify the regulator in charge of PCIe port power. > > + The regulator will be enabled when initializing the PCIe host and > > + disabled either as part of the init process or when shutting down > > + the host (optional required). > > + > > + vph-supply: > > + description: Should specify the regulator in charge of VPH one of > > + the three PCIe PHY powers. This regulator can be supplied by both > > + 1.8v and 3.3v voltage supplies (optional required). > > + > > +required: > > + - compatible > > + - reg > > + - reg-names > > + - "#address-cells" > > + - "#size-cells" > > + - device_type > > + - bus-range > > + - ranges > > + - num-lanes > > + - num-viewport > > + - interrupts > > + - interrupt-names > > + - "#interrupt-cells" > > + - interrupt-map-mask > > + - interrupt-map > > + - clocks > > + - clock-names > > + > > +unevaluatedProperties: false > > + > > +examples: > > + - | > > + #include <dt-bindings/clock/imx6qdl-clock.h> > > + #include <dt-bindings/interrupt-controller/arm-gic.h> > > + > > + pcie: pcie@1ffc000 { > > + compatible = "fsl,imx6q-pcie"; > > + reg = <0x01ffc000 0x04000>, > > + <0x01f00000 0x80000>; > > + reg-names = "dbi", "config"; > > + #address-cells = <3>; > > + #size-cells = <2>; > > + device_type = "pci"; > > + bus-range = <0x00 0xff>; > > + ranges = <0x81000000 0 0 0x01f80000 0 > 0x00010000>, > > + <0x82000000 0 0x01000000 0x01000000 0 > 0x00f00000>; > > + num-lanes = <1>; > > + num-viewport = <4>; > > + interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > > + interrupt-names = "msi"; > > + #interrupt-cells = <1>; > > + interrupt-map-mask = <0 0 0 0x7>; > > + interrupt-map = <0 0 0 1 &gpc GIC_SPI 123 > IRQ_TYPE_LEVEL_HIGH>, > > + <0 0 0 2 &gpc GIC_SPI 122 > IRQ_TYPE_LEVEL_HIGH>, > > + <0 0 0 3 &gpc GIC_SPI 121 > IRQ_TYPE_LEVEL_HIGH>, > > + <0 0 0 4 &gpc GIC_SPI 120 > IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&clks IMX6QDL_CLK_PCIE_AXI>, > > + <&clks IMX6QDL_CLK_LVDS1_GATE>, > > + <&clks IMX6QDL_CLK_PCIE_REF_125M>; > > + clock-names = "pcie", "pcie_bus", "pcie_phy"; > > + status = "disabled"; > > Why are you disabling your example? [Richard Zhu] Would remove the disable later. Thanks. > > > + }; > > +... > > diff --git a/MAINTAINERS b/MAINTAINERS index > > f73fef615702..a1e2cc0be9e8 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -14187,7 +14187,7 @@ M: Lucas Stach <l.stach@pengutronix.de> > > L: linux-pci@vger.kernel.org > > L: linux-arm-kernel@lists.infradead.org (moderated for > non-subscribers) > > S: Maintained > > -F: Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt > > +F: Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml > > F: drivers/pci/controller/dwc/*imx6* > > > > PCI DRIVER FOR FU740 > > -- > > 2.25.1 > > > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v1 2/3] arm: dts: imx: fix the schema check errors 2021-08-19 7:14 [PATCH v1 0/3] dt-bindings: PCI: imx6: convert the imx pcie Richard Zhu 2021-08-19 7:14 ` [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema Richard Zhu @ 2021-08-19 7:14 ` Richard Zhu 2021-08-19 11:10 ` Lucas Stach 2021-08-19 7:14 ` [PATCH v1 3/3] arm64: dts: imx8mq: " Richard Zhu 2 siblings, 1 reply; 9+ messages in thread From: Richard Zhu @ 2021-08-19 7:14 UTC (permalink / raw) To: robh, l.stach, galak, shawnguo Cc: devicetree, linux-arm-kernel, linux-kernel, linux-imx, kernel, Richard Zhu - the ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# - num-viewport is required, add it in imx6sx.dtsi. - only one propert is allowed in the compatible string, remove "snps,dw-pcie". Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> --- arch/arm/boot/dts/imx6qdl.dtsi | 6 +++--- arch/arm/boot/dts/imx6qp.dtsi | 2 +- arch/arm/boot/dts/imx6sx.dtsi | 7 ++++--- arch/arm/boot/dts/imx7d.dtsi | 6 +++--- 4 files changed, 11 insertions(+), 10 deletions(-) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index 82e01ce026ea..cb43c72806a1 100644 --- a/arch/arm/boot/dts/imx6qdl.dtsi +++ b/arch/arm/boot/dts/imx6qdl.dtsi @@ -264,7 +264,7 @@ L2: cache-controller@a02000 { }; pcie: pcie@1ffc000 { - compatible = "fsl,imx6q-pcie", "snps,dw-pcie"; + compatible = "fsl,imx6q-pcie"; reg = <0x01ffc000 0x04000>, <0x01f00000 0x80000>; reg-names = "dbi", "config"; @@ -272,8 +272,8 @@ pcie: pcie@1ffc000 { #size-cells = <2>; device_type = "pci"; bus-range = <0x00 0xff>; - ranges = <0x81000000 0 0 0x01f80000 0 0x00010000 /* downstream I/O */ - 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ + ranges = <0x81000000 0 0 0x01f80000 0 0x00010000>, /* downstream I/O */ + <0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ num-lanes = <1>; num-viewport = <4>; interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; diff --git a/arch/arm/boot/dts/imx6qp.dtsi b/arch/arm/boot/dts/imx6qp.dtsi index b310f13a53f2..050365513836 100644 --- a/arch/arm/boot/dts/imx6qp.dtsi +++ b/arch/arm/boot/dts/imx6qp.dtsi @@ -110,5 +110,5 @@ &mmdc0 { }; &pcie { - compatible = "fsl,imx6qp-pcie", "snps,dw-pcie"; + compatible = "fsl,imx6qp-pcie"; }; diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi index 8516730778df..0324f9e17389 100644 --- a/arch/arm/boot/dts/imx6sx.dtsi +++ b/arch/arm/boot/dts/imx6sx.dtsi @@ -1395,16 +1395,17 @@ pwm8: pwm@22b0000 { }; pcie: pcie@8ffc000 { - compatible = "fsl,imx6sx-pcie", "snps,dw-pcie"; + compatible = "fsl,imx6sx-pcie"; reg = <0x08ffc000 0x04000>, <0x08f00000 0x80000>; reg-names = "dbi", "config"; #address-cells = <3>; #size-cells = <2>; device_type = "pci"; bus-range = <0x00 0xff>; - ranges = <0x81000000 0 0 0x08f80000 0 0x00010000 /* downstream I/O */ - 0x82000000 0 0x08000000 0x08000000 0 0x00f00000>; /* non-prefetchable memory */ + ranges = <0x81000000 0 0 0x08f80000 0 0x00010000>, /* downstream I/O */ + <0x82000000 0 0x08000000 0x08000000 0 0x00f00000>; /* non-prefetchable memory */ num-lanes = <1>; + num-viewport = <4>; interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; interrupt-names = "msi"; #interrupt-cells = <1>; diff --git a/arch/arm/boot/dts/imx7d.dtsi b/arch/arm/boot/dts/imx7d.dtsi index b0bcfa9094a3..0cb8c8df87ef 100644 --- a/arch/arm/boot/dts/imx7d.dtsi +++ b/arch/arm/boot/dts/imx7d.dtsi @@ -164,7 +164,7 @@ fec2: ethernet@30bf0000 { }; pcie: pcie@33800000 { - compatible = "fsl,imx7d-pcie", "snps,dw-pcie"; + compatible = "fsl,imx7d-pcie"; reg = <0x33800000 0x4000>, <0x4ff00000 0x80000>; reg-names = "dbi", "config"; @@ -172,8 +172,8 @@ pcie: pcie@33800000 { #size-cells = <2>; device_type = "pci"; bus-range = <0x00 0xff>; - ranges = <0x81000000 0 0 0x4ff80000 0 0x00010000 /* downstream I/O */ - 0x82000000 0 0x40000000 0x40000000 0 0x0ff00000>; /* non-prefetchable memory */ + ranges = <0x81000000 0 0 0x4ff80000 0 0x00010000>, /* downstream I/O */ + <0x82000000 0 0x40000000 0x40000000 0 0x0ff00000>; /* non-prefetchable memory */ num-lanes = <1>; num-viewport = <4>; interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>; -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1 2/3] arm: dts: imx: fix the schema check errors 2021-08-19 7:14 ` [PATCH v1 2/3] arm: dts: imx: fix the schema check errors Richard Zhu @ 2021-08-19 11:10 ` Lucas Stach 2021-08-23 1:52 ` Richard Zhu 0 siblings, 1 reply; 9+ messages in thread From: Lucas Stach @ 2021-08-19 11:10 UTC (permalink / raw) To: Richard Zhu, robh, galak, shawnguo Cc: devicetree, linux-kernel, linux-imx, kernel, linux-arm-kernel Hi Richard, Am Donnerstag, dem 19.08.2021 um 15:14 +0800 schrieb Richard Zhu: > - the ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# I failed to parse this part of the commit message and expected to see an actual change in the addresses of the ranges or something. I think it would be better to phrase this something like: "group ranges property by region" or something to make it clear that this is purely a DTS representation change, with no functional change. > - num-viewport is required, add it in imx6sx.dtsi. > - only one propert is allowed in the compatible string, remove > "snps,dw-pcie". > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> Other than the nit above, this is: Reviewed-by: Lucas Stach <l.stach@pengutronix.de> > --- > arch/arm/boot/dts/imx6qdl.dtsi | 6 +++--- > arch/arm/boot/dts/imx6qp.dtsi | 2 +- > arch/arm/boot/dts/imx6sx.dtsi | 7 ++++--- > arch/arm/boot/dts/imx7d.dtsi | 6 +++--- > 4 files changed, 11 insertions(+), 10 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi > index 82e01ce026ea..cb43c72806a1 100644 > --- a/arch/arm/boot/dts/imx6qdl.dtsi > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > @@ -264,7 +264,7 @@ L2: cache-controller@a02000 { > }; > > pcie: pcie@1ffc000 { > - compatible = "fsl,imx6q-pcie", "snps,dw-pcie"; > + compatible = "fsl,imx6q-pcie"; > reg = <0x01ffc000 0x04000>, > <0x01f00000 0x80000>; > reg-names = "dbi", "config"; > @@ -272,8 +272,8 @@ pcie: pcie@1ffc000 { > #size-cells = <2>; > device_type = "pci"; > bus-range = <0x00 0xff>; > - ranges = <0x81000000 0 0 0x01f80000 0 0x00010000 /* downstream I/O */ > - 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ > + ranges = <0x81000000 0 0 0x01f80000 0 0x00010000>, /* downstream I/O */ > + <0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ > num-lanes = <1>; > num-viewport = <4>; > interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > diff --git a/arch/arm/boot/dts/imx6qp.dtsi b/arch/arm/boot/dts/imx6qp.dtsi > index b310f13a53f2..050365513836 100644 > --- a/arch/arm/boot/dts/imx6qp.dtsi > +++ b/arch/arm/boot/dts/imx6qp.dtsi > @@ -110,5 +110,5 @@ &mmdc0 { > }; > > &pcie { > - compatible = "fsl,imx6qp-pcie", "snps,dw-pcie"; > + compatible = "fsl,imx6qp-pcie"; > }; > diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi > index 8516730778df..0324f9e17389 100644 > --- a/arch/arm/boot/dts/imx6sx.dtsi > +++ b/arch/arm/boot/dts/imx6sx.dtsi > @@ -1395,16 +1395,17 @@ pwm8: pwm@22b0000 { > }; > > pcie: pcie@8ffc000 { > - compatible = "fsl,imx6sx-pcie", "snps,dw-pcie"; > + compatible = "fsl,imx6sx-pcie"; > reg = <0x08ffc000 0x04000>, <0x08f00000 0x80000>; > reg-names = "dbi", "config"; > #address-cells = <3>; > #size-cells = <2>; > device_type = "pci"; > bus-range = <0x00 0xff>; > - ranges = <0x81000000 0 0 0x08f80000 0 0x00010000 /* downstream I/O */ > - 0x82000000 0 0x08000000 0x08000000 0 0x00f00000>; /* non-prefetchable memory */ > + ranges = <0x81000000 0 0 0x08f80000 0 0x00010000>, /* downstream I/O */ > + <0x82000000 0 0x08000000 0x08000000 0 0x00f00000>; /* non-prefetchable memory */ > num-lanes = <1>; > + num-viewport = <4>; > interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > interrupt-names = "msi"; > #interrupt-cells = <1>; > diff --git a/arch/arm/boot/dts/imx7d.dtsi b/arch/arm/boot/dts/imx7d.dtsi > index b0bcfa9094a3..0cb8c8df87ef 100644 > --- a/arch/arm/boot/dts/imx7d.dtsi > +++ b/arch/arm/boot/dts/imx7d.dtsi > @@ -164,7 +164,7 @@ fec2: ethernet@30bf0000 { > }; > > pcie: pcie@33800000 { > - compatible = "fsl,imx7d-pcie", "snps,dw-pcie"; > + compatible = "fsl,imx7d-pcie"; > reg = <0x33800000 0x4000>, > <0x4ff00000 0x80000>; > reg-names = "dbi", "config"; > @@ -172,8 +172,8 @@ pcie: pcie@33800000 { > #size-cells = <2>; > device_type = "pci"; > bus-range = <0x00 0xff>; > - ranges = <0x81000000 0 0 0x4ff80000 0 0x00010000 /* downstream I/O */ > - 0x82000000 0 0x40000000 0x40000000 0 0x0ff00000>; /* non-prefetchable memory */ > + ranges = <0x81000000 0 0 0x4ff80000 0 0x00010000>, /* downstream I/O */ > + <0x82000000 0 0x40000000 0x40000000 0 0x0ff00000>; /* non-prefetchable memory */ > num-lanes = <1>; > num-viewport = <4>; > interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>; ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH v1 2/3] arm: dts: imx: fix the schema check errors 2021-08-19 11:10 ` Lucas Stach @ 2021-08-23 1:52 ` Richard Zhu 0 siblings, 0 replies; 9+ messages in thread From: Richard Zhu @ 2021-08-23 1:52 UTC (permalink / raw) To: Lucas Stach, robh, galak, shawnguo Cc: devicetree, linux-kernel, dl-linux-imx, kernel, linux-arm-kernel > -----Original Message----- > From: Lucas Stach <l.stach@pengutronix.de> > Sent: Thursday, August 19, 2021 7:11 PM > To: Richard Zhu <hongxing.zhu@nxp.com>; robh@kernel.org; > galak@kernel.crashing.org; shawnguo@kernel.org > Cc: devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; dl-linux-imx > <linux-imx@nxp.com>; kernel@pengutronix.de; > linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH v1 2/3] arm: dts: imx: fix the schema check errors > > Hi Richard, > > Am Donnerstag, dem 19.08.2021 um 15:14 +0800 schrieb Richard Zhu: > > - the ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# > > I failed to parse this part of the commit message and expected to see an > actual change in the addresses of the ranges or something. I think it would be > better to phrase this something like: "group ranges property by region" or > something to make it clear that this is purely a DTS representation change, > with no functional change. > [Richard Zhu] Hi Lucas: Thanks. Would change as "group ranges property by region" refer to your review comments. Best Regards Richard Zhu > > - num-viewport is required, add it in imx6sx.dtsi. > > - only one propert is allowed in the compatible string, remove > > "snps,dw-pcie". > > > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> > > Other than the nit above, this is: > Reviewed-by: Lucas Stach <l.stach@pengutronix.de> > > > --- > > arch/arm/boot/dts/imx6qdl.dtsi | 6 +++--- > > arch/arm/boot/dts/imx6qp.dtsi | 2 +- arch/arm/boot/dts/imx6sx.dtsi > > | 7 ++++--- > > arch/arm/boot/dts/imx7d.dtsi | 6 +++--- > > 4 files changed, 11 insertions(+), 10 deletions(-) > > > > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi > > b/arch/arm/boot/dts/imx6qdl.dtsi index 82e01ce026ea..cb43c72806a1 > > 100644 > > --- a/arch/arm/boot/dts/imx6qdl.dtsi > > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > > @@ -264,7 +264,7 @@ L2: cache-controller@a02000 { > > }; > > > > pcie: pcie@1ffc000 { > > - compatible = "fsl,imx6q-pcie", "snps,dw-pcie"; > > + compatible = "fsl,imx6q-pcie"; > > reg = <0x01ffc000 0x04000>, > > <0x01f00000 0x80000>; > > reg-names = "dbi", "config"; > > @@ -272,8 +272,8 @@ pcie: pcie@1ffc000 { > > #size-cells = <2>; > > device_type = "pci"; > > bus-range = <0x00 0xff>; > > - ranges = <0x81000000 0 0 0x01f80000 0 > 0x00010000 /* downstream I/O */ > > - 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; > /* non-prefetchable memory */ > > + ranges = <0x81000000 0 0 0x01f80000 0 > 0x00010000>, /* downstream I/O */ > > + <0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; > /* > > +non-prefetchable memory */ > > num-lanes = <1>; > > num-viewport = <4>; > > interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; diff --git > > a/arch/arm/boot/dts/imx6qp.dtsi b/arch/arm/boot/dts/imx6qp.dtsi index > > b310f13a53f2..050365513836 100644 > > --- a/arch/arm/boot/dts/imx6qp.dtsi > > +++ b/arch/arm/boot/dts/imx6qp.dtsi > > @@ -110,5 +110,5 @@ &mmdc0 { > > }; > > > > &pcie { > > - compatible = "fsl,imx6qp-pcie", "snps,dw-pcie"; > > + compatible = "fsl,imx6qp-pcie"; > > }; > > diff --git a/arch/arm/boot/dts/imx6sx.dtsi > > b/arch/arm/boot/dts/imx6sx.dtsi index 8516730778df..0324f9e17389 > > 100644 > > --- a/arch/arm/boot/dts/imx6sx.dtsi > > +++ b/arch/arm/boot/dts/imx6sx.dtsi > > @@ -1395,16 +1395,17 @@ pwm8: pwm@22b0000 { > > }; > > > > pcie: pcie@8ffc000 { > > - compatible = "fsl,imx6sx-pcie", "snps,dw-pcie"; > > + compatible = "fsl,imx6sx-pcie"; > > reg = <0x08ffc000 0x04000>, <0x08f00000 0x80000>; > > reg-names = "dbi", "config"; > > #address-cells = <3>; > > #size-cells = <2>; > > device_type = "pci"; > > bus-range = <0x00 0xff>; > > - ranges = <0x81000000 0 0 0x08f80000 0 > 0x00010000 /* downstream I/O */ > > - 0x82000000 0 0x08000000 0x08000000 0 0x00f00000>; > /* non-prefetchable memory */ > > + ranges = <0x81000000 0 0 0x08f80000 0 > 0x00010000>, /* downstream I/O */ > > + <0x82000000 0 0x08000000 0x08000000 0 0x00f00000>; > /* > > +non-prefetchable memory */ > > num-lanes = <1>; > > + num-viewport = <4>; > > interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>; > > interrupt-names = "msi"; > > #interrupt-cells = <1>; > > diff --git a/arch/arm/boot/dts/imx7d.dtsi > > b/arch/arm/boot/dts/imx7d.dtsi index b0bcfa9094a3..0cb8c8df87ef > 100644 > > --- a/arch/arm/boot/dts/imx7d.dtsi > > +++ b/arch/arm/boot/dts/imx7d.dtsi > > @@ -164,7 +164,7 @@ fec2: ethernet@30bf0000 { > > }; > > > > pcie: pcie@33800000 { > > - compatible = "fsl,imx7d-pcie", "snps,dw-pcie"; > > + compatible = "fsl,imx7d-pcie"; > > reg = <0x33800000 0x4000>, > > <0x4ff00000 0x80000>; > > reg-names = "dbi", "config"; > > @@ -172,8 +172,8 @@ pcie: pcie@33800000 { > > #size-cells = <2>; > > device_type = "pci"; > > bus-range = <0x00 0xff>; > > - ranges = <0x81000000 0 0 0x4ff80000 0 0x00010000 > /* downstream I/O */ > > - 0x82000000 0 0x40000000 0x40000000 0 0x0ff00000>; /* > non-prefetchable memory */ > > + ranges = <0x81000000 0 0 0x4ff80000 0 0x00010000>, > /* downstream I/O */ > > + <0x82000000 0 0x40000000 0x40000000 0 0x0ff00000>; /* > > +non-prefetchable memory */ > > num-lanes = <1>; > > num-viewport = <4>; > > interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>; > ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v1 3/3] arm64: dts: imx8mq: fix the schema check errors 2021-08-19 7:14 [PATCH v1 0/3] dt-bindings: PCI: imx6: convert the imx pcie Richard Zhu 2021-08-19 7:14 ` [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema Richard Zhu 2021-08-19 7:14 ` [PATCH v1 2/3] arm: dts: imx: fix the schema check errors Richard Zhu @ 2021-08-19 7:14 ` Richard Zhu 2021-08-19 11:11 ` Lucas Stach 2 siblings, 1 reply; 9+ messages in thread From: Richard Zhu @ 2021-08-19 7:14 UTC (permalink / raw) To: robh, l.stach, galak, shawnguo Cc: devicetree, linux-arm-kernel, linux-kernel, linux-imx, kernel, Richard Zhu The ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi index 91df9c5350ae..45895dad85e1 100644 --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi @@ -1364,8 +1364,8 @@ pcie0: pcie@33800000 { #size-cells = <2>; device_type = "pci"; bus-range = <0x00 0xff>; - ranges = <0x81000000 0 0x00000000 0x1ff80000 0 0x00010000 /* downstream I/O 64KB */ - 0x82000000 0 0x18000000 0x18000000 0 0x07f00000>; /* non-prefetchable memory */ + ranges = <0x81000000 0 0x00000000 0x1ff80000 0 0x00010000>, /* downstream I/O 64KB */ + <0x82000000 0 0x18000000 0x18000000 0 0x07f00000>; /* non-prefetchable memory */ num-lanes = <1>; num-viewport = <4>; interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>; @@ -1402,8 +1402,8 @@ pcie1: pcie@33c00000 { #address-cells = <3>; #size-cells = <2>; device_type = "pci"; - ranges = <0x81000000 0 0x00000000 0x27f80000 0 0x00010000 /* downstream I/O 64KB */ - 0x82000000 0 0x20000000 0x20000000 0 0x07f00000>; /* non-prefetchable memory */ + ranges = <0x81000000 0 0x00000000 0x27f80000 0 0x00010000>, /* downstream I/O 64KB */ + <0x82000000 0 0x20000000 0x20000000 0 0x07f00000>; /* non-prefetchable memory */ num-lanes = <1>; num-viewport = <4>; interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>; -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1 3/3] arm64: dts: imx8mq: fix the schema check errors 2021-08-19 7:14 ` [PATCH v1 3/3] arm64: dts: imx8mq: " Richard Zhu @ 2021-08-19 11:11 ` Lucas Stach 0 siblings, 0 replies; 9+ messages in thread From: Lucas Stach @ 2021-08-19 11:11 UTC (permalink / raw) To: Richard Zhu, robh, galak, shawnguo Cc: devicetree, linux-kernel, linux-imx, kernel, linux-arm-kernel Am Donnerstag, dem 19.08.2021 um 15:14 +0800 schrieb Richard Zhu: > The ranges should be aligned to $ref: /schemas/pci/pci-bus.yaml# > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> Same comment as on the last patch, but still: Reviewed-by: Lucas Stach <l.stach@pengutronix.de> > --- > arch/arm64/boot/dts/freescale/imx8mq.dtsi | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi > index 91df9c5350ae..45895dad85e1 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi > @@ -1364,8 +1364,8 @@ pcie0: pcie@33800000 { > #size-cells = <2>; > device_type = "pci"; > bus-range = <0x00 0xff>; > - ranges = <0x81000000 0 0x00000000 0x1ff80000 0 0x00010000 /* downstream I/O 64KB */ > - 0x82000000 0 0x18000000 0x18000000 0 0x07f00000>; /* non-prefetchable memory */ > + ranges = <0x81000000 0 0x00000000 0x1ff80000 0 0x00010000>, /* downstream I/O 64KB */ > + <0x82000000 0 0x18000000 0x18000000 0 0x07f00000>; /* non-prefetchable memory */ > num-lanes = <1>; > num-viewport = <4>; > interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>; > @@ -1402,8 +1402,8 @@ pcie1: pcie@33c00000 { > #address-cells = <3>; > #size-cells = <2>; > device_type = "pci"; > - ranges = <0x81000000 0 0x00000000 0x27f80000 0 0x00010000 /* downstream I/O 64KB */ > - 0x82000000 0 0x20000000 0x20000000 0 0x07f00000>; /* non-prefetchable memory */ > + ranges = <0x81000000 0 0x00000000 0x27f80000 0 0x00010000>, /* downstream I/O 64KB */ > + <0x82000000 0 0x20000000 0x20000000 0 0x07f00000>; /* non-prefetchable memory */ > num-lanes = <1>; > num-viewport = <4>; > interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>; ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-08-24 6:04 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-19 7:14 [PATCH v1 0/3] dt-bindings: PCI: imx6: convert the imx pcie Richard Zhu 2021-08-19 7:14 ` [PATCH v1 1/3] dt-bindings: PCI: imx6: convert the imx pcie controller to dtschema Richard Zhu 2021-08-23 18:34 ` Rob Herring 2021-08-24 6:04 ` Richard Zhu 2021-08-19 7:14 ` [PATCH v1 2/3] arm: dts: imx: fix the schema check errors Richard Zhu 2021-08-19 11:10 ` Lucas Stach 2021-08-23 1:52 ` Richard Zhu 2021-08-19 7:14 ` [PATCH v1 3/3] arm64: dts: imx8mq: " Richard Zhu 2021-08-19 11:11 ` Lucas Stach
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).