Re: [PATCH] dt-bindings: mfd: ti,am3359-tscadc: change clock-names property to enum

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Matt,

mranostay@ti.com wrote on Mon, 24 Oct 2022 03:07:06 -0700:

> Several J7 platforms use adc_tsc_fck instead of fck for clock naming. To
> avoid warnings from dt-schema on invalid clock naming; create an enum with
> both fck, and adc_tsc_fck options.

Looks like I've missed this definition when introducing the bindings.
But in practice this property looks very k3 specific and is not used
anywhere else so would it be worth considering renaming the properties
in the DT instead? There won't be any backward compatibility issue
apparently.

> Signed-off-by: Matt Ranostay <mranostay@ti.com>
> ---
>  Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> index 34bf6a01436f..fbb44531ed88 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> @@ -28,7 +28,9 @@ properties:
>      maxItems: 1
>  
>    clock-names:
> -    const: fck
> +    enum:
> +      - fck
> +      - adc_tsc_fck
>  
>    dmas:
>      items:


Thanks,
Miquèl