* [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU @ 2019-10-04 15:14 Krzysztof Kozlowski 2019-10-04 15:14 ` [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation Krzysztof Kozlowski ` (2 more replies) 0 siblings, 3 replies; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-04 15:14 UTC (permalink / raw) To: Kukjin Kim, Krzysztof Kozlowski, Rob Herring, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, linux-arm-kernel, linux-samsung-soc, devicetree, linux-kernel, dri-devel, linux-serial The clkoutN names of clocks must be unique because they represent unique inputs of clock multiplexer. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml index 73b56fc5bf58..d8e03716f5d2 100644 --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml @@ -53,8 +53,10 @@ properties: List of clock names for particular CLKOUT mux inputs minItems: 1 maxItems: 32 - items: - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' + allOf: + - items: + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' + - uniqueItems: true clocks: minItems: 1 -- 2.17.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation 2019-10-04 15:14 [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Krzysztof Kozlowski @ 2019-10-04 15:14 ` Krzysztof Kozlowski 2019-10-15 19:26 ` Rob Herring 2019-10-04 15:14 ` [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema Krzysztof Kozlowski 2019-10-08 12:38 ` [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Rob Herring 2 siblings, 1 reply; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-04 15:14 UTC (permalink / raw) To: Kukjin Kim, Krzysztof Kozlowski, Rob Herring, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, linux-arm-kernel, linux-samsung-soc, devicetree, linux-kernel, dri-devel, linux-serial Array elements under 'items' should be indented. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- Documentation/devicetree/bindings/gpu/samsung-rotator.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/gpu/samsung-rotator.yaml b/Documentation/devicetree/bindings/gpu/samsung-rotator.yaml index 45ce562435fa..f4dfa6fc724c 100644 --- a/Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +++ b/Documentation/devicetree/bindings/gpu/samsung-rotator.yaml @@ -27,7 +27,7 @@ properties: clock-names: items: - - const: rotator + - const: rotator required: - compatible -- 2.17.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation 2019-10-04 15:14 ` [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation Krzysztof Kozlowski @ 2019-10-15 19:26 ` Rob Herring 0 siblings, 0 replies; 14+ messages in thread From: Rob Herring @ 2019-10-15 19:26 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Kukjin Kim, Krzysztof Kozlowski, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, linux-arm-kernel, linux-samsung-soc, devicetree, linux-kernel, dri-devel, linux-serial On Fri, 4 Oct 2019 17:14:13 +0200, Krzysztof Kozlowski wrote: > Array elements under 'items' should be indented. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > --- > Documentation/devicetree/bindings/gpu/samsung-rotator.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied, thanks. Rob ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema 2019-10-04 15:14 [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Krzysztof Kozlowski 2019-10-04 15:14 ` [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation Krzysztof Kozlowski @ 2019-10-04 15:14 ` Krzysztof Kozlowski 2019-10-15 19:40 ` Rob Herring 2019-10-08 12:38 ` [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Rob Herring 2 siblings, 1 reply; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-04 15:14 UTC (permalink / raw) To: Kukjin Kim, Krzysztof Kozlowski, Rob Herring, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, linux-arm-kernel, linux-samsung-soc, devicetree, linux-kernel, dri-devel, linux-serial Convert Samsung S3C/S5P/Exynos Serial/UART bindings to DT schema format using json-schema. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- .../bindings/mfd/samsung,exynos5433-lpass.txt | 2 +- .../bindings/serial/samsung_uart.txt | 58 ------- .../bindings/serial/samsung_uart.yaml | 148 ++++++++++++++++++ 3 files changed, 149 insertions(+), 59 deletions(-) delete mode 100644 Documentation/devicetree/bindings/serial/samsung_uart.txt create mode 100644 Documentation/devicetree/bindings/serial/samsung_uart.yaml diff --git a/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt b/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt index d759da606f75..30ea27c3936d 100644 --- a/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt +++ b/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt @@ -18,7 +18,7 @@ an optional sub-node. For "samsung,exynos5433-lpass" compatible this includes: UART, SLIMBUS, PCM, I2S, DMAC, Timers 0...4, VIC, WDT 0...1 devices. Bindings of the sub-nodes are described in: - ../serial/samsung_uart.txt + ../serial/samsung_uart.yaml ../sound/samsung-i2s.txt ../dma/arm-pl330.txt diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.txt b/Documentation/devicetree/bindings/serial/samsung_uart.txt deleted file mode 100644 index e85f37ec33f0..000000000000 --- a/Documentation/devicetree/bindings/serial/samsung_uart.txt +++ /dev/null @@ -1,58 +0,0 @@ -* Samsung's UART Controller - -The Samsung's UART controller is used for interfacing SoC with serial -communicaion devices. - -Required properties: -- compatible: should be one of following: - - "samsung,exynos4210-uart" - Exynos4210 SoC, - - "samsung,s3c2410-uart" - compatible with ports present on S3C2410 SoC, - - "samsung,s3c2412-uart" - compatible with ports present on S3C2412 SoC, - - "samsung,s3c2440-uart" - compatible with ports present on S3C2440 SoC, - - "samsung,s3c6400-uart" - compatible with ports present on S3C6400 SoC, - - "samsung,s5pv210-uart" - compatible with ports present on S5PV210 SoC. - -- reg: base physical address of the controller and length of memory mapped - region. - -- interrupts: a single interrupt signal to SoC interrupt controller, - according to interrupt bindings documentation [1]. - -- clock-names: input names of clocks used by the controller: - - "uart" - controller bus clock, - - "clk_uart_baudN" - Nth baud base clock input (N = 0, 1, ...), - according to SoC User's Manual (only N = 0 is allowedfor SoCs without - internal baud clock mux). -- clocks: phandles and specifiers for all clocks specified in "clock-names" - property, in the same order, according to clock bindings documentation [2]. - -[1] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt -[2] Documentation/devicetree/bindings/clock/clock-bindings.txt - -Optional properties: -- samsung,uart-fifosize: The fifo size supported by the UART channel - -Note: Each Samsung UART should have an alias correctly numbered in the -"aliases" node, according to serialN format, where N is the port number -(non-negative decimal integer) as specified by User's Manual of respective -SoC. - -Example: - aliases { - serial0 = &uart0; - serial1 = &uart1; - serial2 = &uart2; - }; - -Example: - uart1: serial@7f005400 { - compatible = "samsung,s3c6400-uart"; - reg = <0x7f005400 0x100>; - interrupt-parent = <&vic1>; - interrupts = <6>; - clock-names = "uart", "clk_uart_baud2", - "clk_uart_baud3"; - clocks = <&clocks PCLK_UART1>, <&clocks PCLK_UART1>, - <&clocks SCLK_UART>; - samsung,uart-fifosize = <16>; - }; diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.yaml b/Documentation/devicetree/bindings/serial/samsung_uart.yaml new file mode 100644 index 000000000000..276bea1c231a --- /dev/null +++ b/Documentation/devicetree/bindings/serial/samsung_uart.yaml @@ -0,0 +1,148 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/serial/samsung_uart.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Samsung S3C, S5P and Exynos SoC UART Controller + +maintainers: + - Krzysztof Kozlowski <krzk@kernel.org> + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +description: |+ + Each Samsung UART should have an alias correctly numbered in the "aliases" + node, according to serialN format, where N is the port number (non-negative + decimal integer) as specified by User's Manual of respective SoC. + +properties: + compatible: + items: + - enum: + - samsung,s3c2410-uart + - samsung,s3c2412-uart + - samsung,s3c2440-uart + - samsung,s3c6400-uart + - samsung,s5pv210-uart + - samsung,exynos4210-uart + + reg: + maxItems: 1 + + clocks: + minItems: 2 + maxItems: 5 + + clock-names: + description: | + List of clock names: + - "uart" - controller bus clock, + - "clk_uart_baudN" - Nth baud base clock input (N = 0, 1, ...). + N = 0 is allowed for SoCs without internal baud clock mux. + minItems: 2 + maxItems: 5 + allOf: + - uniqueItems: true + - oneOf: + - items: + - const: uart + - pattern: '^clk_uart_baud[0-3]$' + - items: + - const: uart + - pattern: '^clk_uart_baud[0-3]$' + - pattern: '^clk_uart_baud[0-3]$' + - items: + - const: uart + - pattern: '^clk_uart_baud[0-3]$' + - pattern: '^clk_uart_baud[0-3]$' + - pattern: '^clk_uart_baud[0-3]$' + - items: + - const: uart + - pattern: '^clk_uart_baud[0-3]$' + - pattern: '^clk_uart_baud[0-3]$' + - pattern: '^clk_uart_baud[0-3]$' + - pattern: '^clk_uart_baud[0-3]$' + + interrupts: + minItems: 1 + maxItems: 2 + + samsung,uart-fifosize: + description: The fifo size supported by the UART channel + allOf: + - $ref: /schemas/types.yaml#/definitions/uint32 + - enum: [16, 64, 256] + +required: + - compatible + - clocks + - clock-names + - interrupts + - reg + +allOf: + - if: + properties: + compatible: + contains: + enum: + - samsung,s3c2410-uart + - samsung,s5pv210-uart + then: + properties: + clocks: + minItems: 2 + maxItems: 3 + clock-names: + minItems: 2 + maxItems: 3 + allOf: + - uniqueItems: true + - oneOf: + - items: + - const: uart + - pattern: '^clk_uart_baud[0-1]$' + - items: + - const: uart + - pattern: '^clk_uart_baud[0-1]$' + - pattern: '^clk_uart_baud[0-1]$' + + - if: + properties: + compatible: + contains: + enum: + - samsung,exynos4210-uart + then: + properties: + clocks: + minItems: 2 + maxItems: 2 + clock-names: + minItems: 2 + maxItems: 2 + allOf: + - uniqueItems: true + - items: + - const: uart + - const: clk_uart_baud0 + +examples: + - | + #include <dt-bindings/clock/samsung,s3c64xx-clock.h> + + aliases { + serial0 = &uart0; + }; + + uart0: serial@7f005000 { + compatible = "samsung,s3c6400-uart"; + reg = <0x7f005000 0x100>; + interrupt-parent = <&vic1>; + interrupts = <5>; + clock-names = "uart", "clk_uart_baud2", + "clk_uart_baud3"; + clocks = <&clocks PCLK_UART0>, <&clocks PCLK_UART0>, + <&clocks SCLK_UART>; + samsung,uart-fifosize = <16>; + }; -- 2.17.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema 2019-10-04 15:14 ` [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema Krzysztof Kozlowski @ 2019-10-15 19:40 ` Rob Herring 2019-11-03 15:56 ` Krzysztof Kozlowski 0 siblings, 1 reply; 14+ messages in thread From: Rob Herring @ 2019-10-15 19:40 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, linux-arm-kernel, linux-samsung-soc, devicetree, linux-kernel, dri-devel, linux-serial On Fri, Oct 04, 2019 at 05:14:14PM +0200, Krzysztof Kozlowski wrote: > Convert Samsung S3C/S5P/Exynos Serial/UART bindings to DT schema format > using json-schema. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > --- > .../bindings/mfd/samsung,exynos5433-lpass.txt | 2 +- > .../bindings/serial/samsung_uart.txt | 58 ------- > .../bindings/serial/samsung_uart.yaml | 148 ++++++++++++++++++ > 3 files changed, 149 insertions(+), 59 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/serial/samsung_uart.txt > create mode 100644 Documentation/devicetree/bindings/serial/samsung_uart.yaml > > diff --git a/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt b/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt > index d759da606f75..30ea27c3936d 100644 > --- a/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt > +++ b/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt > @@ -18,7 +18,7 @@ an optional sub-node. For "samsung,exynos5433-lpass" compatible this includes: > UART, SLIMBUS, PCM, I2S, DMAC, Timers 0...4, VIC, WDT 0...1 devices. > > Bindings of the sub-nodes are described in: > - ../serial/samsung_uart.txt > + ../serial/samsung_uart.yaml > ../sound/samsung-i2s.txt > ../dma/arm-pl330.txt > > diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.txt b/Documentation/devicetree/bindings/serial/samsung_uart.txt > deleted file mode 100644 > index e85f37ec33f0..000000000000 > --- a/Documentation/devicetree/bindings/serial/samsung_uart.txt > +++ /dev/null > @@ -1,58 +0,0 @@ > -* Samsung's UART Controller > - > -The Samsung's UART controller is used for interfacing SoC with serial > -communicaion devices. > - > -Required properties: > -- compatible: should be one of following: > - - "samsung,exynos4210-uart" - Exynos4210 SoC, > - - "samsung,s3c2410-uart" - compatible with ports present on S3C2410 SoC, > - - "samsung,s3c2412-uart" - compatible with ports present on S3C2412 SoC, > - - "samsung,s3c2440-uart" - compatible with ports present on S3C2440 SoC, > - - "samsung,s3c6400-uart" - compatible with ports present on S3C6400 SoC, > - - "samsung,s5pv210-uart" - compatible with ports present on S5PV210 SoC. > - > -- reg: base physical address of the controller and length of memory mapped > - region. > - > -- interrupts: a single interrupt signal to SoC interrupt controller, > - according to interrupt bindings documentation [1]. > - > -- clock-names: input names of clocks used by the controller: > - - "uart" - controller bus clock, > - - "clk_uart_baudN" - Nth baud base clock input (N = 0, 1, ...), > - according to SoC User's Manual (only N = 0 is allowedfor SoCs without > - internal baud clock mux). > -- clocks: phandles and specifiers for all clocks specified in "clock-names" > - property, in the same order, according to clock bindings documentation [2]. > - > -[1] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt > -[2] Documentation/devicetree/bindings/clock/clock-bindings.txt > - > -Optional properties: > -- samsung,uart-fifosize: The fifo size supported by the UART channel > - > -Note: Each Samsung UART should have an alias correctly numbered in the > -"aliases" node, according to serialN format, where N is the port number > -(non-negative decimal integer) as specified by User's Manual of respective > -SoC. > - > -Example: > - aliases { > - serial0 = &uart0; > - serial1 = &uart1; > - serial2 = &uart2; > - }; > - > -Example: > - uart1: serial@7f005400 { > - compatible = "samsung,s3c6400-uart"; > - reg = <0x7f005400 0x100>; > - interrupt-parent = <&vic1>; > - interrupts = <6>; > - clock-names = "uart", "clk_uart_baud2", > - "clk_uart_baud3"; > - clocks = <&clocks PCLK_UART1>, <&clocks PCLK_UART1>, > - <&clocks SCLK_UART>; > - samsung,uart-fifosize = <16>; > - }; > diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.yaml b/Documentation/devicetree/bindings/serial/samsung_uart.yaml > new file mode 100644 > index 000000000000..276bea1c231a > --- /dev/null > +++ b/Documentation/devicetree/bindings/serial/samsung_uart.yaml > @@ -0,0 +1,148 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/serial/samsung_uart.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Samsung S3C, S5P and Exynos SoC UART Controller > + > +maintainers: > + - Krzysztof Kozlowski <krzk@kernel.org> > + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> > + > +description: |+ > + Each Samsung UART should have an alias correctly numbered in the "aliases" > + node, according to serialN format, where N is the port number (non-negative > + decimal integer) as specified by User's Manual of respective SoC. > + > +properties: > + compatible: > + items: > + - enum: > + - samsung,s3c2410-uart > + - samsung,s3c2412-uart > + - samsung,s3c2440-uart > + - samsung,s3c6400-uart > + - samsung,s5pv210-uart > + - samsung,exynos4210-uart > + > + reg: > + maxItems: 1 > + > + clocks: > + minItems: 2 > + maxItems: 5 > + > + clock-names: > + description: | > + List of clock names: > + - "uart" - controller bus clock, > + - "clk_uart_baudN" - Nth baud base clock input (N = 0, 1, ...). Kind of redundant to list this here. > + N = 0 is allowed for SoCs without internal baud clock mux. You can add a description with the item: items: - pattern: '^clk_uart_baud[0-3]$' description: N = 0 is allowed for SoCs without internal baud clock mux. Or leave it here is fine too. > + minItems: 2 > + maxItems: 5 > + allOf: > + - uniqueItems: true You can drop this. > + - oneOf: > + - items: > + - const: uart > + - pattern: '^clk_uart_baud[0-3]$' > + - items: > + - const: uart > + - pattern: '^clk_uart_baud[0-3]$' > + - pattern: '^clk_uart_baud[0-3]$' > + - items: > + - const: uart > + - pattern: '^clk_uart_baud[0-3]$' > + - pattern: '^clk_uart_baud[0-3]$' > + - pattern: '^clk_uart_baud[0-3]$' > + - items: > + - const: uart > + - pattern: '^clk_uart_baud[0-3]$' > + - pattern: '^clk_uart_baud[0-3]$' > + - pattern: '^clk_uart_baud[0-3]$' > + - pattern: '^clk_uart_baud[0-3]$' You only need the last case. With 'minItems: 2', the additional items here will just be ignored. > + > + interrupts: > + minItems: 1 > + maxItems: 2 What is each item? > + > + samsung,uart-fifosize: > + description: The fifo size supported by the UART channel > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - enum: [16, 64, 256] > + > +required: > + - compatible > + - clocks > + - clock-names > + - interrupts > + - reg > + > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - samsung,s3c2410-uart > + - samsung,s5pv210-uart > + then: > + properties: > + clocks: > + minItems: 2 > + maxItems: 3 > + clock-names: > + minItems: 2 > + maxItems: 3 > + allOf: > + - uniqueItems: true > + - oneOf: > + - items: > + - const: uart > + - pattern: '^clk_uart_baud[0-1]$' > + - items: > + - const: uart > + - pattern: '^clk_uart_baud[0-1]$' > + - pattern: '^clk_uart_baud[0-1]$' Similar simplification can be done here. > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - samsung,exynos4210-uart > + then: > + properties: > + clocks: > + minItems: 2 > + maxItems: 2 > + clock-names: > + minItems: 2 > + maxItems: 2 > + allOf: > + - uniqueItems: true > + - items: > + - const: uart > + - const: clk_uart_baud0 > + > +examples: > + - | > + #include <dt-bindings/clock/samsung,s3c64xx-clock.h> > + > + aliases { > + serial0 = &uart0; > + }; Drop this. It will cause a warning if we enforce 'aliases' to be at the root node. > + > + uart0: serial@7f005000 { > + compatible = "samsung,s3c6400-uart"; > + reg = <0x7f005000 0x100>; > + interrupt-parent = <&vic1>; > + interrupts = <5>; > + clock-names = "uart", "clk_uart_baud2", > + "clk_uart_baud3"; > + clocks = <&clocks PCLK_UART0>, <&clocks PCLK_UART0>, > + <&clocks SCLK_UART>; > + samsung,uart-fifosize = <16>; > + }; > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema 2019-10-15 19:40 ` Rob Herring @ 2019-11-03 15:56 ` Krzysztof Kozlowski 0 siblings, 0 replies; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-11-03 15:56 UTC (permalink / raw) To: Rob Herring Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, linux-arm-kernel, linux-samsung-soc, devicetree, linux-kernel, dri-devel, linux-serial On Tue, Oct 15, 2019 at 02:40:06PM -0500, Rob Herring wrote: > On Fri, Oct 04, 2019 at 05:14:14PM +0200, Krzysztof Kozlowski wrote: > > Convert Samsung S3C/S5P/Exynos Serial/UART bindings to DT schema format > > using json-schema. > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > > --- > > .../bindings/mfd/samsung,exynos5433-lpass.txt | 2 +- > > .../bindings/serial/samsung_uart.txt | 58 ------- > > .../bindings/serial/samsung_uart.yaml | 148 ++++++++++++++++++ > > 3 files changed, 149 insertions(+), 59 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/serial/samsung_uart.txt > > create mode 100644 Documentation/devicetree/bindings/serial/samsung_uart.yaml > > > > diff --git a/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt b/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt > > index d759da606f75..30ea27c3936d 100644 > > --- a/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt > > +++ b/Documentation/devicetree/bindings/mfd/samsung,exynos5433-lpass.txt > > @@ -18,7 +18,7 @@ an optional sub-node. For "samsung,exynos5433-lpass" compatible this includes: > > UART, SLIMBUS, PCM, I2S, DMAC, Timers 0...4, VIC, WDT 0...1 devices. > > > > Bindings of the sub-nodes are described in: > > - ../serial/samsung_uart.txt > > + ../serial/samsung_uart.yaml > > ../sound/samsung-i2s.txt > > ../dma/arm-pl330.txt > > > > diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.txt b/Documentation/devicetree/bindings/serial/samsung_uart.txt > > deleted file mode 100644 > > index e85f37ec33f0..000000000000 > > --- a/Documentation/devicetree/bindings/serial/samsung_uart.txt > > +++ /dev/null > > @@ -1,58 +0,0 @@ > > -* Samsung's UART Controller > > - > > -The Samsung's UART controller is used for interfacing SoC with serial > > -communicaion devices. > > - > > -Required properties: > > -- compatible: should be one of following: > > - - "samsung,exynos4210-uart" - Exynos4210 SoC, > > - - "samsung,s3c2410-uart" - compatible with ports present on S3C2410 SoC, > > - - "samsung,s3c2412-uart" - compatible with ports present on S3C2412 SoC, > > - - "samsung,s3c2440-uart" - compatible with ports present on S3C2440 SoC, > > - - "samsung,s3c6400-uart" - compatible with ports present on S3C6400 SoC, > > - - "samsung,s5pv210-uart" - compatible with ports present on S5PV210 SoC. > > - > > -- reg: base physical address of the controller and length of memory mapped > > - region. > > - > > -- interrupts: a single interrupt signal to SoC interrupt controller, > > - according to interrupt bindings documentation [1]. > > - > > -- clock-names: input names of clocks used by the controller: > > - - "uart" - controller bus clock, > > - - "clk_uart_baudN" - Nth baud base clock input (N = 0, 1, ...), > > - according to SoC User's Manual (only N = 0 is allowedfor SoCs without > > - internal baud clock mux). > > -- clocks: phandles and specifiers for all clocks specified in "clock-names" > > - property, in the same order, according to clock bindings documentation [2]. > > - > > -[1] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt > > -[2] Documentation/devicetree/bindings/clock/clock-bindings.txt > > - > > -Optional properties: > > -- samsung,uart-fifosize: The fifo size supported by the UART channel > > - > > -Note: Each Samsung UART should have an alias correctly numbered in the > > -"aliases" node, according to serialN format, where N is the port number > > -(non-negative decimal integer) as specified by User's Manual of respective > > -SoC. > > - > > -Example: > > - aliases { > > - serial0 = &uart0; > > - serial1 = &uart1; > > - serial2 = &uart2; > > - }; > > - > > -Example: > > - uart1: serial@7f005400 { > > - compatible = "samsung,s3c6400-uart"; > > - reg = <0x7f005400 0x100>; > > - interrupt-parent = <&vic1>; > > - interrupts = <6>; > > - clock-names = "uart", "clk_uart_baud2", > > - "clk_uart_baud3"; > > - clocks = <&clocks PCLK_UART1>, <&clocks PCLK_UART1>, > > - <&clocks SCLK_UART>; > > - samsung,uart-fifosize = <16>; > > - }; > > diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.yaml b/Documentation/devicetree/bindings/serial/samsung_uart.yaml > > new file mode 100644 > > index 000000000000..276bea1c231a > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/serial/samsung_uart.yaml > > @@ -0,0 +1,148 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/serial/samsung_uart.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Samsung S3C, S5P and Exynos SoC UART Controller > > + > > +maintainers: > > + - Krzysztof Kozlowski <krzk@kernel.org> > > + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > + > > +description: |+ > > + Each Samsung UART should have an alias correctly numbered in the "aliases" > > + node, according to serialN format, where N is the port number (non-negative > > + decimal integer) as specified by User's Manual of respective SoC. > > + > > +properties: > > + compatible: > > + items: > > + - enum: > > + - samsung,s3c2410-uart > > + - samsung,s3c2412-uart > > + - samsung,s3c2440-uart > > + - samsung,s3c6400-uart > > + - samsung,s5pv210-uart > > + - samsung,exynos4210-uart > > + > > + reg: > > + maxItems: 1 > > + > > + clocks: > > + minItems: 2 > > + maxItems: 5 > > + > > + clock-names: > > + description: | > > + List of clock names: > > + - "uart" - controller bus clock, > > + - "clk_uart_baudN" - Nth baud base clock input (N = 0, 1, ...). > > Kind of redundant to list this here. > > > + N = 0 is allowed for SoCs without internal baud clock mux. > > You can add a description with the item: > > items: > - pattern: '^clk_uart_baud[0-3]$' > description: N = 0 is allowed for SoCs without internal baud clock mux. > This caused error: properties:clock-names:items:1: Additional properties are not allowed ('description' was unexpected) ../Documentation/devicetree/bindings/Makefile:12: recipe for target 'Documentation/devicetree/bindings/serial/samsung_uart.example.dts' failed I applied rest of comments. Thanks for review! Best regards, Krzysztof ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-04 15:14 [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Krzysztof Kozlowski 2019-10-04 15:14 ` [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation Krzysztof Kozlowski 2019-10-04 15:14 ` [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema Krzysztof Kozlowski @ 2019-10-08 12:38 ` Rob Herring 2019-10-08 12:50 ` Krzysztof Kozlowski 2 siblings, 1 reply; 14+ messages in thread From: Rob Herring @ 2019-10-08 12:38 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Fri, Oct 4, 2019 at 10:14 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > The clkoutN names of clocks must be unique because they represent > unique inputs of clock multiplexer. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > --- > Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > index 73b56fc5bf58..d8e03716f5d2 100644 > --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > @@ -53,8 +53,10 @@ properties: > List of clock names for particular CLKOUT mux inputs > minItems: 1 > maxItems: 32 > - items: > - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > + allOf: > + - items: > + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > + - uniqueItems: true You shouldn't need the 'allOf', just add uniqueItems at the same level as items. Rob ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 12:38 ` [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Rob Herring @ 2019-10-08 12:50 ` Krzysztof Kozlowski 2019-10-08 14:05 ` Rob Herring 0 siblings, 1 reply; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-08 12:50 UTC (permalink / raw) To: Rob Herring Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 08, 2019 at 07:38:14AM -0500, Rob Herring wrote: > On Fri, Oct 4, 2019 at 10:14 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > The clkoutN names of clocks must be unique because they represent > > unique inputs of clock multiplexer. > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > > --- > > Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > index 73b56fc5bf58..d8e03716f5d2 100644 > > --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > @@ -53,8 +53,10 @@ properties: > > List of clock names for particular CLKOUT mux inputs > > minItems: 1 > > maxItems: 32 > > - items: > > - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > + allOf: > > + - items: > > + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > + - uniqueItems: true > > You shouldn't need the 'allOf', just add uniqueItems at the same level as items. If you mean something like: 56 uniqueItems: true 57 items: 58 pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' Then the dt_binding_check fails: dev/linux/Documentation/devicetree/bindings/arm/samsung/pmu.yaml: properties:clock-names: 'uniqueItems' is not one of ['$ref', 'additionalItems', 'additionalProperties', 'allOf', 'anyOf', 'const', 'contains', 'default', 'dependencies', 'deprecated', 'description', 'else', 'enum', 'items', 'if', 'minItems', 'minimum', 'maxItems', 'maximum', 'not', 'oneOf', 'pattern', 'patternProperties', 'properties', 'required', 'then', 'type', 'typeSize', 'unevaluatedProperties'] Best regards, Krzysztof ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 12:50 ` Krzysztof Kozlowski @ 2019-10-08 14:05 ` Rob Herring 2019-10-08 14:17 ` Rob Herring 0 siblings, 1 reply; 14+ messages in thread From: Rob Herring @ 2019-10-08 14:05 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 8, 2019 at 7:50 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On Tue, Oct 08, 2019 at 07:38:14AM -0500, Rob Herring wrote: > > On Fri, Oct 4, 2019 at 10:14 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > > > The clkoutN names of clocks must be unique because they represent > > > unique inputs of clock multiplexer. > > > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > > > --- > > > Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > index 73b56fc5bf58..d8e03716f5d2 100644 > > > --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > @@ -53,8 +53,10 @@ properties: > > > List of clock names for particular CLKOUT mux inputs > > > minItems: 1 > > > maxItems: 32 > > > - items: > > > - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > + allOf: > > > + - items: > > > + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > + - uniqueItems: true > > > > You shouldn't need the 'allOf', just add uniqueItems at the same level as items. > > If you mean something like: > 56 uniqueItems: true > 57 items: > 58 pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > Then the dt_binding_check fails: > > dev/linux/Documentation/devicetree/bindings/arm/samsung/pmu.yaml: properties:clock-names: > 'uniqueItems' is not one of ['$ref', 'additionalItems', 'additionalProperties', 'allOf', 'anyOf', 'const', 'contains', 'default', 'dependencies', 'deprecated', 'description', 'else', 'enum', 'items', 'if', 'minItems', 'minimum', 'maxItems', 'maximum', 'not', 'oneOf', 'pattern', 'patternProperties', 'properties', 'required', 'then', 'type', 'typeSize', 'unevaluatedProperties'] I can add it. The other option is to fix this in the clock schema. I'm not sure if there's a need for duplicate clock-names. Seems unlikely. I'll test that. Rob ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 14:05 ` Rob Herring @ 2019-10-08 14:17 ` Rob Herring 2019-10-08 14:29 ` Krzysztof Kozlowski 0 siblings, 1 reply; 14+ messages in thread From: Rob Herring @ 2019-10-08 14:17 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 8, 2019 at 9:05 AM Rob Herring <robh+dt@kernel.org> wrote: > > On Tue, Oct 8, 2019 at 7:50 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > On Tue, Oct 08, 2019 at 07:38:14AM -0500, Rob Herring wrote: > > > On Fri, Oct 4, 2019 at 10:14 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > > > > > The clkoutN names of clocks must be unique because they represent > > > > unique inputs of clock multiplexer. > > > > > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > > > > --- > > > > Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > index 73b56fc5bf58..d8e03716f5d2 100644 > > > > --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > @@ -53,8 +53,10 @@ properties: > > > > List of clock names for particular CLKOUT mux inputs > > > > minItems: 1 > > > > maxItems: 32 > > > > - items: > > > > - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > + allOf: > > > > + - items: > > > > + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > + - uniqueItems: true > > > > > > You shouldn't need the 'allOf', just add uniqueItems at the same level as items. > > > > If you mean something like: > > 56 uniqueItems: true > > 57 items: > > 58 pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > Then the dt_binding_check fails: > > > > dev/linux/Documentation/devicetree/bindings/arm/samsung/pmu.yaml: properties:clock-names: > > 'uniqueItems' is not one of ['$ref', 'additionalItems', 'additionalProperties', 'allOf', 'anyOf', 'const', 'contains', 'default', 'dependencies', 'deprecated', 'description', 'else', 'enum', 'items', 'if', 'minItems', 'minimum', 'maxItems', 'maximum', 'not', 'oneOf', 'pattern', 'patternProperties', 'properties', 'required', 'then', 'type', 'typeSize', 'unevaluatedProperties'] > > I can add it. > > The other option is to fix this in the clock schema. I'm not sure if > there's a need for duplicate clock-names. Seems unlikely. I'll test > that. Actually, clock-names is already set to be unique. Did you see otherwise? Rob ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 14:17 ` Rob Herring @ 2019-10-08 14:29 ` Krzysztof Kozlowski 2019-10-08 14:38 ` Rob Herring 0 siblings, 1 reply; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-08 14:29 UTC (permalink / raw) To: Rob Herring Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 08, 2019 at 09:17:16AM -0500, Rob Herring wrote: > On Tue, Oct 8, 2019 at 9:05 AM Rob Herring <robh+dt@kernel.org> wrote: > > > > On Tue, Oct 8, 2019 at 7:50 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > > > On Tue, Oct 08, 2019 at 07:38:14AM -0500, Rob Herring wrote: > > > > On Fri, Oct 4, 2019 at 10:14 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > > > > > > > The clkoutN names of clocks must be unique because they represent > > > > > unique inputs of clock multiplexer. > > > > > > > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > > > > > --- > > > > > Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- > > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > > index 73b56fc5bf58..d8e03716f5d2 100644 > > > > > --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > > @@ -53,8 +53,10 @@ properties: > > > > > List of clock names for particular CLKOUT mux inputs > > > > > minItems: 1 > > > > > maxItems: 32 > > > > > - items: > > > > > - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > > + allOf: > > > > > + - items: > > > > > + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > > + - uniqueItems: true > > > > > > > > You shouldn't need the 'allOf', just add uniqueItems at the same level as items. > > > > > > If you mean something like: > > > 56 uniqueItems: true > > > 57 items: > > > 58 pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > > > Then the dt_binding_check fails: > > > > > > dev/linux/Documentation/devicetree/bindings/arm/samsung/pmu.yaml: properties:clock-names: > > > 'uniqueItems' is not one of ['$ref', 'additionalItems', 'additionalProperties', 'allOf', 'anyOf', 'const', 'contains', 'default', 'dependencies', 'deprecated', 'description', 'else', 'enum', 'items', 'if', 'minItems', 'minimum', 'maxItems', 'maximum', 'not', 'oneOf', 'pattern', 'patternProperties', 'properties', 'required', 'then', 'type', 'typeSize', 'unevaluatedProperties'] > > > > I can add it. > > > > The other option is to fix this in the clock schema. I'm not sure if > > there's a need for duplicate clock-names. Seems unlikely. I'll test > > that. > > Actually, clock-names is already set to be unique. Did you see otherwise? Yeah, I duplicated on purpose "clkout1" and it was not reported as an error. That's why I added "uniqueItems". Best regards, Krzysztof ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 14:29 ` Krzysztof Kozlowski @ 2019-10-08 14:38 ` Rob Herring 2019-10-08 15:04 ` Krzysztof Kozlowski 0 siblings, 1 reply; 14+ messages in thread From: Rob Herring @ 2019-10-08 14:38 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 8, 2019 at 9:29 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On Tue, Oct 08, 2019 at 09:17:16AM -0500, Rob Herring wrote: > > On Tue, Oct 8, 2019 at 9:05 AM Rob Herring <robh+dt@kernel.org> wrote: > > > > > > On Tue, Oct 8, 2019 at 7:50 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > > > > > On Tue, Oct 08, 2019 at 07:38:14AM -0500, Rob Herring wrote: > > > > > On Fri, Oct 4, 2019 at 10:14 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > > > > > > > > > The clkoutN names of clocks must be unique because they represent > > > > > > unique inputs of clock multiplexer. > > > > > > > > > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > > > > > > --- > > > > > > Documentation/devicetree/bindings/arm/samsung/pmu.yaml | 6 ++++-- > > > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > > > > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > > > index 73b56fc5bf58..d8e03716f5d2 100644 > > > > > > --- a/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > > > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.yaml > > > > > > @@ -53,8 +53,10 @@ properties: > > > > > > List of clock names for particular CLKOUT mux inputs > > > > > > minItems: 1 > > > > > > maxItems: 32 > > > > > > - items: > > > > > > - pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > > > + allOf: > > > > > > + - items: > > > > > > + pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > > > + - uniqueItems: true > > > > > > > > > > You shouldn't need the 'allOf', just add uniqueItems at the same level as items. > > > > > > > > If you mean something like: > > > > 56 uniqueItems: true > > > > 57 items: > > > > 58 pattern: '^clkout([0-9]|[12][0-9]|3[0-1])$' > > > > > > > > Then the dt_binding_check fails: > > > > > > > > dev/linux/Documentation/devicetree/bindings/arm/samsung/pmu.yaml: properties:clock-names: > > > > 'uniqueItems' is not one of ['$ref', 'additionalItems', 'additionalProperties', 'allOf', 'anyOf', 'const', 'contains', 'default', 'dependencies', 'deprecated', 'description', 'else', 'enum', 'items', 'if', 'minItems', 'minimum', 'maxItems', 'maximum', 'not', 'oneOf', 'pattern', 'patternProperties', 'properties', 'required', 'then', 'type', 'typeSize', 'unevaluatedProperties'] > > > > > > I can add it. > > > > > > The other option is to fix this in the clock schema. I'm not sure if > > > there's a need for duplicate clock-names. Seems unlikely. I'll test > > > that. > > > > Actually, clock-names is already set to be unique. Did you see otherwise? > > Yeah, I duplicated on purpose "clkout1" and it was not reported as an > error. That's why I added "uniqueItems". Are you running using DT_SCHEMA_FILES? If so, you won't get the core schema. Rob ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 14:38 ` Rob Herring @ 2019-10-08 15:04 ` Krzysztof Kozlowski 2019-10-08 15:10 ` Krzysztof Kozlowski 0 siblings, 1 reply; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-08 15:04 UTC (permalink / raw) To: Rob Herring Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 08, 2019 at 09:38:15AM -0500, Rob Herring wrote: > Are you running using DT_SCHEMA_FILES? If so, you won't get the core schema. Ah, yes, now I see proper errors. Thanks for pointing this. I'll send next version of this patch only (if others are ok). Best regards, Krzysztof ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU 2019-10-08 15:04 ` Krzysztof Kozlowski @ 2019-10-08 15:10 ` Krzysztof Kozlowski 0 siblings, 0 replies; 14+ messages in thread From: Krzysztof Kozlowski @ 2019-10-08 15:10 UTC (permalink / raw) To: Rob Herring Cc: Kukjin Kim, Mark Rutland, David Airlie, Daniel Vetter, Lee Jones, Greg Kroah-Hartman, Maciej Falkowski, Marek Szyprowski, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, linux-samsung-soc, devicetree, linux-kernel, dri-devel, open list:SERIAL DRIVERS On Tue, Oct 08, 2019 at 05:04:56PM +0200, Krzysztof Kozlowski wrote: > On Tue, Oct 08, 2019 at 09:38:15AM -0500, Rob Herring wrote: > > Are you running using DT_SCHEMA_FILES? If so, you won't get the core schema. > > Ah, yes, now I see proper errors. Thanks for pointing this. > > I'll send next version of this patch only (if others are ok). > Of course, there will be no v2, this patch can be just skipped. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-11-03 15:56 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-10-04 15:14 [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Krzysztof Kozlowski 2019-10-04 15:14 ` [PATCH 2/3] dt-bindings: gpu: samsung-rotator: Fix indentation Krzysztof Kozlowski 2019-10-15 19:26 ` Rob Herring 2019-10-04 15:14 ` [PATCH 3/3] dt-bindings: serial: Convert Samsung UART bindings to json-schema Krzysztof Kozlowski 2019-10-15 19:40 ` Rob Herring 2019-11-03 15:56 ` Krzysztof Kozlowski 2019-10-08 12:38 ` [PATCH 1/3] dt-bindings: arm: samsung: Force clkoutN names to be unique in PMU Rob Herring 2019-10-08 12:50 ` Krzysztof Kozlowski 2019-10-08 14:05 ` Rob Herring 2019-10-08 14:17 ` Rob Herring 2019-10-08 14:29 ` Krzysztof Kozlowski 2019-10-08 14:38 ` Rob Herring 2019-10-08 15:04 ` Krzysztof Kozlowski 2019-10-08 15:10 ` Krzysztof Kozlowski
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).