[coreboot-gerrit] Change in coreboot[master]: commonlib/region: Allow multiple windows for xlate_region_dev

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47658 )

Change subject: commonlib/region: Allow multiple windows for xlate_region_dev
......................................................................

Patch Set 13:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47658/12/src/soc/intel/apollolake/m... 
File src/soc/intel/apollolake/mmap_boot.c:

https://review.coreboot.org/c/coreboot/+/47658/12/src/soc/intel/apollolake/m... 
PS12, Line 85: map = rdev_mmap(&real_dev.rdev, region_offset(&cbfs_region), region_sz(&cbfs_region));
             : 	if (map)
             : 		rdev_munmap(&real_dev.rdev, map);
> oops should have refreshed, furquan has the same idea. […]

Actually, I changed it back to `region_is_subregion` in patchset #13 to keep it similar to how it was handled before.