On Fri, Sep 27, 2019 at 9:59 AM Krzysztof Kozlowski krzk@kernel.org wrote:
On Fri, 27 Sep 2019 at 16:33, Marek Szyprowski m.szyprowski@samsung.com wrote:
From: Maciej Falkowski m.falkowski@samsung.com
Convert Samsung Image Scaler to newer dt-schema format.
Signed-off-by: Maciej Falkowski m.falkowski@samsung.com Signed-off-by: Marek Szyprowski m.szyprowski@samsung.com
v3:
- Fixed description of 'clocks' property:
rather than 'mscl clock', 'pclk clock'
- Added empty line within if-else statement
- Added 'additionalProperties: false'
- Listed all missing 'properties' in properties scope
Best regards, Maciej Falkowski
.../bindings/gpu/samsung-scaler.txt | 27 ------- .../bindings/gpu/samsung-scaler.yaml | 81 +++++++++++++++++++ 2 files changed, 81 insertions(+), 27 deletions(-) delete mode 100644 Documentation/devicetree/bindings/gpu/samsung-scaler.txt create mode 100644 Documentation/devicetree/bindings/gpu/samsung-scaler.yaml
diff --git a/Documentation/devicetree/bindings/gpu/samsung-scaler.txt b/Documentation/devicetree/bindings/gpu/samsung-scaler.txt deleted file mode 100644 index 9c3d98105dfd..000000000000 --- a/Documentation/devicetree/bindings/gpu/samsung-scaler.txt +++ /dev/null @@ -1,27 +0,0 @@ -* Samsung Exynos Image Scaler
-Required properties:
- compatible : value should be one of the following:
(a) "samsung,exynos5420-scaler" for Scaler IP in Exynos5420(b) "samsung,exynos5433-scaler" for Scaler IP in Exynos5433
- reg : Physical base address of the IP registers and length of memory
mapped region.
- interrupts : Interrupt specifier for scaler interrupt, according to format
specific to interrupt parent.
- clocks : Clock specifier for scaler clock, according to generic clock
bindings. (See Documentation/devicetree/bindings/clock/exynos*.txt)
- clock-names : Names of clocks. For exynos scaler, it should be "mscl"
on 5420 and "pclk", "aclk" and "aclk_xiu" on 5433.-Example:
scaler@12800000 {compatible = "samsung,exynos5420-scaler";reg = <0x12800000 0x1294>;interrupts = <0 220 IRQ_TYPE_LEVEL_HIGH>;clocks = <&clock CLK_MSCL0>;clock-names = "mscl";};diff --git a/Documentation/devicetree/bindings/gpu/samsung-scaler.yaml b/Documentation/devicetree/bindings/gpu/samsung-scaler.yaml new file mode 100644 index 000000000000..5317ac64426a --- /dev/null +++ b/Documentation/devicetree/bindings/gpu/samsung-scaler.yaml @@ -0,0 +1,81 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/gpu/samsung-scaler.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#
+title: Samsung Exynos SoC Image Scaler
+maintainers:
- Inki Dae inki.dae@samsung.com
+properties:
- compatible:
- enum:
- samsung,exynos5420-scaler- samsung,exynos5433-scaler- reg:
- maxItems: 1
- interrupts:
- maxItems: 1
- clocks: {}
- clock-names: {}
- iommus: {}
- power-domains: {}
+if:
- properties:
- compatible:
contains:const: samsung,exynos5420-scaler+then:
- properties:
- clocks:
items:- description: mscl clock- clock-names:
items:- const: mscl+else:
- properties:
- clocks:
items:- description: pclk clock- description: aclk clock- description: aclk_xiu clock- clock-names:
items:- const: pclk- const: aclk- const: aclk_xiu+required:
- compatible
- reg
- interrupts
- clocks
- clock-names
+additionalProperties: false
+examples:
- |
- #include <dt-bindings/clock/exynos5420.h>
- #include <dt-bindings/interrupt-controller/arm-gic.h>
- scaler@12800000 {
compatible = "samsung,exynos5420-scaler";reg = <0x12800000 0x1294>;interrupts = <GIC_SPI 220 IRQ_TYPE_LEVEL_HIGH>;clocks = <&clock CLK_MSCL0>;clock-names = "mscl";- };
+...
You have some left-overs at the end. With cleaning them: Reviewed-by: Krzysztof Kozlowski krzk@kernel.org
You mean the '...'? That's the end of YAML marker. It's not strictly needed.
Rob