On Tue, Mar 11, 2025 at 09:56:24PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> I don't think there should be any need for us to call any of
> pci_enable_device(), pci_disable_device() or pci_set_master()
> from the suspend/resume paths. The config space save/restore should
> take care o
On Tue, Mar 11, 2025 at 09:56:23PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Stop doing the pci_save_state(), except when we need to prevent
> D3 due to BIOS bugs, so that the code in drivers/pci is allowed
> to manage the state of the PCI device. Less chance something
> getting left
On Tue, Mar 11, 2025 at 09:56:20PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> drivers/pci does the pci_save_state()+pci_set_power_state() from
> the _noirq() pm hooks. Move our manual calls (needed for the
> hibernate vs. D3 workaround with buggy BIOSes) towards that same
> point. We c
Commit f67986b0119c04 ("drm/i915/pxp: Promote pxp subsystem to top-level
of i915") added PXP initialization to driver probe path, but didn't add a
respective PXP cleanup on probe error. That lack of cleanup seems
harmless as long as PXP is still unused and idle when a probe failure
occurs and erro
== Series Details ==
Series: drm/i915: Fix harmful driver register/unregister asymmetry
URL : https://patchwork.freedesktop.org/series/146336/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16292 -> Patchwork_146336v1
Summar
== Series Details ==
Series: drm/i915: Fix harmful driver register/unregister asymmetry
URL : https://patchwork.freedesktop.org/series/146336/
State : warning
== Summary ==
Error: dim checkpatch failed
01f7094dd53a drm/i915: Downgrade device register error if injected
67a8d087a526 drm/i915: Fi
Quoting Matt Roper (2025-03-14 18:17:37-03:00)
>On Tue, Mar 11, 2025 at 02:04:50PM -0300, Gustavo Sousa wrote:
>> Update intel_bw.c internally use intel_display. Conversion of the public
>> interface will come as a follow-up.
>>
>> v2:
>> - Prefer intel_uncore_read() for MCHBAR registers. (Ville
Hi,
On Tue, Mar 11, 2025 at 10:33:14AM +0100, Mikolaj Wasiak wrote:
> This test was designed to isolate a bug in tigerlake and dg2 hardware.
> The bug was found and fixed in newer generations.
> Since we won't support any new hardware with this driver, the test
> should now be turned off in the CI
On Tue, Mar 11, 2025 at 02:04:50PM -0300, Gustavo Sousa wrote:
> Update intel_bw.c internally use intel_display. Conversion of the public
> interface will come as a follow-up.
>
> v2:
> - Prefer intel_uncore_read() for MCHBAR registers. (Ville)
> v3:
> - Remove the unnecessary inclusion of int
Starting with commit ec3e00b4ee27 ("drm/i915: stop registering if
drm_dev_register() fails"), we return from i915_driver_register()
immediately if drm_dev_register() fails, skipping remaining registration
steps, and continue only with remaining probe steps. However, the
_unregister() counterpart c
Starting with commit ec3e00b4ee27 ("drm/i915: stop registering if
drm_dev_register() fails"), we return from i915_driver_register()
immediately if drm_dev_register() fails, skipping remaining registration
steps, and continue only with remaining probe steps. However, the
_unregister() counterpart c
Commit 8f460e2c78f2 ("drm/i915: Demidlayer driver loading") which
introduced manual device registration also added a message that is
submitted on device registration failure as an error. If that failure is
triggered by error injection test, that's an expected error, but CI still
reports it as a bu
== Series Details ==
Series: series starting with [1/3] drm/i915/dsi: Fix off by one in
BXT_MIPI_TRANS_VTOTAL
URL : https://patchwork.freedesktop.org/series/146325/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16292 -> Patchwork_146325v1
=
== Series Details ==
Series: drm/i915/selftest: allow larger memory allocation
URL : https://patchwork.freedesktop.org/series/146321/
State : warning
== Summary ==
Error: patch
https://patchwork.freedesktop.org/api/1.0/series/146321/revisions/1/mbox/ not
found
== Series Details ==
Series: series starting with [1/3] drm/i915/dsi: Fix off by one in
BXT_MIPI_TRANS_VTOTAL
URL : https://patchwork.freedesktop.org/series/146325/
State : warning
== Summary ==
Error: dim checkpatch failed
f342c298c703 drm/i915/dsi: Fix off by one in BXT_MIPI_TRANS_VTOTAL
-:
== Series Details ==
Series: series starting with [1/3] drm/i915/dsi: Fix off by one in
BXT_MIPI_TRANS_VTOTAL
URL : https://patchwork.freedesktop.org/series/146325/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separ
== Series Details ==
Series: drm/i915/selftest: allow larger memory allocation
URL : https://patchwork.freedesktop.org/series/146321/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_16291 -> Patchwork_146321v1
Summary
---
== Series Details ==
Series: drm/display: dp: add new DPCD access functions (rev2)
URL : https://patchwork.freedesktop.org/series/145998/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_16290 -> Patchwork_145998v2
Summary
---
Hi Mikolaj,
On 2025-03-14 at 15:42:23 GMT, Mikolaj Wasiak wrote:
> Due to changes in allocator, the size of the allocation for
> contiguous region is not rounded up to a power-of-two and
> instead allocated as is. Thus, change the part of test that
> expected the allocation to fail.
>
> Closes: h
Hi,
On 2025-03-14 at 15:43:02 +0100, Andi Shyti wrote:
> >
> > Fixes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13697
> >
>
> Please don't make it as tag. Fixes has a specific meaning that
> means that the patch is fixing a bug introduced by a specific
> commit and it needs to be
== Series Details ==
Series: Underrun on idle PSR workaround (rev2)
URL : https://patchwork.freedesktop.org/series/145986/
State : failure
== Summary ==
Error: make failed
CALLscripts/checksyscalls.sh
DESCEND objtool
INSTALL libsubcmd_headers
CC [M] drivers/gpu/drm/i915/display/in
Existing DPCD access functions return an error code or the number of
bytes being read / write in case of partial access. However a lot of
drivers either (incorrectly) ignore partial access or mishandle error
codes. In other cases this results in a boilerplate code which compares
returned value with
== Series Details ==
Series: drm/ci: uprev mesa, IGT
URL : https://patchwork.freedesktop.org/series/146298/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16290 -> Patchwork_146298v1
Summary
---
**SUCCESS**
No regr
== Series Details ==
Series: drm/i915: Disable RPG during live selftest (rev3)
URL : https://patchwork.freedesktop.org/series/143886/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16258 -> Patchwork_143886v3
Summary
---
== Series Details ==
Series: drm/i915/dmc: Add debugfs for dc6 counter (rev7)
URL : https://patchwork.freedesktop.org/series/144240/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16274 -> Patchwork_144240v7
Summary
---
Hi Mikolaj,
On Tue, Mar 04, 2025 at 09:43:26AM +0100, Mikolaj Wasiak wrote:
> When the driver is loaded on the system with numa nodes it might be run in
> a kthread, which makes it impossible to use current->mm in the selftest.
> This patch allows the selftest to use current->mm by using active_mm
== Series Details ==
Series: drm/i915/dmc: Add debugfs for dc6 counter (rev7)
URL : https://patchwork.freedesktop.org/series/144240/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16274 -> Patchwork_144240v7
Summary
---
From: Ville Syrjälä
BXT_MIPI_TRANS_VTOTAL must be programmed with vtotal-1
instead of vtotal. Make it so.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/vlv_dsi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/vlv_dsi.c
b/dr
From: Ville Syrjälä
GLK doesn't use the DSI C clock at all, no need to program
the divider for it. Bspec even says: "Do not program this field".
However looks like some firmware versions program this and
some do not. In order to avoid bogus fastset mismatches
we should also filter it out during
Hi, Borah
On Fri, 14 Mar 2025 at 17:18, Borah, Chaitanya Kumar
wrote:
>
> Hello Robin,
>
> Hope you are doing well. I am Chaitanya from the linux graphics team in Intel.
>
> This mail is regarding a regression we are seeing in our CI runs[1] on
> linux-next repository.
>
> Since the version next
Due to changes in allocator, the size of the allocation for
contiguous region is not rounded up to a power-of-two and
instead allocated as is. Thus, change the part of test that
expected the allocation to fail.
Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9311
Signed-off-by: Mi
Hi Mikolaj,
On Tue, Mar 04, 2025 at 09:43:26AM +0100, Mikolaj Wasiak wrote:
> When the driver is loaded on the system with numa nodes it might be run in
> a kthread, which makes it impossible to use current->mm in the selftest.
> This patch allows the selftest to use current->mm by using active_mm
Hi Sebastian,
On Fri, Mar 14, 2025 at 01:37:44PM +, Sebastian Brzezinka wrote:
> On Fri Mar 14, 2025 at 2:12 AM UTC, Andi Shyti wrote:
> > upto -> up to
> > acknowledgement -> acknowledgment
> Both spellings are correct.
thanks a lot for checking this, you are right! The former is UK
english
Hi
On Fri Mar 14, 2025 at 2:12 AM UTC, Andi Shyti wrote:
> upto -> up to
> acknowledgement -> acknowledgment
Both spellings are correct.
--
Best regards,
Sebastian
From: Dmitry Baryshkov
Switch drm_dp_tunnel.c to use new set of DPCD read / write helpers.
Reviewed-by: Lyude Paul
Acked-by: Jani Nikula
Signed-off-by: Dmitry Baryshkov
---
drivers/gpu/drm/display/drm_dp_tunnel.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
d
From: Dmitry Baryshkov
drm_dp_dpcd_read_link_status() follows the "return error code or number
of bytes read" protocol, with the code returning less bytes than
requested in case of some errors. However most of the drivers
interpreted that as "return error code in case of any error". Switch
drm_dp
From: Dmitry Baryshkov
Switch drm_dp_cec.c to use new set of DPCD read / write helpers.
Reviewed-by: Lyude Paul
Acked-by: Jani Nikula
Signed-off-by: Dmitry Baryshkov
---
drivers/gpu/drm/display/drm_dp_cec.c | 37 ++--
1 file changed, 18 insertions(+), 19 delet
From: Dmitry Baryshkov
Existing DPCD access functions return an error code or the number of
bytes being read / write in case of partial access. However a lot of
drivers either (incorrectly) ignore partial access or mishandle error
codes. In other cases this results in a boilerplate code which com
From: Dmitry Baryshkov
Switch drm_dp_helper.c to use new set of DPCD read / write helpers.
Reviewed-by: Lyude Paul
Acked-by: Jani Nikula
Signed-off-by: Dmitry Baryshkov
---
drivers/gpu/drm/display/drm_dp_helper.c | 296 +---
1 file changed, 116 insertions(+), 180
Hi Andi
On Fri Mar 14, 2025 at 2:12 AM UTC, Andi Shyti wrote:
> Hi
>
> Browsing through my endless list of branches I found a few with
> random cleanups that I fix while rading the code. They tend to be
s/rading/reading
The whole series looks good in my opinion.
Reviewed-by: Sebastian Brzezinka
We need to ensure transcoder is supporting Selective Update in case of
Panel Replay as well.
Signed-off-by: Jouni Högander
---
drivers/gpu/drm/i915/display/intel_psr.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
b/d
We have extended using vblank DC workaround mechanism for
Wa_16025596647. Rename related functions and variables:
vblank_wa_num_pipes -> vblank_enable_count
vblank_dc_work -> vblank_notify_work
intel_display_vblank_dc_work -> intel_display_vblank_notify_work
Signed-off-by: Jouni Högander
---
dr
We need to apply/remove workaround for underrun on idle PSR HW issue
(Wa_16025596647) when DC5/6 is enabled/disabled. This patch implements
mechanism to notify PSR about DC5/6 enable/disable and applies/removes the
workaround using this notification.
Bspec: 74115
Signed-off-by: Jouni Högander
--
We need to apply/remove workaround for underrun on idle PSR HW issue
(Wa_16025596647) when new pipe is enabled or pipe is getting disabled. This
patch implements mechanism to notify PSR about pipe enable/disable and
applies/removes the workaround using this notification.
Bspec: 74151
Signed-off-b
We need PIPEDMC_BLOCK_PKGC_SW definitions to implement workaround for
underrun on idle PSR HW issue (Wa_16025596647). Add PIPEDMC_BLOCK_PKGC_SW
register definitions.
Bspec: 71265
Signed-off-by: Jouni Högander
---
drivers/gpu/drm/i915/display/intel_dmc_regs.h | 8
1 file changed, 8 inse
Write PIPEDMC_BLOCK_PKGC_SW when enabling PSR as described in workaround
for underrun on idle PSR HW issue (Wa_16025596647).
Bspec: 74151
Signed-off-by: Jouni Högander
---
drivers/gpu/drm/i915/display/intel_psr.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/drivers/gpu/d
To implement workaround for underrun on idle PSR HW issue (Wa_16025596647)
we need to know enabled. Figure out which non-PSR pipes we will have active
and store it into intel_crtc_state->active_non_psr_pipes. This is currently
assuming only one eDP on a time. I.e. possible secondary eDP with PSR
ca
On 3/12/2025 4:55 PM, Jani Nikula wrote:
On Wed, 12 Mar 2025, Ankit Nautiyal wrote:
Move HAS_* macros to maintain asciibetical order.
Signed-off-by: Ankit Nautiyal
Reviewed-by: Jani Nikula
Thanks, Jani, for the review and for explaining the details. I've pushed
the change to drm-intel-ne
On 2025-03-14 9:17 am, Borah, Chaitanya Kumar wrote:
Hello Robin,
Hope you are doing well. I am Chaitanya from the linux graphics team in Intel.
This mail is regarding a regression we are seeing in our CI runs[1] on
linux-next repository.
Since the version next-20250312 [2], we are seeing the
Hello Robin,
Hope you are doing well. I am Chaitanya from the linux graphics team in Intel.
This mail is regarding a regression we are seeing in our CI runs[1] on
linux-next repository.
Since the version next-20250312 [2], we are seeing the following regression
> -Original Message-
> From: Baolu Lu
> Sent: Thursday, March 13, 2025 7:53 PM
> To: Borah, Chaitanya Kumar
> Cc: baolu...@linux.intel.com; intel-gfx@lists.freedesktop.org; intel-
> x...@lists.freedesktop.org; io...@lists.linux.dev
> Subject: Re: Regression on drm-tip
>
> On 2025/3/13
Uprev IGT to the latest version and update expectation files.
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci/gitlab-ci.yml | 2 +-
.../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt | 8 +-
.../gpu/drm/ci/xfails/amdgpu-stoney-skips.txt | 1 +
drivers/gpu/drm/ci/xfails/i915-aml
The mediatek display driver fails to probe on mt8173-elm-hana and
mt8183-kukui-jacuzzi-juniper-sku16 in v6.14-rc4 due to missing PHY
configurations.
Enable the following PHY drivers for MediaTek platforms:
- CONFIG_PHY_MTK_HDMI=y for HDMI display
- CONFIG_PHY_MTK_MIPI_DSI=y for DSI display
Signed
LAVA was recently patched [1] with a fix on how parameters are parsed in
`lava-test-case`, so we don't need to repeat quotes to send the
arguments properly to it. Uprev mesa to fix this issue.
[1] https://gitlab.com/lava/lava/-/commit/18c9cf79
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci
Uprev mesa to fix lava-test-case argument parsing. LAVA recently fixed
parameter parsing in `lava-test-case` [1], eliminating the need for
repeated quotes in arguments.
Uprev IGT to the latest version. Also update expectation files.
The mediatek display driver fails to probe on mt8173-elm-hana an
== Series Details ==
Series: drm/i915/psr: Add PSR pause/resume reference count
URL : https://patchwork.freedesktop.org/series/146286/
State : warning
== Summary ==
Error: dim checkpatch failed
2eb6a8d53863 drm/i915/psr: Add PSR pause/resume reference count
-:12: WARNING:COMMIT_LOG_LONG_LINE:
== Series Details ==
Series: drm/i915/xe3lpd: Update bandwidth parameters (rev5)
URL : https://patchwork.freedesktop.org/series/11/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16263 -> Patchwork_11v5
Summary
-
Hi Dave & Sima -
The second and likely final batch of i915 features towards v6.15.
BR,
Jani.
drm-intel-next-2025-03-10:
drm/i915 feature pull #2 for v6.15:
Features and functionality:
- FBC dirty rectangle support for display version 30+ (Vinod)
- Update plane scalers via DSB based commits (
58 matches
Mail list logo
