All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] hid: intel-ish-hid: ipc: remove unused macro
@ 2016-11-11  1:40 Even Xu
  2016-11-15 18:09 ` Pandruvada, Srinivas
  0 siblings, 1 reply; 2+ messages in thread
From: Even Xu @ 2016-11-11  1:40 UTC (permalink / raw)
  To: jikos; +Cc: linux-input, srinivas.pandruvada, Even Xu

The macro timed_wait_for() in utils.h isn't used in current ipc
driver, so remove it for avoiding confusion.

Signed-off-by: Even Xu <even.xu@intel.com>
---
 drivers/hid/intel-ish-hid/ipc/utils.h | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/drivers/hid/intel-ish-hid/ipc/utils.h b/drivers/hid/intel-ish-hid/ipc/utils.h
index 5a82123..dc39dfe 100644
--- a/drivers/hid/intel-ish-hid/ipc/utils.h
+++ b/drivers/hid/intel-ish-hid/ipc/utils.h
@@ -19,27 +19,6 @@
 #define	WAIT_FOR_CONNECT_SLICE	(HZ / 10)
 
 /*
- * Waits for specified event when a thread that triggers event can't signal
- * Also, waits *at_least* `timeinc` after condition is satisfied
- */
-#define	timed_wait_for(timeinc, condition)			\
-	do {							\
-		int completed = 0;				\
-		do {						\
-			unsigned long	j;			\
-			int	done = 0;			\
-								\
-			completed = (condition);		\
-			for (j = jiffies, done = 0; !done; ) {	\
-				schedule_timeout(timeinc);	\
-				if (time_is_before_eq_jiffies(j + timeinc)) \
-					done = 1;		\
-			}					\
-		} while (!(completed));				\
-	} while (0)
-
-
-/*
  * Waits for specified event when a thread that triggers event
  * can't signal with timeout (use whenever we may hang)
  */
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/3] hid: intel-ish-hid: ipc: remove unused macro
  2016-11-11  1:40 [PATCH 1/3] hid: intel-ish-hid: ipc: remove unused macro Even Xu
@ 2016-11-15 18:09 ` Pandruvada, Srinivas
  0 siblings, 0 replies; 2+ messages in thread
From: Pandruvada, Srinivas @ 2016-11-15 18:09 UTC (permalink / raw)
  To: jikos@kernel.org, Xu, Even; +Cc: linux-input@vger.kernel.org

On Fri, 2016-11-11 at 09:40 +0800, Even Xu wrote:
> The macro timed_wait_for() in utils.h isn't used in current ipc
> driver, so remove it for avoiding confusion.
> 
> Signed-off-by: Even Xu <even.xu@intel.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>

> ---
>  drivers/hid/intel-ish-hid/ipc/utils.h | 21 ---------------------
>  1 file changed, 21 deletions(-)
> 
> diff --git a/drivers/hid/intel-ish-hid/ipc/utils.h
> b/drivers/hid/intel-ish-hid/ipc/utils.h
> index 5a82123..dc39dfe 100644
> --- a/drivers/hid/intel-ish-hid/ipc/utils.h
> +++ b/drivers/hid/intel-ish-hid/ipc/utils.h
> @@ -19,27 +19,6 @@
>  #define	WAIT_FOR_CONNECT_SLICE	(HZ / 10)
>  
>  /*
> - * Waits for specified event when a thread that triggers event can't
> signal
> - * Also, waits *at_least* `timeinc` after condition is satisfied
> - */
> -#define	timed_wait_for(timeinc, condition)			
> \
> -	do {							
> \
> -		int completed = 0;				\
> -		do {						
> \
> -			unsigned long	j;			
> \
> -			int	done = 0;			
> \
> -								\
> -			completed = (condition);		\
> -			for (j = jiffies, done = 0; !done; ) {	
> \
> -				schedule_timeout(timeinc);	\
> -				if (time_is_before_eq_jiffies(j +
> timeinc)) \
> -					done = 1;		\
> -			}					\
> -		} while (!(completed));				
> \
> -	} while (0)
> -
> -
> -/*
>   * Waits for specified event when a thread that triggers event
>   * can't signal with timeout (use whenever we may hang)
>   */

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-11-15 18:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-11  1:40 [PATCH 1/3] hid: intel-ish-hid: ipc: remove unused macro Even Xu
2016-11-15 18:09 ` Pandruvada, Srinivas

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.