Re: [PATCH v2] ASoC: dt-bindings: sound: atmel-at91sam9g20ek: convert to json-schema

[<Balakrishnan.S@microchip.com>]

Hi Krzysztof,

On 11/12/24 4:59 pm, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 11/12/2024 10:46, Balakrishnan Sambath wrote:
>> +properties:
>> +  compatible:
>> +    const: atmel,at91sam9g20ek-wm8731-audio
>> +
>> +  atmel,model:
>> +    $ref: /schemas/types.yaml#/definitions/string
>> +    description: The user-visible name of this sound complex.
>> +
>> +  atmel,audio-routing:
>> +    $ref: /schemas/types.yaml#/definitions/non-unique-string-array
>> +    description: A list of the connections between audio components.
>> +    items:
>> +      enum:
>> +        # Board Connectors
>> +        - Ext Spk
>> +        - Int Mic
>> +
>> +        # CODEC Pins
>> +        - LOUT
>> +        - ROUT
>> +        - LHPOUT
>> +        - RHPOUT
>> +        - LLINEIN
>> +        - RLINEIN
>> +        - MICIN
> 
> As previously questioned, I don't think you can create infinite amount
> of pairs out of it. Assuming you always connect board with codec
> (because codec-codec are already defined inside the codec), you can have
> max 14 pairs, but most likely less in practice.
I agree with your perspective.Here this binding is specific to the 
AT91SAM9G20EK board,which uses the onboard WM8731 codec for audio 
support.Since some of similar bindings do not explicitly enforce 
maxItems,I assumed this to be a valid case. However, based on my 
understanding from the schematics, 
https://ww1.microchip.com/downloads/en/DeviceDoc/doc6413.pdf, a 
"maxItems: 4" value appears valid.Are you referring to the same 
understanding here?
> 
> Best regards,
> Krzysztof

-- 
Best regards,
Balakrishnan S