From: Hannes Eder <hannes@hanneseder.net>
To: netdev@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 28/43] drivers/net/skfp: fix sparse warning: Should it be static?
Date: Sat, 14 Feb 2009 22:43:15 +0100 [thread overview]
Message-ID: <20090214214210.24377.32244.stgit@vmbox.hanneseder.net> (raw)
In-Reply-To: <20090214210940.23489.95001.stgit@vmbox.hanneseder.net>
Impact: Move function declarations to header file.
Fix this sparse warnings:
drivers/net/skfp/cfm.c:146:6: warning: symbol 'all_selection_criteria' was not declared. Should it be static?
drivers/net/skfp/drvfbi.c:186:6: warning: symbol 'mac1_irq' was not declared. Should it be static?
drivers/net/skfp/drvfbi.c:284:6: warning: symbol 'read_address' was not declared. Should it be static?
drivers/net/skfp/drvfbi.c:323:6: warning: symbol 'init_board' was not declared. Should it be static?
drivers/net/skfp/fplustm.c:72:24: warning: symbol 'fddi_broadcast' was not declared. Should it be static?
drivers/net/skfp/fplustm.c:679:6: warning: symbol 'mac2_irq' was not declared. Should it be static?
drivers/net/skfp/fplustm.c:805:6: warning: symbol 'mac3_irq' was not declared. Should it be static?
drivers/net/skfp/fplustm.c:856:5: warning: symbol 'init_fplus' was not declared. Should it be static?
drivers/net/skfp/pcmplc.c:404:6: warning: symbol 'init_plc' was not declared. Should it be static?
drivers/net/skfp/pcmplc.c:1592:5: warning: symbol 'pcm_status_twisted' was not declared. Should it be static?
drivers/net/skfp/smtinit.c:68:5: warning: symbol 'init_smt' was not declared. Should it be static?
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
drivers/net/skfp/h/smc.h | 17 +++++++++++++++++
drivers/net/skfp/hwmtm.c | 9 ---------
drivers/net/skfp/pcmplc.c | 3 ---
drivers/net/skfp/skfddi.c | 3 ---
drivers/net/skfp/smt.c | 10 ----------
5 files changed, 17 insertions(+), 25 deletions(-)
diff --git a/drivers/net/skfp/h/smc.h b/drivers/net/skfp/h/smc.h
index 9432591..1758d95 100644
--- a/drivers/net/skfp/h/smc.h
+++ b/drivers/net/skfp/h/smc.h
@@ -467,5 +467,22 @@ struct s_smc {
#endif /* DEBUG_BRD && DEBUG */
} ;
+extern const struct fddi_addr fddi_broadcast;
+
+extern void all_selection_criteria(struct s_smc *smc);
+extern void card_stop(struct s_smc *smc);
+extern void init_board(struct s_smc *smc, u_char *mac_addr);
+extern int init_fplus(struct s_smc *smc);
+extern void init_plc(struct s_smc *smc);
+extern int init_smt(struct s_smc *smc, u_char * mac_addr);
+extern void mac1_irq(struct s_smc *smc, u_short stu, u_short stl);
+extern void mac2_irq(struct s_smc *smc, u_short code_s2u, u_short code_s2l);
+extern void mac3_irq(struct s_smc *smc, u_short code_s3u, u_short code_s3l);
+extern int pcm_status_twisted(struct s_smc *smc);
+extern void plc1_irq(struct s_smc *smc);
+extern void plc2_irq(struct s_smc *smc);
+extern void read_address(struct s_smc *smc, u_char * mac_addr);
+extern void timer_irq(struct s_smc *smc);
+
#endif /* _SCMECM_ */
diff --git a/drivers/net/skfp/hwmtm.c b/drivers/net/skfp/hwmtm.c
index 4218e97..d322f1b 100644
--- a/drivers/net/skfp/hwmtm.c
+++ b/drivers/net/skfp/hwmtm.c
@@ -97,23 +97,15 @@ static void mac_drv_clear_txd(struct s_smc *smc);
extern void* mac_drv_get_space(struct s_smc *smc, unsigned int size);
extern void* mac_drv_get_desc_mem(struct s_smc *smc, unsigned int size);
-extern void init_board(struct s_smc *smc, u_char *mac_addr);
extern void mac_drv_fill_rxd(struct s_smc *smc);
-extern void plc1_irq(struct s_smc *smc);
extern void mac_drv_tx_complete(struct s_smc *smc,
volatile struct s_smt_fp_txd *txd);
-extern void plc2_irq(struct s_smc *smc);
-extern void mac1_irq(struct s_smc *smc, u_short stu, u_short stl);
-extern void mac2_irq(struct s_smc *smc, u_short code_s2u, u_short code_s2l);
-extern void mac3_irq(struct s_smc *smc, u_short code_s3u, u_short code_s3l);
-extern void timer_irq(struct s_smc *smc);
extern void mac_drv_rx_complete(struct s_smc *smc,
volatile struct s_smt_fp_rxd *rxd,
int frag_count, int len);
extern void mac_drv_requeue_rxd(struct s_smc *smc,
volatile struct s_smt_fp_rxd *rxd,
int frag_count);
-extern void init_plc(struct s_smc *smc);
extern void mac_drv_clear_rxd(struct s_smc *smc,
volatile struct s_smt_fp_rxd *rxd, int frag_count);
@@ -136,7 +128,6 @@ extern void dma_complete(struct s_smc *smc, volatile union s_fp_descr *descr,
int flag);
#endif
-extern int init_fplus(struct s_smc *smc);
extern int mac_drv_rx_init(struct s_smc *smc, int len, int fc, char *look_ahead,
int la_len);
diff --git a/drivers/net/skfp/pcmplc.c b/drivers/net/skfp/pcmplc.c
index 74e129f..f1df2ec 100644
--- a/drivers/net/skfp/pcmplc.c
+++ b/drivers/net/skfp/pcmplc.c
@@ -198,9 +198,6 @@ static int plc_imsk_na = PL_PCM_CODE | PL_TRACE_PROP | PL_PCM_BREAK |
static const int plc_imsk_act = PL_PCM_CODE | PL_TRACE_PROP | PL_PCM_BREAK |
PL_PCM_ENABLED | PL_SELF_TEST | PL_EBUF_ERR;
-/* external functions */
-void all_selection_criteria(struct s_smc *smc);
-
/* internal functions */
static void pcm_fsm(struct s_smc *smc, struct s_phy *phy, int cmd);
static void pc_rcode_actions(struct s_smc *smc, int bit, struct s_phy *phy);
diff --git a/drivers/net/skfp/skfddi.c b/drivers/net/skfp/skfddi.c
index 9a00e55..e14aec0 100644
--- a/drivers/net/skfp/skfddi.c
+++ b/drivers/net/skfp/skfddi.c
@@ -135,14 +135,11 @@ void dump_data(unsigned char *Data, int length);
// External functions from the hardware module
extern u_int mac_drv_check_space(void);
-extern void read_address(struct s_smc *smc, u_char * mac_addr);
-extern void card_stop(struct s_smc *smc);
extern int mac_drv_init(struct s_smc *smc);
extern void hwm_tx_frag(struct s_smc *smc, char far * virt, u_long phys,
int len, int frame_status);
extern int hwm_tx_init(struct s_smc *smc, u_char fc, int frag_count,
int frame_len, int frame_status);
-extern int init_smt(struct s_smc *smc, u_char * mac_addr);
extern void fddi_isr(struct s_smc *smc);
extern void hwm_rx_frag(struct s_smc *smc, char far * virt, u_long phys,
int len, int frame_status);
diff --git a/drivers/net/skfp/smt.c b/drivers/net/skfp/smt.c
index 805383b..83d16fe 100644
--- a/drivers/net/skfp/smt.c
+++ b/drivers/net/skfp/smt.c
@@ -55,16 +55,6 @@ static const struct fddi_addr SMT_Unknown = {
} ;
/*
- * external variables
- */
-extern const struct fddi_addr fddi_broadcast ;
-
-/*
- * external functions
- */
-int pcm_status_twisted(struct s_smc *smc);
-
-/*
* function prototypes
*/
#ifdef LITTLE_ENDIAN
next prev parent reply other threads:[~2009-02-14 21:43 UTC|newest]
Thread overview: 113+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-14 21:11 [PATCH 00/43] drivers/net: fix (sparse) warnings Hannes Eder
2009-02-14 21:11 ` [PATCH 01/43] drivers/net/at1700.c: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:21 ` David Miller
2009-02-14 21:12 ` [PATCH 02/43] drivers/net/e2100.c: " Hannes Eder
2009-02-15 14:04 ` walter harms
2009-02-15 19:57 ` Hannes Eder
2009-02-16 9:58 ` walter harms
2009-02-18 1:21 ` David Miller
2009-02-14 21:12 ` [PATCH 03/43] drivers/net/ni65.c: fix sparse warnings: " Hannes Eder
2009-02-18 1:21 ` David Miller
2009-02-14 21:12 ` [PATCH 04/43] drivers/net/ns83820.c: fix sparse warnings: fix signedness Hannes Eder
2009-02-18 1:21 ` David Miller
2009-02-14 21:12 ` [PATCH 05/43] drivers/net/niu.c: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:21 ` David Miller
2009-02-14 21:13 ` [PATCH 06/43] drivers/net/lance.c: fix sparse warnings: fix signedness Hannes Eder
2009-02-18 1:21 ` David Miller
2009-02-14 21:13 ` [PATCH 07/43] drivers/net/ni65.c: " Hannes Eder
2009-02-18 1:21 ` David Miller
2009-02-14 21:13 ` [PATCH 08/43] drivers/net/ppp*.c: " Hannes Eder
2009-02-14 21:29 ` Cyrill Gorcunov
2009-02-14 21:38 ` Cyrill Gorcunov
2009-02-14 21:57 ` Hannes Eder
2009-02-15 8:08 ` Cyrill Gorcunov
2009-02-18 1:22 ` David Miller
2009-02-14 21:14 ` [PATCH 09/43] drivers/net/r6040.c: fix sparse warning: Using plain integer as NULL pointer Hannes Eder
2009-02-15 14:00 ` Florian Fainelli
2009-02-18 1:22 ` David Miller
2009-02-14 21:14 ` [PATCH 10/43] drivers/net/s2io.c: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:22 ` David Miller
2009-02-18 1:26 ` David Miller
2009-02-14 21:14 ` [PATCH 11/43] drivers/net/arcnet: " Hannes Eder
2009-02-18 1:26 ` David Miller
2009-02-14 21:14 ` [PATCH 12/43] drivers/net/atl1e: fix sparse warnings: fix signedness Hannes Eder
2009-02-18 1:26 ` David Miller
2009-02-14 21:15 ` [PATCH 13/43] drivers/net/atlx: " Hannes Eder
2009-02-16 2:20 ` Jay Cliburn
2009-02-18 1:26 ` David Miller
2009-02-14 21:15 ` [PATCH 14/43] drivers/net/bonding: fix sparse warnings: context imbalance Hannes Eder
2009-02-18 1:27 ` David Miller
2009-02-14 21:15 ` [PATCH 15/43] drivers/net/bonding: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:27 ` David Miller
2009-02-14 21:16 ` [PATCH 16/43] drivers/net/cxgb3: " Hannes Eder
2009-02-15 14:14 ` walter harms
2009-02-15 20:10 ` Hannes Eder
2009-02-15 21:30 ` Divy Le Ray
2009-02-18 1:27 ` David Miller
2009-02-14 21:16 ` [PATCH 17/43] drivers/net/cxgb3: fix sparse warnings: fix signedness Hannes Eder
2009-02-15 21:32 ` Divy Le Ray
2009-02-18 1:27 ` David Miller
2009-02-14 21:32 ` [PATCH 18/43] drivers/net/e1000e: fix sparse warning: Should it be static? Hannes Eder
2009-02-18 1:27 ` David Miller
2009-02-14 21:33 ` [PATCH 19/43] drivers/net/hamradio: fix warning: format not a string literal and no Hannes Eder
2009-02-18 1:27 ` David Miller
2009-02-14 21:34 ` [PATCH 20/43] drivers/net/hamradio: fix sparse warning: context imbalance Hannes Eder
2009-02-18 1:38 ` David Miller
2009-02-14 21:35 ` [PATCH 21/43] drivers/net/hamradio: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:38 ` David Miller
2009-02-14 21:36 ` [PATCH 22/43] drivers/net/hamradio: fix sparse warnings: fix signedness Hannes Eder
2009-02-18 1:38 ` David Miller
2009-02-14 21:38 ` [PATCH 23/43] drivers/net/igb: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:39 ` David Miller
2009-02-18 21:49 ` [PATCH 23/43 v2] " Hannes Eder
2009-02-19 3:36 ` David Miller
2009-02-14 21:38 ` [PATCH 24/43] drivers/net/ixgbe: fix sparse warning: Should it be static? Hannes Eder
2009-02-16 6:48 ` Waskiewicz Jr, Peter P
2009-02-18 1:39 ` David Miller
2009-02-14 21:39 ` [PATCH 25/43] drivers/net/pcmcia: fix warning: format not a string literal and no Hannes Eder
2009-02-18 1:39 ` David Miller
2009-02-14 21:41 ` [PATCH 26/43] drivers/net/sfc: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-16 0:38 ` Ben Hutchings
2009-02-18 1:39 ` David Miller
2009-02-14 21:42 ` [PATCH 27/43] drivers/net/sfc: fix sparse warnings: Should it be static? Hannes Eder
2009-02-16 0:37 ` Ben Hutchings
2009-02-16 9:38 ` Hannes Eder
2009-02-16 9:54 ` [PATCH] " Hannes Eder
2009-02-19 1:49 ` David Miller
2009-02-18 1:39 ` [PATCH 27/43] " David Miller
2009-02-14 21:43 ` Hannes Eder [this message]
2009-02-18 1:40 ` [PATCH 28/43] drivers/net/skfp: fix sparse warning: " David Miller
2009-02-14 21:44 ` [PATCH 29/43] drivers/net/tokenring: fix warning: format not a string literal and no Hannes Eder
2009-02-18 1:40 ` David Miller
2009-02-14 21:44 ` [PATCH 30/43] drivers/net/tokenring: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 1:41 ` David Miller
2009-02-14 21:46 ` [PATCH 31/43] drivers/net/tokenring: fix sparse warning: cast truncates bits from const value Hannes Eder
2009-02-14 23:24 ` Al Viro
2009-02-15 11:47 ` Hannes Eder
2009-02-15 12:13 ` [PATCH 31/43 v2] " Hannes Eder
2009-02-16 17:29 ` Hannes Eder
2009-02-18 3:49 ` David Miller
2009-02-14 21:46 ` [PATCH 32/43] drivers/net/tulip: fix warning: format not a string literal and no Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:47 ` [PATCH 33/43] drivers/net/tulip: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:47 ` [PATCH 34/43] drivers/net/usb: fix sparse warnings: Should it be static? Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:48 ` [PATCH 35/43] drivers/net/wan: fix warning: format not a string literal and no Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:48 ` [PATCH 36/43] drivers/net/wan: fix sparse warning: symbol shadows an earlier one Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:48 ` [PATCH 37/43] drivers/net/wireless: fix sparse warning: context imbalance Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:49 ` [PATCH 38/43] drivers/net/wireless: fix sparse warnings: symbol shadows an earlier one Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:49 ` [PATCH 39/43] drivers/net/wireless: fix sparse warnings: fix signedness Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:49 ` [PATCH 40/43] drivers/net/wireless/ath9k: fix sparse warnings: Should it be static? Hannes Eder
2009-02-18 3:35 ` David Miller
2009-02-14 21:50 ` [PATCH 41/43] drivers/net/wireless/b43: fix warning: format not a string literal and no Hannes Eder
2009-02-18 3:49 ` David Miller
2009-02-14 21:50 ` [PATCH 42/43] drivers/net/wireless/ipw2x00: fix sparse warnings: fix signedness Hannes Eder
2009-02-18 3:49 ` David Miller
2009-02-14 21:50 ` [PATCH 43/43] drivers/net/wireless/prism54: fix sparse warning: " Hannes Eder
2009-02-18 3:49 ` David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090214214210.24377.32244.stgit@vmbox.hanneseder.net \
--to=hannes@hanneseder.net \
--cc=kernel-janitors@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®