build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59005 )
Change subject: util/spd_tools: Document adding support for a new memory technology ......................................................................
Patch Set 1:
(4 comments)
File util/spd_tools/README.md:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132544): https://review.coreboot.org/c/coreboot/+/59005/comment/377e19ed_ef706101 PS1, Line 596: ### 1. Gather the SPD requirments 'requirments' may be misspelled - perhaps 'requirements'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132544): https://review.coreboot.org/c/coreboot/+/59005/comment/4ca155b0_231ee048 PS1, Line 605: * Platform-specifc requirments. SoC vendors often don't follow the JEDEC specs 'specifc' may be misspelled - perhaps 'specific'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132544): https://review.coreboot.org/c/coreboot/+/59005/comment/0b61f67d_48dfcfe4 PS1, Line 605: * Platform-specifc requirments. SoC vendors often don't follow the JEDEC specs 'requirments' may be misspelled - perhaps 'requirements'?
File util/spd_tools/src/spd_gen/spd_gen.go:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132544): https://review.coreboot.org/c/coreboot/+/59005/comment/72e43fd1_8f69eab9 PS1, Line 48: * defualt values. 'defualt' may be misspelled - perhaps 'default'?