mISDN: use NULL pointer instead of plain integer
authorHannes Eder <hannes@hanneseder.net>
Sat, 13 Dec 2008 05:11:28 +0000 (21:11 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sat, 13 Dec 2008 05:11:28 +0000 (21:11 -0800)
Fix more than 100 (all remaining in mISDN) sparse warnings:

  drivers/isdn/hardware/mISDN/hfcmulti.c:***:**: warning: Using plain integer as NULL pointer
  drivers/isdn/mISDN/dsp_tones.c:***:**: warning: Using plain integer as NULL pointer
  drivers/isdn/mISDN/dsp_pipeline.c:***:**: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Acked-by: Karsten Keil <kkeil@suse.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/hardware/mISDN/hfcmulti.c
drivers/isdn/mISDN/dsp_pipeline.c
drivers/isdn/mISDN/dsp_tones.c

index 1eac03f39d0005e0e75b75f3925e1924193e794b..46684e13618199cf2b40b9e824a26c37f62c67e5 100644 (file)
@@ -887,7 +887,7 @@ vpm_echocan_off(struct hfc_multi *hc, int ch)
 static inline void
 hfcmulti_resync(struct hfc_multi *locked, struct hfc_multi *newmaster, int rm)
 {
-       struct hfc_multi *hc, *next, *pcmmaster = 0;
+       struct hfc_multi *hc, *next, *pcmmaster = NULL;
        u_int *plx_acc_32, pv;
        u_long flags;
 
index 850260ab57d041cf7a55c86061bb134820021c35..5ee6651b45b903c3e1931fe316956e1166ce5d40 100644 (file)
@@ -249,7 +249,7 @@ int dsp_pipeline_build(struct dsp_pipeline *pipeline, const char *cfg)
                name = strsep(&tok, "(");
                args = strsep(&tok, ")");
                if (args && !*args)
-                       args = 0;
+                       args = NULL;
 
                list_for_each_entry_safe(entry, n, &dsp_elements, list)
                        if (!strcmp(entry->elem->name, name)) {
index 23dd0dd215242dd37dfb111383181bdb3f194b4e..de4d0c9ac4834cf70688502b084d9321219fb272 100644 (file)
@@ -238,113 +238,113 @@ struct pattern {
        u32 seq[10];
 } pattern[] = {
        {TONE_GERMAN_DIALTONE,
-       {DATA_GA, 0, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GA, 0, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GA, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GA, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {1900, 0, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_OLDDIALTONE,
-       {DATA_GO, 0, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GO, 0, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GO, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GO, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {1998, 0, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_AMERICAN_DIALTONE,
-       {DATA_DT, 0, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_DT, 0, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_DT, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_DT, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {8000, 0, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_DIALPBX,
-       {DATA_GA, DATA_S, DATA_GA, DATA_S, DATA_GA, DATA_S, 0, 0, 0, 0},
-       {SIZE_GA, SIZE_S, SIZE_GA, SIZE_S, SIZE_GA, SIZE_S, 0, 0, 0, 0},
+       {DATA_GA, DATA_S, DATA_GA, DATA_S, DATA_GA, DATA_S, NULL, NULL, NULL, NULL},
+       {SIZE_GA, SIZE_S, SIZE_GA, SIZE_S, SIZE_GA, SIZE_S, NULL, NULL, NULL, NULL},
        {2000, 2000, 2000, 2000, 2000, 12000, 0, 0, 0, 0} },
 
        {TONE_GERMAN_OLDDIALPBX,
-       {DATA_GO, DATA_S, DATA_GO, DATA_S, DATA_GO, DATA_S, 0, 0, 0, 0},
-       {SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, 0, 0, 0, 0},
+       {DATA_GO, DATA_S, DATA_GO, DATA_S, DATA_GO, DATA_S, NULL, NULL, NULL, NULL},
+       {SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, NULL, NULL, NULL, NULL},
        {2000, 2000, 2000, 2000, 2000, 12000, 0, 0, 0, 0} },
 
        {TONE_AMERICAN_DIALPBX,
-       {DATA_DT, DATA_S, DATA_DT, DATA_S, DATA_DT, DATA_S, 0, 0, 0, 0},
-       {SIZE_DT, SIZE_S, SIZE_DT, SIZE_S, SIZE_DT, SIZE_S, 0, 0, 0, 0},
+       {DATA_DT, DATA_S, DATA_DT, DATA_S, DATA_DT, DATA_S, NULL, NULL, NULL, NULL},
+       {SIZE_DT, SIZE_S, SIZE_DT, SIZE_S, SIZE_DT, SIZE_S, NULL, NULL, NULL, NULL},
        {2000, 2000, 2000, 2000, 2000, 12000, 0, 0, 0, 0} },
 
        {TONE_GERMAN_RINGING,
-       {DATA_GA, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GA, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GA, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GA, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {8000, 32000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_OLDRINGING,
-       {DATA_GO, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GO, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GO, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GO, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {8000, 40000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_AMERICAN_RINGING,
-       {DATA_RI, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_RI, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_RI, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_RI, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {8000, 32000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_RINGPBX,
-       {DATA_GA, DATA_S, DATA_GA, DATA_S, 0, 0, 0, 0, 0, 0},
-       {SIZE_GA, SIZE_S, SIZE_GA, SIZE_S, 0, 0, 0, 0, 0, 0},
+       {DATA_GA, DATA_S, DATA_GA, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GA, SIZE_S, SIZE_GA, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL},
        {4000, 4000, 4000, 28000, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_OLDRINGPBX,
-       {DATA_GO, DATA_S, DATA_GO, DATA_S, 0, 0, 0, 0, 0, 0},
-       {SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, 0, 0, 0, 0, 0, 0},
+       {DATA_GO, DATA_S, DATA_GO, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL},
        {4000, 4000, 4000, 28000, 0, 0, 0, 0, 0, 0} },
 
        {TONE_AMERICAN_RINGPBX,
-       {DATA_RI, DATA_S, DATA_RI, DATA_S, 0, 0, 0, 0, 0, 0},
-       {SIZE_RI, SIZE_S, SIZE_RI, SIZE_S, 0, 0, 0, 0, 0, 0},
+       {DATA_RI, DATA_S, DATA_RI, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_RI, SIZE_S, SIZE_RI, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL},
        {4000, 4000, 4000, 28000, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_BUSY,
-       {DATA_GA, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GA, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GA, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GA, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {4000, 4000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_OLDBUSY,
-       {DATA_GO, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GO, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GO, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GO, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {1000, 5000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_AMERICAN_BUSY,
-       {DATA_BU, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_BU, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_BU, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_BU, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {4000, 4000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_HANGUP,
-       {DATA_GA, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GA, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GA, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GA, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {4000, 4000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_OLDHANGUP,
-       {DATA_GO, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GO, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GO, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GO, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {1000, 5000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_AMERICAN_HANGUP,
-       {DATA_DT, 0, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_DT, 0, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_DT, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_DT, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {8000, 0, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_SPECIAL_INFO,
-       {DATA_S1, DATA_S2, DATA_S3, DATA_S, 0, 0, 0, 0, 0, 0},
-       {SIZE_S1, SIZE_S2, SIZE_S3, SIZE_S, 0, 0, 0, 0, 0, 0},
+       {DATA_S1, DATA_S2, DATA_S3, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_S1, SIZE_S2, SIZE_S3, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL},
        {2666, 2666, 2666, 8002, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_GASSENBESETZT,
-       {DATA_GA, DATA_S, 0, 0, 0, 0, 0, 0, 0, 0},
-       {SIZE_GA, SIZE_S, 0, 0, 0, 0, 0, 0, 0, 0},
+       {DATA_GA, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GA, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {2000, 2000, 0, 0, 0, 0, 0, 0, 0, 0} },
 
        {TONE_GERMAN_AUFSCHALTTON,
-       {DATA_GO, DATA_S, DATA_GO, DATA_S, 0, 0, 0, 0, 0, 0},
-       {SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, 0, 0, 0, 0, 0, 0},
+       {DATA_GO, DATA_S, DATA_GO, DATA_S, NULL, NULL, NULL, NULL, NULL, NULL},
+       {SIZE_GO, SIZE_S, SIZE_GO, SIZE_S, NULL, NULL, NULL, NULL, NULL, NULL},
        {1000, 5000, 1000, 17000, 0, 0, 0, 0, 0, 0} },
 
        {0,
-       {0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
-       {0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
+       {NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
+       {NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL},
        {0, 0, 0, 0, 0, 0, 0, 0, 0, 0} },
 };
 
@@ -467,7 +467,7 @@ dsp_tone_timeout(void *arg)
 
        /* set next tone */
        if (pat->data[index] == DATA_S)
-               dsp_tone_hw_message(dsp, 0, 0);
+               dsp_tone_hw_message(dsp, NULL, 0);
        else
                dsp_tone_hw_message(dsp, pat->data[index], *(pat->siz[index]));
        /* set timer */