[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2) URL : https://patchwork.freedesktop.org/series/121905/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13471_full ->

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Replace dead 01.org link

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915: Replace dead 01.org link URL : https://patchwork.freedesktop.org/series/121995/ State : success == Summary == CI Bug Log - changes from CI_DRM_13472 -> Patchwork_121995v1 Summary ---

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Replace dead 01.org link

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915: Replace dead 01.org link URL : https://patchwork.freedesktop.org/series/121995/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No such file or directory

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: Replace dead 01.org link

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915: Replace dead 01.org link URL : https://patchwork.freedesktop.org/series/121995/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such file or directory

Re: [Intel-gfx] [PATCH 10/20] drm/i915/dp: Add functions to get min/max src input bpc with DSC

2023-08-03 Thread Nautiyal, Ankit K
On 8/2/2023 5:35 PM, Lisovskiy, Stanislav wrote: On Fri, Jul 28, 2023 at 09:41:40AM +0530, Ankit Nautiyal wrote: Separate out functions for getting maximum and minimum input BPC based on platforms, when DSC is used. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp.c

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/huc: silence injected failure in the load via GSC path (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/huc: silence injected failure in the load via GSC path (rev2) URL : https://patchwork.freedesktop.org/series/121080/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13471_full -> Patchwork_121080v2_full

Re: [Intel-gfx] [PATCH v4 1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Andy Shevchenko
On Thu, Aug 03, 2023 at 10:24:46AM -0700, Andrew Morton wrote: > On Thu, 3 Aug 2023 16:19:18 +0300 Andy Shevchenko > wrote: ... > > +#define abs_diff(a, b) ({ \ > > + typeof(a) __a = (a);\ > > + typeof(b) __b = (b);\ > > +

[Intel-gfx] [PATCH] drm/i915: Replace dead 01.org link

2023-08-03 Thread Zhenyu Wang
01.org is dead so replace old gvt link with current wiki page. Signed-off-by: Zhenyu Wang --- MAINTAINERS | 2 +- drivers/gpu/drm/i915/Kconfig | 2 +- drivers/gpu/drm/i915/intel_gvt.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/vdsc: Fix first_line_bpg_offset calculation

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/vdsc: Fix first_line_bpg_offset calculation URL : https://patchwork.freedesktop.org/series/121967/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471_full -> Patchwork_121967v1_full

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect URL : https://patchwork.freedesktop.org/series/121965/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471_full -> Patchwork_121965v1_full

Re: [Intel-gfx] [PATCH v4 00/29] drm/i915/gvt: KVM: KVMGT fixes and page-track cleanups

2023-08-03 Thread Sean Christopherson
On Fri, 28 Jul 2023 18:35:06 -0700, Sean Christopherson wrote: > Fix a handful of minor bugs in KVMGT, and overhaul KVM's page-track APIs > to provide a leaner and cleaner interface. The motivation for this > series is to (significantly) reduce the number of KVM APIs that KVMGT > uses, with a

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h URL : https://patchwork.freedesktop.org/series/121959/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13471_full -> Patchwork_121959v1_full

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2) URL : https://patchwork.freedesktop.org/series/121905/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471 -> Patchwork_121905v2

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2) URL : https://patchwork.freedesktop.org/series/121905/ State : warning == Summary == Error: dim sparse failed /home/kbuild/linux/maintainer-tools/dim:

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v2,1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete (rev2) URL : https://patchwork.freedesktop.org/series/121905/ State : warning == Summary == Error: dim checkpatch failed

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/huc: silence injected failure in the load via GSC path (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/huc: silence injected failure in the load via GSC path (rev2) URL : https://patchwork.freedesktop.org/series/121080/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471 -> Patchwork_121080v2

Re: [Intel-gfx] [PATCH v4 1/1] drm/i915/pxp: Optimize GET_PARAM:PXP_STATUS

2023-08-03 Thread Teres Alexis, Alan Previn
On Wed, 2023-08-02 at 11:25 -0700, Teres Alexis, Alan Previn wrote: > After recent discussions with Mesa folks, it was requested > that we optimize i915's GET_PARAM for the PXP_STATUS without > changing the UAPI spec. > > Add these additional optimizations: >- If any PXP initializatoin flow

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/huc: silence injected failure in the load via GSC path (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/huc: silence injected failure in the load via GSC path (rev2) URL : https://patchwork.freedesktop.org/series/121080/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/huc: silence injected failure in the load via GSC path (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/huc: silence injected failure in the load via GSC path (rev2) URL : https://patchwork.freedesktop.org/series/121080/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No such

Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/pxp/mtl: intel_pxp_init_hw needs runtime-pm inside pm-complete

2023-08-03 Thread Ceraolo Spurio, Daniele
On 8/2/2023 12:06 PM, Alan Previn wrote: In the case of failed suspend flow or cases where the kernel does not go into full suspend but goes from suspend_prepare back to resume_complete, we get called for a pm_complete but without runtime_pm guaranteed. Thus, ensure we take the runtime_pm

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Dmitry Baryshkov
On Thu, 3 Aug 2023 at 23:47, Simon Ser wrote: > > On Thursday, August 3rd, 2023 at 22:44, Laurent Pinchart > wrote: > > > On Thu, Aug 03, 2023 at 03:31:16PM +, Simon Ser wrote: > > > > > On Thursday, August 3rd, 2023 at 17:22, Simon Ser cont...@emersion.fr > > > wrote: > > > > > > > The

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Simon Ser
On Thursday, August 3rd, 2023 at 22:44, Laurent Pinchart wrote: > On Thu, Aug 03, 2023 at 03:31:16PM +, Simon Ser wrote: > > > On Thursday, August 3rd, 2023 at 17:22, Simon Ser cont...@emersion.fr wrote: > > > > > The KMS docs describe "subconnector" to be defined as "downstream port" >

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Laurent Pinchart
On Thu, Aug 03, 2023 at 03:31:16PM +, Simon Ser wrote: > On Thursday, August 3rd, 2023 at 17:22, Simon Ser wrote: > > > The KMS docs describe "subconnector" to be defined as "downstream port" for > > DP. > > Can USB-C (or USB) be seen as a DP downstream port? > > To expand on this a bit:

Re: [Intel-gfx] [PATCH v5 19/22] drm: restore CONFIG_DRM_USE_DYNAMIC_DEBUG un-BROKEN

2023-08-03 Thread jim . cromie
On Thu, Aug 3, 2023 at 1:14 AM kernel test robot wrote: > > > hi, Jim Cromie, > > we send this report to you to consult that if there is any limitation to use > this CONFIG_DRM_USE_DYNAMIC_DEBUG? > attached config is a randconfig which has CONFIG_DRM_USE_DYNAMIC_DEBUG, the > kernel built with it

Re: [Intel-gfx] [PATCH v5 19/22] drm: restore CONFIG_DRM_USE_DYNAMIC_DEBUG un-BROKEN

2023-08-03 Thread jim . cromie
g-x015-20230731/gcc-12/fb82a8bb4e30dcf042c48563987ad3a24a416f5d/vmlinuz-6.5.0-rc2-00390-gfb82a8bb4e30 branch=linux-review/Jim-Cromie/drm-use-correct-ccflags-y-syntax/20230802-010749 job=/lkp/jobs/scheduled/vm-meta-58/boot-1-yocto-i386-minimal-20190520.cgz-x86_64-randconfig-x015-20230731-fb82a8b

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/sdvo: fix panel_type initialization

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: fix panel_type initialization URL : https://patchwork.freedesktop.org/series/121955/ State : success == Summary == CI Bug Log - changes from CI_DRM_13469_full -> Patchwork_121955v1_full Summary

[Intel-gfx] [PULL] drm-intel-next

2023-08-03 Thread Rodrigo Vivi
Hi Dave and Daniel, Here goes our first pull request of this round. drm-intel-next-2023-08-03: - Removing unused declarations (Arnd, Gustavo) - ICL+ DSI modeset sequence fixes (Ville) - Improvements on HDCP (Suraj) - Fixes and clean up on MTL Display (Mika Kahola, Lee, RK, Nirmoy, Chaitanya) -

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/vdsc: Fix first_line_bpg_offset calculation

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/vdsc: Fix first_line_bpg_offset calculation URL : https://patchwork.freedesktop.org/series/121967/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471 -> Patchwork_121967v1 Summary

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/vdsc: Fix first_line_bpg_offset calculation

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/vdsc: Fix first_line_bpg_offset calculation URL : https://patchwork.freedesktop.org/series/121967/ State : warning == Summary == Error: dim sparse failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No such file or directory

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/vdsc: Fix first_line_bpg_offset calculation

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/vdsc: Fix first_line_bpg_offset calculation URL : https://patchwork.freedesktop.org/series/121967/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No such file or directory

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect URL : https://patchwork.freedesktop.org/series/121965/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471 -> Patchwork_121965v1

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect URL : https://patchwork.freedesktop.org/series/121965/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect URL : https://patchwork.freedesktop.org/series/121965/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h URL : https://patchwork.freedesktop.org/series/121959/ State : success == Summary == CI Bug Log - changes from CI_DRM_13471 -> Patchwork_121959v1

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h URL : https://patchwork.freedesktop.org/series/121959/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such file

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Patchwork
== Series Details == Series: series starting with [v4,1/1] drm/i915: Move abs_diff() to math.h URL : https://patchwork.freedesktop.org/series/121959/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such

Re: [Intel-gfx] [PATCH v4 1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Andrew Morton
On Thu, 3 Aug 2023 16:19:18 +0300 Andy Shevchenko wrote: > abs_diff() belongs to math.h. Move it there. > This will allow others to use it. > > ... > > --- a/include/linux/math.h > +++ b/include/linux/math.h > @@ -155,6 +155,13 @@ __STRUCT_FRACT(u32) >

[Intel-gfx] ✗ Fi.CI.BAT: failure for Add DSC PPS readout (rev6)

2023-08-03 Thread Patchwork
== Series Details == Series: Add DSC PPS readout (rev6) URL : https://patchwork.freedesktop.org/series/120456/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13471 -> Patchwork_120456v6 Summary --- **FAILURE**

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Add DSC PPS readout (rev6)

2023-08-03 Thread Patchwork
== Series Details == Series: Add DSC PPS readout (rev6) URL : https://patchwork.freedesktop.org/series/120456/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such file or directory

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add DSC PPS readout (rev6)

2023-08-03 Thread Patchwork
== Series Details == Series: Add DSC PPS readout (rev6) URL : https://patchwork.freedesktop.org/series/120456/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No such file or directory

Re: [Intel-gfx] [PATCH] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread Kandpal, Suraj
> Thank you for the comments. It seems the correction is for DSC 1.1 and DSC > 1.2 at the same time. > Can you submit a patch for the correction? Because the original patch is > specifically for the original specification DSC 1.1. > Hi I have sent a patch fixing the dsc

[Intel-gfx] [PATCH] drm/i915/vdsc: Fix first_line_bpg_offset calculation

2023-08-03 Thread Suraj Kandpal
On checking DSC1.1 Errata and DSC 1.2 spec the current formula we were using was incorrect to calculate first_line_bpg_offset. The new fixed formula is derived from C model. Signed-off-by: Suraj Kandpal --- drivers/gpu/drm/i915/display/intel_vdsc.c | 15 --- 1 file changed, 12

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Dmitry Baryshkov
On Thu, 3 Aug 2023 at 18:43, Simon Ser wrote: > > On Thursday, August 3rd, 2023 at 17:36, Dmitry Baryshkov > wrote: > > > On Thu, 3 Aug 2023 at 18:31, Simon Ser cont...@emersion.fr wrote: > > > > > On Thursday, August 3rd, 2023 at 17:22, Simon Ser cont...@emersion.fr > > > wrote: > > > > > > >

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Simon Ser
On Thursday, August 3rd, 2023 at 17:36, Dmitry Baryshkov wrote: > On Thu, 3 Aug 2023 at 18:31, Simon Ser cont...@emersion.fr wrote: > > > On Thursday, August 3rd, 2023 at 17:22, Simon Ser cont...@emersion.fr wrote: > > > > > The KMS docs describe "subconnector" to be defined as "downstream

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Dmitry Baryshkov
On Thu, 3 Aug 2023 at 18:31, Simon Ser wrote: > > On Thursday, August 3rd, 2023 at 17:22, Simon Ser wrote: > > > The KMS docs describe "subconnector" to be defined as "downstream port" for > > DP. > > Can USB-C (or USB) be seen as a DP downstream port? > > To expand on this a bit: I'm wondering

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Simon Ser
On Thursday, August 3rd, 2023 at 17:22, Simon Ser wrote: > The KMS docs describe "subconnector" to be defined as "downstream port" for > DP. > Can USB-C (or USB) be seen as a DP downstream port? To expand on this a bit: I'm wondering if we're mixing apples and oranges here. The current values

Re: [Intel-gfx] [PATCH] drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect

2023-08-03 Thread Jani Nikula
On Thu, 03 Aug 2023, Juha-Pekka Heikkila wrote: > If drm_edid_raw returned NULL on error don't try accessing anything behind > that NULL > > Signed-off-by: Juha-Pekka Heikkila Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_sdvo.c | 2 +- > 1 file changed, 1 insertion(+),

Re: [Intel-gfx] [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-03 Thread Simon Ser
On Wednesday, August 2nd, 2023 at 21:23, Dmitry Baryshkov wrote: > >> >> + { DRM_MODE_SUBCONNECTOR_USB, "USB" }, /* DP */ > >> > > >> > Should this be DRM_MODE_SUBCONNECTOR_USB_C and "USB-C", in case we get > >> > another USB type later ? > >> > >> Hmm, which id should I use

[Intel-gfx] [PULL] drm-misc-next

2023-08-03 Thread Thomas Zimmermann
Hi Dave and Daniel, this is the pull request for drm-misc-next for this week. There are plenty of small fixes and cleanups, but nothing stands out. Best regards Thomas drm-misc-next-2023-08-03: drm-misc-next for v6.6: UAPI Changes: * virtio: * Support sync objects Cross-subsystem

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev3)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev3) URL : https://patchwork.freedesktop.org/series/121882/ State : success == Summary == CI Bug Log - changes from CI_DRM_13469 -> Patchwork_121882v3

[Intel-gfx] [PATCH] drm/i915/sdvo: ignore returned broken edid on intel_sdvo_tmds_sink_detect

2023-08-03 Thread Juha-Pekka Heikkila
If drm_edid_raw returned NULL on error don't try accessing anything behind that NULL Signed-off-by: Juha-Pekka Heikkila --- drivers/gpu/drm/i915/display/intel_sdvo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_sdvo.c

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev3)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev3) URL : https://patchwork.freedesktop.org/series/121882/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such file

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev3)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev3) URL : https://patchwork.freedesktop.org/series/121882/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild/linux/maintainer-tools/dim: line 50: /home/kbuild/.dimrc: No such

