On 24-01-29 17:08:29, Dmitry Baryshkov wrote: > On Mon, 29 Jan 2024 at 15:19, Abel Vesa <abel.v...@linaro.org> wrote: > > > > From: Abhinav Kumar <quic_abhin...@quicinc.com> > > > > On platforms where the endpoint used is on port@0, looking for port@1 > > instead results in just ignoring the max link-frequencies altogether. > > Look at port@0 first, then, if not found, look for port@1. > > NAK. Platforms do not "use port@0". It is for the connection between > DPU and DP, while the link-frequencies property is for the link > between DP controller and the actual display.
I messed up. This patch is not needed, plus the author is wrong. Will drop in the next version. Sorry about that. > > > > > Signed-off-by: Abhinav Kumar <quic_abhin...@quicinc.com> > > Signed-off-by: Abel Vesa <abel.v...@linaro.org> > > --- > > drivers/gpu/drm/msm/dp/dp_parser.c | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/msm/dp/dp_parser.c > > b/drivers/gpu/drm/msm/dp/dp_parser.c > > index 7032dcc8842b..eec5b8b83f4b 100644 > > --- a/drivers/gpu/drm/msm/dp/dp_parser.c > > +++ b/drivers/gpu/drm/msm/dp/dp_parser.c > > @@ -97,7 +97,11 @@ static u32 dp_parser_link_frequencies(struct device_node > > *of_node) > > u64 frequency = 0; > > int cnt; > > > > - endpoint = of_graph_get_endpoint_by_regs(of_node, 1, 0); /* port@1 > > */ > > + endpoint = of_graph_get_endpoint_by_regs(of_node, 0, 0); /* port@0 > > */ > > + > > + if (!endpoint) > > + endpoint = of_graph_get_endpoint_by_regs(of_node, 1, 0); /* > > port@1 */ > > + > > if (!endpoint) > > return 0; > > > > > > -- > > 2.34.1 > > > > > -- > With best wishes > Dmitry