[coreboot-gerrit] Change in coreboot[master]: commonlib/region: [WIP] Add region protect API

[Rizwan Qureshi (Code Review)]

Rizwan Qureshi has uploaded this change for review. ( https://review.coreboot.org/28724

Change subject: commonlib/region: [WIP] Add region protect API
......................................................................

commonlib/region: [WIP] Add region protect API

Add "protect" operation to the region_device_ops.
This can be used for write protecting the region that is
represented by the device.
Also add an API redev_protect() which will expose the protect
operation to the library user.

Change-Id: I4c9376e2c2c7a4852f13c65824c6cd64a1c6ac0a
Signed-off-by: Rizwan Qureshi 
---
M src/commonlib/include/commonlib/region.h
M src/commonlib/region.c
2 files changed, 22 insertions(+), 4 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/28724/1

diff --git a/src/commonlib/include/commonlib/region.h b/src/commonlib/include/commonlib/region.h
index 45484dd..901fd35 100644
--- a/src/commonlib/include/commonlib/region.h
+++ b/src/commonlib/include/commonlib/region.h
@@ -63,6 +63,11 @@
 ssize_t rdev_eraseat(const struct region_device *rd, size_t offset,
    		size_t size);

+/*
+ * Returns < 0 on error otherwise returns 0 on success, < 0 0n error.
+ */
+int rdev_protect(const struct region_device *rd);
+
 /****************************************
  *  Implementation of a region device   *
  ****************************************/
@@ -75,6 +80,10 @@
 int rdev_chain(struct region_device *child, const struct region_device *parent,
    	size_t offset, size_t size);

+struct region {
+	size_t offset;
+	size_t size;
+};

/* A region_device operations. */
 struct region_device_ops {
@@ -84,12 +93,9 @@
    ssize_t (*writeat)(const struct region_device *, const void *, size_t,
    	size_t);
    ssize_t (*eraseat)(const struct region_device *, size_t, size_t);
+	int (*protect)(const struct region *);
 };

-struct region {
-	size_t offset;
-	size_t size;
-};

struct region_device {
    const struct region_device *root;
diff --git a/src/commonlib/region.c b/src/commonlib/region.c
index 541a125..58b5fa4 100644
--- a/src/commonlib/region.c
+++ b/src/commonlib/region.c
@@ -151,6 +151,18 @@
    return rdev->ops->eraseat(rdev, req.offset, req.size);
 }

+int rdev_protect(const struct region_device *rd)
+{
+	const struct region_device *rdev;
+
+	rdev = rdev_root(rd);
+
+	if (rdev->ops->protect == NULL)
+		return -1;
+
+	return rdev->ops->protect(region_device_region(rdev));
+}
+
 int rdev_chain(struct region_device *child, const struct region_device *parent,
    	size_t offset, size_t size)
 {