* [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema
@ 2022-03-28 2:11 Kunihiko Hayashi
2022-03-28 12:51 ` Rob Herring
2022-03-28 16:22 ` Rob Herring
0 siblings, 2 replies; 6+ messages in thread
From: Kunihiko Hayashi @ 2022-03-28 2:11 UTC (permalink / raw)
To: Bjorn Helgaas, Rob Herring, Krzysztof Kozlowski
Cc: Masami Hiramatsu, linux-pci, devicetree, linux-arm-kernel,
linux-kernel, Kunihiko Hayashi
Convert the file into a JSON description at the yaml format.
Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
---
.../bindings/pci/socionext,uniphier-pcie.yaml | 100 ++++++++++++++++++
.../devicetree/bindings/pci/uniphier-pcie.txt | 82 --------------
MAINTAINERS | 2 +-
3 files changed, 101 insertions(+), 83 deletions(-)
create mode 100644 Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
delete mode 100644 Documentation/devicetree/bindings/pci/uniphier-pcie.txt
diff --git a/Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml b/Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
new file mode 100644
index 000000000000..57176f62f955
--- /dev/null
+++ b/Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
@@ -0,0 +1,100 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/pci/socionext,uniphier-pcie.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Socionext UniPhier PCIe host controller
+
+description: |
+ UniPhier PCIe host controller is based on the Synopsys DesignWare
+ PCI core. It shares common features with the PCIe DesignWare core and
+ inherits common properties defined in
+ Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml.
+
+maintainers:
+ - Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
+
+allOf:
+ - $ref: /schemas/pci/snps,dw-pcie.yaml#
+
+properties:
+ compatible:
+ enum:
+ - socionext,uniphier-pcie
+
+ reg:
+ minItems: 3
+ maxItems: 4
+
+ reg-names:
+ oneOf:
+ - items:
+ - const: dbi
+ - const: link
+ - const: config
+ - items:
+ - const: dbi
+ - const: link
+ - const: config
+ - const: atu
+
+ clocks:
+ maxItems: 1
+
+ resets:
+ maxItems: 1
+
+ num-viewport: true
+
+ num-lanes: true
+
+ phys:
+ maxItems: 1
+
+ phy-names:
+ const: pcie-phy
+
+required:
+ - compatible
+ - reg
+ - reg-names
+ - clocks
+ - resets
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ pcie: pcie@66000000 {
+ compatible = "socionext,uniphier-pcie";
+ reg-names = "dbi", "link", "config";
+ reg = <0x66000000 0x1000>, <0x66010000 0x10000>, <0x2fff0000 0x10000>;
+ #address-cells = <3>;
+ #size-cells = <2>;
+ clocks = <&sys_clk 24>;
+ resets = <&sys_rst 24>;
+ num-lanes = <1>;
+ num-viewport = <1>;
+ bus-range = <0x0 0xff>;
+ device_type = "pci";
+ ranges = <0x81000000 0 0x00000000 0x2ffe0000 0 0x00010000>,
+ <0x82000000 0 0x00000000 0x20000000 0 0x0ffe0000>;
+ phy-names = "pcie-phy";
+ phys = <&pcie_phy>;
+ #interrupt-cells = <1>;
+ interrupt-names = "dma", "msi";
+ interrupts = <0 224 4>, <0 225 4>;
+ interrupt-map-mask = <0 0 0 7>;
+ interrupt-map = <0 0 0 1 &pcie_intc 0>,
+ <0 0 0 2 &pcie_intc 1>,
+ <0 0 0 3 &pcie_intc 2>,
+ <0 0 0 4 &pcie_intc 3>;
+
+ pcie_intc: legacy-interrupt-controller {
+ interrupt-controller;
+ #interrupt-cells = <1>;
+ interrupt-parent = <&gic>;
+ interrupts = <0 226 4>;
+ };
+ };
diff --git a/Documentation/devicetree/bindings/pci/uniphier-pcie.txt b/Documentation/devicetree/bindings/pci/uniphier-pcie.txt
deleted file mode 100644
index 359585db049f..000000000000
--- a/Documentation/devicetree/bindings/pci/uniphier-pcie.txt
+++ /dev/null
@@ -1,82 +0,0 @@
-Socionext UniPhier PCIe host controller bindings
-
-This describes the devicetree bindings for PCIe host controller implemented
-on Socionext UniPhier SoCs.
-
-UniPhier PCIe host controller is based on the Synopsys DesignWare PCI core.
-It shares common functions with the PCIe DesignWare core driver and inherits
-common properties defined in
-Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml.
-
-Required properties:
-- compatible: Should be "socionext,uniphier-pcie".
-- reg: Specifies offset and length of the register set for the device.
- According to the reg-names, appropriate register sets are required.
-- reg-names: Must include the following entries:
- "dbi" - controller configuration registers
- "link" - SoC-specific glue layer registers
- "config" - PCIe configuration space
- "atu" - iATU registers for DWC version 4.80 or later
-- clocks: A phandle to the clock gate for PCIe glue layer including
- the host controller.
-- resets: A phandle to the reset line for PCIe glue layer including
- the host controller.
-- interrupts: A list of interrupt specifiers. According to the
- interrupt-names, appropriate interrupts are required.
-- interrupt-names: Must include the following entries:
- "dma" - DMA interrupt
- "msi" - MSI interrupt
-
-Optional properties:
-- phys: A phandle to generic PCIe PHY. According to the phy-names, appropriate
- phys are required.
-- phy-names: Must be "pcie-phy".
-
-Required sub-node:
-- legacy-interrupt-controller: Specifies interrupt controller for legacy PCI
- interrupts.
-
-Required properties for legacy-interrupt-controller:
-- interrupt-controller: identifies the node as an interrupt controller.
-- #interrupt-cells: specifies the number of cells needed to encode an
- interrupt source. The value must be 1.
-- interrupt-parent: Phandle to the parent interrupt controller.
-- interrupts: An interrupt specifier for legacy interrupt.
-
-Example:
-
- pcie: pcie@66000000 {
- compatible = "socionext,uniphier-pcie", "snps,dw-pcie";
- status = "disabled";
- reg-names = "dbi", "link", "config";
- reg = <0x66000000 0x1000>, <0x66010000 0x10000>,
- <0x2fff0000 0x10000>;
- #address-cells = <3>;
- #size-cells = <2>;
- clocks = <&sys_clk 24>;
- resets = <&sys_rst 24>;
- num-lanes = <1>;
- num-viewport = <1>;
- bus-range = <0x0 0xff>;
- device_type = "pci";
- ranges =
- /* downstream I/O */
- <0x81000000 0 0x00000000 0x2ffe0000 0 0x00010000
- /* non-prefetchable memory */
- 0x82000000 0 0x00000000 0x20000000 0 0x0ffe0000>;
- #interrupt-cells = <1>;
- interrupt-names = "dma", "msi";
- interrupts = <0 224 4>, <0 225 4>;
- interrupt-map-mask = <0 0 0 7>;
- interrupt-map = <0 0 0 1 &pcie_intc 0>, /* INTA */
- <0 0 0 2 &pcie_intc 1>, /* INTB */
- <0 0 0 3 &pcie_intc 2>, /* INTC */
- <0 0 0 4 &pcie_intc 3>; /* INTD */
-
- pcie_intc: legacy-interrupt-controller {
- interrupt-controller;
- #interrupt-cells = <1>;
- interrupt-parent = <&gic>;
- interrupts = <0 226 4>;
- };
- };
diff --git a/MAINTAINERS b/MAINTAINERS
index 4cc47b2dbdc9..c1d377be991c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15337,7 +15337,7 @@ PCIE DRIVER FOR SOCIONEXT UNIPHIER
M: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
L: linux-pci@vger.kernel.org
S: Maintained
-F: Documentation/devicetree/bindings/pci/uniphier-pcie*
+F: Documentation/devicetree/bindings/pci/socionext,uniphier-pcie*
F: drivers/pci/controller/dwc/pcie-uniphier*
PCIE DRIVER FOR ST SPEAR13XX
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema
2022-03-28 2:11 [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema Kunihiko Hayashi
@ 2022-03-28 12:51 ` Rob Herring
2022-03-28 16:23 ` Rob Herring
2022-03-28 16:22 ` Rob Herring
1 sibling, 1 reply; 6+ messages in thread
From: Rob Herring @ 2022-03-28 12:51 UTC (permalink / raw)
To: Kunihiko Hayashi
Cc: linux-kernel, devicetree, linux-pci, linux-arm-kernel,
Rob Herring, Krzysztof Kozlowski, Bjorn Helgaas,
Masami Hiramatsu
On Mon, 28 Mar 2022 11:11:38 +0900, Kunihiko Hayashi wrote:
> Convert the file into a JSON description at the yaml format.
>
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
> .../bindings/pci/socionext,uniphier-pcie.yaml | 100 ++++++++++++++++++
> .../devicetree/bindings/pci/uniphier-pcie.txt | 82 --------------
> MAINTAINERS | 2 +-
> 3 files changed, 101 insertions(+), 83 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
> delete mode 100644 Documentation/devicetree/bindings/pci/uniphier-pcie.txt
>
Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.
Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.
Full log is available here: https://patchwork.ozlabs.org/patch/1609988
pcie@66000000: compatible: ['socionext,uniphier-pcie', 'snps,dw-pcie'] is too long
arch/arm64/boot/dts/socionext/uniphier-ld20-akebi96.dt.yaml
arch/arm64/boot/dts/socionext/uniphier-ld20-global.dt.yaml
arch/arm64/boot/dts/socionext/uniphier-ld20-ref.dt.yaml
arch/arm64/boot/dts/socionext/uniphier-pxs3-ref.dt.yaml
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema
2022-03-28 2:11 [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema Kunihiko Hayashi
2022-03-28 12:51 ` Rob Herring
@ 2022-03-28 16:22 ` Rob Herring
2022-03-29 10:54 ` Kunihiko Hayashi
1 sibling, 1 reply; 6+ messages in thread
From: Rob Herring @ 2022-03-28 16:22 UTC (permalink / raw)
To: Kunihiko Hayashi
Cc: Bjorn Helgaas, Krzysztof Kozlowski, Masami Hiramatsu, linux-pci,
devicetree, linux-arm-kernel, linux-kernel
On Mon, Mar 28, 2022 at 11:11:38AM +0900, Kunihiko Hayashi wrote:
> Convert the file into a JSON description at the yaml format.
>
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
> .../bindings/pci/socionext,uniphier-pcie.yaml | 100 ++++++++++++++++++
> .../devicetree/bindings/pci/uniphier-pcie.txt | 82 --------------
> MAINTAINERS | 2 +-
> 3 files changed, 101 insertions(+), 83 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
> delete mode 100644 Documentation/devicetree/bindings/pci/uniphier-pcie.txt
>
> diff --git a/Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml b/Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
> new file mode 100644
> index 000000000000..57176f62f955
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
> @@ -0,0 +1,100 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/pci/socionext,uniphier-pcie.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Socionext UniPhier PCIe host controller
> +
> +description: |
> + UniPhier PCIe host controller is based on the Synopsys DesignWare
> + PCI core. It shares common features with the PCIe DesignWare core and
> + inherits common properties defined in
> + Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml.
> +
> +maintainers:
> + - Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> +
> +allOf:
> + - $ref: /schemas/pci/snps,dw-pcie.yaml#
> +
> +properties:
> + compatible:
> + enum:
> + - socionext,uniphier-pcie
> +
> + reg:
> + minItems: 3
> + maxItems: 4
> +
> + reg-names:
> + oneOf:
> + - items:
> + - const: dbi
> + - const: link
> + - const: config
> + - items:
> + - const: dbi
> + - const: link
> + - const: config
> + - const: atu
You can have just the 2nd list plus 'minItems: 3' to do the same thing.
> +
> + clocks:
> + maxItems: 1
> +
> + resets:
> + maxItems: 1
> +
> + num-viewport: true
> +
> + num-lanes: true
> +
> + phys:
> + maxItems: 1
> +
> + phy-names:
> + const: pcie-phy
> +
> +required:
> + - compatible
> + - reg
> + - reg-names
> + - clocks
> + - resets
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + pcie: pcie@66000000 {
> + compatible = "socionext,uniphier-pcie";
> + reg-names = "dbi", "link", "config";
> + reg = <0x66000000 0x1000>, <0x66010000 0x10000>, <0x2fff0000 0x10000>;
> + #address-cells = <3>;
> + #size-cells = <2>;
> + clocks = <&sys_clk 24>;
> + resets = <&sys_rst 24>;
> + num-lanes = <1>;
> + num-viewport = <1>;
> + bus-range = <0x0 0xff>;
> + device_type = "pci";
> + ranges = <0x81000000 0 0x00000000 0x2ffe0000 0 0x00010000>,
> + <0x82000000 0 0x00000000 0x20000000 0 0x0ffe0000>;
> + phy-names = "pcie-phy";
> + phys = <&pcie_phy>;
> + #interrupt-cells = <1>;
> + interrupt-names = "dma", "msi";
> + interrupts = <0 224 4>, <0 225 4>;
> + interrupt-map-mask = <0 0 0 7>;
> + interrupt-map = <0 0 0 1 &pcie_intc 0>,
> + <0 0 0 2 &pcie_intc 1>,
> + <0 0 0 3 &pcie_intc 2>,
> + <0 0 0 4 &pcie_intc 3>;
> +
> + pcie_intc: legacy-interrupt-controller {
> + interrupt-controller;
> + #interrupt-cells = <1>;
> + interrupt-parent = <&gic>;
> + interrupts = <0 226 4>;
> + };
> + };
> diff --git a/Documentation/devicetree/bindings/pci/uniphier-pcie.txt b/Documentation/devicetree/bindings/pci/uniphier-pcie.txt
> deleted file mode 100644
> index 359585db049f..000000000000
> --- a/Documentation/devicetree/bindings/pci/uniphier-pcie.txt
> +++ /dev/null
> @@ -1,82 +0,0 @@
> -Socionext UniPhier PCIe host controller bindings
> -
> -This describes the devicetree bindings for PCIe host controller implemented
> -on Socionext UniPhier SoCs.
> -
> -UniPhier PCIe host controller is based on the Synopsys DesignWare PCI core.
> -It shares common functions with the PCIe DesignWare core driver and inherits
> -common properties defined in
> -Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml.
> -
> -Required properties:
> -- compatible: Should be "socionext,uniphier-pcie".
> -- reg: Specifies offset and length of the register set for the device.
> - According to the reg-names, appropriate register sets are required.
> -- reg-names: Must include the following entries:
> - "dbi" - controller configuration registers
> - "link" - SoC-specific glue layer registers
> - "config" - PCIe configuration space
> - "atu" - iATU registers for DWC version 4.80 or later
> -- clocks: A phandle to the clock gate for PCIe glue layer including
> - the host controller.
> -- resets: A phandle to the reset line for PCIe glue layer including
> - the host controller.
> -- interrupts: A list of interrupt specifiers. According to the
> - interrupt-names, appropriate interrupts are required.
> -- interrupt-names: Must include the following entries:
> - "dma" - DMA interrupt
> - "msi" - MSI interrupt
> -
> -Optional properties:
> -- phys: A phandle to generic PCIe PHY. According to the phy-names, appropriate
> - phys are required.
> -- phy-names: Must be "pcie-phy".
> -
> -Required sub-node:
> -- legacy-interrupt-controller: Specifies interrupt controller for legacy PCI
> - interrupts.
> -
> -Required properties for legacy-interrupt-controller:
> -- interrupt-controller: identifies the node as an interrupt controller.
> -- #interrupt-cells: specifies the number of cells needed to encode an
> - interrupt source. The value must be 1.
> -- interrupt-parent: Phandle to the parent interrupt controller.
> -- interrupts: An interrupt specifier for legacy interrupt.
> -
> -Example:
> -
> - pcie: pcie@66000000 {
> - compatible = "socionext,uniphier-pcie", "snps,dw-pcie";
> - status = "disabled";
> - reg-names = "dbi", "link", "config";
> - reg = <0x66000000 0x1000>, <0x66010000 0x10000>,
> - <0x2fff0000 0x10000>;
> - #address-cells = <3>;
> - #size-cells = <2>;
> - clocks = <&sys_clk 24>;
> - resets = <&sys_rst 24>;
> - num-lanes = <1>;
> - num-viewport = <1>;
> - bus-range = <0x0 0xff>;
> - device_type = "pci";
> - ranges =
> - /* downstream I/O */
> - <0x81000000 0 0x00000000 0x2ffe0000 0 0x00010000
> - /* non-prefetchable memory */
> - 0x82000000 0 0x00000000 0x20000000 0 0x0ffe0000>;
> - #interrupt-cells = <1>;
> - interrupt-names = "dma", "msi";
> - interrupts = <0 224 4>, <0 225 4>;
> - interrupt-map-mask = <0 0 0 7>;
> - interrupt-map = <0 0 0 1 &pcie_intc 0>, /* INTA */
> - <0 0 0 2 &pcie_intc 1>, /* INTB */
> - <0 0 0 3 &pcie_intc 2>, /* INTC */
> - <0 0 0 4 &pcie_intc 3>; /* INTD */
> -
> - pcie_intc: legacy-interrupt-controller {
> - interrupt-controller;
> - #interrupt-cells = <1>;
> - interrupt-parent = <&gic>;
> - interrupts = <0 226 4>;
> - };
> - };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4cc47b2dbdc9..c1d377be991c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15337,7 +15337,7 @@ PCIE DRIVER FOR SOCIONEXT UNIPHIER
> M: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> L: linux-pci@vger.kernel.org
> S: Maintained
> -F: Documentation/devicetree/bindings/pci/uniphier-pcie*
> +F: Documentation/devicetree/bindings/pci/socionext,uniphier-pcie*
> F: drivers/pci/controller/dwc/pcie-uniphier*
>
> PCIE DRIVER FOR ST SPEAR13XX
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema
2022-03-28 12:51 ` Rob Herring
@ 2022-03-28 16:23 ` Rob Herring
2022-03-29 10:55 ` Kunihiko Hayashi
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2022-03-28 16:23 UTC (permalink / raw)
To: Kunihiko Hayashi
Cc: linux-kernel, devicetree, linux-pci, linux-arm-kernel,
Krzysztof Kozlowski, Bjorn Helgaas, Masami Hiramatsu
On Mon, Mar 28, 2022 at 07:51:05AM -0500, Rob Herring wrote:
> On Mon, 28 Mar 2022 11:11:38 +0900, Kunihiko Hayashi wrote:
> > Convert the file into a JSON description at the yaml format.
> >
> > Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> > ---
> > .../bindings/pci/socionext,uniphier-pcie.yaml | 100 ++++++++++++++++++
> > .../devicetree/bindings/pci/uniphier-pcie.txt | 82 --------------
> > MAINTAINERS | 2 +-
> > 3 files changed, 101 insertions(+), 83 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
> > delete mode 100644 Documentation/devicetree/bindings/pci/uniphier-pcie.txt
> >
>
> Running 'make dtbs_check' with the schema in this patch gives the
> following warnings. Consider if they are expected or the schema is
> incorrect. These may not be new warnings.
>
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
>
> Full log is available here: https://patchwork.ozlabs.org/patch/1609988
>
>
> pcie@66000000: compatible: ['socionext,uniphier-pcie', 'snps,dw-pcie'] is too long
> arch/arm64/boot/dts/socionext/uniphier-ld20-akebi96.dt.yaml
> arch/arm64/boot/dts/socionext/uniphier-ld20-global.dt.yaml
> arch/arm64/boot/dts/socionext/uniphier-ld20-ref.dt.yaml
> arch/arm64/boot/dts/socionext/uniphier-pxs3-ref.dt.yaml
Ignore this if your intent is to fix these by dropping 'snps,dw-pcie'. I
think that is the right thing to do. 'snps,dw-pcie' is not too
meaningful.
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema
2022-03-28 16:22 ` Rob Herring
@ 2022-03-29 10:54 ` Kunihiko Hayashi
0 siblings, 0 replies; 6+ messages in thread
From: Kunihiko Hayashi @ 2022-03-29 10:54 UTC (permalink / raw)
To: Rob Herring
Cc: Bjorn Helgaas, Krzysztof Kozlowski, Masami Hiramatsu, linux-pci,
devicetree, linux-arm-kernel, linux-kernel
Hi Rob,
Thank you for reviewing.
On 2022/03/29 1:22, Rob Herring wrote:
> On Mon, Mar 28, 2022 at 11:11:38AM +0900, Kunihiko Hayashi wrote:
>> Convert the file into a JSON description at the yaml format.
>>
>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
>> ---
>> .../bindings/pci/socionext,uniphier-pcie.yaml | 100 ++++++++++++++++++
>> .../devicetree/bindings/pci/uniphier-pcie.txt | 82 --------------
>> MAINTAINERS | 2 +-
>> 3 files changed, 101 insertions(+), 83 deletions(-)
>> create mode 100644
> Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
>> delete mode 100644 Documentation/devicetree/bindings/pci/uniphier-pcie.
> txt
[snip]
>> +properties:
>> + compatible:
>> + enum:
>> + - socionext,uniphier-pcie
>> +
>> + reg:
>> + minItems: 3
>> + maxItems: 4
>> +
>> + reg-names:
>> + oneOf:
>> + - items:
>> + - const: dbi
>> + - const: link
>> + - const: config
>> + - items:
>> + - const: dbi
>> + - const: link
>> + - const: config
>> + - const: atu
>
> You can have just the 2nd list plus 'minItems: 3' to do the same thing.
I see. I'll rewrite it that way.
Thank you,
---
Best Regards
Kunihiko Hayashi
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema
2022-03-28 16:23 ` Rob Herring
@ 2022-03-29 10:55 ` Kunihiko Hayashi
0 siblings, 0 replies; 6+ messages in thread
From: Kunihiko Hayashi @ 2022-03-29 10:55 UTC (permalink / raw)
To: Rob Herring
Cc: Bjorn Helgaas, Krzysztof Kozlowski, Masami Hiramatsu, linux-pci,
devicetree, linux-arm-kernel, linux-kernel
Hi Rob,
On 2022/03/29 1:23, Rob Herring wrote:
> On Mon, Mar 28, 2022 at 07:51:05AM -0500, Rob Herring wrote:
>> On Mon, 28 Mar 2022 11:11:38 +0900, Kunihiko Hayashi wrote:
>>> Convert the file into a JSON description at the yaml format.
>>>
>>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
>>> ---
>>> .../bindings/pci/socionext,uniphier-pcie.yaml | 100
> ++++++++++++++++++
>>> .../devicetree/bindings/pci/uniphier-pcie.txt | 82 --------------
>>> MAINTAINERS | 2 +-
>>> 3 files changed, 101 insertions(+), 83 deletions(-)
>>> create mode 100644
> Documentation/devicetree/bindings/pci/socionext,uniphier-pcie.yaml
>>> delete mode 100644
> Documentation/devicetree/bindings/pci/uniphier-pcie.txt
>>>
>>
>> Running 'make dtbs_check' with the schema in this patch gives the
>> following warnings. Consider if they are expected or the schema is
>> incorrect. These may not be new warnings.
>>
>> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
>> This will change in the future.
>>
>> Full log is available here: https://patchwork.ozlabs.org/patch/1609988
>>
>>
>> pcie@66000000: compatible: ['socionext,uniphier-pcie', 'snps,dw-pcie']
> is too long
>> arch/arm64/boot/dts/socionext/uniphier-ld20-akebi96.dt.yaml
>> arch/arm64/boot/dts/socionext/uniphier-ld20-global.dt.yaml
>> arch/arm64/boot/dts/socionext/uniphier-ld20-ref.dt.yaml
>> arch/arm64/boot/dts/socionext/uniphier-pxs3-ref.dt.yaml
>
> Ignore this if your intent is to fix these by dropping 'snps,dw-pcie'. I
> think that is the right thing to do. 'snps,dw-pcie' is not too
> meaningful.
I see. I should remove "snps,dw-pcie" from the existing devicetree, so
I'll fix it as a devicetree patch.
Thank you,
---
Best Regards
Kunihiko Hayashi
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-03-29 10:55 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-28 2:11 [PATCH] dt-bindings: PCI: uniphier: Convert uniphier-pcie.txt to json-schema Kunihiko Hayashi
2022-03-28 12:51 ` Rob Herring
2022-03-28 16:23 ` Rob Herring
2022-03-29 10:55 ` Kunihiko Hayashi
2022-03-28 16:22 ` Rob Herring
2022-03-29 10:54 ` Kunihiko Hayashi
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).