linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: display: Drop unneeded quotes
@ 2023-03-17 23:36 Rob Herring
  2023-03-18  8:29 ` Laurent Pinchart
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Rob Herring @ 2023-03-17 23:36 UTC (permalink / raw)
  To: Robin van der Gracht, Miguel Ojeda, Krzysztof Kozlowski,
	Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
	Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter,
	Chun-Kuang Hu, Philipp Zabel, Rob Clark, Abhinav Kumar,
	Dmitry Baryshkov, Sean Paul, Thierry Reding, Sam Ravnborg,
	Kieran Bingham, Jonathan Hunter, Jyri Sarha, Tomi Valkeinen,
	Matthias Brugger, AngeloGioacchino Del Regno
  Cc: devicetree, linux-kernel, dri-devel, linux-mediatek,
	linux-arm-msm, freedreno, linux-renesas-soc, linux-tegra,
	linux-arm-kernel

Cleanup bindings dropping unneeded quotes. Once all these are fixed,
checking for this can be enabled in yamllint.

Signed-off-by: Rob Herring <robh@kernel.org>
---
 .../bindings/auxdisplay/holtek,ht16k33.yaml    |  2 +-
 .../bindings/display/bridge/nxp,ptn3460.yaml   |  2 +-
 .../display/bridge/toshiba,tc358767.yaml       |  2 +-
 .../bindings/display/dp-aux-bus.yaml           |  2 +-
 .../display/mediatek/mediatek,hdmi.yaml        |  2 +-
 .../display/msm/dsi-controller-main.yaml       |  8 ++++----
 .../bindings/display/msm/dsi-phy-10nm.yaml     |  2 +-
 .../bindings/display/panel/ronbo,rb070d30.yaml |  2 +-
 .../bindings/display/renesas,du.yaml           |  4 ++--
 .../display/tegra/nvidia,tegra114-mipi.yaml    |  2 +-
 .../display/tegra/nvidia,tegra124-sor.yaml     | 12 ++++++------
 .../display/tegra/nvidia,tegra186-dc.yaml      |  4 ++--
 .../tegra/nvidia,tegra186-dsi-padctl.yaml      |  2 +-
 .../display/tegra/nvidia,tegra20-dsi.yaml      | 12 ++++++------
 .../display/tegra/nvidia,tegra20-hdmi.yaml     |  6 +++---
 .../bindings/display/ti/ti,am65x-dss.yaml      |  2 +-
 .../display/xylon,logicvc-display.yaml         | 18 +++++++++---------
 17 files changed, 42 insertions(+), 42 deletions(-)

diff --git a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
index fc4873deb76f..4f6ffb8182a9 100644
--- a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
+++ b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
@@ -10,7 +10,7 @@ maintainers:
   - Robin van der Gracht <robin@protonic.nl>
 
 allOf:
-  - $ref: "/schemas/input/matrix-keymap.yaml#"
+  - $ref: /schemas/input/matrix-keymap.yaml#
 
 properties:
   compatible:
diff --git a/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml b/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
index 107dd138e6c6..cdeb67bc05f0 100644
--- a/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
@@ -18,7 +18,7 @@ properties:
     maxItems: 1
 
   edid-emulation:
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
     description:
       The EDID emulation entry to use
       Value  Resolution  Description
diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
index 140927884418..e1494b5007cb 100644
--- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
@@ -23,7 +23,7 @@ properties:
         i2c address of the bridge, 0x68 or 0x0f, depending on bootstrap pins
 
   clock-names:
-    const: "ref"
+    const: ref
 
   clocks:
     maxItems: 1
diff --git a/Documentation/devicetree/bindings/display/dp-aux-bus.yaml b/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
index 5e4afe9f98fb..0ece7b01790b 100644
--- a/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
+++ b/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
@@ -26,7 +26,7 @@ description:
 
 properties:
   $nodename:
-    const: "aux-bus"
+    const: aux-bus
 
   panel:
     $ref: panel/panel-common.yaml#
diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
index 8afdd67d6780..b90b6d18a828 100644
--- a/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
+++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
@@ -50,7 +50,7 @@ properties:
       - const: hdmi
 
   mediatek,syscon-hdmi:
-    $ref: '/schemas/types.yaml#/definitions/phandle-array'
+    $ref: /schemas/types.yaml#/definitions/phandle-array
     items:
       - items:
           - description: phandle to system configuration registers
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index e75a3efe4dac..2188d7c9b0bb 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -74,7 +74,7 @@ properties:
 
   syscon-sfpb:
     description: A phandle to mmss_sfpb syscon node (only for DSIv2).
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   qcom,dual-dsi-mode:
     type: boolean
@@ -105,14 +105,14 @@ properties:
     type: object
 
   ports:
-    $ref: "/schemas/graph.yaml#/properties/ports"
+    $ref: /schemas/graph.yaml#/properties/ports
     description: |
       Contains DSI controller input and output ports as children, each
       containing one endpoint subnode.
 
     properties:
       port@0:
-        $ref: "/schemas/graph.yaml#/$defs/port-base"
+        $ref: /schemas/graph.yaml#/$defs/port-base
         unevaluatedProperties: false
         description: |
           Input endpoints of the controller.
@@ -128,7 +128,7 @@ properties:
                   enum: [ 0, 1, 2, 3 ]
 
       port@1:
-        $ref: "/schemas/graph.yaml#/$defs/port-base"
+        $ref: /schemas/graph.yaml#/$defs/port-base
         unevaluatedProperties: false
         description: |
           Output endpoints of the controller.
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
index 3ec466c3ab38..e6b00d7387ce 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
@@ -58,7 +58,7 @@ properties:
       maximum: 31
 
   qcom,phy-drive-ldo-level:
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
     description:
       The PHY LDO has an amplitude tuning feature to adjust the LDO output
       for the HSTX drive. Use supported levels (mV) to offset the drive level
diff --git a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
index d67617f6f74a..95ce22c6787a 100644
--- a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
@@ -37,7 +37,7 @@ properties:
 
   backlight:
     description: Backlight used by the panel
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/display/renesas,du.yaml b/Documentation/devicetree/bindings/display/renesas,du.yaml
index d4830f52c512..c5b9e6812bce 100644
--- a/Documentation/devicetree/bindings/display/renesas,du.yaml
+++ b/Documentation/devicetree/bindings/display/renesas,du.yaml
@@ -76,7 +76,7 @@ properties:
     unevaluatedProperties: false
 
   renesas,cmms:
-    $ref: "/schemas/types.yaml#/definitions/phandle-array"
+    $ref: /schemas/types.yaml#/definitions/phandle-array
     items:
       maxItems: 1
     description:
@@ -84,7 +84,7 @@ properties:
       available DU channel.
 
   renesas,vsps:
-    $ref: "/schemas/types.yaml#/definitions/phandle-array"
+    $ref: /schemas/types.yaml#/definitions/phandle-array
     items:
       items:
         - description: phandle to VSP instance that serves the DU channel
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
index d5ca8cf86e8e..f448624dd779 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
@@ -38,7 +38,7 @@ properties:
     description: The number of cells in a MIPI calibration specifier.
       Should be 1. The single cell specifies a bitmask of the pads that
       need to be calibrated for a given device.
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
     const: 1
 
 additionalProperties: false
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
index 907fb0baccae..70f0e45c71d6 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
@@ -69,12 +69,12 @@ properties:
   # Tegra186 and later
   nvidia,interface:
     description: index of the SOR interface
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
 
   nvidia,ddc-i2c-bus:
     description: phandle of an I2C controller used for DDC EDID
       probing
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   nvidia,hpd-gpio:
     description: specifies a GPIO used for hotplug detection
@@ -82,23 +82,23 @@ properties:
 
   nvidia,edid:
     description: supplies a binary EDID blob
-    $ref: "/schemas/types.yaml#/definitions/uint8-array"
+    $ref: /schemas/types.yaml#/definitions/uint8-array
 
   nvidia,panel:
     description: phandle of a display panel, required for eDP
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   nvidia,xbar-cfg:
     description: 5 cells containing the crossbar configuration.
       Each lane of the SOR, identified by the cell's index, is
       mapped via the crossbar to the pad specified by the cell's
       value.
-    $ref: "/schemas/types.yaml#/definitions/uint32-array"
+    $ref: /schemas/types.yaml#/definitions/uint32-array
 
   # optional when driving an eDP output
   nvidia,dpaux:
     description: phandle to a DispayPort AUX interface
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
 allOf:
   - if:
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
index 265a60d79d89..ce4589466a18 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
@@ -60,13 +60,13 @@ properties:
   nvidia,outputs:
     description: A list of phandles of outputs that this display
       controller can drive.
-    $ref: "/schemas/types.yaml#/definitions/phandle-array"
+    $ref: /schemas/types.yaml#/definitions/phandle-array
 
   nvidia,head:
     description: The number of the display controller head. This
       is used to setup the various types of output to receive
       video data from the given head.
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
 
 additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
index e5a6145c8c53..da75b71e8ece 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
@@ -29,7 +29,7 @@ properties:
       - const: dsi
 
 allOf:
-  - $ref: "/schemas/reset/reset.yaml"
+  - $ref: /schemas/reset/reset.yaml
 
 additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
index 511cbe74e729..59e1dc0813e7 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
@@ -59,12 +59,12 @@ properties:
     description: Should contain a phandle and a specifier specifying
       which pads are used by this DSI output and need to be
       calibrated. See nvidia,tegra114-mipi.yaml for details.
-    $ref: "/schemas/types.yaml#/definitions/phandle-array"
+    $ref: /schemas/types.yaml#/definitions/phandle-array
 
   nvidia,ddc-i2c-bus:
     description: phandle of an I2C controller used for DDC EDID
       probing
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   nvidia,hpd-gpio:
     description: specifies a GPIO used for hotplug detection
@@ -72,19 +72,19 @@ properties:
 
   nvidia,edid:
     description: supplies a binary EDID blob
-    $ref: "/schemas/types.yaml#/definitions/uint8-array"
+    $ref: /schemas/types.yaml#/definitions/uint8-array
 
   nvidia,panel:
     description: phandle of a display panel
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   nvidia,ganged-mode:
     description: contains a phandle to a second DSI controller to
       gang up with in order to support up to 8 data lanes
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
 allOf:
-  - $ref: "../dsi-controller.yaml#"
+  - $ref: ../dsi-controller.yaml#
   - if:
       properties:
         compatible:
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
index f65e59cfffa7..f77197e4869f 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
@@ -68,7 +68,7 @@ properties:
   nvidia,ddc-i2c-bus:
     description: phandle of an I2C controller used for DDC EDID
       probing
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   nvidia,hpd-gpio:
     description: specifies a GPIO used for hotplug detection
@@ -76,11 +76,11 @@ properties:
 
   nvidia,edid:
     description: supplies a binary EDID blob
-    $ref: "/schemas/types.yaml#/definitions/uint8-array"
+    $ref: /schemas/types.yaml#/definitions/uint8-array
 
   nvidia,panel:
     description: phandle of a display panel
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
 
   "#sound-dai-cells":
     const: 0
diff --git a/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml b/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
index 5c7d2cbc4aac..4247280d6c3c 100644
--- a/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
+++ b/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
@@ -88,7 +88,7 @@ properties:
           The DSS DPI output port node from video port 2
 
   ti,am65x-oldi-io-ctrl:
-    $ref: "/schemas/types.yaml#/definitions/phandle"
+    $ref: /schemas/types.yaml#/definitions/phandle
     description:
       phandle to syscon device node mapping OLDI IO_CTRL registers.
       The mapped range should point to OLDI_DAT0_IO_CTRL, map it and
diff --git a/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml b/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
index fc02c5d50ce4..87404d72ea37 100644
--- a/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
+++ b/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
@@ -89,25 +89,25 @@ properties:
     description: Display output colorspace (C_DISPLAY_COLOR_SPACE).
 
   xylon,display-depth:
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
     description: Display output depth (C_PIXEL_DATA_WIDTH).
 
   xylon,row-stride:
-    $ref: "/schemas/types.yaml#/definitions/uint32"
+    $ref: /schemas/types.yaml#/definitions/uint32
     description: Fixed number of pixels in a framebuffer row (C_ROW_STRIDE).
 
   xylon,dithering:
-    $ref: "/schemas/types.yaml#/definitions/flag"
+    $ref: /schemas/types.yaml#/definitions/flag
     description: Dithering module is enabled (C_XCOLOR)
 
   xylon,background-layer:
-    $ref: "/schemas/types.yaml#/definitions/flag"
+    $ref: /schemas/types.yaml#/definitions/flag
     description: |
       The last layer is used to display a black background (C_USE_BACKGROUND).
       The layer must still be registered.
 
   xylon,layers-configurable:
-    $ref: "/schemas/types.yaml#/definitions/flag"
+    $ref: /schemas/types.yaml#/definitions/flag
     description: |
       Configuration of layers' size, position and offset is enabled
       (C_USE_SIZE_POSITION).
@@ -131,7 +131,7 @@ properties:
             maxItems: 1
 
           xylon,layer-depth:
-            $ref: "/schemas/types.yaml#/definitions/uint32"
+            $ref: /schemas/types.yaml#/definitions/uint32
             description: Layer depth (C_LAYER_X_DATA_WIDTH).
 
           xylon,layer-colorspace:
@@ -151,19 +151,19 @@ properties:
             description: Alpha mode for the layer (C_LAYER_X_ALPHA_MODE).
 
           xylon,layer-base-offset:
-            $ref: "/schemas/types.yaml#/definitions/uint32"
+            $ref: /schemas/types.yaml#/definitions/uint32
             description: |
               Offset in number of lines (C_LAYER_X_OFFSET) starting from the
               video RAM base (C_VMEM_BASEADDR), only for version 3.
 
           xylon,layer-buffer-offset:
-            $ref: "/schemas/types.yaml#/definitions/uint32"
+            $ref: /schemas/types.yaml#/definitions/uint32
             description: |
               Offset in number of lines (C_BUFFER_*_OFFSET) starting from the
               layer base offset for the second buffer used in double-buffering.
 
           xylon,layer-primary:
-            $ref: "/schemas/types.yaml#/definitions/flag"
+            $ref: /schemas/types.yaml#/definitions/flag
             description: |
               Layer should be registered as a primary plane (exactly one is
               required).
-- 
2.39.2


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: display: Drop unneeded quotes
  2023-03-17 23:36 [PATCH] dt-bindings: display: Drop unneeded quotes Rob Herring
@ 2023-03-18  8:29 ` Laurent Pinchart
  2023-03-19 12:06 ` Krzysztof Kozlowski
  2023-03-20 15:43 ` Chun-Kuang Hu
  2 siblings, 0 replies; 4+ messages in thread
