Re: [PATCH v2 1/5] dt-bindings: iio: drop unneeded quotes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 24 Jan 2023 09:10:33 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote:

> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Reviewed-by: Dmitry Rokosov <ddrokosov@xxxxxxxxxxxxxx> # memsensing
> Reviewed-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx> # sama5d2-adc
> Reviewed-by: Puranjay Mohan <puranjay12@xxxxxxxxx> # tmp117
> Acked-by: Rob Herring <robh@xxxxxxxxxx>
> Acked-by: Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx> # ad7292
Hi Krzysztof,

Series applied to the togreg branch of iio.git, initially pushed out
(once build test finishes) as testing for 0-day to poke at.

One trivial comment. It's helpful for any series with more than
one or two patches to have a cover letter even if there isn't much to say
as it provides somewhere for people to give tags for the whole series etc
that b4 will then pick up with out the maintainer having to be careful
that the tag was really meant for the whole series.

Also gives a nice place for me to reply to when saying I picked up the
series :)

Thanks,

Jonathan



> ---
>  .../devicetree/bindings/iio/accel/memsensing,msa311.yaml  | 5 ++---
>  Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
>  Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
>  .../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml    | 2 +-
>  Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
>  .../devicetree/bindings/iio/adc/ingenic,adc.yaml          | 4 ++--
>  .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml    | 4 ++--
>  .../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml    | 2 +-
>  .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml   | 2 +-
>  .../devicetree/bindings/iio/adc/st,stm32-adc.yaml         | 8 ++++----
>  .../devicetree/bindings/iio/adc/ti,ads131e08.yaml         | 2 +-
>  Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
>  .../devicetree/bindings/iio/dac/lltc,ltc1660.yaml         | 4 ++--
>  .../devicetree/bindings/iio/dac/lltc,ltc2632.yaml         | 4 ++--
>  .../devicetree/bindings/iio/dac/st,stm32-dac.yaml         | 4 ++--
>  Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
>  .../devicetree/bindings/iio/temperature/ti,tmp117.yaml    | 6 +++---
>  17 files changed, 29 insertions(+), 30 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> index 23528dcaa073..d530ec041fe7 100644
> --- a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> @@ -1,9 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> -
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: MEMSensing digital 3-Axis accelerometer
>  
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index 75a7184a4735..35ed04350e28 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -61,7 +61,7 @@ required:
>  
>  patternProperties:
>    "^channel@([0-9]|1[0-5])$":
> -    $ref: "adc.yaml"
> +    $ref: adc.yaml
>      type: object
>      description: |
>        Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index 1bfbeed6f299..7cc4ddc4e9b7 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -43,7 +43,7 @@ required:
>  
>  patternProperties:
>    "^channel@[0-7]$":
> -    $ref: "adc.yaml"
> +    $ref: adc.yaml
>      type: object
>      description: |
>        Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> index 31f840d59303..4817b840977a 100644
> --- a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> @@ -41,7 +41,7 @@ properties:
>      description: Startup time expressed in ms, it depends on SoC.
>  
>    atmel,trigger-edge-type:
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description:
>        One of possible edge types for the ADTRG hardware trigger pin.
>        When the specific edge type is detected, the conversion will
> diff --git a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> index 77605f17901c..9c57eb13f892 100644
> --- a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/adc/avia-hx711.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/adc/avia-hx711.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: AVIA HX711 ADC chip for weight cells
>  
> diff --git a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
> index 517e8b1fcb73..b71c951e6d02 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019-2020 Artur Rojek
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/adc/ingenic,adc.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/adc/ingenic,adc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Ingenic JZ47xx ADC controller IIO
>  
> diff --git a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> index 2c93fb41f172..f7b3fde4115a 100644
> --- a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Marcus Folkesson <marcus.folkesson@xxxxxxxxx>
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Microchip MCP3911 Dual channel analog front end (ADC)
>  
> diff --git a/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml b/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
> index 8b743742a5f9..ba86c7b7d622 100644
> --- a/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
> @@ -69,7 +69,7 @@ required:
>  
>  patternProperties:
>    "^channel@[0-7]$":
> -    $ref: "adc.yaml"
> +    $ref: adc.yaml
>      type: object
>      description: |
>        Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> index 81c87295912c..e27d094cfa05 100644
> --- a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> @@ -52,7 +52,7 @@ properties:
>    vdd-supply: true
>  
>    samsung,syscon-phandle:
> -    $ref: '/schemas/types.yaml#/definitions/phandle'
> +    $ref: /schemas/types.yaml#/definitions/phandle
>      description:
>        Phandle to the PMU system controller node (to access the ADC_PHY
>        register on Exynos3250/4x12/5250/5420/5800).
> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> index 1c340c95df16..995cbf8cefc6 100644
> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/adc/st,stm32-adc.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/adc/st,stm32-adc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: STMicroelectronics STM32 ADC
>  
> @@ -80,7 +80,7 @@ properties:
>      description:
>        Phandle to system configuration controller. It can be used to control the
>        analog circuitry on stm32mp1.
> -    $ref: "/schemas/types.yaml#/definitions/phandle-array"
> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>  
>    interrupt-controller: true
>  
> @@ -341,7 +341,7 @@ patternProperties:
>      patternProperties:
>        "^channel@([0-9]|1[0-9])$":
>          type: object
> -        $ref: "adc.yaml"
> +        $ref: adc.yaml
>          description: Represents the external channels which are connected to the ADC.
>  
>          properties:
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
> index 55c2c73626f4..890f125d422c 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
> @@ -77,7 +77,7 @@ required:
>  
>  patternProperties:
>    "^channel@([0-7])$":
> -    $ref: "adc.yaml"
> +    $ref: adc.yaml
>      type: object
>      description: |
>        Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml b/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
> index bdf3bba2d750..32c52f9fe18b 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
> @@ -41,7 +41,7 @@ required:
>  
>  patternProperties:
>    "^channel@[0-7]$":
> -    $ref: "adc.yaml"
> +    $ref: adc.yaml
>      type: object
>  
>      properties:
> diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> index 133b0f867992..c9f51d00fa8f 100644
> --- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> @@ -2,8 +2,8 @@
>  # Copyright 2019 Marcus Folkesson <marcus.folkesson@xxxxxxxxx>
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Linear Technology Micropower octal 8-Bit and 10-Bit DACs
>  
> diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
> index b1eb77335d05..c9e3be3b5754 100644
> --- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: Linear Technology LTC263x 12-/10-/8-Bit Rail-to-Rail DAC
>  
> diff --git a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
> index 0f1bf1110122..04045b932bd2 100644
> --- a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/dac/st,stm32-dac.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/dac/st,stm32-dac.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
>  title: STMicroelectronics STM32 DAC
>  
> diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> index 68b481c63318..decf022335d8 100644
> --- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> @@ -63,7 +63,7 @@ properties:
>      description: if defined provides VDD IO power to the sensor.
>  
>    st,drdy-int-pin:
> -    $ref: '/schemas/types.yaml#/definitions/uint32'
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: |
>        The pin on the package that will be used to signal data ready
>      enum:
> diff --git a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> index 347bc16a4671..c4f1c69f9330 100644
> --- a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> +++ b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> @@ -1,10 +1,10 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/iio/temperature/ti,tmp117.yaml#";
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#";
> +$id: http://devicetree.org/schemas/iio/temperature/ti,tmp117.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>  
> -title: "TI TMP117 - Digital temperature sensor with integrated NV memory"
> +title: TI TMP117 - Digital temperature sensor with integrated NV memory
>  
>  description: |
>      TI TMP117 - Digital temperature sensor with integrated NV memory that supports




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux