Received: by 2002:a17:90a:c8b:0:0:0:0 with SMTP id v11csp2291524pja; Fri, 19 Apr 2019 11:23:54 -0700 (PDT) X-Google-Smtp-Source: APXvYqwfBZZs/4xex0hagklXyJ+wwcjtfh2kQA76kcPGUkp9mRTKG+ps6YOtZ6ZQAxeRo3gJ8ahn X-Received: by 2002:a62:12c9:: with SMTP id 70mr5362685pfs.156.1555698234046; Fri, 19 Apr 2019 11:23:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1555698234; cv=none; d=google.com; s=arc-20160816; b=ODZCXHFIz4lplb4JUkegrcd9+hV87Dq7br/fVn9OhMDDG4SnmZgFnmCLVuoQURzlNC Qz0Ff4+0JFNKhyxiGuf9P3qT0plKTlPHMR+zGsY8vD5yB05CdJfuEzS4g1Q/e/HKdeKC RE1PKr+iM4vVXsh2spHqAvcUIuuIjQMdSnmZ4clcAmXaL3YACTbSYdvvmEb+9uf7BxZF 0l9IKecBgZ8MEL7gmgRpvmpGdRS7HLZrfvsr6SUYf2c1M5OoaKkMPwfqIh404DXmTEdi STYLip5ftK8ImwMu8/T1KzhDKxrHF2qXkn9PJAa46K/eQGcaH2pm0vZPICgECshRHSsB x71g== 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=XG8zAcN/Cdib2jKfVHm3MDgQxp0vn5cRdxCvErmhPxo=; b=fas9xs8fRL4jeWjAAuffW93f0NpW6E4KpHX2mD07dx59p7TZNwcVAxlPk2ZIWvYH2A 7Cku81Y0h59oYt/DfXd14Sijohb6o8wqfa2tM+4K2TF3ZTdEf0GyKZMvH60/AdRwqK0V CHHiYyjYZBLnijL3IJJheTLbCGaStsEIUZa5NbrkR/BIC2FUHs1rlV+aSU2ruaei6lmY rKCaoXL4gdJFea86pVLGiVc9ylxMCfted9hCsC8j7mN6l+mwG+6jknZ0m2983WfaYsfI e9y8+iFYIYlWXH208ukFSLvkkjTZN5940joVlOUyYs5OAnJI7GCMY7XGM2o4ofUwYwng cBOw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=lPcOCrrQ; 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 f1si6187545pff.158.2019.04.19.11.23.38; Fri, 19 Apr 2019 11:23:54 -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=lPcOCrrQ; 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 S1727791AbfDSSVn (ORCPT + 99 others); Fri, 19 Apr 2019 14:21:43 -0400 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:38528 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727378AbfDSSVl (ORCPT ); Fri, 19 Apr 2019 14:21:41 -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=XG8zAcN/Cdib2jKfVHm3MDgQxp0vn5cRdxCvErmhPxo=; b=lPcOCrrQG/9tpNtFqLHxtdrVN mb67sKYJQFy5ANW24x4Ccz4QR8e7AV9piCpIbcKgDYbOUoKIaUiSwVGXFISviJFTShYc6KftMznTM iWC1XCiOwvTlC3CoiqxVnL1FhRSfXtB2Sw34Njl8RSrfFyAoVMjh+FjnWqzo+P6UmQPhg=; Received: from 188.29.164.213.threembb.co.uk ([188.29.164.213] 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 1hHVHV-0008Fv-OF; Fri, 19 Apr 2019 15:18:25 +0000 Received: by finisterre.ee.mobilebroadband (Postfix, from userid 1000) id 575F0441D3B; Fri, 19 Apr 2019 16:18:23 +0100 (BST) Date: Fri, 19 Apr 2019 16:18:23 +0100 From: Mark Brown To: Sowjanya Komatineni Cc: thierry.reding@gmail.com, jonathanh@nvidia.com, talho@nvidia.com, robh+dt@kernel.org, mark.rutland@arm.com, kyarlagadda@nvidia.com, ldewangan@nvidia.com, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH V3 5/9] spi: export spi core function spi_set_cs Message-ID: <20190419151823.GS2803@sirena.org.uk> References: <1555363834-32155-1-git-send-email-skomatineni@nvidia.com> <1555363834-32155-6-git-send-email-skomatineni@nvidia.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="QHhm1I6mwQR20oIa" Content-Disposition: inline In-Reply-To: <1555363834-32155-6-git-send-email-skomatineni@nvidia.com> X-Cookie: Shah, shah! Ayatollah you so! 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 --QHhm1I6mwQR20oIa Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Apr 15, 2019 at 02:30:30PM -0700, Sowjanya Komatineni wrote: > This patch exports spi_set_cs of the spi core to allow SPI masters > to use when gpio based chip select is needed. This isn't really what I meant when I said it'd be good to use the core GPIO code - this function doesn't do a huge amount really and the usage of it in your subsequent patch for the driver isn't exactly joined up with the little it does (which is mainly swapping in the GPIO chip select instead of the hardware chip select) isn't used in your driver usage of this as far as I can see. The bulk of the chip select handling code in the core is actually in transfer_one_message() which your driver doesn't use as it's got it's own implementation of that; I've not looked in enough detail to figure out if it could use it. --QHhm1I6mwQR20oIa Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAly55r4ACgkQJNaLcl1U h9ABFQf/di4BhVkLLzj0LzAWfq+W3aQXLwjMllikpbSVi4yBAjMMBCiY/hJ694sK t74oXpOBl1I+nU8XTCCo2UQgwEkIg5nVLHQJqoALeiqtM8W26w9xCeN4I3skqA1m lftMW84AiOm3Rj8FXw62w8WvnugSHO5D+YMbL1DHlltQw2RBM0K6mVrENcuCu+eZ 3BVN3SzoKeuWgJV0HrZhFklXSPjbYUHKlWESz2gNukz9mmP7O+IPPAx6YJ5qbQac Ek1vYs20B7Ldl3EmbRDIdoiiiPssmHNAqT7wiG+WDrehx2f4SF2LTEHQknLypBim /NexZZx63UoROPuN2ifzuZSjJT2bZQ== =bYUd -----END PGP SIGNATURE----- --QHhm1I6mwQR20oIa--