Skip to contentSkip to conversation

libertas: drop useless default_fw_name variable

PATCH In review Andres Salomon 2 participants 2 messages Nov 16, 2007 – Nov 20, 2007 Networking drivers (wireless)

lore
Share & tools
lore
#0 lore raw

PATCH 1/4 libertas: drop useless default_fw_name variable

---
 drivers/net/wireless/libertas/if_usb.c |    7 +------
 1 files changed, 1 insertions(+), 6 deletions(-)
drivers/net/wireless/libertas/if_usb.c +1 -6
index 063572f..3cf9379 100644
@@ -20,9 +20,8 @@
 #define MESSAGE_HEADER_LEN	4
 
 static const char usbdriver_name[] = "usb8xxx";
-static u8 *default_fw_name = "usb8388.bin";
 
-static char *libertas_fw_name = NULL;
+static char *libertas_fw_name = "usb8388.bin";
 module_param_named(fw_name, libertas_fw_name, charp, 0644);
 
 /*
@@ -1129,10 +1128,6 @@ static int if_usb_init_module(void)
 
 	lbs_deb_enter(LBS_DEB_MAIN);
 
-	if (libertas_fw_name == NULL) {
-		libertas_fw_name = default_fw_name;
-	}
-
 	ret = usb_register(&if_usb_driver);
 
 	lbs_deb_leave_args(LBS_DEB_MAIN, "ret %d", ret);
Signature · 1 line
1.5.3.4
#1 lore raw
Quoted 1 line · Andres Salomon

As discussed on IRC, I'll clean this series of 4 patches up for 2.6.25 and the unified libertas namespace and resubmit. Thanks!

Dan

Quoted 30 lines
---
 drivers/net/wireless/libertas/if_usb.c |    7 +------
 1 files changed, 1 insertions(+), 6 deletions(-)
drivers/net/wireless/libertas/if_usb.c +1 -6
index 063572f..3cf9379 100644
@@ -20,9 +20,8 @@
 #define MESSAGE_HEADER_LEN	4
 
 static const char usbdriver_name[] = "usb8xxx";
-static u8 *default_fw_name = "usb8388.bin";
 
-static char *libertas_fw_name = NULL;
+static char *libertas_fw_name = "usb8388.bin";
 module_param_named(fw_name, libertas_fw_name, charp, 0644);
 
 /*
@@ -1129,10 +1128,6 @@ static int if_usb_init_module(void)
 
 	lbs_deb_enter(LBS_DEB_MAIN);
 
-	if (libertas_fw_name == NULL) {
-		libertas_fw_name = default_fw_name;
-	}
-
 	ret = usb_register(&if_usb_driver);
 
 	lbs_deb_leave_args(LBS_DEB_MAIN, "ret %d", ret);