> -Original Message-
> From: Hogander, Jouni
> Sent: Tuesday, April 9, 2024 2:28 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Manna, Animesh ; Hogander, Jouni
>
> Subject: [PATCH] drm/i915/psr: Use crtc_state->port_clock instead of
> intel_dp->link_rate
>
> Intel_dp->link_rate is
On Wed, 10 Apr 2024 04:42:46 -0700, Ville Syrjälä wrote:
>
> On Tue, Apr 09, 2024 at 09:28:55PM -0700, Ashutosh Dixit wrote:
> > There are no hwmon selftests so there is no need to enable hwmon for
> > selftests. So enable hwmon only for real driver load.
> >
> > Closes:
On Wed, 10 Apr 2024 06:53:15 -0700, Andi Shyti wrote:
>
Hi Andi,
> please use "git format-patch -v 3 ..." which generates subject
> [PATCH v3] ...". Otherwise it gets confusing to see the patch
> that needs to be reviewed.
Sure, sorry!
>
> On Tue, Apr 09, 2024 at 11:05:49PM -0700, Ashutosh
== Series Details ==
Series: drm/i915/dg2: wait for HuC load completion before running selftests
URL : https://patchwork.freedesktop.org/series/132300/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14561 -> Patchwork_132300v1
On Wed, Apr 10, 2024 at 08:28:40PM +0200, Janusz Krzysztofik wrote:
Hi Lucas,
On Friday, 5 April 2024 19:54:53 CEST Janusz Krzysztofik wrote:
KUnit can provide KTAP reports from test modules via debugfs files, one
per test suite. Using that source of test results instead of extracting
them
On Wed, Apr 10, 2024 at 01:05:07PM +0300, Jani Nikula wrote:
We've accumulated enough Intel specific header files under include/drm
that they warrant a subdirectory of their own. Clean up the top drm
header directory by moving the Intel files under include/drm/intel.
Since i915 is most
== Series Details ==
Series: drm/i915: Don't enable hwmon for selftests (rev4)
URL : https://patchwork.freedesktop.org/series/132243/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14561 -> Patchwork_132243v4
Summary
Hi Dave and Sima,
Here goes drm-intel-fixes-2024-04-10:
Display fixes:
- Couple CDCLK programming fixes (Ville)
- HDCP related fix (Suraj)
- 4 Bigjoiner related fixes (Ville)
Core fix:
- Fix for a circular locking around GuC on reset+wedged case (John)
Thanks,
Rodrigo.
The following changes
== Series Details ==
Series: drm/i915: Don't enable hwmon for selftests (rev4)
URL : https://patchwork.freedesktop.org/series/132243/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/edid: Parse topology block for all DispID structure v1.x
URL : https://patchwork.freedesktop.org/series/132292/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14561 -> Patchwork_132292v1
On DG2, submissions to VCS engines tied to a gem context are blocked
until the HuC is loaded. Since some selftests do use a gem context,
wait for the HuC load to complete before running the tests to avoid
contamination.
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10564
== Series Details ==
Series: drm/i915/pps: pass display to PPS register macros (rev2)
URL : https://patchwork.freedesktop.org/series/132286/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14560 -> Patchwork_132286v2
Summary
There are no hwmon selftests so there is no need to enable hwmon for
selftests. So enable hwmon only for real driver load.
v2: Move the logic inside i915_hwmon.c
v3: Move is_i915_selftest definition to i915_selftest.h (Badal)
v4: s/is_i915_selftest/is_i915_selftest_module/
Closes:
== Series Details ==
Series: drm/i915/pps: pass display to PPS register macros (rev2)
URL : https://patchwork.freedesktop.org/series/132286/
State : warning
== Summary ==
Error: dim checkpatch failed
760f2ae26bc9 drm/i915/pps: pass display to PPS register macros
-:57: WARNING:LONG_LINE: line
== Series Details ==
Series: drm/i915/dsi: stop relying on implicit dev_priv variable
URL : https://patchwork.freedesktop.org/series/132285/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14559 -> Patchwork_132285v1
Summary
== Series Details ==
Series: drm/i915/dsi: stop relying on implicit dev_priv variable
URL : https://patchwork.freedesktop.org/series/132285/
State : warning
== Summary ==
Error: dim checkpatch failed
02b70fe522db drm/i915/dsi: remove unused _MIPIA_AUTOPWG register definition
1b83a12b11bd
Hi Lucas,
On Friday, 5 April 2024 19:54:53 CEST Janusz Krzysztofik wrote:
> KUnit can provide KTAP reports from test modules via debugfs files, one
> per test suite. Using that source of test results instead of extracting
> them from dmesg, where they may be interleaved with other kernel
From: Ville Syrjälä
DisplayID spec v1.3 revision history notes do claim that
the toplogy block was added in v1.3 so requiring structure
v1.2 would seem correct, but there is at least one EDID in
edid.tv with a topology block and structure v1.0. And
there are also EDIDs with DisplayID structure
tree/branch:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
branch HEAD: 6ebf211bb11dfc004a2ff73a9de5386fa309c430 Add linux-next specific
files for 20240410
Error/Warning reports:
https://lore.kernel.org/oe-kbuild-all/202404102353.cv1gujk3-...@intel.com
Error
On Wed, 10 Apr 2024, Jani Nikula wrote:
> Pass struct intel_display pointer to PPS register macros intead of
> implicitly relying on the dev_priv local variable.
>
> Signed-off-by: Jani Nikula
Whoops, accidentally sent the same patch twice, with slightly different
commit messages.
--
Jani
Pass struct intel_display pointer to PPS register macros intead of
implicitly relying on the dev_priv local variable.
Signed-off-by: Jani Nikula
---
Mostly see how this feels. Might be better to wait for [1] to land so we
could more easily pass display pointer everywhere. I don't really want
Pass struct intel_display pointer to PPS register macros intead of
implicitly relying on the dev_priv local variable.
Signed-off-by: Jani Nikula
---
Mostly see how this feels. Might be better to wait for [1] to land so we
could more easily pass display pointer everywhere.
[1]
Stop relying on the dev_priv local variable in the DSI register
macros. Pass i915 pointer to the macros. Move the MIPI DSI MMIO base
selection to a different level, passing it to _MMIO_MIPI() and doing the
addition there.
Signed-off-by: Jani Nikula
---
Tip: Applying the patch and using 'git
All the BXT specific macros have BXT_ prefix, do the same for VLV for
consistency.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/vlv_dsi.c | 6 +++---
drivers/gpu/drm/i915/display/vlv_dsi_regs.h | 4 ++--
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git
There are other unused registers, but this is also unusable and
inadequate. Remove.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/vlv_dsi_regs.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/vlv_dsi_regs.h
Rip the band-aid, it's not so bad after all. Well, at least for the
somewhat isolated VLV DSI. But it's a start.
Jani Nikula (3):
drm/i915/dsi: remove unused _MIPIA_AUTOPWG register definition
drm/i915/dsi: add VLV_ prefix to VLV only register macros
drm/i915/dsi: pass i915 to register
On Mon, Jan 29, 2024 at 06:42:39PM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/dp: Add jitter WAs for MST/FEC/DSC links
> URL : https://patchwork.freedesktop.org/series/129277/
> State : success
Patchset is pushed to drm-intel-next, fixing the WA code comment in
patch
On Fri, Apr 05, 2024 at 12:21:59PM +0530, Mitul Golani wrote:
> Correct struct member name to 'mode' instead of 'operation mode'
> in 'drm_dp_as_sdp' structure description.
>
> Fixes: 0bbb8f594e33 ("drm/dp: Add Adaptive Sync SDP logging")
Probably good to avoid this 'Fixes:' tag, and only use
== Series Details ==
Series: series starting with [1/2] drm/print: drop include debugfs.h and
include where needed
URL : https://patchwork.freedesktop.org/series/132280/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14557 -> Patchwork_132280v1
On Wed, Apr 10, 2024 at 05:34:13PM +0300, Ville Syrjälä wrote:
> On Tue, Apr 09, 2024 at 06:12:09PM -, Patchwork wrote:
> > == Series Details ==
> >
> > Series: drm/i915: Bigjoiner modeset sequence redesign and MST support (rev4)
> > URL : https://patchwork.freedesktop.org/series/131797/
>
== Series Details ==
Series: series starting with [1/2] drm/print: drop include debugfs.h and
include where needed
URL : https://patchwork.freedesktop.org/series/132280/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked
== Series Details ==
Series: drm/i915: Implement Audio WA_14020863754 (rev2)
URL : https://patchwork.freedesktop.org/series/132255/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14557 -> Patchwork_132255v2
Summary
---
Hi Dave, Sima,
Still low amount of patches this week!
Cheers,
~Maarten
drm-misc-next-2024-04-10:
drm-misc-next for v6.10:
Cross-subsystem Changes:
- Add Tomi as Xilinx maintainer.
- Add sound bindings to DT.
Core Changes:
- Make DP helper depend on KMS helper.
Driver Changes:
- Assorted
On 10.04.2024 16:14, Jani Nikula wrote:
Surprisingly many places depend on debugfs.h to be included via
drm_print.h. Fix them.
Signed-off-by: Jani Nikula
---
Reviewed-by: Andrzej Hajda
Regards
Andrzej
Cc: Andrzej Hajda
Cc: Neil Armstrong
Cc: Robert Foss
Cc: Laurent Pinchart
Cc:
On Tue, Apr 09, 2024 at 06:12:09PM -, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: Bigjoiner modeset sequence redesign and MST support (rev4)
> URL : https://patchwork.freedesktop.org/series/131797/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from
On Wed, Apr 10, 2024 at 01:05:09PM +0300, Jani Nikula wrote:
> Clean up the top level include/drm directory by grouping all the Intel
> specific files under a common subdirectory.
>
> Cc: Daniel Vetter
> Cc: Dave Airlie
> Cc: Lucas De Marchi
> Signed-off-by: Jani Nikula
Reviewed-by: Andi
Never include where a forward declaration will suffice.
Signed-off-by: Jani Nikula
---
include/drm/drm_print.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index 561c3b96b6fd..089950ad8681 100644
---
Surprisingly many places depend on debugfs.h to be included via
drm_print.h. Fix them.
Signed-off-by: Jani Nikula
---
Cc: Andrzej Hajda
Cc: Neil Armstrong
Cc: Robert Foss
Cc: Laurent Pinchart
Cc: Jonas Karlman
Cc: Jernej Skrabec
Cc: Maarten Lankhorst
Cc: Maxime Ripard
Cc: Thomas
Hi Jani,
On Wed, Apr 10, 2024 at 01:05:08PM +0300, Jani Nikula wrote:
> Use <> instead of "" for including headers from include/, even if the
> file is in the same directory.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Andi Shyti
Thanks,
Andi
On Wed, 10 Apr 2024 13:08:57 +0200
Sebastian Andrzej Siewior wrote:
> On 2024-04-09 11:55:33 [-0400], Steven Rostedt wrote:
> > I believe you need to do it in the .c file:
> >
> > Can you try something like this?
> >
> …
> > ?
>
> I tried and nothing changed because the lowlevel config
Hi Ashutosh,
please use "git format-patch -v 3 ..." which generates subject
[PATCH v3] ...". Otherwise it gets confusing to see the patch
that needs to be reviewed.
On Tue, Apr 09, 2024 at 11:05:49PM -0700, Ashutosh Dixit wrote:
> There are no hwmon selftests so there is no need to enable hwmon
WA_14020863754: Corner case with Min Hblank Fix can cause
audio hang
Issue: Previously a fix was made to avoid issues with extremely
small hblanks, called the "Min Hblank Fix". However, this can
potentially cause an audio hang.
Workaround :
During "Audio Programming Sequence" Audio Enabling -
> -Original Message-
> From: Jani Nikula
> Sent: Wednesday, April 10, 2024 3:47 PM
> To: Shankar, Uma ; intel-gfx@lists.freedesktop.org;
> intel...@lists.freedesktop.org
> Cc: Borah, Chaitanya Kumar ; Shankar, Uma
>
> Subject: Re: [PATCH] drm/i915: Implement Audio WA_14020863754
>
>
On Wed, Apr 10, 2024 at 07:41:42AM +, Hogander, Jouni wrote:
> On Wed, 2024-04-10 at 07:35 +, Manna, Animesh wrote:
> >
> >
> > > -Original Message-
> > > From: Hogander, Jouni
> > > Sent: Wednesday, April 10, 2024 12:54 PM
> > > To: Manna, Animesh ; intel-
> > >
== Series Details ==
Series: drm/i915: PREEMPT_RT related fixups. (rev6)
URL : https://patchwork.freedesktop.org/series/95463/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14554 -> Patchwork_95463v6
Summary
---
On Tue, 02 Apr 2024, Jani Nikula wrote:
> Ensure drm headers build, are self-contained, have header guards, and
> have no kernel-doc warnings, when CONFIG_DRM_HEADER_TEST=y.
>
> The mechanism follows similar patters used in i915, xe, and usr/include.
>
> To cover include/drm, we need to recurse
On Tue, Apr 09, 2024 at 09:28:55PM -0700, Ashutosh Dixit wrote:
> There are no hwmon selftests so there is no need to enable hwmon for
> selftests. So enable hwmon only for real driver load.
>
> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10366
> Signed-off-by: Ashutosh Dixit
Why
On Mon, 08 Apr 2024, Ville Syrjälä wrote:
> On Mon, Apr 08, 2024 at 12:43:57PM +0300, Jani Nikula wrote:
>> Continue with placing debugfs next to the implementation.
>>
>> Signed-off-by: Jani Nikula
>> ---
>> .../drm/i915/display/intel_display_debugfs.c | 27 ++--
>>
== Series Details ==
Series: drm/i915: PREEMPT_RT related fixups. (rev6)
URL : https://patchwork.freedesktop.org/series/95463/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915: PREEMPT_RT related fixups. (rev6)
URL : https://patchwork.freedesktop.org/series/95463/
State : warning
== Summary ==
Error: dim checkpatch failed
dc8b0da9b41f drm/i915: Use preempt_disable/enable_rt() where recommended
-:7: WARNING:COMMIT_LOG_LONG_LINE:
== Series Details ==
Series: drm: move Intel drm headers to a subdirectory (rev2)
URL : https://patchwork.freedesktop.org/series/132264/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14554 -> Patchwork_132264v2
Summary
On 2024-04-09 11:55:33 [-0400], Steven Rostedt wrote:
> I believe you need to do it in the .c file:
>
> Can you try something like this?
>
…
> ?
I tried and nothing changed because the lowlevel config option isn't the
problem. What about I drop this and replace 4/10 from this series with
the
== Series Details ==
Series: drm: move Intel drm headers to a subdirectory (rev2)
URL : https://patchwork.freedesktop.org/series/132264/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm: move Intel drm headers to a subdirectory (rev2)
URL : https://patchwork.freedesktop.org/series/132264/
State : warning
== Summary ==
Error: dim checkpatch failed
0a89e58276c7 drm/i915: use system include for drm headers
0ef68bf2213b drm: move intel-gtt.h
== Series Details ==
Series: drm: move Intel drm headers to a subdirectory
URL : https://patchwork.freedesktop.org/series/132264/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14554 -> Patchwork_132264v1
Summary
---
== Series Details ==
Series: drm: move Intel drm headers to a subdirectory
URL : https://patchwork.freedesktop.org/series/132264/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm: move Intel drm headers to a subdirectory
URL : https://patchwork.freedesktop.org/series/132264/
State : warning
== Summary ==
Error: dim checkpatch failed
8c970e29f651 drm/i915: use system include for drm headers
81251adf0df4 drm: move intel-gtt.h under
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
v2: Also fix comment in intel_pci_config.h (Ilpo)
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Bjorn Helgaas
Cc: Hans de Goede
Cc: Ilpo Järvinen
Signed-off-by:
On Wed, 10 Apr 2024, Ilpo Järvinen wrote:
> Could you please also address the comment in
> drivers/gpu/drm/i915/intel_pci_config.h.
Ack.
--
Jani Nikula, Intel
On Wed, 10 Apr 2024, Jani Nikula wrote:
> Clean up the top level include/drm directory by grouping all the Intel
> specific files under a common subdirectory.
>
> Cc: Daniel Vetter
> Cc: Dave Airlie
> Cc: Lucas De Marchi
> Cc: Bjorn Helgaas
> Cc: Hans de Goede
> Cc: Ilpo Järvinen
>
On Wed, 10 Apr 2024, Uma Shankar wrote:
> WA_14020863754: Corner case with Min Hblank Fix can cause
> audio hang
>
> Issue: Previously a fix was made to avoid issues with extremely
> small hblanks, called the "Min Hblank Fix". However, this can
> potentially cause an audio hang.
>
> Workaround :
On Wed, 10 Apr 2024, Patchwork wrote:
> == Series Details ==
>
> Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
> URL : https://patchwork.freedesktop.org/series/131414/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_14552 ->
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/xe/xe_pci.c | 2 +-
include/drm/{ => intel}/xe_pciids.h | 0
2 files
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Tomas Winkler
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display_types.h| 2 +-
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Bjorn Helgaas
Cc: Hans de Goede
Cc: Ilpo Järvinen
Signed-off-by: Jani Nikula
---
arch/x86/kernel/early-quirks.c
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Bjorn Helgaas
Signed-off-by: Jani Nikula
---
arch/x86/kernel/early-quirks.c | 2 +-
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Tomas Winkler
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/pxp/intel_pxp_tee.c | 2 +-
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_hdmi.c | 2 +-
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Tomas Winkler
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Signed-off-by: Jani Nikula
---
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Cc: Tomas Winkler
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/gt/uc/intel_gsc_proxy.c | 2 +-
Clean up the top level include/drm directory by grouping all the Intel
specific files under a common subdirectory.
Cc: Daniel Vetter
Cc: Dave Airlie
Cc: Lucas De Marchi
Signed-off-by: Jani Nikula
---
drivers/char/agp/intel-agp.c | 2 +-
drivers/char/agp/intel-gtt.c
Use <> instead of "" for including headers from include/, even if the
file is in the same directory.
Signed-off-by: Jani Nikula
---
include/drm/i915_component.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h
index
We've accumulated enough Intel specific header files under include/drm
that they warrant a subdirectory of their own. Clean up the top drm
header directory by moving the Intel files under include/drm/intel.
Since i915 is most impacted, I suggest merging the lot via
drm-intel-next. Please ack if
== Series Details ==
Series: drm/i915: Implement Audio WA_14020863754
URL : https://patchwork.freedesktop.org/series/132255/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14552 -> Patchwork_132255v1
Summary
---
WA_14020863754: Corner case with Min Hblank Fix can cause
audio hang
Issue: Previously a fix was made to avoid issues with extremely
small hblanks, called the "Min Hblank Fix". However, this can
potentially cause an audio hang.
Workaround :
During "Audio Programming Sequence" Audio Enabling -
== Series Details ==
Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
URL : https://patchwork.freedesktop.org/series/131414/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14552 -> Patchwork_131414v3
== Series Details ==
Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
URL : https://patchwork.freedesktop.org/series/131414/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
On Wed, 2024-04-10 at 07:35 +, Manna, Animesh wrote:
>
>
> > -Original Message-
> > From: Hogander, Jouni
> > Sent: Wednesday, April 10, 2024 12:54 PM
> > To: Manna, Animesh ; intel-
> > g...@lists.freedesktop.org
> > Cc: ville.syrj...@linux.intel.com; Murthy, Arun R
> >
> >
> -Original Message-
> From: Hogander, Jouni
> Sent: Wednesday, April 10, 2024 12:54 PM
> To: Manna, Animesh ; intel-
> g...@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Murthy, Arun R
> Subject: Re: [PATCH v4] drm/i915/panelreplay: Panel replay workaround with
> VRR
>
>
On Thu, 2024-03-28 at 10:13 +0530, Animesh Manna wrote:
> Panel Replay VSC SDP not getting sent when VRR is enabled
> and W1 and W2 are 0. So Program Set Context Latency in
> TRANS_SET_CONTEXT_LATENCY register to at least a value of 1.
>
> HSD: 14015406119
>
> v1: Initial version.
> v2: Update
== Series Details ==
Series: drm/i915: Don't enable hwmon for selftests (rev3)
URL : https://patchwork.freedesktop.org/series/132243/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14552 -> Patchwork_132243v3
Summary
On 10-04-2024 11:35, Ashutosh Dixit wrote:
There are no hwmon selftests so there is no need to enable hwmon for
selftests. So enable hwmon only for real driver load.
v2: Move the logic inside i915_hwmon.c
v3: Move is_i915_selftest definition to i915_selftest.h (Badal)
Closes:
== Series Details ==
Series: drm/i915: Don't enable hwmon for selftests (rev3)
URL : https://patchwork.freedesktop.org/series/132243/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915/psr: Use crtc_state->port_clock instead of intel_dp->link_rate
(rev2)
URL : https://patchwork.freedesktop.org/series/132198/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14552 -> Patchwork_132198v2
On Tue, 09 Apr 2024 22:17:38 -0700, Nilawar, Badal wrote:
>
> > +#if IS_ENABLED(CONFIG_DRM_I915_SELFTEST)
> > + return i915_selftest.live || i915_selftest.perf || i915_selftest.mock;
> > +#else
> > + return false;
> > +#endif
> > +}
> I think you moved this function here as this is only used
There are no hwmon selftests so there is no need to enable hwmon for
selftests. So enable hwmon only for real driver load.
v2: Move the logic inside i915_hwmon.c
v3: Move is_i915_selftest definition to i915_selftest.h (Badal)
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10366
86 matches
Mail list logo
