Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp4324702yba; Sun, 12 May 2019 10:07:09 -0700 (PDT) X-Google-Smtp-Source: APXvYqzZQgJSI9Z+P//S0SfIzYNkEFPOYHzeTiVstxlliv6WzoeKcmo3+fo8ENUlBBt/9njLaVo1 X-Received: by 2002:a17:902:84:: with SMTP id a4mr26185393pla.210.1557680829849; Sun, 12 May 2019 10:07:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1557680829; cv=none; d=google.com; s=arc-20160816; b=CU0uQZscroJzF9NWi3w1u+qxX53vYXXkdb6HhSF3+CJf1mxwhNu06lchJnbCe8CcvX 17NBO2taWqY9fNd9l7n3yn97yij/ryfUHoSyJfW0z20Ze8+mTaZCNksxkppMV0lhnZ0G 32/SdLuWIoJpM3dec7Pc/gHiBcMsZcOUzrReh0KYzLzTkGen33QCN3ubcKXh+lWZQaXh g8rKXtjAtL+J4K0tFA6S/Z2HPQ90uouHpNMqE9nrXhwzfkVOcPC1ZnXeaz5bZItG2kfq +C/N/qWAYEV6JZnYnztuFkQ5CAKkdgx6sbOTdwY4QCrOYsDxHoiajCx8D8ADlQEwzEwZ 54uA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=9RG2AODhAu0ddI6bREDEwdc3wqRcusF77Y1+ZDoUoqI=; b=k/NXXyBrOD0m0XSxCDhod297B/9mX0OWDEbztzSiLYOgbLi+GwXtI3IFxPl4Vdvrxo NBWrFZancdGz08e0/4f4TYKGYJXvIGzVHOiFz2UEE6kyA+YAx+2l8niG5L0/dJhJg8V3 o657lI1tqtlUF2icwSQ6vsknKQ/reaipiJEOxJNsXHVOUsZzkJDoipidUf1+UmMqJoxm Ralz8KiREGmq50psgjaHVvb3OiKnjSjL6E36+xMm1Sai/acQRiOqzz10IAq8f1SV91Rw MZ9bGo0WI6ibGQ2hZZjyTRJyfKXnNtq3Uva3yV/NqxJiWtZBCd2wxAMWu7f0Pv6AGt9X fvZQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=Z5MaOZyQ; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l193si14607905pfc.233.2019.05.12.10.06.54; Sun, 12 May 2019 10:07:09 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=Z5MaOZyQ; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727066AbfELRFy (ORCPT + 99 others); Sun, 12 May 2019 13:05:54 -0400 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:49684 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727015AbfELRFv (ORCPT ); Sun, 12 May 2019 13:05:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sirena.org.uk; s=20170815-heliosphere; h=In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=9RG2AODhAu0ddI6bREDEwdc3wqRcusF77Y1+ZDoUoqI=; b=Z5MaOZyQWC3c99sXzxJ2QgNM4 GhhJ95ow9EFGIB1Z3aBaNn0yZ0YLAVrYBDO8eZ4LpYM+zKWIN7545KbFXZEc1BQM45klI5sCGKxfH p9MdXFRu9k9EqApl0snS8Md197flO+KVS0KtZLk7az3fDlQj3eg6eM2bbGhmgmSkEsarE=; Received: from [81.145.206.43] (helo=finisterre.sirena.org.uk) by heliosphere.sirena.org.uk with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1hPrut-00044V-N5; Sun, 12 May 2019 17:05:39 +0000 Received: by finisterre.sirena.org.uk (Postfix, from userid 1000) id A1BE644003B; Sun, 12 May 2019 08:54:17 +0100 (BST) Date: Sun, 12 May 2019 16:54:17 +0900 From: Mark Brown To: Viorel Suman Cc: Liam Girdwood , Jaroslav Kysela , Takashi Iwai , "S.j. Wang" , Julia Lawall , Daniel Baluta , Colin Ian King , "alsa-devel@alsa-project.org" , "linux-kernel@vger.kernel.org" , dl-linux-imx , Viorel Suman Subject: Re: [PATCH 1/2] ASoC: ak4458: rstn_control - return a non-zero on error only Message-ID: <20190512075417.GG21483@sirena.org.uk> References: <1557408607-25115-1-git-send-email-viorel.suman@nxp.com> <1557408607-25115-2-git-send-email-viorel.suman@nxp.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="RMedoP2+Pr6Rq0N2" Content-Disposition: inline In-Reply-To: <1557408607-25115-2-git-send-email-viorel.suman@nxp.com> X-Cookie: HOST SYSTEM RESPONDING, PROBABLY UP... User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --RMedoP2+Pr6Rq0N2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, May 09, 2019 at 01:30:36PM +0000, Viorel Suman wrote: > - return ret; > + /* Return a negative error code only. */ > + return (ret < 0 ? ret : 0); Please write normal conditional statements to help people who have to read the code. --RMedoP2+Pr6Rq0N2 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlzX0SgACgkQJNaLcl1U h9Bpqgf/Z4TM2ROcn75WSzHPLXR43KHE2salosbyrC5M/Mg9blRgsevh8gtJwHCW e2IYJiCmHH80IKs6jMo2MOCXJZ4ib5XsvEd3ZUa1vAiajsIUEqUxJxBTgJY/8Lj6 KvgIc6i7qfd+8gtgvwenOntPYH1vdcJvDlI2pgLtJX1mA8zA8WAqkv8kwbHddQQp c03uvUyOErkh0duoT4f5WBfsnky2kQh0oZWfREtPdLq4O/mck4zUCj+XJ5Ah0UMs HP09a4nkC21r+NyYb7tJ2kqs7Mf89F/UW3B2xEx9Xdi0IDsYnONIF95UfIberk5V jtLqEpVlKdRGgZxNUnqqpsIJmbrpUw== =G/aP -----END PGP SIGNATURE----- --RMedoP2+Pr6Rq0N2--