drivers/gpu/drm/msm/dp/dp_catalog.c | 15 +++++++-- drivers/gpu/drm/msm/dp/dp_debug.c | 61 +++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/msm/dp/dp_panel.h | 2 ++ 3 files changed, 76 insertions(+), 2 deletions(-)
DP Test Patten Generator is a very useful tool to debug issues such
as blank screen or corruption seen on the DP monitor by isolating it
to whether the corruption is coming from further upstream such as DPU
OR from the DP controller and below. It was noted in [1] that this API
is unused. Rather than dropping the API, it should be fixed and used.
Hence, this series fixes the DP Test Patten Generator API and also utilizes
it by adding a debugfs for it.
[1] : https://patchwork.freedesktop.org/patch/623508/?series=141074&rev=1
To: Rob Clark <robdclark@gmail.com>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Sean Paul <sean@poorly.run>
To: Marijn Suijten <marijn.suijten@somainline.org>
To: David Airlie <airlied@gmail.com>
To: Simona Vetter <simona@ffwll.ch>
To: Stephen Boyd <swboyd@chromium.org>
To: Kuogee Hsieh <quic_khsieh@quicinc.com>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: linux-kernel@vger.kernel.org
Cc: Jessica Zhang <quic_jesszhan@quicinc.com>
Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
---
Abhinav Kumar (3):
drm/msm/dp: account for widebus in msm_dp_catalog_panel_tpg_enable()
drm/msm/dp: do not touch the MMSS_DP_INTF_CONFIG for tpg
drm/msm/dp: add a debugfs node for using tpg
drivers/gpu/drm/msm/dp/dp_catalog.c | 15 +++++++--
drivers/gpu/drm/msm/dp/dp_debug.c | 61 +++++++++++++++++++++++++++++++++++++
drivers/gpu/drm/msm/dp/dp_panel.h | 2 ++
3 files changed, 76 insertions(+), 2 deletions(-)
---
base-commit: 798bb342e0416d846cf67f4725a3428f39bfb96b
change-id: 20241202-tpg-3f7543c036ac
Best regards,
--
Abhinav Kumar <quic_abhinavk@quicinc.com>
On Mon, Dec 02, 2024 at 12:41:57PM -0800, Abhinav Kumar wrote: > DP Test Patten Generator is a very useful tool to debug issues such > as blank screen or corruption seen on the DP monitor by isolating it > to whether the corruption is coming from further upstream such as DPU > OR from the DP controller and below. It was noted in [1] that this API > is unused. Rather than dropping the API, it should be fixed and used. > > Hence, this series fixes the DP Test Patten Generator API and also utilizes > it by adding a debugfs for it. > > [1] : https://patchwork.freedesktop.org/patch/623508/?series=141074&rev=1 I'd prefer for this to be rebased on top of [2]. The series has been posted a month ago. [2] https://patchwork.freedesktop.org/series/141074/ > > To: Rob Clark <robdclark@gmail.com> > To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > To: Sean Paul <sean@poorly.run> > To: Marijn Suijten <marijn.suijten@somainline.org> > To: David Airlie <airlied@gmail.com> > To: Simona Vetter <simona@ffwll.ch> > To: Stephen Boyd <swboyd@chromium.org> > To: Kuogee Hsieh <quic_khsieh@quicinc.com> > Cc: linux-arm-msm@vger.kernel.org > Cc: dri-devel@lists.freedesktop.org > Cc: freedreno@lists.freedesktop.org > Cc: linux-kernel@vger.kernel.org > Cc: Jessica Zhang <quic_jesszhan@quicinc.com> > > Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com> > --- > Abhinav Kumar (3): > drm/msm/dp: account for widebus in msm_dp_catalog_panel_tpg_enable() > drm/msm/dp: do not touch the MMSS_DP_INTF_CONFIG for tpg > drm/msm/dp: add a debugfs node for using tpg > > drivers/gpu/drm/msm/dp/dp_catalog.c | 15 +++++++-- > drivers/gpu/drm/msm/dp/dp_debug.c | 61 +++++++++++++++++++++++++++++++++++++ > drivers/gpu/drm/msm/dp/dp_panel.h | 2 ++ > 3 files changed, 76 insertions(+), 2 deletions(-) > --- > base-commit: 798bb342e0416d846cf67f4725a3428f39bfb96b > change-id: 20241202-tpg-3f7543c036ac > > Best regards, > -- > Abhinav Kumar <quic_abhinavk@quicinc.com> > -- With best wishes Dmitry
On 12/3/2024 6:31 AM, Dmitry Baryshkov wrote: > On Mon, Dec 02, 2024 at 12:41:57PM -0800, Abhinav Kumar wrote: >> DP Test Patten Generator is a very useful tool to debug issues such >> as blank screen or corruption seen on the DP monitor by isolating it >> to whether the corruption is coming from further upstream such as DPU >> OR from the DP controller and below. It was noted in [1] that this API >> is unused. Rather than dropping the API, it should be fixed and used. >> >> Hence, this series fixes the DP Test Patten Generator API and also utilizes >> it by adding a debugfs for it. >> >> [1] : https://patchwork.freedesktop.org/patch/623508/?series=141074&rev=1 > > I'd prefer for this to be rebased on top of [2]. The series has been > posted a month ago. > > [2] https://patchwork.freedesktop.org/series/141074/ > Well, the review of that series which tried dropping the tpg led to this one really. I will review that one within a couple of days. Lets first agree on that series before deciding to rebase. >> >> To: Rob Clark <robdclark@gmail.com> >> To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> >> To: Sean Paul <sean@poorly.run> >> To: Marijn Suijten <marijn.suijten@somainline.org> >> To: David Airlie <airlied@gmail.com> >> To: Simona Vetter <simona@ffwll.ch> >> To: Stephen Boyd <swboyd@chromium.org> >> To: Kuogee Hsieh <quic_khsieh@quicinc.com> >> Cc: linux-arm-msm@vger.kernel.org >> Cc: dri-devel@lists.freedesktop.org >> Cc: freedreno@lists.freedesktop.org >> Cc: linux-kernel@vger.kernel.org >> Cc: Jessica Zhang <quic_jesszhan@quicinc.com> >> >> Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com> >> --- >> Abhinav Kumar (3): >> drm/msm/dp: account for widebus in msm_dp_catalog_panel_tpg_enable() >> drm/msm/dp: do not touch the MMSS_DP_INTF_CONFIG for tpg >> drm/msm/dp: add a debugfs node for using tpg >> >> drivers/gpu/drm/msm/dp/dp_catalog.c | 15 +++++++-- >> drivers/gpu/drm/msm/dp/dp_debug.c | 61 +++++++++++++++++++++++++++++++++++++ >> drivers/gpu/drm/msm/dp/dp_panel.h | 2 ++ >> 3 files changed, 76 insertions(+), 2 deletions(-) >> --- >> base-commit: 798bb342e0416d846cf67f4725a3428f39bfb96b >> change-id: 20241202-tpg-3f7543c036ac >> >> Best regards, >> -- >> Abhinav Kumar <quic_abhinavk@quicinc.com> >> >
On Tue, 3 Dec 2024 at 21:42, Abhinav Kumar <quic_abhinavk@quicinc.com> wrote: > > > > On 12/3/2024 6:31 AM, Dmitry Baryshkov wrote: > > On Mon, Dec 02, 2024 at 12:41:57PM -0800, Abhinav Kumar wrote: > >> DP Test Patten Generator is a very useful tool to debug issues such > >> as blank screen or corruption seen on the DP monitor by isolating it > >> to whether the corruption is coming from further upstream such as DPU > >> OR from the DP controller and below. It was noted in [1] that this API > >> is unused. Rather than dropping the API, it should be fixed and used. > >> > >> Hence, this series fixes the DP Test Patten Generator API and also utilizes > >> it by adding a debugfs for it. > >> > >> [1] : https://patchwork.freedesktop.org/patch/623508/?series=141074&rev=1 > > > > I'd prefer for this to be rebased on top of [2]. The series has been > > posted a month ago. > > > > [2] https://patchwork.freedesktop.org/series/141074/ > > > > Well, the review of that series which tried dropping the tpg led to this > one really. > > I will review that one within a couple of days. Lets first agree on that > series before deciding to rebase. SGTM. Note, v2 kept the TPG in place, as you have asked. > > >> > >> To: Rob Clark <robdclark@gmail.com> > >> To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > >> To: Sean Paul <sean@poorly.run> > >> To: Marijn Suijten <marijn.suijten@somainline.org> > >> To: David Airlie <airlied@gmail.com> > >> To: Simona Vetter <simona@ffwll.ch> > >> To: Stephen Boyd <swboyd@chromium.org> > >> To: Kuogee Hsieh <quic_khsieh@quicinc.com> > >> Cc: linux-arm-msm@vger.kernel.org > >> Cc: dri-devel@lists.freedesktop.org > >> Cc: freedreno@lists.freedesktop.org > >> Cc: linux-kernel@vger.kernel.org > >> Cc: Jessica Zhang <quic_jesszhan@quicinc.com> > >> > >> Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com> > >> --- > >> Abhinav Kumar (3): > >> drm/msm/dp: account for widebus in msm_dp_catalog_panel_tpg_enable() > >> drm/msm/dp: do not touch the MMSS_DP_INTF_CONFIG for tpg > >> drm/msm/dp: add a debugfs node for using tpg > >> > >> drivers/gpu/drm/msm/dp/dp_catalog.c | 15 +++++++-- > >> drivers/gpu/drm/msm/dp/dp_debug.c | 61 +++++++++++++++++++++++++++++++++++++ > >> drivers/gpu/drm/msm/dp/dp_panel.h | 2 ++ > >> 3 files changed, 76 insertions(+), 2 deletions(-) > >> --- > >> base-commit: 798bb342e0416d846cf67f4725a3428f39bfb96b > >> change-id: 20241202-tpg-3f7543c036ac > >> > >> Best regards, > >> -- > >> Abhinav Kumar <quic_abhinavk@quicinc.com> > >> > > -- With best wishes Dmitry
© 2016 - 2024 Red Hat, Inc.