Hi,

+static int do_mtd_otp_write(struct cmd_tbl *cmdtp, int flag, int argc,
+                char *const argv[])
+{
..

+    printf("Caution! OTP data bits can't be erased! Continue (y/n)?\n");

Please note, that with current SPI-NOR flashes this is not true and
there is usually some kind of erase command for the OTP bits. Only
the region lock is permanent and with that set, no more write or erase
is possible.

I see, so may be just rephrase this message, like just "Continue? (y/n)?"

I don't have any strong opinion here. The code may or may not be used
to write to the OTP of an SPI-NOR flash. Also, this patchset doesn't
support the erase op (I guess u-boot's MTD won't support it either). So
from an user's perspective these OTP data can't be erased ;)

-michael

Reply via email to