Commit 34336ec0 authored by Ben Hutchings's avatar Ben Hutchings Committed by David S. Miller

cxgb3: declare MODULE_FIRMWARE

Replace run-time string formatting with preprocessor string
manipulation.
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
Acked-by: default avatarDivy Le Ray <divy@chelsio.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 45229b42
...@@ -125,11 +125,9 @@ enum { /* adapter interrupt-maintained statistics */ ...@@ -125,11 +125,9 @@ enum { /* adapter interrupt-maintained statistics */
IRQ_NUM_STATS /* keep last */ IRQ_NUM_STATS /* keep last */
}; };
enum { #define TP_VERSION_MAJOR 1
TP_VERSION_MAJOR = 1, #define TP_VERSION_MINOR 1
TP_VERSION_MINOR = 1, #define TP_VERSION_MICRO 0
TP_VERSION_MICRO = 0
};
#define S_TP_VERSION_MAJOR 16 #define S_TP_VERSION_MAJOR 16
#define M_TP_VERSION_MAJOR 0xFF #define M_TP_VERSION_MAJOR 0xFF
......
...@@ -44,6 +44,7 @@ ...@@ -44,6 +44,7 @@
#include <linux/rtnetlink.h> #include <linux/rtnetlink.h>
#include <linux/firmware.h> #include <linux/firmware.h>
#include <linux/log2.h> #include <linux/log2.h>
#include <linux/stringify.h>
#include <asm/uaccess.h> #include <asm/uaccess.h>
#include "common.h" #include "common.h"
...@@ -992,11 +993,21 @@ static int bind_qsets(struct adapter *adap) ...@@ -992,11 +993,21 @@ static int bind_qsets(struct adapter *adap)
return err; return err;
} }
#define FW_FNAME "cxgb3/t3fw-%d.%d.%d.bin" #define FW_VERSION __stringify(FW_VERSION_MAJOR) "." \
#define TPSRAM_NAME "cxgb3/t3%c_psram-%d.%d.%d.bin" __stringify(FW_VERSION_MINOR) "." __stringify(FW_VERSION_MICRO)
#define FW_FNAME "cxgb3/t3fw-" FW_VERSION ".bin"
#define TPSRAM_VERSION __stringify(TP_VERSION_MAJOR) "." \
__stringify(TP_VERSION_MINOR) "." __stringify(TP_VERSION_MICRO)
#define TPSRAM_NAME "cxgb3/t3%c_psram-" TPSRAM_VERSION ".bin"
#define AEL2005_OPT_EDC_NAME "cxgb3/ael2005_opt_edc.bin" #define AEL2005_OPT_EDC_NAME "cxgb3/ael2005_opt_edc.bin"
#define AEL2005_TWX_EDC_NAME "cxgb3/ael2005_twx_edc.bin" #define AEL2005_TWX_EDC_NAME "cxgb3/ael2005_twx_edc.bin"
#define AEL2020_TWX_EDC_NAME "cxgb3/ael2020_twx_edc.bin" #define AEL2020_TWX_EDC_NAME "cxgb3/ael2020_twx_edc.bin"
MODULE_FIRMWARE(FW_FNAME);
MODULE_FIRMWARE("cxgb3/t3b_psram-" TPSRAM_VERSION ".bin");
MODULE_FIRMWARE("cxgb3/t3c_psram-" TPSRAM_VERSION ".bin");
MODULE_FIRMWARE(AEL2005_OPT_EDC_NAME);
MODULE_FIRMWARE(AEL2005_TWX_EDC_NAME);
MODULE_FIRMWARE(AEL2020_TWX_EDC_NAME);
static inline const char *get_edc_fw_name(int edc_idx) static inline const char *get_edc_fw_name(int edc_idx)
{ {
...@@ -1067,16 +1078,13 @@ int t3_get_edc_fw(struct cphy *phy, int edc_idx, int size) ...@@ -1067,16 +1078,13 @@ int t3_get_edc_fw(struct cphy *phy, int edc_idx, int size)
static int upgrade_fw(struct adapter *adap) static int upgrade_fw(struct adapter *adap)
{ {
int ret; int ret;
char buf[64];
const struct firmware *fw; const struct firmware *fw;
struct device *dev = &adap->pdev->dev; struct device *dev = &adap->pdev->dev;
snprintf(buf, sizeof(buf), FW_FNAME, FW_VERSION_MAJOR, ret = request_firmware(&fw, FW_FNAME, dev);
FW_VERSION_MINOR, FW_VERSION_MICRO);
ret = request_firmware(&fw, buf, dev);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "could not upgrade firmware: unable to load %s\n", dev_err(dev, "could not upgrade firmware: unable to load %s\n",
buf); FW_FNAME);
return ret; return ret;
} }
ret = t3_load_fw(adap, fw->data, fw->size); ret = t3_load_fw(adap, fw->data, fw->size);
...@@ -1120,8 +1128,7 @@ static int update_tpsram(struct adapter *adap) ...@@ -1120,8 +1128,7 @@ static int update_tpsram(struct adapter *adap)
if (!rev) if (!rev)
return 0; return 0;
snprintf(buf, sizeof(buf), TPSRAM_NAME, rev, snprintf(buf, sizeof(buf), TPSRAM_NAME, rev);
TP_VERSION_MAJOR, TP_VERSION_MINOR, TP_VERSION_MICRO);
ret = request_firmware(&tpsram, buf, dev); ret = request_firmware(&tpsram, buf, dev);
if (ret < 0) { if (ret < 0) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment