All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2 V2] TAS5756M codec driver
@ 2020-02-27 23:23 Charles-Antoine Couret
  2020-02-27 23:23   ` Charles-Antoine Couret
  0 siblings, 1 reply; 5+ messages in thread
From: Charles-Antoine Couret @ 2020-02-27 23:23 UTC (permalink / raw)
  To: alsa-devel; +Cc: Charles-Antoine Couret

V2 changes:
 * Fix bug in driver itself
 * Change device tree attribute names
 * Convert device tree documentation from txt to YAML format

Charles-Antoine Couret (2):
  ASoC: add TAS5756m driver
  ASoC: tas5756m: Add DT binding document

 .../devicetree/bindings/sound/tas5756m.yaml   |   64 +
 sound/soc/codecs/Kconfig                      |    8 +
 sound/soc/codecs/Makefile                     |    2 +
 sound/soc/codecs/tas5756m.c                   | 2161 ++++++++
 sound/soc/codecs/tas5756m.h                   |  351 ++
 sound/soc/codecs/tas5756m_hf3_cfg.h           | 4508 ++++++++++++++++
 sound/soc/codecs/tas5756m_hf4_cfg.h           | 4510 +++++++++++++++++
 sound/soc/codecs/tas5756m_hf6_cfg.h           | 4510 +++++++++++++++++
 sound/soc/codecs/tas5756m_hf7_cfg.h           | 4510 +++++++++++++++++
 9 files changed, 20624 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.yaml
 create mode 100644 sound/soc/codecs/tas5756m.c
 create mode 100644 sound/soc/codecs/tas5756m.h
 create mode 100644 sound/soc/codecs/tas5756m_hf3_cfg.h
 create mode 100644 sound/soc/codecs/tas5756m_hf4_cfg.h
 create mode 100644 sound/soc/codecs/tas5756m_hf6_cfg.h
 create mode 100644 sound/soc/codecs/tas5756m_hf7_cfg.h

-- 
2.24.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 2/2 V2] ASoC: tas5756m: Add DT binding document
  2020-02-27 23:23 [PATCH 0/2 V2] TAS5756M codec driver Charles-Antoine Couret
@ 2020-02-27 23:23   ` Charles-Antoine Couret
  0 siblings, 0 replies; 5+ messages in thread
From: Charles-Antoine Couret @ 2020-02-27 23:23 UTC (permalink / raw)
  To: alsa-devel; +Cc: Charles-Antoine Couret, devicetree

Document the bindings for the tas5756m driver.

Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
---
 .../devicetree/bindings/sound/tas5756m.yaml   | 64 +++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.yaml

diff --git a/Documentation/devicetree/bindings/sound/tas5756m.yaml b/Documentation/devicetree/bindings/sound/tas5756m.yaml
new file mode 100644
index 000000000000..47be70f1633a
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/tas5756m.yaml
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+
+$id: "http://devicetree.org/schemas/tas5756m.yaml#"
+$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+
+title: TAS5756M audio codec Device Tree Bindings
+
+maintainers:
+  - Charles-Antoine Couret <charles-antoine.couret@mind.be>
+
+description: |
+  You can read datasheets there:
+  http://www.ti.com/lit/ds/symlink/tas5754m.pdf
+  http://www.ti.com/lit/ds/symlink/tas5756m.pdf
+
+  TAS5754M datasheet has a more complete datasheet about register mapping
+  which is common with TAS5756M.
+
+  Those devices have a programmable DSP whith several modes named hybridflow.
+  More details about it there: http://www.ti.com/lit/ug/slau577a/slau577a.pdf
+
+properties:
+  compatible:
+    enum:
+      - "ti,tas5756m"
+      - "ti,tas5754m"
+
+  reg:
+    maxItems: 1
+    description: I2C slave address
+
+  "#sound-dai-cells":
+    const: 0
+
+  mute-gpios
+    maxItems: 1
+    description: GPIO wired to the mute pin.
+
+  ti,hybridflow
+    description: |
+      Select the HybridFlow DSP program.
+      If not supplied default DSP program is used instead.
+    allOf:
+      - $ref: /schemas/types.yaml#/definitions/uint32
+      - enum: [1, 2, 3, 4, 5, 6, 7, 8, 9]
+
+additionalProperties: false
+
+required:
+  - "#sound-dai-cells"
+  - compatible
+  - reg
+
+examples:
+  - |
+    tas5756m: tas5756m@4c {
+        compatible = "ti,tas5756m";
+        reg = <0x4c>;
+        #sound-dai-cells = <0>;
+
+        ti,hybridflow = <6>;
+        mute-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
+    };
-- 
2.24.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/2 V2] ASoC: tas5756m: Add DT binding document
@ 2020-02-27 23:23   ` Charles-Antoine Couret
  0 siblings, 0 replies; 5+ messages in thread
From: Charles-Antoine Couret @ 2020-02-27 23:23 UTC (permalink / raw)
  To: alsa-devel; +Cc: devicetree, Charles-Antoine Couret

Document the bindings for the tas5756m driver.

Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
---
 .../devicetree/bindings/sound/tas5756m.yaml   | 64 +++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.yaml

diff --git a/Documentation/devicetree/bindings/sound/tas5756m.yaml b/Documentation/devicetree/bindings/sound/tas5756m.yaml
new file mode 100644
index 000000000000..47be70f1633a
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/tas5756m.yaml
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+
+$id: "http://devicetree.org/schemas/tas5756m.yaml#"
+$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+
+title: TAS5756M audio codec Device Tree Bindings
+
+maintainers:
+  - Charles-Antoine Couret <charles-antoine.couret@mind.be>
+
+description: |
+  You can read datasheets there:
+  http://www.ti.com/lit/ds/symlink/tas5754m.pdf
+  http://www.ti.com/lit/ds/symlink/tas5756m.pdf
+
+  TAS5754M datasheet has a more complete datasheet about register mapping
+  which is common with TAS5756M.
+
+  Those devices have a programmable DSP whith several modes named hybridflow.
+  More details about it there: http://www.ti.com/lit/ug/slau577a/slau577a.pdf
+
+properties:
+  compatible:
+    enum:
+      - "ti,tas5756m"
+      - "ti,tas5754m"
+
+  reg:
+    maxItems: 1
+    description: I2C slave address
+
+  "#sound-dai-cells":
+    const: 0
+
+  mute-gpios
+    maxItems: 1
+    description: GPIO wired to the mute pin.
+
+  ti,hybridflow
+    description: |
+      Select the HybridFlow DSP program.
+      If not supplied default DSP program is used instead.
+    allOf:
+      - $ref: /schemas/types.yaml#/definitions/uint32
+      - enum: [1, 2, 3, 4, 5, 6, 7, 8, 9]
+
+additionalProperties: false
+
+required:
+  - "#sound-dai-cells"
+  - compatible
+  - reg
+
+examples:
+  - |
+    tas5756m: tas5756m@4c {
+        compatible = "ti,tas5756m";
+        reg = <0x4c>;
+        #sound-dai-cells = <0>;
+
+        ti,hybridflow = <6>;
+        mute-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
+    };
-- 
2.24.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2 V2] ASoC: tas5756m: Add DT binding document
  2020-02-27 23:23   ` Charles-Antoine Couret
