From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH V4 3/5] AC97 driver for mpc5200 Date: Mon, 25 May 2009 21:47:24 -0600 Message-ID: References: <20090525221406.21370.30326.stgit@terra> <20090525221511.21370.81405.stgit@terra> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-gx0-f158.google.com (mail-gx0-f158.google.com [209.85.217.158]) by alsa0.perex.cz (Postfix) with ESMTP id 0BD5C2454A for ; Tue, 26 May 2009 05:47:44 +0200 (CEST) Received: by gxk2 with SMTP id 2so6912902gxk.8 for ; Mon, 25 May 2009 20:47:44 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Timur Tabi Cc: linuxppc-dev@ozlabs.org, alsa-devel@alsa-project.org, broonie@sirena.org.uk List-Id: alsa-devel@alsa-project.org On Mon, May 25, 2009 at 9:41 PM, Timur Tabi wrote: > On Mon, May 25, 2009 at 5:15 PM, Jon Smirl wrote: > >> + =A0 =A0 =A0 spin_event_timeout(1, 3, 0, rc); > > Ugh, don't do this. =A0Just use udelay! heh; indeed. My objection to udelay is that it burns cycles it the test loop that could be used to decide whether or not the loop is complete. Not an issue when the explicit goal is to burn a fixed number of cycles. g. -- = Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-gx0-f220.google.com (mail-gx0-f220.google.com [209.85.217.220]) by ozlabs.org (Postfix) with ESMTP id 8CD05DE0E3 for ; Tue, 26 May 2009 13:47:45 +1000 (EST) Received: by gxk20 with SMTP id 20so2391299gxk.9 for ; Mon, 25 May 2009 20:47:44 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <20090525221406.21370.30326.stgit@terra> <20090525221511.21370.81405.stgit@terra> From: Grant Likely Date: Mon, 25 May 2009 21:47:24 -0600 Message-ID: Subject: Re: [alsa-devel] [PATCH V4 3/5] AC97 driver for mpc5200 To: Timur Tabi Content-Type: text/plain; charset=ISO-8859-1 Cc: linuxppc-dev@ozlabs.org, alsa-devel@alsa-project.org, broonie@sirena.org.uk List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, May 25, 2009 at 9:41 PM, Timur Tabi wrote: > On Mon, May 25, 2009 at 5:15 PM, Jon Smirl wrote: > >> + =A0 =A0 =A0 spin_event_timeout(1, 3, 0, rc); > > Ugh, don't do this. =A0Just use udelay! heh; indeed. My objection to udelay is that it burns cycles it the test loop that could be used to decide whether or not the loop is complete. Not an issue when the explicit goal is to burn a fixed number of cycles. g. --=20 Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd.