Re: [PATCH 03/13] dt-bindings: a2b: Analog Devices AD24xx devices

[Krzysztof Kozlowski <krzk@kernel.org>]

On 21/05/2024 09:24, Alvin Šipraga wrote:

>>> +
>>> +required:
>>> +  - compatible
>>> +
>>> +unevaluatedProperties: false
>>
>> Sorry, but not. No resources, nothing here. Do not create bindings just
>> to instantiate drivers.
> 
> Do you mean that there is no need to introduce a binding for this codec
> if it has the same bindings as dai-common.yaml?

No, I said you do not have absolutely any resources here, so your
binding is empty. There is no need for such binding. You just want to
treat DT as way to instantiate drivers, which is a no-go.

> 
> Basically that is the case, but #sound-dai-cells should be <0>. Is that
> not enough?
> 
> I am OK to just drop the binding if you think so, but I would think that
> the compatible string should be somewhere in the bindings. Could you
> explain a little more what you mean?

Why do you need compatible? Which piece of hardware, with its own
resources, is being described here?

Just put dai-cells in parent node.


...


>>> +
>>> +examples:
>>> +  - |
>>> +    sync_clk: sync-clock {
>>
>> Drop, not related.
> 
> If the clock is required (as it is) then I have to reference some
> phandle in the example, 

Why?

> else the example will fail the check (missing
> required property 'clocks'). That's why I put it here. Please advise.

Let me answer indirectly: do you see any binding doing this? No. There
is almost none, so this should be a hint that it is not needed.



Best regards,
Krzysztof