== Series Details ==
Series: drm/i915: replace X86_FEATURE_PAT with pat_enabled()
URL : https://patchwork.freedesktop.org/series/97482/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10952_full -> Patchwork_21723_full
Summar
== Series Details ==
Series: drm/i915: Fix error pointer dereference in i915_gem_do_execbuffer()
(rev4)
URL : https://patchwork.freedesktop.org/series/96969/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10953 -> Patchwork_21725
===
> -Original Message-
> From: Jani Nikula
> Sent: Wednesday, December 1, 2021 5:52 PM
> To: Srivatsa, Anusha ; intel-
> g...@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [v3 0/3] Introduce Raptor Lake S
>
> On Wed, 01 Dec 2021, Anusha Srivatsa wrote:
> > Raptor Lake S(RPL-S) is a v
== Series Details ==
Series: static analysis failure
URL : https://patchwork.freedesktop.org/series/97486/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10953 -> Patchwork_21724
Summary
---
**SUCCESS**
No regressi
From: Dan Carpenter
Originally "out_fence" was set using out_fence = sync_file_create() but
which returns NULL, but now it is set with out_fence = eb_requests_create()
which returns error pointers. The error path needs to be modified to
avoid an Oops in the "goto err_request;" path.
Fixes: 5444
Hi Arunpravin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on v5.16-rc3]
[cannot apply to drm/drm-next drm-tip/drm-tip next-20211201]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when
fix for null ptr dereferences.
Signed-off-by: Pallavi Mishra
---
drivers/gpu/drm/i915/gem/i915_gem_ttm.c | 3 +++
drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c | 4 +++-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_ttm.c
b/drivers/gpu/drm/i
fix for null ptr dereferences
Pallavi Mishra (1):
static analysis failure
drivers/gpu/drm/i915/gem/i915_gem_ttm.c | 3 +++
drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c | 4 +++-
2 files changed, 6 insertions(+), 1 deletion(-)
--
2.25.1
== Series Details ==
Series: Bump DMC to v2.14 on ADL-P (rev2)
URL : https://patchwork.freedesktop.org/series/97477/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10951_full -> Patchwork_21722_full
Summary
---
**SUCC
> -Original Message-
> From: Ville Syrjälä
> Sent: Wednesday, December 1, 2021 8:33 PM
> To: Srinivas, Vidya
> Cc: intel-gfx@lists.freedesktop.org; Yashashvi, Shantam
>
> Subject: Re: [PATCH] drm/i915: Add PLANE_CUS_CTL restriction in max_width
>
> On Wed, Dec 01, 2021 at 09:17:27AM
== Series Details ==
Series: drm/i915: Plane register cleanup
URL : https://patchwork.freedesktop.org/series/97467/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10950_full -> Patchwork_21718_full
Summary
---
**SUCCE
== Series Details ==
Series: drm/i915: replace X86_FEATURE_PAT with pat_enabled()
URL : https://patchwork.freedesktop.org/series/97482/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10952 -> Patchwork_21723
Summary
---
On Tue, 30 Nov 2021 05:20:05 -0800, Anshuman Gupta wrote:
>
> gt_pm selftest calculates engine ticks cycles and wall time
> cycles by delta of respective engine elapsed TIMESTAMP and ktime
> for period of 1000us.
> It compares the engine ticks cycles with wall time cycles.
>
> Disable local cpu int
@Jani @Manasi
Bump.
On Mon, 2021-11-08 at 15:52 -0800, Navare, Manasi wrote:
> On Mon, Nov 01, 2021 at 12:25:21PM +0200, Jani Nikula wrote:
> > On Mon, 28 Jun 2021, Madhumitha Tolakanahalli Pradeep
> > wrote:
> > > PCH display HPD IRQ is not detected with default filter value.
> > > So, PP_CONTR
Hi Ben, Josh, Kyle,
Kindly add the below i915 changes to linux-firmware:
The following changes since commit
b0e898fbaf377c99a36aac6fdeb7250003648ca4:
linux-firmware: Update firmware file for Intel Bluetooth 9462 (2021-
11-23 12:31:45 -0500)
are available in the Git repository at:
git://ano
Hi Arunpravin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on v5.16-rc3]
[cannot apply to drm/drm-next drm-tip/drm-tip next-20211201]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when
On Wed, Dec 01, 2021 at 02:24:04PM -0800, Madhumitha Tolakanahalli Pradeep
wrote:
Changes since v2.12:
- Release notes for v2.13:
1. Fix for simple flip queue with DC6v
- Release notes for v2.14:
1. Fix for flip queue roll over cases with DC6v
2. Enhancement for residency
3
Hi all,
Today's linux-next merge of the drm-intel-gt tree got a conflict in:
drivers/gpu/drm/i915/display/intel_fbc.c
between commit:
d06188234427 ("drm/i915/fbc: s/dev_priv/i915/")
from the drm-intel tree and commit:
cca084692394 ("drm/i915: Use per device iommu check")
from the drm-i
PAT can be disabled on boot with "nopat" in the command line. Replace
one x86-ism with another, which is slightly more correct to prepare for
supporting other architectures.
Cc: Matt Roper
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 8
drivers/gpu/drm
== Series Details ==
Series: Bump DMC to v2.14 on ADL-P (rev2)
URL : https://patchwork.freedesktop.org/series/97477/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10951 -> Patchwork_21722
Summary
---
**SUCCESS**
N
== Series Details ==
Series: drm/i915: Re-use i915 macros for checking PTEs (rev9)
URL : https://patchwork.freedesktop.org/series/97090/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10951 -> Patchwork_21721
Summary
---
== Series Details ==
Series: drm/i915: Re-use i915 macros for checking PTEs (rev9)
URL : https://patchwork.freedesktop.org/series/97090/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
3809a1d72d34 drm/i915: Re-use i915 macros for checking PTEs
-:6: WARNING:COMMIT_LOG_LONG_LINE:
Adding PR for CI to pick the firmware,
The following changes since commit f5d519563ac9d2d1f382a817aae5ec5473811ac8:
linux-firmware: Update AMD cpu microcode (2021-11-15 12:49:19 -0500)
are available in the Git repository at:
git://anongit.freedesktop.org/drm/drm-firmware adlp_dmc_v2.14
for
Changes since v2.12:
- Release notes for v2.13:
1. Fix for simple flip queue with DC6v
- Release notes for v2.14:
1. Fix for flip queue roll over cases with DC6v
2. Enhancement for residency
3. Workaround for 3Dlut restore issue
v2: Commit message update (Imre)
Signed-
== Series Details ==
Series: series starting with [v4,1/6] drm: move the buddy allocator from i915
into common drm
URL : https://patchwork.freedesktop.org/series/97476/
State : failure
== Summary ==
Applying: drm: move the buddy allocator from i915 into common drm
Using index info to reconstr
== Series Details ==
Series: drm/i915: Re-use i915 macros for checking PTEs (rev8)
URL : https://patchwork.freedesktop.org/series/97090/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10951 -> Patchwork_21719
Summary
---
== Series Details ==
Series: drm/i915: Re-use i915 macros for checking PTEs (rev8)
URL : https://patchwork.freedesktop.org/series/97090/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
6a888e8bac16 drm/i915: Re-use i915 macros for checking PTEs
-:6: WARNING:COMMIT_LOG_LONG_LINE:
== Series Details ==
Series: drm/i915: break intel_display_types.h dependency on i915_drv.h (rev2)
URL : https://patchwork.freedesktop.org/series/97173/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949_full -> Patchwork_21716_full
===
== Series Details ==
Series: drm/i915: Plane register cleanup
URL : https://patchwork.freedesktop.org/series/97467/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10950 -> Patchwork_21718
Summary
---
**SUCCESS**
No
== Series Details ==
Series: series starting with [1/2] Revert "drm/i915: Implement Wa_1508744258"
(rev2)
URL : https://patchwork.freedesktop.org/series/97105/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949_full -> Patchwork_21715_full
===
== Series Details ==
Series: drm/i915: Plane register cleanup
URL : https://patchwork.freedesktop.org/series/97467/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915: Plane register cleanup
URL : https://patchwork.freedesktop.org/series/97467/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
2a489c8cf0ae drm/i915: Get rid of the 64bit PLANE_CC_VAL mmio
-:69: WARNING:LONG_LINE: line length of 102 exceeds 10
== Series Details ==
Series: drm/i915: Re-use i915 macros for checking PTEs (rev7)
URL : https://patchwork.freedesktop.org/series/97090/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10950 -> Patchwork_21717
Summary
---
== Series Details ==
Series: drm/i915: Re-use i915 macros for checking PTEs (rev7)
URL : https://patchwork.freedesktop.org/series/97090/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
68b441f63a8e drm/i915: Re-use i915 macros for checking PTEs
-:6: WARNING:COMMIT_LOG_LONG_LINE:
== Series Details ==
Series: drm/i915/adl_p: Add adl-p ddc pin mapping (rev2)
URL : https://patchwork.freedesktop.org/series/97009/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949_full -> Patchwork_21714_full
Summary
--
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> There is no real point in having this two stage
> skl_program_plane*() vs. skl_plane_update*() wrapper stuff.
> All we need to do is determine the correct color plane and
> we're done.
Reviewed-by: José Roberto de
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Add a few small helpers to calculate the color key register
> values. Cleans up skl_program_plane_arm() a bit.
Reviewed-by: José Roberto de Souza
>
> Signed-off-by: Ville Syrjälä
> ---
> .../drm/i915/display/s
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Extract the PLANE_AUX_DIST stuff into a small helper to
> dclutter skl_program_plane_arm() a bit.
Reviewed-by: José Roberto de Souza
>
> Signed-off-by: Ville Syrjälä
> ---
> .../drm/i915/display/skl_universal
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Polish the skl+ universal plane register defines by
> using REG_BIT() & co.
>
> The defines are also currently spread around in some
> semi-random fashion. Collect them up into one place.
>
> Signed-off-by: Ville
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Replace the "sizes are 0 based" stuff with just straight
> up -1 where needed. Less confusing all around.
>
> Signed-off-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/display/intel_sprite.c | 26 --
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Rename the PLANE_CUS_CTL Y plane selection bits to actually
> say "Y plane".
>
Reviewed-by: José Roberto de Souza
> Signed-off-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/display/intel_display.c | 8 ---
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Let's just stick to 32bit mmio accesses so we can get rid
> of the bare "uncore" reg access in display code. The register
> are defined as 32bit in the spec anyway.
>
> We could define a 64bit "de" variant I suppos
On Wed, 2021-12-01 at 17:25 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Rename the YUV byte order bits to be a bit more consistent.
Why rename bits not used? Would be better already nuke it.
Anyways up to you.
Reviewed-by: José Roberto de Souza
>
> Signed-off-by: Ville Syrjälä
> -
From: Ville Syrjälä
Use REG_BIT() & co. to polish the vlv/chv sprite plane registers.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_sprite.c | 9 +-
drivers/gpu/drm/i915/i915_reg.h | 103
2 files changed, 70 insertions(+), 42 deletions(-
On Wed, Dec 01, 2021 at 03:57:12PM +0200, Jani Nikula wrote:
> Break the dependency on i915_drv.h.
>
> Signed-off-by: Jani Nikula
Looks reasonable.
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/display/intel_display_types.h| 9 -
> drivers/gpu/drm/i915/display/intel_dp
On Wed, Dec 01, 2021 at 09:17:27AM +0530, Vidya Srinivas wrote:
> PLANE_CUS_CTL has a restriction of 4096 width even though
> PLANE_SIZE and scaler size registers supports max 5120.
> Take care of this restriction in max_width.
>
> Without this patch, when 5k content is sent on HDR plane
> with NV
On Fri, Nov 19, 2021 at 06:09:31AM -0800, José Roberto de Souza wrote:
> Those two workarounds needs to be implemented in UMD, KMD only needs
> to whitelist the registers, so here only adding the workaround number
> to facilitate future workaroud table checks.
>
> Signed-off-by: José Roberto de So
== Series Details ==
Series: drm/i915: Skip remap_io_mapping() for non-x86 platforms (rev4)
URL : https://patchwork.freedesktop.org/series/96855/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949_full -> Patchwork_21713_full
==
== Series Details ==
Series: Introduce Raptor Lake S (rev4)
URL : https://patchwork.freedesktop.org/series/96869/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949_full -> Patchwork_21712_full
Summary
---
**SUCCESS
From: Ville Syrjälä
Extract the PLANE_AUX_DIST stuff into a small helper to
dclutter skl_program_plane_arm() a bit.
Signed-off-by: Ville Syrjälä
---
.../drm/i915/display/skl_universal_plane.c| 35 ---
1 file changed, 23 insertions(+), 12 deletions(-)
diff --git a/drivers/g
From: Ville Syrjälä
Rename the PLANE_CUS_CTL Y plane selection bits to actually
say "Y plane".
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 8
drivers/gpu/drm/i915/i915_reg.h | 8
2 files changed, 8 insertions(+), 8 deletions(-)
From: Ville Syrjälä
Rename the YUV byte order bits to be a bit more consistent.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/skl_universal_plane.c | 8
drivers/gpu/drm/i915/i915_reg.h| 14 +++---
2 files changed, 11 insertions(+), 11 delet
From: Ville Syrjälä
There is no real point in having this two stage
skl_program_plane*() vs. skl_plane_update*() wrapper stuff.
All we need to do is determine the correct color plane and
we're done.
Signed-off-by: Ville Syrjälä
---
.../drm/i915/display/skl_universal_plane.c| 53 ++-
From: Ville Syrjälä
Add a few small helpers to calculate the color key register
values. Cleans up skl_program_plane_arm() a bit.
Signed-off-by: Ville Syrjälä
---
.../drm/i915/display/skl_universal_plane.c| 45 +--
1 file changed, 32 insertions(+), 13 deletions(-)
diff --gi
From: Ville Syrjälä
Use REG_BIT() & co. to polish the cursor plane registers.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_cursor.c | 25 ---
drivers/gpu/drm/i915/display/intel_display.c | 4 +-
drivers/gpu/drm/i915/i915_reg.h | 71 +++-
From: Ville Syrjälä
Use REG_BIT() & co. to polish the g4x+ sprite plane registers.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_sprite.c | 12 ++--
drivers/gpu/drm/i915/i915_reg.h | 73 +
2 files changed, 53 insertions(+), 32 deletions(-)
From: Ville Syrjälä
Replace the "sizes are 0 based" stuff with just straight
up -1 where needed. Less confusing all around.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_sprite.c | 26 ---
.../drm/i915/display/skl_universal_plane.c| 6 +
2 files
From: Ville Syrjälä
Use REG_BIT() & co. to polish the ivb+ sprite plane registers.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_sprite.c | 20 +++--
drivers/gpu/drm/i915/i915_reg.h | 81 +
2 files changed, 62 insertions(+), 39 deletions(-)
From: Ville Syrjälä
There's no need to have separate masks for the stride bitfield
in PLANE_STRIDE for different platforms. All the extra bits
are hardcoded to zero anyway.
Also the masks we're using now don't even match the actual hardware
since the bitfield was only 10 bits on skl/derivatives,
From: Ville Syrjälä
Use REG_BIT() & co. for the pre-skl primary plane registers.
Also give everything a consistent namespace.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/i9xx_plane.c| 99 +
drivers/gpu/drm/i915/display/intel_display.c | 13 +--
drivers/g
From: Ville Syrjälä
Polish the skl+ universal plane register defines by
using REG_BIT() & co.
The defines are also currently spread around in some
semi-random fashion. Collect them up into one place.
Signed-off-by: Ville Syrjälä
---
.../drm/i915/display/skl_universal_plane.c| 36 ++--
dr
From: Ville Syrjälä
Let's just stick to 32bit mmio accesses so we can get rid
of the bare "uncore" reg access in display code. The register
are defined as 32bit in the spec anyway.
We could define a 64bit "de" variant I suppose, but doesn't
really make much sense just for this one case, and when
From: Ville Syrjälä
Bunch of cleanup around plane registers, and a bit of
reshuffling in the skl+ universal plane code.
Ville Syrjälä (14):
drm/i915: Get rid of the 64bit PLANE_CC_VAL mmio
drm/i915: Rename plane YUV order bits
drm/i915: Get rid of the "sizes are 0 based" stuff
drm/i915:
On Wed, Dec 01, 2021 at 03:57:11PM +0200, Jani Nikula wrote:
> Avoid looking into the guts of struct drm_i915_private in
> headers. Again, converting an inline function to a macro is less than
> ideal, but avoids having to pull in i915_drv.h just for the to_i915()
> part.
Ugly, but gets the job do
On Wed, Dec 01, 2021 at 03:57:10PM +0200, Jani Nikula wrote:
> It's not the ideal location, but a better alternative than
> i915_drv.h. The goal is to break the intel_display_types.h to i915_drv.h
> dependency.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/
On Wed, Dec 01, 2021 at 03:57:09PM +0200, Jani Nikula wrote:
> This is far from ideal, but it reduces the i915_drv.h dependency from
> intel_display_types.h. Maybe in the future we'll need a better split.
Yeah, looks pretty temporary until we move the wm code into some
actually sensible place.
Re
On Wed, Dec 01, 2021 at 03:57:07PM +0200, Jani Nikula wrote:
> Move a number of crtc/pipe related functions to intel_crtc.[ch], and
> un-inline to avoid looking into struct drm_i915_private guts in header
> files.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/d
On Wed, Dec 01, 2021 at 03:57:08PM +0200, Jani Nikula wrote:
> Move fb functions where they belong, and un-inline to avoid looking into
> struct drm_i915_private guts in header files.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/display/intel_display_
On Wed, Dec 01, 2021 at 03:57:06PM +0200, Jani Nikula wrote:
> There are only three call sites remaining for
> intel_wait_for_vblank(). Remove the function, and open code it to avoid
> new users from showing up.
>
> v2:
> - Use intel_crtc_wait_for_next_vblank() (Ville)
>
> Signed-off-by: Jani Nik
On Wed, Dec 01, 2021 at 03:57:05PM +0200, Jani Nikula wrote:
> The "get" in the name implies reference counting, remove it. This also
> makes the function conform to naming style.
>
> Suggested-by: Ville Syrjälä
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/
On Wed, Dec 01, 2021 at 03:57:03PM +0200, Jani Nikula wrote:
> intel_wait_for_vblank() goes through a pipe to crtc lookup, while in
> most cases we already have the crtc available. Avoid the extra lookups
> by adding an intel_crtc based helper.
>
> v2:
> - Add intel_crtc_wait_for_next_vblank() hel
On Wed, Dec 01, 2021 at 03:57:04PM +0200, Jani Nikula wrote:
> The "get" in the name implies reference counting, remove it. This also
> makes the function conform to naming style.
>
> Suggested-by: Ville Syrjälä
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> .../gpu/drm/i915
On Tue, 30 Nov 2021 at 09:21, Maarten Lankhorst
wrote:
>
> We will need the lock to unbind the vma, and wait for bind to complete.
> Remove the special casing for the !ww path, and force ww locking for all.
>
> Changes since v1:
> - Pass err to for_i915_gem_ww handling for -EDEADLK handling.
>
> S
== Series Details ==
Series: drm/i915: break intel_display_types.h dependency on i915_drv.h (rev2)
URL : https://patchwork.freedesktop.org/series/97173/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949 -> Patchwork_21716
== Series Details ==
Series: series starting with [1/2] Revert "drm/i915: Implement Wa_1508744258"
(rev2)
URL : https://patchwork.freedesktop.org/series/97105/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949 -> Patchwork_21715
=
== Series Details ==
Series: drm/i915: break intel_display_types.h dependency on i915_drv.h (rev2)
URL : https://patchwork.freedesktop.org/series/97173/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separate
== Series Details ==
Series: drm/i915: break intel_display_types.h dependency on i915_drv.h (rev2)
URL : https://patchwork.freedesktop.org/series/97173/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
e378d10ac676 drm/i915/display: add intel_crtc_wait_for_next_vblank() and use it
Break the dependency on i915_drv.h.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display_types.h| 9 -
drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c | 1 +
drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +-
drivers/gpu/drm/i915/display/intel
Avoid looking into the guts of struct drm_i915_private in
headers. Again, converting an inline function to a macro is less than
ideal, but avoids having to pull in i915_drv.h just for the to_i915()
part.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display_types.h | 6 +-
It's not the ideal location, but a better alternative than
i915_drv.h. The goal is to break the intel_display_types.h to i915_drv.h
dependency.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display.h | 24
drivers/gpu/drm/i915/i915_drv.h | 24
This is far from ideal, but it reduces the i915_drv.h dependency from
intel_display_types.h. Maybe in the future we'll need a better split.
Signed-off-by: Jani Nikula
---
.../drm/i915/display/intel_display_types.h| 1 +
drivers/gpu/drm/i915/i915_drv.h | 64 +---
dr
Move fb functions where they belong, and un-inline to avoid looking into
struct drm_i915_private guts in header files.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display_types.h | 10 --
drivers/gpu/drm/i915/display/intel_fb.c| 10 ++
drivers/gp
Move a number of crtc/pipe related functions to intel_crtc.[ch], and
un-inline to avoid looking into struct drm_i915_private guts in header
files.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_crtc.c | 42 ++
drivers/gpu/drm/i915/display/intel_crtc.h |
There are only three call sites remaining for
intel_wait_for_vblank(). Remove the function, and open code it to avoid
new users from showing up.
v2:
- Use intel_crtc_wait_for_next_vblank() (Ville)
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_cdclk.c | 2 +-
drivers/
The "get" in the name implies reference counting, remove it. This also
makes the function conform to naming style.
Suggested-by: Ville Syrjälä
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display_types.h | 2 +-
drivers/gpu/drm/i915/intel_pm.c| 4 ++--
2
The "get" in the name implies reference counting, remove it. This also
makes the function conform to naming style.
Suggested-by: Ville Syrjälä
Signed-off-by: Jani Nikula
---
.../gpu/drm/i915/display/intel_atomic_plane.c| 2 +-
drivers/gpu/drm/i915/display/intel_cdclk.c | 2 +-
drive
intel_wait_for_vblank() goes through a pipe to crtc lookup, while in
most cases we already have the crtc available. Avoid the extra lookups
by adding an intel_crtc based helper.
v2:
- Add intel_crtc_wait_for_next_vblank() helper (Ville)
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/displa
v2 of https://patchwork.freedesktop.org/series/97173/
BR,
Jani.
Jani Nikula (10):
drm/i915/display: add intel_crtc_wait_for_next_vblank() and use it
drm/i915/crtc: rename intel_get_crtc_for_pipe() to
intel_crtc_for_pipe()
drm/i915/crtc: rename intel_get_crtc_for_plane() to
intel_cr
== Series Details ==
Series: drm/i915/adl_p: Add adl-p ddc pin mapping (rev2)
URL : https://patchwork.freedesktop.org/series/97009/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949 -> Patchwork_21714
Summary
---
*
On Tue, Nov 30, 2021 at 11:14 AM Marco Elver wrote:
>
> The non-interrupt portion of interrupt stack traces before interrupt
> entry is usually arbitrary. Therefore, saving stack traces of interrupts
> (that include entries before interrupt entry) to stack depot leads to
> unbounded stackdepot gro
== Series Details ==
Series: drm/i915: Skip remap_io_mapping() for non-x86 platforms (rev4)
URL : https://patchwork.freedesktop.org/series/96855/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949 -> Patchwork_21713
Summar
On 01/12/2021 11:15, Maarten Lankhorst wrote:
On 30-11-2021 19:38, Tvrtko Ursulin wrote:
On 30/11/2021 11:17, Maarten Lankhorst wrote:
On 30-11-2021 09:54, Tvrtko Ursulin wrote:
Hi,
On 29/11/2021 13:47, Maarten Lankhorst wrote:
New version of the series, with feedback from previous serie
Hi Michal,
> >> fist of all thanks for taking a look at this, I was eagerly
> >> waiting for reviewers.
> >>
> >> On Tue, Nov 30, 2021 at 01:07:30PM -0800, Lucas De Marchi wrote:
> >>> On Sun, Nov 28, 2021 at 01:09:26PM +0200, Andi Shyti wrote:
> >>> > Starting from a patch from Matt to_root_gt()
== Series Details ==
Series: drm/i915: Skip remap_io_mapping() for non-x86 platforms (rev4)
URL : https://patchwork.freedesktop.org/series/96855/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915: Skip remap_io_mapping() for non-x86 platforms (rev4)
URL : https://patchwork.freedesktop.org/series/96855/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
e1ed6afae3fa drm/i915: Skip remap_io_mapping() for non-x86 platforms
-:63: WARNING:FIL
== Series Details ==
Series: Introduce Raptor Lake S (rev4)
URL : https://patchwork.freedesktop.org/series/96869/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10949 -> Patchwork_21712
Summary
---
**SUCCESS**
No r
On Wed, 01 Dec 2021, Anusha Srivatsa wrote:
> Raptor Lake S(RPL-S) is a version 12
> Display, Media and Render. For all i915
> purposes it is the same as Alder Lake S (ADL-S).
>
> The series introduces it as a subplatform
> of ADL-S. The one difference is the GuC
> submission which is default on R
== Series Details ==
Series: Introduce Raptor Lake S (rev4)
URL : https://patchwork.freedesktop.org/series/96869/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
On 12/1/21 12:25, Christian König wrote:
Am 01.12.21 um 12:04 schrieb Thomas Hellström (Intel):
On 12/1/21 11:32, Christian König wrote:
Am 01.12.21 um 11:15 schrieb Thomas Hellström (Intel):
[SNIP]
What we could do is to avoid all this by not calling the callback
with the lock held in t
== Series Details ==
Series: Introduce Raptor Lake S (rev4)
URL : https://patchwork.freedesktop.org/series/96869/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
91c35932ece3 drm/i915/rpl-s: Add PCI IDS for Raptor Lake S
-:112: ERROR:COMPLEX_MACRO: Macros with complex values shou
1 - 100 of 130 matches
Mail list logo