Re: [Intel-gfx] [PATCH v3] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread Tseng, William
Thank you for the comment. I am not aware of the Errata of DSC 1.1. Then this patch does not seem applicable to the revised calculation of the parameter first_line_bpg_offset. Maybe a new patch should be created for the correction, in order to fix the problem which is not found on the SOC

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/sdvo: fix panel_type initialization

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: fix panel_type initialization URL : https://patchwork.freedesktop.org/series/121955/ State : success == Summary == CI Bug Log - changes from CI_DRM_13469 -> Patchwork_121955v1 Summary ---

Re: [Intel-gfx] [PATCH] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread Tseng, William
Thank you for the comments. It seems the correction is for DSC 1.1 and DSC 1.2 at the same time. Can you submit a patch for the correction? Because the original patch is specifically for the original specification DSC 1.1. -Original Message- From: Nautiyal, Ankit K Sent: Wednesday,

Re: [Intel-gfx] [PATCH 1/4] drm/i915/cx0: Add intel_cx0_get_owned_lane_mask()

2023-08-03 Thread Gustavo Sousa
Quoting Taylor, Clinton A (2023-08-02 18:41:27-03:00) >On Tue, 2023-07-25 at 18:27 -0300, Gustavo Sousa wrote: >> There are more parts of C10/C20 programming that need to take owned >> lanes into account. Define the function intel_cx0_get_owned_lane_mask() >> and use it. There will be new users of

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/sdvo: fix panel_type initialization

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: fix panel_type initialization URL : https://patchwork.freedesktop.org/series/121955/ State : warning == Summary == Error: dim sparse failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such file or directory

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/sdvo: fix panel_type initialization

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/sdvo: fix panel_type initialization URL : https://patchwork.freedesktop.org/series/121955/ State : warning == Summary == Error: dim checkpatch failed /home/kbuild2/linux/maintainer-tools/dim: line 50: /home/kbuild2/.dimrc: No such file or directory

