* [PATCH v3] arm64: dts: qcom: sm6115: Add EUD dt node and dwc3 connector
@ 2023-01-04 10:12 Bhupesh Sharma
2023-05-26 19:02 ` Bjorn Andersson
0 siblings, 1 reply; 3+ messages in thread
From: Bhupesh Sharma @ 2023-01-04 10:12 UTC (permalink / raw)
To: linux-arm-msm
Cc: quic_schowdhu, agross, andersson, konrad.dybcio, linux-kernel,
bhupesh.linux, bhupesh.sharma, robh+dt, devicetree
Add the Embedded USB Debugger(EUD) device tree node for
SM6115 / SM4250 SoC.
The node contains EUD base register region, EUD mode manager
register region and TCSR Check register region along with the
interrupt entry.
Also add the typec connector node for EUD which is attached to
EUD node via port. EUD is also attached to DWC3 node via port.
To enable the role switch, we need to set dr_mode = "otg" property
for 'usb_dwc3' sub-node in the board dts file.
Also the EUD device can be enabled on a board once linux is boot'ed
by setting:
$ echo 1 > /sys/bus/platform/drivers/qcom_eud/../enable
Cc: Souradeep Chowdhury <quic_schowdhu@quicinc.com>
Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
---
- v2 can be viewed here: https://lore.kernel.org/linux-arm-msm/20230104052047.3945983-1-bhupesh.sharma@linaro.org
- In v3, fix the errors reported by '$ make dtbs_check' for the port sub node:
ports:
'oneOf' conditional failed, one must be fixed:
'port' is a required property
'#address-cells' is a required property
'#size-cells' is a required property
- This patch is based on my earlier sm6115 usb related changes, which can
be seen here:
https://lore.kernel.org/linux-arm-msm/20221215094532.589291-1-bhupesh.sharma@linaro.org/
- This patch is also dependent on my sm6115 eud dt-binding and driver changes
(v2) sent earlier, which can be seen here:
https://lore.kernel.org/linux-arm-msm/20230103150419.3923421-1-bhupesh.sharma@linaro.org/
arch/arm64/boot/dts/qcom/sm6115.dtsi | 46 ++++++++++++++++++++++++++++
1 file changed, 46 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
index 030763187cc3f..a1a4f659587f3 100644
--- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
@@ -170,6 +170,18 @@ core3 {
};
};
+ eud_typec: connector {
+ compatible = "usb-c-connector";
+
+ ports {
+ port@0 {
+ con_eud: endpoint {
+ remote-endpoint = <&eud_con>;
+ };
+ };
+ };
+ };
+
firmware {
scm: scm {
compatible = "qcom,scm-sm6115", "qcom,scm";
@@ -565,6 +577,33 @@ gcc: clock-controller@1400000 {
#power-domain-cells = <1>;
};
+ eud: eud@1610000 {
+ compatible = "qcom,sm6115-eud", "qcom,eud";
+ reg = <0x01610000 0x2000>,
+ <0x01612000 0x1000>,
+ <0x003e5018 0x4>;
+ interrupts = <GIC_SPI 189 IRQ_TYPE_LEVEL_HIGH>;
+ status = "disabled";
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ port@0 {
+ reg = <0>;
+ eud_ep: endpoint {
+ remote-endpoint = <&usb2_role_switch>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+ eud_con: endpoint {
+ remote-endpoint = <&con_eud>;
+ };
+ };
+ };
+ };
+
usb_hsphy: phy@1613000 {
compatible = "qcom,sm6115-qusb2-phy";
reg = <0x01613000 0x180>;
@@ -1064,6 +1103,13 @@ usb_dwc3: usb@4e00000 {
snps,has-lpm-erratum;
snps,hird-threshold = /bits/ 8 <0x10>;
snps,usb3_lpm_capable;
+ usb-role-switch;
+
+ port {
+ usb2_role_switch: endpoint {
+ remote-endpoint = <&eud_ep>;
+ };
+ };
};
};
--
2.38.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] arm64: dts: qcom: sm6115: Add EUD dt node and dwc3 connector
2023-01-04 10:12 [PATCH v3] arm64: dts: qcom: sm6115: Add EUD dt node and dwc3 connector Bhupesh Sharma
@ 2023-05-26 19:02 ` Bjorn Andersson
2023-05-26 19:27 ` Bhupesh Sharma
0 siblings, 1 reply; 3+ messages in thread
From: Bjorn Andersson @ 2023-05-26 19:02 UTC (permalink / raw)
To: Bhupesh Sharma
Cc: linux-arm-msm, quic_schowdhu, agross, konrad.dybcio, linux-kernel,
bhupesh.linux, robh+dt, devicetree
On Wed, Jan 04, 2023 at 03:42:28PM +0530, Bhupesh Sharma wrote:
> Add the Embedded USB Debugger(EUD) device tree node for
> SM6115 / SM4250 SoC.
>
> The node contains EUD base register region, EUD mode manager
> register region and TCSR Check register region along with the
> interrupt entry.
>
> Also add the typec connector node for EUD which is attached to
> EUD node via port. EUD is also attached to DWC3 node via port.
>
> To enable the role switch, we need to set dr_mode = "otg" property
> for 'usb_dwc3' sub-node in the board dts file.
>
> Also the EUD device can be enabled on a board once linux is boot'ed
> by setting:
> $ echo 1 > /sys/bus/platform/drivers/qcom_eud/../enable
>
I was under the impression that you where working on some updates to EUD
and was going to resend this, but I don't find a new version, and I
don't see an on-list reply from me.
So just to make sure this isn't lost any longer... Can you please respin
this to correct the dtc warnings?
Thanks,
Bjorn
> Cc: Souradeep Chowdhury <quic_schowdhu@quicinc.com>
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
> ---
> - v2 can be viewed here: https://lore.kernel.org/linux-arm-msm/20230104052047.3945983-1-bhupesh.sharma@linaro.org
> - In v3, fix the errors reported by '$ make dtbs_check' for the port sub node:
> ports:
> 'oneOf' conditional failed, one must be fixed:
> 'port' is a required property
> '#address-cells' is a required property
> '#size-cells' is a required property
> - This patch is based on my earlier sm6115 usb related changes, which can
> be seen here:
> https://lore.kernel.org/linux-arm-msm/20221215094532.589291-1-bhupesh.sharma@linaro.org/
> - This patch is also dependent on my sm6115 eud dt-binding and driver changes
> (v2) sent earlier, which can be seen here:
> https://lore.kernel.org/linux-arm-msm/20230103150419.3923421-1-bhupesh.sharma@linaro.org/
>
> arch/arm64/boot/dts/qcom/sm6115.dtsi | 46 ++++++++++++++++++++++++++++
> 1 file changed, 46 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> index 030763187cc3f..a1a4f659587f3 100644
> --- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> @@ -170,6 +170,18 @@ core3 {
> };
> };
>
> + eud_typec: connector {
> + compatible = "usb-c-connector";
> +
> + ports {
> + port@0 {
> + con_eud: endpoint {
> + remote-endpoint = <&eud_con>;
> + };
> + };
> + };
> + };
> +
> firmware {
> scm: scm {
> compatible = "qcom,scm-sm6115", "qcom,scm";
> @@ -565,6 +577,33 @@ gcc: clock-controller@1400000 {
> #power-domain-cells = <1>;
> };
>
> + eud: eud@1610000 {
> + compatible = "qcom,sm6115-eud", "qcom,eud";
> + reg = <0x01610000 0x2000>,
> + <0x01612000 0x1000>,
> + <0x003e5018 0x4>;
> + interrupts = <GIC_SPI 189 IRQ_TYPE_LEVEL_HIGH>;
> + status = "disabled";
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + port@0 {
> + reg = <0>;
> + eud_ep: endpoint {
> + remote-endpoint = <&usb2_role_switch>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> + eud_con: endpoint {
> + remote-endpoint = <&con_eud>;
> + };
> + };
> + };
> + };
> +
> usb_hsphy: phy@1613000 {
> compatible = "qcom,sm6115-qusb2-phy";
> reg = <0x01613000 0x180>;
> @@ -1064,6 +1103,13 @@ usb_dwc3: usb@4e00000 {
> snps,has-lpm-erratum;
> snps,hird-threshold = /bits/ 8 <0x10>;
> snps,usb3_lpm_capable;
> + usb-role-switch;
> +
> + port {
> + usb2_role_switch: endpoint {
> + remote-endpoint = <&eud_ep>;
> + };
> + };
> };
> };
>
> --
> 2.38.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] arm64: dts: qcom: sm6115: Add EUD dt node and dwc3 connector
2023-05-26 19:02 ` Bjorn Andersson
@ 2023-05-26 19:27 ` Bhupesh Sharma
0 siblings, 0 replies; 3+ messages in thread
From: Bhupesh Sharma @ 2023-05-26 19:27 UTC (permalink / raw)
To: Bjorn Andersson
Cc: linux-arm-msm, quic_schowdhu, agross, konrad.dybcio, linux-kernel,
bhupesh.linux, robh+dt, devicetree
Hi Bjorn,
On Sat, 27 May 2023 at 00:28, Bjorn Andersson <andersson@kernel.org> wrote:
>
> On Wed, Jan 04, 2023 at 03:42:28PM +0530, Bhupesh Sharma wrote:
> > Add the Embedded USB Debugger(EUD) device tree node for
> > SM6115 / SM4250 SoC.
> >
> > The node contains EUD base register region, EUD mode manager
> > register region and TCSR Check register region along with the
> > interrupt entry.
> >
> > Also add the typec connector node for EUD which is attached to
> > EUD node via port. EUD is also attached to DWC3 node via port.
> >
> > To enable the role switch, we need to set dr_mode = "otg" property
> > for 'usb_dwc3' sub-node in the board dts file.
> >
> > Also the EUD device can be enabled on a board once linux is boot'ed
> > by setting:
> > $ echo 1 > /sys/bus/platform/drivers/qcom_eud/../enable
> >
>
> I was under the impression that you where working on some updates to EUD
> and was going to resend this, but I don't find a new version, and I
> don't see an on-list reply from me.
>
> So just to make sure this isn't lost any longer... Can you please respin
> this to correct the dtc warnings?
Yes, we already have the v6 for this series here:
https://lore.kernel.org/linux-arm-msm/20230517211756.2483552-1-bhupesh.sharma@linaro.org/
Thanks,
Bhupesh
> > Cc: Souradeep Chowdhury <quic_schowdhu@quicinc.com>
> > Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
> > ---
> > - v2 can be viewed here: https://lore.kernel.org/linux-arm-msm/20230104052047.3945983-1-bhupesh.sharma@linaro.org
> > - In v3, fix the errors reported by '$ make dtbs_check' for the port sub node:
> > ports:
> > 'oneOf' conditional failed, one must be fixed:
> > 'port' is a required property
> > '#address-cells' is a required property
> > '#size-cells' is a required property
> > - This patch is based on my earlier sm6115 usb related changes, which can
> > be seen here:
> > https://lore.kernel.org/linux-arm-msm/20221215094532.589291-1-bhupesh.sharma@linaro.org/
> > - This patch is also dependent on my sm6115 eud dt-binding and driver changes
> > (v2) sent earlier, which can be seen here:
> > https://lore.kernel.org/linux-arm-msm/20230103150419.3923421-1-bhupesh.sharma@linaro.org/
> >
> > arch/arm64/boot/dts/qcom/sm6115.dtsi | 46 ++++++++++++++++++++++++++++
> > 1 file changed, 46 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> > index 030763187cc3f..a1a4f659587f3 100644
> > --- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> > @@ -170,6 +170,18 @@ core3 {
> > };
> > };
> >
> > + eud_typec: connector {
> > + compatible = "usb-c-connector";
> > +
> > + ports {
> > + port@0 {
> > + con_eud: endpoint {
> > + remote-endpoint = <&eud_con>;
> > + };
> > + };
> > + };
> > + };
> > +
> > firmware {
> > scm: scm {
> > compatible = "qcom,scm-sm6115", "qcom,scm";
> > @@ -565,6 +577,33 @@ gcc: clock-controller@1400000 {
> > #power-domain-cells = <1>;
> > };
> >
> > + eud: eud@1610000 {
> > + compatible = "qcom,sm6115-eud", "qcom,eud";
> > + reg = <0x01610000 0x2000>,
> > + <0x01612000 0x1000>,
> > + <0x003e5018 0x4>;
> > + interrupts = <GIC_SPI 189 IRQ_TYPE_LEVEL_HIGH>;
> > + status = "disabled";
> > +
> > + ports {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + port@0 {
> > + reg = <0>;
> > + eud_ep: endpoint {
> > + remote-endpoint = <&usb2_role_switch>;
> > + };
> > + };
> > +
> > + port@1 {
> > + reg = <1>;
> > + eud_con: endpoint {
> > + remote-endpoint = <&con_eud>;
> > + };
> > + };
> > + };
> > + };
> > +
> > usb_hsphy: phy@1613000 {
> > compatible = "qcom,sm6115-qusb2-phy";
> > reg = <0x01613000 0x180>;
> > @@ -1064,6 +1103,13 @@ usb_dwc3: usb@4e00000 {
> > snps,has-lpm-erratum;
> > snps,hird-threshold = /bits/ 8 <0x10>;
> > snps,usb3_lpm_capable;
> > + usb-role-switch;
> > +
> > + port {
> > + usb2_role_switch: endpoint {
> > + remote-endpoint = <&eud_ep>;
> > + };
> > + };
> > };
> > };
> >
> > --
> > 2.38.1
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-05-26 19:27 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-04 10:12 [PATCH v3] arm64: dts: qcom: sm6115: Add EUD dt node and dwc3 connector Bhupesh Sharma
2023-05-26 19:02 ` Bjorn Andersson
2023-05-26 19:27 ` Bhupesh Sharma
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).