diff --git a/src/include/boot_device.h b/src/include/boot_device.h index 3964a9508c..cc70442e1e 100644 --- a/src/include/boot_device.h +++ b/src/include/boot_device.h @@ -18,9 +18,18 @@ #include +/* + * Please note that the read-only boot device may not be coherent with + * the read-write boot device. Thus, mixing mmap() and writeat() is + * most likely not to work so don't rely on such semantics. + */ + /* Return the region_device for the read-only boot device. */ const struct region_device *boot_device_ro(void); +/* Return the region_device for the read-write boot device. */ +const struct region_device *boot_device_rw(void); + /* * Create a sub-region of the read-only boot device. * Returns 0 on success, < 0 on error. @@ -28,6 +37,13 @@ const struct region_device *boot_device_ro(void); int boot_device_ro_subregion(const struct region *sub, struct region_device *subrd); +/* + * Create a sub-region of the read-write boot device. + * Returns 0 on success, < 0 on error. + */ +int boot_device_rw_subregion(const struct region *sub, + struct region_device *subrd); + /* * Initialize the boot device. This may be called multiple times within * a stage so boot device implementations should account for this behavior. diff --git a/src/lib/boot_device.c b/src/lib/boot_device.c index 42fe5dfd35..e7968f4fa9 100644 --- a/src/lib/boot_device.c +++ b/src/lib/boot_device.c @@ -20,18 +20,30 @@ void __attribute__((weak)) boot_device_init(void) /* Provide weak do-nothing init. */ } +static int boot_device_subregion(const struct region *sub, + struct region_device *subrd, + const struct region_device *parent) +{ + if (parent == NULL) + return -1; + + return rdev_chain(subrd, parent, region_offset(sub), region_sz(sub)); +} + int boot_device_ro_subregion(const struct region *sub, struct region_device *subrd) { - const struct region_device *boot_dev; - /* Ensure boot device has been initialized at least once. */ boot_device_init(); - boot_dev = boot_device_ro(); - - if (boot_dev == NULL) - return -1; - - return rdev_chain(subrd, boot_dev, region_offset(sub), region_sz(sub)); + return boot_device_subregion(sub, subrd, boot_device_ro()); +} + +int boot_device_rw_subregion(const struct region *sub, + struct region_device *subrd) +{ + /* Ensure boot device has been initialized at least once. */ + boot_device_init(); + + return boot_device_subregion(sub, subrd, boot_device_rw()); }