Re: [Intel-gfx] [PATCH] drm/i915/guc: Fix potential null pointer deref in GuC 'steal id' test

2023-08-03 Thread Andi Shyti
Hi John, On Wed, Aug 02, 2023 at 11:49:40AM -0700, john.c.harri...@intel.com wrote: > From: John Harrison > > It was noticed that if the very first 'stealing' request failed to > create for some reason then the 'steal all ids' loop would immediately > exit with 'last' still being NULL. The test

[Intel-gfx] [PATCH v4 1/1] drm/i915: Move abs_diff() to math.h

2023-08-03 Thread Andy Shevchenko
abs_diff() belongs to math.h. Move it there. This will allow others to use it. Reviewed-by: Jiri Slaby # tty/serial Acked-by: Jani Nikula Acked-by: Greg Kroah-Hartman Reviewed-by: Andi Shyti Reviewed-by: Philipp Zabel # gpu/ipu-v3 Signed-off-by: Andy Shevchenko --- v4: - Cc'ed to Andrew (as

[Intel-gfx] [PATCH v6 8/8] drm/i915/display: Compare the readout dsc pps params

2023-08-03 Thread Suraj Kandpal
With the dsc config being readout and filled in crtc_state add macros and use them to compare current and previous PPS param in DSC. --v2 -Remove version check [Jani] -Remove dupe macro for dsc pipe compare and use the existing ones [Jani] Signed-off-by: Suraj Kandpal ---

