RE: [PATCH v3 0/6] Configure imx8mp dsp node for rproc usage
From: Peng Fan
Date: Wed Dec 11 2024 - 02:16:40 EST
> Subject: [PATCH v3 0/6] Configure imx8mp dsp node for rproc usage
>
> DSP found in i.MX8MP SOC can be used by multiple frameworks in
> order to enable various applications:
> - rproc/rpmsg framework, used to load for example Zephyr
> samples
> - Sound Open Firmware, used to enable various audio processing
> pipelines.
>
> Current dsp node was configured with SOF in mind but it doesn't work
> well with imx8mp-evk dts. SOF controls audio IPs from firmware side
> while imx8mp-evk preffers to control audio IPs from Linux side.
>
> So, configure 'dsp' node to be used with rproc scenario and later will
> add a separate dts or an overlay to configure the node for SOF.
>
> This patch series configures and enables dsp node to be used with
> rproc.
>
> Changes since v1:
> - document syscon compatible as pointed by Peng Fan
> - do not disable dsp_reserved node in the dtsi file as pointed
> by Ahmad Fatoum.
> Changes since v2:
> - Fix dts example in patch 4/6
> - Frank Li commented that we should handle the reset using the
> reset
> interface. I have two objections on this: we are using some bits
> from audiomix
> for reset together some other bits from DAP region. Also, the
> implementation is already upstream!
> Frank Li agreed that it's up to Rob opinion on this.
>
> Daniel Baluta (6):
> arm64: dts: imx8mp: Add mu2 root clock
> arm64: dts: imx8mp: Configure dsp node for rproc usage
> arm64: dts: imx8mp: Add DSP clocks
> dt-bindings: clock: imx8mp: Add syscon compatible
> arm64: dts: imx8mp: Add fsl,dsp-ctrl property for dsp
> arm64: dts: Add dsp rproc related mem regions
>
> .../bindings/clock/imx8mp-audiomix.yaml | 6 +++--
> arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 10 +++++++++
> arch/arm64/boot/dts/freescale/imx8mp.dtsi | 22 ++++++++++++------
> -
> 3 files changed, 28 insertions(+), 10 deletions(-)
>
Patchset looks good to me:
Reviewed-by: Peng Fan <peng.fan@xxxxxxx>