On 25-10-21, 17:37, Dmitry Baryshkov wrote: > On 14/10/2021 17:13, Dmitry Baryshkov wrote: > > On 07/10/2021 10:08, Vinod Koul wrote: > > > @@ -572,8 +574,22 @@ static struct msm_display_topology > > > dpu_encoder_get_topology( > > > topology.num_enc = 0; > > > topology.num_intf = intf_count; > > > + drm_enc = &dpu_enc->base; > > > + priv = drm_enc->dev->dev_private; > > > + if (priv && priv->dsc) { > > > + /* In case of Display Stream Compression DSC, we would use > > > + * 2 encoders, 2 line mixers and 1 interface > > > + * this is power optimal and can drive upto (including) 4k > > > + * screens > > > + */ > > > + topology.num_enc = 2; > > > + topology.num_intf = 1; > > > + topology.num_lm = 2; > > > > So, here you'd set the topology.num_rm. > > I meant 'num_dsc', please excuse the typo. No worries, i had assumed such and made it num_dsc -- ~Vinod