From: Laurent Pinchart @ 2023-03-18  8:29 UTC (permalink / raw)
  To: Rob Herring
  Cc: Robin van der Gracht, Miguel Ojeda, Krzysztof Kozlowski,
	Andrzej Hajda, Neil Armstrong, Robert Foss, Jonas Karlman,
	Jernej Skrabec, David Airlie, Daniel Vetter, Chun-Kuang Hu,
	Philipp Zabel, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
	Sean Paul, Thierry Reding, Sam Ravnborg, Kieran Bingham,
	Jonathan Hunter, Jyri Sarha, Tomi Valkeinen, Matthias Brugger,
	AngeloGioacchino Del Regno, devicetree, linux-kernel, dri-devel,
	linux-mediatek, linux-arm-msm, freedreno, linux-renesas-soc,
	linux-tegra, linux-arm-kernel

Hi Rob,

Thank you for the patch.

On Fri, Mar 17, 2023 at 06:36:24PM -0500, Rob Herring wrote:
> Cleanup bindings dropping unneeded quotes. Once all these are fixed,
> checking for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>

> ---
>  .../bindings/auxdisplay/holtek,ht16k33.yaml    |  2 +-
>  .../bindings/display/bridge/nxp,ptn3460.yaml   |  2 +-
>  .../display/bridge/toshiba,tc358767.yaml       |  2 +-
>  .../bindings/display/dp-aux-bus.yaml           |  2 +-
>  .../display/mediatek/mediatek,hdmi.yaml        |  2 +-
>  .../display/msm/dsi-controller-main.yaml       |  8 ++++----
>  .../bindings/display/msm/dsi-phy-10nm.yaml     |  2 +-
>  .../bindings/display/panel/ronbo,rb070d30.yaml |  2 +-
>  .../bindings/display/renesas,du.yaml           |  4 ++--
>  .../display/tegra/nvidia,tegra114-mipi.yaml    |  2 +-
>  .../display/tegra/nvidia,tegra124-sor.yaml     | 12 ++++++------
>  .../display/tegra/nvidia,tegra186-dc.yaml      |  4 ++--
>  .../tegra/nvidia,tegra186-dsi-padctl.yaml      |  2 +-
>  .../display/tegra/nvidia,tegra20-dsi.yaml      | 12 ++++++------
>  .../display/tegra/nvidia,tegra20-hdmi.yaml     |  6 +++---
>  .../bindings/display/ti/ti,am65x-dss.yaml      |  2 +-
>  .../display/xylon,logicvc-display.yaml         | 18 +++++++++---------
>  17 files changed, 42 insertions(+), 42 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
> index fc4873deb76f..4f6ffb8182a9 100644
> --- a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
> +++ b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
> @@ -10,7 +10,7 @@ maintainers:
>    - Robin van der Gracht <robin@protonic.nl>
>  
>  allOf:
> -  - $ref: "/schemas/input/matrix-keymap.yaml#"
> +  - $ref: /schemas/input/matrix-keymap.yaml#
>  
>  properties:
>    compatible:
> diff --git a/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml b/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
> index 107dd138e6c6..cdeb67bc05f0 100644
> --- a/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
> @@ -18,7 +18,7 @@ properties:
>      maxItems: 1
>  
>    edid-emulation:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description:
>        The EDID emulation entry to use
>        Value  Resolution  Description
> diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
> index 140927884418..e1494b5007cb 100644
> --- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
> @@ -23,7 +23,7 @@ properties:
>          i2c address of the bridge, 0x68 or 0x0f, depending on bootstrap pins
>  
>    clock-names:
> -    const: "ref"
> +    const: ref
>  
>    clocks:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/display/dp-aux-bus.yaml b/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
> index 5e4afe9f98fb..0ece7b01790b 100644
> --- a/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
> +++ b/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
> @@ -26,7 +26,7 @@ description:
>  
>  properties:
>    $nodename:
> -    const: "aux-bus"
> +    const: aux-bus
>  
>    panel:
>      $ref: panel/panel-common.yaml#
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
> index 8afdd67d6780..b90b6d18a828 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
> @@ -50,7 +50,7 @@ properties:
>        - const: hdmi
>  
>    mediatek,syscon-hdmi:
> -    $ref: '/schemas/types.yaml#/definitions/phandle-array'
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>      items:
>        - items:
>            - description: phandle to system configuration registers
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index e75a3efe4dac..2188d7c9b0bb 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -74,7 +74,7 @@ properties:
>  
>    syscon-sfpb:
>      description: A phandle to mmss_sfpb syscon node (only for DSIv2).
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    qcom,dual-dsi-mode:
>      type: boolean
> @@ -105,14 +105,14 @@ properties:
>      type: object
>  
>    ports:
> -    $ref: "/schemas/graph.yaml#/properties/ports"
> +    $ref: /schemas/graph.yaml#/properties/ports
>      description: |
>        Contains DSI controller input and output ports as children, each
>        containing one endpoint subnode.
>  
>      properties:
>        port@0:
> -        $ref: "/schemas/graph.yaml#/$defs/port-base"
> +        $ref: /schemas/graph.yaml#/$defs/port-base
>          unevaluatedProperties: false
>          description: |
>            Input endpoints of the controller.
> @@ -128,7 +128,7 @@ properties:
>                    enum: [ 0, 1, 2, 3 ]
>  
>        port@1:
> -        $ref: "/schemas/graph.yaml#/$defs/port-base"
> +        $ref: /schemas/graph.yaml#/$defs/port-base
>          unevaluatedProperties: false
>          description: |
>            Output endpoints of the controller.
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> index 3ec466c3ab38..e6b00d7387ce 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> @@ -58,7 +58,7 @@ properties:
>        maximum: 31
>  
>    qcom,phy-drive-ldo-level:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description:
>        The PHY LDO has an amplitude tuning feature to adjust the LDO output
>        for the HSTX drive. Use supported levels (mV) to offset the drive level
> diff --git a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
> index d67617f6f74a..95ce22c6787a 100644
> --- a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
> @@ -37,7 +37,7 @@ properties:
>  
>    backlight:
>      description: Backlight used by the panel
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/display/renesas,du.yaml b/Documentation/devicetree/bindings/display/renesas,du.yaml
> index d4830f52c512..c5b9e6812bce 100644
> --- a/Documentation/devicetree/bindings/display/renesas,du.yaml
> +++ b/Documentation/devicetree/bindings/display/renesas,du.yaml
> @@ -76,7 +76,7 @@ properties:
>      unevaluatedProperties: false
>  
>    renesas,cmms:
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>      items:
>        maxItems: 1
>      description:
> @@ -84,7 +84,7 @@ properties:
>        available DU channel.
>  
>    renesas,vsps:
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>      items:
>        items:
>          - description: phandle to VSP instance that serves the DU channel
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
> index d5ca8cf86e8e..f448624dd779 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
> @@ -38,7 +38,7 @@ properties:
>      description: The number of cells in a MIPI calibration specifier.
>        Should be 1. The single cell specifies a bitmask of the pads that
>        need to be calibrated for a given device.
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      const: 1
>  
>  additionalProperties: false
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
> index 907fb0baccae..70f0e45c71d6 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
> @@ -69,12 +69,12 @@ properties:
>    # Tegra186 and later
>    nvidia,interface:
>      description: index of the SOR interface
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>  
>    nvidia,ddc-i2c-bus:
>      description: phandle of an I2C controller used for DDC EDID
>        probing
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    nvidia,hpd-gpio:
>      description: specifies a GPIO used for hotplug detection
> @@ -82,23 +82,23 @@ properties:
>  
>    nvidia,edid:
>      description: supplies a binary EDID blob
> -    $ref: "/schemas/types.yaml#/definitions/uint8-array"
> +    $ref: /schemas/types.yaml#/definitions/uint8-array
>  
>    nvidia,panel:
>      description: phandle of a display panel, required for eDP
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    nvidia,xbar-cfg:
>      description: 5 cells containing the crossbar configuration.
>        Each lane of the SOR, identified by the cell's index, is
>        mapped via the crossbar to the pad specified by the cell's
>        value.
> -    $ref: "/schemas/types.yaml#/definitions/uint32-array"
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>  
>    # optional when driving an eDP output
>    nvidia,dpaux:
>      description: phandle to a DispayPort AUX interface
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>  allOf:
>    - if:
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
> index 265a60d79d89..ce4589466a18 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
> @@ -60,13 +60,13 @@ properties:
>    nvidia,outputs:
>      description: A list of phandles of outputs that this display
>        controller can drive.
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>  
>    nvidia,head:
>      description: The number of the display controller head. This
>        is used to setup the various types of output to receive
>        video data from the given head.
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>  
>  additionalProperties: false
>  
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
> index e5a6145c8c53..da75b71e8ece 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
> @@ -29,7 +29,7 @@ properties:
>        - const: dsi
>  
>  allOf:
> -  - $ref: "/schemas/reset/reset.yaml"
> +  - $ref: /schemas/reset/reset.yaml
>  
>  additionalProperties: false
>  
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
> index 511cbe74e729..59e1dc0813e7 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
> @@ -59,12 +59,12 @@ properties:
>      description: Should contain a phandle and a specifier specifying
>        which pads are used by this DSI output and need to be
>        calibrated. See nvidia,tegra114-mipi.yaml for details.
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>  
>    nvidia,ddc-i2c-bus:
>      description: phandle of an I2C controller used for DDC EDID
>        probing
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    nvidia,hpd-gpio:
>      description: specifies a GPIO used for hotplug detection
> @@ -72,19 +72,19 @@ properties:
>  
>    nvidia,edid:
>      description: supplies a binary EDID blob
> -    $ref: "/schemas/types.yaml#/definitions/uint8-array"
> +    $ref: /schemas/types.yaml#/definitions/uint8-array
>  
>    nvidia,panel:
>      description: phandle of a display panel
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    nvidia,ganged-mode:
>      description: contains a phandle to a second DSI controller to
>        gang up with in order to support up to 8 data lanes
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>  allOf:
> -  - $ref: "../dsi-controller.yaml#"
> +  - $ref: ../dsi-controller.yaml#
>    - if:
>        properties:
>          compatible:
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
> index f65e59cfffa7..f77197e4869f 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
> @@ -68,7 +68,7 @@ properties:
>    nvidia,ddc-i2c-bus:
>      description: phandle of an I2C controller used for DDC EDID
>        probing
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    nvidia,hpd-gpio:
>      description: specifies a GPIO used for hotplug detection
> @@ -76,11 +76,11 @@ properties:
>  
>    nvidia,edid:
>      description: supplies a binary EDID blob
> -    $ref: "/schemas/types.yaml#/definitions/uint8-array"
> +    $ref: /schemas/types.yaml#/definitions/uint8-array
>  
>    nvidia,panel:
>      description: phandle of a display panel
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>  
>    "#sound-dai-cells":
>      const: 0
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml b/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
> index 5c7d2cbc4aac..4247280d6c3c 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
> +++ b/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
> @@ -88,7 +88,7 @@ properties:
>            The DSS DPI output port node from video port 2
>  
>    ti,am65x-oldi-io-ctrl:
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>      description:
>        phandle to syscon device node mapping OLDI IO_CTRL registers.
>        The mapped range should point to OLDI_DAT0_IO_CTRL, map it and
> diff --git a/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml b/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
> index fc02c5d50ce4..87404d72ea37 100644
> --- a/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
> +++ b/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
> @@ -89,25 +89,25 @@ properties:
>      description: Display output colorspace (C_DISPLAY_COLOR_SPACE).
>  
>    xylon,display-depth:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: Display output depth (C_PIXEL_DATA_WIDTH).
>  
>    xylon,row-stride:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: Fixed number of pixels in a framebuffer row (C_ROW_STRIDE).
>  
>    xylon,dithering:
> -    $ref: "/schemas/types.yaml#/definitions/flag"
> +    $ref: /schemas/types.yaml#/definitions/flag
>      description: Dithering module is enabled (C_XCOLOR)
>  
>    xylon,background-layer:
> -    $ref: "/schemas/types.yaml#/definitions/flag"
> +    $ref: /schemas/types.yaml#/definitions/flag
>      description: |
>        The last layer is used to display a black background (C_USE_BACKGROUND).
>        The layer must still be registered.
>  
>    xylon,layers-configurable:
> -    $ref: "/schemas/types.yaml#/definitions/flag"
> +    $ref: /schemas/types.yaml#/definitions/flag
>      description: |
>        Configuration of layers' size, position and offset is enabled
>        (C_USE_SIZE_POSITION).
> @@ -131,7 +131,7 @@ properties:
>              maxItems: 1
>  
>            xylon,layer-depth:
> -            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            $ref: /schemas/types.yaml#/definitions/uint32
>              description: Layer depth (C_LAYER_X_DATA_WIDTH).
>  
>            xylon,layer-colorspace:
> @@ -151,19 +151,19 @@ properties:
>              description: Alpha mode for the layer (C_LAYER_X_ALPHA_MODE).
>  
>            xylon,layer-base-offset:
> -            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            $ref: /schemas/types.yaml#/definitions/uint32
>              description: |
>                Offset in number of lines (C_LAYER_X_OFFSET) starting from the
>                video RAM base (C_VMEM_BASEADDR), only for version 3.
>  
>            xylon,layer-buffer-offset:
> -            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            $ref: /schemas/types.yaml#/definitions/uint32
>              description: |
>                Offset in number of lines (C_BUFFER_*_OFFSET) starting from the
>                layer base offset for the second buffer used in double-buffering.
>  
>            xylon,layer-primary:
> -            $ref: "/schemas/types.yaml#/definitions/flag"
> +            $ref: /schemas/types.yaml#/definitions/flag
>              description: |
>                Layer should be registered as a primary plane (exactly one is
>                required).

