diff mbox series

[01/54] ASoC: dt-bindings: Add WM8978 Binding

Message ID 20210721140424.725744-2-maxime@cerno.tech
State Superseded
Headers show
Series ARM: dts: Last round of DT schema fixes | expand

Commit Message

Maxime Ripard July 21, 2021, 2:03 p.m. UTC
Even though we had the wm8978 driver for some time and a number of
boards using it already, we never had a binding for it. Let's add it
based on what the driver expects and the boards are providing.

Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
---
 .../devicetree/bindings/sound/wlf,wm8978.yaml | 58 +++++++++++++++++++
 1 file changed, 58 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/wlf,wm8978.yaml

Comments

Rob Herring (Arm) July 23, 2021, 9:32 p.m. UTC | #1
On Wed, Jul 21, 2021 at 04:03:31PM +0200, Maxime Ripard wrote:
> Even though we had the wm8978 driver for some time and a number of

> boards using it already, we never had a binding for it. Let's add it

> based on what the driver expects and the boards are providing.

> 

> Cc: alsa-devel@alsa-project.org

> Cc: devicetree@vger.kernel.org

> Cc: Liam Girdwood <lgirdwood@gmail.com>

> Cc: Mark Brown <broonie@kernel.org>

> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

> ---

>  .../devicetree/bindings/sound/wlf,wm8978.yaml | 58 +++++++++++++++++++

>  1 file changed, 58 insertions(+)

>  create mode 100644 Documentation/devicetree/bindings/sound/wlf,wm8978.yaml

> 

> diff --git a/Documentation/devicetree/bindings/sound/wlf,wm8978.yaml b/Documentation/devicetree/bindings/sound/wlf,wm8978.yaml

> new file mode 100644

> index 000000000000..6761380261e3

> --- /dev/null

> +++ b/Documentation/devicetree/bindings/sound/wlf,wm8978.yaml

> @@ -0,0 +1,58 @@

> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)

> +%YAML 1.2

> +---

> +$id: http://devicetree.org/schemas/sound/wlf,wm8978.yaml#

> +$schema: http://devicetree.org/meta-schemas/core.yaml#

> +

> +title: Wolfson WM8978 Codec Device Tree Bindings

> +

> +maintainers:

> +  - Mark Brown <broonie@kernel.org>


No one from Wolfson cares about this device?

Otherwise,

Reviewed-by: Rob Herring <robh@kernel.org>


> +

> +properties:

> +  '#sound-dai-cells':

> +    const: 0

> +

> +  compatible:

> +    const: wlf,wm8978

> +

> +  reg:

> +    maxItems: 1

> +

> +  spi-max-frequency:

> +    maximum: 526000

> +

> +required:

> +  - '#sound-dai-cells'

> +  - compatible

> +  - reg

> +

> +additionalProperties: false

> +

> +examples:

> +  - |

> +    spi {

> +        #address-cells = <1>;

> +        #size-cells = <0>;

> +

> +        codec@0 {

> +            #sound-dai-cells = <0>;

> +            compatible = "wlf,wm8978";

> +            reg = <0>;

> +            spi-max-frequency = <500000>;

> +        };

> +    };

> +

> +  - |

> +    i2c {

> +        #address-cells = <1>;

> +        #size-cells = <0>;

> +

> +        codec@0 {

> +            #sound-dai-cells = <0>;

> +            compatible = "wlf,wm8978";

> +            reg = <0>;

> +        };

> +    };

> +

> +...

> -- 

> 2.31.1

> 

>
Mark Brown July 26, 2021, 12:17 a.m. UTC | #2
On Fri, Jul 23, 2021 at 03:32:51PM -0600, Rob Herring wrote:
> On Wed, Jul 21, 2021 at 04:03:31PM +0200, Maxime Ripard wrote:


> > Even though we had the wm8978 driver for some time and a number of

> > boards using it already, we never had a binding for it. Let's add it

> > based on what the driver expects and the boards are providing.


> > +maintainers:

> > +  - Mark Brown <broonie@kernel.org>


> No one from Wolfson cares about this device?


> Otherwise,


