[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/braswell/acpi/lpc.asl: Allocate used ROM size only

[Frans Hendriks (Code Review)]

Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31822 )

Change subject: soc/intel/braswell/acpi/lpc.asl: Allocate used ROM size only
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31822/2/src/soc/intel/braswell/acpi/lpc.asl 
File src/soc/intel/braswell/acpi/lpc.asl:

https://review.coreboot.org/#/c/31822/2/src/soc/intel/braswell/acpi/lpc.asl@... 
PS2, Line 48: 				      ((CONFIG_COREBOOT_ROMSIZE_KB*1024) + 1),
> was this really tested? the parentheses look wrong

Yes, tested and working.
Please not that this is part of second argement which is 
0xffffffff - ((CONFIG_COREBOOT_ROMSIZE_KB*1024) + 1). This contains twice '(' and twice ')'