Commit ed246b9e authored by Tobin C. Harding's avatar Tobin C. Harding Committed by Greg Kroah-Hartman

staging: ks7010: remove unnecessary function parameter

Function ks7010_upload_firmware() takes as parameters, two struct
pointers, one of which is a member of the other. This is unnecessary
since one can be accessed via the other.

Remove function parameter and fix all call sites.
Signed-off-by: default avatarTobin C. Harding <me@tobin.cc>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent b3882852
...@@ -709,9 +709,9 @@ static int ks7010_sdio_data_compare(struct ks_wlan_private *priv, u32 address, ...@@ -709,9 +709,9 @@ static int ks7010_sdio_data_compare(struct ks_wlan_private *priv, u32 address,
return ret; return ret;
} }
static int ks7010_upload_firmware(struct ks_wlan_private *priv, static int ks7010_upload_firmware(struct ks_sdio_card *card)
struct ks_sdio_card *card)
{ {
struct ks_wlan_private *priv = card->priv;
unsigned int size, offset, n = 0; unsigned int size, offset, n = 0;
unsigned char *rom_buf; unsigned char *rom_buf;
unsigned char rw_data = 0; unsigned char rw_data = 0;
...@@ -997,8 +997,7 @@ static int ks7010_sdio_probe(struct sdio_func *func, ...@@ -997,8 +997,7 @@ static int ks7010_sdio_probe(struct sdio_func *func,
ks7010_init_defaults(priv); ks7010_init_defaults(priv);
/* Upload firmware */ ret = ks7010_upload_firmware(card);
ret = ks7010_upload_firmware(priv, card); /* firmware load */
if (ret) { if (ret) {
dev_err(&card->func->dev, dev_err(&card->func->dev,
"ks7010: firmware load failed !! return code = %d\n", "ks7010: firmware load failed !! return code = %d\n",
......
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