From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH] ASoC: fsl_ssi: separately enable and disable TIE/RIE in trigger() Date: Tue, 29 Oct 2013 07:18:21 -0500 Message-ID: <526FA78D.2090601@tabi.org> References: <1383044688-28596-1-git-send-email-b42378@freescale.com> <526FA330.3080209@tabi.org> <20131029115711.GA9605@MrMyself> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from muin.pair.com (muin.pair.com [209.68.1.55]) by alsa0.perex.cz (Postfix) with ESMTP id C04EE26160D for ; Tue, 29 Oct 2013 13:18:23 +0100 (CET) In-Reply-To: <20131029115711.GA9605@MrMyself> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Nicolin Chen Cc: alsa-devel@alsa-project.org, broonie@kernel.org, linuxppc-dev@lists.ozlabs.org List-Id: alsa-devel@alsa-project.org Nicolin Chen wrote: > Well, actually I just wanted to clear T/RDMAE to disable DMA request, > but it seems to be much easier to do it like this based on current > code and disabling unused interrupts should be better right?:) It's not better if it complicates the code and has no real impact. The code has been running fine for years the way it is. Unless you can show me that it actually makes a difference, I will have to NACK this patch.