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 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 A663FC28B30 for ; Thu, 20 Mar 2025 11:32:08 +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:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=sDE3FP9MMei9OGkWxGgOkSQsozkDSHKT7GetEyH6Vbc=; b=gyi/GyWQstoMue vHORsAoyNR1zlr3nnkhUG8/WBoF3ebJ+dhqNrl6yeLLbQhJRPLsIzKmHgX0mxOYjnlhn8FJDqofXN Ky4SeU6ZJVomO/7nberzvySj0XCVhpix8e3PSLTEGH2PmPGahe/TMR9ky6nhoItL1yGf0JUSB4R3m vXMNEBQlZV5oUWUov6WFSBxTCJiNr9azqy7+6VOo0UobV5xNuMHLr6hDZPI3R21w37ebtWUgyJ/Xq h+uO9g4EkewkiU9gsg+aemWxE+neAFn+Zsj0XKPZXGQzCD367/d2UD4cJsqWeBDj/ZFzwkFPsJa2l eMoTECBjsFfLpOYvzVEA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tvE85-0000000BxKS-2VRK; Thu, 20 Mar 2025 11:32:05 +0000 Received: from relay2-d.mail.gandi.net ([217.70.183.194]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tvE82-0000000BxK4-0zL3 for linux-mtd@lists.infradead.org; Thu, 20 Mar 2025 11:32:04 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id DEE7543150; Thu, 20 Mar 2025 11:31:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1742470319; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=CWcS7+LaqjyqkZYE4vdWzKlwoH9+A6aKT6QyHshPnaQ=; b=P9K6ugwTO5leK8GhtCbUb1gS4TsDUg1MMCfoFYEMUHnzfjo9NOyJgV9vAVd+IfShl0JvLt wf8xjXLHDumMs8GFxwFZVkjOxmR3ty7vc4FaqlMFjV6fTVxyqVKHkd7U6dJIdWxeZibCO4 Sfo/8NByF4Nf0NehnKfkMOPgx+9cv+Chsqu0lWcqAERS3nzkOTXLsaqQsuWGtd+l9dehVh g4wNO4Ev3O6h4nTINUUjTppwm+t8UapIREdpcvZJxujPhvODlflgJe2YYDQhm7A750mGSJ egj4gHhqdftxnptwAlRt5QXbSyMcFVAIIrR3CfRMqIwITP1+H7/nnz8mLJSwCw== From: Miquel Raynal To: Tudor Ambarus Cc: Richard Weinberger , Vignesh Raghavendra , Santhosh Kumar K , Pratyush Yadav , Michael Walle , Thomas Petazzoni , Steam Lin , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 01/21] mtd: spinand: Use more specific naming for the reset op In-Reply-To: (Tudor Ambarus's message of "Thu, 20 Mar 2025 08:08:50 +0000") References: <20250307-winbond-6-14-rc1-octal-v1-0-45c1e074ad74@bootlin.com> <20250307-winbond-6-14-rc1-octal-v1-1-45c1e074ad74@bootlin.com> <9004166e-5535-4024-8114-9fdb217407bb@linaro.org> <87v7skrgjl.fsf@bootlin.com> <87iko50vvb.fsf@bootlin.com> User-Agent: mu4e 1.12.7; emacs 29.4 Date: Thu, 20 Mar 2025 12:31:55 +0100 Message-ID: <87msdgvsyc.fsf@bootlin.com> MIME-Version: 1.0 X-GND-State: clean X-GND-Score: -100 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeefvddrtddtgddugeekuddtucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuifetpfffkfdpucggtfgfnhhsuhgsshgtrhhisggvnecuuegrihhlohhuthemuceftddunecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjughrpefhvfevufgjfhgffffkgggtgfesthhqredttderjeenucfhrhhomhepofhiqhhuvghlucftrgihnhgrlhcuoehmihhquhgvlhdrrhgrhihnrghlsegsohhothhlihhnrdgtohhmqeenucggtffrrghtthgvrhhnpeegvdduleeihfettdfgieevtdffjeeggfefveetudeludekieffhfeiffduvdehffenucffohhmrghinhepfihinhgsohhnugdrtghomhenucfkphepledvrddukeegrdduuddtrdeftdenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepihhnvghtpeelvddrudekgedruddutddrfedtpdhhvghloheplhhotggrlhhhohhsthdpmhgrihhlfhhrohhmpehmihhquhgvlhdrrhgrhihnrghlsegsohhothhlihhnrdgtohhmpdhnsggprhgtphhtthhopedutddprhgtphhtthhopehtuhguohhrrdgrmhgsrghruhhssehlihhnrghrohdrohhrghdprhgtphhtthhopehrihgthhgrrhgusehnohgurdgrthdprhgtphhtthhopehvihhgnhgvshhhrhesthhirdgtohhmpdhrtghpthhtohepshdqkheisehtihdrtghomhdprhgtphhtthhopehprhgrthihuhhshheskhgvrhhnvghlrdhorhhgpdhrtghpthhto hepmhhitghhrggvlhesfigrlhhlvgdrtggtpdhrtghpthhtohepthhhohhmrghsrdhpvghtrgiiiihonhhisegsohhothhlihhnrdgtohhmpdhrtghpthhtohepshhtlhhinhdvseifihhnsghonhgurdgtohhm X-GND-Sasl: miquel.raynal@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250320_043202_888946_325CDAEF X-CRM114-Status: GOOD ( 18.66 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org Cj4+PiBIb3cgZG8geW91IGZlZWwgYWJvdXQgYSBPUC1Bbi1Cbi1Dbi10cmFuc2FjdGlvbi1mb3Jt YXQsCj4+PiAJd2hlcmUgQSwgQiwgQyBpcyB7MSwgMiwgNCwgOH0KPj4+IAluIGlzIHtTLCBEfQo+ Pj4gCXRyYW5zYWN0aW9uIGZvcm1hdCBpcyB7MCwxLDIsM317QSxCLEMsLi4ufQo+Pj4KPj4+IENh cmUgbXVzdCBiZSB0YWtlbiBjYXJlIG9mIGF0IHRoZSB0cmFuc2FjdGlvbiBmb3JtYXQsIGFzIEkg c2VlIHRoZXJlIGFyZQo+Pj4gYSBmZXcsIGRlcGVuZGluZyBvbiB0aGUgeFNQSSBwcm9maWxlIGFu ZCBwcm90b2NvbCBtb2RlLgo+PiAKPj4gSSBtdXN0IGFkbWl0IEkgcmVhbGx5IGRpc2xpa2UgdGhl IHRyYW5zYWN0aW9uIGZvcm1hdCBhYmJyZXZpYXRpb24gYmVjYXVzZQo+Cj4gSSdtIG5vdCBpbiBs b3ZlIHdpdGggdGhlICJ0cmFuc2FjdGlvbiBmb3JtYXQiIGVpdGhlci4KPgo+PiBpdCBpcyBzcGVj aWZpYyB0byBhIHNwZWMgd2UgZG8gbm90IGZvbGxvdyBhbmQgaXMgbm90IGV4cGxpY2l0LiBJIGhh dmUgYQo+PiBzdHJvbmcgcHJlZmVyZW5jZSB0b3dhcmRzIGtlZXBpbmcganVzdCAiQW4tQm4tQ24i LCBsaWtlIEkgZGlkIGluIHRoaXMKPgo+IE9rYXkuCj4KPj4gc2VyaWVzLiBJZiBwZW9wbGUgYXJl IGluIGRvdWJ0LCB0aGV5IGNhbiBjaGVjayB3aGF0IGlzIGluIHRoZSBvcCwgaXQgaXMKPj4gc2Vs ZiBleHBsYW5hdG9yeS4gSSBjaG9zZSB0aGlzIGluIHRoZSBmaXJzdCBwbGFjZSBiZWNhdXNlIGl0 IGlzIG1vcmUKPj4gZXhwbGljaXQgdGhhbiBqdXN0ICJBbiIgd2hpY2ggbWF5IGZlZWwgbGlrZSBh IGNyb3BwZWQKPj4gYWNyb255bS4gRnVydGhlcm1vcmUsIG1vc3Qgb2N0YWwgY2FwYWJsZSBkYXRh c2hlZXRzIEkndmUgbG9va2VkIGF0IHNlZW0KPj4gdG8gdXNlIHRoZSAiQW4tQm4tQ24iIGZvcm1h dCwgZXZlbiBmb3Igc2luZ2xlIG9wY29kZXMuCj4KPiBTbyBmb3IgUkVTRVQsIFdSRU4sIFdSREkg YW5kIG90aGVyIG9wY29kZXMgdGhhdCBkb24ndCByZXF1aXJlIGFkZHJlc3MKPiBhbmQgZGF0YSwg YXJlIHRob3NlIGRhdGFzaGVldHMgcmVmZXJyaW5nIHRvIHRoZXNlIG9wY29kZXMgYXMgMS0xLTEg Y29tbWFuZHM/Cj4gQ2FuIHlvdSBwb2ludCBtZSB0byBvbmUgb2YgdGhlc2UgZGF0YXNoZWV0cywg cGxlYXNlPwoKQWxsIFdpbmJvbmQgZGF0YXNoZWV0cyB1c2UgdGhpcyBjb252ZW50aW9uOgpodHRw czovL3d3dy53aW5ib25kLmNvbS9leHBvcnQvc2l0ZXMvd2luYm9uZC9kYXRhc2hlZXQvVzM1TjAx SldfRGF0YXNoZWV0X0JyaWVmLnBkZgpTZWUgNy4xLjIgSW5zdHJ1Y3Rpb24gU2V0IFRhYmxlIHAu MjUuCgpJIGNoZWNrZWQgTWFjcm9uaXggYW5kIEdpZ2FkZXZpY2UgZGF0YXNoZWV0cywgdGhleSBz aW1wbHkgbmV2ZXIgIm5hbWUiCnRoZXNlIG9wY29kZXMgY2xlYXJseS4KCj4gSW4gU1BJIE5PUiwg V1JFTiBpcyBqdXN0IFdSRU4sIHJlZ2FyZGxlc3Mgb2YgdGhlIG51bWJlciBvZiBsaW5lcyBpdCBp cwo+IHNlbnQgb24sIDEsIDQsIG9yIDguIEhvdyB3b3VsZCB5b3UgbW9kZWwgaXQgaGVyZSwgMS0w LTAsIDQtMC0wLCA4LTAtMAo+IHVzaW5nIHRoZSBzYW1lIG9wY29kZT8KCkkgZ3Vlc3MsIHllcy4K Cj4+IFNvIHVubGVzcyB0aGVyZSBhcmUgc3Ryb25nIGFyZ3VtZW50cyBhZ2FpbnN0LCBJJ2QgcHJl ZmVyIHRvIGtlZXAgdGhlCj4+IGN1cnJlbnQgZm9ybWF0dGluZy4gVEJILCBpdCBpcyBub3Qgc2V0 IGluIHN0b25lIGFuZCBjYW4gc3RpbGwgZXZvbHZlCj4+IGxhdGVyIGlmIHRoYXQncyBuZWVkZWQu Cj4KPiBJIGZpbmQgdGhlIG5hbWluZyBzY2hlbWUgQW4tQm4tQ24gZ29vZCwgSSdtIGFyZ3Vpbmcg dGhhdCBpdCdzIG5vdCBuZWVkZWQKPiBldmVyeXdoZXJlLCBvcGNvZGUgb25seSBjb21tYW5kcyBh cmUgYmV0dGVyIG9mZiB3aXRoIGl0LCBJIHRoaW5rLgoKT2ssIEkgY2FuIGRyb3AgdGhlIC0wLTAg aW4gdGhpcyBjYXNlIGlmIHlvdSBwcmVmZXIuCgpUaGFua3MsCk1pcXXDqGwKCgpfX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTGludXggTVREIGRp c2N1c3Npb24gbWFpbGluZyBsaXN0Cmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4v bGlzdGluZm8vbGludXgtbXRkLwo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net [217.70.183.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4C6A1189F36 for ; Thu, 20 Mar 2025 11:32:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.194 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742470329; cv=none; b=aKKk8FIzP/NZSdRLfI86fMjhfrwqLtPJrc+K6PAb/oOnV3ldggL//8bKBY0ITq9FPQJudpdxOfLWeeW40PBBC+0hWxtYM8piyVTDYPaDTr1FEpFbTtWvlyT7VpbZ2e3xPz7aPBLNd+c+gs9gOM83Qzt6WyjNoUOee00jPpoMgOs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742470329; c=relaxed/simple; bh=bIWxxHLaEUs7nKR8UquB5Bxi4qsYCUmHqTaYzR326DA=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=aZ4wr/9IC60PDuu/fyoPgDAzU02jVmpYGycfHqWnq8CVk2AxlfFm33TJyK80a3qTuUmd9IHkX5Ef8xmMk4TujIavlo3L+U7O+Ip2e8vaD0UpLdR2RaU04Q7+Y7lAkWqlN0P2gGub0dVvqNpn6gbCZ2VZRGdcrPWJLw9qBojGT5A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=P9K6ugwT; arc=none smtp.client-ip=217.70.183.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="P9K6ugwT" Received: by mail.gandi.net (Postfix) with ESMTPSA id DEE7543150; Thu, 20 Mar 2025 11:31:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1742470319; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=CWcS7+LaqjyqkZYE4vdWzKlwoH9+A6aKT6QyHshPnaQ=; b=P9K6ugwTO5leK8GhtCbUb1gS4TsDUg1MMCfoFYEMUHnzfjo9NOyJgV9vAVd+IfShl0JvLt wf8xjXLHDumMs8GFxwFZVkjOxmR3ty7vc4FaqlMFjV6fTVxyqVKHkd7U6dJIdWxeZibCO4 Sfo/8NByF4Nf0NehnKfkMOPgx+9cv+Chsqu0lWcqAERS3nzkOTXLsaqQsuWGtd+l9dehVh g4wNO4Ev3O6h4nTINUUjTppwm+t8UapIREdpcvZJxujPhvODlflgJe2YYDQhm7A750mGSJ egj4gHhqdftxnptwAlRt5QXbSyMcFVAIIrR3CfRMqIwITP1+H7/nnz8mLJSwCw== From: Miquel Raynal To: Tudor Ambarus Cc: Richard Weinberger , Vignesh Raghavendra , Santhosh Kumar K , Pratyush Yadav , Michael Walle , Thomas Petazzoni , Steam Lin , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 01/21] mtd: spinand: Use more specific naming for the reset op In-Reply-To: (Tudor Ambarus's message of "Thu, 20 Mar 2025 08:08:50 +0000") References: <20250307-winbond-6-14-rc1-octal-v1-0-45c1e074ad74@bootlin.com> <20250307-winbond-6-14-rc1-octal-v1-1-45c1e074ad74@bootlin.com> <9004166e-5535-4024-8114-9fdb217407bb@linaro.org> <87v7skrgjl.fsf@bootlin.com> <87iko50vvb.fsf@bootlin.com> User-Agent: mu4e 1.12.7; emacs 29.4 Date: Thu, 20 Mar 2025 12:31:55 +0100 Message-ID: <87msdgvsyc.fsf@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-GND-State: clean X-GND-Score: -100 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeefvddrtddtgddugeekuddtucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuifetpfffkfdpucggtfgfnhhsuhgsshgtrhhisggvnecuuegrihhlohhuthemuceftddunecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjughrpefhvfevufgjfhgffffkgggtgfesthhqredttderjeenucfhrhhomhepofhiqhhuvghlucftrgihnhgrlhcuoehmihhquhgvlhdrrhgrhihnrghlsegsohhothhlihhnrdgtohhmqeenucggtffrrghtthgvrhhnpeegvdduleeihfettdfgieevtdffjeeggfefveetudeludekieffhfeiffduvdehffenucffohhmrghinhepfihinhgsohhnugdrtghomhenucfkphepledvrddukeegrdduuddtrdeftdenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepihhnvghtpeelvddrudekgedruddutddrfedtpdhhvghloheplhhotggrlhhhohhsthdpmhgrihhlfhhrohhmpehmihhquhgvlhdrrhgrhihnrghlsegsohhothhlihhnrdgtohhmpdhnsggprhgtphhtthhopedutddprhgtphhtthhopehtuhguohhrrdgrmhgsrghruhhssehlihhnrghrohdrohhrghdprhgtphhtthhopehrihgthhgrrhgusehnohgurdgrthdprhgtphhtthhopehvihhgnhgvshhhrhesthhirdgtohhmpdhrtghpthhtohepshdqkheisehtihdrtghomhdprhgtphhtthhopehprhgrthihuhhshheskhgvrhhnvghlrdhorhhgpdhrtghpthhto hepmhhitghhrggvlhesfigrlhhlvgdrtggtpdhrtghpthhtohepthhhohhmrghsrdhpvghtrgiiiihonhhisegsohhothhlihhnrdgtohhmpdhrtghpthhtohepshhtlhhinhdvseifihhnsghonhgurdgtohhm X-GND-Sasl: miquel.raynal@bootlin.com >>> How do you feel about a OP-An-Bn-Cn-transaction-format, >>> where A, B, C is {1, 2, 4, 8} >>> n is {S, D} >>> transaction format is {0,1,2,3}{A,B,C,...} >>> >>> Care must be taken care of at the transaction format, as I see there are >>> a few, depending on the xSPI profile and protocol mode. >>=20 >> I must admit I really dislike the transaction format abbreviation because > > I'm not in love with the "transaction format" either. > >> it is specific to a spec we do not follow and is not explicit. I have a >> strong preference towards keeping just "An-Bn-Cn", like I did in this > > Okay. > >> series. If people are in doubt, they can check what is in the op, it is >> self explanatory. I chose this in the first place because it is more >> explicit than just "An" which may feel like a cropped >> acronym. Furthermore, most octal capable datasheets I've looked at seem >> to use the "An-Bn-Cn" format, even for single opcodes. > > So for RESET, WREN, WRDI and other opcodes that don't require address > and data, are those datasheets referring to these opcodes as 1-1-1 comman= ds? > Can you point me to one of these datasheets, please? All Winbond datasheets use this convention: https://www.winbond.com/export/sites/winbond/datasheet/W35N01JW_Datasheet_B= rief.pdf See 7.1.2 Instruction Set Table p.25. I checked Macronix and Gigadevice datasheets, they simply never "name" these opcodes clearly. > In SPI NOR, WREN is just WREN, regardless of the number of lines it is > sent on, 1, 4, or 8. How would you model it here, 1-0-0, 4-0-0, 8-0-0 > using the same opcode? I guess, yes. >> So unless there are strong arguments against, I'd prefer to keep the >> current formatting. TBH, it is not set in stone and can still evolve >> later if that's needed. > > I find the naming scheme An-Bn-Cn good, I'm arguing that it's not needed > everywhere, opcode only commands are better off with it, I think. Ok, I can drop the -0-0 in this case if you prefer. Thanks, Miqu=C3=A8l