Attention is currently required from: Jason Glenesk, Matt DeVillier, Felix Held.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69071 )
Change subject: soc/amd/morgana/data_fabric: Add register bitslice struct ......................................................................
Patch Set 1:
(5 comments)
File src/soc/amd/morgana/include/soc/data_fabric.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161871): https://review.coreboot.org/c/coreboot/+/69071/comment/6e93535c_a48e0bda PS1, Line 21: uint32_t : 1; /* [ 2.. 2] */ space prohibited before that ':' (ctx:WxW)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161871): https://review.coreboot.org/c/coreboot/+/69071/comment/5ec8c752_e71eef9d PS1, Line 23: uint32_t : 12; /* [15.. 4] */ space prohibited before that ':' (ctx:WxW)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161871): https://review.coreboot.org/c/coreboot/+/69071/comment/87d6002f_5a9899f1 PS1, Line 25: uint32_t : 10; /* [31..22] */ space prohibited before that ':' (ctx:WxW)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161871): https://review.coreboot.org/c/coreboot/+/69071/comment/dba9c59b_fec68d4a PS1, Line 36: uint32_t : 1; /* [15..15] */ space prohibited before that ':' (ctx:WxW)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161871): https://review.coreboot.org/c/coreboot/+/69071/comment/c235aaf5_c7bab795 PS1, Line 38: uint32_t : 8; /* [31..24] */ space prohibited before that ':' (ctx:WxW)