diff mbox series

[v5,1/2] ASoC: dt-bindings: max98520: add initial bindings

Message ID 20211020083055.23625-1-george.song@maximintegrated.com
State Not Applicable, archived
Headers show
Series [v5,1/2] ASoC: dt-bindings: max98520: add initial bindings | expand

Checks

Context Check Description
robh/checkpatch success
robh/dt-meta-schema success
robh/dtbs-check success

Commit Message

George Song Oct. 20, 2021, 8:30 a.m. UTC
add initial bindings for max98520 audio amplifier

Signed-off-by: George Song <george.song@maximintegrated.com>
---
 .../bindings/sound/maxim,max98520.yaml        | 36 +++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/maxim,max98520.yaml

Comments

Rob Herring (Arm) Oct. 22, 2021, 11:37 p.m. UTC | #1
On Wed, 20 Oct 2021 17:30:54 +0900, George Song wrote:
> add initial bindings for max98520 audio amplifier
> 
> Signed-off-by: George Song <george.song@maximintegrated.com>
> ---
>  .../bindings/sound/maxim,max98520.yaml        | 36 +++++++++++++++++++
>  1 file changed, 36 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/maxim,max98520.yaml
> 


Please add Acked-by/Reviewed-by tags when posting new versions. However,
there's no need to repost patches *only* to add the tags. The upstream
maintainer will do that for acks received on the version they apply.

If a tag was not added on purpose, please state why and what changed.
George Song Oct. 25, 2021, 7:21 a.m. UTC | #2
> On Wed, 20 Oct 2021 17:30:54 +0900, George Song wrote:
> > add initial bindings for max98520 audio amplifier
> >
> > Signed-off-by: George Song <george.song@maximintegrated.com>
> > ---
> >  .../bindings/sound/maxim,max98520.yaml        | 36
> +++++++++++++++++++
> >  1 file changed, 36 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/sound/maxim,max98520.yaml
> >
> 
> 
> Please add Acked-by/Reviewed-by tags when posting new versions.
> However, there's no need to repost patches *only* to add the tags.
> The upstream maintainer will do that for acks received on the
> version they apply.
> 
> If a tag was not added on purpose, please state why and what changed.
OK I`ll add Reviewed-by tag which is remained from you in v3 patch 
as "Reviewed-by: Rob Herring <robh@kernel.org>"
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/maxim,max98520.yaml b/Documentation/devicetree/bindings/sound/maxim,max98520.yaml
new file mode 100644
index 000000000000..b6509cb2c8e0
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/maxim,max98520.yaml
@@ -0,0 +1,36 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/sound/maxim,max98520.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim Integrated MAX98520 Speaker Amplifier Driver
+
+maintainers:
+  - George Song <george.song@maximintegrated.com>
+
+properties:
+  compatible:
+    const: maxim,max98520
+
+  reg:
+    maxItems: 1
+    description: I2C address of the device.
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+      #address-cells = <1>;
+      #size-cells = <0>;
+      max98520: amplifier@38 {
+        compatible = "maxim,max98520";
+        reg = <0x38>;
+      };
+    };
+