OMAP2+: hwmod: allow multiple calls to omap_hwmod_init()
authorPaul Walmsley <paul@pwsan.com>
Wed, 23 Feb 2011 07:14:06 +0000 (00:14 -0700)
committerPaul Walmsley <paul@pwsan.com>
Mon, 28 Feb 2011 19:04:25 +0000 (12:04 -0700)
There's no longer any reason why we should prevent multiple
calls to omap_hwmod_init().  It is now simply used to register an
array of hwmods.

This should allow a subset of hwmods (e.g., hwmods
handling the system clocksource and clockevents) to be registered
earlier than the remaining mass of hwmods.

Signed-off-by: Paul Walmsley <paul@pwsan.com>
Cc: BenoƮt Cousson <b-cousson@ti.com>
Cc: Kevin Hilman <khilman@ti.com>
arch/arm/mach-omap2/omap_hwmod.c

index 9e89a58711b73d9730c9c6dd21291eb2dbe21e0e..2a7ab6adee82dfcb2dd6f7e57b2ab1e7f403369a 100644 (file)
@@ -162,9 +162,6 @@ static LIST_HEAD(omap_hwmod_list);
 /* mpu_oh: used to add/remove MPU initiator from sleepdep list */
 static struct omap_hwmod *mpu_oh;
 
-/* inited: 0 if omap_hwmod_init() has not yet been called; 1 otherwise */
-static u8 inited;
-
 
 /* Private functions */
 
@@ -1595,26 +1592,20 @@ int omap_hwmod_for_each(int (*fn)(struct omap_hwmod *oh, void *data),
  */
 int __init omap_hwmod_init(struct omap_hwmod **ohs)
 {
-       struct omap_hwmod *oh;
-       int r;
-
-       if (inited)
-               return -EINVAL;
-
-       inited = 1;
+       int r, i;
 
        if (!ohs)
                return 0;
 
-       oh = *ohs;
-       while (oh) {
-               if (omap_chip_is(oh->omap_chip)) {
-                       r = _register(oh);
-                       WARN(r, "omap_hwmod: %s: _register returned "
-                            "%d\n", oh->name, r);
-               }
-               oh = *++ohs;
-       }
+       i = 0;
+       do {
+               if (!omap_chip_is(ohs[i]->omap_chip))
+                       continue;
+
+               r = _register(ohs[i]);
+               WARN(r, "omap_hwmod: %s: _register returned %d\n", ohs[i]->name,
+                    r);
+       } while (ohs[++i]);
 
        return 0;
 }