build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36590 )
Change subject: [UNTESTED]sb/intel/common: Deal with different save states ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36590/1/src/southbridge/intel/commo... File src/southbridge/intel/common/smihandler.c:
https://review.coreboot.org/c/coreboot/+/36590/1/src/southbridge/intel/commo... PS1, Line 265: ret = (u32*)save_state_reg(apm_node, SS_EAX); "(foo*)" should be "(foo *)"
https://review.coreboot.org/c/coreboot/+/36590/1/src/southbridge/intel/commo... PS1, Line 269: param = (u32*)save_state_reg(apm_node, SS_EBX); "(foo*)" should be "(foo *)"