* [PATCH v2 0/2] Add code to manage DSP related clocks @ 2021-09-03 14:53 Daniel Baluta 2021-09-03 14:53 ` [PATCH v2 1/2] ASoC: SOF: imx: " Daniel Baluta 2021-09-03 14:53 ` [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation Daniel Baluta 0 siblings, 2 replies; 9+ messages in thread From: Daniel Baluta @ 2021-09-03 14:53 UTC (permalink / raw) To: broonie, pierre-louis.bossart, lgirdwood, robh+dt, ranjani.sridharan, kai.vehmanen Cc: devicetree, shawnguo, kernel, festevam, linux-imx, peter.ujfalusi, alsa-devel, linux-kernel, s-anna, Daniel Baluta From: Daniel Baluta <daniel.baluta@nxp.com> This code is based on top of SOF topic/sof-dev branch and we want to have a review with ALSA and Device Tree communities then it will be merged to SOF tree and then merged into ALSA tree. DSP node on the Linux kernel side must also take care of enabling DAI/DMA related clocks. By design we choose to manage DAI/DMA clocks from the kernel side because of the architecture of some i.MX8 boards. Clocks are handled by a special M4 core which runs a special firmware called SCFW (System Controler firmware). This communicates with A cores running Linux via a special Messaging Unit and implements a custom API which is already implemented by the Linux kernel i.MX clocks implementation. Note that these clocks are optional. We can use the DSP without them. Changes since v1: - used clk bulk API as suggested by mark Daniel Baluta (2): ASoC: SOF: imx: Add code to manage DSP related clocks dt-bindings: dsp: fsl: Add DSP optional clocks documentation .../devicetree/bindings/dsp/fsl,dsp.yaml | 33 ++++++++++++++ sound/soc/sof/imx/imx-common.c | 44 +++++++++++++++++++ sound/soc/sof/imx/imx-common.h | 13 ++++++ sound/soc/sof/imx/imx8.c | 37 ++++++++++++++++ sound/soc/sof/imx/imx8m.c | 34 ++++++++++++++ 5 files changed, 161 insertions(+) -- 2.27.0 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 1/2] ASoC: SOF: imx: Add code to manage DSP related clocks 2021-09-03 14:53 [PATCH v2 0/2] Add code to manage DSP related clocks Daniel Baluta @ 2021-09-03 14:53 ` Daniel Baluta 2021-09-03 16:06 ` Pierre-Louis Bossart 2021-09-03 14:53 ` [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation Daniel Baluta 1 sibling, 1 reply; 9+ messages in thread From: Daniel Baluta @ 2021-09-03 14:53 UTC (permalink / raw) To: broonie, pierre-louis.bossart, lgirdwood, robh+dt, ranjani.sridharan, kai.vehmanen Cc: devicetree, shawnguo, kernel, festevam, linux-imx, peter.ujfalusi, alsa-devel, linux-kernel, s-anna, Daniel Baluta From: Daniel Baluta <daniel.baluta@nxp.com> There are two types of clocks: * DSP IP clocks * DAI clocks This clocks are necessary in order to power up DSP and DAIs. We choose to enable DAI clocks here because of the way i.MX8/i.MX8X design handles resources (including clocks). All clocks are managed by a separate core (named SCU) which communicates with Linux managed ARM core via a well known API. We parse and enable the clocks in probe function and disable them in remove function. Future patches will introduce Power Management support so that we disable clocks while DSP is not used or system enters power save. Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> --- sound/soc/sof/imx/imx-common.c | 44 ++++++++++++++++++++++++++++++++++ sound/soc/sof/imx/imx-common.h | 13 ++++++++++ sound/soc/sof/imx/imx8.c | 37 ++++++++++++++++++++++++++++ sound/soc/sof/imx/imx8m.c | 34 ++++++++++++++++++++++++++ 4 files changed, 128 insertions(+) diff --git a/sound/soc/sof/imx/imx-common.c b/sound/soc/sof/imx/imx-common.c index 8826ef94f04a..f9d650ad3846 100644 --- a/sound/soc/sof/imx/imx-common.c +++ b/sound/soc/sof/imx/imx-common.c @@ -74,4 +74,48 @@ void imx8_dump(struct snd_sof_dev *sdev, u32 flags) } EXPORT_SYMBOL(imx8_dump); +int imx8_parse_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks) +{ + int ret; + + ret = devm_clk_bulk_get(sdev->dev, clks->num_dsp_clks, clks->dsp_clks); + if (ret) { + dev_err(sdev->dev, "Failed to request DSP clocks\n"); + return ret; + } + + ret = devm_clk_bulk_get_optional(sdev->dev, clks->num_dai_clks, clks->dai_clks); + if (ret) { + dev_err(sdev->dev, "Failed to request DAI clks\n"); + return ret; + } + + return 0; +} +EXPORT_SYMBOL(imx8_parse_clocks); + +int imx8_enable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks) +{ + int ret; + + ret = clk_bulk_prepare_enable(clks->num_dsp_clks, clks->dsp_clks); + if (ret) + return ret; + ret = clk_bulk_prepare_enable(clks->num_dai_clks, clks->dai_clks); + if (ret) { + clk_bulk_disable_unprepare(clks->num_dsp_clks, clks->dsp_clks); + return ret; + } + + return 0; +} +EXPORT_SYMBOL(imx8_enable_clocks); + +void imx8_disable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks) +{ + clk_bulk_disable_unprepare(clks->num_dsp_clks, clks->dsp_clks); + clk_bulk_disable_unprepare(clks->num_dai_clks, clks->dai_clks); +} +EXPORT_SYMBOL(imx8_disable_clocks); + MODULE_LICENSE("Dual BSD/GPL"); diff --git a/sound/soc/sof/imx/imx-common.h b/sound/soc/sof/imx/imx-common.h index 1cc7d6704182..54fba9fcd861 100644 --- a/sound/soc/sof/imx/imx-common.h +++ b/sound/soc/sof/imx/imx-common.h @@ -3,6 +3,8 @@ #ifndef __IMX_COMMON_H__ #define __IMX_COMMON_H__ +#include <linux/clk.h> + #define EXCEPT_MAX_HDR_SIZE 0x400 #define IMX8_STACK_DUMP_SIZE 32 @@ -13,4 +15,15 @@ void imx8_get_registers(struct snd_sof_dev *sdev, void imx8_dump(struct snd_sof_dev *sdev, u32 flags); +struct imx_clocks { + struct clk_bulk_data *dsp_clks; + int num_dsp_clks; + struct clk_bulk_data *dai_clks; + int num_dai_clks; +}; + +int imx8_parse_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks); +int imx8_enable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks); +void imx8_disable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks); + #endif diff --git a/sound/soc/sof/imx/imx8.c b/sound/soc/sof/imx/imx8.c index fc1720c211a3..5370d34edd61 100644 --- a/sound/soc/sof/imx/imx8.c +++ b/sound/soc/sof/imx/imx8.c @@ -41,6 +41,25 @@ #define MBOX_OFFSET 0x800000 #define MBOX_SIZE 0x1000 +/* DSP clocks */ +struct clk_bulk_data imx8_dsp_clks[] = { + { .id = "ipg" }, + { .id = "ocram" }, + { .id = "core" }, +}; + +/* DAI clocks */ +struct clk_bulk_data imx8_dai_clks[] = { + { .id = "esai0_core" }, + { .id = "esai0_extal" }, + { .id = "esai0_spba" }, + { .id = "sai1_bus" }, + { .id = "sai1_mclk0" }, + { .id = "sai1_mclk1" }, + { .id = "sai1_mclk2" }, + { .id = "sai1_mclk3" }, +}; + struct imx8_priv { struct device *dev; struct snd_sof_dev *sdev; @@ -57,6 +76,7 @@ struct imx8_priv { struct device **pd_dev; struct device_link **link; + struct imx_clocks *clks; }; static void imx8_get_reply(struct snd_sof_dev *sdev) @@ -223,6 +243,10 @@ static int imx8_probe(struct snd_sof_dev *sdev) if (!priv) return -ENOMEM; + priv->clks = devm_kzalloc(&pdev->dev, sizeof(*priv->clks), GFP_KERNEL); + if (!priv->clks) + return -ENOMEM; + sdev->num_cores = 1; sdev->pdata->hw_pdata = priv; priv->dev = sdev->dev; @@ -336,6 +360,18 @@ static int imx8_probe(struct snd_sof_dev *sdev) /* set default mailbox offset for FW ready message */ sdev->dsp_box.offset = MBOX_OFFSET; + /* init clocks info */ + priv->clks->dsp_clks = imx8_dsp_clks; + priv->clks->num_dsp_clks = ARRAY_SIZE(imx8_dsp_clks); + priv->clks->dai_clks = imx8_dai_clks; + priv->clks->num_dai_clks = ARRAY_SIZE(imx8_dai_clks); + + ret = imx8_parse_clocks(sdev, priv->clks); + if (ret < 0) + goto exit_pdev_unregister; + + imx8_enable_clocks(sdev, priv->clks); + return 0; exit_pdev_unregister: @@ -354,6 +390,7 @@ static int imx8_remove(struct snd_sof_dev *sdev) struct imx8_priv *priv = sdev->pdata->hw_pdata; int i; + imx8_disable_clocks(sdev, priv->clks); platform_device_unregister(priv->ipc_dev); for (i = 0; i < priv->num_domains; i++) { diff --git a/sound/soc/sof/imx/imx8m.c b/sound/soc/sof/imx/imx8m.c index 30624fafc632..fea1b72bebaa 100644 --- a/sound/soc/sof/imx/imx8m.c +++ b/sound/soc/sof/imx/imx8m.c @@ -23,6 +23,21 @@ #define MBOX_OFFSET 0x800000 #define MBOX_SIZE 0x1000 +struct clk_bulk_data imx8m_dsp_clks[] = { + { .id = "ipg" }, + { .id = "ocram" }, + { .id = "core" }, +}; + +struct clk_bulk_data imx8m_dai_clks[] = { + { .id = "sai3_bus" }, + { .id = "sai3_mclk0" }, + { .id = "sai3_mclk1" }, + { .id = "sai3_mclk2" }, + { .id = "sai3_mclk3" }, + { .id = "sdma3_root" }, +}; + struct imx8m_priv { struct device *dev; struct snd_sof_dev *sdev; @@ -30,6 +45,8 @@ struct imx8m_priv { /* DSP IPC handler */ struct imx_dsp_ipc *dsp_ipc; struct platform_device *ipc_dev; + + struct imx_clocks *clks; }; static void imx8m_get_reply(struct snd_sof_dev *sdev) @@ -143,6 +160,10 @@ static int imx8m_probe(struct snd_sof_dev *sdev) if (!priv) return -ENOMEM; + priv->clks = devm_kzalloc(&pdev->dev, sizeof(*priv->clks), GFP_KERNEL); + if (!priv->clks) + return -ENOMEM; + sdev->num_cores = 1; sdev->pdata->hw_pdata = priv; priv->dev = sdev->dev; @@ -211,6 +232,18 @@ static int imx8m_probe(struct snd_sof_dev *sdev) /* set default mailbox offset for FW ready message */ sdev->dsp_box.offset = MBOX_OFFSET; + /* init clocks info */ + priv->clks->dsp_clks = imx8m_dsp_clks; + priv->clks->num_dsp_clks = ARRAY_SIZE(imx8m_dsp_clks); + priv->clks->dai_clks = imx8m_dai_clks; + priv->clks->num_dai_clks = ARRAY_SIZE(imx8m_dai_clks); + + ret = imx8_parse_clocks(sdev, priv->clks); + if (ret < 0) + goto exit_pdev_unregister; + + imx8_enable_clocks(sdev, priv->clks); + return 0; exit_pdev_unregister: @@ -222,6 +255,7 @@ static int imx8m_remove(struct snd_sof_dev *sdev) { struct imx8m_priv *priv = sdev->pdata->hw_pdata; + imx8_disable_clocks(sdev, priv->clks); platform_device_unregister(priv->ipc_dev); return 0; -- 2.27.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] ASoC: SOF: imx: Add code to manage DSP related clocks 2021-09-03 14:53 ` [PATCH v2 1/2] ASoC: SOF: imx: " Daniel Baluta @ 2021-09-03 16:06 ` Pierre-Louis Bossart 0 siblings, 0 replies; 9+ messages in thread From: Pierre-Louis Bossart @ 2021-09-03 16:06 UTC (permalink / raw) To: Daniel Baluta, broonie, lgirdwood, robh+dt, ranjani.sridharan, kai.vehmanen Cc: devicetree, shawnguo, kernel, festevam, linux-imx, peter.ujfalusi, alsa-devel, linux-kernel, s-anna, Daniel Baluta On 9/3/21 9:53 AM, Daniel Baluta wrote: > From: Daniel Baluta <daniel.baluta@nxp.com> > > There are two types of clocks: > * DSP IP clocks > * DAI clocks > > This clocks are necessary in order to power up DSP and DAIs. > > We choose to enable DAI clocks here because of the way i.MX8/i.MX8X > design handles resources (including clocks). > > All clocks are managed by a separate core (named SCU) which communicates > with Linux managed ARM core via a well known API. > > We parse and enable the clocks in probe function and disable them in > remove function. > > Future patches will introduce Power Management support so that we > disable clocks while DSP is not used or system enters power save. > > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > --- > sound/soc/sof/imx/imx-common.c | 44 ++++++++++++++++++++++++++++++++++ > sound/soc/sof/imx/imx-common.h | 13 ++++++++++ > sound/soc/sof/imx/imx8.c | 37 ++++++++++++++++++++++++++++ > sound/soc/sof/imx/imx8m.c | 34 ++++++++++++++++++++++++++ > 4 files changed, 128 insertions(+) > > diff --git a/sound/soc/sof/imx/imx-common.c b/sound/soc/sof/imx/imx-common.c > index 8826ef94f04a..f9d650ad3846 100644 > --- a/sound/soc/sof/imx/imx-common.c > +++ b/sound/soc/sof/imx/imx-common.c > @@ -74,4 +74,48 @@ void imx8_dump(struct snd_sof_dev *sdev, u32 flags) > } > EXPORT_SYMBOL(imx8_dump); > > +int imx8_parse_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks) > +{ > + int ret; > + > + ret = devm_clk_bulk_get(sdev->dev, clks->num_dsp_clks, clks->dsp_clks); > + if (ret) { > + dev_err(sdev->dev, "Failed to request DSP clocks\n"); > + return ret; > + } > + > + ret = devm_clk_bulk_get_optional(sdev->dev, clks->num_dai_clks, clks->dai_clks); > + if (ret) { > + dev_err(sdev->dev, "Failed to request DAI clks\n"); > + return ret; > + } > + > + return 0; > +} > +EXPORT_SYMBOL(imx8_parse_clocks); > + > +int imx8_enable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks) > +{ > + int ret; > + > + ret = clk_bulk_prepare_enable(clks->num_dsp_clks, clks->dsp_clks); > + if (ret) > + return ret; > + ret = clk_bulk_prepare_enable(clks->num_dai_clks, clks->dai_clks); > + if (ret) { > + clk_bulk_disable_unprepare(clks->num_dsp_clks, clks->dsp_clks); > + return ret; > + } > + > + return 0; > +} > +EXPORT_SYMBOL(imx8_enable_clocks); > + > +void imx8_disable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks) > +{ > + clk_bulk_disable_unprepare(clks->num_dsp_clks, clks->dsp_clks); > + clk_bulk_disable_unprepare(clks->num_dai_clks, clks->dai_clks); > +} > +EXPORT_SYMBOL(imx8_disable_clocks); > + > MODULE_LICENSE("Dual BSD/GPL"); > diff --git a/sound/soc/sof/imx/imx-common.h b/sound/soc/sof/imx/imx-common.h > index 1cc7d6704182..54fba9fcd861 100644 > --- a/sound/soc/sof/imx/imx-common.h > +++ b/sound/soc/sof/imx/imx-common.h > @@ -3,6 +3,8 @@ > #ifndef __IMX_COMMON_H__ > #define __IMX_COMMON_H__ > > +#include <linux/clk.h> > + > #define EXCEPT_MAX_HDR_SIZE 0x400 > #define IMX8_STACK_DUMP_SIZE 32 > > @@ -13,4 +15,15 @@ void imx8_get_registers(struct snd_sof_dev *sdev, > > void imx8_dump(struct snd_sof_dev *sdev, u32 flags); > > +struct imx_clocks { > + struct clk_bulk_data *dsp_clks; > + int num_dsp_clks; > + struct clk_bulk_data *dai_clks; > + int num_dai_clks; > +}; > + > +int imx8_parse_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks); > +int imx8_enable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks); > +void imx8_disable_clocks(struct snd_sof_dev *sdev, struct imx_clocks *clks); > + > #endif > diff --git a/sound/soc/sof/imx/imx8.c b/sound/soc/sof/imx/imx8.c > index fc1720c211a3..5370d34edd61 100644 > --- a/sound/soc/sof/imx/imx8.c > +++ b/sound/soc/sof/imx/imx8.c > @@ -41,6 +41,25 @@ > #define MBOX_OFFSET 0x800000 > #define MBOX_SIZE 0x1000 > > +/* DSP clocks */ > +struct clk_bulk_data imx8_dsp_clks[] = { > + { .id = "ipg" }, > + { .id = "ocram" }, > + { .id = "core" }, > +}; > + > +/* DAI clocks */ > +struct clk_bulk_data imx8_dai_clks[] = { > + { .id = "esai0_core" }, > + { .id = "esai0_extal" }, > + { .id = "esai0_spba" }, > + { .id = "sai1_bus" }, > + { .id = "sai1_mclk0" }, > + { .id = "sai1_mclk1" }, > + { .id = "sai1_mclk2" }, > + { .id = "sai1_mclk3" }, > +}; > + > struct imx8_priv { > struct device *dev; > struct snd_sof_dev *sdev; > @@ -57,6 +76,7 @@ struct imx8_priv { > struct device **pd_dev; > struct device_link **link; > > + struct imx_clocks *clks; > }; > > static void imx8_get_reply(struct snd_sof_dev *sdev) > @@ -223,6 +243,10 @@ static int imx8_probe(struct snd_sof_dev *sdev) > if (!priv) > return -ENOMEM; > > + priv->clks = devm_kzalloc(&pdev->dev, sizeof(*priv->clks), GFP_KERNEL); > + if (!priv->clks) > + return -ENOMEM; > + > sdev->num_cores = 1; > sdev->pdata->hw_pdata = priv; > priv->dev = sdev->dev; > @@ -336,6 +360,18 @@ static int imx8_probe(struct snd_sof_dev *sdev) > /* set default mailbox offset for FW ready message */ > sdev->dsp_box.offset = MBOX_OFFSET; > > + /* init clocks info */ > + priv->clks->dsp_clks = imx8_dsp_clks; > + priv->clks->num_dsp_clks = ARRAY_SIZE(imx8_dsp_clks); > + priv->clks->dai_clks = imx8_dai_clks; > + priv->clks->num_dai_clks = ARRAY_SIZE(imx8_dai_clks); > + > + ret = imx8_parse_clocks(sdev, priv->clks); > + if (ret < 0) > + goto exit_pdev_unregister; > + > + imx8_enable_clocks(sdev, priv->clks); > + > return 0; > > exit_pdev_unregister: > @@ -354,6 +390,7 @@ static int imx8_remove(struct snd_sof_dev *sdev) > struct imx8_priv *priv = sdev->pdata->hw_pdata; > int i; > > + imx8_disable_clocks(sdev, priv->clks); > platform_device_unregister(priv->ipc_dev); > > for (i = 0; i < priv->num_domains; i++) { > diff --git a/sound/soc/sof/imx/imx8m.c b/sound/soc/sof/imx/imx8m.c > index 30624fafc632..fea1b72bebaa 100644 > --- a/sound/soc/sof/imx/imx8m.c > +++ b/sound/soc/sof/imx/imx8m.c > @@ -23,6 +23,21 @@ > #define MBOX_OFFSET 0x800000 > #define MBOX_SIZE 0x1000 > > +struct clk_bulk_data imx8m_dsp_clks[] = { > + { .id = "ipg" }, > + { .id = "ocram" }, > + { .id = "core" }, > +}; > + > +struct clk_bulk_data imx8m_dai_clks[] = { > + { .id = "sai3_bus" }, > + { .id = "sai3_mclk0" }, > + { .id = "sai3_mclk1" }, > + { .id = "sai3_mclk2" }, > + { .id = "sai3_mclk3" }, > + { .id = "sdma3_root" }, > +}; > + > struct imx8m_priv { > struct device *dev; > struct snd_sof_dev *sdev; > @@ -30,6 +45,8 @@ struct imx8m_priv { > /* DSP IPC handler */ > struct imx_dsp_ipc *dsp_ipc; > struct platform_device *ipc_dev; > + > + struct imx_clocks *clks; > }; > > static void imx8m_get_reply(struct snd_sof_dev *sdev) > @@ -143,6 +160,10 @@ static int imx8m_probe(struct snd_sof_dev *sdev) > if (!priv) > return -ENOMEM; > > + priv->clks = devm_kzalloc(&pdev->dev, sizeof(*priv->clks), GFP_KERNEL); > + if (!priv->clks) > + return -ENOMEM; > + > sdev->num_cores = 1; > sdev->pdata->hw_pdata = priv; > priv->dev = sdev->dev; > @@ -211,6 +232,18 @@ static int imx8m_probe(struct snd_sof_dev *sdev) > /* set default mailbox offset for FW ready message */ > sdev->dsp_box.offset = MBOX_OFFSET; > > + /* init clocks info */ > + priv->clks->dsp_clks = imx8m_dsp_clks; > + priv->clks->num_dsp_clks = ARRAY_SIZE(imx8m_dsp_clks); > + priv->clks->dai_clks = imx8m_dai_clks; > + priv->clks->num_dai_clks = ARRAY_SIZE(imx8m_dai_clks); > + > + ret = imx8_parse_clocks(sdev, priv->clks); > + if (ret < 0) > + goto exit_pdev_unregister; > + > + imx8_enable_clocks(sdev, priv->clks); > + > return 0; > > exit_pdev_unregister: > @@ -222,6 +255,7 @@ static int imx8m_remove(struct snd_sof_dev *sdev) > { > struct imx8m_priv *priv = sdev->pdata->hw_pdata; > > + imx8_disable_clocks(sdev, priv->clks); > platform_device_unregister(priv->ipc_dev); > > return 0; > ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation 2021-09-03 14:53 [PATCH v2 0/2] Add code to manage DSP related clocks Daniel Baluta 2021-09-03 14:53 ` [PATCH v2 1/2] ASoC: SOF: imx: " Daniel Baluta @ 2021-09-03 14:53 ` Daniel Baluta 2021-09-03 16:53 ` Rob Herring 1 sibling, 1 reply; 9+ messages in thread From: Daniel Baluta @ 2021-09-03 14:53 UTC (permalink / raw) To: broonie, pierre-louis.bossart, lgirdwood, robh+dt, ranjani.sridharan, kai.vehmanen Cc: devicetree, shawnguo, kernel, festevam, linux-imx, peter.ujfalusi, alsa-devel, linux-kernel, s-anna, Daniel Baluta From: Daniel Baluta <daniel.baluta@nxp.com> DSP node on the Linux kernel side must also take care of enabling DAI/DMA related clocks. By design we choose to manage DAI/DMA clocks from the kernel side because of the architecture of some i.MX8 boards. Clocks are handled by a special M4 core which runs a special firmware called SCFW (System Controler firmware). This communicates with A cores running Linux via a special Messaging Unit and implements a custom API which is already implemented by the Linux kernel i.MX clocks implementation. Note that these clocks are optional. We can use the DSP without them. Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> --- .../devicetree/bindings/dsp/fsl,dsp.yaml | 33 +++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml index 7afc9f2be13a..1453668c0194 100644 --- a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +++ b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml @@ -24,16 +24,49 @@ properties: maxItems: 1 clocks: + minItems: 3 items: - description: ipg clock - description: ocram clock - description: core clock + - description: esai0 core clock for accessing registers + - description: esai0 baud clock + - description: esai0 system clock + - description: esai0 spba clock required when ESAI is placed in slave mode + - description: SAI1 bus clock + - description: SAI1 master clock 0 + - description: SAI1 master clock 1 + - description: SAI1 master clock 2 + - description: SAI1 master clock 3 + - description: SAI3 bus clock + - description: SAI3 master clock 0 + - description: SAI3 master clock 1 + - description: SAI3 master clock 2 + - description: SAI3 master clock 3 + - description: SDMA3 root clock used for accessing registers + clock-names: + minItems: 3 items: - const: ipg - const: ocram - const: core + - const: esai0_core + - const: esai0_extal + - const: esai0_fsys + - const: esai0_spba + - const: sai1_bus + - const: sai1_mclk0 + - const: sai1_mclk1 + - const: sai1_mclk2 + - const: sai1_mclk3 + - const: sai3_bus + - const: sai3_mclk0 + - const: sai3_mclk1 + - const: sai3_mclk2 + - const: sai3_mclk3 + - const: smda3_root power-domains: description: -- 2.27.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation 2021-09-03 14:53 ` [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation Daniel Baluta @ 2021-09-03 16:53 ` Rob Herring 2021-09-04 14:50 ` Daniel Baluta 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2021-09-03 16:53 UTC (permalink / raw) To: Daniel Baluta Cc: broonie, pierre-louis.bossart, lgirdwood, ranjani.sridharan, kai.vehmanen, devicetree, shawnguo, kernel, festevam, linux-imx, peter.ujfalusi, alsa-devel, linux-kernel, s-anna, Daniel Baluta On Fri, Sep 03, 2021 at 05:53:40PM +0300, Daniel Baluta wrote: > From: Daniel Baluta <daniel.baluta@nxp.com> > > DSP node on the Linux kernel side must also take care of enabling > DAI/DMA related clocks. > > By design we choose to manage DAI/DMA clocks from the kernel side because of > the architecture of some i.MX8 boards. > > Clocks are handled by a special M4 core which runs a special firmware > called SCFW (System Controler firmware). > > This communicates with A cores running Linux via a special Messaging > Unit and implements a custom API which is already implemented by the > Linux kernel i.MX clocks implementation. > > Note that these clocks are optional. We can use the DSP without them. > > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> > --- > .../devicetree/bindings/dsp/fsl,dsp.yaml | 33 +++++++++++++++++++ > 1 file changed, 33 insertions(+) > > diff --git a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > index 7afc9f2be13a..1453668c0194 100644 > --- a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > +++ b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > @@ -24,16 +24,49 @@ properties: > maxItems: 1 > > clocks: > + minItems: 3 > items: > - description: ipg clock > - description: ocram clock > - description: core clock > + - description: esai0 core clock for accessing registers > + - description: esai0 baud clock > + - description: esai0 system clock > + - description: esai0 spba clock required when ESAI is placed in slave mode > + - description: SAI1 bus clock > + - description: SAI1 master clock 0 > + - description: SAI1 master clock 1 > + - description: SAI1 master clock 2 > + - description: SAI1 master clock 3 > + - description: SAI3 bus clock > + - description: SAI3 master clock 0 > + - description: SAI3 master clock 1 > + - description: SAI3 master clock 2 > + - description: SAI3 master clock 3 > + - description: SDMA3 root clock used for accessing registers Sigh, I just rejected this kind of thing for the other i.MX8 DSP binding[1]. Add a reference to the h/w block and then get the clocks (and other resources) from there. Rob [1] https://lore.kernel.org/linux-devicetree/YTDq%2FkWFPLHUnHMN@robh.at.kernel.org/ ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation 2021-09-03 16:53 ` Rob Herring @ 2021-09-04 14:50 ` Daniel Baluta 2021-09-07 13:09 ` Rob Herring 0 siblings, 1 reply; 9+ messages in thread From: Daniel Baluta @ 2021-09-04 14:50 UTC (permalink / raw) To: Rob Herring Cc: Daniel Baluta, Mark Brown, Pierre-Louis Bossart, Liam Girdwood, Ranjani Sridharan, Kai Vehmanen, Devicetree List, Shawn Guo, Pengutronix Kernel Team, Fabio Estevam, dl-linux-imx, Péter Ujfalusi, Linux-ALSA, Linux Kernel Mailing List, s-anna, Daniel Baluta On Fri, Sep 3, 2021 at 8:11 PM Rob Herring <robh@kernel.org> wrote: > > On Fri, Sep 03, 2021 at 05:53:40PM +0300, Daniel Baluta wrote: > > From: Daniel Baluta <daniel.baluta@nxp.com> > > > > DSP node on the Linux kernel side must also take care of enabling > > DAI/DMA related clocks. > > > > By design we choose to manage DAI/DMA clocks from the kernel side because of > > the architecture of some i.MX8 boards. > > > > Clocks are handled by a special M4 core which runs a special firmware > > called SCFW (System Controler firmware). > > > > This communicates with A cores running Linux via a special Messaging > > Unit and implements a custom API which is already implemented by the > > Linux kernel i.MX clocks implementation. > > > > Note that these clocks are optional. We can use the DSP without them. > > > > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> > > --- > > .../devicetree/bindings/dsp/fsl,dsp.yaml | 33 +++++++++++++++++++ > > 1 file changed, 33 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > > index 7afc9f2be13a..1453668c0194 100644 > > --- a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > > +++ b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > > @@ -24,16 +24,49 @@ properties: > > maxItems: 1 > > > > clocks: > > + minItems: 3 > > items: > > - description: ipg clock > > - description: ocram clock > > - description: core clock > > + - description: esai0 core clock for accessing registers > > + - description: esai0 baud clock > > + - description: esai0 system clock > > + - description: esai0 spba clock required when ESAI is placed in slave mode > > + - description: SAI1 bus clock > > + - description: SAI1 master clock 0 > > + - description: SAI1 master clock 1 > > + - description: SAI1 master clock 2 > > + - description: SAI1 master clock 3 > > + - description: SAI3 bus clock > > + - description: SAI3 master clock 0 > > + - description: SAI3 master clock 1 > > + - description: SAI3 master clock 2 > > + - description: SAI3 master clock 3 > > + - description: SDMA3 root clock used for accessing registers > > Sigh, I just rejected this kind of thing for the other i.MX8 DSP > binding[1]. > > Add a reference to the h/w block and then get the clocks (and other > resources) from there. The H/W block is controlled by the DSP firmware. So, we don't want to use the Linux kernel driver (thus the H/W block device tree node). The only thing that we cannot control from the DSP firmware are the clocks hence we handle them in the DSP node. We moved the DAI clocks under the DSP node as I think you suggested here: https://www.lkml.org/lkml/2020/3/12/969 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation 2021-09-04 14:50 ` Daniel Baluta @ 2021-09-07 13:09 ` Rob Herring 2021-09-09 11:20 ` Daniel Baluta 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2021-09-07 13:09 UTC (permalink / raw) To: Daniel Baluta Cc: Daniel Baluta, Mark Brown, Pierre-Louis Bossart, Liam Girdwood, Ranjani Sridharan, Kai Vehmanen, Devicetree List, Shawn Guo, Pengutronix Kernel Team, Fabio Estevam, dl-linux-imx, Péter Ujfalusi, Linux-ALSA, Linux Kernel Mailing List, Suman Anna, Daniel Baluta, Shengjiu Wang On Sat, Sep 4, 2021 at 9:51 AM Daniel Baluta <daniel.baluta@gmail.com> wrote: > > On Fri, Sep 3, 2021 at 8:11 PM Rob Herring <robh@kernel.org> wrote: > > > > On Fri, Sep 03, 2021 at 05:53:40PM +0300, Daniel Baluta wrote: > > > From: Daniel Baluta <daniel.baluta@nxp.com> > > > > > > DSP node on the Linux kernel side must also take care of enabling > > > DAI/DMA related clocks. > > > > > > By design we choose to manage DAI/DMA clocks from the kernel side because of > > > the architecture of some i.MX8 boards. > > > > > > Clocks are handled by a special M4 core which runs a special firmware > > > called SCFW (System Controler firmware). > > > > > > This communicates with A cores running Linux via a special Messaging > > > Unit and implements a custom API which is already implemented by the > > > Linux kernel i.MX clocks implementation. > > > > > > Note that these clocks are optional. We can use the DSP without them. > > > > > > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> > > > --- > > > .../devicetree/bindings/dsp/fsl,dsp.yaml | 33 +++++++++++++++++++ > > > 1 file changed, 33 insertions(+) > > > > > > diff --git a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > > > index 7afc9f2be13a..1453668c0194 100644 > > > --- a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > > > +++ b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml > > > @@ -24,16 +24,49 @@ properties: > > > maxItems: 1 > > > > > > clocks: > > > + minItems: 3 > > > items: > > > - description: ipg clock > > > - description: ocram clock > > > - description: core clock > > > + - description: esai0 core clock for accessing registers > > > + - description: esai0 baud clock > > > + - description: esai0 system clock > > > + - description: esai0 spba clock required when ESAI is placed in slave mode > > > + - description: SAI1 bus clock > > > + - description: SAI1 master clock 0 > > > + - description: SAI1 master clock 1 > > > + - description: SAI1 master clock 2 > > > + - description: SAI1 master clock 3 > > > + - description: SAI3 bus clock > > > + - description: SAI3 master clock 0 > > > + - description: SAI3 master clock 1 > > > + - description: SAI3 master clock 2 > > > + - description: SAI3 master clock 3 > > > + - description: SDMA3 root clock used for accessing registers > > > > Sigh, I just rejected this kind of thing for the other i.MX8 DSP > > binding[1]. > > > > Add a reference to the h/w block and then get the clocks (and other > > resources) from there. > > The H/W block is controlled by the DSP firmware. So, we don't want > to use the Linux kernel driver (thus the H/W block device tree node). 'status' is how you disable a device to not be used by the OS. The information about that device's resources are already in DT, we don't need to duplicate that here. If you want a list of devices assigned to the DSP here, that would be okay. > The only thing that we cannot control from the DSP firmware are the clocks > hence we handle them in the DSP node. > > We moved the DAI clocks under the DSP node as I think you suggested here: > > https://www.lkml.org/lkml/2020/3/12/969 No, that's certainly not what I was suggesting. The resources in the DSP node should be the h/w resources of the DSP itself. Rob ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation 2021-09-07 13:09 ` Rob Herring @ 2021-09-09 11:20 ` Daniel Baluta 2021-09-09 17:15 ` Rob Herring 0 siblings, 1 reply; 9+ messages in thread From: Daniel Baluta @ 2021-09-09 11:20 UTC (permalink / raw) To: Rob Herring Cc: Daniel Baluta, Mark Brown, Pierre-Louis Bossart, Liam Girdwood, Ranjani Sridharan, Kai Vehmanen, Devicetree List, Shawn Guo, Pengutronix Kernel Team, Fabio Estevam, dl-linux-imx, Péter Ujfalusi, Linux-ALSA, Linux Kernel Mailing List, Suman Anna, Daniel Baluta, Shengjiu Wang > > The H/W block is controlled by the DSP firmware. So, we don't want > > to use the Linux kernel driver (thus the H/W block device tree node). > > 'status' is how you disable a device to not be used by the OS. > > The information about that device's resources are already in DT, we > don't need to duplicate that here. If you want a list of devices > assigned to the DSP here, that would be okay. Thanks! This is a very good idea. I was thinking at a totally different thing. So having something like this: dsp { hw-block-list = <&sai1>, <&sai2>; } And then inside the DSP driver we can get access to sai1 clocks. Do you know of any standard property name? > > > The only thing that we cannot control from the DSP firmware are the clocks > > hence we handle them in the DSP node. > > > > We moved the DAI clocks under the DSP node as I think you suggested here: > > > > https://www.lkml.org/lkml/2020/3/12/969 > > No, that's certainly not what I was suggesting. The resources in the > DSP node should be the h/w resources of the DSP itself. I see thanks! ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation 2021-09-09 11:20 ` Daniel Baluta @ 2021-09-09 17:15 ` Rob Herring 0 siblings, 0 replies; 9+ messages in thread From: Rob Herring @ 2021-09-09 17:15 UTC (permalink / raw) To: Daniel Baluta Cc: Daniel Baluta, Mark Brown, Pierre-Louis Bossart, Liam Girdwood, Ranjani Sridharan, Kai Vehmanen, Devicetree List, Shawn Guo, Pengutronix Kernel Team, Fabio Estevam, dl-linux-imx, Péter Ujfalusi, Linux-ALSA, Linux Kernel Mailing List, Suman Anna, Daniel Baluta, Shengjiu Wang On Thu, Sep 9, 2021 at 6:21 AM Daniel Baluta <daniel.baluta@gmail.com> wrote: > > > > The H/W block is controlled by the DSP firmware. So, we don't want > > > to use the Linux kernel driver (thus the H/W block device tree node). > > > > 'status' is how you disable a device to not be used by the OS. > > > > The information about that device's resources are already in DT, we > > don't need to duplicate that here. If you want a list of devices > > assigned to the DSP here, that would be okay. > > Thanks! This is a very good idea. I was thinking at a totally different thing. > > So having something like this: > > dsp { > > > hw-block-list = <&sai1>, <&sai2>; > > } Yes. > And then inside the DSP driver we can get access to sai1 clocks. Do > you know of any standard property name? There isn't. So it needs a vendor prefix. There's been some discussions around 'system devicetree' where all processors (like the DSP) view of the system get represented. Device assignment is one of the issues to solve with that, but it's not anywhere close to having something to help here. Rob ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-09-09 17:16 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-09-03 14:53 [PATCH v2 0/2] Add code to manage DSP related clocks Daniel Baluta 2021-09-03 14:53 ` [PATCH v2 1/2] ASoC: SOF: imx: " Daniel Baluta 2021-09-03 16:06 ` Pierre-Louis Bossart 2021-09-03 14:53 ` [PATCH v2 2/2] dt-bindings: dsp: fsl: Add DSP optional clocks documentation Daniel Baluta 2021-09-03 16:53 ` Rob Herring 2021-09-04 14:50 ` Daniel Baluta 2021-09-07 13:09 ` Rob Herring 2021-09-09 11:20 ` Daniel Baluta 2021-09-09 17:15 ` Rob Herring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).