[coreboot-gerrit] Patch set updated for coreboot: commonlib/region: Add helpers for dynamic initialization of region dev

[Furquan Shaikh (furquan@google.com)]

Furquan Shaikh (furquan at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15267

-gerrit

commit c98c5e0242d327faa9eb75b62c48331d64245f04
Author: Furquan Shaikh <furquan at google.com>
Date:   Sun Jun 19 23:16:45 2016 -0700

    commonlib/region: Add helpers for dynamic initialization of region dev
    
    This allows initialization of runtime region devices and xlate region
    devices where all parameters cannot be statically determined.
    
    BUG=chrome-os-partner:54563
    
    Change-Id: Ia6e1b695fed3bbfa08598d1593e650fc1465d41f
    Signed-off-by: Furquan Shaikh <furquan at google.com>
---
 src/commonlib/include/commonlib/region.h | 11 +++++++++++
 src/commonlib/region.c                   | 23 +++++++++++++++++++++++
 2 files changed, 34 insertions(+)

diff --git a/src/commonlib/include/commonlib/region.h b/src/commonlib/include/commonlib/region.h
index 4e32b09..634132a 100644
--- a/src/commonlib/include/commonlib/region.h
+++ b/src/commonlib/include/commonlib/region.h
@@ -90,6 +90,11 @@ struct region_device {
 		},					\
 	}
 
+/* Helper to dynamically initialize region device. */
+void region_device_init(struct region_device *rdev,
+			const struct region_device_ops *ops, size_t offset,
+			size_t size);
+
 static inline size_t region_offset(const struct region *r)
 {
 	return r->offset;
@@ -187,4 +192,10 @@ extern const struct region_device_ops xlate_rdev_ops;
 		.rdev = REGION_DEV_INIT(&xlate_rdev_ops, 0,  (parent_sz_)),\
 	}
 
+/* Helper to dynamically initialize xlate region device. */
+void xlate_region_device_init(struct xlate_region_device *xdev,
+			      const struct region_device *access_dev,
+			      size_t sub_offset, size_t sub_size,
+			      size_t parent_size);
+
 #endif /* _REGION_H_ */
diff --git a/src/commonlib/region.c b/src/commonlib/region.c
index bfc5fc8..6f71074 100644
--- a/src/commonlib/region.c
+++ b/src/commonlib/region.c
@@ -133,6 +133,29 @@ void mem_region_device_init(struct mem_region_device *mdev, void *base,
 	mdev->rdev.region.size = size;
 }
 
+void region_device_init(struct region_device *rdev,
+			const struct region_device_ops *ops, size_t offset,
+			size_t size)
+{
+	memset(rdev, 0, sizeof(*rdev));
+	rdev->root = NULL;
+	rdev->ops = ops;
+	rdev->region.offset = offset;
+	rdev->region.size = size;
+}
+
+void xlate_region_device_init(struct xlate_region_device *xdev,
+			      const struct region_device *access_dev,
+			      size_t sub_offset, size_t sub_size,
+			      size_t parent_size)
+{
+	memset(xdev, 0, sizeof(*xdev));
+	xdev->access_dev = access_dev;
+	xdev->sub_region.offset = sub_offset;
+	xdev->sub_region.size = sub_size;
+	region_device_init(&xdev->rdev, &xlate_rdev_ops, 0, parent_size);
+}
+
 static void *mdev_mmap(const struct region_device *rd, size_t offset,
 			size_t size __unused)
 {