-- 
Regards,

Laurent Pinchart

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: display: Drop unneeded quotes
  2023-03-17 23:36 [PATCH] dt-bindings: display: Drop unneeded quotes Rob Herring
  2023-03-18  8:29 ` Laurent Pinchart
@ 2023-03-19 12:06 ` Krzysztof Kozlowski
  2023-03-20 15:43 ` Chun-Kuang Hu
  2 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2023-03-19 12:06 UTC (permalink / raw)
  To: Rob Herring, Robin van der Gracht, Miguel Ojeda,
	Krzysztof Kozlowski, Andrzej Hajda, Neil Armstrong, Robert Foss,
	Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie,
	Daniel Vetter, Chun-Kuang Hu, Philipp Zabel, Rob Clark,
	Abhinav Kumar, Dmitry Baryshkov, Sean Paul, Thierry Reding,
	Sam Ravnborg, Kieran Bingham, Jonathan Hunter, Jyri Sarha,
	Tomi Valkeinen, Matthias Brugger, AngeloGioacchino Del Regno
  Cc: devicetree, linux-kernel, dri-devel, linux-mediatek,
	linux-arm-msm, freedreno, linux-renesas-soc, linux-tegra,
	linux-arm-kernel

On 18/03/2023 00:36, Rob Herring wrote:
> Cleanup bindings dropping unneeded quotes. Once all these are fixed,
> checking for this can be enabled in yamllint.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: display: Drop unneeded quotes
  2023-03-17 23:36 [PATCH] dt-bindings: display: Drop unneeded quotes Rob Herring
  2023-03-18  8:29 ` Laurent Pinchart
  2023-03-19 12:06 ` Krzysztof Kozlowski
@ 2023-03-20 15:43 ` Chun-Kuang Hu
  2 siblings, 0 replies; 4+ messages in thread
