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

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak 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 12: Code-Review+1

(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);
if I understand this part right, wouldn't
```
if (!region_overlap(&real_dev.rdev, &cbfs_region))
     printk(error);
```
work to avoid the unnecessary mmap op ? I guess mmap for x86 boot media is a noop for now so it's not a big deal.