[Intel-gfx] [PATCH v6 7/8] drm/i915/vdsc: Fill the intel_dsc_get_pps_config function

2023-08-03 Thread Suraj Kandpal
We have setup both the read and write functions so we can move ahead and fill in all the readout state from PPS register into the crtc_state so we can send it for comparision. --v2 -Shorten comment to just PPSX rather than having the whole "Readout PPSX register" [Jani] -Remove pps_temp

[Intel-gfx] [PATCH v6 6/8] drm/i915/vdsc: Remove unused dsc registers

2023-08-03 Thread Suraj Kandpal
Now that we have macros that can fetch dsc register values based on pipe and pps parameters we can go ahead and remove all the unused register. Signed-off-by: Suraj Kandpal --- .../gpu/drm/i915/display/intel_vdsc_regs.h| 259 ++ 1 file changed, 24 insertions(+), 235

[Intel-gfx] [PATCH v6 5/8] drm/i915/vdsc: Add function to write in PPS register

2023-08-03 Thread Suraj Kandpal
Now that we have a function that reads any PPS register based on intel_dsc_pps enum provided lets create a function that can write on any PPS. --v2 -Changes need as PPS enum was dropped -Remove duplicated code in intel_dsc_write_pps_reg [Jani] --v3 -Use dsc_split instead of num_vdsc_instances

