diff mbox series

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

Message ID SL2P216MB23377A60BEC4396ADFA78A6A8C512@SL2P216MB2337.KORP216.PROD.OUTLOOK.COM
State New
Headers show
Series [v3,1/2] ASoC: dt-bindings: irondevice,sma1307: Add initial DT binding | expand

Commit Message

Kiseok Jo Nov. 4, 2024, 7:33 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         | 54 ++++++++++++++++++++++
 1 file changed, 54 insertions(+)

--
2.39.2

Comments

Mark Brown Nov. 4, 2024, 4:41 p.m. UTC | #1
On Mon, Nov 04, 2024 at 10:17:51AM -0600, Rob Herring wrote:
> On Mon, Nov 04, 2024 at 07:33:47AM +0000, Ki-Seok Jo wrote:

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

> Is this difference visible to software? The package is not, so that part 
> is irrelevant.

It seems reasonable to allow it as a compatible if it's sold as a
separate part, even if we don't need to care.
Mark Brown Nov. 7, 2024, 9:52 p.m. UTC | #2
On Mon, 04 Nov 2024 07:33:47 +0000, Ki-Seok Jo wrote:
> This adds the schema binding for the Iron Device SMA1307 Amp
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/2] ASoC: dt-bindings: irondevice,sma1307: Add initial DT binding
      (no commit info)
[2/2] ASoC: sma1307: Add driver for Iron Device SMA1307
      commit: 576c57e6b4c1d734bcb7cc33dde9a99a9383b520

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
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..0bb4ee664
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml
@@ -0,0 +1,54 @@ 
+# 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>;
+        };
+    };