Attention is currently required from: Paul Menzel.
Justin Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79755?usp=email )
Change subject: mb/dell: Add Dell Precision M4800 (Haswell) ......................................................................
Patch Set 4:
(3 comments)
This change is ready for review.
Commit Message:
https://review.coreboot.org/c/coreboot/+/79755/comment/5bf6a19d_e6d64f53 : PS3, Line 9: workstation
Is it a laptop?
Correct, it is indeed a laptop. Maybe we could call it a mobile workstation?
https://review.coreboot.org/c/coreboot/+/79755/comment/9f15df36_60de4542 : PS3, Line 38: - Physical Wireless switch
What does that mean? Wifi does not work, but the system things it’s still enabled?
I should clarify that these two commits are necessary for wifi to work: https://review.coreboot.org/c/coreboot/+/79011/1 and https://review.coreboot.org/c/coreboot/+/77534/7.
Without them, I believe I observed that the kernel detects the wifi card but can't enable it.
Patchset:
PS3:
Welcome to coreboot, and thank you for this great contribution.
Thank you for the warm welcome.