== Series Details ==
Series: Remove unnecessary GuC err capture noise
URL : https://patchwork.freedesktop.org/series/103709/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11620 -> Patchwork_103709v1
Summary
---
This series remove unnecessary GuC err capture noise.
Alan Previn (1):
drm/i915/guc: Remove unnecessary GuC err capture noise
.../gpu/drm/i915/gt/uc/intel_guc_capture.c| 77 +--
1 file changed, 2 insertions(+), 75 deletions(-)
--
2.25.1
GuC error capture blurts some debug messages about empty
register lists for certain register types on engines during
firmware initialization.
These are not errors or warnings, so get rid of them.
Signed-off-by: Alan Previn
---
.../gpu/drm/i915/gt/uc/intel_guc_capture.c| 77
On Thu, Apr 28, 2022 at 01:18:42PM +0100, Tvrtko Ursulin wrote:
>
> Hi,
>
> On 28/04/2022 00:07, Matt Roper wrote:
> > Rather than storing subslice masks internally as u8[] (inside the sseu
> > structure) and u32 (everywhere else), let's move over to using an
> > intel_sseu_ss_mask_t typedef
== Series Details ==
Series: series starting with [1/2] drm/i915/dmabuf: dmabuf cleanup
URL : https://patchwork.freedesktop.org/series/103696/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11618_full -> Patchwork_103696v1_full
== Series Details ==
Series: drm/i915/dmc: Add MMIO range restrictions
URL : https://patchwork.freedesktop.org/series/103693/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11618_full -> Patchwork_103693v1_full
Summary
== Series Details ==
Series: drm/i915/dmc: Load DMC on DG2 (rev4)
URL : https://patchwork.freedesktop.org/series/103625/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11618_full -> Patchwork_103625v4_full
Summary
---
On Fri, May 06, 2022 at 10:13:28AM -0700, Umesh Nerlige Ramappa wrote:
On Wed, May 04, 2022 at 07:09:09PM +0100, Tvrtko Ursulin wrote:
On 04/05/2022 18:35, Umesh Nerlige Ramappa wrote:
On Wed, May 04, 2022 at 09:10:42AM +0100, Tvrtko Ursulin wrote:
On 03/05/2022 20:49, Umesh Nerlige Ramappa
== Series Details ==
Series: series starting with [1/2] drm/i915/dmabuf: dmabuf cleanup
URL : https://patchwork.freedesktop.org/series/103696/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11618 -> Patchwork_103696v1
Update open coded for loop to use the standard scatterlist
for_each_sg API.
Cc: Tvrtko Ursulin
Signed-off-by: Michael J. Ruhl
---
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
Some minor cleanup of some variables for consistency.
Normalize struct sg_table to sgt.
Normalize struct dma_buf_attachment to attach.
checkpatch issues sizeof(), !NULL updates.
Cc: Tvrtko Ursulin
Signed-off-by: Michael J. Ruhl
---
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 47
While studying this code I noticed a few inconsistencies and possible
cleanups.
Please consider these patches.
On Fri, 2022-05-06 at 18:28 +, Hogander, Jouni wrote:
> On Fri, 2022-05-06 at 15:29 +, Souza, Jose wrote:
> > On Fri, 2022-05-06 at 08:48 +0300, Jouni Högander wrote:
> > > Currently we have some corner cases where area calculation
> > > fails. For
> > > these sel fetch are calculation
== Series Details ==
Series: Expose max and current bpc via debugfs (rev4)
URL : https://patchwork.freedesktop.org/series/102502/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617_full -> Patchwork_102502v4_full
Summary
== Series Details ==
Series: drm/i915/dmc: Add MMIO range restrictions
URL : https://patchwork.freedesktop.org/series/103693/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11618 -> Patchwork_103693v1
Summary
---
On Fri, 2022-05-06 at 15:29 +, Souza, Jose wrote:
> On Fri, 2022-05-06 at 08:48 +0300, Jouni Högander wrote:
> > Currently we have some corner cases where area calculation
> > fails. For
> > these sel fetch are calculation ends up having update area as y1 =
> > 0,
> > y2 = 4. Instead of these
== Series Details ==
Series: drm/i915/dmc: Add MMIO range restrictions
URL : https://patchwork.freedesktop.org/series/103693/
State : warning
== Summary ==
Error: dim checkpatch failed
dfd1ba4479c1 drm/i915/dmc: Add MMIO range restrictions
-:77: WARNING:LONG_LINE: line length of 101 exceeds
== Series Details ==
Series: drm/i915/dmc: Load DMC on DG2 (rev4)
URL : https://patchwork.freedesktop.org/series/103625/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11618 -> Patchwork_103625v4
Summary
---
On Fri, May 06, 2022 at 01:10:24PM +0300, Jani Nikula wrote:
> +int drm_edid_to_sad(const struct edid *edid, struct cea_sad **sads)
> +{
> + struct drm_edid drm_edid = {
> + .edid = edid,
> + .size = edid_size(edid),
> + };
> +
> + return _drm_edid_to_sad(_edid,
On Fri, May 06, 2022 at 01:10:09PM +0300, Jani Nikula wrote:
> We have an iterator for this, use it. It does include the base block,
> but its tag is 0 and will be skipped.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/drm_edid.c | 8 +---
> 1 file
On Fri, May 06, 2022 at 01:10:08PM +0300, Jani Nikula wrote:
> Only one of the conditions can be true.
>
> Suggested-by: Ville Syrjälä
> Signed-off-by: Jani Nikula
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/drm_edid.c | 12 ++--
> 1 file changed, 6 insertions(+), 6
Bspec has added some steps that check forDMC MMIO range before
programming them
v2: Fix for CI
v3: move register defines to .h (Anusha)
- Check MMIO restrictions per pipe
- Add MMIO restricton for v1 dmc header as well (Lucas)
v4: s/_PICK/_PICK_EVEN and use it only for Pipe DMC scenario.
- clean
On Fri, May 06, 2022 at 10:23:41AM -0700, Lucas De Marchi wrote:
> On Thu, May 05, 2022 at 02:38:05PM -0700, Matt Roper wrote:
> > From: Stuart Summers
> >
> > Although we already strip 3D-specific flags from PIPE_CONTROL
> > instructions when submitting to a compute engine, there are some
> >
On Thu, May 05, 2022 at 02:38:05PM -0700, Matt Roper wrote:
From: Stuart Summers
Although we already strip 3D-specific flags from PIPE_CONTROL
instructions when submitting to a compute engine, there are some
additional flags that need to be removed when the platform as a whole
lacks a 3D
== Series Details ==
Series: series starting with [1/2] drm/i915/display: Add smem fallback
allocation for dpt
URL : https://patchwork.freedesktop.org/series/103678/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617_full -> Patchwork_103678v1_full
Add Support for DC states on Dg2.
v2: Add dc9 as the max supported DC states and disable DC5.
v3: set max_dc to 0. (Imre)
v4: Add FIXME (Rodrigo)
Cc: Imre Deak
Cc: Rodrigo Vivi
Signed-off-by: Anusha Srivatsa
Reviewed-by: Rodrigo Vivi (v1)
---
While DG2 supports DC5 and DC9, some of the tests in
fast-feedback blew up DG2 when the tests forced transition
from dc5->dc9 on suspend and dc9->dc5 on resume. Some local
experiments performed with Rodrigo on a RIL system showed promising
results when dc5 was completely diabled and i915 took
On Wed, May 04, 2022 at 07:09:09PM +0100, Tvrtko Ursulin wrote:
On 04/05/2022 18:35, Umesh Nerlige Ramappa wrote:
On Wed, May 04, 2022 at 09:10:42AM +0100, Tvrtko Ursulin wrote:
On 03/05/2022 20:49, Umesh Nerlige Ramappa wrote:
On Tue, May 03, 2022 at 09:42:52AM +0100, Tvrtko Ursulin wrote:
On Thu, May 05, 2022 at 02:38:03PM -0700, Matt Roper wrote:
From: Ayaz A Siddiqui
v2 (MattR):
- Clarify comment above RING_CMD_CCTL programming.
- Remove bspec reference from field definition. (Lucas)
- Add WARN if we try to use a (presumably uninitialized) wb_index of 0.
On most platforms
On 06.05.2022 15:22, Javier Martinez Canillas wrote:
Commit d258d00fb9c7 ("fbdev: efifb: Cleanup fb_info in .fb_destroy rather
than .remove") attempted to fix a use-after-free error due driver freeing
the fb_info in the .remove handler instead of doing it in .fb_destroy.
But ironically that
== Series Details ==
Series: drm/i915/pxp: fix sparse warning for not declared symbol
URL : https://patchwork.freedesktop.org/series/103673/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617_full -> Patchwork_103673v1_full
On 5/6/2022 00:18, Tvrtko Ursulin wrote:
On 05/05/2022 19:36, John Harrison wrote:
On 5/5/2022 10:21, Belgaumkar, Vinay wrote:
On 5/5/2022 5:13 AM, Tvrtko Ursulin wrote:
On 05/05/2022 06:40, Vinay Belgaumkar wrote:
SLPC min/max frequency updates require H2G calls. We are seeing
timeouts when
On 5/6/2022 12:18 AM, Tvrtko Ursulin wrote:
On 05/05/2022 19:36, John Harrison wrote:
On 5/5/2022 10:21, Belgaumkar, Vinay wrote:
On 5/5/2022 5:13 AM, Tvrtko Ursulin wrote:
On 05/05/2022 06:40, Vinay Belgaumkar wrote:
SLPC min/max frequency updates require H2G calls. We are seeing
== Series Details ==
Series: drm/i915: Eliminate PIPECONF RMWs from .color_commit() (rev4)
URL : https://patchwork.freedesktop.org/series/102666/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617_full -> Patchwork_102666v4_full
== Series Details ==
Series: Expose max and current bpc via debugfs (rev4)
URL : https://patchwork.freedesktop.org/series/102502/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617 -> Patchwork_102502v4
Summary
---
== Series Details ==
Series: Expose max and current bpc via debugfs (rev4)
URL : https://patchwork.freedesktop.org/series/102502/
State : warning
== Summary ==
Error: dim checkpatch failed
e8b34c163d1c drm/debug: Expose connector's max supported bpc via debugfs
-:21: WARNING:BAD_SIGN_OFF:
Hi Javier,
On Fri, May 06, 2022 at 03:22:25PM +0200, Javier Martinez Canillas wrote:
> Commit d258d00fb9c7 ("fbdev: efifb: Cleanup fb_info in .fb_destroy rather
> than .remove") attempted to fix a use-after-free error due driver freeing
> the fb_info in the .remove handler instead of doing it in
On Fri, 2022-05-06 at 08:48 +0300, Jouni Högander wrote:
> Current update area calculation is not handling situation where
> e.g. cursor plane is fully or partially outside pipe area.
>
> Fix this by checking damage area against pipe_src area using
> drm_rect_intersect.
>
> Closes:
On 5/6/2022 12:51 AM, Tvrtko Ursulin wrote:
On 05/05/2022 19:56, John Harrison wrote:
On 5/4/2022 16:46, Daniele Ceraolo Spurio wrote:
From: Matthew Brost
In GuC submission mode the EU priority must be updated by the GuC
rather
than the driver as the GuC owns the programming of the
On Fri, 2022-05-06 at 08:48 +0300, Jouni Högander wrote:
> Currently we have some corner cases where area calculation fails. For
> these sel fetch are calculation ends up having update area as y1 = 0,
> y2 = 4. Instead of these values safer option is full update.
Aren't you able to reproduce
Hi Jani,
On Fri, May 06, 2022 at 02:53:50PM +0300, Jani Nikula wrote:
> On Sat, 19 Mar 2022, Andi Shyti wrote:
> > +#define INTEL_GT_RPS_SYSFS_ATTR(_name, _mode, _show, _store) \
> > + struct device_attribute dev_attr_gt_##_name = __ATTR(gt_##_name, _mode,
> > _show, _store); \
> > + struct
== Series Details ==
Series: fbdev: efifb: Fix a use-after-free due early fb_info cleanup
URL : https://patchwork.freedesktop.org/series/103680/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11617 -> Patchwork_103680v1
On Fri, May 06, 2022 at 08:21:46AM +0100, Tvrtko Ursulin wrote:
>
> On 05/05/2022 21:59, Matt Roper wrote:
> > On Tue, May 03, 2022 at 09:05:43AM +0100, Tvrtko Ursulin wrote:
> > >
> > > On 02/05/2022 17:34, Matt Roper wrote:
> > > > This patch adds the basic definitions needed to support
> > >
== Series Details ==
Series: series starting with [1/2] drm/i915/display: Add smem fallback
allocation for dpt
URL : https://patchwork.freedesktop.org/series/103678/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617 -> Patchwork_103678v1
== Series Details ==
Series: series starting with [1/2] drm/i915/display: Add smem fallback
allocation for dpt
URL : https://patchwork.freedesktop.org/series/103678/
State : warning
== Summary ==
Error: dim checkpatch failed
cb0cd39c87aa drm/i915/display: Add smem fallback allocation for dpt
== Series Details ==
Series: drm/i915/pxp: fix sparse warning for not declared symbol
URL : https://patchwork.freedesktop.org/series/103673/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617 -> Patchwork_103673v1
Summary
== Series Details ==
Series: Expose max and current bpc via debugfs (rev3)
URL : https://patchwork.freedesktop.org/series/102502/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11616_full -> Patchwork_102502v3_full
Summary
== Series Details ==
Series: drm/i915/pxp: fix sparse warning for not declared symbol
URL : https://patchwork.freedesktop.org/series/103673/
State : warning
== Summary ==
Error: dim checkpatch failed
151a8d341bdd drm/i915/pxp: fix sparse warning for not declared symbol
-:4:
== Series Details ==
Series: drm/i915: Eliminate PIPECONF RMWs from .color_commit() (rev4)
URL : https://patchwork.freedesktop.org/series/102666/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11617 -> Patchwork_102666v4
Am 06.05.22 um 15:22 schrieb Javier Martinez Canillas:
Commit d258d00fb9c7 ("fbdev: efifb: Cleanup fb_info in .fb_destroy rather
than .remove") attempted to fix a use-after-free error due driver freeing
the fb_info in the .remove handler instead of doing it in .fb_destroy.
But ironically that
== Series Details ==
Series: Fixes for selective fetch area calculation
URL : https://patchwork.freedesktop.org/series/103659/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11616_full -> Patchwork_103659v1_full
Summary
Commit d258d00fb9c7 ("fbdev: efifb: Cleanup fb_info in .fb_destroy rather
than .remove") attempted to fix a use-after-free error due driver freeing
the fb_info in the .remove handler instead of doing it in .fb_destroy.
But ironically that change introduced yet another use-after-free since the
From: CQ Tang
Display might allocate a smem object and call
i915_vma_pin_iomap(), the existing code will fail.
This fix was suggested by Chris P Wilson, that we pin
the smem with i915_gem_object_pin_map_unlocked().
Signed-off-by: CQ Tang
Signed-off-by: Juha-Pekka Heikkila
Cc: Chris Wilson
Add fallback smem allocation for dpt if stolen memory allocation failed.
Signed-off-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/display/intel_dpt.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dpt.c
== Series Details ==
Series: drm/edid: introduce struct drm_edid (rev2)
URL : https://patchwork.freedesktop.org/series/103665/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11616 -> Patchwork_103665v2
Summary
---
== Series Details ==
Series: drm/edid: introduce struct drm_edid (rev2)
URL : https://patchwork.freedesktop.org/series/103665/
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: introduce struct drm_edid (rev2)
URL : https://patchwork.freedesktop.org/series/103665/
State : warning
== Summary ==
Error: dim checkpatch failed
49478b13e157 drm/edid: use else-if in CTA extension parsing
253ce252ebb2 drm/edid: convert
Fix:
drivers/gpu/drm/i915/pxp/intel_pxp_debugfs.c:61:6: warning: symbol
'intel_pxp_debugfs_register' was not declared. Should it be static?
Sort and remove the redundant pxp prefixes from the includes while at
it.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/pxp/intel_pxp_debugfs.c | 5
On Fri, 15 Apr 2022, Imre Deak wrote:
> Atm the port -> DDI and AUX power domain mapping is specified by relying
> on the aliasing of the platform specific intel_display_power_domain enum
> values. For instance D12+ platforms refer to the 'D' port and power
> domain instances, which doesn't match
On Sat, 19 Mar 2022, Andi Shyti wrote:
> +#define INTEL_GT_RPS_SYSFS_ATTR(_name, _mode, _show, _store) \
> + struct device_attribute dev_attr_gt_##_name = __ATTR(gt_##_name, _mode,
> _show, _store); \
> + struct device_attribute dev_attr_rps_##_name = __ATTR(rps_##_name,
> _mode, _show,
Hey, do you ever run sparse on your driver? Whenever our CI ends up
recompiling amdgpu, there's quite the spew. See below.
You're not alone, but, sorry to say, the _Static_assert() from
amdgv_sriovmsg.h has been pretty obnoxious for quite some time now.
First, I don't think you should be using
== Series Details ==
Series: drm/edid: introduce struct drm_edid
URL : https://patchwork.freedesktop.org/series/103665/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11616 -> Patchwork_103665v1
Summary
---
== Series Details ==
Series: drm/edid: introduce struct drm_edid
URL : https://patchwork.freedesktop.org/series/103665/
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: introduce struct drm_edid
URL : https://patchwork.freedesktop.org/series/103665/
State : warning
== Summary ==
Error: dim checkpatch failed
d886c554f1a8 drm/edid: use else-if in CTA extension parsing
4cbaa4d285a2 drm/edid: convert
== Series Details ==
Series: Expose max and current bpc via debugfs (rev3)
URL : https://patchwork.freedesktop.org/series/102502/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11616 -> Patchwork_102502v3
Summary
---
== Series Details ==
Series: Expose max and current bpc via debugfs (rev3)
URL : https://patchwork.freedesktop.org/series/102502/
State : warning
== Summary ==
Error: dim checkpatch failed
a180df33d330 drm/debug: Expose connector's max supported bpc via debugfs
-:21: WARNING:BAD_SIGN_OFF:
Hi Dave & Daniel -
drm-intel-next-2022-05-06:
drm/i915 feature pull #2 for v5.19:
Features and functionality:
- Add first set of DG2 PCI IDs for "motherboard down" designs (Matt Roper)
- Add initial RPL-P PCI IDs as ADL-P subplatform (Matt Atwood)
Refactoring and cleanups:
- Power well
== Series Details ==
Series: Fixes for selective fetch area calculation
URL : https://patchwork.freedesktop.org/series/103659/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11616 -> Patchwork_103659v1
Summary
---
On Fri, 06 May 2022, Jani Nikula wrote:
> We'll need to propagate drm_edid everywhere.
I seem to have copy-pasted a TAB in some of the commit messages, in a
way that does not show up in git log.
>
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/drm_edid.c | 50
We'll need to propagate drm_edid everywhere. Also make version_greater()
a function for type safety.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_displayid.c | 16
drivers/gpu/drm/drm_edid.c | 17 ++---
include/drm/drm_displayid.h | 6 +++---
include/drm/drm_edid.h | 6 --
4 files changed,
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index b453bbf2f642..82db7afb4f8e 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 82a242a71ecf..b453bbf2f642 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 21 +++--
1 file changed, 11 insertions(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 706552ae00ea..82a242a71ecf 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 40 --
1 file changed, 25 insertions(+), 15 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 36 +++-
1 file changed, 23 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 39 --
1 file changed, 25 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 37 -
1 file changed, 24 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 36 ++--
1 file changed, 18 insertions(+), 18 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
v2: Drop incorrect NULL name check (Dan Carpenter)
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 24
1 file changed, 16 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 27 ++-
1 file changed, 14 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 4b2d3453fc5c..b8deced8de01
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 52 --
1 file changed, 27 insertions(+), 25 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 3931c3318cf7..27655da6ef9c 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 88bb6c7ac97c..3931c3318cf7 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index b5d7347f32d9..88bb6c7ac97c 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index fdc6ad651d3f..1abdd88ff64b
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 1abdd88ff64b..b5d7347f32d9 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 95 ++
1 file changed, 46 insertions(+), 49 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index c9d48fbd0a76..74038be72ad3 100644
---
We'll need to propagate drm_edid everywhere.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 50 +++---
1 file changed, 31 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
We'll need to propagate drm_edid everywhere. This is a clunky start, but
a start nonetheless. We'll eventually convert all of the EDID parsing to
struct drm_edid.
Initially, we'll just create the struct drm_edid in stack. This will be
the compat layer for legacy struct edid code. In the future,
Introduce new opaque type struct drm_edid to encapsulate the EDID data
and the size allocated for it. The contents will be private to
drm_edid.c.
There are a number of reasons for adding a container around struct edid:
* struct edid is a raw blob pointer to data that usually originates
outside
We have an iterator for this, use it. It does include the base block,
but its tag is 0 and will be skipped.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c
Only one of the conditions can be true.
Suggested-by: Ville Syrjälä
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_edid.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index
Introduce struct drm_edid and start converting struct edid * to struct
drm_edid * in the EDID parsing. The rationale for drm_edid is explained
in depth in commit message for patch 3.
Initially, there's little functional benefit in the series, it's mostly
just prep work. It's also intended to be
== Series Details ==
Series: drm/i915/guc/rc: Use i915_probe_error instead of drm_error
URL : https://patchwork.freedesktop.org/series/103656/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11614_full -> Patchwork_103656v1_full
Hi YueHaibing,
On Fri, May 06, 2022 at 11:26:52AM +0800, YueHaibing wrote:
> drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c: In function ‘act_freq_mhz_show’:
> drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c:276:20: error: implicit
> declaration of function ‘sysfs_gt_attribute_r_max_func’
>
== Series Details ==
Series: drm/i915/gt: Fix build error without CONFIG_PM
URL : https://patchwork.freedesktop.org/series/103654/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11614_full -> Patchwork_103654v1_full
Summary
On 06/05/2022 04:26, YueHaibing wrote:
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c: In function ‘act_freq_mhz_show’:
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c:276:20: error: implicit declaration
of function ‘sysfs_gt_attribute_r_max_func’
[-Werror=implicit-function-declaration]
276 |
On 05/05/2022 19:56, John Harrison wrote:
On 5/4/2022 16:46, Daniele Ceraolo Spurio wrote:
From: Matthew Brost
In GuC submission mode the EU priority must be updated by the GuC rather
than the driver as the GuC owns the programming of the context
descriptor.
Given that the GuC code uses
1 - 100 of 103 matches
Mail list logo
