From patchwork Wed Dec 15 11:01:03 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tudor Ambarus X-Patchwork-Id: 12696250 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 28DA9C433F5 for ; Wed, 15 Dec 2021 11:06:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:To :From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=R7MTAGnM1/VsdEKsKmjB5ZIErOD82Hwxwe8wyxVzN28=; b=ueJyBOx21DW3KM 7VnrfCrXudRt8tSQsBn3tiuQEACTHRzdJz5SUsJYYTuAzoR9eM7b6AF6jxJA1i5Rg155SteYWOXmU 8+c8lxUV95m/+1kLfxB1TIMpG3th+63sG8jHXftyJDUQwRBle3UTeHOcc4008Xkr8CZXxSxf4Ueav G0vHVcIRzJPYMUYU6S0FNC8da/m4c/Bk0gCSosyP97A+gP1ncU5MlG3amMkJA6fvNMMWjh11u55ax t1MY1M9f6rn4IiV31XfqBOcG7GVtxgWFn74nMDwJwEtIfz0lAgxcDBENIS76BOs9YpxUaH5q3Er/+ XU5sYFvurpxX0Fw7DQsQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mxS5i-000O11-0i; Wed, 15 Dec 2021 11:04:58 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mxS2E-000M9D-N5 for linux-arm-kernel@lists.infradead.org; Wed, 15 Dec 2021 11:01:24 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1639566082; x=1671102082; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=7qxF1rA3WXG5vGkMtaJGRRx4q17WASpZMPPVSSKMp4E=; b=t/tNp10IIAbSmRtbZslGHuqde/cVJC7x7IA29W5aCuHF5T52rql3MKJH q7DhJFNw5c4DYR5CmDmZUiFi/N1xp54mlmDJQOuEHNihxW0DkQT7/413G kNG/uHggY5rvO8qeLWHyWnNL4D3LXu7VCA72bPJ69Zr8Uw2oE1WauNt2x dc/sHMsg03a2pLxtfio/AhSWwsyj66MFLq05Cq4Yjca8mSCnlMPVDw1pV aJbZr5guepGlf1A9J8hHCLMPPAHPdN1KWkWbb7vPDCqHkiG5bFSiSZMBm vsimWsD0Op18rM8iQl5cYjaqdShVar+A7SlwQkiqIPW4TfEgVNNmJlVzs Q==; IronPort-SDR: rt0MniAA6+tOR8zkPgA15sOR02cmfmIJBrTMU11WBH8riBw6+s4phEY9z+gqRBdm6K8fL1vIqX P+wlRdTlH9x3fw6+pmVFZ3umjBbWFZY7mJNxq4rXLrzRKoV5skOFtgO+GxJNH0yWAtWpWDJfaK 041INyAJy7orwm6JNIJiM+kQRadfzl8X4XwDpDrO5SHDJAyQEKyzcOVhlmm3w6zzE5tOzIZqdB a5vBxb/CRBNUw8u57yN1/LosRNFulfDVhKdPBIuUQalhMEXCxQeekkL+1zybc04hYpl+SZnRfE xZqN1CggSikst3QcO1U+EpUK X-IronPort-AV: E=Sophos;i="5.88,207,1635231600"; d="scan'208";a="147304249" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 15 Dec 2021 04:01:20 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Wed, 15 Dec 2021 04:01:20 -0700 Received: from ROB-ULT-M18064N.mchp-main.com (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Wed, 15 Dec 2021 04:01:18 -0700 From: Tudor Ambarus To: Subject: [PATCH v3 00/12] dmaengine: at_xdmac: Various fixes Date: Wed, 15 Dec 2021 13:01:03 +0200 Message-ID: <20211215110115.191749-1-tudor.ambarus@microchip.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211215_030122_861374_77A1D085 X-CRM114-Status: UNSURE ( 7.69 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Tudor Ambarus , mripard@kernel.org, linux-kernel@vger.kernel.org, ludovic.desroches@microchip.com, dmaengine@vger.kernel.org, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Bugs identified when debugging a hang encountered when operating an octal DTR SPI NOR memory. The culprit was the flash, not the DMA driver, so all these bugs are not experienced in real life, they are all theoretical fixes. Nevertheless the bugs are there and I think they should be squashed. Tested the serial with DMA on sama5d2_xplained. Tested QSPI with DMA on sama7g5ek. All went well. v3: - drop tty patches, they were applied by Greg. - improve all commit descriptions - split v2's patch 13/13 in two: one removing a level of indentation and one fixing the race. v2: - drop local chan_rx local variable in patch 3/13, focus just on fixes for now. - collect Richard's Acked-by tag. - add details in the cover letter about what tests were performed. Tudor Ambarus (12): dmaengine: at_xdmac: Don't start transactions at tx_submit level dmaengine: at_xdmac: Start transfer for cyclic channels in issue_pending dmaengine: at_xdmac: Print debug message after realeasing the lock dmaengine: at_xdmac: Fix concurrency over chan's completed_cookie dmaengine: at_xdmac: Fix race for the tx desc callback dmaengine: at_xdmac: Move the free desc to the tail of the desc list dmaengine: at_xdmac: Fix concurrency over xfers_list dmaengine: at_xdmac: Remove a level of indentation in at_xdmac_advance_work() dmaengine: at_xdmac: Fix lld view setting dmaengine: at_xdmac: Fix at_xdmac_lld struct definition dmaengine: at_xdmac: Remove a level of indentation in at_xdmac_tasklet() dmaengine: at_xdmac: Fix race over irq_status drivers/dma/at_xdmac.c | 186 ++++++++++++++++++++--------------------- 1 file changed, 89 insertions(+), 97 deletions(-)