[coreboot-gerrit] Change in coreboot[master]: driver/intel/pmc_mux/conn: Add type-c port info to cbmem

[Furquan Shaikh (Code Review)]

Attention is currently required from: Nico Huber, Tim Wawrzynczak, Nick Vaccaro, Patrick Rudolph.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57345 )

Change subject: driver/intel/pmc_mux/conn: Add type-c port info to cbmem
......................................................................

Patch Set 18:

(1 comment)

File src/drivers/intel/pmc_mux/conn/conn.c:

https://review.coreboot.org/c/coreboot/+/57345/comment/b1f9fcea_5bfd9104 
PS18, Line 46: 		printk(BIOS_ERR, "ERROR: No space for Type-C port info!\n");
> I always wonder what happens in CBMEM on a resume, would this trigger?

That is an excellent point. This will report error in case of S3 resume because CBMEM entry is already allocated and initialized. We will have to add a check `acpi_is_wakeup_s3()` and return early from this function if the system is on S3 resume path.