✓ Fi.CI.BAT: success for Enable Wa_14019159160 and Wa_16019325821 for MTL

2024-02-23 Thread Patchwork
== Series Details == Series: Enable Wa_14019159160 and Wa_16019325821 for MTL URL : https://patchwork.freedesktop.org/series/130335/ State : success == Summary == CI Bug Log - changes from CI_DRM_14330 -> Patchwork_130335v1 Summary ---

✗ Fi.CI.SPARSE: warning for Enable Wa_14019159160 and Wa_16019325821 for MTL

2024-02-23 Thread Patchwork
== Series Details == Series: Enable Wa_14019159160 and Wa_16019325821 for MTL URL : https://patchwork.freedesktop.org/series/130335/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

✗ Fi.CI.CHECKPATCH: warning for Enable Wa_14019159160 and Wa_16019325821 for MTL

2024-02-23 Thread Patchwork
== Series Details == Series: Enable Wa_14019159160 and Wa_16019325821 for MTL URL : https://patchwork.freedesktop.org/series/130335/ State : warning == Summary == Error: dim checkpatch failed 5e1af201c616 drm/i915: Enable Wa_16019325821 7c8b332ab8fd drm/i915/guc: Add support for w/a KLVs

✓ Fi.CI.BAT: success for drm/i915/guc: Correct capture of EIR register on hang

2024-02-23 Thread Patchwork
== Series Details == Series: drm/i915/guc: Correct capture of EIR register on hang URL : https://patchwork.freedesktop.org/series/130330/ State : success == Summary == CI Bug Log - changes from CI_DRM_14330 -> Patchwork_130330v1 Summary

✗ Fi.CI.BUILD: failure for tracing/treewide: Remove second parameter of __assign_str() (rev2)

2024-02-23 Thread Patchwork
== Series Details == Series: tracing/treewide: Remove second parameter of __assign_str() (rev2) URL : https://patchwork.freedesktop.org/series/130320/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/130320/revisions/2/mbox/ not applied Applying:

✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Don't explode when the dig port we don't have an AUX CH

2024-02-23 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Don't explode when the dig port we don't have an AUX CH URL : https://patchwork.freedesktop.org/series/130329/ State : success == Summary == CI Bug Log - changes from CI_DRM_14330 -> Patchwork_130329v1

✗ Fi.CI.BAT: failure for drm/i915/guc: Simplify/extend platform check for Wa_14018913170 (rev2)

2024-02-23 Thread Patchwork
== Series Details == Series: drm/i915/guc: Simplify/extend platform check for Wa_14018913170 (rev2) URL : https://patchwork.freedesktop.org/series/130022/ State : failure == Summary == CI Bug Log - changes from CI_DRM_14330 -> Patchwork_130022v2

✓ Fi.CI.BAT: success for series starting with [1/3] drm/i915/bios: bump expected child device size (rev3)

2024-02-23 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/bios: bump expected child device size (rev3) URL : https://patchwork.freedesktop.org/series/129378/ State : success == Summary == CI Bug Log - changes from CI_DRM_14330 -> Patchwork_129378v3

✗ Fi.CI.SPARSE: warning for series starting with [1/3] drm/i915/bios: bump expected child device size (rev3)

2024-02-23 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/bios: bump expected child device size (rev3) URL : https://patchwork.freedesktop.org/series/129378/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked

Re: [PATCH v2 00/21] drm/i915: Add Display Port tunnel BW allocation support

2024-02-23 Thread Ville Syrjälä
On Tue, Feb 20, 2024 at 11:18:20PM +0200, Imre Deak wrote: > This is v2 of [1], with the following changes: > > - Several functional/typo/formatting fixes, detailed in the patches. > - Move the BW allocation from encoder hooks to > intel_atomic_commit_tail() fixing the allocation for MST

Re: [PATCH v2 11/21] drm/i915/dp: Add way to get active pipes with syncing commits

2024-02-23 Thread Ville Syrjälä
On Sat, Feb 24, 2024 at 12:09:41AM +0200, Imre Deak wrote: > On Fri, Feb 23, 2024 at 11:11:45PM +0200, Ville Syrjälä wrote: > > On Tue, Feb 20, 2024 at 11:18:31PM +0200, Imre Deak wrote: > > > Add a way to get the active pipes through a given DP port by syncing > > > against a related pending

Re: [PATCH v2 11/21] drm/i915/dp: Add way to get active pipes with syncing commits

2024-02-23 Thread Imre Deak
On Fri, Feb 23, 2024 at 11:11:45PM +0200, Ville Syrjälä wrote: > On Tue, Feb 20, 2024 at 11:18:31PM +0200, Imre Deak wrote: > > Add a way to get the active pipes through a given DP port by syncing > > against a related pending non-blocking commit. Atm > > intel_dp_get_active_pipes() will only try

Re: [PATCH v2 12/21] drm/i915/dp: Add support for DP tunnel BW allocation

2024-02-23 Thread Ville Syrjälä
On Tue, Feb 20, 2024 at 11:18:32PM +0200, Imre Deak wrote: > +static void queue_retry_work(struct intel_atomic_state *state, > + struct drm_dp_tunnel *tunnel, > + const struct intel_crtc_state *crtc_state) > +{ > + struct drm_i915_private *i915

Re: [PATCH v2 02/21] drm/dp: Add support for DP tunneling

2024-02-23 Thread Ville Syrjälä
On Tue, Feb 20, 2024 at 11:18:22PM +0200, Imre Deak wrote: > +static inline void drm_dp_tunnel_ref_put(struct drm_dp_tunnel_ref > *tunnel_ref) > +{ > + drm_dp_tunnel_put(tunnel_ref->tunnel, _ref->tracker); Should we set tunnel_ref->tunnel=NULL here? -- Ville Syrjälä Intel

Re: [PATCH v2 16/21] drm/i915/dp: Allocate/free DP tunnel BW in the encoder enable/disable hooks

2024-02-23 Thread Ville Syrjälä
On Tue, Feb 20, 2024 at 11:18:36PM +0200, Imre Deak wrote: > Allocate and free the DP tunnel BW required by a stream while > enabling/disabling the stream during a modeset. > > v2: > - Move the allocation up from encoder hooks to > intel_atomic_commit_tail(). Subject should be adjusted to

Re: [PATCH v2 11/21] drm/i915/dp: Add way to get active pipes with syncing commits

2024-02-23 Thread Ville Syrjälä
On Tue, Feb 20, 2024 at 11:18:31PM +0200, Imre Deak wrote: > Add a way to get the active pipes through a given DP port by syncing > against a related pending non-blocking commit. Atm > intel_dp_get_active_pipes() will only try to sync a given pipe and if > that would block ignore the pipe. A

[PATCH v3 0/3] Enable Wa_14019159160 and Wa_16019325821 for MTL

2024-02-23 Thread John . C . Harrison
From: John Harrison Enable Wa_14019159160 and Wa_16019325821 for MTL RCS/CCS workarounds for MTL. v2: Fix bug in WA KLV implementation (offset not being reset to start of list). Add better comment to prep patch about how KLVs can be added. Add a module parameter override and disable the w/a

[PATCH v3 2/3] drm/i915/guc: Add support for w/a KLVs

2024-02-23 Thread John . C . Harrison
From: John Harrison To prevent running out of bits, new w/a enable flags are being added via a KLV system instead of a 32 bit flags word. Signed-off-by: John Harrison Reviewed-by: Vinay Belgaumkar --- .../gpu/drm/i915/gt/uc/abi/guc_errors_abi.h | 1 +

[PATCH v3 1/3] drm/i915: Enable Wa_16019325821

2024-02-23 Thread John . C . Harrison
From: John Harrison Some platforms require holding RCS context switches until CCS is idle (the reverse w/a of Wa_14014475959). Some platforms require both versions. Signed-off-by: John Harrison Reviewed-by: Vinay Belgaumkar --- drivers/gpu/drm/i915/gt/gen8_engine_cs.c | 19

[PATCH v3 3/3] drm/i915/guc: Enable Wa_14019159160

2024-02-23 Thread John . C . Harrison
From: John Harrison Use the new w/a KLV support to enable a MTL w/a. Note, this w/a is a super-set of Wa_16019325821, so requires turning that one as well as setting the new flag for Wa_14019159160 itself. Signed-off-by: John Harrison Reviewed-by: Vinay Belgaumkar ---

Re: [FYI][PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Steven Rostedt
On Fri, 23 Feb 2024 13:46:53 -0500 Steven Rostedt wrote: > Now one thing I could do is to not remove the parameter, but just add: > > WARN_ON_ONCE((src) != __data_offsets->item##_ptr_); > > in the __assign_str() macro to make sure that it's still the same that is > assigned. But I'm not

[PATCH] drm/i915/guc: Correct capture of EIR register on hang

2024-02-23 Thread John . C . Harrison
From: John Harrison The EIR register (0x20B0) was being included in the engine class list for render and compute as the absolute register address. However, it is actually a ring register available on all engines at an offset of (base) + 0xB0. As it was included as an RCS engine but with the

[PATCH 2/2] drm/i915: Simplify aux_ch_to_digital_port()

2024-02-23 Thread Ville Syrjala
From: Ville Syrjälä Just return the correct thing from within the loop to make the code more readable. We have no ref counts/etc. to deal with here so no point in breaking from the loop just to return something. Signed-off-by: Ville Syrjälä --- .../drm/i915/display/intel_display_power_well.c

[PATCH 1/2] drm/i915: Don't explode when the dig port we don't have an AUX CH

2024-02-23 Thread Ville Syrjala
From: Ville Syrjälä The icl+ power well code currently assumes that every AUX power well maps to an encoder which is using said power well. That is by no menas guaranteed as we: - only register encoders for ports declared in the VBT - combo PHY HDMI-only encoder no longer get an AUX CH since

[PATCH v3] drm/i915/guc: Simplify/extend platform check for Wa_14018913170

2024-02-23 Thread John . C . Harrison
From: John Harrison The above w/a is required for every platform that the i915 driver supports. It is fixed on the latest platforms but they are only supported by Xe instead of i915. So just remove the platform check completely and keep the code simple. v2: Add extra comment (review feedback

Re: i915 build error on drm-misc-next

2024-02-23 Thread Abhinav Kumar
On 2/23/2024 11:35 AM, Rodrigo Vivi wrote: On Fri, Feb 23, 2024 at 09:47:11AM -0800, Abhinav Kumar wrote: CC Dmitry Hi Rodrigo On 2/23/2024 9:00 AM, Rodrigo Vivi wrote: On Fri, Feb 23, 2024 at 08:50:06AM -0700, Jeffrey Hugo wrote: With the x86_64_defconfig I see the following when

Re: [FYI][PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Steven Rostedt
On Fri, 23 Feb 2024 14:50:49 -0500 Kent Overstreet wrote: > Tangentially related though, what would make me really happy is if we > could create the string with in the TP__fast_assign() section. I have to > have a bunch of annoying wrappers right now because the string length > has to be known

Re: [PATCH 07/12] drm/i915: Use drm_printer more extensively in intel_crtc_state_dump()

2024-02-23 Thread Ville Syrjälä
On Thu, Feb 22, 2024 at 04:57:21PM -0500, Rodrigo Vivi wrote: > On Thu, Feb 15, 2024 at 06:40:50PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Convert all the direct drm_dbg_kms() stuff in intel_crtc_state_dump() > > over to drm_printf() since we now have the drm_printer around.

Re: [PATCH 06/12] drm/i915: Convert intel_dpll_dump_hw_state() to drm_printer

2024-02-23 Thread Ville Syrjälä
On Thu, Feb 22, 2024 at 04:54:07PM -0500, Rodrigo Vivi wrote: > On Thu, Feb 15, 2024 at 06:40:49PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Utilize drm_printer in pipe_config_pll_mismatch() to avoid > > a bit of code duplication. > > > > To achieve this we need to plumb the

Re: [PATCH 02/12] drm/i915: Include CRTC info in infoframe mismatch prints

2024-02-23 Thread Ville Syrjälä
On Thu, Feb 22, 2024 at 04:47:39PM -0500, Rodrigo Vivi wrote: > On Thu, Feb 15, 2024 at 06:40:45PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Most crtc state mismatches include the CRTC id+name in the > > prints. Also include it in the infoframe mismatches. > > > >

Re: [PATCH 01/12] drm/i915: Indicate which pipe failed the fastset check overall

2024-02-23 Thread Ville Syrjälä
On Thu, Feb 22, 2024 at 04:46:12PM -0500, Rodrigo Vivi wrote: > On Thu, Feb 15, 2024 at 06:40:44PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > intel_crtc_check_fastset() is done per-pipe, so it would be nice > > to know which pipe it was that failed its checkup. > > > >

Re: i915 build error on drm-misc-next

2024-02-23 Thread Rodrigo Vivi
On Fri, Feb 23, 2024 at 09:47:11AM -0800, Abhinav Kumar wrote: > CC Dmitry > > Hi Rodrigo > > On 2/23/2024 9:00 AM, Rodrigo Vivi wrote: > > On Fri, Feb 23, 2024 at 08:50:06AM -0700, Jeffrey Hugo wrote: > > > With the x86_64_defconfig I see the following when building drm-misc-next: > > > > > >

Re: [PATCH] drm/i915/guc: Add Compute context hint

2024-02-23 Thread Rodrigo Vivi
On Fri, Feb 23, 2024 at 10:31:41AM -0800, Belgaumkar, Vinay wrote: > > On 2/23/2024 12:51 AM, Tvrtko Ursulin wrote: > > > > On 22/02/2024 23:31, Belgaumkar, Vinay wrote: > > > > > > On 2/22/2024 7:32 AM, Tvrtko Ursulin wrote: > > > > > > > > On 21/02/2024 21:28, Rodrigo Vivi wrote: > > > > >

Re: [FYI][PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Steven Rostedt
On Fri, 23 Feb 2024 10:30:45 -0800 Jeff Johnson wrote: > On 2/23/2024 9:56 AM, Steven Rostedt wrote: > > From: "Steven Rostedt (Google)" > > > > [ > >This is a treewide change. I will likely re-create this patch again in > >the second week of the merge window of v6.9 and submit it

Re: [PATCH] drm/i915/guc: Add Compute context hint

2024-02-23 Thread Belgaumkar, Vinay
On 2/23/2024 12:51 AM, Tvrtko Ursulin wrote: On 22/02/2024 23:31, Belgaumkar, Vinay wrote: On 2/22/2024 7:32 AM, Tvrtko Ursulin wrote: On 21/02/2024 21:28, Rodrigo Vivi wrote: On Wed, Feb 21, 2024 at 09:42:34AM +, Tvrtko Ursulin wrote: On 21/02/2024 00:14, Vinay Belgaumkar wrote:

✗ Fi.CI.BUILD: failure for tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Patchwork
== Series Details == Series: tracing/treewide: Remove second parameter of __assign_str() URL : https://patchwork.freedesktop.org/series/130320/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/130320/revisions/1/mbox/ not applied Applying:

Re: [FYI][PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Steven Rostedt
On Fri, 23 Feb 2024 12:56:34 -0500 Steven Rostedt wrote: > Note, the same updates will need to be done for: > > __assign_str_len() > __assign_rel_str() > __assign_rel_str_len() Correction: The below macros do not pass in their source to the entry macros, so they will not need to be

✓ Fi.CI.BAT: success for drm/i915/display: Save a few bytes of memory in intel_backlight_device_register()

2024-02-23 Thread Patchwork
== Series Details == Series: drm/i915/display: Save a few bytes of memory in intel_backlight_device_register() URL : https://patchwork.freedesktop.org/series/130319/ State : success == Summary == CI Bug Log - changes from CI_DRM_14328 -> Patchwork_130319v1

Re: i915 build error on drm-misc-next

2024-02-23 Thread Abhinav Kumar
CC Dmitry Hi Rodrigo On 2/23/2024 9:00 AM, Rodrigo Vivi wrote: On Fri, Feb 23, 2024 at 08:50:06AM -0700, Jeffrey Hugo wrote: With the x86_64_defconfig I see the following when building drm-misc-next: CC drivers/gpu/drm/i915/display/intel_crt.o CC

[PATCH] drm/i915/display: Save a few bytes of memory in intel_backlight_device_register()

2024-02-23 Thread Christophe JAILLET
'name' may still be "intel_backlight" when backlight_device_register() is called. In such a case, using kstrdup_const() saves a memory duplication when dev_set_name() is called in backlight_device_register(). Use kfree_const() accordingly. Signed-off-by: Christophe JAILLET --- Compile tested

Re: [PATCH v2 2/4] ASoC: codecs: hda: Skip HDMI/DP registration if i915 is missing

2024-02-23 Thread Cezary Rojewski
On 2024-02-23 3:44 PM, Takashi Iwai wrote: On Fri, 23 Feb 2024 12:46:24 +0100, Cezary Rojewski wrote: If i915 does not support given platform but the hardware i.e.: HDAudio codec is still there, the codec-probing procedure will succeed for such device but the follow up initialization will

Re: i915 build error on drm-misc-next

2024-02-23 Thread Rodrigo Vivi
On Fri, Feb 23, 2024 at 08:50:06AM -0700, Jeffrey Hugo wrote: > With the x86_64_defconfig I see the following when building drm-misc-next: > > CC drivers/gpu/drm/i915/display/intel_crt.o > CC drivers/gpu/drm/i915/display/intel_cx0_phy.o > CC

✓ Fi.CI.IGT: success for drm/i915: Add missing ; to __assign_str() macros in tracepoint code

2024-02-23 Thread Patchwork
== Series Details == Series: drm/i915: Add missing ; to __assign_str() macros in tracepoint code URL : https://patchwork.freedesktop.org/series/130279/ State : success == Summary == CI Bug Log - changes from CI_DRM_14322_full -> Patchwork_130279v1_full

✓ Fi.CI.IGT: success for drm/i915/panelreplay: Move out psr_init_dpcd() from init_connector()

2024-02-23 Thread Patchwork
== Series Details == Series: drm/i915/panelreplay: Move out psr_init_dpcd() from init_connector() URL : https://patchwork.freedesktop.org/series/130276/ State : success == Summary == CI Bug Log - changes from CI_DRM_14322_full -> Patchwork_130276v1_full

Re: [PATCH v2 0/4] ALSA/ASoC: Conditionally skip i915 init and cleanups

2024-02-23 Thread Takashi Iwai
On Fri, 23 Feb 2024 12:46:22 +0100, Cezary Rojewski wrote: > > A small set of changes to improve initialization of the audio stack on > HDAudio devices and pair of cleanups. > > As the first change is the most important one here, following is the > technical background for it: > > Commit

Re: [PATCH v2 2/4] ASoC: codecs: hda: Skip HDMI/DP registration if i915 is missing

2024-02-23 Thread Takashi Iwai
On Fri, 23 Feb 2024 12:46:24 +0100, Cezary Rojewski wrote: > > If i915 does not support given platform but the hardware i.e.: HDAudio > codec is still there, the codec-probing procedure will succeed for such > device but the follow up initialization will always end up with -ENODEV. > > While bus

Re: [PATCH v2 02/21] drm/dp: Add support for DP tunneling

2024-02-23 Thread Imre Deak
On Fri, Feb 23, 2024 at 08:25:38AM +0200, Shankar, Uma wrote: > [...] > > diff --git a/drivers/gpu/drm/display/Kconfig > > b/drivers/gpu/drm/display/Kconfig > > index 09712b88a5b83..c0f56888c3280 100644 > > --- a/drivers/gpu/drm/display/Kconfig > > +++ b/drivers/gpu/drm/display/Kconfig > [...] >

Re: [PATCH 1/2] drm/ttm: improve idle/busy handling v4

2024-02-23 Thread Christian König
Am 06.02.24 um 13:56 schrieb Christian König: Am 06.02.24 um 13:53 schrieb Thomas Hellström: Hi, Christian, On Fri, 2024-01-26 at 15:09 +0100, Christian König wrote: Previously we would never try to move a BO into the preferred placements when it ever landed in a busy placement since those

Re: [PATCH v2 1/4] ALSA: hda: Skip i915 initialization on CNL/LKF-based platforms

2024-02-23 Thread Rodrigo Vivi
On Fri, Feb 23, 2024 at 12:46:23PM +0100, Cezary Rojewski wrote: > Commit 78f613ba1efb ("drm/i915: finish removal of CNL") and its friends > removed support for i915 for all CNL-based platforms. HDAudio library, > however, still treats such platforms as valid candidates for i915 > binding. Update

✗ Fi.CI.BAT: failure for ALSA/ASoC: Conditionally skip i915 init and cleanups (rev2)

2024-02-23 Thread Patchwork
== Series Details == Series: ALSA/ASoC: Conditionally skip i915 init and cleanups (rev2) URL : https://patchwork.freedesktop.org/series/130271/ State : failure == Summary == CI Bug Log - changes from CI_DRM_14325 -> Patchwork_130271v2

[PATCH v2 4/4] ALSA: hda: Reuse for_each_pcm_streams()

2024-02-23 Thread Cezary Rojewski
Use the macro to improve readability. Signed-off-by: Cezary Rojewski --- sound/pci/hda/hda_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c index 12f02cdc9659..2cac337f5263 100644 --- a/sound/pci/hda/hda_codec.c

[PATCH v2 3/4] ASoC: codecs: hda: Cleanup error messages

2024-02-23 Thread Cezary Rojewski
Be cohesive and use same pattern in each error message. Signed-off-by: Cezary Rojewski --- sound/soc/codecs/hda.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/sound/soc/codecs/hda.c b/sound/soc/codecs/hda.c index d9e7cd8aada2..8a9d0674555e 100644 ---

[PATCH v2 2/4] ASoC: codecs: hda: Skip HDMI/DP registration if i915 is missing

2024-02-23 Thread Cezary Rojewski
If i915 does not support given platform but the hardware i.e.: HDAudio codec is still there, the codec-probing procedure will succeed for such device but the follow up initialization will always end up with -ENODEV. While bus could filter out address '2' which Intel's HDMI/DP codecs always

[PATCH v2 1/4] ALSA: hda: Skip i915 initialization on CNL/LKF-based platforms

2024-02-23 Thread Cezary Rojewski
Commit 78f613ba1efb ("drm/i915: finish removal of CNL") and its friends removed support for i915 for all CNL-based platforms. HDAudio library, however, still treats such platforms as valid candidates for i915 binding. Update query mechanism to reflect changes made in drm tree. At the same time,

[PATCH v2 0/4] ALSA/ASoC: Conditionally skip i915 init and cleanups

2024-02-23 Thread Cezary Rojewski
A small set of changes to improve initialization of the audio stack on HDAudio devices and pair of cleanups. As the first change is the most important one here, following is the technical background for it: Commit 78f613ba1efb ("drm/i915: finish removal of CNL") and its friends removed support

✗ Fi.CI.IGT: failure for LNL display

2024-02-23 Thread Patchwork
== Series Details == Series: LNL display URL : https://patchwork.freedesktop.org/series/130255/ State : failure == Summary == CI Bug Log - changes from CI_DRM_14320_full -> Patchwork_130255v1_full Summary --- **FAILURE** Serious

✓ Fi.CI.BAT: success for Enable Adaptive Sync SDP Support for DP (rev10)

2024-02-23 Thread Patchwork
== Series Details == Series: Enable Adaptive Sync SDP Support for DP (rev10) URL : https://patchwork.freedesktop.org/series/126829/ State : success == Summary == CI Bug Log - changes from CI_DRM_14319 -> Patchwork_126829v10 Summary ---

RE: [PATCH v2 21/21] drm/i915/dp: Enable DP tunnel BW allocation mode

2024-02-23 Thread Shankar, Uma
> -Original Message- > From: dri-devel On Behalf Of Imre > Deak > Sent: Wednesday, February 21, 2024 2:49 AM > To: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Subject: [PATCH v2 21/21] drm/i915/dp: Enable DP tunnel BW allocation mode > > Detect DP tunnels and

RE: [PATCH v2 20/21] drm/i915/dp: Read DPRX for all long HPD pulses

2024-02-23 Thread Shankar, Uma
> -Original Message- > From: Intel-gfx On Behalf Of Imre > Deak > Sent: Wednesday, February 21, 2024 2:49 AM > To: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Subject: [PATCH v2 20/21] drm/i915/dp: Read DPRX for all long HPD pulses > > The TBT DP tunnel BW

Re: [PATCH 1/4] ALSA: hda: Skip i915 initialization on CNL/LKF-based platforms

2024-02-23 Thread Cezary Rojewski
On 2024-02-23 9:47 AM, Takashi Iwai wrote: On Thu, 22 Feb 2024 21:54:42 +0100, Rodrigo Vivi wrote: ... @@ -127,15 +128,26 @@ static int i915_component_master_match(struct device *dev, int subcomponent, /* check whether Intel graphics is present and reachable */ static int

RE: [PATCH v2 19/21] drm/i915/dp: Suspend/resume DP tunnels

2024-02-23 Thread Shankar, Uma
> -Original Message- > From: Intel-gfx On Behalf Of Imre > Deak > Sent: Wednesday, February 21, 2024 2:49 AM > To: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Subject: [PATCH v2 19/21] drm/i915/dp: Suspend/resume DP tunnels > > Suspend and resume DP tunnels

RE: [PATCH v2 17/21] drm/i915/dp: Handle DP tunnel IRQs

2024-02-23 Thread Shankar, Uma
> -Original Message- > From: Intel-gfx On Behalf Of Imre > Deak > Sent: Wednesday, February 21, 2024 2:49 AM > To: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Subject: [PATCH v2 17/21] drm/i915/dp: Handle DP tunnel IRQs > > Handle DP tunnel IRQs a sink (or

✓ Fi.CI.BAT: success for HDCP MST Type1 fixes (rev5)

2024-02-23 Thread Patchwork
== Series Details == Series: HDCP MST Type1 fixes (rev5) URL : https://patchwork.freedesktop.org/series/129925/ State : success == Summary == CI Bug Log - changes from CI_DRM_14324 -> Patchwork_129925v5 Summary --- **SUCCESS** No

RE: [PATCH v2 13/21] drm/i915/dp: Add DP tunnel atomic state and check BW limit

2024-02-23 Thread Shankar, Uma
> -Original Message- > From: dri-devel On Behalf Of Imre > Deak > Sent: Wednesday, February 21, 2024 2:49 AM > To: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Cc: Ville Syrjälä > Subject: [PATCH v2 13/21] drm/i915/dp: Add DP tunnel atomic state and check > BW

✗ Fi.CI.SPARSE: warning for HDCP MST Type1 fixes (rev5)

2024-02-23 Thread Patchwork
== Series Details == Series: HDCP MST Type1 fixes (rev5) URL : https://patchwork.freedesktop.org/series/129925/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately. +./arch/x86/include/asm/bitops.h:116:1:

Re: [PATCH] drm/i915/guc: Add Compute context hint

2024-02-23 Thread Tvrtko Ursulin
On 22/02/2024 23:31, Belgaumkar, Vinay wrote: On 2/22/2024 7:32 AM, Tvrtko Ursulin wrote: On 21/02/2024 21:28, Rodrigo Vivi wrote: On Wed, Feb 21, 2024 at 09:42:34AM +, Tvrtko Ursulin wrote: On 21/02/2024 00:14, Vinay Belgaumkar wrote: Allow user to provide a context hint. When this

Re: [PATCH 1/4] ALSA: hda: Skip i915 initialization on CNL/LKF-based platforms

2024-02-23 Thread Takashi Iwai
On Thu, 22 Feb 2024 21:54:42 +0100, Rodrigo Vivi wrote: > > On Thu, Feb 22, 2024 at 06:53:12PM +0100, Cezary Rojewski wrote: > > On 2024-02-22 6:24 PM, Ville Syrjälä wrote: > > > On Thu, Feb 22, 2024 at 06:06:11PM +0100, Cezary Rojewski wrote: > > > > Commit 78f613ba1efb ("drm/i915: finish

[PATCH 12/13] drm/i915/hdcp: Allocate stream id after HDCP AKE stage

2024-02-23 Thread Suraj Kandpal
Allocate stream id after HDCP AKE stage and not before so that it can also be done during link integrity check. Right now for MST scenarios LIC fails after hdcp enablement for this reason. --v2 -no need for else block in prepare_streams function [Ankit] --v3 -remove intel_hdcp argument from

[PATCH 07/13] drm/i915/hdcp: HDCP Capability for the downstream device

2024-02-23 Thread Suraj Kandpal
Currently we are only checking capability of remote device and not immediate downstream device but during capability check we need are concerned with only the HDCP capability of downstream device. During i915_display_info reporting we need HDCP Capability for both the monitors and downstream

[PATCH 11/13] drm/i915/hdcp: Don't enable HDCP1.4 directly from check_link

2024-02-23 Thread Suraj Kandpal
Whenever LIC fails instead of moving from ENABLED to DESIRED CP property we directly enable HDCP1.4 without informing the userspace of this failure in link integrity check. Now we will just update the value to DESIRED send the event to userspace and then continue with the normal flow of HDCP

[PATCH 13/13] drm/i915/hdcp: Read Rxcaps for robustibility

2024-02-23 Thread Suraj Kandpal
We see some monitors and docks report incorrect hdcp version and capability in first few reads so we read rx_caps three times before we conclude the monitor's or docks HDCP capability --v2 -Add comment to justify the 3 time read loop for hdcp capability[Ankit] Signed-off-by: Suraj Kandpal

[PATCH 09/13] drm/i915/hdcp: Extract hdcp structure from correct connector

2024-02-23 Thread Suraj Kandpal
Currently intel_hdcp is not being extracted from primary connector this patch fixes that. Fixes: 524240b231ea ("drm/i915/hdcp: Propagate aux info in DP HDCP functions") Signed-off-by: Suraj Kandpal Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 7 +-- 1 file

[PATCH 10/13] drm/i915/hdcp: Don't enable HDCP2.2 directly from check_link

2024-02-23 Thread Suraj Kandpal
Whenever LIC fails instead of moving from ENABLED to DESIRED CP property we directly enable HDCP2.2 without informing the userspace of this failure in link integrity check. Now we will just update the value to DESIRED send the event to userspace and then continue with the normal flow of HDCP

[PATCH 08/13] drm/i915/hdcp: Remove additional timing for reading mst hdcp message

2024-02-23 Thread Suraj Kandpal
Now that we have moved back to direct reads the additional timing is not required hence this can be removed. --v2 -Add Fixes tag [Ankit] Fixes: 3974f9c17bb9 ("drm/i915/hdcp: Adjust timeout for read in DPMST Scenario") Signed-off-by: Suraj Kandpal Reviewed-by: Ankit Nautiyal ---

[PATCH 05/13] drm/i915/hdcp: Rename hdcp capable functions

2024-02-23 Thread Suraj Kandpal
Rename hdcp_capable and hdcp_2_2_capable to hdcp_get_capability and hdcp_2_2_get_capability to properly reflect what these functions are doing. Signed-off-by: Suraj Kandpal Reviewed-by: Ankit Nautiyal --- .../drm/i915/display/intel_display_debugfs.c | 4 ++--

[PATCH 02/13] drm/i915/hdcp: Move source hdcp2 checks into its own function

2024-02-23 Thread Suraj Kandpal
Move checks on the source side for HDCP2.2 into its own function so that they can be used in the HDCP remote capability check function. Signed-off-by: Suraj Kandpal Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_hdcp.c | 20 +--- 1 file changed, 17

[PATCH 06/13] drm/i915/hdcp: Add new remote capability check shim function

2024-02-23 Thread Suraj Kandpal
Create a remote HDCP capability shim function which can read the remote monitor HDCP capability when in MST configuration. --v2 -Add an assertion to make sure only mst encoder call this remote_cap function [Ankit] --v3 -rename remote_hdcp_cap to remote_hdcp_capability [Jani] Signed-off-by:

[PATCH 04/13] drm/i915/hdcp: Pass drm_dp_aux to read_bcaps function

2024-02-23 Thread Suraj Kandpal
Pass drm_dp_aux to intel_dp_hdcp_read_bcaps function so as to aid in reading the bcaps for the remote monitor later on. Signed-off-by: Suraj Kandpal Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-)

[PATCH 00/13] HDCP MST Type1 fixes

2024-02-23 Thread Suraj Kandpal
We were seeing a blank screen whenever Type1 content was played. This was due to extra timing which was taken as we had moved to remote read and writes previously for MST scenario, which in turn was done as we were not able to do direct read and writes to the immediate downstream device. The

[PATCH 03/13] drm/i915/hdcp: Refactor intel_dp_hdcp2_capable

2024-02-23 Thread Suraj Kandpal
Break intel_dp_hdcp2_capable so that the common the code can be reused for the remote capability check. Signed-off-by: Suraj Kandpal Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git

[PATCH 01/13] drm/i915/hdcp: Move to direct reads for HDCP

2024-02-23 Thread Suraj Kandpal
Even for MST scenarios we need to do direct reads only on the immediate downstream device the rest of the authentication is taken care by that device. Remote reads will only be used to check capability of the monitors in MST topology. --v2 -Add fixes tag [Ankit] -Derive aux where needed rather

RE: [PATCH v2 11/21] drm/i915/dp: Add way to get active pipes with syncing commits

2024-02-23 Thread Shankar, Uma
> -Original Message- > From: dri-devel On Behalf Of Imre > Deak > Sent: Wednesday, February 21, 2024 2:49 AM > To: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Cc: Ville Syrjälä > Subject: [PATCH v2 11/21] drm/i915/dp: Add way to get active pipes with > syncing >