Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.1/patches/2230389/?format=api
{ "id": 2230389, "url": "http://patchwork.ozlabs.org/api/1.1/patches/2230389/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-mtd/patch/20260429-winbond-v6-18-rc1-cont-read-v3-5-0f38b3c229ad@bootlin.com/", "project": { "id": 3, "url": "http://patchwork.ozlabs.org/api/1.1/projects/3/?format=api", "name": "Linux MTD development", "link_name": "linux-mtd", "list_id": "linux-mtd.lists.infradead.org", "list_email": "linux-mtd@lists.infradead.org", "web_url": null, "scm_url": null, "webscm_url": null }, "msgid": "<20260429-winbond-v6-18-rc1-cont-read-v3-5-0f38b3c229ad@bootlin.com>", "date": "2026-04-29T17:56:42", "name": "[v3,05/11] spi: spi-mem: Create a secondary read operation", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "e7cb58836ae88e52bc5787208b0717008528c598", "submitter": { "id": 73368, "url": "http://patchwork.ozlabs.org/api/1.1/people/73368/?format=api", "name": "Miquel Raynal", "email": "miquel.raynal@bootlin.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-mtd/patch/20260429-winbond-v6-18-rc1-cont-read-v3-5-0f38b3c229ad@bootlin.com/mbox/", "series": [ { "id": 502120, "url": "http://patchwork.ozlabs.org/api/1.1/series/502120/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-mtd/list/?series=502120", "date": "2026-04-29T17:56:40", "name": "mtd: spinand: Winbond continuous read support", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/502120/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2230389/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2230389/checks/", "tags": {}, "headers": { "Return-Path": "\n <linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n secure) header.d=lists.infradead.org header.i=@lists.infradead.org\n header.a=rsa-sha256 header.s=bombadil.20210309 header.b=DF1c1L64;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n unprotected) header.d=bootlin.com header.i=@bootlin.com header.a=rsa-sha256\n header.s=dkim header.b=K0amzfyc;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=none (no SPF record) smtp.mailfrom=lists.infradead.org\n (client-ip=2607:7c80:54:3::133; helo=bombadil.infradead.org;\n envelope-from=linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org;\n receiver=patchwork.ozlabs.org)" ], "Received": [ "from bombadil.infradead.org (bombadil.infradead.org\n [IPv6:2607:7c80:54:3::133])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g5Q5K16Qcz1yHZ\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 30 Apr 2026 03:58:13 +1000 (AEST)", "from localhost ([::1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux))\n\tid 1wI9Al-000000040iy-3zq9;\n\tWed, 29 Apr 2026 17:58:07 +0000", "from smtpout-02.galae.net ([185.246.84.56])\n\tby bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux))\n\tid 1wI9Ab-000000040VW-2c5b\n\tfor linux-mtd@lists.infradead.org;\n\tWed, 29 Apr 2026 17:58:01 +0000", "from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233])\n\tby smtpout-02.galae.net (Postfix) with ESMTPS id 2B37A1A3453;\n\tWed, 29 Apr 2026 17:57:56 +0000 (UTC)", "from mail.galae.net (mail.galae.net [212.83.136.155])\n\tby smtpout-01.galae.net (Postfix) with ESMTPS id F35B55FD43;\n\tWed, 29 Apr 2026 17:57:55 +0000 (UTC)", "from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon)\n with ESMTPSA id 3FF5A1072B173;\n\tWed, 29 Apr 2026 19:57:50 +0200 (CEST)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20210309; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Cc:To:In-Reply-To:References:Message-Id\n\t:MIME-Version:Subject:Date:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=dZthBDWBtpz42D1LwWTcn+QZThKswwxJL0HMVzqYxXM=; b=DF1c1L64OK7Ie7\n\tGpDmnoeVHJ/8tLfy84LcM9NOIHkf5pgYnYIuAQlLIcXvqPp1EM67tvLgkMD6E1Rq5L4DS2wROZtJ7\n\tsIXTGjlNccb+g0xcVPk1EOT/52s+ctnI9BM8tBbghZLEK3vjaWe6iMvRG1i8DVaekjaGsUGvLSB5I\n\t6p/N/DxMF/iZxfUDmytNPR1r3uka/600n+EuVRjt5EG3+KplPT8bVVTwV3YrtqLkHgCA8n8yFHTjP\n\tbDxtUYnO9cXw4d1Ww1U2Vw9cxDOpUzrMFv1uf+BjKkkeL2FrUV5pwI501Y/v6qYplyZguvKDyT79R\n\tDDSG5ljvIOgwvSUXA21Q==;", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim;\n\tt=1777485474; h=from:subject:date:message-id:to:cc:mime-version:content-type:\n\t content-transfer-encoding:in-reply-to:references;\n\tbh=ChgtJK72vZUlsiBaKPcyKbWrS3ajSynsgsTRx3iHXAU=;\n\tb=K0amzfycc9kXe6uiG4uUt/UiAhIdiGMzKOl7+OtAUig03P9w5Oi4pi71lfOK4VzCWadfpc\n\tnS5TLgH8UeOpLv9oqplFB7SfTVmfJbMNIhuGL5isL94d1zok7pU+S8QA5DWBkSbgh28SJz\n\tDZearlJ5Cu/VO0kJdyM9+tAQscMA0+OXCO9beDN4ZJPGNWzVEq1MJI7sPFDuRqVpC63fLQ\n\tPic34fN8lxySr8+En0zBofRfLA+dt1XFD2tD6zmSWls95oUd8rvbzJXRh5bZmIbqP8kUKa\n\tE5JC9PYLpexMp71IOMLC7TSlwz1RJMfwhawHgZ7crGtjiqwa80VGQE5DnFcuTw==" ], "From": "Miquel Raynal <miquel.raynal@bootlin.com>", "Date": "Wed, 29 Apr 2026 19:56:42 +0200", "Subject": "[PATCH v3 05/11] spi: spi-mem: Create a secondary read operation", "MIME-Version": "1.0", "Message-Id": "\n <20260429-winbond-v6-18-rc1-cont-read-v3-5-0f38b3c229ad@bootlin.com>", "References": "\n <20260429-winbond-v6-18-rc1-cont-read-v3-0-0f38b3c229ad@bootlin.com>", "In-Reply-To": "\n <20260429-winbond-v6-18-rc1-cont-read-v3-0-0f38b3c229ad@bootlin.com>", "To": "Mark Brown <broonie@kernel.org>, Richard Weinberger <richard@nod.at>,\n Vignesh Raghavendra <vigneshr@ti.com>, Michael Walle <mwalle@kernel.org>,\n Miquel Raynal <miquel.raynal@bootlin.com>,\n Takahiro Kuwano <takahiro.kuwano@infineon.com>,\n Lorenzo Bianconi <lorenzo@kernel.org>, Ray Liu <ray.liu@airoha.com>,\n Chin-Ting Kuo <chin-ting_kuo@aspeedtech.com>, =?utf-8?q?C=C3=A9dric_Le_Goa?=\n\t=?utf-8?q?ter?= <clg@kaod.org>, Joel Stanley <joel@jms.id.au>,\n Andrew Jeffery <andrew@codeconstruct.com.au>,\n Avi Fishman <avifishman70@gmail.com>, Tomer Maimon <tmaimon77@gmail.com>,\n Tali Perry <tali.perry1@gmail.com>, Patrick Venture <venture@google.com>,\n Nancy Yuen <yuenn@google.com>, Benjamin Fair <benjaminfair@google.com>,\n Maxime Coquelin <mcoquelin.stm32@gmail.com>,\n Alexandre Torgue <alexandre.torgue@foss.st.com>, =?utf-8?q?Jonathan_Neusch?=\n\t=?utf-8?q?=C3=A4fer?= <j.neuschaefer@gmx.net>", "Cc": "Pratyush Yadav <pratyush@kernel.org>,\n Thomas Petazzoni <thomas.petazzoni@bootlin.com>,\n Steam Lin <STLin2@winbond.com>, Santhosh Kumar K <s-k6@ti.com>,\n linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org,\n linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org,\n linux-aspeed@lists.ozlabs.org, openbmc@lists.ozlabs.org,\n linux-stm32@st-md-mailman.stormreply.com", "X-Mailer": "b4 0.14.3", "X-Last-TLS-Session-Version": "TLSv1.3", "X-CRM114-Version": "20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ", "X-CRM114-CacheID": "sfid-20260429_105757_797949_6CDAA3E4 ", "X-CRM114-Status": "GOOD ( 19.02 )", "X-Spam-Score": "-2.1 (--)", "X-Spam-Report": "Spam detection software,\n running on the system \"bombadil.infradead.org\",\n has NOT identified this incoming email as spam. The original\n message has been attached to this so you can view it or label\n similar future email. If you have any questions, see\n the administrator of that system for details.\n Content preview: In some situations,\n direct mappings may need to use different\n operation templates. For instance, when enabling continuous reads, Winbond\n SPI NANDs no longer expect address cycles because they would be ignoring\n them otherwise. Hence, right after the command opcode, they start countin\n [...]\n Content analysis details: (-2.1 points, 5.0 required)\n pts rule name description\n ---- ----------------------\n --------------------------------------------------\n -0.0 SPF_PASS SPF: sender matches SPF record\n -0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n author's\n domain\n 0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n not necessarily valid\n -0.1 DKIM_VALID Message has at least one valid DKIM or DK\n signature\n -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from\n envelope-from domain\n -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n [score: 0.0000]", "X-BeenThere": "linux-mtd@lists.infradead.org", "X-Mailman-Version": "2.1.34", "Precedence": "list", "List-Id": "Linux MTD discussion mailing list <linux-mtd.lists.infradead.org>", "List-Unsubscribe": "<http://lists.infradead.org/mailman/options/linux-mtd>,\n <mailto:linux-mtd-request@lists.infradead.org?subject=unsubscribe>", "List-Archive": "<http://lists.infradead.org/pipermail/linux-mtd/>", "List-Post": "<mailto:linux-mtd@lists.infradead.org>", "List-Help": "<mailto:linux-mtd-request@lists.infradead.org?subject=help>", "List-Subscribe": "<http://lists.infradead.org/mailman/listinfo/linux-mtd>,\n <mailto:linux-mtd-request@lists.infradead.org?subject=subscribe>", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Sender": "\"linux-mtd\" <linux-mtd-bounces@lists.infradead.org>", "Errors-To": "linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org" }, "content": "In some situations, direct mappings may need to use different\noperation templates.\n\nFor instance, when enabling continuous reads, Winbond SPI NANDs no\nlonger expect address cycles because they would be ignoring them\notherwise. Hence, right after the command opcode, they start counting\ndummy cycles, followed by the data cycles as usual.\n\nThis breaks the assumptions of \"reads from cache\" always being done\nidentically once the best variant has been picked up, across the\nlifetime of the system.\n\nIn order to support this feature, we must give direct mapping more than\na single operation template to use, in order to switch to using\nsecondary operations upon request by the upper layer.\n\nCreate the concept of optional secondary operation template, which may\nor may not be fulfilled by the SPI NAND and SPI NOR cores. If the\nunderlying SPI controller does not leverage any kind of direct mapping\nacceleration, the feature has no impact and can be freely\nused. Otherwise, the controller driver needs to opt-in for using this\nfeature, if supported.\n\nThe condition checked to know whether a secondary operation has been\nprovided or not is to look for a non zero opcode to limit the creation\nof extra variables. In practice, the opcode 0x00 exist, but is not\nrelated to any cache related operation.\n\nAcked-by: Mark Brown <broonie@kernel.org>\nSigned-off-by: Miquel Raynal <miquel.raynal@bootlin.com>\n---\nThe choice of defining two variables named primary and secondary instead\nof using an array of templates is on purpose, to simplify the reading. I\nfind less obvious the use of an array here but this is personal taste.\n---\n drivers/spi/spi-mem.c | 17 +++++++++++++++++\n include/linux/spi/spi-mem.h | 5 +++++\n 2 files changed, 22 insertions(+)", "diff": "diff --git a/drivers/spi/spi-mem.c b/drivers/spi/spi-mem.c\nindex e2eaa1ba4ff6..f64eda9bbd9f 100644\n--- a/drivers/spi/spi-mem.c\n+++ b/drivers/spi/spi-mem.c\n@@ -713,6 +713,23 @@ spi_mem_dirmap_create(struct spi_mem *mem,\n \tif (info->primary_op_tmpl.data.dir == SPI_MEM_NO_DATA)\n \t\treturn ERR_PTR(-EINVAL);\n \n+\t/* Apply similar constraints to the secondary template */\n+\tif (info->secondary_op_tmpl.cmd.opcode) {\n+\t\tif (!info->secondary_op_tmpl.addr.nbytes ||\n+\t\t info->secondary_op_tmpl.addr.nbytes > 8)\n+\t\t\treturn ERR_PTR(-EINVAL);\n+\n+\t\tif (info->secondary_op_tmpl.data.dir == SPI_MEM_NO_DATA)\n+\t\t\treturn ERR_PTR(-EINVAL);\n+\n+\t\tif (!spi_mem_supports_op(mem, &info->secondary_op_tmpl))\n+\t\t\treturn ERR_PTR(-EOPNOTSUPP);\n+\n+\t\tif (ctlr->mem_ops && ctlr->mem_ops->dirmap_create &&\n+\t\t !spi_mem_controller_is_capable(ctlr, secondary_op_tmpl))\n+\t\t\treturn ERR_PTR(-EOPNOTSUPP);\n+\t}\n+\n \tdesc = kzalloc_obj(*desc);\n \tif (!desc)\n \t\treturn ERR_PTR(-ENOMEM);\ndiff --git a/include/linux/spi/spi-mem.h b/include/linux/spi/spi-mem.h\nindex 9a96ddace3eb..2012a3b2ef91 100644\n--- a/include/linux/spi/spi-mem.h\n+++ b/include/linux/spi/spi-mem.h\n@@ -227,6 +227,8 @@ struct spi_mem_op {\n * struct spi_mem_dirmap_info - Direct mapping information\n * @op_tmpl: operation template that should be used by the direct mapping when\n *\t the memory device is accessed\n+ * @secondary_op_tmpl: secondary template, may be used as an alternative to the\n+ * primary template (decided by the upper layer)\n * @offset: absolute offset this direct mapping is pointing to\n * @length: length in byte of this direct mapping\n *\n@@ -239,6 +241,7 @@ struct spi_mem_op {\n struct spi_mem_dirmap_info {\n \tstruct spi_mem_op *op_tmpl;\n \tstruct spi_mem_op primary_op_tmpl;\n+\tstruct spi_mem_op secondary_op_tmpl;\n \tu64 offset;\n \tu64 length;\n };\n@@ -382,12 +385,14 @@ struct spi_controller_mem_ops {\n * @swap16: Supports swapping bytes on a 16 bit boundary when configured in\n *\t Octal DTR\n * @per_op_freq: Supports per operation frequency switching\n+ * @secondary_op_tmpl: Supports leveraging a secondary memory operation template\n */\n struct spi_controller_mem_caps {\n \tbool dtr;\n \tbool ecc;\n \tbool swap16;\n \tbool per_op_freq;\n+\tbool secondary_op_tmpl;\n };\n \n #define spi_mem_controller_is_capable(ctlr, cap)\t\\\n", "prefixes": [ "v3", "05/11" ] }