Re: [PATCH 5/8] drm/i915: Add mdclk_cdclk_ratio to intel_dbuf_state

2024-03-05 Thread Gustavo Sousa
Quoting Matt Roper (2024-03-04 20:25:31-03:00) >On Mon, Mar 04, 2024 at 03:30:24PM -0300, Gustavo Sousa wrote: >> CDCLK programming Xe2LPD always selects the CDCLK PLL as source for the > >I think something got a bit muddled while rewriting this sentence. >Maybe the first two words were supposed

Re: [PATCH 5/8] drm/i915: Add mdclk_cdclk_ratio to intel_dbuf_state

2024-03-04 Thread Matt Roper
On Mon, Mar 04, 2024 at 03:30:24PM -0300, Gustavo Sousa wrote: > CDCLK programming Xe2LPD always selects the CDCLK PLL as source for the I think something got a bit muddled while rewriting this sentence. Maybe the first two words were supposed to be dropped? Otherwise, Reviewed-by: Matt Roper

[PATCH 5/8] drm/i915: Add mdclk_cdclk_ratio to intel_dbuf_state

2024-03-04 Thread Gustavo Sousa
CDCLK programming Xe2LPD always selects the CDCLK PLL as source for the MDCLK. Because of that, the ratio between MDCLK and CDCLK is not be constant anymore. As such, make sure to have the current ratio available in intel_dbuf_state so that it can be used during dbuf programming. Note that we