Staging: android: Alarm driver cleanups
authorDae S. Kim <dae@velatum.com>
Wed, 29 Aug 2012 15:08:33 +0000 (17:08 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 4 Sep 2012 21:23:33 +0000 (14:23 -0700)
Little cleanups. Enum value ANDROID_ALARM_TYPE_COUNT was treated as
an alarm type within a switch statement. That condition was unreachable
though.

Signed-off-by: Dae S. Kim <dae@velatum.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/android/alarm-dev.c

index 5b7064005188d9ac9299ee24e79e9c163ef785df..a9b293ff3cc8350b8fc1faed4f32dab3fb118873 100644 (file)
@@ -67,10 +67,8 @@ static struct devalarm alarms[ANDROID_ALARM_TYPE_COUNT];
 
 static int is_wakeup(enum android_alarm_type type)
 {
-       if (type == ANDROID_ALARM_RTC_WAKEUP ||
-                       type == ANDROID_ALARM_ELAPSED_REALTIME_WAKEUP)
-               return 1;
-       return 0;
+       return (type == ANDROID_ALARM_RTC_WAKEUP ||
+               type == ANDROID_ALARM_ELAPSED_REALTIME_WAKEUP);
 }
 
 
@@ -85,12 +83,9 @@ static void devalarm_start(struct devalarm *alrm, ktime_t exp)
 
 static int devalarm_try_to_cancel(struct devalarm *alrm)
 {
-       int ret;
        if (is_wakeup(alrm->type))
-               ret = alarm_try_to_cancel(&alrm->u.alrm);
-       else
-               ret = hrtimer_try_to_cancel(&alrm->u.hrt);
-       return ret;
+               return alarm_try_to_cancel(&alrm->u.alrm);
+       return hrtimer_try_to_cancel(&alrm->u.hrt);
 }
 
 static void devalarm_cancel(struct devalarm *alrm)
@@ -223,10 +218,12 @@ from_old_alarm_set:
                case ANDROID_ALARM_ELAPSED_REALTIME:
                        get_monotonic_boottime(&tmp_time);
                        break;
-               case ANDROID_ALARM_TYPE_COUNT:
                case ANDROID_ALARM_SYSTEMTIME:
                        ktime_get_ts(&tmp_time);
                        break;
+               default:
+                       rv = -EINVAL;
+                       goto err1;
                }
                if (copy_to_user((void __user *)arg, &tmp_time,
                    sizeof(tmp_time))) {