Re: [PATCH net-next v2] dpll: zl3073x: Allow to use custom phase measure averaging factor

[Ivan Vecera <ivecera@redhat.com>]

On 17. 09. 25 1:26 odp., Kubalewski, Arkadiusz wrote:
>> From: Jakub Kicinski <kuba@kernel.org>
>> Sent: Tuesday, September 16, 2025 1:47 AM
>>
>> cc: Arkadiusz
>>
>> On Thu, 11 Sep 2025 09:23:01 +0200 Ivan Vecera wrote:
>>> The DPLL phase measurement block uses an exponential moving average,
>>> calculated using the following equation:
>>>
>>>                         2^N - 1                1
>>> curr_avg = prev_avg * --------- + new_val * -----
>>>                           2^N                 2^N
>>>
>>> Where curr_avg is phase offset reported by the firmware to the driver,
>>> prev_avg is previous averaged value and new_val is currently measured
>>> value for particular reference.
>>>
>>> New measurements are taken approximately 40 Hz or at the frequency of
>>> the reference (whichever is lower).
>>>
>>> The driver currently uses the averaging factor N=2 which prioritizes
>>> a fast response time to track dynamic changes in the phase. But for
>>> applications requiring a very stable and precise reading of the average
>>> phase offset, and where rapid changes are not expected, a higher factor
>>> would be appropriate.
>>>
>>> Add devlink device parameter phase_offset_avg_factor to allow a user
>>> set tune the averaging factor via devlink interface.
>>
>> Is averaging phase offset normal for DPLL devices?
>> If it is we should probably add this to the official API.
>> If it isn't we should probably default to smallest possible history?
>>
> 
> AFAIK, our phase offset measurement uses similar mechanics, but the algorithm
> is embedded in the DPLL device FW and currently not user controlled.
> Although it might happen that one day we would also provide such knob,
> if useful for users, no plans for it now.
>  From this perspective I would rather see it in dpll api, especially
> this relates to the phase measurement which is already there, the value
> being shared by multiple dpll devices seems HW related, but also seem not a
> problem, as long as a change would notify each device it relates with.

What if the averaging is implemented in different HW differently? As I
mentioned the Microchip HW uses exponential moving average but
a different HW can do it differently.

> Does frequency offset measurement for EEC DPLL would also use the same value?

Nope, this only affects phase offset measurement. AFAIK there is no such
tuning knob for FFO or frequency measurement in general...
Is it correct Prathosh?

Thanks,
Ivan