Re: [PATCH v2 0/8] Add DFU, emmc and usb boot for TI am62x

Tom Rini
2023-05-31T21:16:25Z

[-- Attachment #1: Type: text/plain, Size: 2352 bytes --]

On Thu, Apr 06, 2023 at 08:55:34PM +0200, Sjoerd Simons wrote:

> This series adds more boot sources for the TI am62x. For that the dts'
> are synced from the upstream ti-next git tree (to add usb nodes), some
> dwc3 glue is and finally the default configuration is tuned to add
> support for DFU and USB (host and gadget)
>
>
> Changes in v2:
> - Update dts sync to v6.3-rc5
> - Switch dwc3 glue to a seperate driver rather then in dwc-generic
> - Minimize config changes to just DFU configuration
> - Only enable usb port 0 DFU in SPL
> - Create a seperate defconfig for R5
>
> Sjoerd Simons (8):
> omap: timer: add ti,am654-timer compatibility
> arm: mach-k3: am62: Add timer0 id to the dev list
> arm: dts: k3-am62: Bump dtsi from linux
> arm: dts: k3-am625-sk: Enable emmc in SPL
> usb: dwc3: Add dwc3 glue driver for am62
> configs: am62: Add configs for enabling USB in U-Boot
> arm: dts: k3-am625-sk: Enable usb ports in u-boot
> configs: am62x_evm_*: Enable USB and DFU support
>
> arch/arm/dts/k3-am62-main.dtsi | 259 +++++++++++++++++++++++---
> arch/arm/dts/k3-am62-mcu.dtsi | 49 +++++
> arch/arm/dts/k3-am62-wakeup.dtsi | 4 +-
> arch/arm/dts/k3-am625-r5-sk.dts | 2 +-
> arch/arm/dts/k3-am625-sk-u-boot.dtsi | 27 ++-
> arch/arm/dts/k3-am625-sk.dts | 95 +++++-----
> arch/arm/dts/k3-am625.dtsi | 52 ++++++
> arch/arm/mach-k3/am62x/dev-data.c | 1 +
> configs/am62x_evm_a53_defconfig | 35 +++-
> configs/am62x_evm_r5_usbdfu_defconfig | 116 ++++++++++++
> drivers/timer/omap-timer.c | 1 +
> drivers/usb/dwc3/Kconfig | 14 ++
> drivers/usb/dwc3/Makefile | 1 +
> drivers/usb/dwc3/dwc3-am62.c | 127 +++++++++++++
> include/configs/am62x_evm.h | 10 +-
> 15 files changed, 701 insertions(+), 92 deletions(-)
> create mode 100644 configs/am62x_evm_r5_usbdfu_defconfig
> create mode 100644 drivers/usb/dwc3/dwc3-am62.c
> Tested-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
> Tested-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>

This seems, conceptually, fine. But as we're getting the TI dts files
in sync with the kernel, I'm deferring this version and you'll want to
rebase and re-post once everything has settled.

--
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]