From: Daniel Walker Date: Fri, 12 Jun 2009 18:51:16 +0000 (-0700) Subject: staging: android: binder: add enum usage in function arguments X-Git-Tag: v2.6.32-rc1~668^2~638 X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=fd0a2f07d9b37fa52ece5bdec4f1130e080c1f51;p=~shefty%2Frdma-dev.git staging: android: binder: add enum usage in function arguments Declare the binder_deferred_state enum, and use the new enum for one of the binder_defer_work function arguments. This should keep the argument within the confines of the enum instead of the whole int range. Signed-off-by: Daniel Walker Cc: Brian Swetland Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index e1e6adedd4d..77207110fcd 100644 --- a/drivers/staging/android/binder.c +++ b/drivers/staging/android/binder.c @@ -243,7 +243,7 @@ struct binder_buffer { uint8_t data[0]; }; -enum { +enum binder_deferred_state { BINDER_DEFERRED_PUT_FILES = 0x01, BINDER_DEFERRED_FLUSH = 0x02, BINDER_DEFERRED_RELEASE = 0x04, @@ -326,7 +326,8 @@ struct binder_transaction { uid_t sender_euid; }; -static void binder_defer_work(struct binder_proc *proc, int defer); +static void +binder_defer_work(struct binder_proc *proc, enum binder_deferred_state defer); /* * copied from get_unused_fd_flags @@ -3071,7 +3072,8 @@ static void binder_deferred_func(struct work_struct *work) } static DECLARE_WORK(binder_deferred_work, binder_deferred_func); -static void binder_defer_work(struct binder_proc *proc, int defer) +static void +binder_defer_work(struct binder_proc *proc, enum binder_deferred_state defer) { mutex_lock(&binder_deferred_lock); proc->deferred_work |= defer;