* [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate()
2026-01-08 21:16 [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Brian Masney
@ 2026-01-08 21:16 ` Brian Masney
2026-01-22 0:26 ` Alexandre Belloni
2026-01-10 19:11 ` (subset) [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Bjorn Andersson
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Brian Masney @ 2026-01-08 21:16 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Brian Masney, Alexandre Belloni,
linux-rtc
The divider_round_rate() function is now deprecated, so let's migrate
to divider_determine_rate() instead so that this deprecated API can be
removed.
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
To: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: linux-rtc@vger.kernel.org
---
drivers/rtc/rtc-ac100.c | 75 +++++++++++++++++++++++++------------------------
1 file changed, 38 insertions(+), 37 deletions(-)
diff --git a/drivers/rtc/rtc-ac100.c b/drivers/rtc/rtc-ac100.c
index 33626311fa781b5ce90dcc472f948dc933bbc897..16aca4431da8c029e6195d8a3c9fe75fa95d0bc0 100644
--- a/drivers/rtc/rtc-ac100.c
+++ b/drivers/rtc/rtc-ac100.c
@@ -140,42 +140,16 @@ static unsigned long ac100_clkout_recalc_rate(struct clk_hw *hw,
AC100_CLKOUT_DIV_WIDTH);
}
-static long ac100_clkout_round_rate(struct clk_hw *hw, unsigned long rate,
- unsigned long prate)
-{
- unsigned long best_rate = 0, tmp_rate, tmp_prate;
- int i;
-
- if (prate == AC100_RTC_32K_RATE)
- return divider_round_rate(hw, rate, &prate, NULL,
- AC100_CLKOUT_DIV_WIDTH,
- CLK_DIVIDER_POWER_OF_TWO);
-
- for (i = 0; ac100_clkout_prediv[i].div; i++) {
- tmp_prate = DIV_ROUND_UP(prate, ac100_clkout_prediv[i].val);
- tmp_rate = divider_round_rate(hw, rate, &tmp_prate, NULL,
- AC100_CLKOUT_DIV_WIDTH,
- CLK_DIVIDER_POWER_OF_TWO);
-
- if (tmp_rate > rate)
- continue;
- if (rate - tmp_rate < best_rate - tmp_rate)
- best_rate = tmp_rate;
- }
-
- return best_rate;
-}
-
static int ac100_clkout_determine_rate(struct clk_hw *hw,
struct clk_rate_request *req)
{
- struct clk_hw *best_parent;
+ int i, ret, num_parents = clk_hw_get_num_parents(hw);
+ struct clk_hw *best_parent = NULL;
unsigned long best = 0;
- int i, num_parents = clk_hw_get_num_parents(hw);
for (i = 0; i < num_parents; i++) {
struct clk_hw *parent = clk_hw_get_parent_by_index(hw, i);
- unsigned long tmp, prate;
+ unsigned long prate;
/*
* The clock has two parents, one is a fixed clock which is
@@ -199,13 +173,40 @@ static int ac100_clkout_determine_rate(struct clk_hw *hw,
prate = clk_hw_get_rate(parent);
- tmp = ac100_clkout_round_rate(hw, req->rate, prate);
-
- if (tmp > req->rate)
- continue;
- if (req->rate - tmp < req->rate - best) {
- best = tmp;
- best_parent = parent;
+ if (prate == AC100_RTC_32K_RATE) {
+ struct clk_rate_request div_req = *req;
+
+ div_req.best_parent_rate = prate;
+
+ ret = divider_determine_rate(hw, &div_req, NULL,
+ AC100_CLKOUT_DIV_WIDTH,
+ CLK_DIVIDER_POWER_OF_TWO);
+ if (ret != 0 || div_req.rate > req->rate)
+ continue;
+ else if (req->rate - div_req.rate < req->rate - best) {
+ best = div_req.rate;
+ best_parent = parent;
+ }
+ } else {
+ int j;
+
+ for (j = 0; ac100_clkout_prediv[j].div; j++) {
+ struct clk_rate_request div_req = *req;
+ unsigned long tmp_prate;
+
+ tmp_prate = DIV_ROUND_UP(prate, ac100_clkout_prediv[j].div);
+ div_req.best_parent_rate = tmp_prate;
+
+ ret = divider_determine_rate(hw, &div_req, NULL,
+ AC100_CLKOUT_DIV_WIDTH,
+ CLK_DIVIDER_POWER_OF_TWO);
+ if (ret != 0 || div_req.rate > req->rate)
+ continue;
+ else if (req->rate - div_req.rate < req->rate - best) {
+ best = div_req.rate;
+ best_parent = parent;
+ }
+ }
}
}
@@ -213,7 +214,7 @@ static int ac100_clkout_determine_rate(struct clk_hw *hw,
return -EINVAL;
req->best_parent_hw = best_parent;
- req->best_parent_rate = best;
+ req->best_parent_rate = clk_hw_get_rate(best_parent);
req->rate = best;
return 0;
--
2.52.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate()
2026-01-08 21:16 ` [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate() Brian Masney
@ 2026-01-22 0:26 ` Alexandre Belloni
2026-01-22 0:48 ` Brian Masney
0 siblings, 1 reply; 9+ messages in thread
From: Alexandre Belloni @ 2026-01-22 0:26 UTC (permalink / raw)
To: Brian Masney
Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel,
linux-rtc
Hello,
On 08/01/2026 16:16:21-0500, Brian Masney wrote:
> The divider_round_rate() function is now deprecated, so let's migrate
> to divider_determine_rate() instead so that this deprecated API can be
> removed.
>
> Signed-off-by: Brian Masney <bmasney@redhat.com>
>
> ---
> To: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: linux-rtc@vger.kernel.org
> ---
> drivers/rtc/rtc-ac100.c | 75 +++++++++++++++++++++++++------------------------
> 1 file changed, 38 insertions(+), 37 deletions(-)
>
> diff --git a/drivers/rtc/rtc-ac100.c b/drivers/rtc/rtc-ac100.c
> index 33626311fa781b5ce90dcc472f948dc933bbc897..16aca4431da8c029e6195d8a3c9fe75fa95d0bc0 100644
> --- a/drivers/rtc/rtc-ac100.c
> +++ b/drivers/rtc/rtc-ac100.c
> @@ -140,42 +140,16 @@ static unsigned long ac100_clkout_recalc_rate(struct clk_hw *hw,
> AC100_CLKOUT_DIV_WIDTH);
> }
>
> -static long ac100_clkout_round_rate(struct clk_hw *hw, unsigned long rate,
> - unsigned long prate)
> -{
> - unsigned long best_rate = 0, tmp_rate, tmp_prate;
> - int i;
> -
> - if (prate == AC100_RTC_32K_RATE)
> - return divider_round_rate(hw, rate, &prate, NULL,
> - AC100_CLKOUT_DIV_WIDTH,
> - CLK_DIVIDER_POWER_OF_TWO);
> -
> - for (i = 0; ac100_clkout_prediv[i].div; i++) {
> - tmp_prate = DIV_ROUND_UP(prate, ac100_clkout_prediv[i].val);
> - tmp_rate = divider_round_rate(hw, rate, &tmp_prate, NULL,
> - AC100_CLKOUT_DIV_WIDTH,
> - CLK_DIVIDER_POWER_OF_TWO);
> -
> - if (tmp_rate > rate)
> - continue;
> - if (rate - tmp_rate < best_rate - tmp_rate)
> - best_rate = tmp_rate;
> - }
> -
> - return best_rate;
> -}
> -
> static int ac100_clkout_determine_rate(struct clk_hw *hw,
> struct clk_rate_request *req)
> {
> - struct clk_hw *best_parent;
> + int i, ret, num_parents = clk_hw_get_num_parents(hw);
> + struct clk_hw *best_parent = NULL;
> unsigned long best = 0;
> - int i, num_parents = clk_hw_get_num_parents(hw);
>
> for (i = 0; i < num_parents; i++) {
> struct clk_hw *parent = clk_hw_get_parent_by_index(hw, i);
> - unsigned long tmp, prate;
> + unsigned long prate;
>
> /*
> * The clock has two parents, one is a fixed clock which is
> @@ -199,13 +173,40 @@ static int ac100_clkout_determine_rate(struct clk_hw *hw,
>
> prate = clk_hw_get_rate(parent);
>
> - tmp = ac100_clkout_round_rate(hw, req->rate, prate);
> -
> - if (tmp > req->rate)
> - continue;
> - if (req->rate - tmp < req->rate - best) {
> - best = tmp;
> - best_parent = parent;
> + if (prate == AC100_RTC_32K_RATE) {
> + struct clk_rate_request div_req = *req;
> +
> + div_req.best_parent_rate = prate;
> +
> + ret = divider_determine_rate(hw, &div_req, NULL,
> + AC100_CLKOUT_DIV_WIDTH,
> + CLK_DIVIDER_POWER_OF_TWO);
> + if (ret != 0 || div_req.rate > req->rate)
> + continue;
This leaves a braces inbalance
> + else if (req->rate - div_req.rate < req->rate - best) {
> + best = div_req.rate;
> + best_parent = parent;
> + }
> + } else {
> + int j;
> +
> + for (j = 0; ac100_clkout_prediv[j].div; j++) {
> + struct clk_rate_request div_req = *req;
> + unsigned long tmp_prate;
> +
> + tmp_prate = DIV_ROUND_UP(prate, ac100_clkout_prediv[j].div);
> + div_req.best_parent_rate = tmp_prate;
> +
> + ret = divider_determine_rate(hw, &div_req, NULL,
> + AC100_CLKOUT_DIV_WIDTH,
> + CLK_DIVIDER_POWER_OF_TWO);
> + if (ret != 0 || div_req.rate > req->rate)
> + continue;
Ditto.
> + else if (req->rate - div_req.rate < req->rate - best) {
> + best = div_req.rate;
> + best_parent = parent;
> + }
> + }
> }
> }
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate()
2026-01-22 0:26 ` Alexandre Belloni
@ 2026-01-22 0:48 ` Brian Masney
2026-01-22 2:02 ` Alexandre Belloni
0 siblings, 1 reply; 9+ messages in thread
From: Brian Masney @ 2026-01-22 0:48 UTC (permalink / raw)
To: Alexandre Belloni
Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel,
linux-rtc
Hi Alexandre,
On Thu, Jan 22, 2026 at 01:26:09AM +0100, Alexandre Belloni wrote:
> On 08/01/2026 16:16:21-0500, Brian Masney wrote:
> > The divider_round_rate() function is now deprecated, so let's migrate
> > to divider_determine_rate() instead so that this deprecated API can be
> > removed.
> >
> > Signed-off-by: Brian Masney <bmasney@redhat.com>
> >
> > ---
> > To: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > Cc: linux-rtc@vger.kernel.org
> > ---
> > drivers/rtc/rtc-ac100.c | 75 +++++++++++++++++++++++++------------------------
> > 1 file changed, 38 insertions(+), 37 deletions(-)
> >
> > diff --git a/drivers/rtc/rtc-ac100.c b/drivers/rtc/rtc-ac100.c
> > index 33626311fa781b5ce90dcc472f948dc933bbc897..16aca4431da8c029e6195d8a3c9fe75fa95d0bc0 100644
> > --- a/drivers/rtc/rtc-ac100.c
> > +++ b/drivers/rtc/rtc-ac100.c
> > @@ -140,42 +140,16 @@ static unsigned long ac100_clkout_recalc_rate(struct clk_hw *hw,
> > AC100_CLKOUT_DIV_WIDTH);
> > }
> >
> > -static long ac100_clkout_round_rate(struct clk_hw *hw, unsigned long rate,
> > - unsigned long prate)
> > -{
> > - unsigned long best_rate = 0, tmp_rate, tmp_prate;
> > - int i;
> > -
> > - if (prate == AC100_RTC_32K_RATE)
> > - return divider_round_rate(hw, rate, &prate, NULL,
> > - AC100_CLKOUT_DIV_WIDTH,
> > - CLK_DIVIDER_POWER_OF_TWO);
> > -
> > - for (i = 0; ac100_clkout_prediv[i].div; i++) {
> > - tmp_prate = DIV_ROUND_UP(prate, ac100_clkout_prediv[i].val);
> > - tmp_rate = divider_round_rate(hw, rate, &tmp_prate, NULL,
> > - AC100_CLKOUT_DIV_WIDTH,
> > - CLK_DIVIDER_POWER_OF_TWO);
> > -
> > - if (tmp_rate > rate)
> > - continue;
> > - if (rate - tmp_rate < best_rate - tmp_rate)
> > - best_rate = tmp_rate;
> > - }
> > -
> > - return best_rate;
> > -}
> > -
> > static int ac100_clkout_determine_rate(struct clk_hw *hw,
> > struct clk_rate_request *req)
> > {
> > - struct clk_hw *best_parent;
> > + int i, ret, num_parents = clk_hw_get_num_parents(hw);
> > + struct clk_hw *best_parent = NULL;
> > unsigned long best = 0;
> > - int i, num_parents = clk_hw_get_num_parents(hw);
> >
> > for (i = 0; i < num_parents; i++) {
> > struct clk_hw *parent = clk_hw_get_parent_by_index(hw, i);
> > - unsigned long tmp, prate;
> > + unsigned long prate;
> >
> > /*
> > * The clock has two parents, one is a fixed clock which is
> > @@ -199,13 +173,40 @@ static int ac100_clkout_determine_rate(struct clk_hw *hw,
> >
> > prate = clk_hw_get_rate(parent);
> >
> > - tmp = ac100_clkout_round_rate(hw, req->rate, prate);
> > -
> > - if (tmp > req->rate)
> > - continue;
> > - if (req->rate - tmp < req->rate - best) {
> > - best = tmp;
> > - best_parent = parent;
> > + if (prate == AC100_RTC_32K_RATE) {
> > + struct clk_rate_request div_req = *req;
> > +
> > + div_req.best_parent_rate = prate;
> > +
> > + ret = divider_determine_rate(hw, &div_req, NULL,
> > + AC100_CLKOUT_DIV_WIDTH,
> > + CLK_DIVIDER_POWER_OF_TWO);
> > + if (ret != 0 || div_req.rate > req->rate)
> > + continue;
>
> This leaves a braces inbalance
To be clear, you also want the braces around the if like this:
if (ret != 0 || div_req.rate > req->rate) {
continue;
} else if (req->rate - div_req.rate < req->rate - best) {
best = div_req.rate;
best_parent = parent;
}
Brian
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate()
2026-01-22 0:48 ` Brian Masney
@ 2026-01-22 2:02 ` Alexandre Belloni
0 siblings, 0 replies; 9+ messages in thread
From: Alexandre Belloni @ 2026-01-22 2:02 UTC (permalink / raw)
To: Brian Masney
Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel,
linux-rtc
On 21/01/2026 19:48:19-0500, Brian Masney wrote:
> Hi Alexandre,
>
> On Thu, Jan 22, 2026 at 01:26:09AM +0100, Alexandre Belloni wrote:
> > On 08/01/2026 16:16:21-0500, Brian Masney wrote:
> > > The divider_round_rate() function is now deprecated, so let's migrate
> > > to divider_determine_rate() instead so that this deprecated API can be
> > > removed.
> > >
> > > Signed-off-by: Brian Masney <bmasney@redhat.com>
> > >
> > > ---
> > > To: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > > Cc: linux-rtc@vger.kernel.org
> > > ---
> > > drivers/rtc/rtc-ac100.c | 75 +++++++++++++++++++++++++------------------------
> > > 1 file changed, 38 insertions(+), 37 deletions(-)
> > >
> > > diff --git a/drivers/rtc/rtc-ac100.c b/drivers/rtc/rtc-ac100.c
> > > index 33626311fa781b5ce90dcc472f948dc933bbc897..16aca4431da8c029e6195d8a3c9fe75fa95d0bc0 100644
> > > --- a/drivers/rtc/rtc-ac100.c
> > > +++ b/drivers/rtc/rtc-ac100.c
> > > @@ -140,42 +140,16 @@ static unsigned long ac100_clkout_recalc_rate(struct clk_hw *hw,
> > > AC100_CLKOUT_DIV_WIDTH);
> > > }
> > >
> > > -static long ac100_clkout_round_rate(struct clk_hw *hw, unsigned long rate,
> > > - unsigned long prate)
> > > -{
> > > - unsigned long best_rate = 0, tmp_rate, tmp_prate;
> > > - int i;
> > > -
> > > - if (prate == AC100_RTC_32K_RATE)
> > > - return divider_round_rate(hw, rate, &prate, NULL,
> > > - AC100_CLKOUT_DIV_WIDTH,
> > > - CLK_DIVIDER_POWER_OF_TWO);
> > > -
> > > - for (i = 0; ac100_clkout_prediv[i].div; i++) {
> > > - tmp_prate = DIV_ROUND_UP(prate, ac100_clkout_prediv[i].val);
> > > - tmp_rate = divider_round_rate(hw, rate, &tmp_prate, NULL,
> > > - AC100_CLKOUT_DIV_WIDTH,
> > > - CLK_DIVIDER_POWER_OF_TWO);
> > > -
> > > - if (tmp_rate > rate)
> > > - continue;
> > > - if (rate - tmp_rate < best_rate - tmp_rate)
> > > - best_rate = tmp_rate;
> > > - }
> > > -
> > > - return best_rate;
> > > -}
> > > -
> > > static int ac100_clkout_determine_rate(struct clk_hw *hw,
> > > struct clk_rate_request *req)
> > > {
> > > - struct clk_hw *best_parent;
> > > + int i, ret, num_parents = clk_hw_get_num_parents(hw);
> > > + struct clk_hw *best_parent = NULL;
> > > unsigned long best = 0;
> > > - int i, num_parents = clk_hw_get_num_parents(hw);
> > >
> > > for (i = 0; i < num_parents; i++) {
> > > struct clk_hw *parent = clk_hw_get_parent_by_index(hw, i);
> > > - unsigned long tmp, prate;
> > > + unsigned long prate;
> > >
> > > /*
> > > * The clock has two parents, one is a fixed clock which is
> > > @@ -199,13 +173,40 @@ static int ac100_clkout_determine_rate(struct clk_hw *hw,
> > >
> > > prate = clk_hw_get_rate(parent);
> > >
> > > - tmp = ac100_clkout_round_rate(hw, req->rate, prate);
> > > -
> > > - if (tmp > req->rate)
> > > - continue;
> > > - if (req->rate - tmp < req->rate - best) {
> > > - best = tmp;
> > > - best_parent = parent;
> > > + if (prate == AC100_RTC_32K_RATE) {
> > > + struct clk_rate_request div_req = *req;
> > > +
> > > + div_req.best_parent_rate = prate;
> > > +
> > > + ret = divider_determine_rate(hw, &div_req, NULL,
> > > + AC100_CLKOUT_DIV_WIDTH,
> > > + CLK_DIVIDER_POWER_OF_TWO);
> > > + if (ret != 0 || div_req.rate > req->rate)
> > > + continue;
> >
> > This leaves a braces inbalance
>
> To be clear, you also want the braces around the if like this:
>
> if (ret != 0 || div_req.rate > req->rate) {
> continue;
> } else if (req->rate - div_req.rate < req->rate - best) {
> best = div_req.rate;
> best_parent = parent;
> }
>
Yes, you can change that and add my ack on v2, I'm fine with this going
through clk.
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: (subset) [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends
2026-01-08 21:16 [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Brian Masney
2026-01-08 21:16 ` [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate() Brian Masney
@ 2026-01-10 19:11 ` Bjorn Andersson
2026-01-15 21:05 ` Dmitry Baryshkov
` (2 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Bjorn Andersson @ 2026-01-10 19:11 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Brian Masney
Cc: linux-clk, linux-kernel, Chen Wang, Inochi Amaoto, sophgo,
Chen-Yu Tsai, Maxime Ripard, Jernej Skrabec, Samuel Holland,
linux-arm-kernel, linux-sunxi, Alexandre Belloni, linux-rtc,
Andreas Färber, Manivannan Sadhasivam, linux-actions,
Keguang Zhang, linux-mips, Taichi Sugaya, Takao Orito,
Jacky Huang, Shan-Chun Hung, Vladimir Zapolskiy,
Piotr Wojtaszczyk, linux-arm-msm, Orson Zhai, Baolin Wang,
Chunyan Zhang, Maxime Coquelin, Alexandre Torgue, linux-stm32,
Michal Simek, Rob Clark, Dmitry Baryshkov, David Airlie,
Simona Vetter, Abhinav Kumar, Jessica Zhang, Sean Paul,
Marijn Suijten, dri-devel, freedreno, Vinod Koul, Neil Armstrong,
linux-phy
On Thu, 08 Jan 2026 16:16:18 -0500, Brian Masney wrote:
> Here's a series that gets rid of the deprecated APIs
> divider_round_rate(), divider_round_rate_parent(), and
> divider_ro_round_rate_parent() since these functions are just wrappers
> for the determine_rate variant.
>
> Note that when I converted some of these drivers from round_rate to
> determine_rate, this was mistakenly converted to the following in some
> cases:
>
> [...]
Applied, thanks!
[14/27] clk: qcom: alpha-pll: convert from divider_round_rate() to divider_determine_rate()
commit: e1f08613e113f02a3ec18c9a7964de97f940acbf
[15/27] clk: qcom: regmap-divider: convert from divider_ro_round_rate() to divider_ro_determine_rate()
commit: 35a48f41b63f67c490f3a2a89b042536be67cf0f
[16/27] clk: qcom: regmap-divider: convert from divider_round_rate() to divider_determine_rate()
commit: b2f36d675e09299d9aee395c6f83d8a95d4c9441
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends
2026-01-08 21:16 [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Brian Masney
2026-01-08 21:16 ` [PATCH 03/27] rtc: ac100: convert from divider_round_rate() to divider_determine_rate() Brian Masney
2026-01-10 19:11 ` (subset) [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Bjorn Andersson
@ 2026-01-15 21:05 ` Dmitry Baryshkov
2026-01-21 22:53 ` Brian Masney
2026-02-04 15:44 ` (subset) " Vinod Koul
4 siblings, 0 replies; 9+ messages in thread
From: Dmitry Baryshkov @ 2026-01-15 21:05 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Brian Masney
Cc: linux-clk, linux-kernel, Chen Wang, Inochi Amaoto, sophgo,
Chen-Yu Tsai, Maxime Ripard, Jernej Skrabec, Samuel Holland,
linux-arm-kernel, linux-sunxi, Alexandre Belloni, linux-rtc,
Andreas Färber, Manivannan Sadhasivam, linux-actions,
Keguang Zhang, linux-mips, Taichi Sugaya, Takao Orito,
Jacky Huang, Shan-Chun Hung, Vladimir Zapolskiy,
Piotr Wojtaszczyk, Bjorn Andersson, linux-arm-msm, Orson Zhai,
Baolin Wang, Chunyan Zhang, Maxime Coquelin, Alexandre Torgue,
linux-stm32, Michal Simek, Rob Clark, Dmitry Baryshkov,
David Airlie, Simona Vetter, Abhinav Kumar, Jessica Zhang,
Sean Paul, Marijn Suijten, dri-devel, freedreno, Vinod Koul,
Neil Armstrong, linux-phy
On Thu, 08 Jan 2026 16:16:18 -0500, Brian Masney wrote:
> Here's a series that gets rid of the deprecated APIs
> divider_round_rate(), divider_round_rate_parent(), and
> divider_ro_round_rate_parent() since these functions are just wrappers
> for the determine_rate variant.
>
> Note that when I converted some of these drivers from round_rate to
> determine_rate, this was mistakenly converted to the following in some
> cases:
>
> [...]
Applied to msm-next, thanks!
[24/27] drm/msm/dsi_phy_14nm: convert from divider_round_rate() to divider_determine_rate()
https://gitlab.freedesktop.org/lumag/msm/-/commit/1d232f793d4d
Best regards,
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends
2026-01-08 21:16 [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Brian Masney
` (2 preceding siblings ...)
2026-01-15 21:05 ` Dmitry Baryshkov
@ 2026-01-21 22:53 ` Brian Masney
2026-02-04 15:44 ` (subset) " Vinod Koul
4 siblings, 0 replies; 9+ messages in thread
From: Brian Masney @ 2026-01-21 22:53 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Chen Wang, Inochi Amaoto, sophgo,
Chen-Yu Tsai, Maxime Ripard, Jernej Skrabec, Samuel Holland,
linux-arm-kernel, linux-sunxi, Alexandre Belloni, linux-rtc,
Andreas Färber, Manivannan Sadhasivam, linux-actions,
Keguang Zhang, linux-mips, Taichi Sugaya, Takao Orito,
Jacky Huang, Shan-Chun Hung, Vladimir Zapolskiy,
Piotr Wojtaszczyk, Bjorn Andersson, linux-arm-msm, Orson Zhai,
Baolin Wang, Chunyan Zhang, Maxime Coquelin, Alexandre Torgue,
linux-stm32, Michal Simek, Rob Clark, Dmitry Baryshkov,
David Airlie, Simona Vetter, Abhinav Kumar, Jessica Zhang,
Sean Paul, Marijn Suijten, dri-devel, freedreno, Vinod Koul,
Neil Armstrong, linux-phy
Hi Stephen,
On Thu, Jan 08, 2026 at 04:16:18PM -0500, Brian Masney wrote:
> Here's a series that gets rid of the deprecated APIs
> divider_round_rate(), divider_round_rate_parent(), and
> divider_ro_round_rate_parent() since these functions are just wrappers
> for the determine_rate variant.
I sent you a GIT PULL for what can go to Linus for the upcoming merge
window from this series:
https://lore.kernel.org/linux-clk/aXFYU324yQ6uBmk0@redhat.com/T/#u
Thanks,
Brian
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: (subset) [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends
2026-01-08 21:16 [PATCH 00/27] clk: remove deprecated API divider_round_rate() and friends Brian Masney
` (3 preceding siblings ...)
2026-01-21 22:53 ` Brian Masney
@ 2026-02-04 15:44 ` Vinod Koul
4 siblings, 0 replies; 9+ messages in thread
From: Vinod Koul @ 2026-02-04 15:44 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Brian Masney
Cc: linux-clk, linux-kernel, Chen Wang, Inochi Amaoto, sophgo,
Chen-Yu Tsai, Maxime Ripard, Jernej Skrabec, Samuel Holland,
linux-arm-kernel, linux-sunxi, Alexandre Belloni, linux-rtc,
Andreas Färber, Manivannan Sadhasivam, linux-actions,
Keguang Zhang, linux-mips, Taichi Sugaya, Takao Orito,
Jacky Huang, Shan-Chun Hung, Vladimir Zapolskiy,
Piotr Wojtaszczyk, Bjorn Andersson, linux-arm-msm, Orson Zhai,
Baolin Wang, Chunyan Zhang, Maxime Coquelin, Alexandre Torgue,
linux-stm32, Michal Simek, Rob Clark, Dmitry Baryshkov,
David Airlie, Simona Vetter, Abhinav Kumar, Jessica Zhang,
Sean Paul, Marijn Suijten, dri-devel, freedreno, Neil Armstrong,
linux-phy
On Thu, 08 Jan 2026 16:16:18 -0500, Brian Masney wrote:
> Here's a series that gets rid of the deprecated APIs
> divider_round_rate(), divider_round_rate_parent(), and
> divider_ro_round_rate_parent() since these functions are just wrappers
> for the determine_rate variant.
>
> Note that when I converted some of these drivers from round_rate to
> determine_rate, this was mistakenly converted to the following in some
> cases:
>
> [...]
Applied, thanks!
[25/27] phy: ti: phy-j721e-wiz: convert from divider_round_rate() to divider_determine_rate()
commit: dbeea86fecef7cf2b93aded4525d74f6277376ef
Best regards,
--
~Vinod
^ permalink raw reply [flat|nested] 9+ messages in thread