Re: [PATCH 1/3] arm64: dts: qcom: eliza: Describe the ADSP and USB related nodes

[Abel Vesa]

On 26-03-31 15:37:08, Konrad Dybcio wrote:
> On 3/31/26 12:37 PM, Abel Vesa wrote:
> > Describe the ADSP remoteproc node along with its dependencies, including
> > the IPCC mailbox, AOSS QMP and SMP2P links used for communication.
> > 
> > The Eliza SoC features a USB 3.1 Gen 2 controller connected to a QMP
> > combo PHY and an SNPS eUSB2 PHY. Describe them.
> > 
> > Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> > ---
> 
> [...]
> 
> > +		usb_hsphy: phy@88e3000 {
> > +			compatible = "qcom,eliza-snps-eusb2-phy",
> > +				     "qcom,sm8550-snps-eusb2-phy";
> > +			reg = <0x0 0x088e3000 0x0 0x154>;
> > +			#phy-cells = <0>;
> > +
> > +			clocks = <&rpmhcc RPMH_CXO_CLK>;
> 
> This is TCSR_USB2_CLKREF_EN

Good point. Will fix.

> 
> 
> > +		usb: usb@a600000 {
> > +			compatible = "qcom,eliza-dwc3", "qcom,snps-dwc3";
> 
> Does the device suspend and resume successfully?

Well, tested with pm_test devices and it does suspend and resume
successfully, but there is this:

[   54.584126] dwc3-qcom a600000.usb: port-1 HS-PHY not in L2

But if I'm not mistaken, this is valid accross all SNPS eUSB2 PHYs, on
all platforms that have them.

Please correct me if I'm wrong.

Sorry for the late reply.