[flashrom] [PATCH] Add support for GigaDevice GD25VQ41B

Stefan Tauner stefan.tauner at alumni.tuwien.ac.at
Fri Feb 12 22:42:08 CET 2016


On Fri, 12 Feb 2016 12:39:51 +0200
Urja Rannikko <urjaman at gmail.com> wrote:

> >> +             .feature_bits   = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_QPI,  
> These arent "truly/100%" OTP, but one-time-lockable "security
> registers" - the effect is most likely
> the same (=if they're used they're most likely locked and that makes
> them effectively ROM), so
> I'm ok with the FEATURE_OTP, but maybe change the comment, my suggestion:
> /* Lockable Security Registers: 3 * 512B pages, read 0x48, write 0x42,
> erase 0x44 */

Hm... That is most likely my fault. I think there are quite some other
chips where I deliberately added the OTP comment (and flag) in the same
form although they are one-time-lock-only.
Rationale: I wanted to make any chip with any permanently lockable bits
easily recognizable before starting to work on adding support for it.

Therefore I think the original comment is the way to go for the time
being (no need to resend the patch though, Hatim).

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner




More information about the flashrom mailing list