linux-kernel - Re: [PATCH] bus: imx-aipstz: set default value for opacr registers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAA+D8AOSzBNOoET0_aJRYhf9dsDu4mQqGWso9YNN_HNQU1582w@mail.gmail.com>
Date: Fri, 23 Jan 2026 17:59:33 +0800
From: Shengjiu Wang <shengjiu.wang@...il.com>
To: Daniel Baluta <daniel.baluta@....nxp.com>
Cc: 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
Subject: Re: [PATCH] bus: imx-aipstz: set default value for opacr registers

On Fri, Jan 23, 2026 at 5:11 PM Daniel Baluta <daniel.baluta@....nxp.com> wrote:
>
> 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.

right,  I will remove them.

Best regards
Shengjiu Wang
>
>