diff mbox series

dt-bindings: sound: sun8i-a33-codec: Add Allwinner A64 codec compatible fallback

Message ID 20201011211543.155561-1-peron.clem@gmail.com
State Superseded
Headers show
Series dt-bindings: sound: sun8i-a33-codec: Add Allwinner A64 codec compatible fallback | expand

Checks

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

Commit Message

Clément Péron Oct. 11, 2020, 9:15 p.m. UTC
make dtbs_check report a warning because the documentation
for the A64 codec compatible is missing.

The A64 codec compatible is actually a simple fallback to the A33.

Reflect this in the dt-bindings Documentation.

Signed-off-by: Clément Péron <peron.clem@gmail.com>
---
 .../bindings/sound/allwinner,sun8i-a33-codec.yaml           | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Maxime Ripard Oct. 12, 2020, 7:50 a.m. UTC | #1
Hi!

On Sun, Oct 11, 2020 at 11:15:42PM +0200, Clément Péron wrote:
> make dtbs_check report a warning because the documentation
> for the A64 codec compatible is missing.
> 
> The A64 codec compatible is actually a simple fallback to the A33.
> 
> Reflect this in the dt-bindings Documentation.
> 
> Signed-off-by: Clément Péron <peron.clem@gmail.com>

This patch is already in the ASoC tree and linux-next:

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/commit/Documentation/devicetree/bindings/sound/allwinner,sun8i-a33-codec.yaml?id=cef305d4eb0733f25215793ed30b056a7db9bb62

Maxime
Clément Péron Oct. 12, 2020, 8:37 a.m. UTC | #2
Hi Maxime and all,

On Mon, 12 Oct 2020 at 09:50, Maxime Ripard <maxime@cerno.tech> wrote:
>
> Hi!
>
> On Sun, Oct 11, 2020 at 11:15:42PM +0200, Clément Péron wrote:
> > make dtbs_check report a warning because the documentation
> > for the A64 codec compatible is missing.
> >
> > The A64 codec compatible is actually a simple fallback to the A33.
> >
> > Reflect this in the dt-bindings Documentation.
> >
> > Signed-off-by: Clément Péron <peron.clem@gmail.com>
>
> This patch is already in the ASoC tree and linux-next:

I was indeed on top of sunxi/for-next and not linux-next/master

I will test on top of linux-next for the next ones.

Sorry for the duplicate patch and the noise :(.
Clement

>
> https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/commit/Documentation/devicetree/bindings/sound/allwinner,sun8i-a33-codec.yaml?id=cef305d4eb0733f25215793ed30b056a7db9bb62
>
> Maxime
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/allwinner,sun8i-a33-codec.yaml b/Documentation/devicetree/bindings/sound/allwinner,sun8i-a33-codec.yaml
index 55d28268d2f4..e78a22496e7c 100644
--- a/Documentation/devicetree/bindings/sound/allwinner,sun8i-a33-codec.yaml
+++ b/Documentation/devicetree/bindings/sound/allwinner,sun8i-a33-codec.yaml
@@ -15,7 +15,11 @@  properties:
     const: 0
 
   compatible:
-    const: allwinner,sun8i-a33-codec
+    oneOf:
+      - const: allwinner,sun8i-a33-codec
+      - items:
+        - const: allwinner,sun50i-a64-codec
+        - const: allwinner,sun8i-a33-codec
 
   reg:
     maxItems: 1