mbox series

[v3,0/2] AM64: Add USB support

Message ID 20210317043007.18272-1-a-govindraju@ti.com (mailing list archive)
Headers show
Series AM64: Add USB support | expand

Message

Aswath Govindraju March 17, 2021, 4:30 a.m. UTC
The following series of patches, add USB support for AM642 evm.

USB test logs,
https://pastebin.ubuntu.com/p/YSQRBWGmzd/

Changes since v2:
- dropped compatible string "ti,j721e-usb" leading to DT schema errors
- Reran test logs
- Couldn't pick up reviewed-by from kishon as a change was made in the
  patch

Changes since v1:
- Rebased the patches on top of ti-k3-dts-next
- Added test logs

Aswath Govindraju (2):
  arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem
  arm64: dts: ti: k3-am642-evm: Add USB support

 arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 30 ++++++++++++++++++++++++
 arch/arm64/boot/dts/ti/k3-am642-evm.dts  | 18 ++++++++++++++
 2 files changed, 48 insertions(+)

Comments

Kishon Vijay Abraham I March 17, 2021, 2:52 p.m. UTC | #1
Aswath and Nishanth,

On 17/03/21 10:00 am, Aswath Govindraju wrote:
> The following series of patches, add USB support for AM642 evm.
> 
> USB test logs,
> https://pastebin.ubuntu.com/p/YSQRBWGmzd/

Vinod has provided stable tag [1]
git://git.kernel.org/pub/scm/linux/kernel/git/phy/linux-phy.git
tags/ti-serdes-for-5.13

This should unblock us from up-streaming SK along with this.

Thanks
Kishon

[1] -> http://lore.kernel.org/r/YFGjPNonIlA4Anar@vkoul-mobl.Dlink
> 
> Changes since v2:
> - dropped compatible string "ti,j721e-usb" leading to DT schema errors
> - Reran test logs
> - Couldn't pick up reviewed-by from kishon as a change was made in the
>   patch
> 
> Changes since v1:
> - Rebased the patches on top of ti-k3-dts-next
> - Added test logs
> 
> Aswath Govindraju (2):
>   arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem
>   arm64: dts: ti: k3-am642-evm: Add USB support
> 
>  arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 30 ++++++++++++++++++++++++
>  arch/arm64/boot/dts/ti/k3-am642-evm.dts  | 18 ++++++++++++++
>  2 files changed, 48 insertions(+)
>
Kishon Vijay Abraham I March 17, 2021, 4:08 p.m. UTC | #2
Hi,

On 17/03/21 8:22 pm, Kishon Vijay Abraham I wrote:
> Aswath and Nishanth,
> 
> On 17/03/21 10:00 am, Aswath Govindraju wrote:
>> The following series of patches, add USB support for AM642 evm.
>>
>> USB test logs,
>> https://pastebin.ubuntu.com/p/YSQRBWGmzd/
> 
> Vinod has provided stable tag [1]
> git://git.kernel.org/pub/scm/linux/kernel/git/phy/linux-phy.git
> tags/ti-serdes-for-5.13
> 
> This should unblock us from up-streaming SK along with this.

In case you prefer to send SK support as follow up patches

Reviewed-by: Kishon Vijay Abraham I <kishon@ti.com>

Thanks
Kishon

> 
> Thanks
> Kishon
> 
> [1] -> http://lore.kernel.org/r/YFGjPNonIlA4Anar@vkoul-mobl.Dlink
>>
>> Changes since v2:
>> - dropped compatible string "ti,j721e-usb" leading to DT schema errors
>> - Reran test logs
>> - Couldn't pick up reviewed-by from kishon as a change was made in the
>>   patch
>>
>> Changes since v1:
>> - Rebased the patches on top of ti-k3-dts-next
>> - Added test logs
>>
>> Aswath Govindraju (2):
>>   arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem
>>   arm64: dts: ti: k3-am642-evm: Add USB support
>>
>>  arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 30 ++++++++++++++++++++++++
>>  arch/arm64/boot/dts/ti/k3-am642-evm.dts  | 18 ++++++++++++++
>>  2 files changed, 48 insertions(+)
>>
Nishanth Menon March 17, 2021, 7:10 p.m. UTC | #3
On Wed, 17 Mar 2021 10:00:04 +0530, Aswath Govindraju wrote:
> The following series of patches, add USB support for AM642 evm.
> 
> USB test logs,
> https://pastebin.ubuntu.com/p/YSQRBWGmzd/
> 
> Changes since v2:
> - dropped compatible string "ti,j721e-usb" leading to DT schema errors
> - Reran test logs
> - Couldn't pick up reviewed-by from kishon as a change was made in the
>   patch
> 
> [...]

Hi Aswath Govindraju,

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/2] arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem
      commit: d06a661309d30b654b74a4633dd78804ef16369f
[2/2] arm64: dts: ti: k3-am642-evm: Add USB support
      commit: 04a80a75baa1c80f7e5096147b6173c13ca3d3e0


All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] git://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git
Nishanth Menon March 17, 2021, 7:16 p.m. UTC | #4
On 20:22-20210317, Kishon Vijay Abraham I wrote:
> Aswath and Nishanth,
> 
> On 17/03/21 10:00 am, Aswath Govindraju wrote:
> > The following series of patches, add USB support for AM642 evm.
> > 
> > USB test logs,
> > https://pastebin.ubuntu.com/p/YSQRBWGmzd/
> 
> Vinod has provided stable tag [1]
> git://git.kernel.org/pub/scm/linux/kernel/git/phy/linux-phy.git
> tags/ti-serdes-for-5.13
> 
> This should unblock us from up-streaming SK along with this.
> 

Awesome. Thanks Vinod and Kishon for helping co-ordinate this.

Ashwath,

Please highlight this detail as part SK USB3 addendum series and I can
pick it up as part of pulling that series in. our window is starting to
narrow, so lets respin asap.