Re: [PATCH] mtd: spi-nor: Enable locking for n25q00a

From: "Michael Walle" <mwalle@kernel.org>
To: "Miquel Raynal" <miquel.raynal@bootlin.com>,
	"Sean Anderson" <sean.anderson@linux.dev>
Cc: "Tudor Ambarus" <tudor.ambarus@linaro.org>,
	"Pratyush Yadav" <pratyush@kernel.org>,
	<linux-mtd@lists.infradead.org>,
	"Richard Weinberger" <richard@nod.at>,
	<linux-kernel@vger.kernel.org>,
	"Vignesh Raghavendra" <vigneshr@ti.com>
Subject: Re: [PATCH] mtd: spi-nor: Enable locking for n25q00a
Date: Wed, 12 Nov 2025 14:34:20 +0100	[thread overview]
Message-ID: <DE6R76H9N8H0.NKHN0YUTE8LL@kernel.org> (raw)
In-Reply-To: <87v7jfgzfw.fsf@bootlin.com>

[-- Attachment #1.1: Type: text/plain, Size: 617 bytes --]

On Wed Nov 12, 2025 at 2:20 PM CET, Miquel Raynal wrote:
> I forgot to mention: I don't like this interface because it is not very
> user friendly, but this is uAPI, so set in stone. As part of my journey
> in the SPI NOR swp.c file, I wrote a debugfs interface to help
> visualizing what is actually locked. It is absolutely trivial to do and
> helps a lot. We might want to use that for writing some kind of testing
> procedure — I will share it soon.

Yes please, that sounds very good. We might also want to just raw
dump the SR (and maybe CR/SR2 if we somehow can infer it's
existence).

-michael

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 297 bytes --]

[-- Attachment #2: Type: text/plain, Size: 144 bytes --]

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/