mbox series

[v2,0/2] dw9807 binding and driver compatible changes

Message ID 20220329212654.587451-1-sakari.ailus@linux.intel.com
Headers show
Series dw9807 binding and driver compatible changes | expand

Message

Sakari Ailus March 29, 2022, 9:26 p.m. UTC 
Hello everyone,

These two patches convert dw9807 bindings to json-schema and add a new
compatible string to the dw9807-vcm driver in order to support existing
ACPI firmware.

since v1:

- Use stronger language in a comment related to a compatible string for
  old firmware.

- Remove extra newlines and quotes from bindings.

Sakari Ailus (2):
  dw9807-vcm: Add "dongwoon,dw9807" compatible string
  dt-bindings: Convert Dongwoon dw9807-vcm bindings to json-schema

 .../media/i2c/dongwoon,dw9807-vcm.txt         |  9 ----
 .../media/i2c/dongwoon,dw9807-vcm.yaml        | 43 +++++++++++++++++++
 drivers/media/i2c/dw9807-vcm.c                |  2 +
 3 files changed, 45 insertions(+), 9 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
 create mode 100644 Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml

Comments

Krzysztof Kozlowski March 30, 2022, 7:15 a.m. UTC | #1 
On 29/03/2022 23:26, Sakari Ailus wrote:
> Convert the old text based dw9807-vcm chip DT bindings to json-schema.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
>  .../media/i2c/dongwoon,dw9807-vcm.txt         |  9 ----
>  .../media/i2c/dongwoon,dw9807-vcm.yaml        | 43 +++++++++++++++++++
>  2 files changed, 43 insertions(+), 9 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
>  create mode 100644 Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml
> 
> diff --git a/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt b/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
> deleted file mode 100644
> index c4701f1eaaf63..0000000000000
> --- a/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -Dongwoon Anatech DW9807 voice coil lens driver
> -
> -DW9807 is a 10-bit DAC with current sink capability. It is intended for
> -controlling voice coil lenses.
> -
> -Mandatory properties:
> -
> -- compatible: "dongwoon,dw9807-vcm"
> -- reg: I2C slave address
> diff --git a/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml b/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml
> new file mode 100644
> index 0000000000000..0cf153ed547fe
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml
> @@ -0,0 +1,43 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +# Copyright (C) 2018, 2021 Intel Corporation
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/media/i2c/dongwoon,dw9807-vcm.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Dongwoon Anatech DW9807 voice coil lens driver
> +
> +maintainers:
> +  - Sakari Ailus <sakari.ailus@linux.intel.com>
> +
> +description: |
> +  DW9807 is a 10-bit DAC with current sink capability. It is intended for
> +  controlling voice coil lenses.
> +
> +properties:
> +
> +  compatible:
> +    const: "dongwoon,dw9807-vcm"

Nothing changed here about new lines and quotes.

Best regards,
Krzysztof
Sakari Ailus March 30, 2022, 7:27 a.m. UTC | #2 
On Wed, Mar 30, 2022 at 09:15:47AM +0200, Krzysztof Kozlowski wrote:
> On 29/03/2022 23:26, Sakari Ailus wrote:
> > Convert the old text based dw9807-vcm chip DT bindings to json-schema.
> > 
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > ---
> >  .../media/i2c/dongwoon,dw9807-vcm.txt         |  9 ----
> >  .../media/i2c/dongwoon,dw9807-vcm.yaml        | 43 +++++++++++++++++++
> >  2 files changed, 43 insertions(+), 9 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
> >  create mode 100644 Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml
> > 
> > diff --git a/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt b/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
> > deleted file mode 100644
> > index c4701f1eaaf63..0000000000000
> > --- a/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.txt
> > +++ /dev/null
> > @@ -1,9 +0,0 @@
> > -Dongwoon Anatech DW9807 voice coil lens driver
> > -
> > -DW9807 is a 10-bit DAC with current sink capability. It is intended for
> > -controlling voice coil lenses.
> > -
> > -Mandatory properties:
> > -
> > -- compatible: "dongwoon,dw9807-vcm"
> > -- reg: I2C slave address
> > diff --git a/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml b/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml
> > new file mode 100644
> > index 0000000000000..0cf153ed547fe
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807-vcm.yaml
> > @@ -0,0 +1,43 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +# Copyright (C) 2018, 2021 Intel Corporation
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/media/i2c/dongwoon,dw9807-vcm.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Dongwoon Anatech DW9807 voice coil lens driver
> > +
> > +maintainers:
> > +  - Sakari Ailus <sakari.ailus@linux.intel.com>
> > +
> > +description: |
> > +  DW9807 is a 10-bit DAC with current sink capability. It is intended for
> > +  controlling voice coil lenses.
> > +
> > +properties:
> > +
> > +  compatible:
> > +    const: "dongwoon,dw9807-vcm"
> 
> Nothing changed here about new lines and quotes.

Oops. You're right. Accidentally sent wrong patches, will resend in a
moment...