From: Lucas De Marchi <lucas.demarchi@intel.com>
To: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Christoph Hellwig <hch@infradead.org>,
Jani Nikula <jani.nikula@intel.com>,
intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@ffwll.ch>
Subject: Re: [Intel-gfx] [PATCH 02/30] drm/i915/display: split DISPLAY_VER 9 and 10 in intel_setup_outputs()
Date: Mon, 26 Jul 2021 07:01:53 -0700 [thread overview]
Message-ID: <20210726140153.adlmwxqdaiujfzp6@ldmartin-desk2> (raw)
In-Reply-To: <YP6MU/zzQTwWKOyD@intel.com>
On Mon, Jul 26, 2021 at 06:20:03AM -0400, Rodrigo Vivi wrote:
>On Sat, Jul 24, 2021 at 10:02:15PM -0700, Lucas De Marchi wrote:
>> On Sat, Jul 24, 2021 at 06:41:21PM +0100, Christoph Hellwig wrote:
>> > Still tests fine:
>> >
>> > Tested-by: Christoph Hellwig <hch@lst.de>
>>
>> I just pushed this to drm-intel-next as part of another series and
>> added your Tested-by.
>>
>> Rodrigo, can you pick this up for -fixes? This should go with your other
>> patch to fix the port mask, too.
>
>done.
>
>But while doing this and reviewing this series at the same time
>I got myself wondering if we shouldn't remove the PORT_F support
>entirely...
well, there is still ICL with some skus having it. I'm not sure we
actually have that sku out in the wild, but if we do, we wouldn't be
able to remove it.
Lucas De Marchi
>
>>
>> Thanks for the bug report and test.
>>
>> Lucas De Marchi
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
WARNING: multiple messages have this Message-ID (diff)
From: Lucas De Marchi <lucas.demarchi@intel.com>
To: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Christoph Hellwig <hch@infradead.org>,
Jani Nikula <jani.nikula@intel.com>,
intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@ffwll.ch>
Subject: Re: [Intel-gfx] [PATCH 02/30] drm/i915/display: split DISPLAY_VER 9 and 10 in intel_setup_outputs()
Date: Mon, 26 Jul 2021 07:01:53 -0700 [thread overview]
Message-ID: <20210726140153.adlmwxqdaiujfzp6@ldmartin-desk2> (raw)
In-Reply-To: <YP6MU/zzQTwWKOyD@intel.com>
On Mon, Jul 26, 2021 at 06:20:03AM -0400, Rodrigo Vivi wrote:
>On Sat, Jul 24, 2021 at 10:02:15PM -0700, Lucas De Marchi wrote:
>> On Sat, Jul 24, 2021 at 06:41:21PM +0100, Christoph Hellwig wrote:
>> > Still tests fine:
>> >
>> > Tested-by: Christoph Hellwig <hch@lst.de>
>>
>> I just pushed this to drm-intel-next as part of another series and
>> added your Tested-by.
>>
>> Rodrigo, can you pick this up for -fixes? This should go with your other
>> patch to fix the port mask, too.
>
>done.
>
>But while doing this and reviewing this series at the same time
>I got myself wondering if we shouldn't remove the PORT_F support
>entirely...
well, there is still ICL with some skus having it. I'm not sure we
actually have that sku out in the wild, but if we do, we wouldn't be
able to remove it.
Lucas De Marchi
>
>>
>> Thanks for the bug report and test.
>>
>> Lucas De Marchi
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2021-07-26 14:02 UTC|newest]
Thread overview: 143+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-24 0:10 [PATCH 00/30] Remove CNL support Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 0:10 ` [PATCH 01/30] drm/i915: fix not reading DSC disable fuse in GLK Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 18:03 ` Matt Roper
2021-07-24 18:03 ` [Intel-gfx] " Matt Roper
2021-07-24 0:10 ` [PATCH 02/30] drm/i915/display: split DISPLAY_VER 9 and 10 in intel_setup_outputs() Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 17:41 ` Christoph Hellwig
2021-07-25 5:02 ` Lucas De Marchi
2021-07-25 5:02 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:20 ` Rodrigo Vivi
2021-07-26 10:20 ` Rodrigo Vivi
2021-07-26 14:01 ` Lucas De Marchi [this message]
2021-07-26 14:01 ` Lucas De Marchi
2021-07-24 0:10 ` [PATCH 03/30] drm/i915/display: remove PORT_F workaround for CNL Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 18:05 ` Matt Roper
2021-07-24 18:05 ` [Intel-gfx] " Matt Roper
2021-07-24 0:10 ` [PATCH 04/30] drm/i915/display: remove explicit CNL handling from intel_cdclk.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 18:12 ` Matt Roper
2021-07-24 18:12 ` [Intel-gfx] " Matt Roper
2021-07-27 6:12 ` Lucas De Marchi
2021-07-27 6:12 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 0:10 ` [PATCH 05/30] drm/i915/display: remove explicit CNL handling from intel_color.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 18:14 ` Matt Roper
2021-07-24 18:14 ` [Intel-gfx] " Matt Roper
2021-07-24 0:10 ` [PATCH 06/30] drm/i915/display: remove explicit CNL handling from intel_combo_phy.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 18:17 ` Matt Roper
2021-07-24 18:17 ` [Intel-gfx] " Matt Roper
2021-07-24 0:10 ` [PATCH 07/30] drm/i915/display: remove explicit CNL handling from intel_crtc.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 18:17 ` Matt Roper
2021-07-24 18:17 ` [Intel-gfx] " Matt Roper
2021-07-24 0:10 ` [PATCH 08/30] drm/i915/display: remove explicit CNL handling from intel_ddi.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 11:00 ` Rodrigo Vivi
2021-07-26 11:00 ` Rodrigo Vivi
2021-07-24 0:10 ` [PATCH 09/30] drm/i915/display: remove explicit CNL handling from intel_display_debugfs.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 11:06 ` Rodrigo Vivi
2021-07-26 11:06 ` Rodrigo Vivi
2021-07-24 0:10 ` [PATCH 10/30] drm/i915/display: remove explicit CNL handling from intel_dmc.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 11:06 ` Rodrigo Vivi
2021-07-26 11:06 ` Rodrigo Vivi
2021-07-26 23:21 ` Matt Roper
2021-07-26 23:21 ` [Intel-gfx] " Matt Roper
2021-07-24 0:10 ` [PATCH 11/30] drm/i915/display: remove explicit CNL handling from intel_dp.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:41 ` Rodrigo Vivi
2021-07-26 10:41 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:10 ` [PATCH 12/30] drm/i915/display: remove explicit CNL handling from intel_dpll_mgr.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:35 ` Rodrigo Vivi
2021-07-26 10:35 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:10 ` [PATCH 13/30] drm/i915/display: remove explicit CNL handling from intel_vdsc.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 11:05 ` Rodrigo Vivi
2021-07-26 11:05 ` Rodrigo Vivi
2021-07-24 0:10 ` [PATCH 14/30] drm/i915/display: remove explicit CNL handling from skl_universal_plane.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:22 ` Rodrigo Vivi
2021-07-26 10:22 ` Rodrigo Vivi
2021-07-24 0:10 ` [PATCH 15/30] drm/i915/display: remove explicit CNL handling from intel_display_power.c Lucas De Marchi
2021-07-24 0:10 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:28 ` Rodrigo Vivi
2021-07-26 10:28 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 16/30] drm/i915/display: remove CNL ddi buf translation tables Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:56 ` Rodrigo Vivi
2021-07-26 10:56 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 17/30] drm/i915/display: rename CNL references in skl_scaler.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:11 ` Rodrigo Vivi
2021-07-26 10:11 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 18/30] drm/i915: remove explicit CNL handling from i915_irq.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:59 ` Rodrigo Vivi
2021-07-26 10:59 ` [Intel-gfx] " Rodrigo Vivi
2021-07-26 15:06 ` Lucas De Marchi
2021-07-26 15:06 ` [Intel-gfx] " Lucas De Marchi
2021-07-24 0:11 ` [PATCH 19/30] drm/i915: remove explicit CNL handling from intel_pm.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 11:08 ` Rodrigo Vivi
2021-07-26 11:08 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 20/30] drm/i915: remove explicit CNL handling from intel_mocs.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:21 ` Rodrigo Vivi
2021-07-26 10:21 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 21/30] drm/i915: remove explicit CNL handling from intel_pch.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:14 ` Rodrigo Vivi
2021-07-26 10:14 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 22/30] drm/i915: remove explicit CNL handling from intel_wopcm.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:57 ` Rodrigo Vivi
2021-07-26 10:57 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 23/30] drm/i915/gt: remove explicit CNL handling from intel_sseu.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 23:21 ` Matt Roper
2021-07-26 23:21 ` [Intel-gfx] " Matt Roper
2021-07-24 0:11 ` [PATCH 24/30] drm/i915: rename CNL references in intel_dram.c Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:22 ` Rodrigo Vivi
2021-07-26 10:22 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 25/30] drm/i915/gt: rename CNL references in intel_engine.h Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:38 ` Rodrigo Vivi
2021-07-26 10:38 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 26/30] drm/i915: finish removal of CNL Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:21 ` Rodrigo Vivi
2021-07-26 10:21 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 27/30] drm/i915: remove GRAPHICS_VER == 10 Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:08 ` Rodrigo Vivi
2021-07-26 10:08 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 28/30] drm/i915: rename/remove CNL registers Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:06 ` Rodrigo Vivi
2021-07-26 10:06 ` Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 29/30] drm/i915: replace random CNL comments Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:09 ` Rodrigo Vivi
2021-07-26 10:09 ` [Intel-gfx] " Rodrigo Vivi
2021-07-24 0:11 ` [PATCH 30/30] drm/i915: switch num_scalers/num_sprites to consider DISPLAY_VER Lucas De Marchi
2021-07-24 0:11 ` [Intel-gfx] " Lucas De Marchi
2021-07-26 10:13 ` Rodrigo Vivi
2021-07-26 10:13 ` Rodrigo Vivi
2021-07-27 6:59 ` Lucas De Marchi
2021-07-27 6:59 ` Lucas De Marchi
2021-07-24 0:40 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Remove CNL support Patchwork
2021-07-24 0:41 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-07-24 1:10 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-07-24 2:05 ` [PATCH 00/30] " Jason Ekstrand
2021-07-24 2:05 ` [Intel-gfx] " Jason Ekstrand
2021-07-24 12:37 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for " Patchwork
2021-07-26 11:07 ` Rodrigo Vivi
2021-07-26 13:59 ` Lucas De Marchi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210726140153.adlmwxqdaiujfzp6@ldmartin-desk2 \
--to=lucas.demarchi@intel.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=hch@infradead.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@intel.com \
--cc=rodrigo.vivi@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.