Re: [PATCH v2 4/5] clk: rockchip: pll: let rockchip_rk3588_pll_k_get update m directly

Hi Alexey,

On 7/23/26 5:04 PM, Alexey Charkov wrote:
> Selecting the next integer multiplier m is coupled to setting a negative
> fractional coefficient k. The current code checks for negative k in two
> separate places, which is error-prone.
> 
> Let rockchip_rk3588_pll_k_get update m directly, to make it the single
> source of truth for the final value of the integer multiplier m, which
> also reduces the number of scattered conditional branches in the code.
> 

Reviewed-by: Quentin Schulz <[email protected]>

Thanks!
Quentin