| Message ID | 20241210125338.104959-1-daniel.baluta@nxp.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Configure imx8mp dsp node for rproc usage | expand |
> 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@nxp.com>
On 12/10/2024 2:53 PM, Daniel Baluta wrote: > 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 For the series: Reviewed-by: Iuliana Prodan <iuliana.prodan@nxp.com> Thanks, Iulia
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(-)