staging: ks7010: make loading config file optional
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Tue, 31 May 2016 10:56:23 +0000 (12:56 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Jun 2016 05:42:53 +0000 (22:42 -0700)
We have sane defaults, so we don't need to bail out if there is no
config file. Note that the config file should go away completely in
favour of configuration mechanisms already upstream.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks7010_config.c

index 7f273dafaf3583139e82ce495afb96da084d5127..06ff8852e17f3369b2f66f99c01e4d1e8eb80b9a 100644 (file)
@@ -203,7 +203,6 @@ int ks_wlan_read_config_file(ks_wlan_private *priv)
 
        const struct firmware *fw_entry;
        struct device *dev = NULL;
-       int retval;
        char cfg_file[]=CFG_FILE;
        char *cur_p, *end_p;
        char wk_buff[256], *wk_p;
@@ -254,10 +253,9 @@ int ks_wlan_read_config_file(ks_wlan_private *priv)
        priv->reg.rate_set.size = 12;
 
        dev = &priv->ks_wlan_hw.sdio_card->func->dev;
-       if((retval = request_firmware(&fw_entry, cfg_file, dev)) !=0 ){
-               DPRINTK(1, "error request_firmware() file=%s ret=%d\n", cfg_file, retval);
-               return 1;
-       }
+       /* If no cfg file, stay with the defaults */
+       if (request_firmware_direct(&fw_entry, cfg_file, dev))
+               return 0;
 
        DPRINTK(4, "success request_firmware() file=%s size=%d\n", cfg_file, fw_entry->size);
        cur_p = fw_entry->data;