From: Chun-Kuang Hu @ 2023-03-20 15:43 UTC (permalink / raw)
  To: Rob Herring
  Cc: Robin van der Gracht, Miguel Ojeda, Krzysztof Kozlowski,
	Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
	Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter,
	Chun-Kuang Hu, Philipp Zabel, Rob Clark, Abhinav Kumar,
	Dmitry Baryshkov, Sean Paul, Thierry Reding, Sam Ravnborg,
	Kieran Bingham, Jonathan Hunter, Jyri Sarha, Tomi Valkeinen,
	Matthias Brugger, AngeloGioacchino Del Regno, devicetree,
	linux-kernel, dri-devel, linux-mediatek, linux-arm-msm,
	freedreno, linux-renesas-soc, linux-tegra, linux-arm-kernel

Hi, Rob:

Rob Herring <robh@kernel.org> 於 2023年3月18日 週六 上午7:36寫道:
>
> Cleanup bindings dropping unneeded quotes. Once all these are fixed,
> checking for this can be enabled in yamllint.

For MediaTek part,

Reviewed-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>

>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  .../bindings/auxdisplay/holtek,ht16k33.yaml    |  2 +-
>  .../bindings/display/bridge/nxp,ptn3460.yaml   |  2 +-
>  .../display/bridge/toshiba,tc358767.yaml       |  2 +-
>  .../bindings/display/dp-aux-bus.yaml           |  2 +-
>  .../display/mediatek/mediatek,hdmi.yaml        |  2 +-
>  .../display/msm/dsi-controller-main.yaml       |  8 ++++----
>  .../bindings/display/msm/dsi-phy-10nm.yaml     |  2 +-
>  .../bindings/display/panel/ronbo,rb070d30.yaml |  2 +-
>  .../bindings/display/renesas,du.yaml           |  4 ++--
>  .../display/tegra/nvidia,tegra114-mipi.yaml    |  2 +-
>  .../display/tegra/nvidia,tegra124-sor.yaml     | 12 ++++++------
>  .../display/tegra/nvidia,tegra186-dc.yaml      |  4 ++--
>  .../tegra/nvidia,tegra186-dsi-padctl.yaml      |  2 +-
>  .../display/tegra/nvidia,tegra20-dsi.yaml      | 12 ++++++------
>  .../display/tegra/nvidia,tegra20-hdmi.yaml     |  6 +++---
>  .../bindings/display/ti/ti,am65x-dss.yaml      |  2 +-
>  .../display/xylon,logicvc-display.yaml         | 18 +++++++++---------
>  17 files changed, 42 insertions(+), 42 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
> index fc4873deb76f..4f6ffb8182a9 100644
> --- a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
> +++ b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
> @@ -10,7 +10,7 @@ maintainers:
>    - Robin van der Gracht <robin@protonic.nl>
>
>  allOf:
> -  - $ref: "/schemas/input/matrix-keymap.yaml#"
> +  - $ref: /schemas/input/matrix-keymap.yaml#
>
>  properties:
>    compatible:
> diff --git a/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml b/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
> index 107dd138e6c6..cdeb67bc05f0 100644
> --- a/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/nxp,ptn3460.yaml
> @@ -18,7 +18,7 @@ properties:
>      maxItems: 1
>
>    edid-emulation:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description:
>        The EDID emulation entry to use
>        Value  Resolution  Description
> diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
> index 140927884418..e1494b5007cb 100644
> --- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
> @@ -23,7 +23,7 @@ properties:
>          i2c address of the bridge, 0x68 or 0x0f, depending on bootstrap pins
>
>    clock-names:
> -    const: "ref"
> +    const: ref
>
>    clocks:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/display/dp-aux-bus.yaml b/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
> index 5e4afe9f98fb..0ece7b01790b 100644
> --- a/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
> +++ b/Documentation/devicetree/bindings/display/dp-aux-bus.yaml
> @@ -26,7 +26,7 @@ description:
>
>  properties:
>    $nodename:
> -    const: "aux-bus"
> +    const: aux-bus
>
>    panel:
>      $ref: panel/panel-common.yaml#
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
> index 8afdd67d6780..b90b6d18a828 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,hdmi.yaml
> @@ -50,7 +50,7 @@ properties:
>        - const: hdmi
>
>    mediatek,syscon-hdmi:
> -    $ref: '/schemas/types.yaml#/definitions/phandle-array'
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>      items:
>        - items:
>            - description: phandle to system configuration registers
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index e75a3efe4dac..2188d7c9b0bb 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -74,7 +74,7 @@ properties:
>
>    syscon-sfpb:
>      description: A phandle to mmss_sfpb syscon node (only for DSIv2).
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    qcom,dual-dsi-mode:
>      type: boolean
> @@ -105,14 +105,14 @@ properties:
>      type: object
>
>    ports:
> -    $ref: "/schemas/graph.yaml#/properties/ports"
> +    $ref: /schemas/graph.yaml#/properties/ports
>      description: |
>        Contains DSI controller input and output ports as children, each
>        containing one endpoint subnode.
>
>      properties:
>        port@0:
> -        $ref: "/schemas/graph.yaml#/$defs/port-base"
> +        $ref: /schemas/graph.yaml#/$defs/port-base
>          unevaluatedProperties: false
>          description: |
>            Input endpoints of the controller.
> @@ -128,7 +128,7 @@ properties:
>                    enum: [ 0, 1, 2, 3 ]
>
>        port@1:
> -        $ref: "/schemas/graph.yaml#/$defs/port-base"
> +        $ref: /schemas/graph.yaml#/$defs/port-base
>          unevaluatedProperties: false
>          description: |
>            Output endpoints of the controller.
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> index 3ec466c3ab38..e6b00d7387ce 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> @@ -58,7 +58,7 @@ properties:
>        maximum: 31
>
>    qcom,phy-drive-ldo-level:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description:
>        The PHY LDO has an amplitude tuning feature to adjust the LDO output
>        for the HSTX drive. Use supported levels (mV) to offset the drive level
> diff --git a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
> index d67617f6f74a..95ce22c6787a 100644
> --- a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
> @@ -37,7 +37,7 @@ properties:
>
>    backlight:
>      description: Backlight used by the panel
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/display/renesas,du.yaml b/Documentation/devicetree/bindings/display/renesas,du.yaml
> index d4830f52c512..c5b9e6812bce 100644
> --- a/Documentation/devicetree/bindings/display/renesas,du.yaml
> +++ b/Documentation/devicetree/bindings/display/renesas,du.yaml
> @@ -76,7 +76,7 @@ properties:
>      unevaluatedProperties: false
>
>    renesas,cmms:
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>      items:
>        maxItems: 1
>      description:
> @@ -84,7 +84,7 @@ properties:
>        available DU channel.
>
>    renesas,vsps:
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>      items:
>        items:
>          - description: phandle to VSP instance that serves the DU channel
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
> index d5ca8cf86e8e..f448624dd779 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra114-mipi.yaml
> @@ -38,7 +38,7 @@ properties:
>      description: The number of cells in a MIPI calibration specifier.
>        Should be 1. The single cell specifies a bitmask of the pads that
>        need to be calibrated for a given device.
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      const: 1
>
>  additionalProperties: false
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
> index 907fb0baccae..70f0e45c71d6 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml
> @@ -69,12 +69,12 @@ properties:
>    # Tegra186 and later
>    nvidia,interface:
>      description: index of the SOR interface
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>
>    nvidia,ddc-i2c-bus:
>      description: phandle of an I2C controller used for DDC EDID
>        probing
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    nvidia,hpd-gpio:
>      description: specifies a GPIO used for hotplug detection
> @@ -82,23 +82,23 @@ properties:
>
>    nvidia,edid:
>      description: supplies a binary EDID blob
> -    $ref: "/schemas/types.yaml#/definitions/uint8-array"
> +    $ref: /schemas/types.yaml#/definitions/uint8-array
>
>    nvidia,panel:
>      description: phandle of a display panel, required for eDP
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    nvidia,xbar-cfg:
>      description: 5 cells containing the crossbar configuration.
>        Each lane of the SOR, identified by the cell's index, is
>        mapped via the crossbar to the pad specified by the cell's
>        value.
> -    $ref: "/schemas/types.yaml#/definitions/uint32-array"
> +    $ref: /schemas/types.yaml#/definitions/uint32-array
>
>    # optional when driving an eDP output
>    nvidia,dpaux:
>      description: phandle to a DispayPort AUX interface
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>  allOf:
>    - if:
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
> index 265a60d79d89..ce4589466a18 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml
> @@ -60,13 +60,13 @@ properties:
>    nvidia,outputs:
>      description: A list of phandles of outputs that this display
>        controller can drive.
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>
>    nvidia,head:
>      description: The number of the display controller head. This
>        is used to setup the various types of output to receive
>        video data from the given head.
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>
>  additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
> index e5a6145c8c53..da75b71e8ece 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra186-dsi-padctl.yaml
> @@ -29,7 +29,7 @@ properties:
>        - const: dsi
>
>  allOf:
> -  - $ref: "/schemas/reset/reset.yaml"
> +  - $ref: /schemas/reset/reset.yaml
>
>  additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
> index 511cbe74e729..59e1dc0813e7 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dsi.yaml
> @@ -59,12 +59,12 @@ properties:
>      description: Should contain a phandle and a specifier specifying
>        which pads are used by this DSI output and need to be
>        calibrated. See nvidia,tegra114-mipi.yaml for details.
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>
>    nvidia,ddc-i2c-bus:
>      description: phandle of an I2C controller used for DDC EDID
>        probing
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    nvidia,hpd-gpio:
>      description: specifies a GPIO used for hotplug detection
> @@ -72,19 +72,19 @@ properties:
>
>    nvidia,edid:
>      description: supplies a binary EDID blob
> -    $ref: "/schemas/types.yaml#/definitions/uint8-array"
> +    $ref: /schemas/types.yaml#/definitions/uint8-array
>
>    nvidia,panel:
>      description: phandle of a display panel
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    nvidia,ganged-mode:
>      description: contains a phandle to a second DSI controller to
>        gang up with in order to support up to 8 data lanes
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>  allOf:
> -  - $ref: "../dsi-controller.yaml#"
> +  - $ref: ../dsi-controller.yaml#
>    - if:
>        properties:
>          compatible:
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
> index f65e59cfffa7..f77197e4869f 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-hdmi.yaml
> @@ -68,7 +68,7 @@ properties:
>    nvidia,ddc-i2c-bus:
>      description: phandle of an I2C controller used for DDC EDID
>        probing
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    nvidia,hpd-gpio:
>      description: specifies a GPIO used for hotplug detection
> @@ -76,11 +76,11 @@ properties:
>
>    nvidia,edid:
>      description: supplies a binary EDID blob
> -    $ref: "/schemas/types.yaml#/definitions/uint8-array"
> +    $ref: /schemas/types.yaml#/definitions/uint8-array
>
>    nvidia,panel:
>      description: phandle of a display panel
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>
>    "#sound-dai-cells":
>      const: 0
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml b/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
> index 5c7d2cbc4aac..4247280d6c3c 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
> +++ b/Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml
> @@ -88,7 +88,7 @@ properties:
>            The DSS DPI output port node from video port 2
>
>    ti,am65x-oldi-io-ctrl:
> -    $ref: "/schemas/types.yaml#/definitions/phandle"
> +    $ref: /schemas/types.yaml#/definitions/phandle
>      description:
>        phandle to syscon device node mapping OLDI IO_CTRL registers.
>        The mapped range should point to OLDI_DAT0_IO_CTRL, map it and
> diff --git a/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml b/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
> index fc02c5d50ce4..87404d72ea37 100644
> --- a/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
> +++ b/Documentation/devicetree/bindings/display/xylon,logicvc-display.yaml
> @@ -89,25 +89,25 @@ properties:
>      description: Display output colorspace (C_DISPLAY_COLOR_SPACE).
>
>    xylon,display-depth:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: Display output depth (C_PIXEL_DATA_WIDTH).
>
>    xylon,row-stride:
> -    $ref: "/schemas/types.yaml#/definitions/uint32"
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: Fixed number of pixels in a framebuffer row (C_ROW_STRIDE).
>
>    xylon,dithering:
> -    $ref: "/schemas/types.yaml#/definitions/flag"
> +    $ref: /schemas/types.yaml#/definitions/flag
>      description: Dithering module is enabled (C_XCOLOR)
>
>    xylon,background-layer:
> -    $ref: "/schemas/types.yaml#/definitions/flag"
> +    $ref: /schemas/types.yaml#/definitions/flag
>      description: |
>        The last layer is used to display a black background (C_USE_BACKGROUND).
>        The layer must still be registered.
>
>    xylon,layers-configurable:
> -    $ref: "/schemas/types.yaml#/definitions/flag"
> +    $ref: /schemas/types.yaml#/definitions/flag
>      description: |
>        Configuration of layers' size, position and offset is enabled
>        (C_USE_SIZE_POSITION).
> @@ -131,7 +131,7 @@ properties:
>              maxItems: 1
>
>            xylon,layer-depth:
> -            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            $ref: /schemas/types.yaml#/definitions/uint32
>              description: Layer depth (C_LAYER_X_DATA_WIDTH).
>
>            xylon,layer-colorspace:
> @@ -151,19 +151,19 @@ properties:
>              description: Alpha mode for the layer (C_LAYER_X_ALPHA_MODE).
>
>            xylon,layer-base-offset:
> -            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            $ref: /schemas/types.yaml#/definitions/uint32
>              description: |
>                Offset in number of lines (C_LAYER_X_OFFSET) starting from the
>                video RAM base (C_VMEM_BASEADDR), only for version 3.
>
>            xylon,layer-buffer-offset:
> -            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            $ref: /schemas/types.yaml#/definitions/uint32
>              description: |
>                Offset in number of lines (C_BUFFER_*_OFFSET) starting from the
>                layer base offset for the second buffer used in double-buffering.
>
>            xylon,layer-primary:
> -            $ref: "/schemas/types.yaml#/definitions/flag"
> +            $ref: /schemas/types.yaml#/definitions/flag
>              description: |
>                Layer should be registered as a primary plane (exactly one is
>                required).
> --
> 2.39.2
>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-03-20 15:44 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-17 23:36 [PATCH] dt-bindings: display: Drop unneeded quotes Rob Herring
2023-03-18  8:29 ` Laurent Pinchart
2023-03-19 12:06 ` Krzysztof Kozlowski
2023-03-20 15:43 ` Chun-Kuang Hu

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).