[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