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

Reply via email to