Em Mon, 19 Jun 2023 20:20:16 +0300 Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> escreveu: > Some existing and new users may benefit from the intlog2() and > intlog10() APIs, make them wide available. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > Documentation/driver-api/media/dtv-common.rst | 2 +- > drivers/media/dvb-core/Makefile | 2 +- > drivers/media/dvb-frontends/af9013_priv.h | 2 +- > drivers/media/dvb-frontends/af9033_priv.h | 2 +- > drivers/media/dvb-frontends/cxd2820r_priv.h | 2 +- > drivers/media/dvb-frontends/cxd2841er.c | 2 +- > .../dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt2_mon.c | 2 +- > .../dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt_mon.c | 2 +- > drivers/media/dvb-frontends/cxd2880/cxd2880_top.c | 2 +- > drivers/media/dvb-frontends/dib7000p.c | 2 +- > drivers/media/dvb-frontends/dib8000.c | 2 +- > drivers/media/dvb-frontends/dib9000.c | 2 +- > drivers/media/dvb-frontends/drxk_hard.c | 2 +- > drivers/media/dvb-frontends/lgdt3305.c | 2 +- > drivers/media/dvb-frontends/lgdt3306a.c | 2 +- > drivers/media/dvb-frontends/lgdt330x.c | 2 +- > drivers/media/dvb-frontends/m88ds3103_priv.h | 2 +- > drivers/media/dvb-frontends/mn88443x.c | 2 +- > drivers/media/dvb-frontends/mn88472_priv.h | 2 +- > drivers/media/dvb-frontends/mn88473_priv.h | 2 +- > drivers/media/dvb-frontends/or51132.c | 2 +- > drivers/media/dvb-frontends/or51211.c | 2 +- > drivers/media/dvb-frontends/rtl2830_priv.h | 2 +- > drivers/media/dvb-frontends/rtl2832_priv.h | 2 +- > drivers/media/dvb-frontends/si2165.c | 2 +- > drivers/media/dvb-frontends/stv0367.c | 2 +- > drivers/media/dvb-frontends/tc90522.c | 2 +- > drivers/media/dvb-frontends/tda10048.c | 2 +- > include/{media/dvb_math.h => linux/int_log.h} | 7 +++---- > lib/math/Makefile | 2 +- > .../media/dvb-core/dvb_math.c => lib/math/int_log.c | 11 ++++++----- > 31 files changed, 38 insertions(+), 38 deletions(-) > rename include/{media/dvb_math.h => linux/int_log.h} (92%) > rename drivers/media/dvb-core/dvb_math.c => lib/math/int_log.c (95%) > > diff --git a/Documentation/driver-api/media/dtv-common.rst b/Documentation/driver-api/media/dtv-common.rst > index f8b2c4dc8170..cf234e0e101a 100644 > --- a/Documentation/driver-api/media/dtv-common.rst > +++ b/Documentation/driver-api/media/dtv-common.rst > @@ -9,7 +9,7 @@ Math functions > Provide some commonly-used math functions, usually required in order to > estimate signal strength and signal to noise measurements in dB. > > -.. kernel-doc:: include/media/dvb_math.h > +.. kernel-doc:: include/linux/int_log.h Hmm... maybe the best would be to move the kernel-doc tag to be on a more generic place, as this will now be used outside media, and we don't want to risk having multiple kernel-doc entries to the same header - as it breaks cross-references. After fixed: Reviewed-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > > > DVB devices > diff --git a/drivers/media/dvb-core/Makefile b/drivers/media/dvb-core/Makefile > index 62b028ded9f7..1cb3ca67bed9 100644 > --- a/drivers/media/dvb-core/Makefile > +++ b/drivers/media/dvb-core/Makefile > @@ -8,6 +8,6 @@ dvb-vb2-$(CONFIG_DVB_MMAP) := dvb_vb2.o > > dvb-core-objs := dvbdev.o dmxdev.o dvb_demux.o \ > dvb_ca_en50221.o dvb_frontend.o \ > - $(dvb-net-y) dvb_ringbuffer.o $(dvb-vb2-y) dvb_math.o > + $(dvb-net-y) dvb_ringbuffer.o $(dvb-vb2-y) > > obj-$(CONFIG_DVB_CORE) += dvb-core.o > diff --git a/drivers/media/dvb-frontends/af9013_priv.h b/drivers/media/dvb-frontends/af9013_priv.h > index 3b9b9424fe1a..bba7a9693a23 100644 > --- a/drivers/media/dvb-frontends/af9013_priv.h > +++ b/drivers/media/dvb-frontends/af9013_priv.h > @@ -12,7 +12,7 @@ > #define AF9013_PRIV_H > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "af9013.h" > #include <linux/firmware.h> > #include <linux/i2c-mux.h> > diff --git a/drivers/media/dvb-frontends/af9033_priv.h b/drivers/media/dvb-frontends/af9033_priv.h > index 0e64da0cdeab..7560da75ef00 100644 > --- a/drivers/media/dvb-frontends/af9033_priv.h > +++ b/drivers/media/dvb-frontends/af9033_priv.h > @@ -14,7 +14,7 @@ > #include <linux/math64.h> > #include <linux/regmap.h> > #include <linux/kernel.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > struct reg_val { > u32 reg; > diff --git a/drivers/media/dvb-frontends/cxd2820r_priv.h b/drivers/media/dvb-frontends/cxd2820r_priv.h > index 9b4d9cf8563d..605320bbc12b 100644 > --- a/drivers/media/dvb-frontends/cxd2820r_priv.h > +++ b/drivers/media/dvb-frontends/cxd2820r_priv.h > @@ -11,7 +11,7 @@ > > #include <linux/dvb/version.h> > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "cxd2820r.h" > #include <linux/gpio/driver.h> /* For gpio_chip */ > #include <linux/math64.h> > diff --git a/drivers/media/dvb-frontends/cxd2841er.c b/drivers/media/dvb-frontends/cxd2841er.c > index 5431f922f55e..ef403a9fb753 100644 > --- a/drivers/media/dvb-frontends/cxd2841er.c > +++ b/drivers/media/dvb-frontends/cxd2841er.c > @@ -22,7 +22,7 @@ > #include <linux/dynamic_debug.h> > #include <linux/kernel.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include <media/dvb_frontend.h> > #include "cxd2841er.h" > #include "cxd2841er_priv.h" > diff --git a/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt2_mon.c b/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt2_mon.c > index 604580bf7cf7..4e173dd87ecf 100644 > --- a/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt2_mon.c > +++ b/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt2_mon.c > @@ -11,7 +11,7 @@ > #include "cxd2880_tnrdmd_dvbt2.h" > #include "cxd2880_tnrdmd_dvbt2_mon.h" > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > static const int ref_dbm_1000[4][8] = { > {-96000, -95000, -94000, -93000, -92000, -92000, -98000, -97000}, > diff --git a/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt_mon.c b/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt_mon.c > index fedc3b4a2fa0..86d5a1e4022a 100644 > --- a/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt_mon.c > +++ b/drivers/media/dvb-frontends/cxd2880/cxd2880_tnrdmd_dvbt_mon.c > @@ -11,7 +11,7 @@ > #include "cxd2880_tnrdmd_dvbt.h" > #include "cxd2880_tnrdmd_dvbt_mon.h" > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > static const int ref_dbm_1000[3][5] = { > {-93000, -91000, -90000, -89000, -88000}, > diff --git a/drivers/media/dvb-frontends/cxd2880/cxd2880_top.c b/drivers/media/dvb-frontends/cxd2880/cxd2880_top.c > index d5b1b3788e39..f67b6d24b8d4 100644 > --- a/drivers/media/dvb-frontends/cxd2880/cxd2880_top.c > +++ b/drivers/media/dvb-frontends/cxd2880/cxd2880_top.c > @@ -11,7 +11,7 @@ > #include <linux/spi/spi.h> > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > #include "cxd2880.h" > #include "cxd2880_tnrdmd_mon.h" > diff --git a/drivers/media/dvb-frontends/dib7000p.c b/drivers/media/dvb-frontends/dib7000p.c > index a90d2f51868f..b791e687d2e2 100644 > --- a/drivers/media/dvb-frontends/dib7000p.c > +++ b/drivers/media/dvb-frontends/dib7000p.c > @@ -13,7 +13,7 @@ > #include <linux/mutex.h> > #include <asm/div64.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include <media/dvb_frontend.h> > > #include "dib7000p.h" > diff --git a/drivers/media/dvb-frontends/dib8000.c b/drivers/media/dvb-frontends/dib8000.c > index fe19d127abb3..2abda7d1cb6e 100644 > --- a/drivers/media/dvb-frontends/dib8000.c > +++ b/drivers/media/dvb-frontends/dib8000.c > @@ -13,7 +13,7 @@ > #include <linux/mutex.h> > #include <asm/div64.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > #include <media/dvb_frontend.h> > > diff --git a/drivers/media/dvb-frontends/dib9000.c b/drivers/media/dvb-frontends/dib9000.c > index 914ca820c174..1c57587a917a 100644 > --- a/drivers/media/dvb-frontends/dib9000.c > +++ b/drivers/media/dvb-frontends/dib9000.c > @@ -11,7 +11,7 @@ > #include <linux/i2c.h> > #include <linux/mutex.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include <media/dvb_frontend.h> > > #include "dib9000.h" > diff --git a/drivers/media/dvb-frontends/drxk_hard.c b/drivers/media/dvb-frontends/drxk_hard.c > index 3301ef75d441..6ad4f202f1bf 100644 > --- a/drivers/media/dvb-frontends/drxk_hard.c > +++ b/drivers/media/dvb-frontends/drxk_hard.c > @@ -20,7 +20,7 @@ > #include <media/dvb_frontend.h> > #include "drxk.h" > #include "drxk_hard.h" > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > static int power_down_dvbt(struct drxk_state *state, bool set_power_mode); > static int power_down_qam(struct drxk_state *state); > diff --git a/drivers/media/dvb-frontends/lgdt3305.c b/drivers/media/dvb-frontends/lgdt3305.c > index 62d743988919..c15d3735d34c 100644 > --- a/drivers/media/dvb-frontends/lgdt3305.c > +++ b/drivers/media/dvb-frontends/lgdt3305.c > @@ -10,7 +10,7 @@ > #include <asm/div64.h> > #include <linux/dvb/frontend.h> > #include <linux/slab.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "lgdt3305.h" > > static int debug; > diff --git a/drivers/media/dvb-frontends/lgdt3306a.c b/drivers/media/dvb-frontends/lgdt3306a.c > index 70258884126b..3c6650f6e9a3 100644 > --- a/drivers/media/dvb-frontends/lgdt3306a.c > +++ b/drivers/media/dvb-frontends/lgdt3306a.c > @@ -12,7 +12,7 @@ > #include <asm/div64.h> > #include <linux/kernel.h> > #include <linux/dvb/frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "lgdt3306a.h" > #include <linux/i2c-mux.h> > > diff --git a/drivers/media/dvb-frontends/lgdt330x.c b/drivers/media/dvb-frontends/lgdt330x.c > index 83565209c3b1..97a10996c7fa 100644 > --- a/drivers/media/dvb-frontends/lgdt330x.c > +++ b/drivers/media/dvb-frontends/lgdt330x.c > @@ -28,7 +28,7 @@ > #include <asm/byteorder.h> > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "lgdt330x_priv.h" > #include "lgdt330x.h" > > diff --git a/drivers/media/dvb-frontends/m88ds3103_priv.h b/drivers/media/dvb-frontends/m88ds3103_priv.h > index aa5306f40201..594ad9cbc2cc 100644 > --- a/drivers/media/dvb-frontends/m88ds3103_priv.h > +++ b/drivers/media/dvb-frontends/m88ds3103_priv.h > @@ -10,7 +10,7 @@ > > #include <media/dvb_frontend.h> > #include "m88ds3103.h" > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include <linux/firmware.h> > #include <linux/i2c-mux.h> > #include <linux/regmap.h> > diff --git a/drivers/media/dvb-frontends/mn88443x.c b/drivers/media/dvb-frontends/mn88443x.c > index 2ce5692bc22c..db2921c736af 100644 > --- a/drivers/media/dvb-frontends/mn88443x.c > +++ b/drivers/media/dvb-frontends/mn88443x.c > @@ -10,7 +10,7 @@ > #include <linux/gpio/consumer.h> > #include <linux/of_device.h> > #include <linux/regmap.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > #include "mn88443x.h" > > diff --git a/drivers/media/dvb-frontends/mn88472_priv.h b/drivers/media/dvb-frontends/mn88472_priv.h > index 337562723f88..41f14bd67bfd 100644 > --- a/drivers/media/dvb-frontends/mn88472_priv.h > +++ b/drivers/media/dvb-frontends/mn88472_priv.h > @@ -9,7 +9,7 @@ > #define MN88472_PRIV_H > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "mn88472.h" > #include <linux/firmware.h> > #include <linux/regmap.h> > diff --git a/drivers/media/dvb-frontends/mn88473_priv.h b/drivers/media/dvb-frontends/mn88473_priv.h > index eca7f4e2b769..e9daaacfa22f 100644 > --- a/drivers/media/dvb-frontends/mn88473_priv.h > +++ b/drivers/media/dvb-frontends/mn88473_priv.h > @@ -9,7 +9,7 @@ > #define MN88473_PRIV_H > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "mn88473.h" > #include <linux/math64.h> > #include <linux/firmware.h> > diff --git a/drivers/media/dvb-frontends/or51132.c b/drivers/media/dvb-frontends/or51132.c > index 24de1b115158..355f3598627b 100644 > --- a/drivers/media/dvb-frontends/or51132.c > +++ b/drivers/media/dvb-frontends/or51132.c > @@ -27,7 +27,7 @@ > #include <linux/slab.h> > #include <asm/byteorder.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include <media/dvb_frontend.h> > #include "or51132.h" > > diff --git a/drivers/media/dvb-frontends/or51211.c b/drivers/media/dvb-frontends/or51211.c > index ddcaea5c9941..ae732dc5116e 100644 > --- a/drivers/media/dvb-frontends/or51211.c > +++ b/drivers/media/dvb-frontends/or51211.c > @@ -26,7 +26,7 @@ > #include <linux/slab.h> > #include <asm/byteorder.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include <media/dvb_frontend.h> > #include "or51211.h" > > diff --git a/drivers/media/dvb-frontends/rtl2830_priv.h b/drivers/media/dvb-frontends/rtl2830_priv.h > index fae78ed78522..ae1fc24a4d84 100644 > --- a/drivers/media/dvb-frontends/rtl2830_priv.h > +++ b/drivers/media/dvb-frontends/rtl2830_priv.h > @@ -9,7 +9,7 @@ > #define RTL2830_PRIV_H > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "rtl2830.h" > #include <linux/i2c-mux.h> > #include <linux/math64.h> > diff --git a/drivers/media/dvb-frontends/rtl2832_priv.h b/drivers/media/dvb-frontends/rtl2832_priv.h > index 5f79f95b9475..f11ba038d5f0 100644 > --- a/drivers/media/dvb-frontends/rtl2832_priv.h > +++ b/drivers/media/dvb-frontends/rtl2832_priv.h > @@ -14,7 +14,7 @@ > #include <linux/bitops.h> > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "rtl2832.h" > > struct rtl2832_dev { > diff --git a/drivers/media/dvb-frontends/si2165.c b/drivers/media/dvb-frontends/si2165.c > index 72810efd1a96..434d003bf397 100644 > --- a/drivers/media/dvb-frontends/si2165.c > +++ b/drivers/media/dvb-frontends/si2165.c > @@ -19,7 +19,7 @@ > #include <linux/regmap.h> > > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "si2165_priv.h" > #include "si2165.h" > > diff --git a/drivers/media/dvb-frontends/stv0367.c b/drivers/media/dvb-frontends/stv0367.c > index 95e376f23506..a93f40617469 100644 > --- a/drivers/media/dvb-frontends/stv0367.c > +++ b/drivers/media/dvb-frontends/stv0367.c > @@ -15,7 +15,7 @@ > #include <linux/slab.h> > #include <linux/i2c.h> > > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > > #include "stv0367.h" > #include "stv0367_defs.h" > diff --git a/drivers/media/dvb-frontends/tc90522.c b/drivers/media/dvb-frontends/tc90522.c > index 879f028f9682..1f8cbf45554a 100644 > --- a/drivers/media/dvb-frontends/tc90522.c > +++ b/drivers/media/dvb-frontends/tc90522.c > @@ -21,7 +21,7 @@ > #include <linux/kernel.h> > #include <linux/math64.h> > #include <linux/dvb/frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "tc90522.h" > > #define TC90522_I2C_THRU_REG 0xfe > diff --git a/drivers/media/dvb-frontends/tda10048.c b/drivers/media/dvb-frontends/tda10048.c > index 0b3f6999515e..3cb4e5270e4f 100644 > --- a/drivers/media/dvb-frontends/tda10048.c > +++ b/drivers/media/dvb-frontends/tda10048.c > @@ -16,7 +16,7 @@ > #include <linux/math64.h> > #include <asm/div64.h> > #include <media/dvb_frontend.h> > -#include <media/dvb_math.h> > +#include <linux/int_log.h> > #include "tda10048.h" > > #define TDA10048_DEFAULT_FIRMWARE "dvb-fe-tda10048-1.0.fw" > diff --git a/include/media/dvb_math.h b/include/linux/int_log.h > similarity index 92% > rename from include/media/dvb_math.h > rename to include/linux/int_log.h > index 8690ec42954d..332306202464 100644 > --- a/include/media/dvb_math.h > +++ b/include/linux/int_log.h > @@ -1,6 +1,5 @@ > /* > - * dvb-math provides some complex fixed-point math > - * operations shared between the dvb related stuff > + * Provides fixed-point logarithm operations. > * > * Copyright (C) 2006 Christoph Pfister (christophpfister@xxxxxxxxx) > * > @@ -15,8 +14,8 @@ > * GNU Lesser General Public License for more details. > */ > > -#ifndef __DVB_MATH_H > -#define __DVB_MATH_H > +#ifndef __LINUX_INT_LOG_H > +#define __LINUX_INT_LOG_H > > #include <linux/types.h> > > diff --git a/lib/math/Makefile b/lib/math/Makefile > index bfac26ddfc22..91fcdb0c9efe 100644 > --- a/lib/math/Makefile > +++ b/lib/math/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0-only > -obj-y += div64.o gcd.o lcm.o int_pow.o int_sqrt.o reciprocal_div.o > +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_PRIME_NUMBERS) += prime_numbers.o > diff --git a/drivers/media/dvb-core/dvb_math.c b/lib/math/int_log.c > similarity index 95% > rename from drivers/media/dvb-core/dvb_math.c > rename to lib/math/int_log.c > index dc90564d7f34..322df25a22d6 100644 > --- a/drivers/media/dvb-core/dvb_math.c > +++ b/lib/math/int_log.c > @@ -1,6 +1,5 @@ > /* > - * dvb-math provides some complex fixed-point math > - * operations shared between the dvb related stuff > + * Provides fixed-point logarithm operations. > * > * Copyright (C) 2006 Christoph Pfister (christophpfister@xxxxxxxxx) > * > @@ -16,10 +15,12 @@ > */ > > #include <linux/bitops.h> > +#include <linux/export.h> > +#include <linux/int_log.h> > #include <linux/kernel.h> > -#include <linux/module.h> > +#include <linux/types.h> > + > #include <asm/bug.h> > -#include <media/dvb_math.h> > > static const unsigned short logtable[256] = { > 0x0000, 0x0171, 0x02e0, 0x044e, 0x05ba, 0x0725, 0x088e, 0x09f7, > @@ -53,7 +54,7 @@ static const unsigned short logtable[256] = { > 0xe82a, 0xe8ef, 0xe9b3, 0xea77, 0xeb3b, 0xebfe, 0xecc1, 0xed83, > 0xee45, 0xef06, 0xefc8, 0xf088, 0xf149, 0xf209, 0xf2c8, 0xf387, > 0xf446, 0xf505, 0xf5c3, 0xf680, 0xf73e, 0xf7fb, 0xf8b7, 0xf973, > - 0xfa2f, 0xfaea, 0xfba5, 0xfc60, 0xfd1a, 0xfdd4, 0xfe8e, 0xff47 > + 0xfa2f, 0xfaea, 0xfba5, 0xfc60, 0xfd1a, 0xfdd4, 0xfe8e, 0xff47, > }; > > unsigned int intlog2(u32 value)