Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933693AbdGTFal (ORCPT ); Thu, 20 Jul 2017 01:30:41 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:36610 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750740AbdGTFaj (ORCPT ); Thu, 20 Jul 2017 01:30:39 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 42A0760867 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=varada@codeaurora.org Date: Thu, 20 Jul 2017 11:00:29 +0530 From: Varadarajan Narayanan To: Mark Brown Cc: robh+dt@kernel.org, mark.rutland@arm.com, andy.gross@linaro.org, david.brown@linaro.org, linux-spi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, Sham Muthayyan Subject: Re: [PATCH v4 01/14] spi: qup: Enable chip select support Message-ID: <20170720053029.GA25583@codeaurora.org> References: <1498556731-13087-1-git-send-email-varada@codeaurora.org> <1498556731-13087-2-git-send-email-varada@codeaurora.org> <20170717160151.teozg2ni4mqxotbd@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170717160151.teozg2ni4mqxotbd@sirena.org.uk> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1735 Lines: 41 Brown, On Mon, Jul 17, 2017 at 05:01:51PM +0100, Mark Brown wrote: > On Tue, Jun 27, 2017 at 03:15:18PM +0530, Varadarajan Narayanan wrote: > > > the chip select support was removed earlier in commit > > 4a8573abe965115bc5b064401fd669b74e985258. Since the chip > > Please include human readable descriptions of things like commits and > issues being discussed in e-mail in your mails, this makes them much > easier for humans to read especially when they have no internet access. > I do frequently catch up on my mail on flights or while otherwise > travelling so this is even more pressing for me than just being about > making things a bit easier to read. Sorry. The commit being referred to is (https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4a8573abe965115bc5b064401fd669b74e985258) commit 4a8573abe965115bc5b064401fd669b74e985258 Author: Andy Gross Date: Thu Jun 12 14:34:10 2014 -0500 spi: qup: Remove chip select function This patch removes the chip select function. Chip select should instead be supported using GPIOs, defining the DT entry "cs-gpios", and letting the SPI core assert/deassert the chip select as it sees fit. The chip select control inside the controller is buggy. It is supposed to automatically assert the chip select based on the activity in the controller, but it is buggy and doesn't work at all. So instead we elect to use GPIOs. Signed-off-by: Andy Gross Signed-off-by: Mark Brown Will update the commit log and repost the patches. Thanks Varada -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation