Re: [PATCH v4 08/39] drm/msm/dp: allow dp_ctrl stream APIs to use any panel passed to it

[Dmitry Baryshkov]

On Tue, May 19, 2026 at 03:52:35PM +0800, Yongxing Mou wrote:
> 
> 
> On 4/12/2026 1:38 AM, Dmitry Baryshkov wrote:
> > On Fri, Apr 10, 2026 at 05:33:43PM +0800, Yongxing Mou wrote:
> > > From: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> > > 
> > > Currently, the dp_ctrl stream APIs operate on their own dp_panel
> > > which is stored inside the dp_ctrl's private struct. However with MST,
> > > the stored panel represents the fixed link and not the sinks which
> > > are hotplugged. Allow the stream related APIs to work on the panel
> > > which is passed to them rather than the stored one. For SST cases,
> > > this shall continue to use the stored dp_panel.
> > 
> > Hmm. Why? Can't we get rid of it (in the followup patch)?
> > 
> In MST, the stream panel is bound to the connector, but in SST the panel is
> still stored in msm_dp_display_private and msm_dp_ctrl_private, so
>   dropping it does not seem necessary at this point. The purpose of these
> changes is to allow the functions to operate on the MST panel data passed in
> per stream.

I understand the panel being stored in msm_dp_display_private. Can we
drop it from msm_dp_ctrl_private?

> > > 
> > > Signed-off-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> > > Signed-off-by: Yongxing Mou <yongxing.mou@xxxxxxxxxxxxxxxx>
> > > ---
> > >   drivers/gpu/drm/msm/dp/dp_ctrl.c    | 21 +++++++++++----------
> > >   drivers/gpu/drm/msm/dp/dp_ctrl.h    |  2 +-
> > >   drivers/gpu/drm/msm/dp/dp_display.c |  2 +-
> > >   3 files changed, 13 insertions(+), 12 deletions(-)
> > > 

-- 
With best wishes
Dmitry