Hi Dmitry,
On 30/04/24 15:47, Dmitry Baryshkov wrote:
On Tue, Apr 30, 2024 at 02:41:18PM +0530, Vignesh Raman wrote:
Stop vendoring the testlist into the kernel. Instead, use the
testlist from the IGT build to ensure we do not miss renamed
or newly added tests.
Signed-off-by: Vignesh Raman
== Series Details ==
Series: Enable Aux Based EDP HDR (rev5)
URL : https://patchwork.freedesktop.org/series/132009/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14716 -> Patchwork_132009v5
Summary
---
**SUCCESS**
== Series Details ==
Series: Enable Aux Based EDP HDR (rev5)
URL : https://patchwork.freedesktop.org/series/132009/
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:
== Series Details ==
Series: Fixes in hdcp remote capability (rev2)
URL : https://patchwork.freedesktop.org/series/133047/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14716 -> Patchwork_133047v2
Summary
---
Write panel override luminance values which helps the TCON decide
if tone mapping needs to be enabled or not.
Signed-off-by: Suraj Kandpal
Reviewed-by: Sebastian Wick
---
.../drm/i915/display/intel_dp_aux_backlight.c | 25 +++
1 file changed, 25 insertions(+)
diff --git
As of now whenerver HDR is switched on we use the PWM to change the
backlight as opposed to AUX based backlight changes in terms of nits.
This patch writes to the appropriate DPCD registers to enable aux
based backlight using values in nits.
--v2
-Fix max_cll and max_fall assignment [Jani]
-Fix
Rename intel to intel cap which is present inside panel.edp struct
to make it more sensible to tell us what it contains.
Signed-off-by: Suraj Kandpal
Reviewed-by: Arun R Murthy
---
drivers/gpu/drm/i915/display/intel_display_types.h | 2 +-
.../gpu/drm/i915/display/intel_dp_aux_backlight.c |
Drop comments for EDP HDR DPCD registers as the code and
conditions will tell us what can be written where.
--v2
-Drop the comments altogether instead of just renaming them
[Sebastian]
Signed-off-by: Suraj Kandpal
Reviewed-by: Arun R Murthy
---
Change INTEL_EDP_HDR_TCON_SDP_COLORIMETRY enable to
INTEL_EDP_HDR_TCON_SDP_OVERRIDE_AUX as this bit tells TCON to
ignore DPCD colorimetry values and take the one's sent through
SDP.
--v2
-Fix typo in commit message [Arun]
Signed-off-by: Suraj Kandpal
Reviewed-by: Arun R Murthy
---
Add checks to see the HDR capability of TCON panel.
Signed-off-by: Suraj Kandpal
Reviewed-by: Arun R Murthy
---
drivers/gpu/drm/i915/display/intel_display_types.h| 5 +
drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c | 10 ++
2 files changed, 15 insertions(+)
diff --git
This series enables Aux based EDP HDR and backlight controls.
The DPCD written to are intel proprietary and are filled
based on the specs that were provided to TCON vendors.
Signed-off-by: Suraj Kandpal
Suraj Kandpal (7):
drm/i915/dp: Make has_gamut_metadata_dip() non static
drm/i915/dp:
Make has_gamut_metadata_dip() non static so it can also be used to
at other places eg in intel_dp_aux_backlight. So that we can check
if HW is capable of sending SDP which helps us decide if we use
AUX based HDR control or via SDP.
--v2
-State reason the function is needed [Arun]
Signed-off-by:
This patch series aims to avoid page fault errors caused by aux
early assignment as connector may not be mst and using encoder
to check for connector type.
Signed-off-by: Suraj Kandpal
Suraj Kandpal (2):
drm/i915/hdcp: Move aux assignment after connector type check
drm/i915/hdcp: Check
Check mst_port field in intel_connector to check connector type
rather than rely on encoder as it may not be attached to connector
at times.
Signed-off-by: Suraj Kandpal
---
drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Move assignment of aux after connector type check as port may not
exist if connector is not DPMST.
--v2
-Fix unwanted change in intel_encoder check [Jani]
Signed-off-by: Suraj Kandpal
---
drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff
>-Original Message-
>From: Vivi, Rodrigo
>Sent: Monday, May 6, 2024 10:52 PM
>To: Kumar, Naveen1
>Cc: intel-gfx@lists.freedesktop.org; Shankar, Uma ;
>Kulkarni, Vandita ; Nikula, Jani
>; Belgaumkar, Vinay ;
>Borah, Chaitanya Kumar
>Subject: Re: [PATCH] drm/i915/display: Introduce
== Series Details ==
Series: Introducing I915_FORMAT_MOD_4_TILED_XE2_CCS Modifier for Xe2
URL : https://patchwork.freedesktop.org/series/133253/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14715 -> Patchwork_133253v1
== Series Details ==
Series: Introducing I915_FORMAT_MOD_4_TILED_XE2_CCS Modifier for Xe2
URL : https://patchwork.freedesktop.org/series/133253/
State : warning
== Summary ==
Error: dim checkpatch failed
e953532ce0c8 drm/fourcc: define Intel Xe2 related tile4 ccs modifier
4ef786f4f7f8
== Series Details ==
Series: drm/xe: Nuke xe's copy of intel_fbdev_fb.h
URL : https://patchwork.freedesktop.org/series/133248/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14715 -> Patchwork_133248v1
Summary
---
== Series Details ==
Series: drm/xe: Nuke xe's copy of intel_fbdev_fb.h
URL : https://patchwork.freedesktop.org/series/133248/
State : warning
== Summary ==
Error: dim checkpatch failed
72b807c1045a drm/xe: Nuke xe's copy of intel_fbdev_fb.h
-:26: WARNING:FILE_PATH_CHANGES: added, moved or
== Series Details ==
Series: Revert "drm/i915: Remove extra multi-gt pm-references"
URL : https://patchwork.freedesktop.org/series/133245/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14715 -> Patchwork_133245v1
Summary
[cc += Thomas]
On Mon, May 06, 2024 at 08:43:43PM +0200, Johannes Stezenbach wrote:
> so, git bisect pointed to:
>
> commit 701d2054fa317188cd4039c84e72c73254013b23
> Author: Thomas Zimmermann
> Date: Tue Jun 13 13:07:13 2023 +0200
>
> fbdev: Make support for userspace interfaces
== Series Details ==
Series: Revert "drm/i915: Remove extra multi-gt pm-references"
URL : https://patchwork.freedesktop.org/series/133245/
State : warning
== Summary ==
Error: dim checkpatch failed
decb51cfa6e8 Revert "drm/i915: Remove extra multi-gt pm-references"
-:26:
Add I915_FORMAT_MOD_4_TILED_XE2_CCS to possible created tiling for new
framebuffer
on Xe driver.
Signed-off-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/display/intel_display.c | 1 +
drivers/gpu/drm/i915/display/intel_fb.c| 10 ++
Add I915_FORMAT_MOD_4_TILED_XE2_CCS to possible created tiling for new
framebuffer
on Xe driver.
Signed-off-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/xe/display/xe_plane_initial.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/xe/display/xe_plane_initial.c
Add Tile4 type ccs modifier to indicate presence of compression on Xe2
Signed-off-by: Juha-Pekka Heikkila
---
include/uapi/drm/drm_fourcc.h | 12
1 file changed, 12 insertions(+)
diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h
index
These patches introduce I915_FORMAT_MOD_4_TILED_XE2_CCS modifier, which,
from the kernel's perspective, behaves similarly to `I915_FORMAT_MOD_4_TILED`.
This new modifier is primarily intended for user space to effectively monitor
compression status, especially when dealing with a mix of compressed
Hi,
so, git bisect pointed to:
commit 701d2054fa317188cd4039c84e72c73254013b23
Author: Thomas Zimmermann
Date: Tue Jun 13 13:07:13 2023 +0200
fbdev: Make support for userspace interfaces configurable
Add Kconfig option CONFIG_FB_DEVICE and make the virtual fbdev
...
I think
From: Ville Syrjälä
For some reason xe and i915 each have an identical (fortunately)
copy of intel_fbdev_fb.h. The xe copy actually only gets included
by xe's intel_fbdev_fb.c, and the i915 copy by everyone else,
include intel_fbdev.c which is the actual caller of the
functions declared in the
On 5/6/2024 8:02 PM, Janusz Krzysztofik wrote:
This reverts commit 1f33dc0c1189efb9ae19c6fc22b64dd3e26261fb.
There was a patch supposed to fix an issue of illegal attempts to free a
still active i915 VMA object when parking a GT believed to be idle,
reported by CI on 2-GT Meteor Lake. As a
On Mon, May 06, 2024 at 07:51:47PM +0300, Ville Syrjälä wrote:
> On Mon, May 06, 2024 at 05:16:50PM +0300, Jani Nikula wrote:
> >
> > *return in subject
> >
> > On Mon, 06 May 2024, Ville Syrjala wrote:
> > > From: Ville Syrjälä
> > >
> > > Change intel_fbdev_fb_alloc() to return struct
This reverts commit 1f33dc0c1189efb9ae19c6fc22b64dd3e26261fb.
There was a patch supposed to fix an issue of illegal attempts to free a
still active i915 VMA object when parking a GT believed to be idle,
reported by CI on 2-GT Meteor Lake. As a solution, an extra wakeref for
a Primary GT was
On Wed, Apr 24, 2024 at 5:59 AM Kandpal, Suraj wrote:
>
>
>
> > -Original Message-
> > From: Sebastian Wick
> > Sent: Wednesday, April 24, 2024 2:51 AM
> > To: Kandpal, Suraj
> > Cc: intel-gfx@lists.freedesktop.org; Borah, Chaitanya Kumar
> > ; Shankar, Uma
> > ; Nautiyal, Ankit K ;
> >
On Mon, May 06, 2024 at 06:03:17AM -0400, Kumar, Naveen1 wrote:
>
>
> >-Original Message-
> >From: Vivi, Rodrigo
> >Sent: Saturday, April 6, 2024 3:39 AM
> >To: intel-gfx@lists.freedesktop.org
> >Cc: Vivi, Rodrigo ; Shankar, Uma
> >; Kulkarni, Vandita ;
> >Kumar, Naveen1 ; Nikula, Jani
On Mon, May 06, 2024 at 05:16:50PM +0300, Jani Nikula wrote:
>
> *return in subject
>
> On Mon, 06 May 2024, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Change intel_fbdev_fb_alloc() to return struct intel_fb instead
> > of struct drm_framebuffer. Let's us eliminate some annoying
> >
On Mon, May 06, 2024 at 05:03:59PM +0300, Jani Nikula wrote:
> On Mon, 06 May 2024, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > skl_plane_max_stride() is pretty messy. Streamline it and
> > split it into clear skl+ vs. adl+ variants.
> >
> > TODO: Deal with icl and tgl strude limits
On 5/3/2024 3:13 PM, Jakub Kicinski wrote:
> On Tue, 30 Apr 2024 17:38:09 + Easwar Hariharan wrote:
>> I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave"
>> with more appropriate terms. Inspired by and following on to Wolfram's
>> series to fix drivers/i2c/[1], fix
Hi Janusz,
On 4/16/2024 6:40 PM, Rodrigo Vivi wrote:
On Tue, Apr 16, 2024 at 10:09:46AM +0200, Janusz Krzysztofik wrote:
Hi Rodrigo,
On Tuesday, 16 April 2024 03:16:31 CEST Rodrigo Vivi wrote:
On Mon, Apr 15, 2024 at 09:53:09PM +0200, Janusz Krzysztofik wrote:
We defer actually closing,
*return in subject
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Change intel_fbdev_fb_alloc() to return struct intel_fb instead
> of struct drm_framebuffer. Let's us eliminate some annoying
> aliasing variables in the fbdev setup code.
You'll need to enable DRM_XE=m
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Make the 'fb' pointers const in the pinning code. We never
> want to mutate these. Also nuke a few aliasing fb vs. intel_fb
> cases by just using the more specific type everywhere in the
> same function.
I was half expecting to
== Series Details ==
Series: drm/i915/gt: Disarm breadcrumbs if engines are already idle (rev3)
URL : https://patchwork.freedesktop.org/series/132786/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14711 -> Patchwork_132786v3
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Be a bit more consistent in our use of integer types in
> the fb related calculatiosn. u32 we generally only use
> for ggtt offsets and such, and everything else can be regular
> (unsigned) ints.
>
> There's also an
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Pull the "does this plane need a physical address?" check into
> a small helper.
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8
>
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Fence regions are only relevant for GGTT, not DPT. Drop the
> pointless 'uses_fence' argument from intel_pin_fb_obj_dpt().
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
>
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> skl_plane_max_stride() is pretty messy. Streamline it and
> split it into clear skl+ vs. adl+ variants.
>
> TODO: Deal with icl and tgl strude limits properly
>
> Signed-off-by: Ville Syrjälä
> ---
>
== Series Details ==
Series: drm/i915/gt: Disarm breadcrumbs if engines are already idle (rev3)
URL : https://patchwork.freedesktop.org/series/132786/
State : warning
== Summary ==
Error: dim checkpatch failed
fe6b28d83f8d drm/i915/gt: Disarm breadcrumbs if engines are already idle
-:15:
On Mon, 06 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Plane .max_stride() is alreayd a vfunc so having one made
*already
> up of two branches based on the display version is silly.
> Split i9xx_plane_max_stride() into gen2 vs. gen3 variants
> so that we get rid of said check.
== Series Details ==
Series: drm/i915: Plane fb refactoring
URL : https://patchwork.freedesktop.org/series/133231/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14711 -> Patchwork_133231v1
Summary
---
**SUCCESS**
== Series Details ==
Series: drm/i915: Plane fb refactoring
URL : https://patchwork.freedesktop.org/series/133231/
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:
From: Ville Syrjälä
Rename the fb pinning functions such that their name directly
informs us what gets pinned into which address space.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_dpt.c | 6 +--
drivers/gpu/drm/i915/display/intel_dpt.h | 6 +--
From: Ville Syrjälä
We use a mix of 'fb' vs. 'ifbdev->fb' in the same function.
Both should be pointing at the same thing. Make things less
confusing by just getting existing fb from 'ifbdev->fb' at the
start and then sticking with the local 'fb' until the very end.
And we'll also change
From: Ville Syrjälä
Change intel_fbdev_fb_alloc() to return struct intel_fb instead
of struct drm_framebuffer. Let's us eliminate some annoying
aliasing variables in the fbdev setup code.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_fbdev.c| 10 +-
From: Ville Syrjälä
Make the 'fb' pointers const in the pinning code. We never
want to mutate these. Also nuke a few aliasing fb vs. intel_fb
cases by just using the more specific type everywhere in the
same function.
Signed-off-by: Ville Syrjälä
---
From: Ville Syrjälä
Be a bit more consistent in our use of integer types in
the fb related calculatiosn. u32 we generally only use
for ggtt offsets and such, and everything else can be regular
(unsigned) ints.
There's also an overabundance of consts for local variables
in
From: Ville Syrjälä
Pull the "does this plane need a physical address?" check into
a small helper.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8
drivers/gpu/drm/i915/display/intel_atomic_plane.h | 1 +
From: Ville Syrjälä
Fence regions are only relevant for GGTT, not DPT. Drop the
pointless 'uses_fence' argument from intel_pin_fb_obj_dpt().
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_fb_pin.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
From: Ville Syrjälä
skl_plane_max_stride() is pretty messy. Streamline it and
split it into clear skl+ vs. adl+ variants.
TODO: Deal with icl and tgl strude limits properly
Signed-off-by: Ville Syrjälä
---
.../drm/i915/display/skl_universal_plane.c| 65 +++
1 file
From: Ville Syrjälä
Plane .max_stride() is alreayd a vfunc so having one made
up of two branches based on the display version is silly.
Split i9xx_plane_max_stride() into gen2 vs. gen3 variants
so that we get rid of said check.
Signed-off-by: Ville Syrjälä
---
From: Ville Syrjälä
A bit of cleanup/refactoring around plane fb stuff.
This is mainly prep work for a slightly bigger rework
of alignment handling.
Ville Syrjälä (9):
drm/i915: Split gen2 vs. gen3 .max_stride()
drm/i915: Clean up skl+ plane stride limits
drm/i915: Drop 'uses_fence'
On Sat, May 04, 2024 at 04:29:48PM -0700, Niranjana Vishwanathapura wrote:
please use intel...@lists.freedesktop.org for patches touching only xe.
probably a good idea to use some gitconfig identities:
[sendemail "i915"]
to = intel-gfx@lists.freedesktop.org
[sendemail "xe"]
to =
On Mon, May 06, 2024 at 12:15:13PM +, Murthy, Arun R wrote:
>
> > -Original Message-
> > From: Ville Syrjälä
> > Sent: Monday, May 6, 2024 5:36 PM
> > To: Murthy, Arun R
> > Cc: intel-gfx@lists.freedesktop.org; Nikula, Jani
> > Subject: Re: [PATCH] drm/i915/display/dp: Remove
> -Original Message-
> From: Ville Syrjälä
> Sent: Monday, May 6, 2024 5:36 PM
> To: Murthy, Arun R
> Cc: intel-gfx@lists.freedesktop.org; Nikula, Jani
> Subject: Re: [PATCH] drm/i915/display/dp: Remove UHBR13.5 for icl
>
> On Mon, May 06, 2024 at 02:36:25PM +0530, Arun R Murthy
On Mon, May 06, 2024 at 01:09:02PM +0300, Jani Nikula wrote:
> Avoid the implicit dev_priv local variable use, and pass dev_priv
> explicitly to the PIPE_CRC_CTL register macro.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/display/intel_pipe_crc.c |
On Mon, May 06, 2024 at 01:09:03PM +0300, Jani Nikula wrote:
> Avoid the implicit dev_priv local variable use, and pass dev_priv
> explicitly to the PIPE_CRC_RES_* register macros.
>
> Signed-off-by: Jani Nikula
> ---
> .../gpu/drm/i915/display/intel_display_irq.c | 24 ++-
>
On Mon, May 06, 2024 at 01:09:04PM +0300, Jani Nikula wrote:
> Avoid the implicit dev_priv local variable use, and pass dev_priv
> explicitly to the PORT_DFT2_G4X register macro.
Looks like we only use this on VLV/CHV, so could define a fixed
offset _VLV variant.
But this works too
Reviewed-by:
On Mon, May 06, 2024 at 02:36:25PM +0530, Arun R Murthy wrote:
> UHBR13.5 is not supported in icl+ as well. This has been removed for mtl
> by the commit caf3d748f646 ("drm/i915/dp: Remove support for UHBR13.5")
icl/tgl/adl don't support UHBR at all. The numbers are
there for dg2 presumably,
== Series Details ==
Series: drm/xe: More fb pinning optimizations.
URL : https://patchwork.freedesktop.org/series/133228/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14711 -> Patchwork_133228v1
Summary
---
On Mon, May 06, 2024 at 12:42:03PM +0300, Jani Nikula wrote:
> On Fri, 03 May 2024, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Define the contents of VBT block 57 (Vswing PreEmphasis Table).
> >
> > The contents is highly platform specific. The columns of the
> > table corresponding to
== Series Details ==
Series: drm/xe: More fb pinning optimizations.
URL : https://patchwork.freedesktop.org/series/133228/
State : warning
== Summary ==
Error: dim checkpatch failed
607761f79c90 drm/xe: Remove safety check from
__xe_ttm_stolen_io_mem_reserve_stolen
280e9e445c55
On Mon, May 06, 2024 at 12:19:01PM +0300, Jani Nikula wrote:
> On Fri, 03 May 2024, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > The LFP data applies to all kinds of display interfaces, so
> > stop calling things by the "LVDS" name.
> >
> > Signed-off-by: Ville Syrjälä
>
> Reviewed-by:
== Series Details ==
Series: drm/i915: Implement Audio WA_14020863754 (rev3)
URL : https://patchwork.freedesktop.org/series/132255/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14711 -> Patchwork_132255v3
Summary
---
== Series Details ==
Series: drm/i915/display: remove implicit dev_priv from pipe crc
URL : https://patchwork.freedesktop.org/series/133224/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14711 -> Patchwork_133224v1
Summary
i915 has this really nice, infrastructure where everything becomes
complicated, GGTT needs eviction, etc..
Lets not do that, and make the dumbest possible interface instead.
Try to retrieve the VMA from old_plane_state, or intel_fbdev if kernel
fb.
Signed-off-by: Maarten Lankhorst
---
Preallocate a DPT when creating the FB, and store it in
i915_address_space. This can be used to prevent an expensive
allocation in the pinning path.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/xe/display/xe_fb_pin.c | 33 +++---
1 file changed, 19 insertions(+), 14
The DPT bo should not be allocated when pinning, but in advance when
creating the framebuffer. Split allocation from bo pinning and GGTT
insertion.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/xe/display/xe_fb_pin.c | 159 +++--
1 file changed, 123 insertions(+), 36
This is invalid with display code when reworking DPT pinning.
The only reason we added it, was because originally all display
allocations also had the bit set.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c | 4
1 file changed, 4 deletions(-)
diff --git
This reduces the latency of pinning framebuffers by
re-using the previous mapping, if available.
Additionally, DPT is preallocated when creating the FB, instead
of performing a bo allocation on every pin.
No changes since v3, just a rebase from conflicts.
Maarten Lankhorst (4):
drm/xe: Remove
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 -
Avoid the implicit dev_priv local variable use, and pass dev_priv
explicitly to the PORT_DFT2_G4X register macro.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_pipe_crc.c | 8
drivers/gpu/drm/i915/i915_reg.h | 2 +-
2 files changed, 5 insertions(+), 5
Avoid the implicit dev_priv local variable use, and pass dev_priv
explicitly to the PIPE_CRC_RES_* register macros.
Signed-off-by: Jani Nikula
---
.../gpu/drm/i915/display/intel_display_irq.c | 24 ++-
drivers/gpu/drm/i915/i915_reg.h | 22 -
2
Avoid the implicit dev_priv local variable use, and pass dev_priv
explicitly to the PIPE_CRC_CTL register macro.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_pipe_crc.c | 12 ++--
drivers/gpu/drm/i915/i915_reg.h | 2 +-
2 files changed, 7
Continue removing implicit dev_priv local variable use.
Jani Nikula (3):
drm/i915: pass dev_priv explicitly to PIPE_CRC_CTL
drm/i915: pass dev_priv explicitly to PIPE_CRC_*
drm/i915: pass dev_priv explicitly to PORT_DFT2_G4X
.../gpu/drm/i915/display/intel_display_irq.c | 24
>-Original Message-
>From: Vivi, Rodrigo
>Sent: Saturday, April 6, 2024 3:39 AM
>To: intel-gfx@lists.freedesktop.org
>Cc: Vivi, Rodrigo ; Shankar, Uma
>; Kulkarni, Vandita ;
>Kumar, Naveen1 ; Nikula, Jani
>; Belgaumkar, Vinay
>Subject: [PATCH] drm/i915/display: Introduce Display
== Series Details ==
Series: drm/i915/display/dp: Remove UHBR13.5 for icl
URL : https://patchwork.freedesktop.org/series/133216/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14711 -> Patchwork_133216v1
Summary
---
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> I got curious about what gems (or turds) might be hiding
> inside the BDB blocks we aren't parsing. So I undertook the
> effort to dig up the definition for pretty much all of them.
>
> Unfortunately I didn't find anything
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 253 (PRD Table).
>
> Unfortunately the block has two definitions, with the cutoff
> supposedly happening on ICL vs. TGL. Also according to some
> notes it might be that the VBIOS (if that's still
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 50 (MIPI).
>
> This was some easly attempt at a MIPI DSI stuff. I'm not sure
> this was ever actually used (I certainly don't have any VBTs
> with this block), but here's some kind of definition
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 57 (Vswing PreEmphasis Table).
>
> The contents is highly platform specific. The columns of the
> table corresponding to some set of PHY/etc registers. The rows
> corresponding to all legal
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 55 (RGB Palette Table).
>
> Note that I've not actually seen any real world VBTs with this
> block.
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
>
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 51 (Fixed Set Mode Table).
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 11 +++
> 1 file changed, 11
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 46 (Chromaticity For Narrow Gamut
> Panel). One entry per panel.
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
but...
> ---
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 26
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 28 (EFP DTD).
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 26 (TV Options).
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 12
> 1 file changed, 12 insertions(+)
>
>
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 20 (OEM Customizable Modes).
> Each entry is either 26 or 28 bytes, depending on the BDB version.
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
>
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of block 18 (Driver Rotation).
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 12
> 1 file changed, 12 insertions(+)
>
>
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 12 (Driver Persistent Algorithm).
>
> Signed-off-by: Ville Syrjälä
Baffled.
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 19 +++
> 1 file
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 10 (Mode Removal Table).
>
> There seem to be two variants:
> - 8 byte entries for desktop systems
> - 10 byte entries for mobile systems, with the extra
> panel_flags being a bitmask of LFPs
>
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents for VBT blocks:
> - Block 6 (Extended MMIO Register Table)
> - Block 7 (IO Software Flag Table)
> - Block 8 (MMIO SWF Register Table)
>
> All of these use the same basic layout, with two known variants:
> -
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Define the contents of VBT block 4 (Mode Support List).
>
> Slightly crazy layout with a variable length list at the start,
> followed by the length of said list.
>
> No real idea what these "Intel mode numbers" really are. What
On Fri, 03 May 2024, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Document which VBT blocks were defined in which BDB version,
> for the cases where the spec actually states this accurately.
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Jani Nikula
> ---
>
1 - 100 of 112 matches
Mail list logo