[flashrom-gerrit] Change in flashrom[master]: writeprotect.h: add structure to represent chip wp configuration bits

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/58478 )

Change subject: writeprotect.h: add structure to represent chip wp configuration bits
......................................................................

writeprotect.h: add structure to represent chip wp configuration bits

Add `struct wp_bits` for representing values of all WP bits in a chip's
status/config register(s).

It allows most WP code to store and manipulate a chip's configuration
without knowing the exact layout of bits in the chip's status registers.

Supporting other chips may require additional fields to be added to the
structure.

BUG=b:195381327,b:153800563
BRANCH=none
TEST=flashrom --wp-{enable,disable,range,list,status} at end of patch series

Change-Id: I17dee630248ce7b51e624a6e46d7097d5d0de809
Signed-off-by: Nikolai Artemiev 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/58478
Tested-by: build bot (Jenkins) 
Reviewed-by: Anastasia Klimchuk 
Reviewed-by: Edward O'Callaghan 
Reviewed-by: Nico Huber 
---
M writeprotect.h
1 file changed, 36 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved
  Edward O'Callaghan: Looks good to me, approved
  Anastasia Klimchuk: Looks good to me, approved

diff --git a/writeprotect.h b/writeprotect.h
index 8510226..2f473f7 100644
--- a/writeprotect.h
+++ b/writeprotect.h
@@ -18,6 +18,42 @@
 #ifndef __WRITEPROTECT_H__
 #define __WRITEPROTECT_H__ 1

+#include <stdint.h>
+#include <stdbool.h>
+#include <stddef.h>
+
 #define MAX_BP_BITS 4

+/*
+ * Description of a chip's write protection configuration.
+ *
+ * It allows most WP code to store and manipulate a chip's configuration
+ * without knowing the exact layout of bits in the chip's status registers.
+ */
+struct wp_bits  {
+	/* Status register protection bit (SRP) */
+	bool srp_bit_present;
+	uint8_t srp;
+
+	/* Status register lock bit (SRL) */
+	bool srl_bit_present;
+	uint8_t srl;
+
+	/* Complement bit (CMP) */
+	bool cmp_bit_present;
+	uint8_t cmp;
+
+	/* Sector/block protection bit (SEC) */
+	bool sec_bit_present;
+	uint8_t sec;
+
+	/* Top/bottom protection bit (TB) */
+	bool tb_bit_present;
+	uint8_t tb;
+
+	/* Block protection bits (BP) */
+	size_t bp_bit_count;
+	uint8_t bp[MAX_BP_BITS];
+};
+
 #endif /* !__WRITEPROTECT_H__ */

38 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.