5 comments:
File src/soc/intel/tigerlake/early_tcss.c:
nit: extra blank line
Ack
nit: extra blank line
Ack
Patch Set #12, Line 201: static void update_tcss_mux(int port, struct tcss_mux mux_data)
May be better to pass `mux_data` by reference, i.e. […]
good point I will change this to a reference
Patch Set #12, Line 203: struct pmc_ipc_buffer *rbuf = malloc(sizeof(*rbuf));
Does you really need malloc here? It doesn't appear that `rbuf` has a lifetime that is longer than t […]
I think there was a reason for this with the old method of creating the buffers but with the new one you are right will just change this to a normal declaration
Patch Set #12, Line 269: //Add check for connected maybe?
?
Will remove this left over comment
To view, visit change 42079. To unsubscribe, or for help writing mail filters, visit settings.