[Intel-gfx] [PATCH v6 4/8] drm/i915/vdsc: Add function to read any PPS register

2023-08-03 Thread Suraj Kandpal
Add function to read any PPS register based on the intel_dsc_pps enum provided. Add a function which will call the new pps read function and place it in crtc state. Only PPS0 and PPS1 are readout the rest of the registers will be read in upcoming patches. --v2 -Changes in read function as PPS

[Intel-gfx] [PATCH v6 3/8] drm/i915/vdsc: Add func to get no. of vdsc instances per pipe

2023-08-03 Thread Suraj Kandpal
We have a function that gets us the total of the vdsc engines being used but not the no. of vdsc instances being used by each pipe. --v6 -Change function to static Signed-off-by: Suraj Kandpal --- drivers/gpu/drm/i915/display/intel_vdsc.c | 78 +-- 1 file changed, 44

[Intel-gfx] [PATCH v6 2/8] drm/i915/vdsc: Add a check for dsc split cases

2023-08-03 Thread Suraj Kandpal
In intel_vdsc_get_config we only read the primary dsc engine register and not take into account if the other dsc engine is in use and if both registers have the same value or not this patche fixes that by adding a check. --v3 -Remove superfluos new line [Jani] -Fix register naming [Jani] --v5

[Intel-gfx] [PATCH v6 1/8] drm/i915/vdsc: Refactor dsc register field macro

