ARM: tegra: tegra_powergate_is_powered should be static
Olof Johansson [Fri, 9 Sep 2011 00:54:21 +0000 (17:54 -0700)]
Not exported and not used externally.

Also, fix return type. Due to new return type, errors can't be returned
so WARN_ON instead of returning error if a bad parameter is specified.

Signed-off-by: Olof Johansson <olof@lixom.net>
Acked-by: Stephen Warren <swarren@nvidia.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>

arch/arm/mach-tegra/include/mach/powergate.h
arch/arm/mach-tegra/powergate.c

index 401d1b7..39c396d 100644 (file)
@@ -31,7 +31,6 @@
 
 int tegra_powergate_power_on(int id);
 int tegra_powergate_power_off(int id);
-bool tegra_powergate_is_powered(int id);
 int tegra_powergate_remove_clamping(int id);
 
 /* Must be called with clk disabled, and returns with clk enabled */
index 3cee9aa..9483064 100644 (file)
@@ -89,12 +89,11 @@ int tegra_powergate_power_off(int id)
        return tegra_powergate_set(id, false);
 }
 
-bool tegra_powergate_is_powered(int id)
+static bool tegra_powergate_is_powered(int id)
 {
        u32 status;
 
-       if (id < 0 || id >= TEGRA_NUM_POWERGATE)
-               return -EINVAL;
+       WARN_ON(id < 0 || id >= TEGRA_NUM_POWERGATE);
 
        status = pmc_read(PWRGATE_STATUS) & (1 << id);
        return !!status;