[coreboot-gerrit] Change in coreboot[master]: soc/intel/denverton_ns: Hook up SMMSTORE

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47450 )

Change subject: soc/intel/denverton_ns: Hook up SMMSTORE
......................................................................

Patch Set 3: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/47450/3/src/soc/intel/denverton_ns/... 
File src/soc/intel/denverton_ns/smihandler.c:

https://review.coreboot.org/c/coreboot/+/47450/3/src/soc/intel/denverton_ns/... 
PS3, Line 201: static void southbridge_smi_store(void)
             : {
             : 	u8 sub_command, ret;
             : 	em64t100_smm_state_save_area_t *io_smi =
             : 		smi_apmc_find_state_save(APM_CNT_SMMSTORE);
             : 	uint32_t reg_ebx;
             : 
             : 	if (!io_smi)
             : 		return;
             : 	/* Command and return value in EAX */
             : 	sub_command = (io_smi->rax >> 8) & 0xff;
             : 
             : 	/* Parameter buffer in EBX */
             : 	reg_ebx = io_smi->rbx;
             : 
             : 	/* drivers/smmstore/smi.c */
             : 	ret = smmstore_exec(sub_command, (void *)reg_ebx);
             : 	io_smi->rax = ret;
             : }
Is this chip specific?