bcma: return error in bcma_sprom_get() when fallback fails
authorArend van Spriel <arend@broadcom.com>
Tue, 6 Mar 2012 14:50:47 +0000 (15:50 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 6 Mar 2012 20:16:18 +0000 (15:16 -0500)
When not SPROM is available a fallback mechanism is used. However,
when that fails the code currently continues. This patch assures
that the bcma_sprom_get() function aborts when that happens.

Cc: Rafal Milecki <zajec5@gmail.com>
Cc: Hauke Mehrtens <hauke@hauke-m.de>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/bcma/sprom.c

index 916ae25..fba8066 100644 (file)
@@ -49,10 +49,23 @@ int bcma_arch_register_fallback_sprom(int (*sprom_callback)(struct bcma_bus *bus
 static int bcma_fill_sprom_with_fallback(struct bcma_bus *bus,
                                         struct ssb_sprom *out)
 {
-       if (!get_fallback_sprom)
-               return -ENOENT;
+       int err;
 
-       return get_fallback_sprom(bus, out);
+       if (!get_fallback_sprom) {
+               err = -ENOENT;
+               goto fail;
+       }
+
+       err = get_fallback_sprom(bus, out);
+       if (err)
+               goto fail;
+
+       pr_debug("Using SPROM revision %d provided by"
+                " platform.\n", bus->sprom.revision);
+       return 0;
+fail:
+       pr_warn("Using fallback SPROM failed (err %d)\n", err);
+       return err;
 }
 
 /**************************************************
@@ -317,13 +330,7 @@ int bcma_sprom_get(struct bcma_bus *bus)
                 * available for this device in some other storage.
                 */
                err = bcma_fill_sprom_with_fallback(bus, &bus->sprom);
-               if (err) {
-                       pr_warn("Using fallback SPROM failed (err %d)\n", err);
-               } else {
-                       pr_debug("Using SPROM revision %d provided by"
-                                " platform.\n", bus->sprom.revision);
-                       return 0;
-               }
+               return err;
        }
 
        sprom = kcalloc(SSB_SPROMSIZE_WORDS_R4, sizeof(u16),