Patch Set 1:

Patch Set 1:

Please use runtime detection and ssdt code to achieve the same functionality.

The standard Intel FSP disables both SPI1 and PWM. All system using this FSP will not have the SPI1 and PWM enabled. Adding runtime detection will take boot time, where the result will be constant on these system.

I dont know if Google Cyan use standard FSP binary. If so I suggest removing the SPI1 and PWM ASL code.
For Google Cyan the SP1 and PWM ASL code can be added to mainboard directory when required.

I suggest to move this code to Google Cyan?

Since I am not familiar with peripheral usage on Chromebooks, I am adding Matt on CC.

Matt, could You please share Your opinion and possibly review the patch?

View Change

To view, visit change 29417. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iec2ca7520081d00bf7a53d58ee054aa6f23e5606
Gerrit-Change-Number: 29417
Gerrit-PatchSet: 3
Gerrit-Owner: Frans Hendriks <fhendriks@eltan.com>
Gerrit-Reviewer: Frans Hendriks <fhendriks@eltan.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski@3mdeb.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Piotr Król <piotr.krol@3mdeb.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Matt DeVillier <matt.devillier@gmail.com>
Gerrit-Comment-Date: Thu, 04 Apr 2019 08:53:42 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment