* [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-26 18:35 ` Randy Dunlap 0 siblings, 0 replies; 12+ messages in thread From: Randy Dunlap @ 2021-04-26 18:35 UTC (permalink / raw) To: linux-kernel Cc: Randy Dunlap, Ville Syrjälä, Jani Nikula, Damien Lespiau, Daniel Vetter, Jani Nikula, Joonas Lahtinen, Rodrigo Vivi, intel-gfx, dri-devel When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, there are build errors trying to reference backlight_device_{un}register(). Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] fixes this. ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' Fixes: 912e8b12eedb ("drm/i915: register backlight device also when backlight class is a module") Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Jani Nikula <jani.nikula@intel.com> Cc: Damien Lespiau <damien.lespiau@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Cc: Jani Nikula <jani.nikula@linux.intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: intel-gfx@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org --- Found in linux-next but applies to mainline (5.12). drivers/gpu/drm/i915/display/intel_panel.c | 2 +- drivers/gpu/drm/i915/display/intel_panel.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.c +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.c @@ -1254,7 +1254,7 @@ void intel_panel_enable_backlight(const mutex_unlock(&dev_priv->backlight_lock); } -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) static u32 intel_panel_get_backlight(struct intel_connector *connector) { struct drm_i915_private *dev_priv = to_i915(connector->base.dev); --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.h +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.h @@ -54,7 +54,7 @@ u32 intel_panel_invert_pwm_level(struct u32 intel_panel_backlight_level_to_pwm(struct intel_connector *connector, u32 level); u32 intel_panel_backlight_level_from_pwm(struct intel_connector *connector, u32 val); -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) int intel_backlight_device_register(struct intel_connector *connector); void intel_backlight_device_unregister(struct intel_connector *connector); #else /* CONFIG_BACKLIGHT_CLASS_DEVICE */ ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-gfx] [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-26 18:35 ` Randy Dunlap 0 siblings, 0 replies; 12+ messages in thread From: Randy Dunlap @ 2021-04-26 18:35 UTC (permalink / raw) To: linux-kernel Cc: Jani Nikula, Daniel Vetter, Damien Lespiau, Randy Dunlap, dri-devel, intel-gfx When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, there are build errors trying to reference backlight_device_{un}register(). Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] fixes this. ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' Fixes: 912e8b12eedb ("drm/i915: register backlight device also when backlight class is a module") Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Jani Nikula <jani.nikula@intel.com> Cc: Damien Lespiau <damien.lespiau@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Cc: Jani Nikula <jani.nikula@linux.intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: intel-gfx@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org --- Found in linux-next but applies to mainline (5.12). drivers/gpu/drm/i915/display/intel_panel.c | 2 +- drivers/gpu/drm/i915/display/intel_panel.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.c +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.c @@ -1254,7 +1254,7 @@ void intel_panel_enable_backlight(const mutex_unlock(&dev_priv->backlight_lock); } -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) static u32 intel_panel_get_backlight(struct intel_connector *connector) { struct drm_i915_private *dev_priv = to_i915(connector->base.dev); --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.h +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.h @@ -54,7 +54,7 @@ u32 intel_panel_invert_pwm_level(struct u32 intel_panel_backlight_level_to_pwm(struct intel_connector *connector, u32 level); u32 intel_panel_backlight_level_from_pwm(struct intel_connector *connector, u32 val); -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) int intel_backlight_device_register(struct intel_connector *connector); void intel_backlight_device_unregister(struct intel_connector *connector); #else /* CONFIG_BACKLIGHT_CLASS_DEVICE */ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-26 18:35 ` Randy Dunlap 0 siblings, 0 replies; 12+ messages in thread From: Randy Dunlap @ 2021-04-26 18:35 UTC (permalink / raw) To: linux-kernel Cc: Jani Nikula, Daniel Vetter, Damien Lespiau, Randy Dunlap, dri-devel, Rodrigo Vivi, intel-gfx When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, there are build errors trying to reference backlight_device_{un}register(). Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] fixes this. ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' Fixes: 912e8b12eedb ("drm/i915: register backlight device also when backlight class is a module") Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Jani Nikula <jani.nikula@intel.com> Cc: Damien Lespiau <damien.lespiau@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Cc: Jani Nikula <jani.nikula@linux.intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: intel-gfx@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org --- Found in linux-next but applies to mainline (5.12). drivers/gpu/drm/i915/display/intel_panel.c | 2 +- drivers/gpu/drm/i915/display/intel_panel.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.c +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.c @@ -1254,7 +1254,7 @@ void intel_panel_enable_backlight(const mutex_unlock(&dev_priv->backlight_lock); } -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) static u32 intel_panel_get_backlight(struct intel_connector *connector) { struct drm_i915_private *dev_priv = to_i915(connector->base.dev); --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.h +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.h @@ -54,7 +54,7 @@ u32 intel_panel_invert_pwm_level(struct u32 intel_panel_backlight_level_to_pwm(struct intel_connector *connector, u32 level); u32 intel_panel_backlight_level_from_pwm(struct intel_connector *connector, u32 val); -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) int intel_backlight_device_register(struct intel_connector *connector); void intel_backlight_device_unregister(struct intel_connector *connector); #else /* CONFIG_BACKLIGHT_CLASS_DEVICE */ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm: i915: fix build when ACPI is disabled and BACKLIGHT=m 2021-04-26 18:35 ` Randy Dunlap (?) (?) @ 2021-04-26 19:56 ` Patchwork -1 siblings, 0 replies; 12+ messages in thread From: Patchwork @ 2021-04-26 19:56 UTC (permalink / raw) To: Randy Dunlap; +Cc: intel-gfx == Series Details == Series: drm: i915: fix build when ACPI is disabled and BACKLIGHT=m URL : https://patchwork.freedesktop.org/series/89509/ State : warning == Summary == $ dim checkpatch origin/drm-tip 1db2d6629e36 drm: i915: fix build when ACPI is disabled and BACKLIGHT=m -:16: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #16: ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': total: 0 errors, 1 warnings, 0 checks, 16 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm: i915: fix build when ACPI is disabled and BACKLIGHT=m 2021-04-26 18:35 ` Randy Dunlap ` (2 preceding siblings ...) (?) @ 2021-04-26 20:21 ` Patchwork -1 siblings, 0 replies; 12+ messages in thread From: Patchwork @ 2021-04-26 20:21 UTC (permalink / raw) To: Randy Dunlap; +Cc: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 2526 bytes --] == Series Details == Series: drm: i915: fix build when ACPI is disabled and BACKLIGHT=m URL : https://patchwork.freedesktop.org/series/89509/ State : success == Summary == CI Bug Log - changes from CI_DRM_10011 -> Patchwork_19995 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/index.html Known issues ------------ Here are the changes found in Patchwork_19995 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@amdgpu/amd_basic@query-info: - fi-bsw-kefka: NOTRUN -> [SKIP][1] ([fdo#109271]) +17 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/fi-bsw-kefka/igt@amdgpu/amd_basic@query-info.html * igt@gem_exec_suspend@basic-s0: - fi-kbl-soraka: [PASS][2] -> [INCOMPLETE][3] ([i915#155]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/fi-kbl-soraka/igt@gem_exec_suspend@basic-s0.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/fi-kbl-soraka/igt@gem_exec_suspend@basic-s0.html #### Possible fixes #### * igt@i915_selftest@live@execlists: - fi-bsw-kefka: [INCOMPLETE][4] ([i915#2782] / [i915#2940]) -> [PASS][5] [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/fi-bsw-kefka/igt@i915_selftest@live@execlists.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/fi-bsw-kefka/igt@i915_selftest@live@execlists.html [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [i915#155]: https://gitlab.freedesktop.org/drm/intel/issues/155 [i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782 [i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940 Participating hosts (40 -> 37) ------------------------------ Missing (3): fi-tgl-y fi-bsw-cyan fi-bdw-samus Build changes ------------- * Linux: CI_DRM_10011 -> Patchwork_19995 CI-20190529: 20190529 CI_DRM_10011: 7e185a8c4fb2230187a8637e3885be57123cb7eb @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6075: ccf602c569257291045415ff504a6d2460986c28 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_19995: 1db2d6629e365765e76b05550c35dd2434adf895 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 1db2d6629e36 drm: i915: fix build when ACPI is disabled and BACKLIGHT=m == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/index.html [-- Attachment #1.2: Type: text/html, Size: 3178 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm: i915: fix build when ACPI is disabled and BACKLIGHT=m 2021-04-26 18:35 ` Randy Dunlap ` (3 preceding siblings ...) (?) @ 2021-04-27 5:08 ` Patchwork -1 siblings, 0 replies; 12+ messages in thread From: Patchwork @ 2021-04-27 5:08 UTC (permalink / raw) To: Randy Dunlap; +Cc: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 29338 bytes --] == Series Details == Series: drm: i915: fix build when ACPI is disabled and BACKLIGHT=m URL : https://patchwork.freedesktop.org/series/89509/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10011_full -> Patchwork_19995_full ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_19995_full absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_19995_full, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_19995_full: ### IGT changes ### #### Possible regressions #### * igt@gem_userptr_blits@huge-split: - shard-skl: [PASS][1] -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl9/igt@gem_userptr_blits@huge-split.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl1/igt@gem_userptr_blits@huge-split.html Known issues ------------ Here are the changes found in Patchwork_19995_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_persistence@legacy-engines-hostile-preempt: - shard-snb: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#1099]) +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-snb6/igt@gem_ctx_persistence@legacy-engines-hostile-preempt.html * igt@gem_ctx_ringsize@idle@bcs0: - shard-skl: NOTRUN -> [INCOMPLETE][4] ([i915#3316]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@gem_ctx_ringsize@idle@bcs0.html * igt@gem_exec_fair@basic-flow@rcs0: - shard-skl: NOTRUN -> [SKIP][5] ([fdo#109271]) +86 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@gem_exec_fair@basic-flow@rcs0.html * igt@gem_exec_fair@basic-none-solo@rcs0: - shard-kbl: NOTRUN -> [FAIL][6] ([i915#2842]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl1/igt@gem_exec_fair@basic-none-solo@rcs0.html * igt@gem_exec_fair@basic-none@vcs1: - shard-iclb: NOTRUN -> [FAIL][7] ([i915#2842]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb1/igt@gem_exec_fair@basic-none@vcs1.html * igt@gem_exec_fair@basic-none@vecs0: - shard-kbl: [PASS][8] -> [FAIL][9] ([i915#2842]) +1 similar issue [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl4/igt@gem_exec_fair@basic-none@vecs0.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl2/igt@gem_exec_fair@basic-none@vecs0.html * igt@gem_exec_fair@basic-throttle@rcs0: - shard-iclb: [PASS][10] -> [FAIL][11] ([i915#2849]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb2/igt@gem_exec_fair@basic-throttle@rcs0.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb5/igt@gem_exec_fair@basic-throttle@rcs0.html * igt@gem_pwrite@basic-exhaustion: - shard-kbl: NOTRUN -> [WARN][12] ([i915#2658]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl6/igt@gem_pwrite@basic-exhaustion.html * igt@gem_render_copy@x-tiled-to-vebox-yf-tiled: - shard-kbl: NOTRUN -> [SKIP][13] ([fdo#109271]) +134 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl1/igt@gem_render_copy@x-tiled-to-vebox-yf-tiled.html * igt@gem_userptr_blits@vma-merge: - shard-apl: NOTRUN -> [FAIL][14] ([i915#3318]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl8/igt@gem_userptr_blits@vma-merge.html * igt@gem_workarounds@suspend-resume-fd: - shard-kbl: [PASS][15] -> [DMESG-WARN][16] ([i915#180]) +4 similar issues [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl1/igt@gem_workarounds@suspend-resume-fd.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl3/igt@gem_workarounds@suspend-resume-fd.html * igt@i915_pm_dc@dc6-psr: - shard-skl: NOTRUN -> [FAIL][17] ([i915#454]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@i915_pm_dc@dc6-psr.html * igt@i915_suspend@forcewake: - shard-kbl: [PASS][18] -> [INCOMPLETE][19] ([i915#155] / [i915#636]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl6/igt@i915_suspend@forcewake.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl3/igt@i915_suspend@forcewake.html * igt@kms_chamelium@hdmi-hpd-storm-disable: - shard-skl: NOTRUN -> [SKIP][20] ([fdo#109271] / [fdo#111827]) +6 similar issues [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@kms_chamelium@hdmi-hpd-storm-disable.html * igt@kms_chamelium@vga-hpd-for-each-pipe: - shard-kbl: NOTRUN -> [SKIP][21] ([fdo#109271] / [fdo#111827]) +16 similar issues [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl1/igt@kms_chamelium@vga-hpd-for-each-pipe.html * igt@kms_color_chamelium@pipe-a-ctm-limited-range: - shard-apl: NOTRUN -> [SKIP][22] ([fdo#109271] / [fdo#111827]) +25 similar issues [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl2/igt@kms_color_chamelium@pipe-a-ctm-limited-range.html * igt@kms_color_chamelium@pipe-c-ctm-green-to-red: - shard-snb: NOTRUN -> [SKIP][23] ([fdo#109271] / [fdo#111827]) +13 similar issues [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-snb7/igt@kms_color_chamelium@pipe-c-ctm-green-to-red.html * igt@kms_content_protection@lic: - shard-apl: NOTRUN -> [TIMEOUT][24] ([i915#1319]) +2 similar issues [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl8/igt@kms_content_protection@lic.html * igt@kms_content_protection@srm: - shard-kbl: NOTRUN -> [TIMEOUT][25] ([i915#1319]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl1/igt@kms_content_protection@srm.html * igt@kms_content_protection@uevent: - shard-apl: NOTRUN -> [FAIL][26] ([i915#2105]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl7/igt@kms_content_protection@uevent.html * igt@kms_flip@flip-vs-suspend@a-edp1: - shard-skl: [PASS][27] -> [INCOMPLETE][28] ([i915#198]) +1 similar issue [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl4/igt@kms_flip@flip-vs-suspend@a-edp1.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl4/igt@kms_flip@flip-vs-suspend@a-edp1.html * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-msflip-blt: - shard-snb: NOTRUN -> [SKIP][29] ([fdo#109271]) +265 similar issues [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-snb6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-msflip-blt.html * igt@kms_pipe_crc_basic@hang-read-crc-pipe-d: - shard-apl: NOTRUN -> [SKIP][30] ([fdo#109271] / [i915#533]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl6/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb: - shard-skl: NOTRUN -> [FAIL][31] ([fdo#108145] / [i915#265]) +1 similar issue [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb.html * igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb: - shard-skl: NOTRUN -> [FAIL][32] ([i915#265]) [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html * igt@kms_plane_alpha_blend@pipe-c-alpha-basic: - shard-apl: NOTRUN -> [FAIL][33] ([fdo#108145] / [i915#265]) +2 similar issues [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl7/igt@kms_plane_alpha_blend@pipe-c-alpha-basic.html * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc: - shard-skl: [PASS][34] -> [FAIL][35] ([fdo#108145] / [i915#265]) [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl1/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl10/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html * igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping: - shard-skl: NOTRUN -> [SKIP][36] ([fdo#109271] / [i915#2733]) [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping.html * igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2: - shard-apl: NOTRUN -> [SKIP][37] ([fdo#109271] / [i915#658]) +5 similar issues [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl7/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3: - shard-kbl: NOTRUN -> [SKIP][38] ([fdo#109271] / [i915#658]) +4 similar issues [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl6/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-3: - shard-skl: NOTRUN -> [SKIP][39] ([fdo#109271] / [i915#658]) [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl9/igt@kms_psr2_sf@plane-move-sf-dmg-area-3.html * igt@kms_psr@psr2_suspend: - shard-iclb: [PASS][40] -> [SKIP][41] ([fdo#109441]) +2 similar issues [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb2/igt@kms_psr@psr2_suspend.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb5/igt@kms_psr@psr2_suspend.html * igt@kms_sysfs_edid_timing: - shard-apl: NOTRUN -> [FAIL][42] ([IGT#2]) [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl7/igt@kms_sysfs_edid_timing.html * igt@kms_vblank@pipe-c-ts-continuation-dpms-suspend: - shard-kbl: [PASS][43] -> [INCOMPLETE][44] ([i915#155] / [i915#2405]) [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl2/igt@kms_vblank@pipe-c-ts-continuation-dpms-suspend.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl3/igt@kms_vblank@pipe-c-ts-continuation-dpms-suspend.html * igt@kms_vblank@pipe-d-wait-idle: - shard-kbl: NOTRUN -> [SKIP][45] ([fdo#109271] / [i915#533]) +2 similar issues [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl2/igt@kms_vblank@pipe-d-wait-idle.html - shard-skl: NOTRUN -> [SKIP][46] ([fdo#109271] / [i915#533]) [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl9/igt@kms_vblank@pipe-d-wait-idle.html * igt@kms_writeback@writeback-check-output: - shard-apl: NOTRUN -> [SKIP][47] ([fdo#109271] / [i915#2437]) [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl6/igt@kms_writeback@writeback-check-output.html * igt@kms_writeback@writeback-fb-id: - shard-skl: NOTRUN -> [SKIP][48] ([fdo#109271] / [i915#2437]) [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@kms_writeback@writeback-fb-id.html * igt@perf@polling-parameterized: - shard-skl: [PASS][49] -> [FAIL][50] ([i915#1542]) [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl8/igt@perf@polling-parameterized.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl9/igt@perf@polling-parameterized.html * igt@perf@polling-small-buf: - shard-skl: [PASS][51] -> [FAIL][52] ([i915#1722]) [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl1/igt@perf@polling-small-buf.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl7/igt@perf@polling-small-buf.html * igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name: - shard-apl: NOTRUN -> [SKIP][53] ([fdo#109271]) +245 similar issues [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl7/igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name.html * igt@sysfs_clients@fair-0: - shard-apl: NOTRUN -> [SKIP][54] ([fdo#109271] / [i915#2994]) +2 similar issues [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl2/igt@sysfs_clients@fair-0.html * igt@sysfs_clients@pidname: - shard-skl: NOTRUN -> [SKIP][55] ([fdo#109271] / [i915#2994]) +1 similar issue [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@sysfs_clients@pidname.html * igt@sysfs_clients@sema-10: - shard-kbl: NOTRUN -> [SKIP][56] ([fdo#109271] / [i915#2994]) +1 similar issue [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl1/igt@sysfs_clients@sema-10.html #### Possible fixes #### * igt@gem_create@create-clear: - shard-glk: [FAIL][57] ([i915#1888] / [i915#3160]) -> [PASS][58] [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-glk3/igt@gem_create@create-clear.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-glk9/igt@gem_create@create-clear.html * igt@gem_ctx_persistence@many-contexts: - shard-tglb: [FAIL][59] ([i915#2410]) -> [PASS][60] [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-tglb1/igt@gem_ctx_persistence@many-contexts.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-tglb7/igt@gem_ctx_persistence@many-contexts.html * igt@gem_exec_fair@basic-none-vip@rcs0: - shard-kbl: [FAIL][61] ([i915#2842]) -> [PASS][62] [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl4/igt@gem_exec_fair@basic-none-vip@rcs0.html [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl2/igt@gem_exec_fair@basic-none-vip@rcs0.html * igt@gem_exec_fair@basic-pace-share@rcs0: - shard-glk: [FAIL][63] ([i915#2842]) -> [PASS][64] +1 similar issue [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-glk9/igt@gem_exec_fair@basic-pace-share@rcs0.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-glk1/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@gem_exec_fair@basic-pace@rcs0: - shard-kbl: [SKIP][65] ([fdo#109271]) -> [PASS][66] +1 similar issue [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl1/igt@gem_exec_fair@basic-pace@rcs0.html [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl4/igt@gem_exec_fair@basic-pace@rcs0.html * igt@gem_exec_fair@basic-pace@vcs0: - shard-tglb: [FAIL][67] ([i915#2842]) -> [PASS][68] [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-tglb1/igt@gem_exec_fair@basic-pace@vcs0.html [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-tglb7/igt@gem_exec_fair@basic-pace@vcs0.html * igt@gem_exec_whisper@basic-normal-all: - shard-glk: [DMESG-WARN][69] ([i915#118] / [i915#95]) -> [PASS][70] [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-glk4/igt@gem_exec_whisper@basic-normal-all.html [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-glk6/igt@gem_exec_whisper@basic-normal-all.html * igt@gem_mmap_gtt@cpuset-basic-small-copy-odd: - shard-iclb: [FAIL][71] ([i915#307]) -> [PASS][72] +1 similar issue [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb3/igt@gem_mmap_gtt@cpuset-basic-small-copy-odd.html [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb8/igt@gem_mmap_gtt@cpuset-basic-small-copy-odd.html * igt@i915_selftest@mock@requests: - shard-skl: [INCOMPLETE][73] ([i915#198]) -> [PASS][74] [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl6/igt@i915_selftest@mock@requests.html [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl6/igt@i915_selftest@mock@requests.html * igt@i915_suspend@fence-restore-tiled2untiled: - shard-glk: [FAIL][75] -> [PASS][76] [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-glk3/igt@i915_suspend@fence-restore-tiled2untiled.html [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-glk2/igt@i915_suspend@fence-restore-tiled2untiled.html * igt@kms_cursor_crc@pipe-a-cursor-suspend: - shard-kbl: [DMESG-WARN][77] ([i915#180]) -> [PASS][78] +8 similar issues [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl3/igt@kms_cursor_crc@pipe-a-cursor-suspend.html [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl1/igt@kms_cursor_crc@pipe-a-cursor-suspend.html - shard-apl: [DMESG-WARN][79] ([i915#180]) -> [PASS][80] [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-apl6/igt@kms_cursor_crc@pipe-a-cursor-suspend.html [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl7/igt@kms_cursor_crc@pipe-a-cursor-suspend.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic: - shard-skl: [FAIL][81] ([i915#2346]) -> [PASS][82] +1 similar issue [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic.html [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl2/igt@kms_cursor_legacy@flip-vs-cursor-atomic.html * igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1: - shard-skl: [FAIL][83] ([i915#2122]) -> [PASS][84] [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl7/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl10/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html * igt@kms_flip@flip-vs-suspend-interruptible@a-edp1: - shard-skl: [DMESG-WARN][85] ([i915#1982]) -> [PASS][86] [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl4/igt@kms_flip@flip-vs-suspend-interruptible@a-edp1.html [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@kms_flip@flip-vs-suspend-interruptible@a-edp1.html * igt@kms_hdr@bpc-switch: - shard-skl: [FAIL][87] ([i915#1188]) -> [PASS][88] [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl4/igt@kms_hdr@bpc-switch.html [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl7/igt@kms_hdr@bpc-switch.html * igt@kms_psr@psr2_cursor_blt: - shard-iclb: [SKIP][89] ([fdo#109441]) -> [PASS][90] +1 similar issue [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb6/igt@kms_psr@psr2_cursor_blt.html [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb2/igt@kms_psr@psr2_cursor_blt.html * igt@perf@polling-parameterized: - shard-iclb: [FAIL][91] ([i915#1542]) -> [PASS][92] [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb5/igt@perf@polling-parameterized.html [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb3/igt@perf@polling-parameterized.html #### Warnings #### * igt@gem_exec_fair@basic-pace@vecs0: - shard-kbl: [SKIP][93] ([fdo#109271]) -> [FAIL][94] ([i915#2842]) [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl1/igt@gem_exec_fair@basic-pace@vecs0.html [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl4/igt@gem_exec_fair@basic-pace@vecs0.html * igt@i915_pm_rc6_residency@rc6-idle: - shard-iclb: [WARN][95] ([i915#2684]) -> [WARN][96] ([i915#1804] / [i915#2684]) [95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb2/igt@i915_pm_rc6_residency@rc6-idle.html [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb7/igt@i915_pm_rc6_residency@rc6-idle.html * igt@kms_dp_dsc@basic-dsc-enable-edp: - shard-iclb: [DMESG-WARN][97] ([i915#1226]) -> [SKIP][98] ([fdo#109349]) [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb2/igt@kms_dp_dsc@basic-dsc-enable-edp.html [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb7/igt@kms_dp_dsc@basic-dsc-enable-edp.html * igt@kms_psr2_sf@cursor-plane-update-sf: - shard-iclb: [SKIP][99] ([i915#2920]) -> [SKIP][100] ([i915#658]) +2 similar issues [99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb2/igt@kms_psr2_sf@cursor-plane-update-sf.html [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb6/igt@kms_psr2_sf@cursor-plane-update-sf.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-1: - shard-iclb: [SKIP][101] ([i915#658]) -> [SKIP][102] ([i915#2920]) +1 similar issue [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-iclb1/igt@kms_psr2_sf@plane-move-sf-dmg-area-1.html [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-iclb2/igt@kms_psr2_sf@plane-move-sf-dmg-area-1.html * igt@runner@aborted: - shard-kbl: ([FAIL][103], [FAIL][104], [FAIL][105], [FAIL][106], [FAIL][107], [FAIL][108], [FAIL][109], [FAIL][110], [FAIL][111], [FAIL][112]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#2292] / [i915#3002] / [i915#602]) -> ([FAIL][113], [FAIL][114], [FAIL][115], [FAIL][116], [FAIL][117], [FAIL][118], [FAIL][119]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002]) [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl7/igt@runner@aborted.html [104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl3/igt@runner@aborted.html [105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl3/igt@runner@aborted.html [106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl3/igt@runner@aborted.html [107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl7/igt@runner@aborted.html [108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl4/igt@runner@aborted.html [109]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl3/igt@runner@aborted.html [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl7/igt@runner@aborted.html [111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl1/igt@runner@aborted.html [112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-kbl7/igt@runner@aborted.html [113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl3/igt@runner@aborted.html [114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl3/igt@runner@aborted.html [115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl3/igt@runner@aborted.html [116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl4/igt@runner@aborted.html [117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl4/igt@runner@aborted.html [118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl4/igt@runner@aborted.html [119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-kbl7/igt@runner@aborted.html - shard-apl: ([FAIL][120], [FAIL][121], [FAIL][122]) ([i915#1814] / [i915#3002]) -> [FAIL][123] ([i915#3002]) [120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-apl6/igt@runner@aborted.html [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-apl2/igt@runner@aborted.html [122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-apl8/igt@runner@aborted.html [123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-apl1/igt@runner@aborted.html - shard-skl: ([FAIL][124], [FAIL][125], [FAIL][126], [FAIL][127], [FAIL][128], [FAIL][129]) ([i915#1436] / [i915#1814] / [i915#2029] / [i915#2722] / [i915#3002]) -> ([FAIL][130], [FAIL][131]) ([i915#3002]) [124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl2/igt@runner@aborted.html [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl2/igt@runner@aborted.html [126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl6/igt@runner@aborted.html [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl2/igt@runner@aborted.html [128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl9/igt@runner@aborted.html [129]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10011/shard-skl2/igt@runner@aborted.html [130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl8/igt@runner@aborted.html [131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/shard-skl4/igt@runner@aborted.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [IGT#2]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/2 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109349]: https://bugs.freedesktop.org/show_bug.cgi?id=109349 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1099]: https://gitlab.freedesktop.org/drm/intel/issues/1099 [i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118 [i915#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188 [i915#1226]: https://gitlab.freedesktop.org/drm/intel/issues/1226 [i915#1319]: https://gitlab.freedesktop.org/drm/intel/issues/1319 [i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436 [i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542 [i915#155]: https://gitlab.freedesktop.org/drm/intel/issues/155 [i915#1722]: https://gitlab.freedesktop.org/drm/intel/issues/1722 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804 [i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814 [i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888 [i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198 [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982 [i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029 [i915#2105]: https://gitlab.freedesktop.org/drm/intel/issues/2105 [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122 [i915#2292]: https://gitlab.freedesktop.org/drm/intel/issues/2292 [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346 [i915#2405]: https://gitlab.freedesktop.org/drm/intel/issues/2405 [i915#2410]: https://gitlab.freedesktop.org/drm/intel/issues/2410 [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437 [i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265 [i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658 [i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684 [i915#2722]: https://gitlab.freedesktop.org/drm/intel/issues/2722 [i915#2733]: https://gitlab.freedesktop.org/drm/intel/issues/2733 [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842 [i915#2849]: https://gitlab.freedesktop.org/drm/intel/issues/2849 [i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920 [i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994 [i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002 [i915#307]: https://gitlab.freedesktop.org/drm/intel/issues/307 [i915#3160]: https://gitlab.freedesktop.org/drm/intel/issues/3160 [i915#3316]: https://gitlab.freedesktop.org/drm/intel/issues/3316 [i915#3318]: https://gitlab.freedesktop.org/drm/intel/issues/3318 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533 [i915#602]: https://gitlab.freedesktop.org/drm/intel/issues/602 [i915#636]: https://gitlab.freedesktop.org/drm/intel/issues/636 [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658 [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95 Participating hosts (11 -> 11) ------------------------------ No changes in participating hosts Build changes ------------- * Linux: CI_DRM_10011 -> Patchwork_19995 CI-20190529: 20190529 CI_DRM_10011: 7e185a8c4fb2230187a8637e3885be57123cb7eb @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6075: ccf602c569257291045415ff504a6d2460986c28 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_19995: 1db2d6629e365765e76b05550c35dd2434adf895 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19995/index.html [-- Attachment #1.2: Type: text/html, Size: 36444 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m 2021-04-26 18:35 ` Randy Dunlap (?) @ 2021-04-27 8:03 ` Jani Nikula -1 siblings, 0 replies; 12+ messages in thread From: Jani Nikula @ 2021-04-27 8:03 UTC (permalink / raw) To: Randy Dunlap, linux-kernel Cc: Randy Dunlap, Ville Syrjälä, Damien Lespiau, Daniel Vetter, Joonas Lahtinen, Rodrigo Vivi, intel-gfx, dri-devel On Mon, 26 Apr 2021, Randy Dunlap <rdunlap@infradead.org> wrote: > When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and > CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, > there are build errors trying to reference backlight_device_{un}register(). > > Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] > fixes this. I feel like a broken record... CONFIG_DRM_I915=y and CONFIG_BACKLIGHT_CLASS_DEVICE=m is an invalid configuration. The patch at hand just silently hides the problem, leaving you without backlight. i915 should *depend* on backlight, not select it. It would express the dependency without chances for invalid configuration. However, i915 alone can't depend on backlight, all users of backlight should depend on backlight, not select it. Otherwise, you end up with other configuration problems, circular dependencies and whatnot. Everyone should change. See also (*) why select is not a good idea here. I've sent patches to this effect before, got rejected, and the same thing gets repeated ad infinitum. Accepting this patch would stop the inflow of these reports and similar patches, but it does not fix the root cause. It just sweeps the problem under the rug. BR, Jani. (*) Documentation/kbuild/kconfig-language.rst: select should be used with care. select will force a symbol to a value without visiting the dependencies. By abusing select you are able to select a symbol FOO even if FOO depends on BAR that is not set. In general use select only for non-visible symbols (no prompts anywhere) and for symbols with no dependencies. That will limit the usefulness but on the other hand avoid the illegal configurations all over. > > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': > intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': > intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' > > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': > intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': > intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' > > Fixes: 912e8b12eedb ("drm/i915: register backlight device also when backlight class is a module") > Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > Cc: Jani Nikula <jani.nikula@intel.com> > Cc: Damien Lespiau <damien.lespiau@intel.com> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > Cc: Jani Nikula <jani.nikula@linux.intel.com> > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: intel-gfx@lists.freedesktop.org > Cc: dri-devel@lists.freedesktop.org > --- > Found in linux-next but applies to mainline (5.12). > > drivers/gpu/drm/i915/display/intel_panel.c | 2 +- > drivers/gpu/drm/i915/display/intel_panel.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.c > +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.c > @@ -1254,7 +1254,7 @@ void intel_panel_enable_backlight(const > mutex_unlock(&dev_priv->backlight_lock); > } > > -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) > +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) > static u32 intel_panel_get_backlight(struct intel_connector *connector) > { > struct drm_i915_private *dev_priv = to_i915(connector->base.dev); > --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.h > +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.h > @@ -54,7 +54,7 @@ u32 intel_panel_invert_pwm_level(struct > u32 intel_panel_backlight_level_to_pwm(struct intel_connector *connector, u32 level); > u32 intel_panel_backlight_level_from_pwm(struct intel_connector *connector, u32 val); > > -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) > +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) > int intel_backlight_device_register(struct intel_connector *connector); > void intel_backlight_device_unregister(struct intel_connector *connector); > #else /* CONFIG_BACKLIGHT_CLASS_DEVICE */ -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-27 8:03 ` Jani Nikula 0 siblings, 0 replies; 12+ messages in thread From: Jani Nikula @ 2021-04-27 8:03 UTC (permalink / raw) To: Randy Dunlap, linux-kernel Cc: Randy Dunlap, Daniel Vetter, Damien Lespiau, dri-devel, intel-gfx On Mon, 26 Apr 2021, Randy Dunlap <rdunlap@infradead.org> wrote: > When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and > CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, > there are build errors trying to reference backlight_device_{un}register(). > > Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] > fixes this. I feel like a broken record... CONFIG_DRM_I915=y and CONFIG_BACKLIGHT_CLASS_DEVICE=m is an invalid configuration. The patch at hand just silently hides the problem, leaving you without backlight. i915 should *depend* on backlight, not select it. It would express the dependency without chances for invalid configuration. However, i915 alone can't depend on backlight, all users of backlight should depend on backlight, not select it. Otherwise, you end up with other configuration problems, circular dependencies and whatnot. Everyone should change. See also (*) why select is not a good idea here. I've sent patches to this effect before, got rejected, and the same thing gets repeated ad infinitum. Accepting this patch would stop the inflow of these reports and similar patches, but it does not fix the root cause. It just sweeps the problem under the rug. BR, Jani. (*) Documentation/kbuild/kconfig-language.rst: select should be used with care. select will force a symbol to a value without visiting the dependencies. By abusing select you are able to select a symbol FOO even if FOO depends on BAR that is not set. In general use select only for non-visible symbols (no prompts anywhere) and for symbols with no dependencies. That will limit the usefulness but on the other hand avoid the illegal configurations all over. > > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': > intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': > intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' > > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': > intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': > intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' > > Fixes: 912e8b12eedb ("drm/i915: register backlight device also when backlight class is a module") > Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > Cc: Jani Nikula <jani.nikula@intel.com> > Cc: Damien Lespiau <damien.lespiau@intel.com> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > Cc: Jani Nikula <jani.nikula@linux.intel.com> > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: intel-gfx@lists.freedesktop.org > Cc: dri-devel@lists.freedesktop.org > --- > Found in linux-next but applies to mainline (5.12). > > drivers/gpu/drm/i915/display/intel_panel.c | 2 +- > drivers/gpu/drm/i915/display/intel_panel.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.c > +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.c > @@ -1254,7 +1254,7 @@ void intel_panel_enable_backlight(const > mutex_unlock(&dev_priv->backlight_lock); > } > > -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) > +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) > static u32 intel_panel_get_backlight(struct intel_connector *connector) > { > struct drm_i915_private *dev_priv = to_i915(connector->base.dev); > --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.h > +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.h > @@ -54,7 +54,7 @@ u32 intel_panel_invert_pwm_level(struct > u32 intel_panel_backlight_level_to_pwm(struct intel_connector *connector, u32 level); > u32 intel_panel_backlight_level_from_pwm(struct intel_connector *connector, u32 val); > > -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) > +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) > int intel_backlight_device_register(struct intel_connector *connector); > void intel_backlight_device_unregister(struct intel_connector *connector); > #else /* CONFIG_BACKLIGHT_CLASS_DEVICE */ -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-27 8:03 ` Jani Nikula 0 siblings, 0 replies; 12+ messages in thread From: Jani Nikula @ 2021-04-27 8:03 UTC (permalink / raw) To: Randy Dunlap, linux-kernel Cc: Randy Dunlap, Daniel Vetter, Damien Lespiau, dri-devel, Rodrigo Vivi, intel-gfx On Mon, 26 Apr 2021, Randy Dunlap <rdunlap@infradead.org> wrote: > When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and > CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, > there are build errors trying to reference backlight_device_{un}register(). > > Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] > fixes this. I feel like a broken record... CONFIG_DRM_I915=y and CONFIG_BACKLIGHT_CLASS_DEVICE=m is an invalid configuration. The patch at hand just silently hides the problem, leaving you without backlight. i915 should *depend* on backlight, not select it. It would express the dependency without chances for invalid configuration. However, i915 alone can't depend on backlight, all users of backlight should depend on backlight, not select it. Otherwise, you end up with other configuration problems, circular dependencies and whatnot. Everyone should change. See also (*) why select is not a good idea here. I've sent patches to this effect before, got rejected, and the same thing gets repeated ad infinitum. Accepting this patch would stop the inflow of these reports and similar patches, but it does not fix the root cause. It just sweeps the problem under the rug. BR, Jani. (*) Documentation/kbuild/kconfig-language.rst: select should be used with care. select will force a symbol to a value without visiting the dependencies. By abusing select you are able to select a symbol FOO even if FOO depends on BAR that is not set. In general use select only for non-visible symbols (no prompts anywhere) and for symbols with no dependencies. That will limit the usefulness but on the other hand avoid the illegal configurations all over. > > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': > intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': > intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' > > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_register': > intel_panel.c:(.text+0x2ec1): undefined reference to `backlight_device_register' > ld: drivers/gpu/drm/i915/display/intel_panel.o: in function `intel_backlight_device_unregister': > intel_panel.c:(.text+0x2f93): undefined reference to `backlight_device_unregister' > > Fixes: 912e8b12eedb ("drm/i915: register backlight device also when backlight class is a module") > Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > Cc: Jani Nikula <jani.nikula@intel.com> > Cc: Damien Lespiau <damien.lespiau@intel.com> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > Cc: Jani Nikula <jani.nikula@linux.intel.com> > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: intel-gfx@lists.freedesktop.org > Cc: dri-devel@lists.freedesktop.org > --- > Found in linux-next but applies to mainline (5.12). > > drivers/gpu/drm/i915/display/intel_panel.c | 2 +- > drivers/gpu/drm/i915/display/intel_panel.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.c > +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.c > @@ -1254,7 +1254,7 @@ void intel_panel_enable_backlight(const > mutex_unlock(&dev_priv->backlight_lock); > } > > -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) > +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) > static u32 intel_panel_get_backlight(struct intel_connector *connector) > { > struct drm_i915_private *dev_priv = to_i915(connector->base.dev); > --- linux-next-20210426.orig/drivers/gpu/drm/i915/display/intel_panel.h > +++ linux-next-20210426/drivers/gpu/drm/i915/display/intel_panel.h > @@ -54,7 +54,7 @@ u32 intel_panel_invert_pwm_level(struct > u32 intel_panel_backlight_level_to_pwm(struct intel_connector *connector, u32 level); > u32 intel_panel_backlight_level_from_pwm(struct intel_connector *connector, u32 val); > > -#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE) > +#if IS_REACHABLE(CONFIG_BACKLIGHT_CLASS_DEVICE) > int intel_backlight_device_register(struct intel_connector *connector); > void intel_backlight_device_unregister(struct intel_connector *connector); > #else /* CONFIG_BACKLIGHT_CLASS_DEVICE */ -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m 2021-04-27 8:03 ` Jani Nikula (?) @ 2021-04-27 15:31 ` Randy Dunlap -1 siblings, 0 replies; 12+ messages in thread From: Randy Dunlap @ 2021-04-27 15:31 UTC (permalink / raw) To: Jani Nikula, linux-kernel Cc: Ville Syrjälä, Damien Lespiau, Daniel Vetter, Joonas Lahtinen, Rodrigo Vivi, intel-gfx, dri-devel On 4/27/21 1:03 AM, Jani Nikula wrote: > On Mon, 26 Apr 2021, Randy Dunlap <rdunlap@infradead.org> wrote: >> When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and >> CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, >> there are build errors trying to reference backlight_device_{un}register(). >> >> Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] >> fixes this. > > I feel like a broken record... Thanks! :) I'll leave it b0rken as well. > CONFIG_DRM_I915=y and CONFIG_BACKLIGHT_CLASS_DEVICE=m is an invalid > configuration. The patch at hand just silently hides the problem, > leaving you without backlight. > > i915 should *depend* on backlight, not select it. It would express the > dependency without chances for invalid configuration. > > However, i915 alone can't depend on backlight, all users of backlight > should depend on backlight, not select it. Otherwise, you end up with > other configuration problems, circular dependencies and > whatnot. Everyone should change. See also (*) why select is not a good > idea here. > > I've sent patches to this effect before, got rejected, and the same > thing gets repeated ad infinitum. > > Accepting this patch would stop the inflow of these reports and similar > patches, but it does not fix the root cause. It just sweeps the problem > under the rug. > > > BR, > Jani. > > (*) Documentation/kbuild/kconfig-language.rst: > > select should be used with care. select will force > a symbol to a value without visiting the dependencies. > By abusing select you are able to select a symbol FOO even > if FOO depends on BAR that is not set. > In general use select only for non-visible symbols > (no prompts anywhere) and for symbols with no dependencies. > That will limit the usefulness but on the other hand avoid > the illegal configurations all over. Yes, I'm well aware of that. ta. -- ~Randy ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-27 15:31 ` Randy Dunlap 0 siblings, 0 replies; 12+ messages in thread From: Randy Dunlap @ 2021-04-27 15:31 UTC (permalink / raw) To: Jani Nikula, linux-kernel Cc: Daniel Vetter, Damien Lespiau, dri-devel, intel-gfx On 4/27/21 1:03 AM, Jani Nikula wrote: > On Mon, 26 Apr 2021, Randy Dunlap <rdunlap@infradead.org> wrote: >> When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and >> CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, >> there are build errors trying to reference backlight_device_{un}register(). >> >> Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] >> fixes this. > > I feel like a broken record... Thanks! :) I'll leave it b0rken as well. > CONFIG_DRM_I915=y and CONFIG_BACKLIGHT_CLASS_DEVICE=m is an invalid > configuration. The patch at hand just silently hides the problem, > leaving you without backlight. > > i915 should *depend* on backlight, not select it. It would express the > dependency without chances for invalid configuration. > > However, i915 alone can't depend on backlight, all users of backlight > should depend on backlight, not select it. Otherwise, you end up with > other configuration problems, circular dependencies and > whatnot. Everyone should change. See also (*) why select is not a good > idea here. > > I've sent patches to this effect before, got rejected, and the same > thing gets repeated ad infinitum. > > Accepting this patch would stop the inflow of these reports and similar > patches, but it does not fix the root cause. It just sweeps the problem > under the rug. > > > BR, > Jani. > > (*) Documentation/kbuild/kconfig-language.rst: > > select should be used with care. select will force > a symbol to a value without visiting the dependencies. > By abusing select you are able to select a symbol FOO even > if FOO depends on BAR that is not set. > In general use select only for non-visible symbols > (no prompts anywhere) and for symbols with no dependencies. > That will limit the usefulness but on the other hand avoid > the illegal configurations all over. Yes, I'm well aware of that. ta. -- ~Randy _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m @ 2021-04-27 15:31 ` Randy Dunlap 0 siblings, 0 replies; 12+ messages in thread From: Randy Dunlap @ 2021-04-27 15:31 UTC (permalink / raw) To: Jani Nikula, linux-kernel Cc: Daniel Vetter, Damien Lespiau, dri-devel, Rodrigo Vivi, intel-gfx On 4/27/21 1:03 AM, Jani Nikula wrote: > On Mon, 26 Apr 2021, Randy Dunlap <rdunlap@infradead.org> wrote: >> When CONFIG_DRM_I915=y, CONFIG_ACPI is not set, and >> CONFIG_BACKLIGHT_CLASS_DEVICE=m, not due to I915 config, >> there are build errors trying to reference backlight_device_{un}register(). >> >> Changing the use of IS_ENABLED() to IS_REACHABLE() in intel_panel.[ch] >> fixes this. > > I feel like a broken record... Thanks! :) I'll leave it b0rken as well. > CONFIG_DRM_I915=y and CONFIG_BACKLIGHT_CLASS_DEVICE=m is an invalid > configuration. The patch at hand just silently hides the problem, > leaving you without backlight. > > i915 should *depend* on backlight, not select it. It would express the > dependency without chances for invalid configuration. > > However, i915 alone can't depend on backlight, all users of backlight > should depend on backlight, not select it. Otherwise, you end up with > other configuration problems, circular dependencies and > whatnot. Everyone should change. See also (*) why select is not a good > idea here. > > I've sent patches to this effect before, got rejected, and the same > thing gets repeated ad infinitum. > > Accepting this patch would stop the inflow of these reports and similar > patches, but it does not fix the root cause. It just sweeps the problem > under the rug. > > > BR, > Jani. > > (*) Documentation/kbuild/kconfig-language.rst: > > select should be used with care. select will force > a symbol to a value without visiting the dependencies. > By abusing select you are able to select a symbol FOO even > if FOO depends on BAR that is not set. > In general use select only for non-visible symbols > (no prompts anywhere) and for symbols with no dependencies. > That will limit the usefulness but on the other hand avoid > the illegal configurations all over. Yes, I'm well aware of that. ta. -- ~Randy _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2021-04-27 15:33 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-04-26 18:35 [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m Randy Dunlap 2021-04-26 18:35 ` [Intel-gfx] " Randy Dunlap 2021-04-26 18:35 ` Randy Dunlap 2021-04-26 19:56 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2021-04-26 20:21 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2021-04-27 5:08 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 2021-04-27 8:03 ` [PATCH] " Jani Nikula 2021-04-27 8:03 ` [Intel-gfx] " Jani Nikula 2021-04-27 8:03 ` Jani Nikula 2021-04-27 15:31 ` Randy Dunlap 2021-04-27 15:31 ` [Intel-gfx] " Randy Dunlap 2021-04-27 15:31 ` Randy Dunlap
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.