Re: [PATCH v7 0/7] incorporate pm runtime framework and eDP clean up

[Abhinav Kumar]

On 11/8/2023 10:10 AM, Kuogee Hsieh wrote:
> On 11/7/2023 1:23 PM, Dmitry Baryshkov wrote:
> > On Tue, 7 Nov 2023 at 23:01, Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> 
> > wrote:
> > > On 11/6/2023 5:55 PM, Dmitry Baryshkov wrote:
> > > > On Sat, 7 Oct 2023 at 01:55, Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> 
> > > > wrote:
> > > > > The purpose of this patch series is to incorporate pm runtime 
> > > > > framework
> > > > > into MSM eDP/DP driver so that eDP panel can be detected by DRM eDP 
> > > > > panel
> > > > > driver during system probe time. During incorporating procedure, 
> > > > > original
> > > > > customized pm realted fucntions, such as dp_pm_prepare(), 
> > > > > dp_pm_suspend(),
> > > > > dp_pm_resume() and dp_pm_prepare(), are removed and replaced with 
> > > > > functions
> > > > > provided by pm runtiem framework such as pm_runtime_force_suspend() 
> > > > > and
> > > > > pm_runtime_force_resume(). In addition, both eDP aux-bus and irq 
> > > > > handler
> > > > > are bound at system probe time too.
> > > > With this patchset in place I can crash the board using the following
> > > > sequence (SM8350-HDK):
> > > >
> > > > - plug the USBC DP dongle
> > > > - run modetest at any mode, don't press Enter yet
> > > > - unplug the dongle
> > > > - press Enter to stop modetest
> > > >
> > > > => the board resets to Sahara.
> > > >
> > > > Please ping me if you need any additional information from my side.
> > > questiosn,
> > >
> > > 1) which dongle are you used?
> > I have used several Dell and Hama USB-C dongles.
> >
> > > 2) what code branch shoud I used to duplicate this problem.
> > I have pushed my kernel tree to
> > git.codelinaro.org/dmitry.baryshkov/linux.git, branch test-dp-rpm
> > I had several UCSI patches on top, but they should not be relevant.
> git.codelinaro.org/dmitry.baryshkov/linux.git, branch test-dp-rpm <== I 
> synced out his branch and it is still work at my chromebook Kodiak DUT.
> >

Perhaps the gap in test results with the same tree is due to internal 
hpd vs hpd pin. We need to try this on a device which does not use 
internal hpd.

> > > I can not duplicate  system crash problem at my setup kodiak (SM7325)
> > > chrome book with my PM runtime patch series.
> > >
> > > my code base is Linux 6.6-rc2 + pm runtime patch series (7 patches)
> > >
> > > I did:
> > >
> > > 1) plugin either apple dongle (DP-to-HDMI) + 1080p display or DP typeC
> > > cable directly to 1080p display
> > >
> > > 2)  stop ui
> > >
> > > 3) /usr/bin/modetest -M msm -s 34:1920x1080 (see test pattern show at
> > > display)
> > >
> > > 4) unplug apple dongle or DP typeC cable
> > >
> > > 5) hit enter key
> > >
> > > 6) start ui
> > >
> > > 7) display back to login page of chrome book