Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753143AbeAOADm (ORCPT + 1 other); Sun, 14 Jan 2018 19:03:42 -0500 Received: from mail-pg0-f49.google.com ([74.125.83.49]:33830 "EHLO mail-pg0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751255AbeAOADl (ORCPT ); Sun, 14 Jan 2018 19:03:41 -0500 X-Google-Smtp-Source: ACJfBosAogdSt1xHYYIXc/9SinCyPiSdM2jfWVegqCEbpuPtt/dqPrCUPuYSulPcz8h/fgjASHVcAA== Date: Sun, 14 Jan 2018 16:03:38 -0800 From: Nicolin Chen To: "Maciej S. Szmigiero" Cc: timur@tabi.org, broonie@kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, alsa-devel@alsa-project.org, lgirdwood@gmail.com, fabio.estevam@nxp.com, caleb@crome.org, arnaud.mouiche@invoxia.com, lukma@denx.de, kernel@pengutronix.de Subject: Re: [PATCH v2 04/16] ASoC: fsl_ssi: Rename fsl_ssi_disable_val macro Message-ID: <20180115000337.GB2925@Asurada-CZ80> References: <1515652995-15996-1-git-send-email-nicoleotsuka@gmail.com> <1515652995-15996-5-git-send-email-nicoleotsuka@gmail.com> <6dd7d3e8-4673-c56d-c0ce-80df1f537f9b@maciej.szmigiero.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6dd7d3e8-4673-c56d-c0ce-80df1f537f9b@maciej.szmigiero.name> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Sun, Jan 14, 2018 at 11:34:37PM +0100, Maciej S. Szmigiero wrote: > > + /* Check if the opposite stream is active */ > > + aactive = ssi->streams & BIT(!dir); > ^ > Here an implicit assumption that either RX == 0, TX == 1 or > RX == 1, TX == 0 still remains. I would try to get rid of this !dir. However the regvals is defined as regvals[2] so this assumption (either RX == 0 or TX == 0) doesn't look wrong to me. I would prefer to add a comment to limit a potential modification to RX and TX.