Hi Vignesh, On 11/08/2023 19:04, Vignesh Raghavendra wrote: > > There is been a relook at the arch post this driver was upstreamed. > System firmware (SYSFW) is now two separate components: TI Foundational > Security (TIFS) running in a secure island and Device Management (DM) > firmware (runs on boot R5 core) [0] shows boot flow diagram for AM62x. I see, I cannot argue with that. Is this change affecting the entire K3 family or only the new members? If the later then I would seek for conditionality for the new regions as non relooked SoCs these regions are off-limit for the SW. > Security critical items such as PSIL pairing, channel firewalls and > credential configurations are under TIFS and is handled via TI SCI calls > at all times. > > But, things related to resource configuration (to ensure different cores > dont step on each other) is under DM. Linux still needs to talk to DM > for configuring these regions. But, when primary bootloader (R5 SPL) is > running, there isn't a DM firmware (as it runs on the same core after R5 > SPL), it would need to configure DMA resources on its own. > > This update is mainly to aid R5 SPL to reuse kernel DT as is. > Hope that helps Right, so Linux will need to use these soon ;) > > > [0] https://u-boot.readthedocs.io/en/latest/board/ti/am62x_sk.html?highlight=am62#boot-flow > (Similar boot flow for rest of K3 devices barring am65 and am64) OK, if the R5 needs to use DMA before DM then surely there is a need to configure the channel(s). I expect that R5 or DM will reset things back before booting the main CPU. > >> >>> >>> Vignesh Raghavendra (3): >>> dt-bindings: dma: ti: k3-bcdma: Describe cfg register regions >>> dt-bindings: dma: ti: k3-pktdma: Describe cfg register regions >>> dt-bindings: dma: ti: k3-udma: Describe cfg register regions >>> >>> .../devicetree/bindings/dma/ti/k3-bcdma.yaml | 25 +++++++++++++------ >>> .../devicetree/bindings/dma/ti/k3-pktdma.yaml | 18 ++++++++++--- >>> .../devicetree/bindings/dma/ti/k3-udma.yaml | 14 ++++++++--- >>> 3 files changed, 43 insertions(+), 14 deletions(-) >>> >> > -- Péter
