bna: fix sparse warnings/errors
authorstephen hemminger <shemminger@vyatta.com>
Wed, 4 Jan 2012 13:02:24 +0000 (13:02 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 5 Jan 2012 18:23:00 +0000 (13:23 -0500)
This fixes a several sparse warnings.
  * the __iomem tag was being used incorrectly (needs to be a prefix)
  * several variables should have been static since local to one file
  * the firmware was not being forwared declared
    and was const one place and not the other

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/brocade/bna/bna_types.h
drivers/net/ethernet/brocade/bna/bnad.c
drivers/net/ethernet/brocade/bna/bnad.h
drivers/net/ethernet/brocade/bna/cna_fwimg.c

index 8e57fc5c586894d00b9ce6af8b145a0bf6b3b03c..e8d3ab7ea6cbb8b153992b0a151c42b1c5abc6f3 100644 (file)
@@ -427,7 +427,7 @@ struct bna_ethport {
 
 /* Doorbell structure */
 struct bna_ib_dbell {
-       void *__iomem doorbell_addr;
+       void __iomem   *doorbell_addr;
        u32             doorbell_ack;
 };
 
@@ -463,7 +463,7 @@ struct bna_tcb {
        u32             consumer_index;
        volatile u32    *hw_consumer_index;
        u32             q_depth;
-       void *__iomem q_dbell;
+       void __iomem   *q_dbell;
        struct bna_ib_dbell *i_dbell;
        int                     page_idx;
        int                     page_count;
@@ -599,7 +599,7 @@ struct bna_rcb {
        u32             producer_index;
        u32             consumer_index;
        u32             q_depth;
-       void *__iomem q_dbell;
+       void __iomem   *q_dbell;
        int                     page_idx;
        int                     page_count;
        /* Control path */
index 2eddbaa5db47a8266a1362117f2c55afe7f1cdcd..be7d91e4b7852701005947bcefe09aa5bf1d39f3 100644 (file)
@@ -53,9 +53,9 @@ MODULE_PARM_DESC(bna_debugfs_enable, "Enables debugfs feature, default=1,"
  * Global variables
  */
 u32 bnad_rxqs_per_cq = 2;
-u32 bna_id;
-struct mutex bnad_list_mutex;
-LIST_HEAD(bnad_list);
+static u32 bna_id;
+static struct mutex bnad_list_mutex;
+static LIST_HEAD(bnad_list);
 static const u8 bnad_bcast_addr[] =  {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};
 
 /*
index c975ce672f480ab48852939607ccdfe1b1714c49..55824d92699f43cb390c860ce3295f9275f858f6 100644 (file)
@@ -347,7 +347,7 @@ struct bnad_drvinfo {
 /*
  * EXTERN VARIABLES
  */
-extern struct firmware *bfi_fw;
+extern const struct firmware *bfi_fw;
 extern u32             bnad_rxqs_per_cq;
 
 /*
index 725b9fff337f8bbd5d3c95358bee976521147318..cfc22a64157e4962f324cc6dbae101d2d73404ca 100644 (file)
@@ -16,6 +16,7 @@
  * www.brocade.com
  */
 #include <linux/firmware.h>
+#include "bnad.h"
 #include "bfi.h"
 #include "cna.h"