Attention is currently required from: Jason Glenesk, Raul Rangel, Martin Roth, Marshall Dawson, Kangheui Won, Paul Menzel, Zheng Bao, Felix Held. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59308 )
Change subject: amdfwtool: Upgrade "relative address" to four address modes ......................................................................
Patch Set 5:
(6 comments)
File util/amdfwtool/amdfwtool.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133540): https://review.coreboot.org/c/coreboot/+/59308/comment/83117886_cd4f75dc PS5, Line 355: ((mode) == 2 ? (offset) - ctx.current_table : (offset)) )) space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133540): https://review.coreboot.org/c/coreboot/+/59308/comment/3f32e04d_721b4235 PS5, Line 357: #define RUN_TO_OFFSET(ctx, run) ((ctx).address_mode == 0 ? (run) - RUN_BASE(ctx) : (run)) /* TODO: */ line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133540): https://review.coreboot.org/c/coreboot/+/59308/comment/df083602_7ba18760 PS5, Line 478: bdir->header.additional_info_fields.dir_size = table_size / 0x1000;; Statements terminations use 1 semicolon
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133540): https://review.coreboot.org/c/coreboot/+/59308/comment/3c19e76c_bf043444 PS5, Line 955: biosdir->entries[count].address_mode = SET_ADDR_MODE(1, biosdir); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133540): https://review.coreboot.org/c/coreboot/+/59308/comment/934d24a7_c24174e4 PS5, Line 961: biosdir->entries[count].address_mode = SET_ADDR_MODE(1, biosdir); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133540): https://review.coreboot.org/c/coreboot/+/59308/comment/e4d77c7e_3c439d62 PS5, Line 974: biosdir->entries[count].address_mode = SET_ADDR_MODE(1, biosdir); line over 96 characters