[coreboot-gerrit] Change in coreboot[master]: cbfs: Move more stuff into cbfs_boot_lookup()

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39327 )

Change subject: cbfs: Move more stuff into cbfs_boot_lookup()
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39327/3/src/lib/cbfs.c 
File src/lib/cbfs.c:

https://review.coreboot.org/c/coreboot/+/39327/3/src/lib/cbfs.c@62 
PS3, Line 62: 	const struct region_device *bootdev = boot_device_ro();
> I'm using the assumption here that boot_device_ro() is always the root rdev for anything cbfs_get_bo […]

It does help some. I think that instead of calling boot_device_ro() here you can do the following as it's the exact semantic you desire.

bootdev = rdev_root(&fh->data);