Attention is currently required from: David Wu, Tim Wawrzynczak, Zhuohao Lee. Alan Huang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59086 )
Change subject: drivers/net/r8168: Add support for Realtek RT8125 ......................................................................
Patch Set 3:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/59086/comment/0e8a9c1b_7e66b4fb PS2, Line 7: mb/google/brya/var/brask: Make LAN driver to support RT8125
suggestion: […]
Ack
https://review.coreboot.org/c/coreboot/+/59086/comment/f3ab000d_8467e162 PS2, Line 8: : Add new device id 0x8125 to the driver.
suggestion: […]
Ack
File src/drivers/net/r8168.c:
https://review.coreboot.org/c/coreboot/+/59086/comment/3ca17c81_673ef669 PS2, Line 364: 0x8168, : 0x8125,
Now that there is more than one, should we add these to https://review.coreboot. […]
Ack
https://review.coreboot.org/c/coreboot/+/59086/comment/ed916890_99d6ce3e PS2, Line 371: 0x10ec
While you're here, do you mind changing this to `PCI_VENDOR_ID_REALTEK` ?
Ack