2023-03-24 13:34:18

by Roger Quadros

[permalink] [raw]
Subject: [PATCH 0/2] arm64: dts: ti: k3-am625-sk: Enable Type-C dual-role

Hi,

This series enables Type-C port on USB0.

cheers,
-roger

Dhruva Gole (1):
arm64: dts: ti: k3-am625-sk: Add ti,vbus-divider property to usbss1

Roger Quadros (1):
arm64: dts: ti: k3-am625-sk: Enable Type-C port for USB0

arch/arm64/boot/dts/ti/k3-am625-sk.dts | 41 ++++++++++++++++++++++++--
1 file changed, 39 insertions(+), 2 deletions(-)

--
2.34.1


2023-03-24 13:34:24

by Roger Quadros

[permalink] [raw]
Subject: [PATCH 1/2] arm64: dts: ti: k3-am625-sk: Add ti,vbus-divider property to usbss1

From: Dhruva Gole <[email protected]>

The property "ti,vbus-divider" is needed for both usbss0 and usbss1 as
both USB0 and USB1 have the same external voltage divider circuit.

Signed-off-by: Dhruva Gole <[email protected]>
Signed-off-by: Roger Quadros <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am625-sk.dts | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk.dts b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
index 67dc26fc23e4..be027fad5f61 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
@@ -482,6 +482,7 @@ &usbss0 {

&usbss1 {
status = "okay";
+ ti,vbus-divider;
};

&usb0 {
--
2.34.1

2023-03-24 18:24:46

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm64: dts: ti: k3-am625-sk: Add ti,vbus-divider property to usbss1

On 15:31-20230324, Roger Quadros wrote:
> From: Dhruva Gole <[email protected]>
>
> The property "ti,vbus-divider" is needed for both usbss0 and usbss1 as
> both USB0 and USB1 have the same external voltage divider circuit.
>
> Signed-off-by: Dhruva Gole <[email protected]>
> Signed-off-by: Roger Quadros <[email protected]>
> ---
> arch/arm64/boot/dts/ti/k3-am625-sk.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk.dts b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
> index 67dc26fc23e4..be027fad5f61 100644
> --- a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
> @@ -482,6 +482,7 @@ &usbss0 {
>
> &usbss1 {
> status = "okay";
> + ti,vbus-divider;
> };
>

1. Does'nt this need a fixes tag?
2. lakml is missing in CC

> &usb0 {
> --
> 2.34.1
>

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D

2023-03-25 13:05:23

by Roger Quadros

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm64: dts: ti: k3-am625-sk: Add ti,vbus-divider property to usbss1



On 24/03/2023 20:14, Nishanth Menon wrote:
> On 15:31-20230324, Roger Quadros wrote:
>> From: Dhruva Gole <[email protected]>
>>
>> The property "ti,vbus-divider" is needed for both usbss0 and usbss1 as
>> both USB0 and USB1 have the same external voltage divider circuit.
>>
>> Signed-off-by: Dhruva Gole <[email protected]>
>> Signed-off-by: Roger Quadros <[email protected]>
>> ---
>> arch/arm64/boot/dts/ti/k3-am625-sk.dts | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk.dts b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>> index 67dc26fc23e4..be027fad5f61 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>> @@ -482,6 +482,7 @@ &usbss0 {
>>
>> &usbss1 {
>> status = "okay";
>> + ti,vbus-divider;
>> };
>>
>
> 1. Does'nt this need a fixes tag?
> 2. lakml is missing in CC

Will fix both issues.

>
>> &usb0 {
>> --
>> 2.34.1
>>
>

cheers,
-roger