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

Message ID 20230324133150.43224-2-rogerq@kernel.org
State New
Headers
Series arm64: dts: ti: k3-am625-sk: Enable Type-C dual-role |

Commit Message

Roger Quadros March 24, 2023, 1:31 p.m. UTC
  From: Dhruva Gole <d-gole@ti.com>

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 <d-gole@ti.com>
Signed-off-by: Roger Quadros <rogerq@kernel.org>
---
 arch/arm64/boot/dts/ti/k3-am625-sk.dts | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Nishanth Menon March 24, 2023, 6:14 p.m. UTC | #1
On 15:31-20230324, Roger Quadros wrote:
> From: Dhruva Gole <d-gole@ti.com>
> 
> 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 <d-gole@ti.com>
> Signed-off-by: Roger Quadros <rogerq@kernel.org>
> ---
>  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
>
  
Roger Quadros March 25, 2023, 1:03 p.m. UTC | #2
On 24/03/2023 20:14, Nishanth Menon wrote:
> On 15:31-20230324, Roger Quadros wrote:
>> From: Dhruva Gole <d-gole@ti.com>
>>
>> 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 <d-gole@ti.com>
>> Signed-off-by: Roger Quadros <rogerq@kernel.org>
>> ---
>>  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
  

Patch

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 {