2023-08-03 Thread Suraj Kandpal
This patch refactors dsc register related macros that prepares the values to be written in the register. The current bit shifting looks bad and going forward will not serve our purpose to readout dsc register field values the change was suggested by Jani Nikula. Cc: Jani Nikula Signed-off-by:

[Intel-gfx] [PATCH v6 0/8] Add DSC PPS readout

2023-08-03 Thread Suraj Kandpal
Up until now we only verified one or two of the dsc pps params like bits_per_component and bits_per_pixel this patch series aim to readout almost all PPS param and get them compared. Along with that some work on making a common function to read and write PPS param regiters is also done. --v2

Re: [Intel-gfx] [PATCH v4] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-08-03 Thread Daniel Vetter
On Thu, 27 Jul 2023 at 22:13, Zhanjun Dong wrote: > > This attempts to avoid circular locking dependency between flush delayed work > and intel_gt_reset. > Switched from cancel_delayed_work_sync to cancel_delayed_work, the non-sync > version for reset path, it is safe as the worker has the

Re: [Intel-gfx] [PATCH v3] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread Kandpal, Suraj
> This change is required for DSC 1.1 because the current calculation is for DSC > 1.2 and may get a calculated value which is not recommended by DSC 1.1, for > example, the calculated value at 8bpp becomes 15, not the value of 12 > recommened by DSC 1.1. > Hi Tseng, Please find the comments

Re: [Intel-gfx] [PATCH] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread Tseng, William
Thank you for the comments. I have made the corresponding changes to the revised patch in version No.3. -Original Message- From: Kandpal, Suraj Sent: Wednesday, August 2, 2023 7:15 PM To: Tseng, William ; intel-gfx@lists.freedesktop.org Cc: Heikkila, Juha-pekka ; Tseng, William ;

[Intel-gfx] [PATCH v3] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread William Tseng
This change is required for DSC 1.1 because the current calculation is for DSC 1.2 and may get a calculated value which is not recommended by DSC 1.1, for example, the calculated value at 8bpp becomes 15, not the value of 12 recommened by DSC 1.1. v2: - change the if-condition from minor version

Re: [Intel-gfx] [PATCH v4] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-08-03 Thread Andi Shyti
Hi Zhanjun, On Thu, Jul 27, 2023 at 01:13:23PM -0700, Zhanjun Dong wrote: > This attempts to avoid circular locking dependency between flush delayed work > and intel_gt_reset. > Switched from cancel_delayed_work_sync to cancel_delayed_work, the non-sync > version for reset path, it is safe as

[Intel-gfx] [PATCH] drm/i915/sdvo: fix panel_type initialization

2023-08-03 Thread Jani Nikula
Commit 3f9ffce5765d ("drm/i915: Do panel VBT init early if the VBT declares an explicit panel type") started using -1 as the value for unset panel_type. It gets initialized in intel_panel_init_alloc(), but the SDVO code never calls it. Call intel_panel_init_alloc() to initialize the panel,

[Intel-gfx] ✗ Fi.CI.BUILD: failure for Add DSC PPS readout (rev5)

2023-08-03 Thread Patchwork
== Series Details == Series: Add DSC PPS readout (rev5) URL : https://patchwork.freedesktop.org/series/120456/ State : failure == Summary == Error: make failed CALLscripts/checksyscalls.sh DESCEND objtool INSTALL libsubcmd_headers CC [M] drivers/gpu/drm/i915/display/intel_vdsc.o

[Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev2)

2023-08-03 Thread Patchwork
== Series Details == Series: drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1 (rev2) URL : https://patchwork.freedesktop.org/series/121882/ State : failure == Summary == Error: make failed CALLscripts/checksyscalls.sh DESCEND objtool INSTALL libsubcmd_headers CC [M]

