* [linux-review:UPDATE-20191005-005305/Mauro-Carvalho-Chehab/media-cxd2841er-avoid-too-many-status-inquires/20191003-224524 1/1] drivers/media//dvb-frontends/cxd2841er.c:2010:1: note: in expansion of macro 'dev_info'
@ 2019-10-04 17:13 ` kbuild test robot
0 siblings, 0 replies; 2+ messages in thread
From: kbuild test robot @ 2019-10-04 17:13 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 5361 bytes --]
tree: https://github.com/0day-ci/linux/commits/UPDATE-20191005-005305/Mauro-Carvalho-Chehab/media-cxd2841er-avoid-too-many-status-inquires/20191003-224524
head: efac65071ccae96e9185db48174c7f77a2fe5098
commit: efac65071ccae96e9185db48174c7f77a2fe5098 [1/1] media: cxd2841er: avoid too many status inquires
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 7.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout efac65071ccae96e9185db48174c7f77a2fe5098
# save the attached .config to linux build tree
GCC_VERSION=7.4.0 make.cross ARCH=xtensa
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/acpi.h:15:0,
from include/linux/i2c.h:13,
from include/media/dvb_frontend.h:38,
from drivers/media//dvb-frontends/cxd2841er.c:26:
drivers/media//dvb-frontends/cxd2841er.c: In function 'cxd2841er_read_ber':
>> drivers/media//dvb-frontends/cxd2841er.c:2010:27: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
dev_info(&priv->i2c->dev, "BER interval: %d ms", priv->ber_interval);
^
include/linux/device.h:1658:22: note: in definition of macro 'dev_fmt'
#define dev_fmt(fmt) fmt
^~~
>> drivers/media//dvb-frontends/cxd2841er.c:2010:1: note: in expansion of macro 'dev_info'
dev_info(&priv->i2c->dev, "BER interval: %d ms", priv->ber_interval);
^~~~~~~~
drivers/media//dvb-frontends/cxd2841er.c: In function 'cxd2841er_read_ucblocks':
drivers/media//dvb-frontends/cxd2841er.c:2166:27: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
dev_info(&priv->i2c->dev, "UCB interval: %d ms", priv->ucb_interval);
^
include/linux/device.h:1658:22: note: in definition of macro 'dev_fmt'
#define dev_fmt(fmt) fmt
^~~
drivers/media//dvb-frontends/cxd2841er.c:2166:1: note: in expansion of macro 'dev_info'
dev_info(&priv->i2c->dev, "UCB interval: %d ms", priv->ucb_interval);
^~~~~~~~
vim +/dev_info +2010 drivers/media//dvb-frontends/cxd2841er.c
1944
1945 static void cxd2841er_read_ber(struct dvb_frontend *fe)
1946 {
1947 struct dtv_frontend_properties *p = &fe->dtv_property_cache;
1948 struct cxd2841er_priv *priv = fe->demodulator_priv;
1949 u32 ret, bit_error = 0, bit_count = 0;
1950
1951 if (priv->ber_time &&
1952 (!time_after(jiffies, priv->ber_time)))
1953 return;
1954
1955 dev_dbg(&priv->i2c->dev, "%s()\n", __func__);
1956 switch (p->delivery_system) {
1957 case SYS_DVBC_ANNEX_A:
1958 case SYS_DVBC_ANNEX_B:
1959 case SYS_DVBC_ANNEX_C:
1960 ret = cxd2841er_read_ber_c(priv, &bit_error, &bit_count);
1961 break;
1962 case SYS_ISDBT:
1963 ret = cxd2841er_read_ber_i(priv, &bit_error, &bit_count);
1964 break;
1965 case SYS_DVBS:
1966 ret = cxd2841er_mon_read_ber_s(priv, &bit_error, &bit_count);
1967
1968 if (!priv->ber_interval && p->symbol_rate)
1969 priv->ber_interval = (10000000) / (p->symbol_rate / 1000);
1970 break;
1971 case SYS_DVBS2:
1972 ret = cxd2841er_mon_read_ber_s2(priv, &bit_error, &bit_count);
1973
1974 if (!priv->ber_interval && p->symbol_rate)
1975 priv->ber_interval = (10000000) / (p->symbol_rate / 1000);
1976 break;
1977 case SYS_DVBT:
1978 ret = cxd2841er_read_ber_t(priv, &bit_error, &bit_count);
1979 break;
1980 case SYS_DVBT2:
1981 ret = cxd2841er_read_ber_t2(priv, &bit_error, &bit_count);
1982 break;
1983 default:
1984 p->post_bit_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1985 p->post_bit_count.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1986 return;
1987 }
1988
1989 if (!ret) {
1990 p->post_bit_error.stat[0].scale = FE_SCALE_COUNTER;
1991 p->post_bit_error.stat[0].uvalue += bit_error;
1992 p->post_bit_count.stat[0].scale = FE_SCALE_COUNTER;
1993 p->post_bit_count.stat[0].uvalue += bit_count;
1994 } else {
1995 p->post_bit_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1996 p->post_bit_count.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1997 }
1998
1999 /*
2000 * If the per-delivery system doesn't specify, set a default timeout
2001 * that will wait for ~12.5 seconds for 8MHz channels
2002 */
2003 if (!priv->ber_interval && p->bandwidth_hz)
2004 priv->ber_interval = (100000000) / (p->bandwidth_hz / 1000);
2005
2006 if (priv->ber_interval < 1000)
2007 priv->ber_interval = 1000;
2008
2009 // HACK:
> 2010 dev_info(&priv->i2c->dev, "BER interval: %d ms", priv->ber_interval);
2011
2012 priv->ber_time = jiffies + msecs_to_jiffies(priv->ber_interval);
2013 }
2014
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 59595 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* [linux-review:UPDATE-20191005-005305/Mauro-Carvalho-Chehab/media-cxd2841er-avoid-too-many-status-inquires/20191003-224524 1/1] drivers/media//dvb-frontends/cxd2841er.c:2010:1: note: in expansion of macro 'dev_info'
@ 2019-10-04 17:13 ` kbuild test robot
0 siblings, 0 replies; 2+ messages in thread
From: kbuild test robot @ 2019-10-04 17:13 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: kbuild-all, linux-media, 0day robot
[-- Attachment #1: Type: text/plain, Size: 5242 bytes --]
tree: https://github.com/0day-ci/linux/commits/UPDATE-20191005-005305/Mauro-Carvalho-Chehab/media-cxd2841er-avoid-too-many-status-inquires/20191003-224524
head: efac65071ccae96e9185db48174c7f77a2fe5098
commit: efac65071ccae96e9185db48174c7f77a2fe5098 [1/1] media: cxd2841er: avoid too many status inquires
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 7.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout efac65071ccae96e9185db48174c7f77a2fe5098
# save the attached .config to linux build tree
GCC_VERSION=7.4.0 make.cross ARCH=xtensa
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/acpi.h:15:0,
from include/linux/i2c.h:13,
from include/media/dvb_frontend.h:38,
from drivers/media//dvb-frontends/cxd2841er.c:26:
drivers/media//dvb-frontends/cxd2841er.c: In function 'cxd2841er_read_ber':
>> drivers/media//dvb-frontends/cxd2841er.c:2010:27: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
dev_info(&priv->i2c->dev, "BER interval: %d ms", priv->ber_interval);
^
include/linux/device.h:1658:22: note: in definition of macro 'dev_fmt'
#define dev_fmt(fmt) fmt
^~~
>> drivers/media//dvb-frontends/cxd2841er.c:2010:1: note: in expansion of macro 'dev_info'
dev_info(&priv->i2c->dev, "BER interval: %d ms", priv->ber_interval);
^~~~~~~~
drivers/media//dvb-frontends/cxd2841er.c: In function 'cxd2841er_read_ucblocks':
drivers/media//dvb-frontends/cxd2841er.c:2166:27: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
dev_info(&priv->i2c->dev, "UCB interval: %d ms", priv->ucb_interval);
^
include/linux/device.h:1658:22: note: in definition of macro 'dev_fmt'
#define dev_fmt(fmt) fmt
^~~
drivers/media//dvb-frontends/cxd2841er.c:2166:1: note: in expansion of macro 'dev_info'
dev_info(&priv->i2c->dev, "UCB interval: %d ms", priv->ucb_interval);
^~~~~~~~
vim +/dev_info +2010 drivers/media//dvb-frontends/cxd2841er.c
1944
1945 static void cxd2841er_read_ber(struct dvb_frontend *fe)
1946 {
1947 struct dtv_frontend_properties *p = &fe->dtv_property_cache;
1948 struct cxd2841er_priv *priv = fe->demodulator_priv;
1949 u32 ret, bit_error = 0, bit_count = 0;
1950
1951 if (priv->ber_time &&
1952 (!time_after(jiffies, priv->ber_time)))
1953 return;
1954
1955 dev_dbg(&priv->i2c->dev, "%s()\n", __func__);
1956 switch (p->delivery_system) {
1957 case SYS_DVBC_ANNEX_A:
1958 case SYS_DVBC_ANNEX_B:
1959 case SYS_DVBC_ANNEX_C:
1960 ret = cxd2841er_read_ber_c(priv, &bit_error, &bit_count);
1961 break;
1962 case SYS_ISDBT:
1963 ret = cxd2841er_read_ber_i(priv, &bit_error, &bit_count);
1964 break;
1965 case SYS_DVBS:
1966 ret = cxd2841er_mon_read_ber_s(priv, &bit_error, &bit_count);
1967
1968 if (!priv->ber_interval && p->symbol_rate)
1969 priv->ber_interval = (10000000) / (p->symbol_rate / 1000);
1970 break;
1971 case SYS_DVBS2:
1972 ret = cxd2841er_mon_read_ber_s2(priv, &bit_error, &bit_count);
1973
1974 if (!priv->ber_interval && p->symbol_rate)
1975 priv->ber_interval = (10000000) / (p->symbol_rate / 1000);
1976 break;
1977 case SYS_DVBT:
1978 ret = cxd2841er_read_ber_t(priv, &bit_error, &bit_count);
1979 break;
1980 case SYS_DVBT2:
1981 ret = cxd2841er_read_ber_t2(priv, &bit_error, &bit_count);
1982 break;
1983 default:
1984 p->post_bit_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1985 p->post_bit_count.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1986 return;
1987 }
1988
1989 if (!ret) {
1990 p->post_bit_error.stat[0].scale = FE_SCALE_COUNTER;
1991 p->post_bit_error.stat[0].uvalue += bit_error;
1992 p->post_bit_count.stat[0].scale = FE_SCALE_COUNTER;
1993 p->post_bit_count.stat[0].uvalue += bit_count;
1994 } else {
1995 p->post_bit_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1996 p->post_bit_count.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
1997 }
1998
1999 /*
2000 * If the per-delivery system doesn't specify, set a default timeout
2001 * that will wait for ~12.5 seconds for 8MHz channels
2002 */
2003 if (!priv->ber_interval && p->bandwidth_hz)
2004 priv->ber_interval = (100000000) / (p->bandwidth_hz / 1000);
2005
2006 if (priv->ber_interval < 1000)
2007 priv->ber_interval = 1000;
2008
2009 // HACK:
> 2010 dev_info(&priv->i2c->dev, "BER interval: %d ms", priv->ber_interval);
2011
2012 priv->ber_time = jiffies + msecs_to_jiffies(priv->ber_interval);
2013 }
2014
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 59595 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-10-04 17:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-04 17:13 [linux-review:UPDATE-20191005-005305/Mauro-Carvalho-Chehab/media-cxd2841er-avoid-too-many-status-inquires/20191003-224524 1/1] drivers/media//dvb-frontends/cxd2841er.c:2010:1: note: in expansion of macro 'dev_info' kbuild test robot
2019-10-04 17:13 ` kbuild test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.