brcm47xx: BCMA - Consolidate/reduce code in bcma_device_name
authorHauke Mehrtens <hauke@hauke-m.de>
Wed, 28 Mar 2012 09:36:50 +0000 (09:36 +0000)
committerHauke Mehrtens <hauke@hauke-m.de>
Wed, 28 Mar 2012 09:36:50 +0000 (09:36 +0000)
Eliminate unneeded procedure by incorporating code into bcma_device_name

Signed-off-by: Nathan Hintz <nlhintz@hotmail.com>
SVN-Revision: 31120

target/linux/brcm47xx/patches-3.2/230-bcma-find-name-for-non-brcm.patch

index f76c5adec5359a71431ef2c3f1364a0de886d9e9..54addee9666c2609281100f39129981f31251c8f 100644 (file)
@@ -48,7 +48,7 @@
        { BCMA_CORE_MAC_GBIT, "GBit MAC" },
        { BCMA_CORE_DDR12_MEM_CTL, "DDR1/DDR2 Memory Controller" },
        { BCMA_CORE_PCIE_RC, "PCIe Root Complex" },
-@@ -79,16 +80,57 @@ struct bcma_device_id_name bcma_device_n
+@@ -79,16 +80,41 @@ struct bcma_device_id_name bcma_device_n
        { BCMA_CORE_SHIM, "SHIM" },
        { BCMA_CORE_DEFAULT, "Default" },
  };
 +      { BCMA_CORE_MIPS_74K, "MIPS 74K" },
 +};
 +
-+static const char *bcma_scan_device_names(const struct bcma_device_id *id,
-+                                 const struct bcma_device_id_name *device_names,
-+                                 int array_size)
++static const char *bcma_device_name(const struct bcma_device_id *id)
  {
-       int i;
+-      int i;
++      const struct bcma_device_id_name *names;
++      int size, i;
  
 -      if (id->manuf == BCMA_MANUF_BCM) {
 -              for (i = 0; i < ARRAY_SIZE(bcma_device_names); i++) {
 -                      if (bcma_device_names[i].id == id->id)
 -                              return bcma_device_names[i].name;
 -              }
-+      for (i = 0; i < array_size; i++) {
-+              if (device_names[i].id == id->id)
-+                      return device_names[i].name;
-       }
-+      return NULL;
-+}
-+
-+static const char *bcma_device_name(const struct bcma_device_id *id)
-+{
-+      const char *name;
-+      const struct bcma_device_id_name *names;
-+      int size;
-+
 +      /* search manufacturer specific names */
 +      switch (id->manuf) {
 +      case BCMA_MANUF_ARM:
 +              size = ARRAY_SIZE(bcma_mips_device_names);
 +              break;
 +      default:
-+              names = NULL;
-+              break;
-+      }
++              return "UNKNOWN";
+       }
 +
-+      if (names) {
-+              name = bcma_scan_device_names(id, names, size);
-+              if (name)
-+                      return name;
++      for (i = 0; i < size; i++) {
++              if (names[i].id == id->id)
++                      return names[i].name;
 +      }
 +
        return "UNKNOWN";