Attention is currently required from: Shobhit Srivastava, Subrata Banik, Kane Chen, Tim Wawrzynczak, Sridhar Siricilla. Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62861 )
Change subject: soc/intel/adl: Remove IOM Mctp command from TCSS ASL ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/62861/comment/be2c79fd_3db04a8b PS2, Line 14: Due to change in hardware sequence
maybe mention like […]
Done
https://review.coreboot.org/c/coreboot/+/62861/comment/188391bf_23b99d04 PS2, Line 26: MAULIK V VAGHELA
is your name in git config still set to all caps? 😄
Sorry, this was pushed from my another machine. Corrected now across all machines :)
File src/soc/intel/alderlake/acpi/tcss.asl:
https://review.coreboot.org/c/coreboot/+/62861/comment/646fb26f_82b98c01 PS2, Line 694: /* : * If the TCSS Deven is cleared by BIOS Mailbox request, then : * restore to previously saved value of TCSS DEVNE. : */ : Local0 = 0 : While (_SB.PCI0.TXHC.VDID == 0xFFFFFFFF) { : If (DSGS () == 1) { : DSCR (0) : } : Local0++ : If (Local0 == 5) { : Printf("pCode mailbox command failed.") : Break : } : }
Is this part still necessary too? I don't see the reference code doing this.
Ack
https://review.coreboot.org/c/coreboot/+/62861/comment/9fcb7704_b545594f PS2, Line 725: /* : * If the TCSS Deven in normal state, then Save current TCSS DEVEN value and : * clear it. : */ : Local0 = 0 : While (_SB.PCI0.TXHC.VDID != 0xFFFFFFFF) { : If (DSGS () == 0) { : DSCR (1) : } : Local0++ : If (Local0 == 5) { : Printf("pCode mailbox command failed.") : Break : } : }
Which means DSGS and DSCR are also unused now too I think
Ack. Agreed Tim..I have removed Mailbox communication part