* [PATCH next-20140627] drivers/staging/media/solo6x10: sparse warning corrections
@ 2014-06-30 6:45 Sami Laine
0 siblings, 0 replies; only message in thread
From: Sami Laine @ 2014-06-30 6:45 UTC (permalink / raw)
To: ismael.luceno; +Cc: m.chehab, gregkh, linux-media, devel, linux-kernel, trivial
From: Sami Laine <laine.j.sami@gmail.com>
Sparse warning correction:
CHECK drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c
drivers/staging/media/solo6x10/solo6x10-jpeg.h:113:21: warning: symbol 'jpeg_dqt
' was not declared. Should it be static?
The symbol jpeg_dqt is used just in solo6x10-v4l2-enc.c, so I think it
makes sense to declare it static, which suppresses the sparse warning.
Signed-off-by: Sami Laine <laine.j.sami@gmail.com>
---
diff --git a/drivers/staging/media/solo6x10/solo6x10-jpeg.h b/drivers/staging/me
index c5218ce..9e41185 100644
--- a/drivers/staging/media/solo6x10/solo6x10-jpeg.h
+++ b/drivers/staging/media/solo6x10/solo6x10-jpeg.h
@@ -110,7 +110,7 @@ static const unsigned char jpeg_header[] = {
/* This is the byte marker for the start of the DQT */
#define DQT_START 17
#define DQT_LEN 138
-const unsigned char jpeg_dqt[4][DQT_LEN] = {
+static const unsigned char jpeg_dqt[4][DQT_LEN] = {
{
0xff, 0xdb, 0x00, 0x43, 0x00,
0x08, 0x06, 0x06, 0x07, 0x06, 0x05, 0x08, 0x07,
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2014-06-30 6:45 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-30 6:45 [PATCH next-20140627] drivers/staging/media/solo6x10: sparse warning corrections Sami Laine
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.