Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45411 )
Change subject: soc/intel/common/block: Fix Klocwork issue ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/45411/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45411/1//COMMIT_MSG@11 PS1, Line 11: 2. Initialize pointer soc_config with NULL
It should be easiest to make one commit per change, and add `Found-by:` tags.
I'd prefer one commit per change, too.
https://review.coreboot.org/c/coreboot/+/45411/1/src/soc/intel/common/block/... File src/soc/intel/common/block/chip/chip.c:
https://review.coreboot.org/c/coreboot/+/45411/1/src/soc/intel/common/block/... PS1, Line 8: const struct soc_intel_common_config *soc_config = NULL; : const config_t *config; : : config = config_of_soc(); : soc_config = &config->common_soc_config; : : return soc_config; This could just be `return &config_of_soc()->common_soc_config;`
https://review.coreboot.org/c/coreboot/+/45411/1/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/45411/1/src/soc/intel/common/block/... PS1, Line 174: return I'd just skip over the `vb2api_fail` and `vboot_save_data` calls. Otherwise, there won't be a reboot.