On Fri, 16 Jun 2023 08:14:38 -0700, Douglas Anderson wrote: > Trogdor devices use firmware backed by TF-A instead of Qualcomm's > normal TZ. On TF-A we end up mapping memory as cacheable. Specifically, > you can see in Trogdor's TF-A code [1] in qti_sip_mem_assign() that we > call qti_mmap_add_dynamic_region() with MT_RO_DATA. This translates > down to MT_MEMORY instead of MT_NON_CACHEABLE or MT_DEVICE. > > Let's allow devices like trogdor to be described properly by allowing > "dma-coherent" in the SCM node. > > [...] Applied, thanks! [1/4] dt-bindings: firmware: qcom,scm: Document that SCM can be dma-coherent commit: c0877829ada0406233aee5bd54f6813db79d5f1f [2/4] arm64: dts: qcom: sc7180: Mark SCM as dma-coherent for IDP commit: 9a5f0b11e49e27f0a01a73c31d05df4a95bea3fa [3/4] arm64: dts: qcom: sc7180: Mark SCM as dma-coherent for trogdor commit: a54b7fa6b9ab6b4ecb7d9aba6b1a0ce1bcc961e3 [4/4] arm64: dts: qcom: sc7280: Mark SCM as dma-coherent for chrome devices commit: 7b59e8ae92fe089fed8ff1b23e53442ae5b204c9 Best regards, -- Bjorn Andersson <andersson@xxxxxxxxxx>