mbox series

[0/3] Improve K3-am625-sk support (USB, MMC)

Message ID 20221216143624.23708-1-sjoerd@collabora.com (mailing list archive)
Headers show
Series Improve K3-am625-sk support (USB, MMC) | expand

Message

Sjoerd Simons Dec. 16, 2022, 2:36 p.m. UTC
This series picks up a few patches from the TI BSP tree that
unfortunately didn't make it upstream thusfar.

The first patch improve SD card compatibility (allowing U1 class cards
to be used), the remaining ones add USB support.

The type-c connector isn't entirely modelled with these changes as
it goes through a TPS6598 PD controller. Unfortunately the dtb bindings
for that currently require an irq line, which is not connected on E1 and
E2 version boards. The patchese to support this setup unfortunately didn't land
yet[0].. As such the last patch ignored the PD controller and simply
configures usb0 as periphal only rather then mode switch capable, which
at least gives some basic usability of that USB port.

0: https://lore.kernel.org/lkml/f714ee55-ef47-317d-81b9-57020dda064b@ti.com/T/


Aswath Govindraju (2):
  arm64: dts: ti: k3-am62-main: Add support for USB
  arm64: dts: ti: k3-am625-sk: Add support for USB

Nitin Yadav (1):
  arm64: dts: ti: k3-am62-main: Update OTAP and ITAP delay select

 arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 90 ++++++++++++++++++------
 arch/arm64/boot/dts/ti/k3-am625-sk.dts   | 22 ++++++
 2 files changed, 90 insertions(+), 22 deletions(-)

Comments

Martyn Welch Dec. 19, 2022, 1:09 p.m. UTC | #1
On Fri, 2022-12-16 at 15:36 +0100, Sjoerd Simons wrote:
> 
> This series picks up a few patches from the TI BSP tree that
> unfortunately didn't make it upstream thusfar.
> 
> The first patch improve SD card compatibility (allowing U1 class
> cards
> to be used), the remaining ones add USB support.
> 
> The type-c connector isn't entirely modelled with these changes as
> it goes through a TPS6598 PD controller. Unfortunately the dtb
> bindings
> for that currently require an irq line, which is not connected on E1
> and
> E2 version boards. The patchese to support this setup unfortunately
> didn't land
> yet[0].. As such the last patch ignored the PD controller and simply
> configures usb0 as periphal only rather then mode switch capable,
> which
> at least gives some basic usability of that USB port.
> 
> 0:
> https://lore.kernel.org/lkml/f714ee55-ef47-317d-81b9-57020dda064b@ti.com/T/
> 
> 

Series tested with known problematic SD Card and checking for detection
of inserted USB device.

Tested-by: Martyn Welch <martyn.welch@collabora.com>


> Aswath Govindraju (2):
>   arm64: dts: ti: k3-am62-main: Add support for USB
>   arm64: dts: ti: k3-am625-sk: Add support for USB
> 
> Nitin Yadav (1):
>   arm64: dts: ti: k3-am62-main: Update OTAP and ITAP delay select
> 
>  arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 90 ++++++++++++++++++----
> --
>  arch/arm64/boot/dts/ti/k3-am625-sk.dts   | 22 ++++++
>  2 files changed, 90 insertions(+), 22 deletions(-)
>
Vignesh Raghavendra Dec. 20, 2022, 11:32 a.m. UTC | #2
Hi,

On 16/12/22 8:06 pm, Sjoerd Simons wrote:
> 
> This series picks up a few patches from the TI BSP tree that
> unfortunately didn't make it upstream thusfar.
> 
> The first patch improve SD card compatibility (allowing U1 class cards
> to be used), the remaining ones add USB support.
> 
> The type-c connector isn't entirely modelled with these changes as
> it goes through a TPS6598 PD controller. Unfortunately the dtb bindings
> for that currently require an irq line, which is not connected on E1 and
> E2 version boards. The patchese to support this setup unfortunately didn't land
> yet[0].. As such the last patch ignored the PD controller and simply
> configures usb0 as periphal only rather then mode switch capable, which
> at least gives some basic usability of that USB port.
> 
> 0: https://lore.kernel.org/lkml/f714ee55-ef47-317d-81b9-57020dda064b@ti.com/T/
> 
> 
Really appreciate porting and posting these patches to mainline!
Wondering what tree are these patches based on?
I cannot apply cleanly on linux-next.


> Aswath Govindraju (2):
>   arm64: dts: ti: k3-am62-main: Add support for USB
>   arm64: dts: ti: k3-am625-sk: Add support for USB
> 
> Nitin Yadav (1):
>   arm64: dts: ti: k3-am62-main: Update OTAP and ITAP delay select
> 
>  arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 90 ++++++++++++++++++------
>  arch/arm64/boot/dts/ti/k3-am625-sk.dts   | 22 ++++++
>  2 files changed, 90 insertions(+), 22 deletions(-)
> 

Regards
Vignesh
Sjoerd Simons Dec. 20, 2022, 12:41 p.m. UTC | #3
Hey Vignesh,

On Tue, 2022-12-20 at 17:02 +0530, Vignesh Raghavendra wrote:
> Hi,
> 
> On 16/12/22 8:06 pm, Sjoerd Simons wrote:
> > 
> > This series picks up a few patches from the TI BSP tree that
> > unfortunately didn't make it upstream thusfar.
...
> > 0:
> > https://lore.kernel.org/lkml/f714ee55-ef47-317d-81b9-57020dda064b@ti.com/T/
> > 
> > 
> Really appreciate porting and posting these patches to mainline!
> Wondering what tree are these patches based on?
> I cannot apply cleanly on linux-next.

These are against Linux 6.1 ; I'll happy rebase agains next for the
next round or is there a specific tree you prefer to base against?

Regards,
Vignesh Raghavendra Dec. 21, 2022, 2:49 p.m. UTC | #4
Hi Sjoerd

On 12/20/2022 6:11 PM, Sjoerd Simons wrote:
> Hey Vignesh,
> 
> On Tue, 2022-12-20 at 17:02 +0530, Vignesh Raghavendra wrote:
>> Hi,
>>
>> On 16/12/22 8:06 pm, Sjoerd Simons wrote:
>>>
>>> This series picks up a few patches from the TI BSP tree that
>>> unfortunately didn't make it upstream thusfar.
> ...
>>> 0:
>>> https://lore.kernel.org/lkml/f714ee55-ef47-317d-81b9-57020dda064b@ti.com/T/
>>>
>>>
>> Really appreciate porting and posting these patches to mainline!
>> Wondering what tree are these patches based on?
>> I cannot apply cleanly on linux-next.
> 
> These are against Linux 6.1 ; I'll happy rebase agains next for the
> next round or is there a specific tree you prefer to base against?
> 

TI K3 patch queue is at:
https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git ti-next

This is part of linux-next. So, yes, I would like all patches to be
based on -next or above tree.

I see you have already done that in v2.. Thanks!

Regards
Vignesh