build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31701 )
Change subject: arch/x86/acpi: Remove obsolete acpi_gen_regaddr resv field ......................................................................
Patch Set 6:
(6 comments)
https://review.coreboot.org/#/c/31701/6/src/mainboard/lenovo/x201/mainboard.... File src/mainboard/lenovo/x201/mainboard.c:
https://review.coreboot.org/#/c/31701/6/src/mainboard/lenovo/x201/mainboard.... PS6, Line 35: {1, 1, 1000, {0x7f, 1, 2, 0, 1, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31701/6/src/mainboard/lenovo/x201/mainboard.... PS6, Line 36: {2, 1, 500, {0x01, 8, 0, 0, DEFAULT_PMBASE + LV2, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31701/6/src/mainboard/lenovo/x201/mainboard.... PS6, Line 37: {2, 17, 250, {0x01, 8, 0, 0, DEFAULT_PMBASE + LV3, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31701/6/src/mainboard/packardbell/ms2290/mai... File src/mainboard/packardbell/ms2290/mainboard.c:
https://review.coreboot.org/#/c/31701/6/src/mainboard/packardbell/ms2290/mai... PS6, Line 37: {1, 1, 1000, {0x7f, 1, 2, 0, 1, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31701/6/src/mainboard/packardbell/ms2290/mai... PS6, Line 38: {2, 1, 500, {0x01, 8, 0, 0, DEFAULT_PMBASE + LV2, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31701/6/src/mainboard/packardbell/ms2290/mai... PS6, Line 39: {2, 17, 250, {0x01, 8, 0, 0, DEFAULT_PMBASE + LV3, 0}}, space required after that close brace '}'