[PATCH v2 0/3] lib: polynomial: Move to math/ and clean up

[PATCH v2 0/3] lib: polynomial: Move to math/ and clean up

[Andy Shevchenko]

While removing Baikal SoC and platform code pieces I found that this
code belongs to lib/math/ rather than generic lib/. Hence the move and
followed up cleanups.

No functional changes intended.

Changelog v2:
- added missing math.h (Kuan-Wei)
- amended the first sentence in the top comment (Randy)
- collected tags (Randy, Kuan-Wei)

v1: 20260227074957.1144321-1-andriy.shevchenko@linux.intel.com

Andy Shevchenko (3):
  lib: polynomial: Move to math/ subfolder
  lib: math: polynomial: Don't use 'proxy' headers
  lib: math: polynomial: Remove link to non-exist file and fix spelling

 lib/Kconfig                 |  3 ---
 lib/Makefile                |  2 --
 lib/math/Kconfig            |  3 +++
 lib/math/Makefile           |  1 +
 lib/{ => math}/polynomial.c | 23 ++++++++++-------------
 5 files changed, 14 insertions(+), 18 deletions(-)
 rename lib/{ => math}/polynomial.c (88%)

-- 
2.50.1

[PATCH v2 1/3] lib: polynomial: Move to math/ subfolder

[Andy Shevchenko]

The algorithm behind polynomial belongs to our collection of
math equations and expressions handling. Move it to math/
subfolder where others of the kind are located.

Reviewed-by: Kuan-Wei Chiu <visitorckw@gmail.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 lib/Kconfig                 | 3 ---
 lib/Makefile                | 2 --
 lib/math/Kconfig            | 3 +++
 lib/math/Makefile           | 1 +
 lib/{ => math}/polynomial.c | 0
 5 files changed, 4 insertions(+), 5 deletions(-)
 rename lib/{ => math}/polynomial.c (100%)

diff --git a/lib/Kconfig b/lib/Kconfig
index 5be57adcd454..00a9509636c1 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -626,9 +626,6 @@ config PLDMFW
 config ASN1_ENCODER
        tristate
 
-config POLYNOMIAL
-       tristate
-
 config FIRMWARE_TABLE
 	bool
 
diff --git a/lib/Makefile b/lib/Makefile
index 84da412a044f..9ea149e618d2 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -243,8 +243,6 @@ obj-$(CONFIG_MEMREGION) += memregion.o
 obj-$(CONFIG_STMP_DEVICE) += stmp_device.o
 obj-$(CONFIG_IRQ_POLL) += irq_poll.o
 
-obj-$(CONFIG_POLYNOMIAL) += polynomial.o
-
 # stackdepot.c should not be instrumented or call instrumented functions.
 # Prevent the compiler from calling builtins like memcmp() or bcmp() from this
 # file.
diff --git a/lib/math/Kconfig b/lib/math/Kconfig
index 0634b428d0cb..0e6d9cffc5d6 100644
--- a/lib/math/Kconfig
+++ b/lib/math/Kconfig
@@ -5,6 +5,9 @@ config CORDIC
 	  This option provides an implementation of the CORDIC algorithm;
 	  calculations are in fixed point. Module will be called cordic.
 
+config POLYNOMIAL
+       tristate
+
 config PRIME_NUMBERS
 	tristate "Simple prime number generator for testing"
 	help
diff --git a/lib/math/Makefile b/lib/math/Makefile
index d1caba23baa0..9a3850d55b79 100644
--- a/lib/math/Makefile
+++ b/lib/math/Makefile
@@ -2,6 +2,7 @@
 obj-y += div64.o gcd.o lcm.o int_log.o int_pow.o int_sqrt.o reciprocal_div.o
 
 obj-$(CONFIG_CORDIC)		+= cordic.o
+obj-$(CONFIG_POLYNOMIAL)	+= polynomial.o
 obj-$(CONFIG_PRIME_NUMBERS)	+= prime_numbers.o
 obj-$(CONFIG_RATIONAL)		+= rational.o
 
diff --git a/lib/polynomial.c b/lib/math/polynomial.c
similarity index 100%
rename from lib/polynomial.c
rename to lib/math/polynomial.c
-- 
2.50.1

[PATCH v2 2/3] lib: math: polynomial: Don't use 'proxy' headers

[Andy Shevchenko]

Update header inclusions to follow IWYU (Include What You Use)
principle.

Reviewed-by: Kuan-Wei Chiu <visitorckw@gmail.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 lib/math/polynomial.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/math/polynomial.c b/lib/math/polynomial.c
index 66d383445fec..8c622099ef81 100644
--- a/lib/math/polynomial.c
+++ b/lib/math/polynomial.c
@@ -10,7 +10,8 @@
  *
  */
 
-#include <linux/kernel.h>
+#include <linux/export.h>
+#include <linux/math.h>
 #include <linux/module.h>
 #include <linux/polynomial.h>
 
-- 
2.50.1

[PATCH v2 3/3] lib: math: polynomial: Remove link to non-exist file and fix spelling

[Andy Shevchenko]

The Baikal SoC and platform support was dropped from the kernel,
remove the reference to non-exist file. While at it, fix spelling.

Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
Reviewed-by: Kuan-Wei Chiu <visitorckw@gmail.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 lib/math/polynomial.c | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/lib/math/polynomial.c b/lib/math/polynomial.c
index 8c622099ef81..f26677cfeeff 100644
--- a/lib/math/polynomial.c
+++ b/lib/math/polynomial.c
@@ -16,16 +16,13 @@
 #include <linux/polynomial.h>
 
 /*
- * Originally this was part of drivers/hwmon/bt1-pvt.c.
- * There the following conversion is used and should serve as an example here:
+ * The following conversion is an example:
  *
  * The original translation formulae of the temperature (in degrees of Celsius)
  * to PVT data and vice-versa are following:
  *
- * N = 1.8322e-8*(T^4) + 2.343e-5*(T^3) + 8.7018e-3*(T^2) + 3.9269*(T^1) +
- *     1.7204e2
- * T = -1.6743e-11*(N^4) + 8.1542e-8*(N^3) + -1.8201e-4*(N^2) +
- *     3.1020e-1*(N^1) - 4.838e1
+ * N = 1.8322e-8*(T^4) + 2.343e-5*(T^3) + 8.7018e-3*(T^2) + 3.9269*(T^1) + 1.7204e2
+ * T = -1.6743e-11*(N^4) + 8.1542e-8*(N^3) + -1.8201e-4*(N^2) + 3.1020e-1*(N^1) - 4.838e1
  *
  * where T = [-48.380, 147.438]C and N = [0, 1023].
  *
@@ -36,10 +33,9 @@
  * formulae to accept millidegrees of Celsius. Here what they look like after
  * the alterations:
  *
- * N = (18322e-20*(T^4) + 2343e-13*(T^3) + 87018e-9*(T^2) + 39269e-3*T +
- *     17204e2) / 1e4
- * T = -16743e-12*(D^4) + 81542e-9*(D^3) - 182010e-6*(D^2) + 310200e-3*D -
- *     48380
+ * N = (18322e-20*(T^4) + 2343e-13*(T^3) + 87018e-9*(T^2) + 39269e-3*T + 17204e2) / 1e4
+ * T = -16743e-12*(D^4) + 81542e-9*(D^3) - 182010e-6*(D^2) + 310200e-3*D - 48380
+ *
  * where T = [-48380, 147438] mC and N = [0, 1023].
  *
  * static const struct polynomial poly_temp_to_N = {
@@ -69,13 +65,13 @@
  * polynomial_calc - calculate a polynomial using integer arithmetic
  *
  * @poly: pointer to the descriptor of the polynomial
- * @data: input value of the polynimal
+ * @data: input value of the polynomial
  *
  * Calculate the result of a polynomial using only integer arithmetic. For
  * this to work without too much loss of precision the coefficients has to
  * be altered. This is called factor redistribution.
  *
- * Returns the result of the polynomial calculation.
+ * Return: the result of the polynomial calculation.
  */
 long polynomial_calc(const struct polynomial *poly, long data)
 {
-- 
2.50.1