From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Hans Verkuil <hverkuil+cisco@kernel.org>,
linux-iio@vger.kernel.org, chrome-platform@lists.linux.dev,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org
Cc: "Jonathan Cameron" <jic23@kernel.org>,
"David Lechner" <dlechner@baylibre.com>,
"Nuno Sá" <nuno.sa@analog.com>,
"Andy Shevchenko" <andy@kernel.org>,
"Benson Leung" <bleung@chromium.org>,
"Guenter Roeck" <groeck@chromium.org>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Michael Hennerich" <Michael.Hennerich@analog.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
Subject: [PATCH v2 0/6] iio: Introduce and use value of π
Date: Fri, 7 Nov 2025 21:02:58 +0100 [thread overview]
Message-ID: <20251107201005.3156118-1-andriy.shevchenko@linux.intel.com> (raw)
There are a few drivers that use value of π (small Greek PI)
of different precision. Instead of hard coding over and over
convert them to use a defined constant, which one of the patches
in this series introduces. No functional changes involved.
Note, the respective IIO macros are not converted yet by this series.
All of the converted files were checked in disassembled format before
and after the change to see that the value:
a) pre-calculated at compile-time;
b) is the same that is expected.
I didn't put the power-of-10 into the definition names as now they are
the maximum that fits the respective signed type.
Series is supposed to go via IIO tree.
Changelog v2:
- redone to avoid unneeded calculations
- collected tags (Hans)
v1: <20251027143850.2070427-1-andriy.shevchenko@linux.intel.com>
Andy Shevchenko (6):
media: vidtv: Rename PI definition to PI_SAMPLES
units: Add 32- and 64-bit signed values of π
media: dvb-frontends: atbm8830: Convert to use PI definition
iio: cros_ec_sensors: Convert to use PI definition
iio: frequency: ad9523: Convert to use PI definition
iio: position: iqs624-pos: Convert to use PI definition
drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c | 3 ++-
drivers/iio/frequency/ad9523.c | 5 +++--
drivers/iio/position/iqs624-pos.c | 3 ++-
drivers/media/dvb-frontends/atbm8830.c | 5 +++--
drivers/media/test-drivers/vidtv/vidtv_s302m.c | 6 +++---
include/linux/units.h | 6 ++++++
6 files changed, 19 insertions(+), 9 deletions(-)
--
2.50.1
next reply other threads:[~2025-11-07 20:10 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-07 20:02 Andy Shevchenko [this message]
2025-11-07 20:02 ` [PATCH v2 1/6] media: vidtv: Rename PI definition to PI_SAMPLES Andy Shevchenko
2025-11-07 20:03 ` [PATCH v2 2/6] units: Add 32- and 64-bit signed values of π Andy Shevchenko
2025-11-07 21:09 ` David Lechner
2025-11-07 21:14 ` David Lechner
[not found] ` <CAHp75VexvdYyjt1GkbfqOotkFpLeb=io6outJ5dpRqBv2qPNng@mail.gmail.com>
2025-11-09 17:00 ` Jonathan Cameron
2025-11-09 17:09 ` Andy Shevchenko
2025-11-07 20:03 ` [PATCH v2 3/6] media: dvb-frontends: atbm8830: Convert to use PI definition Andy Shevchenko
2025-11-07 20:03 ` [PATCH v2 4/6] iio: cros_ec_sensors: " Andy Shevchenko
2025-11-07 20:03 ` [PATCH v2 5/6] iio: frequency: ad9523: " Andy Shevchenko
2025-11-07 20:03 ` [PATCH v2 6/6] iio: position: iqs624-pos: " Andy Shevchenko
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=20251107201005.3156118-1-andriy.shevchenko@linux.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=Michael.Hennerich@analog.com \
--cc=andy@kernel.org \
--cc=bleung@chromium.org \
--cc=chrome-platform@lists.linux.dev \
--cc=dlechner@baylibre.com \
--cc=dwlsalmeida@gmail.com \
--cc=groeck@chromium.org \
--cc=hverkuil+cisco@kernel.org \
--cc=jic23@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=nuno.sa@analog.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 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).