Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42079 )
Change subject: soc/intel/tigerlake: Add code for early tcss ......................................................................
Patch Set 20:
(4 comments)
https://review.coreboot.org/c/coreboot/+/42079/12/src/soc/intel/tigerlake/ea... File src/soc/intel/tigerlake/early_tcss.c:
https://review.coreboot.org/c/coreboot/+/42079/12/src/soc/intel/tigerlake/ea... PS12, Line 201: static void update_tcss_mux(int port, struct tcss_mux mux_data)
good point I will change this to a reference
Ack
https://review.coreboot.org/c/coreboot/+/42079/12/src/soc/intel/tigerlake/ea... PS12, Line 203: struct pmc_ipc_buffer *rbuf = malloc(sizeof(*rbuf));
I think there was a reason for this with the old method of creating the buffers but with the new one […]
Ack
https://review.coreboot.org/c/coreboot/+/42079/12/src/soc/intel/tigerlake/ea... PS12, Line 269: //Add check for connected maybe?
Will remove this left over comment
Ack
https://review.coreboot.org/c/coreboot/+/42079/12/src/soc/intel/tigerlake/fs... File src/soc/intel/tigerlake/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/42079/12/src/soc/intel/tigerlake/fs... PS12, Line 334: DSK
what is DSK?
Ack