Add connector_type to debug info to differentiate between eDP and DP

[PATCH v3 0/3] Add connector_type to debug info to differentiate between eDP and DP

Posted by Kuogee Hsieh 4 years, 4 months ago

1) Add connector_type to debug info to differentiate between eDP and DP
2) add more debug info to cover dp Phy
3) repalce DRM_DEBUG_DP with drm_debug_dp

Kuogee Hsieh (3):
  drm/msm/dp: add connector type to enhance debug messages
  drm/msm/dp: enhance debug info related to dp phy
  drm/msm/dp:  replace DRM_DEBUG_DP marco with drm_dbg_dp

 drivers/gpu/drm/msm/dp/dp_audio.c   |  49 +++++++++------
 drivers/gpu/drm/msm/dp/dp_catalog.c |  34 ++++++-----
 drivers/gpu/drm/msm/dp/dp_ctrl.c    | 116 +++++++++++++++++++-----------------
 drivers/gpu/drm/msm/dp/dp_display.c | 103 ++++++++++++++++++++++----------
 drivers/gpu/drm/msm/dp/dp_drm.c     |   4 +-
 drivers/gpu/drm/msm/dp/dp_link.c    |  99 +++++++++++++++++-------------
 drivers/gpu/drm/msm/dp/dp_panel.c   |  43 +++++++------
 drivers/gpu/drm/msm/dp/dp_parser.c  |   2 +-
 drivers/gpu/drm/msm/dp/dp_power.c   |  20 ++++---
 9 files changed, 283 insertions(+), 187 deletions(-)

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

Re: [PATCH v3 0/3] Add connector_type to debug info to differentiate between eDP and DP

Posted by Kuogee Hsieh 4 years, 4 months ago

Hi Stephen,

Can you please review this serial patches.

On 2/2/2022 10:56 AM, Kuogee Hsieh wrote:
> 1) Add connector_type to debug info to differentiate between eDP and DP
> 2) add more debug info to cover dp Phy
> 3) repalce DRM_DEBUG_DP with drm_debug_dp
>
> Kuogee Hsieh (3):
>    drm/msm/dp: add connector type to enhance debug messages
>    drm/msm/dp: enhance debug info related to dp phy
>    drm/msm/dp:  replace DRM_DEBUG_DP marco with drm_dbg_dp
>
>   drivers/gpu/drm/msm/dp/dp_audio.c   |  49 +++++++++------
>   drivers/gpu/drm/msm/dp/dp_catalog.c |  34 ++++++-----
>   drivers/gpu/drm/msm/dp/dp_ctrl.c    | 116 +++++++++++++++++++-----------------
>   drivers/gpu/drm/msm/dp/dp_display.c | 103 ++++++++++++++++++++++----------
>   drivers/gpu/drm/msm/dp/dp_drm.c     |   4 +-
>   drivers/gpu/drm/msm/dp/dp_link.c    |  99 +++++++++++++++++-------------
>   drivers/gpu/drm/msm/dp/dp_panel.c   |  43 +++++++------
>   drivers/gpu/drm/msm/dp/dp_parser.c  |   2 +-
>   drivers/gpu/drm/msm/dp/dp_power.c   |  20 ++++---
>   9 files changed, 283 insertions(+), 187 deletions(-)
>

Re: [PATCH v3 0/3] Add connector_type to debug info to differentiate between eDP and DP

Posted by Jani Nikula 4 years, 4 months ago

On Wed, 02 Feb 2022, Kuogee Hsieh <quic_khsieh@quicinc.com> wrote:
> 1) Add connector_type to debug info to differentiate between eDP and DP
> 2) add more debug info to cover dp Phy
> 3) repalce DRM_DEBUG_DP with drm_debug_dp

In the future, please include the drm/msm/dp prefix also in the cover
letter. Thanks.

BR,
Jani.


>
> Kuogee Hsieh (3):
>   drm/msm/dp: add connector type to enhance debug messages
>   drm/msm/dp: enhance debug info related to dp phy
>   drm/msm/dp:  replace DRM_DEBUG_DP marco with drm_dbg_dp
>
>  drivers/gpu/drm/msm/dp/dp_audio.c   |  49 +++++++++------
>  drivers/gpu/drm/msm/dp/dp_catalog.c |  34 ++++++-----
>  drivers/gpu/drm/msm/dp/dp_ctrl.c    | 116 +++++++++++++++++++-----------------
>  drivers/gpu/drm/msm/dp/dp_display.c | 103 ++++++++++++++++++++++----------
>  drivers/gpu/drm/msm/dp/dp_drm.c     |   4 +-
>  drivers/gpu/drm/msm/dp/dp_link.c    |  99 +++++++++++++++++-------------
>  drivers/gpu/drm/msm/dp/dp_panel.c   |  43 +++++++------
>  drivers/gpu/drm/msm/dp/dp_parser.c  |   2 +-
>  drivers/gpu/drm/msm/dp/dp_power.c   |  20 ++++---
>  9 files changed, 283 insertions(+), 187 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center