devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Val Packett <val@packett.cool>
To: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konradybcio@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Abel Vesa <abel.vesa@linaro.org>,
	Rajendra Nayak <quic_rjendra@quicinc.com>,
	Sibi Sankar <sibi.sankar@oss.qualcomm.com>
Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] Modify USB controller nodes for USB on X1E80100
Date: Tue, 14 Oct 2025 06:00:54 -0300	[thread overview]
Message-ID: <d0c18094-7234-450d-bd9c-e9f545de31e2@packett.cool> (raw)
In-Reply-To: <20251014022121.1850871-1-krishna.kurapati@oss.qualcomm.com>

Hi,

On 10/13/25 11:21 PM, Krishna Kurapati wrote:
> This series aims to flatten usb dt nodes and use latest bindings.
> While at it, fix a compile time warning on the HS only controller node.
>
> Tests done:
> 1. On CRD, verified host mode operation of the 3 exposed Type-C ports
> and enumeration of the Goodix fingerprint sensor on the multiport
> usb controller.
>
> 2. Modified the dr_mode to otg for the first 3 controllers and
> verified role switching and device mode operation on the 3 exposed
> Type-C ports.
>
> HS only controller was not tested on any platform.

have you tested suspend-resume?

The flattened dwc driver seems to break it for me on Latitude 7455, upon 
trying to resume the screen never comes back up and the system just 
reboots from zero in a couple seconds instead.

I've looked at the code and I couldn't find the cause so far, but it is 
fine with the legacy driver and not fine with this one :(


Thanks,
~val


  parent reply	other threads:[~2025-10-14  9:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-14  2:21 [PATCH 0/2] Modify USB controller nodes for USB on X1E80100 Krishna Kurapati
2025-10-14  2:21 ` [PATCH 1/2] arm64: dts: qcom: x1e80100: Flatten usb controller nodes Krishna Kurapati
2025-10-14  2:21 ` [PATCH 2/2] arm64: dts: qcom: x1e80100: Fix compile warnings for USB HS controller Krishna Kurapati
2025-10-14  9:00 ` Val Packett [this message]
2025-10-19 11:35   ` [PATCH 0/2] Modify USB controller nodes for USB on X1E80100 Krishna Kurapati PSSNV
2025-10-27 11:46     ` Konrad Dybcio
2025-11-22 22:18       ` Val Packett
2025-10-14 13:29 ` Rob Herring (Arm)

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=d0c18094-7234-450d-bd9c-e9f545de31e2@packett.cool \
    --to=val@packett.cool \
    --cc=abel.vesa@linaro.org \
    --cc=andersson@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=konradybcio@kernel.org \
    --cc=krishna.kurapati@oss.qualcomm.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=quic_rjendra@quicinc.com \
    --cc=robh@kernel.org \
    --cc=sibi.sankar@oss.qualcomm.com \
    /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;
as well as URLs for NNTP newsgroup(s).