OSDN Git Service

staging: rtl8192u: make in ieee80211/ieee80211_rx.c some local functions static
authorAna Rey <anarey@gmail.com>
Fri, 14 Mar 2014 11:34:43 +0000 (12:34 +0100)
committerPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
Mon, 17 Mar 2014 01:16:44 +0000 (18:16 -0700)
Make some local functions (AddReorderEntry, RxReorderIndicate and
parse_subframe) static and fix coding style in this function
declarations when It is necessary.

Fix the following sparse warnings in ieee80211/ieee80211_rx.c

drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:521:1: warning: symbol 'AddReorderEntry' was not declared. Should it be static?
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:604:6: warning: symbol 'RxReorderIndicatePacket' was not declared. Should it be static?
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:774:4: warning: symbol 'parse_subframe' was not declared. Should it be static?

Signed-off-by: Ana Rey <anarey@gmail.com>
Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c

index fc5b848..a98414a 100644 (file)
@@ -517,11 +517,8 @@ drop:
 
        return 1;
 }
-bool
-AddReorderEntry(
-       PRX_TS_RECORD                   pTS,
-       PRX_REORDER_ENTRY               pReorderEntry
-       )
+
+static bool AddReorderEntry(PRX_TS_RECORD pTS, PRX_REORDER_ENTRY pReorderEntry)
 {
        struct list_head *pList = &pTS->RxPendingPktList;
        while(pList->next != &pTS->RxPendingPktList)
@@ -601,10 +598,9 @@ void ieee80211_indicate_packets(struct ieee80211_device *ieee, struct ieee80211_
 }
 
 
-void RxReorderIndicatePacket( struct ieee80211_device *ieee,
-               struct ieee80211_rxb *prxb,
-               PRX_TS_RECORD           pTS,
-               u16                     SeqNum)
+static void RxReorderIndicatePacket(struct ieee80211_device *ieee,
+                                   struct ieee80211_rxb *prxb,
+                                   PRX_TS_RECORD pTS, u16 SeqNum)
 {
        PRT_HIGH_THROUGHPUT     pHTInfo = ieee->pHTInfo;
        PRX_REORDER_ENTRY       pReorderEntry = NULL;
@@ -771,9 +767,9 @@ void RxReorderIndicatePacket( struct ieee80211_device *ieee,
        }
 }
 
-u8 parse_subframe(struct sk_buff *skb,
-                 struct ieee80211_rx_stats *rx_stats,
-                 struct ieee80211_rxb *rxb,u8 *src,u8 *dst)
+static u8 parse_subframe(struct sk_buff *skb,
+                        struct ieee80211_rx_stats *rx_stats,
+                        struct ieee80211_rxb *rxb,u8 *src,u8 *dst)
 {
        struct ieee80211_hdr_3addr  *hdr = (struct ieee80211_hdr_3addr *)skb->data;
        u16             fc = le16_to_cpu(hdr->frame_ctl);