mbox series

[0/2] drm/msm/dp: Fix Type-C Timing

Message ID 20250212034225.2565069-1-james.a.macinnes@gmail.com (mailing list archive)
Headers show
Series drm/msm/dp: Fix Type-C Timing | expand

Message

James A. MacInnes Feb. 12, 2025, 3:42 a.m. UTC
SDM845 DisplayPort output inop on DP Monitor and tears on HDMI.

Fixed
 - SDM845 does not support wide mode.
 - SDM845 does not need porch shift.

Verified functionality on SDM845.

James A. MacInnes (2):
  drm/msm/dp: Disable wide bus support for SDM845
  drm/msm/disp: Correct porch timing for SDM845

 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 8 ++++----
 drivers/gpu/drm/msm/dp/dp_display.c                  | 7 ++++++-
 2 files changed, 10 insertions(+), 5 deletions(-)

Comments

Dmitry Baryshkov Feb. 12, 2025, 11:20 a.m. UTC | #1
On Tue, Feb 11, 2025 at 07:42:23PM -0800, James A. MacInnes wrote:
> SDM845 DisplayPort output inop on DP Monitor and tears on HDMI.
> 
> Fixed
>  - SDM845 does not support wide mode.
>  - SDM845 does not need porch shift.
> 
> Verified functionality on SDM845.

Please use ./scripts/get_maintainer.pl to get the To / Cc lists. Your
messages missed several mailing lists and maybe some of maintainers.

> 
> James A. MacInnes (2):
>   drm/msm/dp: Disable wide bus support for SDM845
>   drm/msm/disp: Correct porch timing for SDM845
> 
>  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 8 ++++----
>  drivers/gpu/drm/msm/dp/dp_display.c                  | 7 ++++++-
>  2 files changed, 10 insertions(+), 5 deletions(-)
> 
> -- 
> 2.43.0
>
James A. MacInnes Feb. 12, 2025, 4:34 p.m. UTC | #2
On Wed, 12 Feb 2025 13:20:01 +0200
Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote:

> On Tue, Feb 11, 2025 at 07:42:23PM -0800, James A. MacInnes wrote:
> > SDM845 DisplayPort output inop on DP Monitor and tears on HDMI.
> > 
> > Fixed
> >  - SDM845 does not support wide mode.
> >  - SDM845 does not need porch shift.
> > 
> > Verified functionality on SDM845.  
> 
> Please use ./scripts/get_maintainer.pl to get the To / Cc lists. Your
> messages missed several mailing lists and maybe some of maintainers.
> 

Will do. The list from get_maintainers.pl was very long and I was
attempting to not spam everyone. On revision I will include everyone.
Marijn Suijten Feb. 12, 2025, 5:24 p.m. UTC | #3
On 2025-02-12 08:34:24, James A. MacInnes wrote:
> On Wed, 12 Feb 2025 13:20:01 +0200
> Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote:
> 
> > On Tue, Feb 11, 2025 at 07:42:23PM -0800, James A. MacInnes wrote:
> > > SDM845 DisplayPort output inop on DP Monitor and tears on HDMI.
> > > 
> > > Fixed
> > >  - SDM845 does not support wide mode.
> > >  - SDM845 does not need porch shift.
> > > 
> > > Verified functionality on SDM845.  
> > 
> > Please use ./scripts/get_maintainer.pl to get the To / Cc lists. Your
> > messages missed several mailing lists and maybe some of maintainers.
> > 
> 
> Will do. The list from get_maintainers.pl was very long and I was
> attempting to not spam everyone. On revision I will include everyone.

May I suggest using b4 for patch handling?  It'll help with tracking revisions,
cover letters, To/Cc, running checkpatch andsoforth:

https://b4.docs.kernel.org/en/latest/

(Another patch of you seemed to have V2 tacked on to the end of the cover letter
title; it's instead supposed to be part of the PREFIX like [PATCH v2 1/2] for
every message.  This tool will automatically handle that for you :D)

- Marijn