> Reviewed-by: Rob Herring <robh@kernel.org>


Well, Wolfson no longer exists since it was bought by Cirrus but copying
in the Cirrus people to see if they want to be listed there.
Richard Fitzgerald July 27, 2021, 12:21 p.m. UTC | #3
On 26/07/2021 01:17, Mark Brown wrote:
> On Fri, Jul 23, 2021 at 03:32:51PM -0600, Rob Herring wrote:

>> On Wed, Jul 21, 2021 at 04:03:31PM +0200, Maxime Ripard wrote:

> 

>>> Even though we had the wm8978 driver for some time and a number of

>>> boards using it already, we never had a binding for it. Let's add it

>>> based on what the driver expects and the boards are providing.

> 

>>> +maintainers:

>>> +  - Mark Brown <broonie@kernel.org>

> 

>> No one from Wolfson cares about this device?

> 

>> Otherwise,

> 

>> Reviewed-by: Rob Herring <robh@kernel.org>

> 

> Well, Wolfson no longer exists since it was bought by Cirrus but copying

> in the Cirrus people to see if they want to be listed there.

> 


People, ownership, and email addresses can change. It's always best to
check MAINTAINERS to see the current support status. In this case it has
a section to pick up those Wolfson drivers, with the correct email
address. The list email will be monitored by multiple people.

WOLFSON MICROELECTRONICS DRIVERS
L:	patches@opensource.cirrus.com
S:	Supported
<SNIP>
F:	include/sound/wm????.h
<SNIP>
F:	sound/soc/codecs/wm*
Mark Brown July 27, 2021, 12:58 p.m. UTC | #4
On Tue, Jul 27, 2021 at 01:21:52PM +0100, Richard Fitzgerald wrote:
> On 26/07/2021 01:17, Mark Brown wrote:

> > On Fri, Jul 23, 2021 at 03:32:51PM -0600, Rob Herring wrote:

> > > On Wed, Jul 21, 2021 at 04:03:31PM +0200, Maxime Ripard wrote:


> > > > Even though we had the wm8978 driver for some time and a number of

> > > > boards using it already, we never had a binding for it. Let's add it

> > > > based on what the driver expects and the boards are providing.


> > > > +maintainers:

> > > > +  - Mark Brown <broonie@kernel.org>


> > > No one from Wolfson cares about this device?


> > > Otherwise,


> > > Reviewed-by: Rob Herring <robh@kernel.org>


> > Well, Wolfson no longer exists since it was bought by Cirrus but copying

> > in the Cirrus people to see if they want to be listed there.


> People, ownership, and email addresses can change. It's always best to

> check MAINTAINERS to see the current support status. In this case it has

> a section to pick up those Wolfson drivers, with the correct email

> address. The list email will be monitored by multiple people.

> 

> WOLFSON MICROELECTRONICS DRIVERS

> L:	patches@opensource.cirrus.com

> S:	Supported

> <SNIP>

> F:	include/sound/wm????.h

> <SNIP>

> F:	sound/soc/codecs/wm*


Might be worth updating that to cover the DT bindings too (Arizona is
the only sound driver for some reason, wm8974 already exists but is not
covered).
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/wlf,wm8978.yaml b/Documentation/devicetree/bindings/sound/wlf,wm8978.yaml
new file mode 100644
index 000000000000..6761380261e3
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/wlf,wm8978.yaml
@@ -0,0 +1,58 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/sound/wlf,wm8978.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Wolfson WM8978 Codec Device Tree Bindings
+
+maintainers:
+  - Mark Brown <broonie@kernel.org>
+
+properties:
+  '#sound-dai-cells':
+    const: 0
+
+  compatible:
+    const: wlf,wm8978
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency:
+    maximum: 526000
+
+required:
+  - '#sound-dai-cells'
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        codec@0 {
+            #sound-dai-cells = <0>;
+            compatible = "wlf,wm8978";
+            reg = <0>;
+            spi-max-frequency = <500000>;
+        };
+    };
+
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        codec@0 {
+            #sound-dai-cells = <0>;
+            compatible = "wlf,wm8978";
+            reg = <0>;
+        };
+    };
+
+...