Mathew King has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36045 )
Change subject: mb/g/drallion: Enable privacy screen on Drallion variant
......................................................................
Patch Set 8:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36045/7/src/mainboard/google/dralli...
File src/mainboard/google/drallion/variants/drallion/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/36045/7/src/mainboard/google/dralli...
PS7, Line 229: 0x0400
Another thing, I currently see kernel complains with this when booting up: […]
According to the ACPI spec A.3.1 _DOS should only exist if "the platform runtime firmware is capable of automatically switching the active display output". So I'm not really sure why the kernel considers this a firmware bug.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/36045
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I79d02bb1b25f0deb49ae4bb852b7ed8c21fd31c7
Gerrit-Change-Number: 36045
Gerrit-PatchSet: 8
Gerrit-Owner: Mathew King
mathewk@chromium.org
Gerrit-Reviewer: Mathew King
mathewk@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Bernardo Perez Priego
bernardo.perez.priego@intel.com
Gerrit-CC: Bora Guvendik
bora.guvendik@intel.com
Gerrit-CC: Rajat Jain
rajatja@google.com
Gerrit-CC: Selma Bensaid
selma.bensaid@intel.com
Gerrit-Comment-Date: Mon, 04 Nov 2019 23:56:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Rajat Jain
rajatja@google.com
Comment-In-Reply-To: Mathew King
mathewk@chromium.org
Gerrit-MessageType: comment