Re: [Intel-gfx] [PATCH v2] drm/i915: Use intel_plane_data_rate for min_cdclk calculation

2020-02-21 Thread Ville Syrjälä
On Fri, Feb 21, 2020 at 02:38:01PM +, Lisovskiy, Stanislav wrote: > On Fri, 2020-02-21 at 16:04 +0200, Ville Syrjälä wrote: > > On Thu, Feb 20, 2020 at 06:08:56PM +0200, Stanislav Lisovskiy wrote: > > > There seems to be a bit of confusing redundancy in a way, how > > > plane data rate/min

Re: [Intel-gfx] [PATCH v2] drm/i915: Use intel_plane_data_rate for min_cdclk calculation

2020-02-21 Thread Lisovskiy, Stanislav
On Fri, 2020-02-21 at 16:04 +0200, Ville Syrjälä wrote: > On Thu, Feb 20, 2020 at 06:08:56PM +0200, Stanislav Lisovskiy wrote: > > There seems to be a bit of confusing redundancy in a way, how > > plane data rate/min cdclk are calculated. > > In fact both min cdclk, pixel rate and plane data rate

Re: [Intel-gfx] [PATCH v2] drm/i915: Use intel_plane_data_rate for min_cdclk calculation

2020-02-21 Thread Ville Syrjälä
On Thu, Feb 20, 2020 at 06:08:56PM +0200, Stanislav Lisovskiy wrote: > There seems to be a bit of confusing redundancy in a way, how > plane data rate/min cdclk are calculated. > In fact both min cdclk, pixel rate and plane data rate are all > part of the same formula as per BSpec. > > However

[Intel-gfx] [PATCH v2] drm/i915: Use intel_plane_data_rate for min_cdclk calculation

2020-02-20 Thread Stanislav Lisovskiy
There seems to be a bit of confusing redundancy in a way, how plane data rate/min cdclk are calculated. In fact both min cdclk, pixel rate and plane data rate are all part of the same formula as per BSpec. However currently we have intel_plane_data_rate, which is used to calculate plane data rate