[M] Change in coreboot[master]: soc/intel/cse_lite: Back up PSR data during CSE FW downgrade

[Anil Kumar K (Code Review)]

Attention is currently required from: Subrata Banik, Jérémy Compostella, Paul Menzel, Sridhar Siricilla.

Anil Kumar K has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74577 )

Change subject: soc/intel/cse_lite: Back up PSR data during CSE FW downgrade
......................................................................

Patch Set 32:

(3 comments)

File src/soc/intel/common/block/cse/cse_lite.c:

https://review.coreboot.org/c/coreboot/+/74577/comment/d6eadd78_c80d6948 
PS27, Line 145: struct psr_heci_fw_downgrade_backup_req {
              : 		struct psr_heci_header header;
              : 	} __packed;
> Why do you need to define this ?

this is to keep the code in sync with the HECI API definition for better readability of the code

https://review.coreboot.org/c/coreboot/+/74577/comment/3781f36b_4058132c 
PS27, Line 153: sizeof(struct psr_heci_fw_downgrade_backup_res)
> is there any issue using sizeof with structure ?

Done

File src/soc/intel/common/block/cse/cse_lite.c:

https://review.coreboot.org/c/coreboot/+/74577/comment/68bb0819_66fccc50 
PS28, Line 142: static enum cb_err cse_send_backup_psr_cmd(struct  psr_heci_fw_downgrade_backup_res *res)
> at the moment this function is only called during downgrade. […]

Done