From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: Sascha Hauer <s.hauer@pengutronix.de>,
Barebox List <barebox@lists.infradead.org>
Subject: Re: [PATCH 08/14] Add rational_best_approximation()
Date: Tue, 18 May 2021 14:23:18 +0200 [thread overview]
Message-ID: <893dc993-2191-da8f-b825-e472927babb7@pengutronix.de> (raw)
In-Reply-To: <20210518114929.12964-9-s.hauer@pengutronix.de>
On 18.05.21 13:49, Sascha Hauer wrote:
> Import rational_best_approximation() from Linux. This is used by an
> upcoming update of the clk_fractional_divider code.
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Reviewed-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
> include/linux/rational.h | 20 ++++++++
> lib/math/Makefile | 1 +
> lib/math/rational.c | 100 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 121 insertions(+)
> create mode 100644 include/linux/rational.h
> create mode 100644 lib/math/rational.c
>
> diff --git a/include/linux/rational.h b/include/linux/rational.h
> new file mode 100644
> index 0000000000..33f5f5fc3e
> --- /dev/null
> +++ b/include/linux/rational.h
> @@ -0,0 +1,20 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * rational fractions
> + *
> + * Copyright (C) 2009 emlix GmbH, Oskar Schirmer <oskar@scara.com>
> + *
> + * helper functions when coping with rational numbers,
> + * e.g. when calculating optimum numerator/denominator pairs for
> + * pll configuration taking into account restricted register size
> + */
> +
> +#ifndef _LINUX_RATIONAL_H
> +#define _LINUX_RATIONAL_H
> +
> +void rational_best_approximation(
> + unsigned long given_numerator, unsigned long given_denominator,
> + unsigned long max_numerator, unsigned long max_denominator,
> + unsigned long *best_numerator, unsigned long *best_denominator);
> +
> +#endif /* _LINUX_RATIONAL_H */
> diff --git a/lib/math/Makefile b/lib/math/Makefile
> index c2c892dd55..756d7dd90d 100644
> --- a/lib/math/Makefile
> +++ b/lib/math/Makefile
> @@ -1,2 +1,3 @@
> obj-y += div64.o
> pbl-y += div64.o
> +obj-y += rational.o
> diff --git a/lib/math/rational.c b/lib/math/rational.c
> new file mode 100644
> index 0000000000..e5367e6a8a
> --- /dev/null
> +++ b/lib/math/rational.c
> @@ -0,0 +1,100 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * rational fractions
> + *
> + * Copyright (C) 2009 emlix GmbH, Oskar Schirmer <oskar@scara.com>
> + * Copyright (C) 2019 Trent Piepho <tpiepho@gmail.com>
> + *
> + * helper functions when coping with rational numbers
> + */
> +
> +#include <linux/rational.h>
> +#include <linux/compiler.h>
> +#include <linux/export.h>
> +#include <linux/kernel.h>
> +
> +/*
> + * calculate best rational approximation for a given fraction
> + * taking into account restricted register size, e.g. to find
> + * appropriate values for a pll with 5 bit denominator and
> + * 8 bit numerator register fields, trying to set up with a
> + * frequency ratio of 3.1415, one would say:
> + *
> + * rational_best_approximation(31415, 10000,
> + * (1 << 8) - 1, (1 << 5) - 1, &n, &d);
> + *
> + * you may look at given_numerator as a fixed point number,
> + * with the fractional part size described in given_denominator.
> + *
> + * for theoretical background, see:
> + * https://en.wikipedia.org/wiki/Continued_fraction
> + */
> +
> +void rational_best_approximation(
> + unsigned long given_numerator, unsigned long given_denominator,
> + unsigned long max_numerator, unsigned long max_denominator,
> + unsigned long *best_numerator, unsigned long *best_denominator)
> +{
> + /* n/d is the starting rational, which is continually
> + * decreased each iteration using the Euclidean algorithm.
> + *
> + * dp is the value of d from the prior iteration.
> + *
> + * n2/d2, n1/d1, and n0/d0 are our successively more accurate
> + * approximations of the rational. They are, respectively,
> + * the current, previous, and two prior iterations of it.
> + *
> + * a is current term of the continued fraction.
> + */
> + unsigned long n, d, n0, d0, n1, d1, n2, d2;
> + n = given_numerator;
> + d = given_denominator;
> + n0 = d1 = 0;
> + n1 = d0 = 1;
> +
> + for (;;) {
> + unsigned long dp, a;
> +
> + if (d == 0)
> + break;
> + /* Find next term in continued fraction, 'a', via
> + * Euclidean algorithm.
> + */
> + dp = d;
> + a = n / d;
> + d = n % d;
> + n = dp;
> +
> + /* Calculate the current rational approximation (aka
> + * convergent), n2/d2, using the term just found and
> + * the two prior approximations.
> + */
> + n2 = n0 + a * n1;
> + d2 = d0 + a * d1;
> +
> + /* If the current convergent exceeds the maxes, then
> + * return either the previous convergent or the
> + * largest semi-convergent, the final term of which is
> + * found below as 't'.
> + */
> + if ((n2 > max_numerator) || (d2 > max_denominator)) {
> + unsigned long t = min((max_numerator - n0) / n1,
> + (max_denominator - d0) / d1);
> +
> + /* This tests if the semi-convergent is closer
> + * than the previous convergent.
> + */
> + if (2u * t > a || (2u * t == a && d0 * dp > d1 * d)) {
> + n1 = n0 + t * n1;
> + d1 = d0 + t * d1;
> + }
> + break;
> + }
> + n0 = n1;
> + n1 = n2;
> + d0 = d1;
> + d1 = d2;
> + }
> + *best_numerator = n1;
> + *best_denominator = d1;
> +}
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2021-05-18 12:24 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-18 11:49 [PATCH 00/14] clk updates Sascha Hauer
2021-05-18 11:49 ` [PATCH 01/14] clk: introduce clk init op Sascha Hauer
2021-05-18 11:57 ` Ahmad Fatoum
2021-05-18 11:49 ` [PATCH 02/14] clk: rename clk_register() to bclk_register() Sascha Hauer
2021-05-18 12:06 ` Ahmad Fatoum
2021-05-18 11:49 ` [PATCH 03/14] clk: introduce struct clk_hw Sascha Hauer
2021-05-18 12:17 ` Ahmad Fatoum
2021-05-19 6:55 ` Sascha Hauer
2021-05-18 11:49 ` [PATCH 04/14] clk: introduce clk_register() Sascha Hauer
2021-05-18 12:21 ` Ahmad Fatoum
2021-05-18 11:49 ` [PATCH 05/14] clk: divider: Add ro ops Sascha Hauer
2021-05-18 12:21 ` Ahmad Fatoum
2021-05-18 11:49 ` [PATCH 06/14] clk: mux: " Sascha Hauer
2021-05-18 11:49 ` [PATCH 07/14] clk: move fixed_factor to include/linux/clk.h Sascha Hauer
2021-05-18 11:49 ` [PATCH 08/14] Add rational_best_approximation() Sascha Hauer
2021-05-18 12:23 ` Ahmad Fatoum [this message]
2021-05-18 11:49 ` [PATCH 09/14] clk: Update fractional divider from Linux Sascha Hauer
2021-05-18 11:49 ` [PATCH 10/14] clk: Add lock to different clock types Sascha Hauer
2021-05-18 11:49 ` [PATCH 11/14] clk: Add Linux functions to register a divider Sascha Hauer
2021-05-18 11:49 ` [PATCH 12/14] clk: Add Linux functions to register a fixed factor clock Sascha Hauer
2021-05-18 11:49 ` [PATCH 13/14] clk: Add Linux functions to register a gate Sascha Hauer
2021-05-18 11:49 ` [PATCH 14/14] clk: Add Linux functions to register a mux Sascha Hauer
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=893dc993-2191-da8f-b825-e472927babb7@pengutronix.de \
--to=a.fatoum@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox