From: Jianhua Lu <lujianhua000@gmail.com>
To: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Cc: linux@roeck-us.net, heikki.krogerus@linux.intel.com,
gregkh@linuxfoundation.org, andersson@kernel.org,
robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
luca.weiss@fairphone.com, linux-usb@vger.kernel.org,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
caleb.connolly@linaro.org, konrad.dybcio@linaro.org,
subbaram@quicinc.com, jackp@quicinc.com,
robertom@qti.qualcomm.com
Subject: Re: [PATCH v6 12/13] arm64: dts: qcom: qrb5165-rb5: Switch on TCPM usb-role-switching for usb_1
Date: Tue, 2 May 2023 00:04:09 +0800 [thread overview]
Message-ID: <ZE_i-a33NO02xEEJ@Gentoo> (raw)
In-Reply-To: <bf5b4a43-192d-f1f7-253d-c60b884fe344@linaro.org>
On Mon, May 01, 2023 at 04:53:22PM +0100, Bryan O'Donoghue wrote:
> On 01/05/2023 16:21, Jianhua Lu wrote:
> > This port node should be moved out to the block of pm8150b_typec rather than
> > usb-c-connector. Otherwise,
> >
> > [ 10.998897] OF: graph: no port node found in /soc@0/spmi@c440000/pmic@2/typec@1500
>
uh.... This is runtime error, it doesn't matter bindings.
I haven't much experience with typc-c, so I can't address code for this.
>
> Ports/endpoints should go in the connector.
>
> Documentation/devicetree/bindings/connector/usb-connector.yaml
>
> grep usb-c-connector arch/arm* -r
>
> This dtsi should be updated
>
> arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi !=
> Documentation/devicetree/bindings/usb/nxp,ptn5110.yaml
>
> ---
> bod
next prev parent reply other threads:[~2023-05-01 16:04 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-01 12:10 [PATCH v6 00/13] Add Qualcomm PMIC TPCM support Bryan O'Donoghue
2023-05-01 12:10 ` [PATCH v6 01/13] dt-bindings: regulator: qcom,usb-vbus-regulator: Mark reg as required Bryan O'Donoghue
2023-05-01 12:11 ` [PATCH v6 02/13] dt-bindings: regulator: qcom,usb-vbus-regulator: Mark regulator-*-microamp required Bryan O'Donoghue
2023-05-01 12:11 ` [PATCH v6 03/13] dt-bindings: phy: qcom,sc7180-qmp-usb3-dp-phy: Add orientation-switch as optional Bryan O'Donoghue
2023-05-01 12:11 ` [PATCH v6 04/13] dt-bindings: phy: qcom,sc7180-qmp-usb3-dp-phy: Add input and output ports Bryan O'Donoghue
2023-05-02 10:45 ` Krzysztof Kozlowski
2023-05-02 10:51 ` Krzysztof Kozlowski
2023-05-01 12:11 ` [PATCH v6 05/13] dt-bindings: usb: Add Qualcomm PMIC Type-C Bryan O'Donoghue
2023-05-02 10:56 ` Krzysztof Kozlowski
2023-05-01 12:11 ` [PATCH v6 06/13] dt-bindings: mfd: qcom,spmi-pmic: Add typec to SPMI device types Bryan O'Donoghue
2023-05-01 12:11 ` [PATCH v6 07/13] usb: typec: qcom: Add Qualcomm PMIC Type-C driver Bryan O'Donoghue
2023-05-01 13:31 ` Guenter Roeck
2023-05-01 17:50 ` Caleb Connolly
2023-05-01 12:11 ` [PATCH v6 08/13] arm64: dts: qcom: sm8250: Define ports for qmpphy orientation-switching Bryan O'Donoghue
2023-05-02 11:01 ` Konrad Dybcio
2023-05-01 12:11 ` [PATCH v6 09/13] arm64: dts: qcom: pm8150b: Add a TCPM description Bryan O'Donoghue
2023-05-01 12:11 ` [PATCH v6 10/13] arm64: dts: qcom: qrb5165-rb5: Switch on Type-C VBUS boost Bryan O'Donoghue
2023-05-02 10:56 ` Konrad Dybcio
2023-05-01 12:11 ` [PATCH v6 11/13] arm64: dts: qcom: qrb5165-rb5: Switch on basic TCPM Bryan O'Donoghue
2023-05-01 15:28 ` Jianhua Lu
2023-05-02 10:57 ` Konrad Dybcio
2023-05-01 12:11 ` [PATCH v6 12/13] arm64: dts: qcom: qrb5165-rb5: Switch on TCPM usb-role-switching for usb_1 Bryan O'Donoghue
2023-05-01 15:21 ` Jianhua Lu
2023-05-01 15:53 ` Bryan O'Donoghue
2023-05-01 16:04 ` Jianhua Lu [this message]
2023-05-01 15:30 ` Jianhua Lu
2023-05-02 11:00 ` Konrad Dybcio
2023-05-02 11:03 ` Bryan O'Donoghue
2023-05-02 11:13 ` Konrad Dybcio
2023-05-02 11:16 ` Bryan O'Donoghue
2023-05-02 11:47 ` Konrad Dybcio
2023-05-02 12:10 ` Bryan O'Donoghue
2023-05-07 21:09 ` Bryan O'Donoghue
2023-05-01 12:11 ` [PATCH v6 13/13] arm64: dts: qcom: qrb5165-rb5: Switch on TCPM orientation-switch for usb_1_qmpphy Bryan O'Donoghue
2023-05-01 15:34 ` Jianhua Lu
2023-05-01 16:00 ` Bryan O'Donoghue
2023-05-01 15:37 ` Jianhua Lu
2023-05-01 17:04 ` Bryan O'Donoghue
2023-05-02 11:01 ` Konrad Dybcio
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=ZE_i-a33NO02xEEJ@Gentoo \
--to=lujianhua000@gmail.com \
--cc=andersson@kernel.org \
--cc=bryan.odonoghue@linaro.org \
--cc=caleb.connolly@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=heikki.krogerus@linux.intel.com \
--cc=jackp@quicinc.com \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=luca.weiss@fairphone.com \
--cc=robertom@qti.qualcomm.com \
--cc=robh+dt@kernel.org \
--cc=subbaram@quicinc.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.