From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from az33egw02.freescale.net (az33egw02.freescale.net [192.88.158.103]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "az33egw02.freescale.net", Issuer "Thawte Premium Server CA" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id 5E295DE02B for ; Fri, 15 May 2009 02:04:03 +1000 (EST) Message-ID: <4A0C40D4.1090508@freescale.com> Date: Thu, 14 May 2009 11:03:32 -0500 From: Timur Tabi MIME-Version: 1.0 To: Grant Likely Subject: Re: [PATCH v7] introduce macro spin_event_timeout() References: <1241560385-26868-1-git-send-email-timur@freescale.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Cc: linuxppc-dev@ozlabs.org, smaclennan@pikatech.com, scottwood@freescale.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Grant Likely wrote: > This version looks good to me. Who's the first user? (I'd like to see > that something is ready to use this before merging it) I was going to do it the other way around - wait until this patch is merged, and then update some code to use it. After all, it's easier to use a macro if it already exists in a header file. :-) Grepping for "while.*in_be" finds lots of candidates. I was originally planning to use this macro in sound/soc/fsl/fsl_ssi.c, but I eliminated the need to poll on a register with commit a4d11fe50c238a7da5225d1399314c3505cbd792 ("ASoC: remove trigger delay in Freescale MPC8610 sound driver"). So now I need to find a new guinea pig. -- Timur Tabi Linux kernel developer at Freescale