@ 2020-02-28 15:08     ` Rob Herring
  -1 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2020-02-28 15:08 UTC (permalink / raw)
  To: Charles-Antoine Couret; +Cc: alsa-devel, Charles-Antoine Couret, devicetree

On Fri, 28 Feb 2020 00:23:27 +0100, Charles-Antoine Couret wrote:
> Document the bindings for the tas5756m driver.
> 
> Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
> ---
>  .../devicetree/bindings/sound/tas5756m.yaml   | 64 +++++++++++++++++++
>  1 file changed, 64 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.yaml
> 

My bot found errors running 'make dt_binding_check' on your patch:

warning: no schema found in file: Documentation/devicetree/bindings/sound/tas5756m.yaml
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/sound/tas5756m.yaml: ignoring, error parsing file
Documentation/devicetree/bindings/display/simple-framebuffer.example.dts:21.16-37.11: Warning (chosen_node_is_root): /example-0/chosen: chosen node must be at root node
Documentation/devicetree/bindings/sound/tas5756m.yaml:  did not find expected <document start>
  in "<unicode string>", line 4, column 1
Documentation/devicetree/bindings/Makefile:12: recipe for target 'Documentation/devicetree/bindings/sound/tas5756m.example.dts' failed
make[1]: *** [Documentation/devicetree/bindings/sound/tas5756m.example.dts] Error 1
Makefile:1263: recipe for target 'dt_binding_check' failed
make: *** [dt_binding_check] Error 2

See https://patchwork.ozlabs.org/patch/1246173
Please check and re-submit.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2 V2] ASoC: tas5756m: Add DT binding document
@ 2020-02-28 15:08     ` Rob Herring
  0 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2020-02-28 15:08 UTC (permalink / raw)
  To: Charles-Antoine Couret; +Cc: devicetree, alsa-devel, Charles-Antoine Couret

On Fri, 28 Feb 2020 00:23:27 +0100, Charles-Antoine Couret wrote:
> Document the bindings for the tas5756m driver.
> 
> Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@mind.be>
> ---
>  .../devicetree/bindings/sound/tas5756m.yaml   | 64 +++++++++++++++++++
>  1 file changed, 64 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.yaml
> 

My bot found errors running 'make dt_binding_check' on your patch:

warning: no schema found in file: Documentation/devicetree/bindings/sound/tas5756m.yaml
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/sound/tas5756m.yaml: ignoring, error parsing file
Documentation/devicetree/bindings/display/simple-framebuffer.example.dts:21.16-37.11: Warning (chosen_node_is_root): /example-0/chosen: chosen node must be at root node
Documentation/devicetree/bindings/sound/tas5756m.yaml:  did not find expected <document start>
  in "<unicode string>", line 4, column 1
Documentation/devicetree/bindings/Makefile:12: recipe for target 'Documentation/devicetree/bindings/sound/tas5756m.example.dts' failed
make[1]: *** [Documentation/devicetree/bindings/sound/tas5756m.example.dts] Error 1
Makefile:1263: recipe for target 'dt_binding_check' failed
make: *** [dt_binding_check] Error 2

See https://patchwork.ozlabs.org/patch/1246173
Please check and re-submit.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-02-28 15:09 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-27 23:23 [PATCH 0/2 V2] TAS5756M codec driver Charles-Antoine Couret
2020-02-27 23:23 ` [PATCH 2/2 V2] ASoC: tas5756m: Add DT binding document Charles-Antoine Couret
2020-02-27 23:23   ` Charles-Antoine Couret
2020-02-28 15:08   ` Rob Herring
2020-02-28 15:08     ` Rob Herring

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.