mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Kuogee Hsieh <quic_khsieh@quicinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: <dri-devel@lists.freedesktop.org>, <robdclark@gmail.com>,
	<sean@poorly.run>, <swboyd@chromium.org>, <dianders@chromium.org>,
	<vkoul@kernel.org>, <daniel@ffwll.ch>, <airlied@gmail.com>,
	<agross@kernel.org>, <andersson@kernel.org>,
	<quic_abhinavk@quicinc.com>, <quic_jesszhan@quicinc.com>,
	<quic_sbillaka@quicinc.com>, <marijn.suijten@somainline.org>,
	<freedreno@lists.freedesktop.org>,
	<linux-arm-msm@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v7 0/7] incorporate pm runtime framework and eDP clean up
Date: Tue, 7 Nov 2023 13:00:24 -0800	[thread overview]
Message-ID: <55a4f98e-2772-e4fd-ae8a-132f92582c78@quicinc.com> (raw)
In-Reply-To: <CAA8EJpoFRp+7GyO=F3Ar21tfG5Yt0cL6zkAquqg7D1XXQjp50Q@mail.gmail.com>


On 11/6/2023 5:55 PM, Dmitry Baryshkov wrote:
> On Sat, 7 Oct 2023 at 01:55, Kuogee Hsieh <quic_khsieh@quicinc.com> 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?

2) what code branch shoud I used to duplicate this problem.

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



  reply	other threads:[~2023-11-07 21:01 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-06 22:55 Kuogee Hsieh
2023-10-06 22:55 ` [PATCH v7 1/7] drm/msm/dp: tie dp_display_irq_handler() with dp driver Kuogee Hsieh
2023-10-06 22:55 ` [PATCH v7 2/7] drm/msm/dp: rename is_connected with link_ready Kuogee Hsieh
2023-10-06 22:55 ` [PATCH v7 3/7] drm/msm/dp: use drm_bridge_hpd_notify() to report HPD status changes Kuogee Hsieh
2023-10-06 22:55 ` [PATCH v7 4/7] drm/msm/dp: move parser->parse() and dp_power_client_init() to probe Kuogee Hsieh
2023-10-07 11:34   ` Dmitry Baryshkov
2023-10-06 22:55 ` [PATCH v7 5/7] drm/msm/dp: incorporate pm_runtime framework into DP driver Kuogee Hsieh
2023-10-07 11:34   ` Dmitry Baryshkov
2023-10-12  1:02   ` Dmitry Baryshkov
2023-10-12  1:04   ` Dmitry Baryshkov
2023-10-06 22:55 ` [PATCH v7 6/7] drm/msm/dp: delete EV_HPD_INIT_SETUP Kuogee Hsieh
2023-10-06 22:55 ` [PATCH v7 7/7] drm/msm/dp: move of_dp_aux_populate_bus() to eDP probe() Kuogee Hsieh
2023-10-10 19:19 ` [PATCH v7 0/7] incorporate pm runtime framework and eDP clean up Stephen Boyd
2023-11-06  0:51 ` [Freedreno] " Leonard Lausen
2023-11-07  1:55 ` Dmitry Baryshkov
2023-11-07 21:00   ` Kuogee Hsieh [this message]
2023-11-07 21:23     ` Dmitry Baryshkov
2023-11-08 18:10       ` Kuogee Hsieh
2023-11-08 18:27         ` Abhinav Kumar
2023-11-09 17:51           ` Kuogee Hsieh
2023-11-09 20:35             ` Dmitry Baryshkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55a4f98e-2772-e4fd-ae8a-132f92582c78@quicinc.com \
    --to=quic_khsieh@quicinc.com \
    --cc=agross@kernel.org \
    --cc=airlied@gmail.com \
    --cc=andersson@kernel.org \
    --cc=daniel@ffwll.ch \
    --cc=dianders@chromium.org \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=freedreno@lists.freedesktop.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marijn.suijten@somainline.org \
    --cc=quic_abhinavk@quicinc.com \
    --cc=quic_jesszhan@quicinc.com \
    --cc=quic_sbillaka@quicinc.com \
    --cc=robdclark@gmail.com \
    --cc=sean@poorly.run \
    --cc=swboyd@chromium.org \
    --cc=vkoul@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®