Re: [PATCH v2] mtd: spi-nor: winbond: Add support for W77Q51NW

["Michael Walle" <mwalle@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1146 bytes --]

On Tue Jul 29, 2025 at 5:51 AM CEST, Marek Vasut wrote:
> On 7/28/25 8:32 AM, Michael Walle wrote:
> >>>> I have limited supply of these devices, so OTP is untested. The flash
> >>>> does have OTP registers, that's why the .otp entry is there. Why should
> >>>> I remove it if the OTP registers are in the chip ?
> >>>
> >>> We only add tested features. I'm just the messenger here :o. Anyway,
> >>> OTP is not really one-time-programmable here, you can write and
> >>> erase it again as long as you don't lock it, if that was your
> >>> concern.
> >>
> >> So how do I test the OTP without locking it ?
> > 
> > flash_otp_{write,info,dump,erase}, just don't use flash_otp_lock.
>
> Thanks. flash_otp_dump -u /dev/mtd0 returns zeroes, so I suspect the OTP 
> is not working. The chip does work even without this entry and the 
> content /sys/kernel/debug/spi-nor/spi0.0/capabilities and 
> /sys/kernel/debug/spi-nor/spi0.0/params did not change, so I think the 
> best way forward is to drop this patch, until I can figure out the OTP 
> on this chip ?

Probably. Good to know that the flash is working for you.

-michael

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