[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <59773aa2-5949-490c-bced-b8d110152505@oss.nxp.com>
Date: Fri, 23 Jan 2026 11:14:04 +0200
From: Daniel Baluta <daniel.baluta@....nxp.com>
To: Shengjiu Wang <shengjiu.wang@....com>, shawnguo@...nel.org,
s.hauer@...gutronix.de, kernel@...gutronix.de, festevam@...il.com,
laurentiu.mihalcea@....com, daniel.baluta@....com, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
shengjiu.wang@...il.com
Subject: Re: [PATCH] bus: imx-aipstz: set default value for opacr registers
On 1/23/26 10:48, Shengjiu Wang wrote:
> The sdma script app_2_mcu needs the permission to access the peripheral
> devices, so need to clear the Supervisor Protect bit of SPBA2, otherwise
> the sdma script can't work.
>
> As the imx-aipstz is a common driver for all apis bus, so set default
> value for all opacr registers, that the peripheral does not require
> supervisor privilege level for accesses.
>
> Signed-off-by: Shengjiu Wang <shengjiu.wang@....com>
> ---
> drivers/bus/imx-aipstz.c | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
<snip>
> static const struct of_device_id imx_aipstz_match_table[] = {
> @@ -84,6 +99,11 @@ static const struct dev_pm_ops imx_aipstz_pm_ops = {
> */
> static const struct imx_aipstz_config imx8mp_aipstz_default_cfg = {
> .mpr0 = 0x77777777,
> + .opacr0 = 0x0,
> + .opacr1 = 0x0,
> + .opacr2 = 0x0,
> + .opacr3 = 0x0,
> + .opacr4 = 0x0,
> };
>
No need to zero these fields as they are zero by default being part of a global var.
Powered by blists - more mailing lists