[coreboot-gerrit] Patch set updated for coreboot: 1f60c73 regions: add memory region device support

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Tue Apr 21 03:59:24 CEST 2015


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9131

-gerrit

commit 1f60c73ea1bb2b9041520b3debf1781f3e6b8441
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Fri Mar 27 01:03:45 2015 -0500

    regions: add memory region device support
    
    Provide common code for using memory-backed region devices.
    This allows in-memory buffers to act as a region device.
    
    Change-Id: I266cd07bbfa16a427c2b31c512e7c87b77f47718
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/include/region.h | 19 +++++++++++++++++++
 src/lib/region.c     | 42 ++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 61 insertions(+)

diff --git a/src/include/region.h b/src/include/region.h
index cb91102..ed0ad57 100644
--- a/src/include/region.h
+++ b/src/include/region.h
@@ -97,4 +97,23 @@ static inline size_t region_sz(const struct region *r)
 	return r->size;
 }
 
+struct mem_region_device {
+	char *base;
+	struct region_device rdev;
+};
+
+/* Iniitalize at runtime a mem_region_device. This would be used when
+ * the base and size are dynamic or can't be known during linking. */
+void mem_region_device_init(struct mem_region_device *mdev, void *base,
+				size_t size);
+
+extern const struct region_device_ops mem_rdev_ops;
+
+/* Statically initialize mem_region_device. */
+#define MEM_REGION_DEV_INIT(base_, size_)				\
+	{								\
+		.base = (void *)(base_),				\
+		.rdev = REGION_DEV_INIT(&mem_rdev_ops, 0, (size_)),	\
+	}
+
 #endif /* _REGION_H_ */
diff --git a/src/lib/region.c b/src/lib/region.c
index 948addd..ca8cd7b 100644
--- a/src/lib/region.c
+++ b/src/lib/region.c
@@ -119,3 +119,45 @@ int rdev_chain(struct region_device *child, const struct region_device *parent,
 
 	return 0;
 }
+
+void mem_region_device_init(struct mem_region_device *mdev, void *base,
+				size_t size)
+{
+	memset(mdev, 0, sizeof(*mdev));
+	mdev->base = base;
+	mdev->rdev.ops = &mem_rdev_ops;
+	mdev->rdev.region.size = size;
+}
+
+static void *mdev_mmap(const struct region_device *rd, size_t offset,
+			size_t size)
+{
+	const struct mem_region_device *mdev;
+
+	mdev = container_of(rd, typeof(*mdev), rdev);
+
+	return &mdev->base[offset];
+}
+
+static int mdev_munmap(const struct region_device *rd, void *mapping)
+{
+	return 0;
+}
+
+static ssize_t mdev_readat(const struct region_device *rd, void *b,
+				size_t offset, size_t size)
+{
+	const struct mem_region_device *mdev;
+
+	mdev = container_of(rd, typeof(*mdev), rdev);
+
+	memcpy(b, &mdev->base[offset], size);
+
+	return size;
+}
+
+const struct region_device_ops mem_rdev_ops = {
+	.mmap = mdev_mmap,
+	.munmap = mdev_munmap,
+	.readat = mdev_readat,
+};



More information about the coreboot-gerrit mailing list