Re: [PATCH v11 01/11] dt-bindings: iio: frequency: add adf41513

[Rodrigo Alencar]

On 26/05/06 03:08PM, Rodrigo Alencar via B4 Relay wrote:
> From: Rodrigo Alencar <rodrigo.alencar@xxxxxxxxxx>
> 
> DT-bindings for ADF41513, an ultralow noise PLL frequency synthesizer that
> can be used to implement local oscillators (LOs) as high as 26.5 GHz.
> Some properties are based upon an existing PLL device properties
> (e.g. ADF4350).

...
 
> +properties:
> +  compatible:
> +    enum:
> +      - adi,adf41510
> +      - adi,adf41513

...

> +  adi,power-up-frequency-mhz:
> +    minimum: 1000
> +    maximum: 26500
> +    default: 10000
> +    description:
> +      The PLL tunes to this frequency during the initialization sequence.
> +      This property should be set to a frequency supported by the loop filter
> +      and VCO used in the design. Range is 1 GHz to 26.5 GHz for ADF41513,
> +      and 1 GHz to 10 GHz for ADF41510.

Sashiko's feedback:

https://sashiko.dev/#/patchset/20260506-adf41513-iio-driver-v11-0-2b7e99cfe8f2%40analog.com?part=1

	Does the schema allow an ADF41510 device to be configured with frequencies
	exceeding its physical hardware capabilities?
	The global maximum for adi,power-up-frequency-mhz is set to 26500, but the
	description notes that the ADF41510 variant only supports up to 10 GHz.
	Should there be an allOf conditional block that restricts the maximum value
	to 10000 when the adi,adf41510 compatible string is used?

That makes sense and will address this in a v12.

-- 
Kind regards,

Rodrigo Alencar