[Intel-gfx] [PATCH 2/2] drm/i915: Add DSC support to MST path

2022-03-17 Thread Stanislav Lisovskiy
purposes(Manasi Navare) - Properly process ret == EDEADLK, thus fixing the regression caused by WARN triggered with modeset_lock. v5: - Removed redundant check(Imre Deak) Acked-by: Imre Deak Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 138

[Intel-gfx] [PATCH 1/2] drm: Add missing DP DSC extended capability definitions.

2022-03-17 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/dp/drm_dp.c| 25 + include/drm/dp/drm_dp_helper.h | 11 ++- 2 files

[Intel-gfx] [PATCH] drm/i915: Write zero wms if we disable planes for icl+

2022-05-18 Thread Stanislav Lisovskiy
Otherwise we seem to get FIFO underruns. It is being disabled anyway, so kind of logical to write those as zeroes, even if disabling is temporary. Signed-off-by: Stanislav Lisovskiy --- .../drm/i915/display/skl_universal_plane.c| 2 +- drivers/gpu/drm/i915/intel_pm.c | 46

[Intel-gfx] [PATCH 1/1] drm/i915/dg2: Bump up CDCLK for DG2

2022-06-14 Thread Stanislav Lisovskiy
We seem to need this W/A same way as for TGL, in order to fix some of the underruns, which we currently have and those not related to PSR. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Intel-gfx] [PATCH 0/1] Bump up CDCLK for DG2

2022-06-14 Thread Stanislav Lisovskiy
We set it to be equal to pixel rate here, just same as we have already for TGL, as that seems to be currently the only solution to fix underruns, not related to PSR. Stanislav Lisovskiy (1): drm/i915/dg2: Bump up CDCLK for DG2 drivers/gpu/drm/i915/display/intel_cdclk.c | 4 ++-- 1 file

[Intel-gfx] [PATCH 0/1] Do not enable PSR2 if no active planes

2022-06-14 Thread Stanislav Lisovskiy
We seem to cause FIFO underruns by doing that. Also it doesn't make sense. Stanislav Lisovskiy (1): drm/i915: Do not enable PSR2/selective fetch if there are no planes drivers/gpu/drm/i915/display/intel_psr.c | 6 ++ 1 file changed, 6 insertions(+) -- 2.24.1.485.gad05a3d8e5

[Intel-gfx] [PATCH 1/1] drm/i915: Do not enable PSR2/selective fetch if there are no planes

2022-06-14 Thread Stanislav Lisovskiy
We seem to enable PSR2 and selective fetch even if there are no active planes. That seems to causes FIFO underruns at least for ADLP. Those are gone if we don't do that. Just adding simple check in intel_psr2_sel_fetch_config_valid seems to do the trick. Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-08-29 Thread Stanislav Lisovskiy
configuration, which isn't the case here. So added that drm_dp_mst_check here, so that we can make sure that try all the bpps before we fail. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff

[Intel-gfx] [PATCH 4/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-08-29 Thread Stanislav Lisovskiy
We are using almost same code to loop through bpps while calling drm_dp_atomic_find_vcpi_slots - lets remove this duplication by introducing a new function intel_dp_mst_find_vcpi_slots_for_bpp Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 88

[Intel-gfx] [PATCH 4/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-08-29 Thread Stanislav Lisovskiy
We are using almost same code to loop through bpps while calling drm_dp_atomic_find_vcpi_slots - lets remove this duplication by introducing a new function intel_dp_mst_find_vcpi_slots_for_bpp v2: Fix pbn_div calculation - shouldn't matter if its DSC or not. Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 1/4] drm: Add missing DP DSC extended capability definitions.

2022-08-29 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 3/4] drm/i915: Add DSC support to MST path

2022-08-29 Thread Stanislav Lisovskiy
ion in intel_dp_dsc_compute_config (don't remember when I lost it) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 73 - drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 157 3 fi

[Intel-gfx] [PATCH 0/4] Add DP MST DSC support to i915

2022-08-29 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (4): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Add DSC support to MST path drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate

[Intel-gfx] [PATCH 2/2] drm/i915: Add DSC support to MST path

2022-08-22 Thread Stanislav Lisovskiy
ion in intel_dp_dsc_compute_config (don't remember when I lost it) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 73 - drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 157 3 fi

[Intel-gfx] [PATCH 1/2] drm: Add missing DP DSC extended capability definitions.

2022-08-22 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Signed-off-by: Stanislav Lisovskiy --- include/drm/display/drm_dp.h

[Intel-gfx] [PATCH 0/2] Add DP MST DSC support to i915

2022-08-22 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (2): drm: Add missing DP DSC extended capability definitions. drm/i915: Add DSC support to MST path drivers/gpu/drm/i915/display/intel_dp.c | 73 - drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers

[Intel-gfx] [PATCH 3/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-09-01 Thread Stanislav Lisovskiy
We would be using almost same code to loop through bpps while calling drm_dp_atomic_find_vcpi_slots - lets remove this duplication by introducing a new function intel_dp_mst_find_vcpi_slots_for_bpp v2: Fix pbn_div calculation - shouldn't matter if its DSC or not. Signed-off-by: Stanislav

[Intel-gfx] [PATCH 4/4] drm/i915: Add DSC support to MST path

2022-09-01 Thread Stanislav Lisovskiy
ion in intel_dp_dsc_compute_config (don't remember when I lost it) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 73 +-- drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 136 3 fi

[Intel-gfx] [PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-09-01 Thread Stanislav Lisovskiy
configuration, which isn't the case here. So added that drm_dp_mst_check here, so that we can make sure that try all the bpps before we fail. Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 ++-- 1 file changed, 14

[Intel-gfx] [PATCH 1/4] drm: Add missing DP DSC extended capability definitions.

2022-09-01 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 0/4] Add DP MST DSC support to i915

2022-09-01 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (4): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function drm/i915: Add DSC support to MST

[Intel-gfx] [PATCH] drm/i915: Start using REG_BIT* macros

2022-09-01 Thread Stanislav Lisovskiy
Lets start to use REG_BIT* macros, instead of (x << 0) like expressions. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/i915_reg.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/dr

[Intel-gfx] [PATCH] drm/i915: Fix intel_dp_atomic_find_vcpi_slots function

2022-09-06 Thread Stanislav Lisovskiy
drm_dp_atomic_find_vcpi_slots no longer exists and needs to be used as drm_dp_atomic_find_time_slots. Also rename the function itself. Signed-off-by: Stanislav Lisovskiy Fixes: 7ae5ab441402 ("Extract drm_dp_atomic_find_vcpi_slots cycle to separate function") --- drivers/gpu/drm/i9

[Intel-gfx] [PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-08-26 Thread Stanislav Lisovskiy
configuration, which isn't the case here. So added that drm_dp_mst_check here, so that we can make sure that try all the bpps before we fail. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff

[Intel-gfx] [PATCH 3/4] drm/i915: Add DSC support to MST path

2022-08-26 Thread Stanislav Lisovskiy
ion in intel_dp_dsc_compute_config (don't remember when I lost it) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 73 - drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 157 3 fi

[Intel-gfx] [PATCH 1/4] drm: Add missing DP DSC extended capability definitions.

2022-08-26 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 0/4] Add DP MST DSC support to i915

2022-08-26 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (4): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Add DSC support to MST path drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate

[Intel-gfx] [PATCH 4/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-08-26 Thread Stanislav Lisovskiy
We are using almost same code to loop through bpps while calling drm_dp_atomic_find_vcpi_slots - lets remove this duplication by introducing a new function intel_dp_mst_find_vcpi_slots_for_bpp Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 85

[Intel-gfx] [PATCH 1/4] drm: Add missing DP DSC extended capability definitions.

2022-09-13 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 4/4] drm/i915: Add DSC support to MST path

2022-09-13 Thread Stanislav Lisovskiy
off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 73 +--- drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 125 3 files changed, 173 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/i

[Intel-gfx] [PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-09-13 Thread Stanislav Lisovskiy
configuration, which isn't the case here. So added that drm_dp_mst_check here, so that we can make sure that try all the bpps before we fail. Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 ++-- 1 file changed, 14

[Intel-gfx] [PATCH 3/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-09-13 Thread Stanislav Lisovskiy
, constant_n no longer needed. Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 48 +++-- 1 file changed, 36 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915

[Intel-gfx] [PATCH 0/4] Add DP MST DSC support to i915

2022-09-13 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (4): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function drm/i915: Add DSC support to MST

[Intel-gfx] [PATCH 4/4] drm/i915: Add DSC support to MST path

2022-09-05 Thread Stanislav Lisovskiy
off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 73 +-- drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 127 3 files changed, 175 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/i

[Intel-gfx] [PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-09-05 Thread Stanislav Lisovskiy
configuration, which isn't the case here. So added that drm_dp_mst_check here, so that we can make sure that try all the bpps before we fail. Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 ++-- 1 file changed, 14

[Intel-gfx] [PATCH 3/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-09-05 Thread Stanislav Lisovskiy
Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 52 +++-- 1 file changed, 39 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c index 24d6a287a6e3

[Intel-gfx] [PATCH 1/4] drm: Add missing DP DSC extended capability definitions.

2022-09-05 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 0/4] Add DP MST DSC support to i915

2022-09-05 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (4): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function drm/i915: Add DSC support to MST

[Intel-gfx] [PATCH 2/2] drm/i915: Add DSC support to MST path

2022-08-10 Thread Stanislav Lisovskiy
: - Rebased - Added a debug to see that we at least try reserving VCPI slots using DSC, because currently its not visible from the logs, thus making debugging more tricky. - Moved timeslots to numerator, where it should be. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 1/2] drm: Add missing DP DSC extended capability definitions.

2022-08-10 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Signed-off-by: Stanislav Lisovskiy --- include/drm/display/drm_dp.h

[Intel-gfx] [PATCH 0/2] Add DP MST DSC support to i915

2022-08-10 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (2): drm: Add missing DP DSC extended capability definitions. drm/i915: Add DSC support to MST path drivers/gpu/drm/i915/display/intel_dp.c | 76 +- drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers

[Intel-gfx] [PATCH 2/2] drm/i915: Add DSC support to MST path

2022-08-15 Thread Stanislav Lisovskiy
config computation, because we need to know already by this moment if uncompressed amount of VCPI slots needed can fit, otherwise we need to use DSC. (thanks to Vinod Govindapillai for pointing this out) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c

[Intel-gfx] [PATCH 1/2] drm: Add missing DP DSC extended capability definitions.

2022-08-15 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Signed-off-by: Stanislav Lisovskiy --- include/drm/display/drm_dp.h

[Intel-gfx] [PATCH 0/2] Add DP MST DSC support to i915

2022-08-15 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (2): drm: Add missing DP DSC extended capability definitions. drm/i915: Add DSC support to MST path drivers/gpu/drm/i915/display/intel_dp.c | 76 -- drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers

[Intel-gfx] [PATCH] drm/i915/mtl: Add support of Tile4 to MTL

2022-12-20 Thread Stanislav Lisovskiy
- lets use -1 in order to make sure all the next gens support it by default(Juha-Pekka Heikkila) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_fb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_fb.c

[Intel-gfx] [PATCH] drm/i915: Implement workaround for DP2 UHBR bandwidth check

2023-01-10 Thread Stanislav Lisovskiy
-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c index 8b0e4defa3f1..1f1f7f5f6501 100644 --- a/drivers

[Intel-gfx] [PATCH] drm/i915: Implement UHBR bandwidth check

2023-01-13 Thread Stanislav Lisovskiy
commit subject(Rodrigo Vivi) - Fixed the error message if check fails(Rodrigo Vivi) Signed-off-by: Stanislav Lisovskiy Reviewed-by: Rodrigo Vivi --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm

[Intel-gfx] [PATCH] drm/i915: Fix timeslots argument for DP DSC SST case

2023-01-09 Thread Stanislav Lisovskiy
/intel/-/issues/6860 Fixes: 52f14682ac4d ("drm/i915: Bpp/timeslot calculation fixes for DP MST DSC") Reviewed-by: Manasi Navare Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/d

[Intel-gfx] [PATCH] drm/i915: Implement workaround for DP2 UHBR bandwidth check

2023-01-02 Thread Stanislav Lisovskiy
According to spec, we should check if output_bpp * pixel_rate is less than DDI clock * 72, if UHBR is used. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b

[Intel-gfx] [PATCH] drm/i915: Fix timeslots argument for DP DSC SST case

2023-01-02 Thread Stanislav Lisovskiy
We now accept timeslots param exactly how the variable sounds: amount of timeslots, but not ratio timeslots/64. So for SST case(when we have all timeslots for use), it should be 64, but not 1. This caused some issues in the tests. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH] Copy highest enabled wm level to disabled wm levels for gen >= 12

2022-12-18 Thread Stanislav Lisovskiy
There was a specific SW workaround requested, which should prevent some watermark issues happening, which requires copying highest enabled wm level to those disabled wm levels(bit 31 is of course still needs to be cleared). Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH] drm/i915/mtl: Add support of Tile4 to MTL

2022-12-19 Thread Stanislav Lisovskiy
We have some Tile4 tests now skipping, which were supposed to be working. So lets make them work, by adding display_ver 14 as supported. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_fb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[Intel-gfx] [PATCH 1/1] drm/i915: Implement workaround for CDCLK PLL disable/enable

2022-11-24 Thread Stanislav Lisovskiy
It was reported that we might get a hung and loss of register access in some cases when CDCLK PLL is disabled and then enabled, while squashing is enabled. As a workaround it was proposed by HW team that SW should disable squashing when CDCLK PLL is being reenabled. Signed-off-by: Stanislav

[Intel-gfx] [PATCH 0/1] Implement workaround for PLL enabling for DG2/MTL

2022-11-24 Thread Stanislav Lisovskiy
and enable squashing later, if needed. Stanislav Lisovskiy (1): drm/i915: Implement workaround for CDCLK PLL disable/enable drivers/gpu/drm/i915/display/intel_cdclk.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) -- 2.37.3

[Intel-gfx] [PATCH 5/6] drm/i915: Extract VESA DSC bpp alignment to separate function

2022-11-23 Thread Stanislav Lisovskiy
multiple blank lines v3: Rename intel_dp_dsc_nearest_vesa_bpp to intel_dp_dsc_nearest_valid_bpp to reflect its meaning more properly. (Manasi Navare) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 50 + drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH 6/6] drm/i915: Bpp/timeslot calculation fixes for DP MST DSC

2022-11-23 Thread Stanislav Lisovskiy
to intel_dp_dsc_nearest_valid_bpp (Manasi Navare) Reviewed-by: Manasi Navare Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 69 ++--- drivers/gpu/drm/i915/display/intel_dp.h | 3 +- drivers/gpu/drm/i915/display/intel_dp_mst.c | 69 + 3 files

[Intel-gfx] [PATCH 2/2] drm/i915: Implement UHBR bandwidth check

2023-01-16 Thread Stanislav Lisovskiy
-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c index e3e7c305fece..b95051fed23d 100644 --- a/drivers/gpu/drm

[Intel-gfx] [PATCH 0/2] We need to have additional checks for DP MST UHBR

2023-01-16 Thread Stanislav Lisovskiy
According to BSpec UHBR might hit hw limitation which must be checked. So this series adds first some generic checker function, which might be used to add this or similar checks in future, then we introduce that particular UHBR check. Stanislav Lisovskiy (2): drm/i915: Add generic constraint

[Intel-gfx] [PATCH 1/2] drm/i915: Add generic constraint checker when determining DP MST DSC bpp

2023-01-16 Thread Stanislav Lisovskiy
There are might be multiple contraints which we need to check while determining if we can use desired compressed bpp, so might be good idea to add a special helper, so that we don't overcomplicate the main bpp calculation function. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH] drm/i915: Implement workaround for CDCLK PLL disable/enable

2023-01-23 Thread Stanislav Lisovskiy
It was reported that we might get a hung and loss of register access in some cases when CDCLK PLL is disabled and then enabled, while squashing is enabled. As a workaround it was proposed by HW team that SW should disable squashing when CDCLK PLL is being reenabled. Signed-off-by: Stanislav

[Intel-gfx] [PATCH] drm/i915: Implement workaround for CDCLK PLL disable/enable

2023-01-30 Thread Stanislav Lisovskiy
(Rodrigo Vivi) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c index 7e16b655c833

[Intel-gfx] [PATCH 6/6] drm/i915: Bpp/timeslot calculation fixes for DP MST DSC

2022-11-03 Thread Stanislav Lisovskiy
align those with VESA bpps and only then calculate required timeslots amount. Some MST hubs started to work only after third change was made. v2: Make kernel test robot happy(claimed there was unitialzed use, while there is none) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 5/6] drm/i915: Extract VESA DSC bpp alignment to separate function

2022-11-03 Thread Stanislav Lisovskiy
multiple blank lines Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 50 + drivers/gpu/drm/i915/display/intel_dp.h | 1 + drivers/gpu/drm/i915/display/intel_dp_mst.c | 1 - 3 files changed, 32 insertions(+), 20 deletions(-) diff

[Intel-gfx] [PATCH 4/6] drm/i915: Add DSC support to MST path

2022-11-01 Thread Stanislav Lisovskiy
off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 57 + drivers/gpu/drm/i915/display/intel_dp.h | 17 +++ drivers/gpu/drm/i915/display/intel_dp_mst.c | 125 3 files changed, 173 insertions(+), 26 deletions(-) diff --git a/drivers/gpu/drm/i

[Intel-gfx] [PATCH 6/6] drm/i915: Bpp/timeslot calculation fixes for DP MST DSC

2022-11-01 Thread Stanislav Lisovskiy
align those with VESA bpps and only then calculate required timeslots amount. Some MST hubs started to work only after third change was made. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 52 ++--- drivers/gpu/drm/i915/display/intel_dp.h | 3

[Intel-gfx] [PATCH 5/6] drm/i915: Extract VESA DSC bpp alignment to separate function

2022-11-01 Thread Stanislav Lisovskiy
-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 51 - drivers/gpu/drm/i915/display/intel_dp.h | 1 + 2 files changed, 33 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c

[Intel-gfx] [PATCH 2/6] drm/i915: Fix intel_dp_mst_compute_link_config

2022-11-01 Thread Stanislav Lisovskiy
configuration, which isn't the case here. So added that drm_dp_mst_check here, so that we can make sure that try all the bpps before we fail. Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 ++-- 1 file changed, 14

[Intel-gfx] [PATCH 3/6] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-11-01 Thread Stanislav Lisovskiy
, constant_n no longer needed. Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 48 +++-- 1 file changed, 36 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915

[Intel-gfx] [PATCH 1/6] drm: Add missing DP DSC extended capability definitions.

2022-11-01 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[Intel-gfx] [PATCH 0/6] Add DP MST DSC support to i915

2022-11-01 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (6): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function drm/i915: Add DSC support to MST

[Intel-gfx] [PATCH 1/2] drm/i915: Add generic constraint checker when determining DP MST DSC bpp

2023-02-02 Thread Stanislav Lisovskiy
There are might be multiple contraints which we need to check while determining if we can use desired compressed bpp, so might be good idea to add a special helper, so that we don't overcomplicate the main bpp calculation function. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 0/2] We need to have additional checks for DP MST UHBR

2023-02-02 Thread Stanislav Lisovskiy
According to BSpec UHBR might hit hw limitation which must be checked. So this series adds first some generic checker function, which might be used to add this or similar checks in future, then we introduce that particular UHBR check. Stanislav Lisovskiy (2): drm/i915: Add generic constraint

[Intel-gfx] [PATCH 2/2] drm/i915: Implement UHBR bandwidth check

2023-02-02 Thread Stanislav Lisovskiy
clock (32 bit per lane for DP2) instead of port clock in the formula(Ville Syrjälä) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c

[Intel-gfx] [PATCH] drm/i915: Power management SAGV/QGV calculation rounding fix

2023-03-10 Thread Stanislav Lisovskiy
the less demanding could be used, thus resulting in waste of power. BSpec: 64636 Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_bw.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_bw.c b/drivers/gpu/drm

[Intel-gfx] [PATCH] drm/i915/display: Communicate display power demands to pcode more accurately

2023-03-17 Thread Stanislav Lisovskiy
ved redundant kernel-doc and indentation(Jani Nikula) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 156 +++-- drivers/gpu/drm/i915/i915_reg.h| 14 ++ 2 files changed, 159 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/i

[Intel-gfx] [PATCH] drm/i915: Implement UHBR bandwidth check

2023-03-24 Thread Stanislav Lisovskiy
- Limit this to only DISPLAY_VER <= 13 HSDES: 1406899791 BSPEC: 49259 Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 29 +++-- 1 file changed, 27 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c

[Intel-gfx] [PATCH] drm/i915/display: Communicate display power demands to pcode

2023-03-20 Thread Stanislav Lisovskiy
ved redundant kernel-doc and indentation(Jani Nikula) v4: - Fixed some checkpatch warnings Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 156 +++-- drivers/gpu/drm/i915/i915_reg.h| 14 ++ 2 files changed, 159 insertions(+), 11 deleti

[Intel-gfx] [PATCH] drm/i915/display: Communicate display power demands to pcode more accurately

2023-02-27 Thread Stanislav Lisovskiy
hen any pipe power well will disable or enable. v2: - Make intel_cdclk_need_serialize static to make CI compiler happy. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 174 +++-- drivers/gpu/drm/i915/display/intel_cdclk.h | 2 + drivers/gpu/

[Intel-gfx] [PATCH] drm/i915: Implement UHBR bandwidth check

2023-02-27 Thread Stanislav Lisovskiy
According to spec, we should check if output_bpp * pixel_rate is less than DDI clock * 72, if UHBR is used. v2: - s/pipe_config/crtc_state/ (Jani Nikula) - Merged previous patch into that one, to remove empty function(Jani Nikula) HSDES: 1406899791 BSPEC: 49259 Signed-off-by: Stanislav

[Intel-gfx] [PATCH] drm/i915/display: Communicate display power demands to pcode more accurately

2023-02-27 Thread Stanislav Lisovskiy
hen any pipe power well will disable or enable. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 174 +++-- drivers/gpu/drm/i915/display/intel_cdclk.h | 2 + drivers/gpu/drm/i915/i915_reg.h| 14 ++ 3 files changed, 176 inserti

[Intel-gfx] [PATCH] drm/i915: Ensure DSC has enough BW and stays within HW limits

2023-03-06 Thread Stanislav Lisovskiy
output BPP's can only be chosen within range of 8 to 27(BSpec 49259). This all applied together allows to fix existing FIFO underruns, which we have in many DSC tests. BSpec: 49259 HSDES: 18027167222 Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp.c | 21

[Intel-gfx] [PATCH] drm/i915/mtl: Copy c10 phy pll sw state from master to slave for bigjoiner

2023-04-21 Thread Stanislav Lisovskiy
-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_display.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index bf391a6cd8d6..83c98791fea3 100644 --- a/drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH] drm/i915/display: Communicate display power demands to pcode

2023-04-25 Thread Stanislav Lisovskiy
kernel-doc and indentation(Jani Nikula) v4: - Fixed some checkpatch warnings v5: - According to HW team comments that change should affect only DG2, fix correspodent platform check to account this. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 156

[Intel-gfx] [PATCH] drm/i915: Use compressed bpp when calculating m/n value for DP MST DSC

2023-03-27 Thread Stanislav Lisovskiy
For obvious reasons, we use compressed bpp instead of pipe bpp for DSC DP SST case. Lets be consistent and use compressed bpp instead of pipe bpp, also in DP MST DSC case. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +- 1 file changed, 1 insertion

[Intel-gfx] [PATCH] drm/i915: Query compressed bpp properly using correct DPCD and DP Spec info

2023-04-06 Thread Stanislav Lisovskiy
DP_DSC_MAX_BITS_PER_PIXEL_LOW/HIGH. This might also allow us to get rid of an ugly compressed bpp recalculation, which we had to add to make some MST hubs usable. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 76 ++--- 1 file changed, 52 insertions(+), 24 deletions

[Intel-gfx] [PATCH] drm/i915: Query compressed bpp properly using correct DPCD and DP Spec info

2023-04-11 Thread Stanislav Lisovskiy
DP_DSC_MAX_BITS_PER_PIXEL_LOW/HIGH. This might also allow us to get rid of an ugly compressed bpp recalculation, which we had to add to make some MST hubs usable. v2: - Fix operator precedence Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 76 ++--- 1 file changed

[Intel-gfx] [PATCH] Copy highest enabled wm level to disabled wm levels for gen >= 9

2023-02-13 Thread Stanislav Lisovskiy
, as it seems that we needed this anyway on all of those(Ville Syrjälä) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/skl_watermark.c | 26 +++- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c b/drivers

[Intel-gfx] [PATCH] drm/i915/display: Communicate display configuration to pcode

2023-02-14 Thread Stanislav Lisovskiy
any power well will disable or enable. v2: - Fixed identation(Stanislav Lisovskiy) - Made conditions more specific(in the commit we declare that we do this for DG2 only, however that commit changes >= to == for many other platforms.(Stanislav Lisovskiy) v3: - Refactored co

[Intel-gfx] [PATCH] drm/i915/display: Communicate display configuration to pcode

2023-02-08 Thread Stanislav Lisovskiy
any power well will disable or enable. v2: - Fixed identation(Stanislav Lisovskiy) - Made conditions more specific(in the commit we declare that we do this for DG2 only, however that commit changes >= to == for many other platforms.(Stanislav Lisovskiy) Signed-off-by: Jigar

[Intel-gfx] [PATCH] drm/i915/display: Communicate display power demands to pcode

2023-05-04 Thread Stanislav Lisovskiy
kernel-doc and indentation(Jani Nikula) v4: - Fixed some checkpatch warnings v5: - According to HW team comments that change should affect only DG2, fix correspodent platform check to account this. v6: - Added one more missing IS_DG2 check(Vinod Govindapillai) Signed-off-by: Stanislav

[Intel-gfx] [PATCH] drm/i915: Start using plane scale factor for relative data rate

2023-07-14 Thread Stanislav Lisovskiy
. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c index 7d9578ebae556

[Intel-gfx] [PATCH] drm/i915: Start using plane scale factor for relative data rate

2023-07-19 Thread Stanislav Lisovskiy
fix it as described in BSpec 68907. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH 2/2] drm/i915: Don't rely that 2 VDSC engines are always enough for pixel rate

2023-07-04 Thread Stanislav Lisovskiy
ump up CDCLK many times for similar reasons. v2: - Use new intel_dsc_get_num_vdsc_instances to determine number of VDSC engines, instead of slice count(Ankit Nautiyal) v3: - s/u8/int/ (Jani Nikula) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.

[Intel-gfx] [PATCH 0/2] Calculate CDCLK more properly when DSC is enabled

2023-07-04 Thread Stanislav Lisovskiy
-compressed case it can be usualy lower) However if we have 2 DSC engines CDCLK can be ~pixel clock / 2 and so on. Lets do the estimation more properly based on amount of VDSC engines used. That most likely is going to fix some FIFO underruns, we are currently having. Stanislav Lisovskiy (2): drm/i915

[Intel-gfx] [PATCH 1/2] drm/i915: Add helper function for getting number of VDSC engines

2023-07-04 Thread Stanislav Lisovskiy
VDSC code. v2: - s/u8/int/ (Jani Nikula) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_vdsc.c | 15 +++ drivers/gpu/drm/i915/display/intel_vdsc.h | 1 + 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH 1/2] drm/i915: Add helper function for getting number of VDSC engines

2023-07-04 Thread Stanislav Lisovskiy
VDSC code. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_vdsc.c | 15 +++ drivers/gpu/drm/i915/display/intel_vdsc.h | 1 + 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c b/drivers/gpu/drm/i915

[Intel-gfx] [PATCH 0/2] Calculate CDCLK more properly when DSC is enabled

2023-07-04 Thread Stanislav Lisovskiy
-compressed case it can be usualy lower) However if we have 2 DSC engines CDCLK can be ~pixel clock / 2 and so on. Lets do the estimation more properly based on amount of VDSC engines used. That most likely is going to fix some FIFO underruns, we are currently having. Stanislav Lisovskiy (2): drm/i915

[Intel-gfx] [PATCH 2/2] drm/i915: Don't rely that 2 VDSC engines are always enough for pixel rate

2023-07-04 Thread Stanislav Lisovskiy
ump up CDCLK many times for similar reasons. v2: - Use new intel_dsc_get_num_vdsc_instances to determine number of VDSC engines, instead of slice count(Ankit Nautiyal) Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_cdclk.c | 13 +++-- 1 file chang

[Intel-gfx] [PATCH] drm/i915: Fix the disabling sequence for Bigjoiner

2023-05-25 Thread Stanislav Lisovskiy
. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index 0490c6412ab5..68958ba0ef49 100644 --- a/drivers/gpu

[Intel-gfx] [PATCH] drm/i915/mst: Make sure all resources are freed for MST topology manager before destroy

2023-06-07 Thread Stanislav Lisovskiy
. Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c index 3ba4a2fef98a..9c587b40cf42 100644 --- a/drivers/gpu/drm/i915

<    2   3   4   5   6   7   8   >