Re: [Intel-gfx] [PATCH v2 0/7] drm: make headers self-contained and drop drmP.h

Hi Daniel. On Mon, May 27, 2019 at 08:18:35AM +0200, Daniel Vetter wrote: > On Sun, May 26, 2019 at 07:35:28PM +0200, Sam Ravnborg wrote: > > While removing use of drmP.h from files in drm/* I > > noticed that I had to add the same include files due to > > dependencies in the header files. > > > >

Re: [Intel-gfx] [PATCH 11/33] fbdev/sh_mobile: remove sh_mobile_lcdc_display_notify

On Mon, May 27, 2019 at 08:13:06AM +0200, Daniel Vetter wrote: > On Sat, May 25, 2019 at 05:01:59PM +0200, Sam Ravnborg wrote: > > Hi Daniel > > > > > It's dead code, and removing it avoids me having to understand > > > what it's doing with lock_fb_info. > > > > I pushed the series through my bui

Re: [Intel-gfx] [PATCH 09/33] fbcon: Remove fbcon_has_exited

Hi Daniel. > But I indeed forgot > to delete the initial assignment of info at the function start. Is that > what you mean here? Yes. Sam ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/icl: Fix AUX-B HW not done issue w/o AUX-A

== Series Details == Series: drm/i915/icl: Fix AUX-B HW not done issue w/o AUX-A URL : https://patchwork.freedesktop.org/series/61123/ State : failure == Summary == CI Bug Log - changes from CI_DRM_6142_full -> Patchwork_13095_full Summary

Re: [Intel-gfx] [PATCH v2 0/7] drm: make headers self-contained and drop drmP.h

On Sun, May 26, 2019 at 07:35:28PM +0200, Sam Ravnborg wrote: > While removing use of drmP.h from files in drm/* I > noticed that I had to add the same include files due to > dependencies in the header files. > > It is better to let the header files be self-contained and > let the users pull in on

Re: [Intel-gfx] [PATCH 11/33] fbdev/sh_mobile: remove sh_mobile_lcdc_display_notify

On Sat, May 25, 2019 at 05:01:59PM +0200, Sam Ravnborg wrote: > Hi Daniel > > > It's dead code, and removing it avoids me having to understand > > what it's doing with lock_fb_info. > > I pushed the series through my build tests which include the sh > architecture. > > One error and one warning

Re: [Intel-gfx] [PATCH 09/33] fbcon: Remove fbcon_has_exited

On Sat, May 25, 2019 at 05:38:26PM +0200, Sam Ravnborg wrote: > Hi Daniel. > > One detail I noticed while brosing the changes. > > > > > @@ -1064,9 +1062,13 @@ static void fbcon_init(struct vc_data *vc, int init) > > int logo = 1, new_rows, new_cols, rows, cols, charcnt = 256; > > int c

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dsi: Call drm_connector_cleanup on vlv_dsi_init error exit path

== Series Details == Series: drm/i915/dsi: Call drm_connector_cleanup on vlv_dsi_init error exit path URL : https://patchwork.freedesktop.org/series/61117/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142_full -> Patchwork_13094_full =

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dsi: Read back pclk set by GOP and use that as pclk (version 3)

== Series Details == Series: drm/i915/dsi: Read back pclk set by GOP and use that as pclk (version 3) URL : https://patchwork.freedesktop.org/series/61115/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142_full -> Patchwork_13093_full =

[Intel-gfx] [PATCH V3 i-g-t] lib: Drop __kms_addfb() wrapper

The function __kms_addfb() and drmModeAddFB2WithModifiers() have a similar code. Due to this similarity, this commit replaces all the occurrences of __kms_addfb() by drmModeAddFB2WithModifiers() and adds the required adaptations. V2: Simon Sir - Fix array data copy V1: Arkadiusz Hiler - Fix ar

[Intel-gfx] ✓ Fi.CI.IGT: success for Refactor to expand subslice mask (rev10)

== Series Details == Series: Refactor to expand subslice mask (rev10) URL : https://patchwork.freedesktop.org/series/59742/ State : success == Summary == CI Bug Log - changes from CI_DRM_6141_full -> Patchwork_13092_full Summary ---

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2)

== Series Details == Series: series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2) URL : https://patchwork.freedesktop.org/series/60921/ State : success == Summary == CI Bug Log - changes from CI_DRM_6141_full -> Patchwork_13091_full =

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/panel: drmP.h removal + small fix

== Series Details == Series: drm/panel: drmP.h removal + small fix URL : https://patchwork.freedesktop.org/series/61157/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13104 Summary --- **SUCCESS**

[Intel-gfx] ✓ Fi.CI.BAT: success for drm: make headers self-contained and drop drmP.h

== Series Details == Series: drm: make headers self-contained and drop drmP.h URL : https://patchwork.freedesktop.org/series/61156/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13103 Summary --- **

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm: make headers self-contained and drop drmP.h

== Series Details == Series: drm: make headers self-contained and drop drmP.h URL : https://patchwork.freedesktop.org/series/61156/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm: make drm/drm_auth.h self contained Okay! Commit: drm: make drm/drm_

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4)

== Series Details == Series: drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4) URL : https://patchwork.freedesktop.org/series/61047/ State : failure == Summary == CI Bug Log - changes from CI_DRM_6141_full -> Patchwork_13090_full

[Intel-gfx] [PATCH v2 1/2] drm/panel: panel-innolux: drop unused variable

The num_supplies variable is not used, delete it. Build tested. Signed-off-by: Sam Ravnborg Cc: Thierry Reding Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/panel/panel-innolux-p079zca.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/panel/panel-innolux-p079zca.c

[Intel-gfx] [PATCH v2 2/2] drm/panel: drop drmP.h usage

Drop use of the deprecated drmP.h header file. While touching the list of include files: - Divide include files in blocks of linux/* video/* drm/* etc. Be consistent in the order of the blocks - Sort individual blocks of include files Signed-off-by: Sam Ravnborg Cc: Thierry Reding Cc: Linus W

[Intel-gfx] [PATCH v2 0/2] drm/panel: drmP.h removal + small fix

The header file drmP.h have been deprecated and this set include a patch to remove the use of drmP.h in all files in drm/panel/*. The patchset has been build tested with x86, arm, ++ in various configs. There is a flux of incoming panels, and removing use of drmP.h from existing files will hopefu

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm: make headers self-contained and drop drmP.h

== Series Details == Series: drm: make headers self-contained and drop drmP.h URL : https://patchwork.freedesktop.org/series/61156/ State : warning == Summary == $ dim checkpatch origin/drm-tip 67d7923073e6 drm: make drm/drm_auth.h self contained -:21: WARNING:SPDX_LICENSE_TAG: Missing or malf

[Intel-gfx] [PATCH v2 7/7] drm: drop use of drmP.h in drm/*

The use of the drmP.h header file is deprecated. Remove use from all files in drm/* so people do not look there and follow a bad example. Build tested allyesconfig,allmodconfig on x86, arm etc. Including alpha that is as always more challenging than the rest. Signed-off-by: Sam Ravnborg Cc: Maar

[Intel-gfx] [PATCH v2 3/7] drm: make drm_crtc_internal.h self-contained

Do not require users of drm_crtc_internal.h to include other files just to let it build Signed-off-by: Sam Ravnborg Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/drm_crtc_internal.h | 24 1 file changed,

[Intel-gfx] [PATCH v2 6/7] drm: make drm_trace.h self-contained

Do not require users of drm_trace.h to include other files just to let it build Signed-off-by: Sam Ravnborg Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/drm_trace.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/

[Intel-gfx] [PATCH v2 4/7] drm: make drm_internal.h self-contained

Do not require users of drm_internal.h to include other files just to let it build Signed-off-by: Sam Ravnborg Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/drm_internal.h | 10 +- 1 file changed, 9 insertions(+), 1 dele

[Intel-gfx] [PATCH v2 0/7] drm: make headers self-contained and drop drmP.h

While removing use of drmP.h from files in drm/* I noticed that I had to add the same include files due to dependencies in the header files. It is better to let the header files be self-contained and let the users pull in only the additional headers files required. So I went ahead and made the rel

[Intel-gfx] [PATCH v2 2/7] drm: make drm/drm_legacy.h self-contained

Do not require users of include/drm/drm_legacy.h to include other files just to let it build. Signed-off-by: Sam Ravnborg Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- include/drm/drm_legacy.h | 12 ++-- 1 file changed, 6 insertions(+),

[Intel-gfx] [PATCH v2 5/7] drm: make drm_legacy.h self-contained

Do not require users of drm_legacy.h to include other files just to let it build Signed-off-by: Sam Ravnborg Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/drm_legacy.h | 4 1 file changed, 4 insertions(+) diff --git a/driv

[Intel-gfx] [PATCH v2 1/7] drm: make drm/drm_auth.h self contained

Do not require users of include/drm/drm_auth.h to include other files just to let it build. Signed-off-by: Sam Ravnborg Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- include/drm/drm_auth.h | 11 +-- 1 file changed, 9 insertions(+), 2 de

[Intel-gfx] ✓ Fi.CI.BAT: success for fbcon notifier begone! (rev4)

== Series Details == Series: fbcon notifier begone! (rev4) URL : https://patchwork.freedesktop.org/series/60843/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13102 Summary --- **SUCCESS** No reg

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for fbcon notifier begone! (rev4)

== Series Details == Series: fbcon notifier begone! (rev4) URL : https://patchwork.freedesktop.org/series/60843/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: dummycon: Sprinkle locking checks Okay! Commit: fbdev: locking check for fb_set_suspend Oka

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for fbcon notifier begone! (rev4)

== Series Details == Series: fbcon notifier begone! (rev4) URL : https://patchwork.freedesktop.org/series/60843/ State : warning == Summary == $ dim checkpatch origin/drm-tip 01d758cc0fca dummycon: Sprinkle locking checks -:45: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/ehl: Introduce Mule Creek Canyon PCH

== Series Details == Series: drm/i915/ehl: Introduce Mule Creek Canyon PCH URL : https://patchwork.freedesktop.org/series/61137/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13101 Summary --- **SUC

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gtt: set err to -ENOMEM on memory allocation failure

== Series Details == Series: drm/i915/gtt: set err to -ENOMEM on memory allocation failure URL : https://patchwork.freedesktop.org/series/61134/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13100 Summary

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: make REG_BIT() and REG_GENMASK() work with variables

== Series Details == Series: drm/i915: make REG_BIT() and REG_GENMASK() work with variables URL : https://patchwork.freedesktop.org/series/61129/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13099 Summary

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Make intel_fuzzy_clock_check available outside of intel_display.c

== Series Details == Series: series starting with [1/2] drm/i915: Make intel_fuzzy_clock_check available outside of intel_display.c URL : https://patchwork.freedesktop.org/series/61127/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13098

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915: Make intel_fuzzy_clock_check available outside of intel_display.c

== Series Details == Series: series starting with [1/2] drm/i915: Make intel_fuzzy_clock_check available outside of intel_display.c URL : https://patchwork.freedesktop.org/series/61127/ State : warning == Summary == $ dim checkpatch origin/drm-tip 55738a76d511 drm/i915: Make intel_fuzzy_clock

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/kvmgt: Use struct_size() helper

== Series Details == Series: drm/i915/kvmgt: Use struct_size() helper URL : https://patchwork.freedesktop.org/series/61124/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13096 Summary --- **SUCCESS*

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [RFC,1/3] kbuild: add support for ensuring headers are self-contained (rev2)

== Series Details == Series: series starting with [RFC,1/3] kbuild: add support for ensuring headers are self-contained (rev2) URL : https://patchwork.freedesktop.org/series/60738/ State : failure == Summary == Applying: kbuild: add support for ensuring headers are self-contained Using index

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/icl: Fix AUX-B HW not done issue w/o AUX-A

== Series Details == Series: drm/i915/icl: Fix AUX-B HW not done issue w/o AUX-A URL : https://patchwork.freedesktop.org/series/61123/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13095 Summary ---

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dsi: Call drm_connector_cleanup on vlv_dsi_init error exit path

== Series Details == Series: drm/i915/dsi: Call drm_connector_cleanup on vlv_dsi_init error exit path URL : https://patchwork.freedesktop.org/series/61117/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13094 ===

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dsi: Read back pclk set by GOP and use that as pclk (version 3)

== Series Details == Series: drm/i915/dsi: Read back pclk set by GOP and use that as pclk (version 3) URL : https://patchwork.freedesktop.org/series/61115/ State : success == Summary == CI Bug Log - changes from CI_DRM_6142 -> Patchwork_13093 ===

[Intel-gfx] ✓ Fi.CI.BAT: success for Refactor to expand subslice mask (rev10)

== Series Details == Series: Refactor to expand subslice mask (rev10) URL : https://patchwork.freedesktop.org/series/59742/ State : success == Summary == CI Bug Log - changes from CI_DRM_6141 -> Patchwork_13092 Summary --- **SUCCESS*

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/dsi: Read back pclk set by GOP and use that as pclk (version 3)

== Series Details == Series: drm/i915/dsi: Read back pclk set by GOP and use that as pclk (version 3) URL : https://patchwork.freedesktop.org/series/61115/ State : warning == Summary == $ dim checkpatch origin/drm-tip 48ee9e30a01e drm/i915: Make intel_fuzzy_clock_check available outside of in

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Refactor to expand subslice mask (rev10)

== Series Details == Series: Refactor to expand subslice mask (rev10) URL : https://patchwork.freedesktop.org/series/59742/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Use local variable for SSEU info in GETPARAM ioctl Okay! Commit: drm/i

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Refactor to expand subslice mask (rev10)

== Series Details == Series: Refactor to expand subslice mask (rev10) URL : https://patchwork.freedesktop.org/series/59742/ State : warning == Summary == $ dim checkpatch origin/drm-tip 1d605edfee68 drm/i915: Use local variable for SSEU info in GETPARAM ioctl 1b9931fa807c drm/i915: Add macro f

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2)

== Series Details == Series: series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2) URL : https://patchwork.freedesktop.org/series/60921/ State : success == Summary == CI Bug Log - changes from CI_DRM_6141 -> Patchwork_13091 ===

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2)

== Series Details == Series: series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2) URL : https://patchwork.freedesktop.org/series/60921/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915:

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2)

== Series Details == Series: series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register (rev2) URL : https://patchwork.freedesktop.org/series/60921/ State : warning == Summary == $ dim checkpatch origin/drm-tip 7c375533ea71 drm/i915: Make sandybri

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4)

== Series Details == Series: drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4) URL : https://patchwork.freedesktop.org/series/61047/ State : success == Summary == CI Bug Log - changes from CI_DRM_6141 -> Patchwork_13090 Summary -

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4)

== Series Details == Series: drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4) URL : https://patchwork.freedesktop.org/series/61047/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Keep user GGTT alive for a minimum of 250ms + +driv

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4)

== Series Details == Series: drm/i915: Keep user GGTT alive for a minimum of 250ms (rev4) URL : https://patchwork.freedesktop.org/series/61047/ State : warning == Summary == $ dim checkpatch origin/drm-tip 49297ed37b43 drm/i915: Keep user GGTT alive for a minimum of 250ms -:177: CHECK:UNCOMMEN

Re: [Intel-gfx] [PATCH v3 0/5] drm/gma500: drop use of drmP.h

Hi Patrik. > v2 of this series is already applied to drm-misc-next Thanks, I had not rebased on top of drm-misc-next - obviously. Sam ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/inte