Peichao Li has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31428 )
Change subject: mb/google/laser: Disable touch screen device that according to SKU ID
......................................................................
Patch Set 3:
Patch Set 3:
(2 comments)
Hi Jett,
Sorry, I don't think so, since as I know Laser and phaser is no different on the firmware section, SKU ID 1 point to phaser clamshell(100e), SKU ID 6 point to Laser14 clamshell. So I think this logic is correct.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31428
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6953c35a5e8c93d88fe63362156faa351e8ee71f
Gerrit-Change-Number: 31428
Gerrit-PatchSet: 3
Gerrit-Owner: Peichao Li
peichao.wang@bitland.corp-partner.google.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Hao He
hao.he@bitland.corp-partner.google.com
Gerrit-Reviewer: Jett Rink
jettrink@google.com
Gerrit-Reviewer: Justin TerAvest
teravest@chromium.org
Gerrit-Reviewer: Peichao Li
peichao.wang@bitland.corp-partner.google.com
Gerrit-Reviewer: Xingyu Wu
wuxy@bitland.corp-partner.google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Sat, 16 Feb 2019 03:34:44 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment