Patchwork isdn: eicon: fix sparse warning: make global functions static

login
register
mail settings
Submitter Hannes Eder
Date Dec. 11, 2008, 3:07 p.m.
Message ID <49458560.0aa1660a.486c.7f09@mx.google.com>
Download mbox | patch
Permalink /patch/13957/
State Changes Requested
Delegated to: David Miller
Headers show

Comments

Hannes Eder - Dec. 11, 2008, 3:07 p.m.
Fix this sparse warnings by making the functions static:

drivers/isdn/hardware/eicon/di.c:356:6: warning: symbol 'isdn_rc' was not declared. Should it be static?
drivers/isdn/hardware/eicon/di.c:558:6: warning: symbol 'isdn_ind' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:595:6: warning: symbol 'api_parse' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:634:6: warning: symbol 'api_save_msg' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:666:6: warning: symbol 'api_load_msg' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:3417:6: warning: symbol 'manufacturer_req' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:3745:6: warning: symbol 'manufacturer_res' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:4077:6: warning: symbol 'control_rc' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:4743:6: warning: symbol 'data_rc' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:4779:6: warning: symbol 'data_ack' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:4805:6: warning: symbol 'sig_ind' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:6173:6: warning: symbol 'SendInfo' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:6349:6: warning: symbol 'SendMultiIE' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:6468:6: warning: symbol 'nl_ind' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:7250:6: warning: symbol 'get_plci' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:7409:6: warning: symbol 'add_d' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:7427:6: warning: symbol 'add_ai' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:7448:6: warning: symbol 'add_b1' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:7912:6: warning: symbol 'add_b23' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:8709:6: warning: symbol 'nl_req_ncci' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:8731:6: warning: symbol 'send_req' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:8866:6: warning: symbol 'listen_check' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:8909:6: warning: symbol 'IndParse' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:8994:6: warning: symbol 'ie_compare' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:9003:6: warning: symbol 'find_cip' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:9071:6: warning: symbol 'SetVoiceChannel' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:9089:6: warning: symbol 'VoiceChannelOff' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:9102:6: warning: symbol 'AdvCodecSupport' was not declared. Should it be static?
drivers/isdn/hardware/eicon/message.c:9198:6: warning: symbol 'CodecIdCheck' was not declared. Should it be static?

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/isdn/hardware/eicon/di.c      |   28 ++++++++--------
 drivers/isdn/hardware/eicon/message.c |   54 ++++++++++++++++----------------
 2 files changed, 41 insertions(+), 41 deletions(-)
Hannes Eder - Dec. 14, 2008, 10:19 p.m.
ups. checkpatch.pl reports a couple of formatting errors, I'll fix
them and resubmit.

-Hannes

