build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33673 )
Change subject: src: Move 'static' is at beginning of declaration ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/33673/1/src/northbridge/intel/x4x/dq_dqs.c File src/northbridge/intel/x4x/dq_dqs.c:
https://review.coreboot.org/#/c/33673/1/src/northbridge/intel/x4x/dq_dqs.c@5... PS1, Line 543: static const u8 emrs1_lut[8][4][4]={ /* [Config][Leveling Rank][Rank] */ spaces required around that '=' (ctx:VxV)
https://review.coreboot.org/#/c/33673/1/src/security/vboot/vboot_crtm.c File src/security/vboot/vboot_crtm.c:
https://review.coreboot.org/#/c/33673/1/src/security/vboot/vboot_crtm.c@32 PS1, Line 32: static const char *fmap_cbfs_names[] = { static const char * array should probably be static const char * const