[Intel-gfx] [PATCH v4 1/2] drm/i915/display/icl: Bump up the hdisplay and vdisplay as per transcoder limits
Maarten Lankhorst
maarten.lankhorst at linux.intel.com
Fri Sep 13 13:36:39 UTC 2019
Hey,
Op 29-07-2019 om 21:17 schreef Manasi Navare:
> Hi Ville,
>
> Thanks for your review, so do we want to merge this as is or
> do we need some function to reject the 8K mode on ICL in intel_dp_mode_valid()?
>
> Manasi
I've pushed this series as-is because it blocks my bigjoiner work. We should probably reject modes in the connector specific functions if we can't handle it. :)
>
> On Fri, Jul 12, 2019 at 11:29:38PM +0300, Ville Syrjälä wrote:
>> On Fri, Jul 12, 2019 at 01:22:13PM -0700, Manasi Navare wrote:
>>> On ICL+, the vertical limits for the transcoders are increased to 8192
>>> and horizontal limits are bumped to 16K so bump up
>>> limits in intel_mode_valid()
>>>
>>> v4:
>>> * Increase the hdisplay to 16K (Ville)
>>> v3:
>>> * Supported starting ICL (Ville)
>>> * Use the higher limits from TRANS_VTOTAL register (Ville)
>>> v2:
>>> * Checkpatch warning (Manasi)
>>>
>>> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>>> Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
>>> Signed-off-by: Manasi Navare <manasi.d.navare at intel.com>
>> Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
>>
>>> ---
>>> drivers/gpu/drm/i915/display/intel_display.c | 9 +++++++--
>>> 1 file changed, 7 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
>>> index f07081815b80..15006764862b 100644
>>> --- a/drivers/gpu/drm/i915/display/intel_display.c
>>> +++ b/drivers/gpu/drm/i915/display/intel_display.c
>>> @@ -15764,8 +15764,13 @@ intel_mode_valid(struct drm_device *dev,
>>> DRM_MODE_FLAG_CLKDIV2))
>>> return MODE_BAD;
>>>
>>> - if (INTEL_GEN(dev_priv) >= 9 ||
>>> - IS_BROADWELL(dev_priv) || IS_HASWELL(dev_priv)) {
>>> + if (INTEL_GEN(dev_priv) >= 11) {
>>> + hdisplay_max = 16384;
>>> + vdisplay_max = 8192;
>>> + htotal_max = 16384;
>>> + vtotal_max = 8192;
>>> + } else if (INTEL_GEN(dev_priv) >= 9 ||
>>> + IS_BROADWELL(dev_priv) || IS_HASWELL(dev_priv)) {
>>> hdisplay_max = 8192; /* FDI max 4096 handled elsewhere */
>>> vdisplay_max = 4096;
>>> htotal_max = 8192;
>>> --
>>> 2.19.1
>> --
>> Ville Syrjälä
>> Intel
More information about the Intel-gfx
mailing list