From: Alex Elder <elder@linaro.org>
Date: Tue, 12 May 2015 02:16:37 +0000 (-0500)
Subject: greybus: loopback: symbolically define max wait time
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=91262c3ab70a02949db294b55b4cd49b8ae6505e;p=linux.git
greybus: loopback: symbolically define max wait time
Use a symbolic constant to define the maximum time (number of
milliseconds) to delay between initiated operations.
Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
---
diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c
index e1e40ea44ccf1..f7538aa5fcd44 100644
--- a/drivers/staging/greybus/loopback.c
+++ b/drivers/staging/greybus/loopback.c
@@ -55,6 +55,7 @@ struct gb_loopback {
#define GB_LOOPBACK_TYPE_PING 0x02
#define GB_LOOPBACK_TYPE_TRANSFER 0x03
+#define GB_LOOPBACK_MS_WAIT_MAX 1000
#define GB_LOOPBACK_SIZE_MAX SZ_4K
/* Define get_version() routine */
@@ -121,8 +122,8 @@ static DEVICE_ATTR_RW(field)
static void gb_loopback_reset_stats(struct gb_loopback *gb);
static void gb_loopback_check_attr(struct gb_loopback *gb)
{
- if (gb->ms_wait > 1000)
- gb->ms_wait = 1000;
+ if (gb->ms_wait > GB_LOOPBACK_MS_WAIT_MAX)
+ gb->ms_wait = GB_LOOPBACK_MS_WAIT_MAX;
if (gb->type > 2)
gb->type = 0;
if (gb->size > GB_LOOPBACK_SIZE_MAX)