public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* [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

* [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

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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox