diff mbox series

[RESEND,v4,1/2] ASoC: dt-bindings: irondevice,sma1307: Add initial DT

Message ID 20241106005800.7520-2-kiseok.jo@irondevice.com
State Not Applicable
Headers show
Series Add a driver for the Iron Device SMA1307 Amp | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Ki-Seok Jo Nov. 6, 2024, 12:57 a.m. UTC
This adds the schema binding for the Iron Device SMA1307 Amp

Signed-off-by: Kiseok Jo <kiseok.jo@irondevice.com>
---
 .../bindings/sound/irondevice,sma1307.yaml    | 53 +++++++++++++++++++
 1 file changed, 53 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml

Comments

Krzysztof Kozlowski Nov. 6, 2024, 1:55 p.m. UTC | #1
On Wed, Nov 06, 2024 at 09:57:59AM +0900, Kiseok Jo wrote:
> This adds the schema binding for the Iron Device SMA1307 Amp
> 

That's a v5. RESEND means you send exactly the same or the same plus
tags.

If you change patch, then it is v5, so the tools know how to handle it
and we do no waste time at reviewing things twice.

> Signed-off-by: Kiseok Jo <kiseok.jo@irondevice.com>

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

Best regards,
Krzysztof
Mark Brown Nov. 6, 2024, 2:05 p.m. UTC | #2
On Wed, Nov 06, 2024 at 02:55:21PM +0100, Krzysztof Kozlowski wrote:
> On Wed, Nov 06, 2024 at 09:57:59AM +0900, Kiseok Jo wrote:
> > This adds the schema binding for the Iron Device SMA1307 Amp

> That's a v5. RESEND means you send exactly the same or the same plus
> tags.

No, this is actually a resend - he's resending the v4 that you replied
to just before this one which was sent with mangled subject lines and
missing changelogs.
Ki-Seok Jo Nov. 6, 2024, 11:28 p.m. UTC | #3
> 
> On Wed, Nov 06, 2024 at 02:55:21PM +0100, Krzysztof Kozlowski wrote:
> > On Wed, Nov 06, 2024 at 09:57:59AM +0900, Kiseok Jo wrote:
> > > This adds the schema binding for the Iron Device SMA1307 Amp
> 
> > That's a v5. RESEND means you send exactly the same or the same plus
> > tags.
> 
> No, this is actually a resend - he's resending the v4 that you replied to just
> before this one which was sent with mangled subject lines and missing
> changelogs.

Oh, I'm sorry, that was my mistake.
I realized the error and send it again using a script, but I accidentally left out the changelogs.
I'll resend it with the changelogs added this time.
Since there are no changes, I'll simply resend it without additional modifications.

Thank you for reviewing and providing feedback.

Best regards,
Kiseok Jo
Mark Brown Nov. 6, 2024, 11:32 p.m. UTC | #4
On Wed, Nov 06, 2024 at 11:28:45PM +0000, Ki-Seok Jo wrote:

> Oh, I'm sorry, that was my mistake.
> I realized the error and send it again using a script, but I accidentally left out the changelogs.
> I'll resend it with the changelogs added this time.
> Since there are no changes, I'll simply resend it without additional modifications.

It's OK, no need for any further resends just now - just try to consider
it in future.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml b/Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml
new file mode 100644
index 000000000..1e2a038d0
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml
@@ -0,0 +1,53 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/sound/irondevice,sma1307.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Iron Device SMA1307 Audio Amplifier
+
+maintainers:
+  - Kiseok Jo <kiseok.jo@irondevice.com>
+
+description:
+  SMA1307 boosted digital speaker amplifier with feedback-loop.
+
+allOf:
+  - $ref: dai-common.yaml#
+
+properties:
+  compatible:
+    enum:
+      - irondevice,sma1307a
+      - irondevice,sma1307aq
+    description:
+      If a 'q' is added, it indicated the product is AEC-Q100
+      qualified for automotive applications. SMA1307A supports
+      both WLCSP and QFN packages. However, SMA1307AQ only
+      supports the QFN package.
+
+  reg:
+    maxItems: 1
+
+  '#sound-dai-cells':
+    const: 1
+
+required:
+  - compatible
+  - reg
+  - '#sound-dai-cells'
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        amplifier@1e {
+            compatible = "irondevice,sma1307a";
+            reg = <0x1e>;
+            #sound-dai-cells = <1>;
+        };
+    };