> -----Original Message-----
> From: Intel-gfx <[email protected]> On Behalf Of Mika
> Kahola
> Sent: Friday, February 13, 2026 5:56 PM
> To: [email protected]; [email protected]
> Cc: Kahola, Mika <[email protected]>
> Subject: [PATCH 10/19] drm/i915/lt_phy: Add xe3pldp .compare_hw_state
> hook
>
> Add .compare_hw_state function pointer for xe3plpd platform to support dpll
> framework.
>
> Signed-off-by: Mika Kahola <[email protected]>
> ---
> drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> index 26b78063dd94..c1d7d9909544 100644
> --- a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> +++ b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> @@ -4655,6 +4655,15 @@ static void xe3plpd_dump_hw_state(struct
> drm_printer *p,
> intel_lt_phy_dump_hw_state(p, &dpll_hw_state->ltpll); }
>
> +static bool xe3plpd_compare_hw_state(const struct intel_dpll_hw_state *_a,
> + const struct intel_dpll_hw_state *_b) {
> + const struct intel_lt_phy_pll_state *a = &_a->ltpll;
> + const struct intel_lt_phy_pll_state *b = &_b->ltpll;
> +
> + return intel_lt_phy_pll_compare_hw_state(a, b); }
The "{}" should be on their own line
Regards,
Suraj Kandpal
> +
> __maybe_unused
> static const struct intel_dpll_mgr xe3plpd_pll_mgr = {
> .dpll_info = xe3plpd_plls,
> @@ -4664,6 +4673,7 @@ static const struct intel_dpll_mgr xe3plpd_pll_mgr
> = {
> .update_active_dpll = icl_update_active_dpll,
> .update_ref_clks = icl_update_dpll_ref_clks,
> .dump_hw_state = xe3plpd_dump_hw_state,
> + .compare_hw_state = xe3plpd_compare_hw_state,
> };
>
> /**
> --
> 2.43.0
