mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] ASoC: dt-bindings: everest,es8328: Document audio graph port
@ 2024-10-19  0:43 Cristian Ciocaltea
  2024-10-21 19:07 ` Rob Herring (Arm)
  2024-10-22 14:07 ` Mark Brown
  0 siblings, 2 replies; 3+ messages in thread
From: Cristian Ciocaltea @ 2024-10-19  0:43 UTC (permalink / raw)
  To: Liam Girdwood, Mark Brown, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, David Yang
  Cc: Heiko Stuebner, kernel, linux-sound, devicetree, linux-kernel

The ES8328/ES8388 audio codec is currently used in conjunction with
audio-graph-card to provide an endpoint for binding with the other side
of the audio link.

This is achieved via the 'port' property, which is not supported by the
binding:

  rk3588s-indiedroid-nova.dtb: audio-codec@11: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
    from schema $id: http://devicetree.org/schemas/sound/everest,es8328.yaml#

Document the missing property.

Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@collabora.com>
---
 Documentation/devicetree/bindings/sound/everest,es8328.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/everest,es8328.yaml b/Documentation/devicetree/bindings/sound/everest,es8328.yaml
index a0f4670fa38c8435b63566f7383c940b4950caa4..ed18e40dcaacf7eed39fa659795efb1518678379 100644
--- a/Documentation/devicetree/bindings/sound/everest,es8328.yaml
+++ b/Documentation/devicetree/bindings/sound/everest,es8328.yaml
@@ -50,6 +50,10 @@ properties:
   HPVDD-supply:
     description: Regulator providing analog output voltage 3.3V
 
+  port:
+    $ref: audio-graph-port.yaml#
+    unevaluatedProperties: false
+
 required:
   - compatible
   - clocks

---
base-commit: f2493655d2d3d5c6958ed996b043c821c23ae8d3
change-id: 20241019-es8328-doc-port-1065f8397e29


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

* Re: [PATCH] ASoC: dt-bindings: everest,es8328: Document audio graph port
  2024-10-19  0:43 [PATCH] ASoC: dt-bindings: everest,es8328: Document audio graph port Cristian Ciocaltea
@ 2024-10-21 19:07 ` Rob Herring (Arm)
  2024-10-22 14:07 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring (Arm) @ 2024-10-21 19:07 UTC (permalink / raw)
  To: Cristian Ciocaltea
  Cc: devicetree, Conor Dooley, Krzysztof Kozlowski, linux-sound,
	kernel, linux-kernel, David Yang, Liam Girdwood, Mark Brown,
	Heiko Stuebner


On Sat, 19 Oct 2024 03:43:14 +0300, Cristian Ciocaltea wrote:
> The ES8328/ES8388 audio codec is currently used in conjunction with
> audio-graph-card to provide an endpoint for binding with the other side
> of the audio link.
> 
> This is achieved via the 'port' property, which is not supported by the
> binding:
> 
>   rk3588s-indiedroid-nova.dtb: audio-codec@11: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
>     from schema $id: http://devicetree.org/schemas/sound/everest,es8328.yaml#
> 
> Document the missing property.
> 
> Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@collabora.com>
> ---
>  Documentation/devicetree/bindings/sound/everest,es8328.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>


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

* Re: [PATCH] ASoC: dt-bindings: everest,es8328: Document audio graph port
  2024-10-19  0:43 [PATCH] ASoC: dt-bindings: everest,es8328: Document audio graph port Cristian Ciocaltea
  2024-10-21 19:07 ` Rob Herring (Arm)
@ 2024-10-22 14:07 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Mark Brown @ 2024-10-22 14:07 UTC (permalink / raw)
  To: Liam Girdwood, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	David Yang, Cristian Ciocaltea
  Cc: Heiko Stuebner, kernel, linux-sound, devicetree, linux-kernel

On Sat, 19 Oct 2024 03:43:14 +0300, Cristian Ciocaltea wrote:
> The ES8328/ES8388 audio codec is currently used in conjunction with
> audio-graph-card to provide an endpoint for binding with the other side
> of the audio link.
> 
> This is achieved via the 'port' property, which is not supported by the
> binding:
> 
> [...]

Applied to

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

Thanks!

[1/1] ASoC: dt-bindings: everest,es8328: Document audio graph port
      commit: e6d20a9b0f376fda3e3c3709a59cefa6c0021784

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


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

end of thread, other threads:[~2024-10-22 14:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-10-19  0:43 [PATCH] ASoC: dt-bindings: everest,es8328: Document audio graph port Cristian Ciocaltea
2024-10-21 19:07 ` Rob Herring (Arm)
2024-10-22 14:07 ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®