* [linuxtv-media:master 1013/1029] drivers/media/dvb-frontends/cx24120.c:806:6: sparse: symbol 'cx24120_calculate_ber_window' was not declared. Should it be static?
@ 2015-05-21 10:16 kbuild test robot
2015-05-21 10:16 ` [PATCH linuxtv-media] cx24120: cx24120_calculate_ber_window() can be static kbuild test robot
0 siblings, 1 reply; 2+ messages in thread
From: kbuild test robot @ 2015-05-21 10:16 UTC (permalink / raw)
To: Jemma Denson
Cc: kbuild-all, Mauro Carvalho Chehab, linux-media, Patrick Boettcher
tree: git://linuxtv.org/media_tree.git master
head: 2a80f296422a01178d0a993479369e94f5830127
commit: ddcb252e41c15b360c0e9a172fbd29d3f0ed18cd [1013/1029] [media] cx24120: Add in dvbv5 stats for bit error rate
reproduce:
# apt-get install sparse
git checkout ddcb252e41c15b360c0e9a172fbd29d3f0ed18cd
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> drivers/media/dvb-frontends/cx24120.c:806:6: sparse: symbol 'cx24120_calculate_ber_window' was not declared. Should it be static?
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH linuxtv-media] cx24120: cx24120_calculate_ber_window() can be static
2015-05-21 10:16 [linuxtv-media:master 1013/1029] drivers/media/dvb-frontends/cx24120.c:806:6: sparse: symbol 'cx24120_calculate_ber_window' was not declared. Should it be static? kbuild test robot
@ 2015-05-21 10:16 ` kbuild test robot
0 siblings, 0 replies; 2+ messages in thread
From: kbuild test robot @ 2015-05-21 10:16 UTC (permalink / raw)
To: Jemma Denson
Cc: kbuild-all, Mauro Carvalho Chehab, linux-media, Patrick Boettcher
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
cx24120.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/dvb-frontends/cx24120.c b/drivers/media/dvb-frontends/cx24120.c
index 2d46780..edff79a 100644
--- a/drivers/media/dvb-frontends/cx24120.c
+++ b/drivers/media/dvb-frontends/cx24120.c
@@ -803,7 +803,7 @@ static int cx24120_get_fec(struct dvb_frontend *fe)
}
/* Calculate ber window time */
-void cx24120_calculate_ber_window(struct cx24120_state *state, u32 rate)
+static void cx24120_calculate_ber_window(struct cx24120_state *state, u32 rate)
{
struct dvb_frontend *fe = &state->frontend;
struct dtv_frontend_properties *c = &fe->dtv_property_cache;
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-05-21 10:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-21 10:16 [linuxtv-media:master 1013/1029] drivers/media/dvb-frontends/cx24120.c:806:6: sparse: symbol 'cx24120_calculate_ber_window' was not declared. Should it be static? kbuild test robot
2015-05-21 10:16 ` [PATCH linuxtv-media] cx24120: cx24120_calculate_ber_window() can be static 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.