[media] dvb-frontends: Add static
authorAlan <gnomes@lxorguk.ukuu.org.uk>
Mon, 20 Jan 2014 18:03:04 +0000 (15:03 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Fri, 4 Jul 2014 18:33:46 +0000 (15:33 -0300)
Add static to tda m_* variables in the header. They don't need to be global.
With some cleanup they could probably even be marked const.

Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=68191

Reported-by: Christian Schneider <christian@ch-sc.de>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/dvb-frontends/tda18271c2dd_maps.h
drivers/media/tuners/r820t.c

index b87661b9df14f714bb07e057f34388f91091288a..f3bca5c237d74bd15d17e66999518571b9573f2f 100644 (file)
@@ -5,7 +5,7 @@ enum HF_S {
        HF_DVBC_8MHZ, HF_DVBC
 };
 
-struct SStandardParam m_StandardTable[] = {
+static struct SStandardParam m_StandardTable[] = {
        {       0,        0, 0x00, 0x00 },    /* HF_None */
        { 6000000,  7000000, 0x1D, 0x2C },    /* HF_B, */
        { 6900000,  8000000, 0x1E, 0x2C },    /* HF_DK, */
@@ -27,7 +27,7 @@ struct SStandardParam m_StandardTable[] = {
        {       0,        0, 0x00, 0x00 },    /* HF_DVBC (Unused) */
 };
 
-struct SMap  m_BP_Filter_Map[] = {
+static struct SMap  m_BP_Filter_Map[] = {
        {   62000000,  0x00 },
        {   84000000,  0x01 },
        {  100000000,  0x02 },
@@ -799,14 +799,14 @@ static struct SRFBandMap  m_RF_Band_Map[7] = {
        {  865000000,  489500000,   697500000,  842000000},
 };
 
-u8 m_Thermometer_Map_1[16] = {
+static u8 m_Thermometer_Map_1[16] = {
        60, 62, 66, 64,
        74, 72, 68, 70,
        90, 88, 84, 86,
        76, 78, 82, 80,
 };
 
-u8 m_Thermometer_Map_2[16] = {
+static u8 m_Thermometer_Map_2[16] = {
        92, 94, 98, 96,
        106, 104, 100, 102,
        122, 120, 116, 118,
index 96ccfebce7cad32f8891c04be9bdddc71b6e152d..a0db64f57a9817c640ae14fb466eb5c0a31965d8 100644 (file)
@@ -1545,7 +1545,7 @@ static int r820t_imr_cross(struct r820t_priv *priv,
                cross[i].value = rc;
 
                if (cross[i].value < tmp.value)
-                       memcpy(&tmp, &cross[i], sizeof(tmp));
+                       tmp = cross[i];
        }
 
        if ((tmp.phase_y & 0x1f) == 1) {        /* y-direction */