Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp476115imu; Mon, 5 Nov 2018 04:07:41 -0800 (PST) X-Google-Smtp-Source: AJdET5ffyjzMBYzYGK38IFsacI1kqLG5hInJoglE1Q7RJqDZ/SOX5E022HeJHA7B2HtVL5x2ZX2v X-Received: by 2002:a63:5b4a:: with SMTP id l10-v6mr17123206pgm.50.1541419661313; Mon, 05 Nov 2018 04:07:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1541419661; cv=none; d=google.com; s=arc-20160816; b=fu5Ec4NPB7nAmq95OK9rETwSH5c4iIue0qk8MgzCw1REPK9m/dfmA3zANQ/pm4HMu3 yI0c63Cw+KVvr9EbXKBxYcLzQedonGNtZpvKBmHge5KM1R4rogsi9cNxJORAZ7FMSaUb cXw64iCbBFLyHQUOVJRyabWHm0Pb+al/vDo0VOItDgbtk34y+s4zEMqchlEAIs3M87du poq457fgTasH24RAZcM2HDt4Q85KJS/H7DWPVHoLy1B58b8YQ6IqB37o0NMCcTP0OmbW RnwnncKdsk/zPSpRRjhwvWnnjqI5HY+UJlsjvdcFatTzwW81cM/uctQEZF0Ex3osQkcd s8AA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:message-id:in-reply-to:subject:cc:to :from:dkim-signature; bh=b4pQejAkMGDDXfWlmS8wWRza9tmARHoPXwD18GZJss0=; b=l3030g7SNvWfkUbldR6YnMpeW8wAotlu2eGoqwEK7vHwKbjqusC6T/JOmQ0ryAIFQU 5ApV6BWYsDIQ2weZejCl8Lufw04fCdqIBpc7Vy6pBp5wiTnR7/PL7AvMiJal61Ymbzqa ImVCi0OQ9cXdudWLcv+VUMTSOYr76C7rTDmJI03GsozqPRZFAsRLcuf16RcYezU9cmd9 78HLvSvy9vEmS0PyzwDZCG7Ndplr+pp1yVagKqp6lWOUhpJo3ewXT5IDUZBlY7eswTcI hp8Sw/vs6YmO557fSFJEhZs2jULc3P1nFy3shKpA5Q3imIEDWfHAgNOK9EZZ/8K/b3b1 ZbHw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=pOMtjh3z; 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 l16-v6si43295720pfb.69.2018.11.05.04.07.26; Mon, 05 Nov 2018 04:07:41 -0800 (PST) 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=pOMtjh3z; 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 S1729736AbeKEV0D (ORCPT + 99 others); Mon, 5 Nov 2018 16:26:03 -0500 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:53140 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727820AbeKEV0C (ORCPT ); Mon, 5 Nov 2018 16:26:02 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sirena.org.uk; s=20170815-heliosphere; h=Date:Message-Id:In-Reply-To: Subject:Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Id:List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner: List-Archive; bh=b4pQejAkMGDDXfWlmS8wWRza9tmARHoPXwD18GZJss0=; b=pOMtjh3zmLav bWwnO7RkVOBopUbqly+cj8Ai+/No8G5lurVFr2HgUs1jE2YON548NwQZxW08beZExN7L8/B2s6sPH OVmkRsVQLMeqBCMJbv3uJkxGwKeIbolxxzJFcQSLrEljBFv9CN5rx9pewXWDYTnSNumprwyjPt7TI lT49k=; Received: from cpc102320-sgyl38-2-0-cust46.18-2.cable.virginm.net ([82.37.168.47] helo=debutante.sirena.org.uk) by heliosphere.sirena.org.uk with esmtpa (Exim 4.89) (envelope-from ) id 1gJdeE-0008NR-8G; Mon, 05 Nov 2018 12:06:26 +0000 Received: by debutante.sirena.org.uk (Postfix, from userid 1000) id CA85A1124D98; Mon, 5 Nov 2018 12:06:25 +0000 (GMT) From: Mark Brown To: Emil Renner Berthing Cc: Heiko Stuebner , Mark Brown , linux-spi@vger.kernel.org, Addy Ke , Mark Brown , Heiko Stuebner , linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org Subject: Applied "spi: rockchip: simplify use_dma logic" to the spi tree In-Reply-To: <20181031105711.19575-10-esmil@mailme.dk> Message-Id: <20181105120625.CA85A1124D98@debutante.sirena.org.uk> Date: Mon, 5 Nov 2018 12:06:25 +0000 (GMT) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The patch spi: rockchip: simplify use_dma logic has been applied to the spi tree at https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark From eff0275e5253604429aedc42b008c5fcaa6cc597 Mon Sep 17 00:00:00 2001 From: Emil Renner Berthing Date: Wed, 31 Oct 2018 11:57:06 +0100 Subject: [PATCH] spi: rockchip: simplify use_dma logic We only need to know if we're using dma when setting up the transfer, so just use a local variable for that. Signed-off-by: Emil Renner Berthing Tested-by: Heiko Stuebner Signed-off-by: Mark Brown --- drivers/spi/spi-rockchip.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/spi/spi-rockchip.c b/drivers/spi/spi-rockchip.c index 45a1479c1a29..ba60cbcd45c2 100644 --- a/drivers/spi/spi-rockchip.c +++ b/drivers/spi/spi-rockchip.c @@ -184,8 +184,6 @@ struct rockchip_spi { void *rx_end; bool cs_asserted[ROCKCHIP_SPI_MAX_CS_NUM]; - - bool use_dma; }; static inline void spi_enable_chip(struct rockchip_spi *rs, bool enable) @@ -450,7 +448,8 @@ static int rockchip_spi_prepare_dma(struct rockchip_spi *rs, } static void rockchip_spi_config(struct rockchip_spi *rs, - struct spi_device *spi, struct spi_transfer *xfer) + struct spi_device *spi, struct spi_transfer *xfer, + bool use_dma) { u32 div = 0; u32 dmacr = 0; @@ -471,7 +470,7 @@ static void rockchip_spi_config(struct rockchip_spi *rs, else cr0 |= CR0_XFM_TO << CR0_XFM_OFFSET; - if (rs->use_dma) { + if (use_dma) { if (xfer->tx_buf) dmacr |= TF_DMA_EN; if (xfer->rx_buf) @@ -537,6 +536,7 @@ static int rockchip_spi_transfer_one( struct spi_transfer *xfer) { struct rockchip_spi *rs = spi_master_get_devdata(master); + bool use_dma; WARN_ON(readl_relaxed(rs->regs + ROCKCHIP_SPI_SSIENR) && (readl_relaxed(rs->regs + ROCKCHIP_SPI_SR) & SR_BUSY)); @@ -559,15 +559,11 @@ static int rockchip_spi_transfer_one( rs->rx = xfer->rx_buf; rs->rx_end = rs->rx + xfer->len; - /* we need prepare dma before spi was enabled */ - if (master->can_dma && master->can_dma(master, spi, xfer)) - rs->use_dma = true; - else - rs->use_dma = false; + use_dma = master->can_dma ? master->can_dma(master, spi, xfer) : false; - rockchip_spi_config(rs, spi, xfer); + rockchip_spi_config(rs, spi, xfer, use_dma); - if (rs->use_dma) + if (use_dma) return rockchip_spi_prepare_dma(rs, master, xfer); return rockchip_spi_pio_transfer(rs); -- 2.19.0.rc2