On Sun, 4 Feb 2024 at 18:45, Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx> wrote: > > When the topology calls for two interfaces on the current fixed topology > of 2 DSC blocks, or uses 1 DSC block for a single interface (e.g. SC7280 > with only one DSC block), there should be no merging of DSC output. > > This is already represented by the return value of > dpu_encoder_use_dsc_merge(), but not yet used to correctly configure > this flag. > > Fixes: 58dca9810749 ("drm/msm/disp/dpu1: Add support for DSC in encoder") > Signed-off-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx> > --- > Note that more changes are needed to properly support the proposed 2:2:2 > and 1:1:1 topology (in contrast to the already-supported 2:2:1 topology), > but this could be a trivial patch to get going separately before all that > extra work is done. > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> -- With best wishes Dmitry