All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Boyd <sboyd@kernel.org>
To: Peng Fan (OSS) <peng.fan@oss.nxp.com>, mturquette@baylibre.com
Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, kernel@pengutronix.de,
	Peng Fan <peng.fan@nxp.com>, Jerome Brunet <jbrunet@baylibre.com>
Subject: Re: [PATCH 0/3] clk: support regmap
Date: Wed, 02 Jun 2021 01:18:44 -0700	[thread overview]
Message-ID: <162262192433.4130789.1017942859005253343@swboyd.mtv.corp.google.com> (raw)
In-Reply-To: <20210528113403.5374-1-peng.fan@oss.nxp.com>

Quoting Peng Fan (OSS) (2021-05-28 04:34:00)
> From: Peng Fan <peng.fan@nxp.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.

> 
> Peng Fan (3):
>   clk: mux: support regmap
>   clk: fractional-divider: support regmap
>   clk: gate: support regmap

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Boyd <sboyd@kernel.org>
To: Peng Fan (OSS) <peng.fan@oss.nxp.com>, mturquette@baylibre.com
Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, kernel@pengutronix.de,
	Peng Fan <peng.fan@nxp.com>, Jerome Brunet <jbrunet@baylibre.com>
Subject: Re: [PATCH 0/3] clk: support regmap
Date: Wed, 02 Jun 2021 01:18:44 -0700	[thread overview]
Message-ID: <162262192433.4130789.1017942859005253343@swboyd.mtv.corp.google.com> (raw)
In-Reply-To: <20210528113403.5374-1-peng.fan@oss.nxp.com>

Quoting Peng Fan (OSS) (2021-05-28 04:34:00)
> From: Peng Fan <peng.fan@nxp.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.

> 
> Peng Fan (3):
>   clk: mux: support regmap
>   clk: fractional-divider: support regmap
>   clk: gate: support regmap

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2021-06-02  8:18 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-28 11:34 [PATCH 0/3] clk: support regmap Peng Fan (OSS)
2021-05-28 11:34 ` Peng Fan (OSS)
2021-05-28 11:34 ` [PATCH 1/3] clk: mux: " Peng Fan (OSS)
2021-05-28 11:34   ` Peng Fan (OSS)
2021-05-28 11:34 ` [PATCH 2/3] clk: fractional-divider: " Peng Fan (OSS)
2021-05-28 11:34   ` Peng Fan (OSS)
2021-05-28 11:34 ` [PATCH 3/3] clk: gate: " Peng Fan (OSS)
2021-05-28 11:34   ` Peng Fan (OSS)
2021-06-02  8:18 ` Stephen Boyd [this message]
2021-06-02  8:18   ` [PATCH 0/3] clk: " Stephen Boyd
2021-06-02  8:21   ` Marc Kleine-Budde
2021-06-02  8:21     ` Marc Kleine-Budde
2021-06-02  8:40     ` Peng Fan
2021-06-02  8:40       ` Peng Fan
2021-06-02  9:32     ` Jerome Brunet
2021-06-02  9:32       ` Jerome Brunet
2021-06-18 14:30       ` Jerome Brunet
2021-06-18 14:30         ` Jerome Brunet
2021-07-27  0:47         ` Stephen Boyd
2021-07-27  0:47           ` Stephen Boyd
2021-06-02  8:39   ` Peng Fan
2021-06-02  8:39     ` Peng Fan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=162262192433.4130789.1017942859005253343@swboyd.mtv.corp.google.com \
    --to=sboyd@kernel.org \
    --cc=jbrunet@baylibre.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=peng.fan@nxp.com \
    --cc=peng.fan@oss.nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.