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

2022-08-26 Thread Lisovskiy, Stanislav
On Thu, Aug 25, 2022 at 07:04:33PM +0300, Govindapillai, Vinod wrote: > On Thu, 2022-08-25 at 18:17 +0300, Lisovskiy, Stanislav wrote: > > On Thu, Aug 25, 2022 at 05:58:19PM +0300, Govindapillai, Vinod wrote: > > > Hi Stan, > > > > > > Some comments inline.. > > > > > > On Mon, 2022-08-22 at

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

2022-08-25 Thread Govindapillai, Vinod
On Thu, 2022-08-25 at 18:17 +0300, Lisovskiy, Stanislav wrote: > On Thu, Aug 25, 2022 at 05:58:19PM +0300, Govindapillai, Vinod wrote: > > Hi Stan, > > > > Some comments inline.. > > > > On Mon, 2022-08-22 at 12:40 +0300, Stanislav Lisovskiy wrote: > > > Whenever we are not able to get enough

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

2022-08-25 Thread Lisovskiy, Stanislav
On Thu, Aug 25, 2022 at 05:58:19PM +0300, Govindapillai, Vinod wrote: > Hi Stan, > > Some comments inline.. > > On Mon, 2022-08-22 at 12:40 +0300, Stanislav Lisovskiy wrote: > > Whenever we are not able to get enough timeslots > > for required PBN, let's try to allocate those > > using DSC, just

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

2022-08-25 Thread Govindapillai, Vinod
Hi Stan, Some comments inline.. On Mon, 2022-08-22 at 12:40 +0300, Stanislav Lisovskiy wrote: > Whenever we are not able to get enough timeslots > for required PBN, let's try to allocate those > using DSC, just same way as we do for SST. > > v2: Removed intel_dp_mst_dsc_compute_config and

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

2022-08-22 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Removed intel_dp_mst_dsc_compute_config and refactored intel_dp_dsc_compute_config to support timeslots as a parameter(Ville Syrjälä) v3: -

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

2022-08-15 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Removed intel_dp_mst_dsc_compute_config and refactored intel_dp_dsc_compute_config to support timeslots as a parameter(Ville Syrjälä) v3: -

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

2022-08-10 Thread Lisovskiy, Stanislav
On Wed, Aug 10, 2022 at 11:25:13AM +0300, Govindapillai, Vinod wrote: > Hi Stan, > > Please find my comments inline > > > On Mon, 2022-04-11 at 19:25 +0300, Stanislav Lisovskiy wrote: > > Whenever we are not able to get enough timeslots > > for required PBN, let's try to allocate those > >

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

2022-08-10 Thread Govindapillai, Vinod
Hi Stan, Please find my comments inline On Mon, 2022-04-11 at 19:25 +0300, Stanislav Lisovskiy wrote: > Whenever we are not able to get enough timeslots > for required PBN, let's try to allocate those > using DSC, just same way as we do for SST. > > v2: Removed intel_dp_mst_dsc_compute_config

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

2022-08-10 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Removed intel_dp_mst_dsc_compute_config and refactored intel_dp_dsc_compute_config to support timeslots as a parameter(Ville Syrjälä) v3: -

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

2022-04-11 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Removed intel_dp_mst_dsc_compute_config and refactored intel_dp_dsc_compute_config to support timeslots as a parameter(Ville Syrjälä)

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

2022-03-21 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Removed intel_dp_mst_dsc_compute_config and refactored intel_dp_dsc_compute_config to support timeslots as a parameter(Ville Syrjälä)

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

2022-03-21 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Add DSC checks to intel_dp_mst_mode_valid_ctx, similar to ones we have in intel_dp_mode_valid(Manasi Navare) v3: Removed redundant edp condition logic

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

2022-03-18 Thread Lisovskiy, Stanislav
On Thu, Mar 17, 2022 at 06:52:28PM +0200, Ville Syrjälä wrote: > On Thu, Mar 17, 2022 at 06:33:53PM +0200, Stanislav Lisovskiy wrote: > > Whenever we are not able to get enough timeslots > > for required PBN, let's try to allocate those > > using DSC, just same way as we do for SST. > > > > Those

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

2022-03-17 Thread Ville Syrjälä
On Thu, Mar 17, 2022 at 06:33:53PM +0200, Stanislav Lisovskiy wrote: > Whenever we are not able to get enough timeslots > for required PBN, let's try to allocate those > using DSC, just same way as we do for SST. > > Those patches are experimental yet, i.e not > for merging, still need to be

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

2022-03-17 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. Those patches are experimental yet, i.e not for merging, still need to be tested with proper DSC display, submitting those to check ig nothing else blows up at