Re: [PATCH v3] arm64: qcom: qcs8300: Add ADSP and CDSP0 fastrpc nodes
From: Konrad Dybcio
Date: Sat Nov 30 2024 - 08:43:20 EST
On 19.11.2024 1:06 PM, Ling Xu wrote:
> Add ADSP and CDSP0 fastrpc nodes for QCS8300 platform.
>
> Signed-off-by: Ling Xu <quic_lxu5@xxxxxxxxxxx>
> ---
> This patch depends on patch https://lore.kernel.org/all/20240904-qcs8300_initial_dtsi-v1-0-d0ea9afdc007@xxxxxxxxxxx/#t
> Changes since v1:
> - Remove duplicate cdsp fastrpc nodes
> - Add adsp memory-region and vmids
> Changes since v2:
> - Remove extra duplicate cdsp fastrpc nodes
You removed effectively- duplicate iommus entries, no nodes were
removed compared to v2.
Also, I hope you gave this a smoke test, as some platforms in the
past had very strict sid/mask matching policies that didn't really
evaluate the effective value.
But it looks good now
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
Konrad