[PATCH] accel/ivpu: Remove unused PLL_CONFIG_DEFAULT
Lizhi Hou
lizhi.hou at amd.com
Fri Aug 8 16:58:50 UTC 2025
On 8/8/25 04:10, Jacek Lawrynowicz wrote:
> This change removes the unnecessary condition, makes the code clearer,
> and silences clang-tidy warning.
>
> Signed-off-by: Jacek Lawrynowicz <jacek.lawrynowicz at linux.intel.com>
> ---
> drivers/accel/ivpu/ivpu_hw_btrs.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/accel/ivpu/ivpu_hw_btrs.c b/drivers/accel/ivpu/ivpu_hw_btrs.c
> index b236c7234daab..afdb3b2aa72a7 100644
> --- a/drivers/accel/ivpu/ivpu_hw_btrs.c
> +++ b/drivers/accel/ivpu/ivpu_hw_btrs.c
> @@ -33,7 +33,6 @@
>
> #define PLL_CDYN_DEFAULT 0x80
> #define PLL_EPP_DEFAULT 0x80
> -#define PLL_CONFIG_DEFAULT 0x0
> #define PLL_REF_CLK_FREQ 50000000ull
> #define PLL_RATIO_TO_FREQ(x) ((x) * PLL_REF_CLK_FREQ)
>
> @@ -303,7 +302,7 @@ static void prepare_wp_request(struct ivpu_device *vdev, struct wp_request *wp,
> wp->epp = 0;
> } else {
> wp->target = hw->pll.pn_ratio;
> - wp->cfg = enable ? PLL_CONFIG_DEFAULT : 0;
> + wp->cfg = 0;
Reviewed-by: Lizhi Hou <lizhi.hou at amd.com>
> wp->cdyn = enable ? PLL_CDYN_DEFAULT : 0;
> wp->epp = enable ? PLL_EPP_DEFAULT : 0;
> }
More information about the dri-devel
mailing list