Re: [PATCH] accel/ivpu: Remove unused PLL_CONFIG_DEFAULT

[Lizhi Hou]

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.lawrynow...@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....@amd.com>

                wp->cdyn = enable ? PLL_CDYN_DEFAULT : 0;
                wp->epp = enable ? PLL_EPP_DEFAULT : 0;
        }