Re: [Intel-gfx] [PATCH v2 2/3] drm/i915/fbc: Make FBC check stolen at use time

2023-08-03 Thread B, Jeevan
> -Original Message- > From: Intel-gfx On Behalf Of > Jouni Högander > Sent: Wednesday, June 14, 2023 10:48 AM > To: intel-gfx@lists.freedesktop.org > Subject: [Intel-gfx] [PATCH v2 2/3] drm/i915/fbc: Make FBC check stolen at > use time > > As a preparation for Xe change stolen memory

Re: [Intel-gfx] [PATCH v5 3/8] drm/i915/vdsc: Add func to get no. of vdsc instances per pipe

2023-08-03 Thread kernel test robot
Hi Suraj, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-tip/drm-tip] url: https://github.com/intel-lab-lkp/linux/commits/Suraj-Kandpal/drm-i915-vdsc-Refactor-dsc-register-field-macro/20230803-151602 base: git://anongit.freedesktop.org/drm/drm-tip

Re: [Intel-gfx] [PATCH] drm/i915: Hold reference to intel_context over life of i915_request

2023-08-03 Thread Andi Shyti
Hi Andrzej, On Fri, Jul 28, 2023 at 09:54:50AM +0200, Andrzej Hajda wrote: > References to i915_requests may be trapped by userspace inside a > sync_file or dmabuf (dma-resv) and held indefinitely across different > proceses. To counter-act the memory leaks, we try to not to keep nit: lose one

Re: [Intel-gfx] [PATCH v5 3/8] drm/i915/vdsc: Add func to get no. of vdsc instances per pipe

2023-08-03 Thread kernel test robot
Hi Suraj, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-tip/drm-tip] url: https://github.com/intel-lab-lkp/linux/commits/Suraj-Kandpal/drm-i915-vdsc-Refactor-dsc-register-field-macro/20230803-151602 base: git://anongit.freedesktop.org/drm/drm-tip

Re: [Intel-gfx] [PATCH v2 1/3] drm/i915: Move stolen memory handling into i915_gem_stolen

2023-08-03 Thread Nirmoy Das
Hi Jouni, On 8/2/2023 9:52 AM, Hogander, Jouni wrote: On Wed, 2023-08-02 at 09:51 +0200, Nirmoy Das wrote: On 8/1/2023 10:33 AM, Hogander, Jouni wrote: On Tue, 2023-08-01 at 10:02 +0200, Nirmoy Das wrote: Hi Jouni, On 6/14/2023 7:17 AM, Jouni Högander wrote: We are preparing for Xe. Xe

Re: [Intel-gfx] [PATCH 5/5] drm/i915: Implement fdinfo memory stats printing

2023-08-03 Thread Tvrtko Ursulin
On 03/08/2023 06:15, Iddamsetty, Aravind wrote: On 27-07-2023 15:43, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Use the newly added drm_print_memory_stats helper to show memory utilisation of our objects in drm/driver specific fdinfo output. To collect the stats we walk the per memory

[Intel-gfx] [PULL] drm-intel-fixes

2023-08-03 Thread Tvrtko Ursulin
Hi Dave, Daniel, Some fixes for the 6.5 RC this week: one for GVT display I2C handling, which came via gvt-fixes merge, one for premature freeing of request memory, and finally one fix for Gen12 AUX invalidatation flow to correctly align it with the documented sequence. Regards, Tvrtko

[Intel-gfx] [PATCH v3] drm/modes: Fix division by zero due to overflow

2023-08-03 Thread Ziqi Zhao
In the bug reported by Syzbot, the variable `den == (1 << 22)` and `mode->vscan == (1 << 10)`, causing the multiplication to overflow and accidentally make `den == 0`. To prevent any chance of overflow, we replace `num` and `den` with 64-bit unsigned integers, and explicitly check if the divisor

[Intel-gfx] [PULL] drm-misc-fixes

2023-08-03 Thread Maxime Ripard
Hi, Here's this week drm-misc-fixes PR Maxime The following changes since commit 39b1320e5dc2b707dfb5c25b0298ce9d4fc05aea: drm/fb-helper: Remove unused inline function drm_fb_helper_defio_init() (2023-07-25 20:38:37 +0200) are available in the Git repository at:

