From: Brian Norris <computersforpeace@gmail.com>
To: <linux-mtd@lists.infradead.org>
Cc: Brian Norris <computersforpeace@gmail.com>
Subject: [PATCH mtd-utils 00/11] flash_{un,}lock upgrades
Date: Mon, 31 Aug 2015 15:34:21 -0700 [thread overview]
Message-ID: <1441060472-82169-1-git-send-email-computersforpeace@gmail.com> (raw)
Hi,
I've done a bit of work to make the flash_lock tool more useful, in order to
help test some new SPI NOR block protection features. Of note:
* added getopt support
* further unified flash_unlock and flash_lock -- the only difference now is
their default feature, and both binaries contain support for all features
* add support for MEMISLOCKED, so we can query arbitrary regions for
protection status
* mtdinfo -M already can dump some block lock information, but you get a
little more flexibility using flash_lock --info, so both seem worth having
IMO
Unsolved issues:
* the MEM{LOCK,UNLOCK,ISLOCKED} ioctls all still use the out-dated 32-bit
erase_info_user struct, which means they'll have problems supporting >=4GB
flash. This isn't a mtd-utils problem, per se, and at the moment, Linux NAND
(the only (?) candidate for >=4GB flash) support doesn't implement any of
the locking APIs. But just an observation I ran across.
* Deprecation plan: it would make sense to deprecate and remove flash_unlock
eventually (in favor of 'flash_lock --unlock'), but this isn't really
pressing, so I don't see a good reason to spend much effort on it.
Enjoy,
Brian
Brian Norris (11):
flash_{un,}lock: nest optional parameters in help message
flash_{un,}lock: switch to getopt library
flash_{un,}lock: support --version flag
flash_{un,}lock: document option flags
flash_{un,}lock: abstract the argument positions
flash_{un,}lock: move args processing to its own function
flash_{un,}lock: support both lock/unlock in the same binary
flash_{un,}lock: add MEMISLOCKED support
flash_{un,}lock: improve strtol() error handling
flash_{un,}lock: don't allow "last byte + 1"
flash_{un,}lock: document block count == -1
flash_unlock.c | 182 ++++++++++++++++++++++++++++++++++++++++++++++++---------
1 file changed, 156 insertions(+), 26 deletions(-)
--
2.5.0.457.gab17608
next reply other threads:[~2015-08-31 22:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-31 22:34 Brian Norris [this message]
2015-08-31 22:34 ` [PATCH mtd-utils 01/11] flash_{un, }lock: nest optional parameters in help message Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 02/11] flash_{un,}lock: switch to getopt library Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 03/11] flash_{un,}lock: support --version flag Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 04/11] flash_{un,}lock: document option flags Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 05/11] flash_{un, }lock: abstract the argument positions Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 06/11] flash_{un, }lock: move args processing to its own function Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 07/11] flash_{un, }lock: support both lock/unlock in the same binary Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 08/11] flash_{un,}lock: add MEMISLOCKED support Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 09/11] flash_{un, }lock: improve strtol() error handling Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 10/11] flash_{un,}lock: don't allow "last byte + 1" Brian Norris
2015-08-31 22:34 ` [PATCH mtd-utils 11/11] flash_{un,}lock: document block count == -1 Brian Norris
2015-11-11 22:13 ` [PATCH mtd-utils 00/11] flash_{un,}lock upgrades Brian Norris
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1441060472-82169-1-git-send-email-computersforpeace@gmail.com \
--to=computersforpeace@gmail.com \
--cc=linux-mtd@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.