diff options
| author | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-02-24 05:55:24 -0500 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-02-24 05:55:24 -0500 |
| commit | 62f0f39b4aa2dce08f08797089e60d945448ca2b (patch) | |
| tree | 844df7ee10b684211400b9f7889b7a963e5d2df3 | |
| parent | d808aa69a7e85dea850ffe7b3d076be696da35be (diff) | |
ARM: cleanup: pwrdm_can_ever_lose_context() checking
pwrdm_can_ever_lose_context() is only ever called from the OMAP GPIO
code, and only with a pointer returned from omap_hwmod_get_pwrdm().
omap_hwmod_get_pwrdm() only ever returns NULL on error, so using
IS_ERR_OR_NULL() to validate the passed pointer is silly. Use a
simpler !ptr check instead.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
| -rw-r--r-- | arch/arm/mach-omap2/powerdomain.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/mach-omap2/powerdomain.c b/arch/arm/mach-omap2/powerdomain.c index dea62a9aad07..36a69189b083 100644 --- a/arch/arm/mach-omap2/powerdomain.c +++ b/arch/arm/mach-omap2/powerdomain.c | |||
| @@ -1054,7 +1054,7 @@ bool pwrdm_can_ever_lose_context(struct powerdomain *pwrdm) | |||
| 1054 | { | 1054 | { |
| 1055 | int i; | 1055 | int i; |
| 1056 | 1056 | ||
| 1057 | if (IS_ERR_OR_NULL(pwrdm)) { | 1057 | if (!pwrdm) { |
| 1058 | pr_debug("powerdomain: %s: invalid powerdomain pointer\n", | 1058 | pr_debug("powerdomain: %s: invalid powerdomain pointer\n", |
| 1059 | __func__); | 1059 | __func__); |
| 1060 | return 1; | 1060 | return 1; |
