From mboxrd@z Thu Jan 1 00:00:00 1970 From: Karl Beldan Subject: Re: [PATCH 1/4 v3] pxa2xx-i2s: Proper hw initialization Date: Thu, 14 May 2009 10:41:41 +0200 Message-ID: <4A0BD945.8000208@gmail.com> References: <4A0BD586.8010406@gmail.com> <4A0BD855.1060000@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ew0-f175.google.com (mail-ew0-f175.google.com [209.85.219.175]) by alsa0.perex.cz (Postfix) with ESMTP id 3546C2436D for ; Thu, 14 May 2009 10:41:44 +0200 (CEST) Received: by ewy23 with SMTP id 23so1415621ewy.32 for ; Thu, 14 May 2009 01:41:43 -0700 (PDT) In-Reply-To: <4A0BD855.1060000@gmail.com> 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: Mark Brown Cc: Russell King , alsa-devel@alsa-project.org, Eric Miao , linux-arm-kernel , Matthieu Dumont List-Id: alsa-devel@alsa-project.org Karl Beldan wrote: > Karl Beldan wrote: >> Make sure we are in a know good state at end of probe : >> Reset FIFO logic and registers, and make sure REC and RPL functions >> along with FIFO service are disabled (SACR0_RST enables REC and RPL). >> >> Resetting loses current settings so remove reset from stream startup. >> Now reset occurs only at probe. >> > > This differs from what you applied in that it includes 'removal of hw reset from > pxa2xx_i2s_startup'. > I understood including it in either one (1/4 or 2/4) is ok, since you had not > pushed it yet. Otherwise I'll have to post a [PATCH 2/4 v3] including the reset, > which would be identical to [PATCH 2/4] (initial version). I meant 'which would be identical to [PATCH 2/4] (initial version) + reset'. -- Karl