* [PATCH 0/4] Introduce clk_find_nearest_rate()
@ 2014-06-30 16:56 Soren Brinkmann
2014-06-30 16:56 ` [PATCH 3/4] net: macb: Use clk_find_nearest_rate() API Soren Brinkmann
0 siblings, 1 reply; 2+ messages in thread
From: Soren Brinkmann @ 2014-06-30 16:56 UTC (permalink / raw)
To: Mike Turquette, Rafael J. Wysocki, Viresh Kumar
Cc: Russell King, Michal Simek, Nicolas Ferre, Uwe Kleine-König,
linux-pm-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
cpufreq-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Soren Brinkmann, Ian Campbell, netdev-u79uwXL29TY76Z2rM5mHXA,
Kumar Gala, devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring,
Pawel Moll, Mark Rutland
On Zynq I encountered issues due to rounding here and there. Often the
issue would have been resolved by rounding towards the requested
frequency. Unfortunately, the CCF does not specify the behavior of
clk_round_rate() in terms of rounding, making this proposed API
call useful for certain use-cases.
An RFC of this series has been discussed here:
https://lkml.org/lkml/2014/5/14/694
Mike: IIRC, you wanted to fix the return type of __clk_round_rate(). Do
you have any patches for that already? I couldn't find anything in
clk-next.
Patch 1 adds the new API call.
2 and 3 apply the new API.
And 4 is a minor fixup of the Zynq DT. The frequencies in the Zynq OPPs
had been specified taking this rounding issues into account, which
is no longer required with this patchset.
Thanks,
Sören
Soren Brinkmann (4):
clk: Introduce 'clk_find_nearest_rate()'
cpufreq: cpu0: Use clk_find_nearest_rate()
net: macb: Use clk_find_nearest_rate() API
ARM: zynq: dt: Use properly rounded frequencies in OPPs
arch/arm/boot/dts/zynq-7000.dtsi | 4 +--
drivers/clk/clk.c | 57 +++++++++++++++++++++++++++++++++++++
drivers/cpufreq/cpufreq-cpu0.c | 3 +-
drivers/net/ethernet/cadence/macb.c | 2 +-
include/linux/clk.h | 9 ++++++
5 files changed, 71 insertions(+), 4 deletions(-)
--
2.0.1.1.gfbfc394
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 3/4] net: macb: Use clk_find_nearest_rate() API
2014-06-30 16:56 [PATCH 0/4] Introduce clk_find_nearest_rate() Soren Brinkmann
@ 2014-06-30 16:56 ` Soren Brinkmann
0 siblings, 0 replies; 2+ messages in thread
From: Soren Brinkmann @ 2014-06-30 16:56 UTC (permalink / raw)
To: Mike Turquette, Rafael J. Wysocki, Viresh Kumar
Cc: Russell King, Michal Simek, Nicolas Ferre, Uwe Kleine-König,
linux-pm, linux-kernel, cpufreq, linux-arm-kernel,
Soren Brinkmann, netdev
The Ethernet clock has to match the specified frequencies as accurately
as possible. clk_round_rate() does not specify how rounding is
implemented. Hence use clk_find_nearest_rate().
Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
---
drivers/net/ethernet/cadence/macb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/cadence/macb.c b/drivers/net/ethernet/cadence/macb.c
index e9daa072ebb4..7b7f5eb1b341 100644
--- a/drivers/net/ethernet/cadence/macb.c
+++ b/drivers/net/ethernet/cadence/macb.c
@@ -223,7 +223,7 @@ static void macb_set_tx_clk(struct clk *clk, int speed, struct net_device *dev)
return;
}
- rate_rounded = clk_round_rate(clk, rate);
+ rate_rounded = clk_find_nearest_rate(clk, rate);
if (rate_rounded < 0)
return;
--
2.0.1.1.gfbfc394
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-06-30 16:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-30 16:56 [PATCH 0/4] Introduce clk_find_nearest_rate() Soren Brinkmann
2014-06-30 16:56 ` [PATCH 3/4] net: macb: Use clk_find_nearest_rate() API Soren Brinkmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).