On Thu, Dec 11, 2008 at 4:07 PM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings by making the functions static:
>
> drivers/isdn/hardware/eicon/di.c:356:6: warning: symbol 'isdn_rc' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/di.c:558:6: warning: symbol 'isdn_ind' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:595:6: warning: symbol 'api_parse' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:634:6: warning: symbol 'api_save_msg' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:666:6: warning: symbol 'api_load_msg' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:3417:6: warning: symbol 'manufacturer_req' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:3745:6: warning: symbol 'manufacturer_res' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:4077:6: warning: symbol 'control_rc' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:4743:6: warning: symbol 'data_rc' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:4779:6: warning: symbol 'data_ack' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:4805:6: warning: symbol 'sig_ind' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:6173:6: warning: symbol 'SendInfo' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:6349:6: warning: symbol 'SendMultiIE' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:6468:6: warning: symbol 'nl_ind' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:7250:6: warning: symbol 'get_plci' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:7409:6: warning: symbol 'add_d' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:7427:6: warning: symbol 'add_ai' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:7448:6: warning: symbol 'add_b1' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:7912:6: warning: symbol 'add_b23' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:8709:6: warning: symbol 'nl_req_ncci' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:8731:6: warning: symbol 'send_req' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:8866:6: warning: symbol 'listen_check' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:8909:6: warning: symbol 'IndParse' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:8994:6: warning: symbol 'ie_compare' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:9003:6: warning: symbol 'find_cip' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:9071:6: warning: symbol 'SetVoiceChannel' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:9089:6: warning: symbol 'VoiceChannelOff' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:9102:6: warning: symbol 'AdvCodecSupport' was not declared. Should it be static?
> drivers/isdn/hardware/eicon/message.c:9198:6: warning: symbol 'CodecIdCheck' was not declared. Should it be static?
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/isdn/hardware/eicon/di.c      |   28 ++++++++--------
>  drivers/isdn/hardware/eicon/message.c |   54 ++++++++++++++++----------------
>  2 files changed, 41 insertions(+), 41 deletions(-)
>
> diff --git a/drivers/isdn/hardware/eicon/di.c b/drivers/isdn/hardware/eicon/di.c
> index 10760b3..d798fb4 100644
> --- a/drivers/isdn/hardware/eicon/di.c
> +++ b/drivers/isdn/hardware/eicon/di.c
> @@ -353,13 +353,13 @@ void scom_clear_int(ADAPTER * a)
>  /*------------------------------------------------------------------*/
>  /* return code handler                                              */
>  /*------------------------------------------------------------------*/
> -byte isdn_rc(ADAPTER * a,
> -             byte Rc,
> -             byte Id,
> -             byte Ch,
> -             word Ref,
> -             dword extended_info_type,
> -             dword extended_info)
> +static byte isdn_rc(ADAPTER * a,
> +                    byte Rc,
> +                    byte Id,
> +                    byte Ch,
> +                    word Ref,
> +                    dword extended_info_type,
> +                    dword extended_info)
>  {
>   ENTITY  * this;
>   byte e_no;
> @@ -555,13 +555,13 @@ byte isdn_rc(ADAPTER * a,
>  /*------------------------------------------------------------------*/
>  /* indication handler                                               */
>  /*------------------------------------------------------------------*/
> -byte isdn_ind(ADAPTER * a,
> -              byte Ind,
> -              byte Id,
> -              byte Ch,
> -              PBUFFER * RBuffer,
> -              byte MInd,
> -              word MLength)
> +static byte isdn_ind(ADAPTER * a,
> +                     byte Ind,
> +                     byte Id,
> +                     byte Ch,
> +                     PBUFFER * RBuffer,
> +                     byte MInd,
> +                     word MLength)
>  {
>   ENTITY  * this;
>   word clength;
> diff --git a/drivers/isdn/hardware/eicon/message.c b/drivers/isdn/hardware/eicon/message.c
> index 599fed8..339e4ed 100644
> --- a/drivers/isdn/hardware/eicon/message.c
> +++ b/drivers/isdn/hardware/eicon/message.c
> @@ -592,7 +592,7 @@ word api_put(APPL   * appl, CAPI_MSG   * msg)
>  /* api_parse function, check the format of api messages             */
>  /*------------------------------------------------------------------*/
>
> -word api_parse(byte   * msg, word length, byte * format, API_PARSE * parms)
> +static word api_parse(byte   * msg, word length, byte * format, API_PARSE * parms)
>  {
>   word i;
>   word p;
> @@ -631,7 +631,7 @@ word api_parse(byte   * msg, word length, byte * format, API_PARSE * parms)
>   return false;
>  }
>
> -void api_save_msg(API_PARSE   *in, byte *format, API_SAVE   *out)
> +static void api_save_msg(API_PARSE   *in, byte *format, API_SAVE   *out)
>  {
>   word i, j, n = 0;
>   byte   *p;
> @@ -663,7 +663,7 @@ void api_save_msg(API_PARSE   *in, byte *format, API_SAVE   *out)
>   out->parms[i].length = 0;
>  }
>
> -void api_load_msg(API_SAVE   *in, API_PARSE   *out)
> +static void api_load_msg(API_SAVE   *in, API_PARSE   *out)
>  {
>   word i;
>
> @@ -3414,7 +3414,7 @@ byte select_b_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci,
>   return false;
>  }
>
> -byte manufacturer_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * parms)
> +static byte manufacturer_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * parms)
>  {
>   word command;
>   word i;
> @@ -3742,7 +3742,7 @@ byte manufacturer_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * p
>  }
>
>
> -byte manufacturer_res(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * msg)
> +static byte manufacturer_res(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * msg)
>  {
>   word indication;
>
> @@ -4074,7 +4074,7 @@ capi_callback_suffix:
>  }
>
>
> -void control_rc(PLCI   * plci, byte req, byte rc, byte ch, byte global_req, byte nl_rc)
> +static void control_rc(PLCI   * plci, byte req, byte rc, byte ch, byte global_req, byte nl_rc)
>  {
>   dword Id;
>   dword rId;
> @@ -4740,7 +4740,7 @@ void control_rc(PLCI   * plci, byte req, byte rc, byte ch, byte global_req, byte
>   }
>  }
>
> -void data_rc(PLCI   * plci, byte ch)
> +static void data_rc(PLCI   * plci, byte ch)
>  {
>   dword Id;
>   DIVA_CAPI_ADAPTER   * a;
> @@ -4776,7 +4776,7 @@ void data_rc(PLCI   * plci, byte ch)
>   }
>  }
>
> -void data_ack(PLCI   * plci, byte ch)
> +static void data_ack(PLCI   * plci, byte ch)
>  {
>   dword Id;
>   DIVA_CAPI_ADAPTER   * a;
> @@ -4802,7 +4802,7 @@ void data_ack(PLCI   * plci, byte ch)
>   }
>  }
>
> -void sig_ind(PLCI   * plci)
> +static void sig_ind(PLCI   * plci)
>  {
>   dword x_Id;
>   dword Id;
> @@ -6170,7 +6170,7 @@ static void SendSetupInfo(APPL   * appl, PLCI   * plci, dword Id, byte   * * par
>  }
>
>
> -void SendInfo(PLCI   * plci, dword Id, byte   * * parms, byte iesent)
> +static void SendInfo(PLCI   * plci, dword Id, byte   * * parms, byte iesent)
>  {
>   word i;
>   word j;
> @@ -6346,7 +6346,7 @@ void SendInfo(PLCI   * plci, dword Id, byte   * * parms, byte iesent)
>  }
>
>
> -byte SendMultiIE(PLCI   * plci, dword Id, byte   * * parms, byte ie_type, dword info_mask, byte setupParse)
> +static byte SendMultiIE(PLCI   * plci, dword Id, byte   * * parms, byte ie_type, dword info_mask, byte setupParse)
>  {
>   word i;
>   word j;
> @@ -6465,7 +6465,7 @@ static void SendSSExtInd(APPL   * appl, PLCI   * plci, dword Id, byte   * * parm
>     }
>  };
>
> -void nl_ind(PLCI   * plci)
> +static void nl_ind(PLCI   * plci)
>  {
>   byte ch;
>   word ncci;
> @@ -7247,7 +7247,7 @@ void nl_ind(PLCI   * plci)
>  /* find a free PLCI                                                 */
>  /*------------------------------------------------------------------*/
>
> -word get_plci(DIVA_CAPI_ADAPTER   * a)
> +static word get_plci(DIVA_CAPI_ADAPTER   * a)
>  {
>   word i,j;
>   PLCI   * plci;
> @@ -7406,7 +7406,7 @@ static void add_ie(PLCI   * plci, byte code, byte   * p, word p_length)
>  /* put a unstructured data into the buffer                          */
>  /*------------------------------------------------------------------*/
>
> -void add_d(PLCI   * plci, word length, byte   * p)
> +static void add_d(PLCI   * plci, word length, byte   * p)
>  {
>   word i;
>
> @@ -7424,7 +7424,7 @@ void add_d(PLCI   * plci, word length, byte   * p)
>  /* parameter buffer                                                 */
>  /*------------------------------------------------------------------*/
>
> -void add_ai(PLCI   * plci, API_PARSE * ai)
> +static void add_ai(PLCI   * plci, API_PARSE * ai)
>  {
>   word i;
>     API_PARSE ai_parms[5];
> @@ -7445,7 +7445,7 @@ void add_ai(PLCI   * plci, API_PARSE * ai)
>  /* put parameter for b1 protocol in the parameter buffer            */
>  /*------------------------------------------------------------------*/
>
> -word add_b1(PLCI   * plci, API_PARSE * bp, word b_channel_info, word b1_facilities)
> +static word add_b1(PLCI   * plci, API_PARSE * bp, word b_channel_info, word b1_facilities)
>  {
>     API_PARSE bp_parms[8];
>     API_PARSE mdm_cfg[9];
> @@ -7909,7 +7909,7 @@ word add_b1(PLCI   * plci, API_PARSE * bp, word b_channel_info, word b1_faciliti
>  /* put parameter for b2 and B3  protocol in the parameter buffer    */
>  /*------------------------------------------------------------------*/
>
> -word add_b23(PLCI   * plci, API_PARSE * bp)
> +static word add_b23(PLCI   * plci, API_PARSE * bp)
>  {
>   word i, fax_control_bits;
>   byte pos, len;
> @@ -8706,7 +8706,7 @@ void sig_req(PLCI   * plci, byte req, byte Id)
>  /* send a request for the network layer entity                      */
>  /*------------------------------------------------------------------*/
>
> -void nl_req_ncci(PLCI   * plci, byte req, byte ncci)
> +static void nl_req_ncci(PLCI   * plci, byte req, byte ncci)
>  {
>   if(!plci) return;
>   if(plci->adapter->adapter_disabled) return;
> @@ -8728,7 +8728,7 @@ void nl_req_ncci(PLCI   * plci, byte req, byte ncci)
>   plci->req_in_start = plci->req_in;
>  }
>
> -void send_req(PLCI   * plci)
> +static void send_req(PLCI   * plci)
>  {
>   ENTITY   * e;
>   word l;
> @@ -8863,7 +8863,7 @@ void send_data(PLCI   * plci)
>   }
>  }
>
> -void listen_check(DIVA_CAPI_ADAPTER   * a)
> +static void listen_check(DIVA_CAPI_ADAPTER   * a)
>  {
>   word i,j;
>   PLCI   * plci;
> @@ -8906,7 +8906,7 @@ void listen_check(DIVA_CAPI_ADAPTER   * a)
>  /* functions for all parameters sent in INDs                        */
>  /*------------------------------------------------------------------*/
>
> -void IndParse(PLCI   * plci, word * parms_id, byte   ** parms, byte multiIEsize)
> +static void IndParse(PLCI   * plci, word * parms_id, byte   ** parms, byte multiIEsize)
>  {
>   word ploc;            /* points to current location within packet */
>   byte w;
> @@ -8991,7 +8991,7 @@ void IndParse(PLCI   * plci, word * parms_id, byte   ** parms, byte multiIEsize)
>  /* try to match a cip from received BC and HLC                      */
>  /*------------------------------------------------------------------*/
>
> -byte ie_compare(byte   * ie1, byte * ie2)
> +static byte ie_compare(byte   * ie1, byte * ie2)
>  {
>   word i;
>   if(!ie1 || ! ie2) return false;
> @@ -9000,7 +9000,7 @@ byte ie_compare(byte   * ie1, byte * ie2)
>   return true;
>  }
>
> -word find_cip(DIVA_CAPI_ADAPTER   * a, byte   * bc, byte   * hlc)
> +static word find_cip(DIVA_CAPI_ADAPTER   * a, byte   * bc, byte   * hlc)
>  {
>   word i;
>   word j;
> @@ -9068,7 +9068,7 @@ static byte AddInfo(byte   **add_i,
>  /* voice and codec features                                         */
>  /*------------------------------------------------------------------*/
>
> -void SetVoiceChannel(PLCI   *plci, byte   *chi, DIVA_CAPI_ADAPTER   * a)
> +static void SetVoiceChannel(PLCI   *plci, byte   *chi, DIVA_CAPI_ADAPTER   * a)
>  {
>   byte voice_chi[] = "\x02\x18\x01";
>   byte channel;
> @@ -9086,7 +9086,7 @@ void SetVoiceChannel(PLCI   *plci, byte   *chi, DIVA_CAPI_ADAPTER   * a)
>   }
>  }
>
> -void VoiceChannelOff(PLCI   *plci)
> +static void VoiceChannelOff(PLCI   *plci)
>  {
>   dbug(1,dprintf("ExtDevOFF"));
>   add_p(plci,FTY,"\x02\x01\x08");             /* B Off */
> @@ -9099,7 +9099,7 @@ void VoiceChannelOff(PLCI   *plci)
>  }
>
>
> -word AdvCodecSupport(DIVA_CAPI_ADAPTER   *a, PLCI   *plci, APPL   *appl, byte hook_listen)
> +static word AdvCodecSupport(DIVA_CAPI_ADAPTER   *a, PLCI   *plci, APPL   *appl, byte hook_listen)
>  {
>   word j;
>   PLCI   *splci;
> @@ -9195,7 +9195,7 @@ word AdvCodecSupport(DIVA_CAPI_ADAPTER   *a, PLCI   *plci, APPL   *appl, byte ho
>  }
>
>
> -void CodecIdCheck(DIVA_CAPI_ADAPTER   *a, PLCI   *plci)
> +static void CodecIdCheck(DIVA_CAPI_ADAPTER   *a, PLCI   *plci)
>  {
>
>   dbug(1,dprintf("CodecIdCheck"));
> --
> 1.5.6.3
>
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/isdn/hardware/eicon/di.c b/drivers/isdn/hardware/eicon/di.c
index 10760b3..d798fb4 100644
--- a/drivers/isdn/hardware/eicon/di.c
+++ b/drivers/isdn/hardware/eicon/di.c
@@ -353,13 +353,13 @@  void scom_clear_int(ADAPTER * a)
 /*------------------------------------------------------------------*/
 /* return code handler                                              */
 /*------------------------------------------------------------------*/
-byte isdn_rc(ADAPTER * a,
-             byte Rc,
-             byte Id,
-             byte Ch,
-             word Ref,
-             dword extended_info_type,
-             dword extended_info)
+static byte isdn_rc(ADAPTER * a,
+                    byte Rc,
+                    byte Id,
+                    byte Ch,
+                    word Ref,
+                    dword extended_info_type,
+                    dword extended_info)
 {
   ENTITY  * this;
   byte e_no;
@@ -555,13 +555,13 @@  byte isdn_rc(ADAPTER * a,
 /*------------------------------------------------------------------*/
 /* indication handler                                               */
 /*------------------------------------------------------------------*/
-byte isdn_ind(ADAPTER * a,
-              byte Ind,
-              byte Id,
-              byte Ch,
-              PBUFFER * RBuffer,
-              byte MInd,
-              word MLength)
+static byte isdn_ind(ADAPTER * a,
+                     byte Ind,
+                     byte Id,
+                     byte Ch,
+                     PBUFFER * RBuffer,
+                     byte MInd,
+                     word MLength)
 {
   ENTITY  * this;
   word clength;
diff --git a/drivers/isdn/hardware/eicon/message.c b/drivers/isdn/hardware/eicon/message.c
index 599fed8..339e4ed 100644
--- a/drivers/isdn/hardware/eicon/message.c
+++ b/drivers/isdn/hardware/eicon/message.c
@@ -592,7 +592,7 @@  word api_put(APPL   * appl, CAPI_MSG   * msg)
 /* api_parse function, check the format of api messages             */
 /*------------------------------------------------------------------*/
 
-word api_parse(byte   * msg, word length, byte * format, API_PARSE * parms)
+static word api_parse(byte   * msg, word length, byte * format, API_PARSE * parms)
 {
   word i;
   word p;
@@ -631,7 +631,7 @@  word api_parse(byte   * msg, word length, byte * format, API_PARSE * parms)
   return false;
 }
 
-void api_save_msg(API_PARSE   *in, byte *format, API_SAVE   *out)
+static void api_save_msg(API_PARSE   *in, byte *format, API_SAVE   *out)
 {
   word i, j, n = 0;
   byte   *p;
@@ -663,7 +663,7 @@  void api_save_msg(API_PARSE   *in, byte *format, API_SAVE   *out)
   out->parms[i].length = 0;
 }
 
-void api_load_msg(API_SAVE   *in, API_PARSE   *out)
+static void api_load_msg(API_SAVE   *in, API_PARSE   *out)
 {
   word i;
 
@@ -3414,7 +3414,7 @@  byte select_b_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci,
   return false;
 }
 
-byte manufacturer_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * parms)
+static byte manufacturer_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * parms)
 {
   word command;
   word i;
@@ -3742,7 +3742,7 @@  byte manufacturer_req(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * p
 }
 
 
-byte manufacturer_res(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * msg)
+static byte manufacturer_res(dword Id, word Number, DIVA_CAPI_ADAPTER   * a, PLCI   * plci, APPL   * appl, API_PARSE * msg)
 {
   word indication;
 
@@ -4074,7 +4074,7 @@  capi_callback_suffix:
 }
 
 
-void control_rc(PLCI   * plci, byte req, byte rc, byte ch, byte global_req, byte nl_rc)
+static void control_rc(PLCI   * plci, byte req, byte rc, byte ch, byte global_req, byte nl_rc)
 {
   dword Id;
   dword rId;
@@ -4740,7 +4740,7 @@  void control_rc(PLCI   * plci, byte req, byte rc, byte ch, byte global_req, byte
   }
 }
 
-void data_rc(PLCI   * plci, byte ch)
+static void data_rc(PLCI   * plci, byte ch)
 {
   dword Id;
   DIVA_CAPI_ADAPTER   * a;
@@ -4776,7 +4776,7 @@  void data_rc(PLCI   * plci, byte ch)
   }
 }
 
-void data_ack(PLCI   * plci, byte ch)
+static void data_ack(PLCI   * plci, byte ch)
 {
   dword Id;
   DIVA_CAPI_ADAPTER   * a;
@@ -4802,7 +4802,7 @@  void data_ack(PLCI   * plci, byte ch)
   }
 }
 
-void sig_ind(PLCI   * plci)
+static void sig_ind(PLCI   * plci)
 {
   dword x_Id;
   dword Id;
@@ -6170,7 +6170,7 @@  static void SendSetupInfo(APPL   * appl, PLCI   * plci, dword Id, byte   * * par
 }
 
 
-void SendInfo(PLCI   * plci, dword Id, byte   * * parms, byte iesent)
+static void SendInfo(PLCI   * plci, dword Id, byte   * * parms, byte iesent)
 {
   word i;
   word j;
@@ -6346,7 +6346,7 @@  void SendInfo(PLCI   * plci, dword Id, byte   * * parms, byte iesent)
 }
 
 
-byte SendMultiIE(PLCI   * plci, dword Id, byte   * * parms, byte ie_type, dword info_mask, byte setupParse)
+static byte SendMultiIE(PLCI   * plci, dword Id, byte   * * parms, byte ie_type, dword info_mask, byte setupParse)
 {
   word i;
   word j;
@@ -6465,7 +6465,7 @@  static void SendSSExtInd(APPL   * appl, PLCI   * plci, dword Id, byte   * * parm
     }
 };
 
-void nl_ind(PLCI   * plci)
+static void nl_ind(PLCI   * plci)
 {
   byte ch;
   word ncci;
@@ -7247,7 +7247,7 @@  void nl_ind(PLCI   * plci)
 /* find a free PLCI                                                 */
 /*------------------------------------------------------------------*/
 
-word get_plci(DIVA_CAPI_ADAPTER   * a)
+static word get_plci(DIVA_CAPI_ADAPTER   * a)
 {
   word i,j;
   PLCI   * plci;
@@ -7406,7 +7406,7 @@  static void add_ie(PLCI   * plci, byte code, byte   * p, word p_length)
 /* put a unstructured data into the buffer                          */
 /*------------------------------------------------------------------*/
 
-void add_d(PLCI   * plci, word length, byte   * p)
+static void add_d(PLCI   * plci, word length, byte   * p)
 {
   word i;
 
@@ -7424,7 +7424,7 @@  void add_d(PLCI   * plci, word length, byte   * p)
 /* parameter buffer                                                 */
 /*------------------------------------------------------------------*/
 
-void add_ai(PLCI   * plci, API_PARSE * ai)
+static void add_ai(PLCI   * plci, API_PARSE * ai)
 {
   word i;
     API_PARSE ai_parms[5];
@@ -7445,7 +7445,7 @@  void add_ai(PLCI   * plci, API_PARSE * ai)
 /* put parameter for b1 protocol in the parameter buffer            */
 /*------------------------------------------------------------------*/
 
-word add_b1(PLCI   * plci, API_PARSE * bp, word b_channel_info, word b1_facilities)
+static word add_b1(PLCI   * plci, API_PARSE * bp, word b_channel_info, word b1_facilities)
 {
     API_PARSE bp_parms[8];
     API_PARSE mdm_cfg[9];
@@ -7909,7 +7909,7 @@  word add_b1(PLCI   * plci, API_PARSE * bp, word b_channel_info, word b1_faciliti
 /* put parameter for b2 and B3  protocol in the parameter buffer    */
 /*------------------------------------------------------------------*/
 
-word add_b23(PLCI   * plci, API_PARSE * bp)
+static word add_b23(PLCI   * plci, API_PARSE * bp)
 {
   word i, fax_control_bits;
   byte pos, len;
@@ -8706,7 +8706,7 @@  void sig_req(PLCI   * plci, byte req, byte Id)
 /* send a request for the network layer entity                      */
 /*------------------------------------------------------------------*/
 
-void nl_req_ncci(PLCI   * plci, byte req, byte ncci)
+static void nl_req_ncci(PLCI   * plci, byte req, byte ncci)
 {
   if(!plci) return;
   if(plci->adapter->adapter_disabled) return;
@@ -8728,7 +8728,7 @@  void nl_req_ncci(PLCI   * plci, byte req, byte ncci)
   plci->req_in_start = plci->req_in;
 }
 
-void send_req(PLCI   * plci)
+static void send_req(PLCI   * plci)
 {
   ENTITY   * e;
   word l;
@@ -8863,7 +8863,7 @@  void send_data(PLCI   * plci)
   }
 }
 
-void listen_check(DIVA_CAPI_ADAPTER   * a)
+static void listen_check(DIVA_CAPI_ADAPTER   * a)
 {
   word i,j;
   PLCI   * plci;
@@ -8906,7 +8906,7 @@  void listen_check(DIVA_CAPI_ADAPTER   * a)
 /* functions for all parameters sent in INDs                        */
 /*------------------------------------------------------------------*/
 
-void IndParse(PLCI   * plci, word * parms_id, byte   ** parms, byte multiIEsize)
+static void IndParse(PLCI   * plci, word * parms_id, byte   ** parms, byte multiIEsize)
 {
   word ploc;            /* points to current location within packet */
   byte w;
@@ -8991,7 +8991,7 @@  void IndParse(PLCI   * plci, word * parms_id, byte   ** parms, byte multiIEsize)
 /* try to match a cip from received BC and HLC                      */
 /*------------------------------------------------------------------*/
 
-byte ie_compare(byte   * ie1, byte * ie2)
+static byte ie_compare(byte   * ie1, byte * ie2)
 {
   word i;
   if(!ie1 || ! ie2) return false;
@@ -9000,7 +9000,7 @@  byte ie_compare(byte   * ie1, byte * ie2)
   return true;
 }
 
-word find_cip(DIVA_CAPI_ADAPTER   * a, byte   * bc, byte   * hlc)
+static word find_cip(DIVA_CAPI_ADAPTER   * a, byte   * bc, byte   * hlc)
 {
   word i;
   word j;
@@ -9068,7 +9068,7 @@  static byte AddInfo(byte   **add_i,
 /* voice and codec features                                         */
 /*------------------------------------------------------------------*/
 
-void SetVoiceChannel(PLCI   *plci, byte   *chi, DIVA_CAPI_ADAPTER   * a)
+static void SetVoiceChannel(PLCI   *plci, byte   *chi, DIVA_CAPI_ADAPTER   * a)
 {
   byte voice_chi[] = "\x02\x18\x01";
   byte channel;
@@ -9086,7 +9086,7 @@  void SetVoiceChannel(PLCI   *plci, byte   *chi, DIVA_CAPI_ADAPTER   * a)
   }
 }
 
-void VoiceChannelOff(PLCI   *plci)
+static void VoiceChannelOff(PLCI   *plci)
 {
   dbug(1,dprintf("ExtDevOFF"));
   add_p(plci,FTY,"\x02\x01\x08");             /* B Off */
@@ -9099,7 +9099,7 @@  void VoiceChannelOff(PLCI   *plci)
 }
 
 
-word AdvCodecSupport(DIVA_CAPI_ADAPTER   *a, PLCI   *plci, APPL   *appl, byte hook_listen)
+static word AdvCodecSupport(DIVA_CAPI_ADAPTER   *a, PLCI   *plci, APPL   *appl, byte hook_listen)
 {
   word j;
   PLCI   *splci;
@@ -9195,7 +9195,7 @@  word AdvCodecSupport(DIVA_CAPI_ADAPTER   *a, PLCI   *plci, APPL   *appl, byte ho
 }
 
 
-void CodecIdCheck(DIVA_CAPI_ADAPTER   *a, PLCI   *plci)
+static void CodecIdCheck(DIVA_CAPI_ADAPTER   *a, PLCI   *plci)
 {
 
   dbug(1,dprintf("CodecIdCheck"));