From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BA203C433DB for ; Sat, 26 Dec 2020 10:06:51 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6A6C320791 for ; Sat, 26 Dec 2020 10:06:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6A6C320791 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=runtux.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe :List-Id:MIME-Version:Message-ID:Subject:To:From:Date:Reply-To:Cc:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:In-Reply-To:References:List-Owner; bh=nGs/RAM0kqc90hCXeZAMQc/ohxoo12AUQPEpnUFqkXI=; b=P2wv3PFW+fBhwTcdy7yXQTiZzS pRNdMa6vB48SkliUH3gtrH7otolgsCcxRT8iPHthruiadWSW7nTXhx05UnerpsnjdJBR4TOCBH9kZ hQUPOz80ib9EQ2cu1otpezEhX17sMcCXQySH0hsvQpu3rRtMOzOENgu0EAZGJ0n/ojwhXY1n8/TSL 3SpK69y7F5j8RPrVNrRkOUks7HpFOt1GsU5KENepwIWdrmAlsFvz0HRy6mUZ+VKL+7ozKtWoa9xre GAIF1s6ePltlPLMvMn86+mIf4Gx6c+UWdtiZJeX6a5Q/3/eKWuH5SQmcGEoeapThMWak455prueFC RSGx73tQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kt6Rh-0005ly-IL; Sat, 26 Dec 2020 10:05:09 +0000 Received: from tux.runtux.com ([176.9.82.136]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kt6Rd-0005lN-Ra for linux-arm-kernel@lists.infradead.org; Sat, 26 Dec 2020 10:05:07 +0000 Received: from localhost (localhost [127.0.0.1]) by tux.runtux.com (Postfix) with ESMTP id 262A86F070; Sat, 26 Dec 2020 10:58:48 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at tux.runtux.com Received: from tux.runtux.com ([127.0.0.1]) by localhost (tux2.runtux.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id IXeIB_tByXV2; Sat, 26 Dec 2020 10:58:46 +0100 (CET) Received: from bee.priv.zoo (62-99-217-90.static.upcbusiness.at [62.99.217.90]) (Authenticated sender: postmaster@runtux.com) by tux.runtux.com (Postfix) with ESMTPSA id 81DFB6F01F; Sat, 26 Dec 2020 10:58:46 +0100 (CET) Received: by bee.priv.zoo (Postfix, from userid 1002) id 7505146C; Sat, 26 Dec 2020 10:58:45 +0100 (CET) Date: Sat, 26 Dec 2020 10:58:45 +0100 From: Ralf Schlatterbeck To: Mark Brown , Maxime Ripard , Chen-Yu Tsai , linux-spi@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: [PATCH] Fix SPI Chipselect/Clock bug for sun6i Message-ID: <20201226095845.c65lhsmluddvwxsl@runtux.com> MIME-Version: 1.0 Content-Disposition: inline X-ray: beware User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201226_050506_056930_F5FE2725 X-CRM114-Status: GOOD ( 18.12 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org This patch makes additional GPIOs usable as chipselects for SPI. It is available for years on the linux-sunxi SPIdev page at http://linux-sunxi.org/SPIdev and probably is originally by the user "Mirko" there. I've tried unsuccessfully to contact the author. Note that contrary to what is stated on the page above, the bug doesn't have cosmetic implications only: The SPI-Driver for the H2 Allwinner processor in the Linux kernel is supposed to support normal GPIOs as additional chipselects but fails to do so without the patch. For oscilloscope screenshots with/without the patch, see my blog post https://blog.runtux.com/posts/2019/04/18/ or the discussion in the armbian forum at https://forum.armbian.com/topic/4330-spi-gpio-chip-select-support/ (my logo there is a penguin). Please bear with me if I'm not following proper procedures, this is my first patch to the kernel in years. Thanks Ralf Schlatterbeck >From 682ae1848b0df00cceb4c76486b971826b2737a9 Mon Sep 17 00:00:00 2001 From: Ralf Schlatterbeck Date: Thu, 11 Apr 2019 16:21:54 +0200 Subject: [PATCH] Fix SPI Chipselect/Clock bug for sun6i Signed-off-by: Ralf Schlatterbeck The current sun6i SPI implementation initializes the transfer too early, resulting in SCK going high before the transer. When using an additional (gpio) chipselect with sun6i, the chipselect is asserted at a time when clock is high, making the SPI transfer fail. This patch from the sunxi spidev page http://linux-sunxi.org/SPIdev fixes this, without the patch no additional gpio chipselects can be used. The relevant code seems to be from the user "Mirko" of the linux-sunxi wiki page, I was unable to contact the original author. --- drivers/spi/spi-sun6i.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi-sun6i.c b/drivers/spi/spi-sun6i.c index 19238e1b76b4..9b292c6ade50 100644 --- a/drivers/spi/spi-sun6i.c +++ b/drivers/spi/spi-sun6i.c @@ -290,6 +290,10 @@ static int sun6i_spi_transfer_one(struct spi_master *master, } sun6i_spi_write(sspi, SUN6I_CLK_CTL_REG, reg); + /* Finally enable the bus - doing so before might raise SCK to HIGH */ + sun6i_spi_write(sspi, SUN6I_GBL_CTL_REG, + sun6i_spi_read(sspi, SUN6I_GBL_CTL_REG) + | SUN6I_GBL_CTL_BUS_ENABLE); /* Setup the transfer now... */ if (sspi->tx_buf) @@ -398,7 +402,7 @@ static int sun6i_spi_runtime_resume(struct device *dev) } sun6i_spi_write(sspi, SUN6I_GBL_CTL_REG, - SUN6I_GBL_CTL_BUS_ENABLE | SUN6I_GBL_CTL_MASTER | SUN6I_GBL_CTL_TP); + SUN6I_GBL_CTL_MASTER | SUN6I_GBL_CTL_TP); return 0; -- 2.20.1 -- Dr. Ralf Schlatterbeck Tel: +43/2243/26465-16 Open Source Consulting www: www.runtux.com Reichergasse 131, A-3411 Weidling email: office@runtux.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel