diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-12-08 18:33:13 -0500 |
---|---|---|
committer | paul <paul@twilight.(none)> | 2009-12-11 19:00:41 -0500 |
commit | 1fda39e6fd13f9f74721d2127f27675a4a0878af (patch) | |
tree | a68c19473be41998d2f2316af438963cc5382d70 /arch | |
parent | 155a22ecd574c3e6a4c59e6a1bf0dd31ce23ff10 (diff) |
OMAP2/3 powerdomain: return errors rather than returning the output of IS_ERR()
IS_ERR returns only 1 or 0, and the functions return a negative error
in other cases anyways.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Paul Walmsley <paul@pwsan.com>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/arm/mach-omap2/powerdomain.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/arm/mach-omap2/powerdomain.c b/arch/arm/mach-omap2/powerdomain.c index 1dd1a3817e8b..47d576883d5c 100644 --- a/arch/arm/mach-omap2/powerdomain.c +++ b/arch/arm/mach-omap2/powerdomain.c | |||
@@ -478,7 +478,7 @@ int pwrdm_add_wkdep(struct powerdomain *pwrdm1, struct powerdomain *pwrdm2) | |||
478 | if (IS_ERR(p)) { | 478 | if (IS_ERR(p)) { |
479 | pr_debug("powerdomain: hardware cannot set/clear wake up of " | 479 | pr_debug("powerdomain: hardware cannot set/clear wake up of " |
480 | "%s when %s wakes up\n", pwrdm1->name, pwrdm2->name); | 480 | "%s when %s wakes up\n", pwrdm1->name, pwrdm2->name); |
481 | return IS_ERR(p); | 481 | return PTR_ERR(p); |
482 | } | 482 | } |
483 | 483 | ||
484 | pr_debug("powerdomain: hardware will wake up %s when %s wakes up\n", | 484 | pr_debug("powerdomain: hardware will wake up %s when %s wakes up\n", |
@@ -511,7 +511,7 @@ int pwrdm_del_wkdep(struct powerdomain *pwrdm1, struct powerdomain *pwrdm2) | |||
511 | if (IS_ERR(p)) { | 511 | if (IS_ERR(p)) { |
512 | pr_debug("powerdomain: hardware cannot set/clear wake up of " | 512 | pr_debug("powerdomain: hardware cannot set/clear wake up of " |
513 | "%s when %s wakes up\n", pwrdm1->name, pwrdm2->name); | 513 | "%s when %s wakes up\n", pwrdm1->name, pwrdm2->name); |
514 | return IS_ERR(p); | 514 | return PTR_ERR(p); |
515 | } | 515 | } |
516 | 516 | ||
517 | pr_debug("powerdomain: hardware will no longer wake up %s after %s " | 517 | pr_debug("powerdomain: hardware will no longer wake up %s after %s " |
@@ -548,7 +548,7 @@ int pwrdm_read_wkdep(struct powerdomain *pwrdm1, struct powerdomain *pwrdm2) | |||
548 | if (IS_ERR(p)) { | 548 | if (IS_ERR(p)) { |
549 | pr_debug("powerdomain: hardware cannot set/clear wake up of " | 549 | pr_debug("powerdomain: hardware cannot set/clear wake up of " |
550 | "%s when %s wakes up\n", pwrdm1->name, pwrdm2->name); | 550 | "%s when %s wakes up\n", pwrdm1->name, pwrdm2->name); |
551 | return IS_ERR(p); | 551 | return PTR_ERR(p); |
552 | } | 552 | } |
553 | 553 | ||
554 | return prm_read_mod_bits_shift(pwrdm1->prcm_offs, PM_WKDEP, | 554 | return prm_read_mod_bits_shift(pwrdm1->prcm_offs, PM_WKDEP, |
@@ -582,7 +582,7 @@ int pwrdm_add_sleepdep(struct powerdomain *pwrdm1, struct powerdomain *pwrdm2) | |||
582 | pr_debug("powerdomain: hardware cannot set/clear sleep " | 582 | pr_debug("powerdomain: hardware cannot set/clear sleep " |
583 | "dependency affecting %s from %s\n", pwrdm1->name, | 583 | "dependency affecting %s from %s\n", pwrdm1->name, |
584 | pwrdm2->name); | 584 | pwrdm2->name); |
585 | return IS_ERR(p); | 585 | return PTR_ERR(p); |
586 | } | 586 | } |
587 | 587 | ||
588 | pr_debug("powerdomain: will prevent %s from sleeping if %s is active\n", | 588 | pr_debug("powerdomain: will prevent %s from sleeping if %s is active\n", |
@@ -621,7 +621,7 @@ int pwrdm_del_sleepdep(struct powerdomain *pwrdm1, struct powerdomain *pwrdm2) | |||
621 | pr_debug("powerdomain: hardware cannot set/clear sleep " | 621 | pr_debug("powerdomain: hardware cannot set/clear sleep " |
622 | "dependency affecting %s from %s\n", pwrdm1->name, | 622 | "dependency affecting %s from %s\n", pwrdm1->name, |
623 | pwrdm2->name); | 623 | pwrdm2->name); |
624 | return IS_ERR(p); | 624 | return PTR_ERR(p); |
625 | } | 625 | } |
626 | 626 | ||
627 | pr_debug("powerdomain: will no longer prevent %s from sleeping if " | 627 | pr_debug("powerdomain: will no longer prevent %s from sleeping if " |
@@ -664,7 +664,7 @@ int pwrdm_read_sleepdep(struct powerdomain *pwrdm1, struct powerdomain *pwrdm2) | |||
664 | pr_debug("powerdomain: hardware cannot set/clear sleep " | 664 | pr_debug("powerdomain: hardware cannot set/clear sleep " |
665 | "dependency affecting %s from %s\n", pwrdm1->name, | 665 | "dependency affecting %s from %s\n", pwrdm1->name, |
666 | pwrdm2->name); | 666 | pwrdm2->name); |
667 | return IS_ERR(p); | 667 | return PTR_ERR(p); |
668 | } | 668 | } |
669 | 669 | ||
670 | return prm_read_mod_bits_shift(pwrdm1->prcm_offs, OMAP3430_CM_SLEEPDEP, | 670 | return prm_read_mod_bits_shift(pwrdm1->prcm_offs, OMAP3430_CM_SLEEPDEP, |