On 10/22/25 5:09 PM, Dmitry Baryshkov wrote: > On Wed, Oct 22, 2025 at 01:19:16PM +0200, Konrad Dybcio wrote: >> On 10/21/25 8:08 PM, Luca Weiss wrote: >>> The 'Agera' PLLs (with clk_agera_pll_configure) do not take some of the >>> parameters that are provided in the vendor driver. Instead the upstream >>> configuration should provide the final user_ctl value that is written to >>> the USER_CTL register. >> >> This is perhaps wishful thinking due to potential complexity, but maybe >> we could add some sanity checks to make sure that putting things in >> unused fields doesn't happen > > Should we just drop those fields and always write the register value?
They're used in other_kind_of_alpha_pll_configure.. and we have a lot of drivers using either of these approaches, so converting that and not breaking anything sounds a little difficult Konrad