[Intel-gfx] [PATCH v5 6/8] drm/i915/vdsc: Remove unused dsc registers

2023-08-03 Thread Suraj Kandpal
Now that we have macros that can fetch dsc register values based on pipe and pps parameters we can go ahead and remove all the unused register. Signed-off-by: Suraj Kandpal --- .../gpu/drm/i915/display/intel_vdsc_regs.h| 259 ++ 1 file changed, 24 insertions(+), 235

[Intel-gfx] [PATCH v5 5/8] drm/i915/vdsc: Add function to write in PPS register

2023-08-03 Thread Suraj Kandpal
Now that we have a function that reads any PPS register based on intel_dsc_pps enum provided lets create a function that can write on any PPS. --v2 -Changes need as PPS enum was dropped -Remove duplicated code in intel_dsc_write_pps_reg [Jani] --v3 -Use dsc_split instead of num_vdsc_instances

[Intel-gfx] [PATCH v5 4/8] drm/i915/vdsc: Add function to read any PPS register

2023-08-03 Thread Suraj Kandpal
Add function to read any PPS register based on the intel_dsc_pps enum provided. Add a function which will call the new pps read function and place it in crtc state. Only PPS0 and PPS1 are readout the rest of the registers will be read in upcoming patches. --v2 -Changes in read function as PPS

[Intel-gfx] [PATCH v5 7/8] drm/i915/vdsc: Fill the intel_dsc_get_pps_config function

2023-08-03 Thread Suraj Kandpal
We have setup both the read and write functions so we can move ahead and fill in all the readout state from PPS register into the crtc_state so we can send it for comparision. --v2 -Shorten comment to just PPSX rather than having the whole "Readout PPSX register" [Jani] -Remove pps_temp

[Intel-gfx] [PATCH v5 2/8] drm/i915/vdsc: Add a check for dsc split cases

2023-08-03 Thread Suraj Kandpal
In intel_vdsc_get_config we only read the primary dsc engine register and not take into account if the other dsc engine is in use and if both registers have the same value or not this patche fixes that by adding a check. --v3 -Remove superfluos new line [Jani] -Fix register naming [Jani] --v5

[Intel-gfx] [PATCH v5 8/8] drm/i915/display: Compare the readout dsc pps params

2023-08-03 Thread Suraj Kandpal
With the dsc config being readout and filled in crtc_state add macros and use them to compare current and previous PPS param in DSC. --v2 -Remove version check [Jani] -Remove dupe macro for dsc pipe compare and use the existing ones [Jani] Signed-off-by: Suraj Kandpal ---

[Intel-gfx] [PATCH v5 1/8] drm/i915/vdsc: Refactor dsc register field macro

2023-08-03 Thread Suraj Kandpal
This patch refactors dsc register related macros that prepares the values to be written in the register. The current bit shifting looks bad and going forward will not serve our purpose to readout dsc register field values the change was suggested by Jani Nikula. Cc: Jani Nikula Signed-off-by:

[Intel-gfx] [PATCH v5 3/8] drm/i915/vdsc: Add func to get no. of vdsc instances per pipe

2023-08-03 Thread Suraj Kandpal
We have a function that gets us the total of the vdsc engines being used but not the no. of vdsc instances being used by each pipe. Signed-off-by: Suraj Kandpal --- drivers/gpu/drm/i915/display/intel_vdsc.c | 78 +-- 1 file changed, 44 insertions(+), 34 deletions(-) diff

[Intel-gfx] [PATCH v5 0/8] Add DSC PPS readout

2023-08-03 Thread Suraj Kandpal
Up until now we only verified one or two of the dsc pps params like bits_per_component and bits_per_pixel this patch series aim to readout almost all PPS param and get them compared. Along with that some work on making a common function to read and write PPS param regiters is also done. --v2

[Intel-gfx] [PATCH v2] drm/i915/xelpd: Calculate first_line_bpg_offset for DSC 1.1

2023-08-03 Thread William Tseng
This change is required for DSC 1.1 because the current calculation is for DSC 1.2 and may get a calculated value which is not recommended by DSC 1.1, for example, the calculated value at 8bpp becomes 15, not the value of 12 recommened by DSC 1.1. v2: - change the if-condition from minor version