As warned by smatch:

        drivers/media/platform/seco-cec/seco-cec.c:338:21:  warning: symbol 
'secocec_cec_adap_ops' was not declared. Should it be static?

This struct should be static. Also, it is const, so declare it
as such.

Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org>
---
 drivers/media/platform/seco-cec/seco-cec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/seco-cec/seco-cec.c 
b/drivers/media/platform/seco-cec/seco-cec.c
index 8308873c53ab..01ada99de723 100644
--- a/drivers/media/platform/seco-cec/seco-cec.c
+++ b/drivers/media/platform/seco-cec/seco-cec.c
@@ -335,7 +335,7 @@ static void secocec_rx_done(struct cec_adapter *adap, u16 
status_val)
        smb_wr16(SECOCEC_STATUS, status_val);
 }
 
-struct cec_adap_ops secocec_cec_adap_ops = {
+static const struct cec_adap_ops secocec_cec_adap_ops = {
        /* Low-level callbacks */
        .adap_enable = secocec_adap_enable,
        .adap_log_addr = secocec_adap_log_addr,
-- 
2.19.1

Reply via email to