snagrecover: am6x: Support J721e platforms#84
Conversation
|
I checked other (newer) TI SoC about their "Product ID" value... and it's a mess. Every time we have to double check the "Product ID" really used in u-boot. We can also check the CTRLMMR_USB_DEVICE_ID register reset value. Here is my draft for other TI SoC (I reordered some lines, removed the "am654x" since it use the wrong device ID) About SoC family naming "am6x", It should be "K3" to include other TI SoC using "K3 Multicore SoC architecture platform". Indeed not all TI SoC are named "AM6x": DRA829, DRA821, TDA4VM... For the J721e, the "AM68p" variant doesn't really exist (it appear only in the TRM). It should not be confused with the "real" AM68 that is a "J721S2" SoC. Do you really want a exhaustive list of SoC or can we use their "generic" names ? or have all variant name? |
rgantois
left a comment
There was a problem hiding this comment.
Hi, thanks a lot for the contribution! Looks good to me, aside from a small nitpick which I've mentionned
Sure, I'd be open to that change.
The granularity is different for each SoC family. For TI, I've decided to be quite specific with the SoC model since the boot procedure is quite complex, so I expect some potential for breaking variations from one model to another. So ideally for TI, we'd have one model name for each separate commercialized SoC. Now for rebranded SoCs where the chip is exactly the same but with a different name, I don't expect an exhaustive list. SoC aliases can always be added later if needed, like we did for i.MX8MP. |
Support the TI J721E SoC, also known as DRA829, also known as TDA4VM. Its recovery flow is almost identical to the typical AM6x flow, the only major difference being an additional "sysfw.bin" binary firmware needed for "Split binary with a split firmware" boot flow [1] (like the AM654). Update send_tiboot3() accordingly to download "sysfw.bin" after "tiboot3.bin". Add a new snagrecover template with all four binaries: tiboot3.bin, sysfw.itb, tispl.bin and u-boot.img. Use USB Vendor ID and Product ID from the TDA4VM TRM in Table 4-66 USB Boot Parameter Table [2]: Vendor ID = 0x451 Product ID = 0x6163 But latter in 4.6.8.1.1 DFU Device Mode chapter the Product ID is the same as the am65 SoC (0x6162). This is likely a typo since Product ID 0x6163 is the one used by u-boot for the J721e EVM board [3]. Add Product ID 0x6163 to the TI rules list. Tested on SK-TDA4VM [4] board using its usb-c port by setting SW1 switch in USB boot mode, see the user manual [5]. Provide tda4vm and dra829 aliases to the generic name j721e since this SoC familly share the same boot flow. snagrecover log: [INFO] SoC model tda4vm is an alias for j721e TI's K3 based J721E SoC familly include: - DRA829JM - DRA829VM - TDA4VM88 - TDA4VM67 - TDA4VM21 These processors differ only in terms of additional features implemented: - ARM R5F: "Quad Core" vs "Hexa Core" - C66x DSP: "Dual core" vs "single core" - DMPAC: "available" vs "missing" - C7x DSP: "available" vs "missing" - GPU: "available" vs "missing" [...] But the J721e SoC core base is the same (This includes the boot flow). [1] https://docs.u-boot.org/en/latest/board/ti/k3.html#boot-flow-variations [2] https://www.ti.com/lit/zip/spruil1 [3] https://source.denx.de/u-boot/u-boot/-/blob/v2026.01/configs/j721e_evm_r5_defconfig#L177 [4] https://www.ti.com/tool/SK-TDA4VM [5] https://www.ti.com/lit/pdf/SPRUJ21 Signed-off-by: Romain Naour <romain.naour@smile.fr>
7dc5d45 to
0fb79f3
Compare
Thanks for the review. I'm agree that we need to be careful between TI K3 SoC variants, but as soon as they share the same datasheet, the boot flow is the same. I'm using the SK-TDA4VM, the BeagleboneAI64 (both using the TDA4VM SoC) and the J721E-EVM (XJ721E experimental) board for evaluation purpose but the customer product board use the DRA829 SoC. I can use snagboot without any trouble. For convenience, I added "tda4vm" and "dra829" aliases for the "j721e". |
|
Nice, thanks! |
Support the J721e/DRA829/TDA4VM SoC family.
Its recovery flow is almost identical to the typical AM6x flow, the only major difference being an additional "sysfw.bin" binary firmware needed for "Split binary with a split firmware" boot flow [1] (like the AM654). Update send_tiboot3() accordingly to download "sysfw.bin" after "tiboot3.bin".
Add a new snagrecover template with all four binaries: tiboot3.bin, sysfw.itb, tispl.bin and u-boot.img.
Use USB Vendor ID and Product ID from the TDA4VM TRM in Table 4-66 USB Boot Parameter Table [2]:
Vendor ID = 0x451
Product ID = 0x6163
But latter in 4.6.8.1.1 DFU Device Mode chapter the Product ID is the same as the am65 SoC (0x6162). This is likely a typo since Product ID 0x6163 is the one used by u-boot for the J721e EVM board [3].
Add Product ID 0x6163 to the TI rules list.
Tested on SK-TDA4VM [4] board using its usb-c port by setting SW1 switch in USB boot mode, see the user manual [5].
[1] https://docs.u-boot.org/en/latest/board/ti/k3.html#boot-flow-variations
[2] https://www.ti.com/lit/zip/spruil1
[3] https://source.denx.de/u-boot/u-boot/-/blob/v2026.01/configs/j721e_evm_r5_defconfig#L177
[4] https://www.ti.com/tool/SK-TDA4VM
[5] https://www.ti.com/lit/pdf/SPRUJ21