Thread (24 messages) 24 messages, 3 authors, 2026-01-18

Re: [PATCH v2 1/7] dt-bindings: sound: sophgo: add CV1800B I2S/TDM controller binding

From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2026-01-18 16:14:10
Also in: linux-riscv, linux-sound, lkml

On 18/01/2026 13:18, Anton D. Stavinskii wrote:
quoted
Again, messed constraints.
quoted
+    items:
+      enum: [rx, tx]
No, it has to be a specific/fixed list.
Here is the question. Can you please help to understand how to describe
this properly. The idea is that TDM module is usable even without
specified dmas. Each TDM can work as clock source, only rx only tx or
both. I can force to use both channels but the user probably will want 
to not consume all the channels for if it is not needed. DMA can provide
channels for something else like SPI/I2C/UART etc. I'm asking because
I'm afraid I will do some mess again here.
If any combination is valid in hardware, then:

minItems: 1
items:
  - enum: [rx, tx]
  - const: tx

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help