linux-kernel - RE: [PATCH 0/3] clk: support regmap

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]

Date:   Wed, 2 Jun 2021 08:40:18 +0000
From:   Peng Fan <peng.fan@....com>
To:     Marc Kleine-Budde <mkl@...gutronix.de>,
        Stephen Boyd <sboyd@...nel.org>,
        "Peng Fan (OSS)" <peng.fan@....nxp.com>,
        "mturquette@...libre.com" <mturquette@...libre.com>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        Jerome Brunet <jbrunet@...libre.com>
Subject: RE: [PATCH 0/3] clk: support regmap

> Subject: Re: [PATCH 0/3] clk: support regmap
> 
> On 6/2/21 10:18 AM, Stephen Boyd wrote:
> > Quoting Peng Fan (OSS) (2021-05-28 04:34:00)
> >> From: Peng Fan <peng.fan@....com>
> >>
> >> To i.MX8ULP, a PCC register provides clk(mux, gate, divider) and
> >> peripheral reset functionality, so we need make sure the access to
> >> the PCC register be protected to avoid concurrent access from clk and
> reset subsystem.
> >>
> >> So let's use regmap here.
> >>
> >> The i.MX specific code will be posted if this patchset is ok for you.
> >
> > We have a couple regmap clk drivers in the tree. Either combine the
> > different regmap clk drivers or duplicate it into the imx directory.
> > I'd prefer we combine them but last time I couldn't agree on the
> > approach when Jerome wanted to do it. Maybe now is the time to combine
> > them all into one common piece of code.
> 
> IMHO for the basic drivers, such as gate, divider, mux, mux-div, etc... it makes
> no sense to have them in an arch specific subdir, like meson.

Yeah. Let's make it as common piece code could be reused by others.

Thanks,
Peng.

> 
> regards,
> Marc
> 
> --
> Pengutronix e.K.                 | Marc Kleine-Budde           |
> Embedded Linux                   | https://www.pengutronix.de  |
> Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
> Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |