Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp4454013imm; Mon, 17 Sep 2018 14:19:35 -0700 (PDT) X-Google-Smtp-Source: ANB0VdYcpAVSPltP+2SoVCyFrrd/4XVSA4KvyPEWUsABcuCYjPle4FP6JuTa7P+yO9pr4fRbjnCi X-Received: by 2002:a62:c4da:: with SMTP id h87-v6mr27744838pfk.39.1537219175083; Mon, 17 Sep 2018 14:19:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1537219175; cv=none; d=google.com; s=arc-20160816; b=GiQuHxApL8A4CSPzuheWssPYTWigCbmKNX2CI4CKJ3cf3AFEc0IU3GCnkdlMEARxpE PqmpHTvY4pMzN9ICLhR+uoRwbzUwGEdJfpEKxva/jqdkFPwa2uf3vVi/PLLyF5l/b2Br Q12q2jQEjvSPRKtlWJOfW8pwLMdxsx//19PNUK1LUF68yZp0JPFLIs0UDhBmAOUfaOfK 2UEnC0pReWOI2N6ywAkaWMYxaMiC4zgN/WLOD191d0lrIcR0ZSoucf050qnb2CSB/rBH iphr18Kn3UzJhGA/5VQ7AKIf+zoH3pyG0lQcUrDhRJJVh+TquT0GxZUSp/f7iDVFRoMO /Qbw== 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=L9hAWRtUg5s0mSPA5IzmRcueXxQmPlmUd5tXOKenZ8s=; b=AAdjRgYGVTE7jZHpfWx0krFP48WNx+Pj2y2/VSGSD3HKfkTEa25PaAPnSOPBHcNcD4 eETCyLBADLB6ZzFJNebOrSw0H3sLOUuaeABdOe237sNdvwvdOake0U3U7n2mmT1kTdRm 0ruh04kDwWkPIVujsihxjTtXj0wSo3lmQNC+3zmi61pRopq4Q2lGQGeCGkmvh/VJ7VZI xGp9LvmxCGCsQ21gqjdkIahu9cxjM28Qv6547p6K+9yFxKFIa4/GMntjadvEB1wMGVEQ 2y793zYv5JJx3WKded/XurBAkvoke5TlDbWRTU5Hno2rsCQAO12I9fY6mpzQxD8JmNUd s0NQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=kgGPICft; 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 c31-v6si15251267pgl.126.2018.09.17.14.19.19; Mon, 17 Sep 2018 14:19:35 -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=kgGPICft; 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 S1728517AbeIRCsE (ORCPT + 99 others); Mon, 17 Sep 2018 22:48:04 -0400 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:35636 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726844AbeIRCsE (ORCPT ); Mon, 17 Sep 2018 22:48:04 -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=L9hAWRtUg5s0mSPA5IzmRcueXxQmPlmUd5tXOKenZ8s=; b=kgGPICftyv8HmWCBiznkEhCZ/ /3+60c3K2Ty35/6sf/rLLOA8YDwMjCzdw+xOC8eaS0uI+BSOG3MVgCcBEQaXEZDJu3quzAK1hMW8v Aoch0MVgHB8SI/x45eJOu/4g28FztmDtS5S7fRiy/GrGofOY+FFKOBNxG/n5qm2M641E0=; Received: from [209.121.128.187] (helo=finisterre.ee.mobilebroadband) by heliosphere.sirena.org.uk with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1g20uj-0000jb-ED; Mon, 17 Sep 2018 21:18:37 +0000 Received: by finisterre.ee.mobilebroadband (Postfix, from userid 1000) id 759B8440078; Mon, 17 Sep 2018 22:18:35 +0100 (BST) Date: Mon, 17 Sep 2018 14:18:35 -0700 From: Mark Brown To: David Lechner Cc: Geert Uytterhoeven , linux-spi , linux-iio@vger.kernel.org, Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , Linux Kernel Mailing List Subject: Re: [PATCH v2 4/4] spi: spi-davinci: Add support for SPI_CS_WORD Message-ID: <20180917211835.GH2471@sirena.org.uk> References: <20180913003920.30600-1-david@lechnology.com> <20180913003920.30600-5-david@lechnology.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="AqCDj3hiknadvR6t" Content-Disposition: inline In-Reply-To: X-Cookie: Universe, n.: 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 --AqCDj3hiknadvR6t Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Sep 13, 2018 at 09:26:48AM -0500, David Lechner wrote: > On 09/13/2018 08:44 AM, Geert Uytterhoeven wrote: > I suppose it might be best to modify the SPI core to say: > if ((spi->mode & SPI_CS_WORD) && (!(ctlr->mode_bits & SPI_CS_WORD) || > gpio_is_valid(spi->cs_gpio)) { > instead of: > if ((spi->mode & SPI_CS_WORD) && !(ctlr->mode_bits & SPI_CS_WORD)) { > Then we could drop the error above. Yes, that makes sense - the same thing is going to apply to any controller with this support. --AqCDj3hiknadvR6t Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlugGioACgkQJNaLcl1U h9Acdgf/X/OeJnFpgtiBslpY7yCDTv4ulyxoYx45ExUrvKW0mJTNGrJa2zjPiX98 j7v4A/s9MJb7t5Rn8I9ZhDVApF0ydTbEDj/sLtyjvdzMVXH/moB48WAYWlCl9Cdm OPdxhZyoO1DyxrszcsUO0o+LsCEQoslL4ru7Wk9wdyOCi05j9yTxPpxlPlq6Wfcu ugs/OVezQYE6EKqCegn/UtDJTrZjFKafY/7Db6SZBKWnXz/ffSV+mc7CHK5QfBrz LHf01dtYhAoURWfH7VMmOukkLeR5C6WFSU7sTWD9FucjGtw009p6LGK4yRtH33yw 8xR4A3ErQos2mFIkWmexZNTnAyLb+w== =Wiqq -----END PGP SIGNATURE----- --AqCDj3hiknadvR6t--