Attention is currently required from: Tarun Tuli, Eran Mitrani, Nick Vaccaro.

Tarun Tuli uploaded patch set #17 to this change.

View Change

mb/google/hatch: Kohaku - ensure FPMCU is power cycled on reset

Leakage from the SPI CS line onto the FPMCU VDD rail was preventing
the FPMCU from fully shutting down on AP reset.

Instead of simply turning off the power rail, now ensure the CS
line is not driven high until late in coreboot.

This ensures it is completely off for the requisite minimum of 200ms
(now measured at approx 1100ms).

BUG=b:245953688
TEST=Confirmed FPMCU is still functional on Kohaku.
Confirmed FpRebootPowerCycle unit test now passes
BRANCH=Hatch

Signed-off-by: Tarun Tuli <tarnutuli@google.com>
Change-Id: I1e7e32f61c3ac1b3154d42821cc1dd4c5d3de303
---
M src/mainboard/google/hatch/variants/kohaku/gpio.c
M src/mainboard/google/hatch/variants/kohaku/ramstage.c
2 files changed, 50 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/68819/17

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1e7e32f61c3ac1b3154d42821cc1dd4c5d3de303
Gerrit-Change-Number: 68819
Gerrit-PatchSet: 17
Gerrit-Owner: Tarun Tuli <taruntuli@google.com>
Gerrit-Reviewer: Eran Mitrani <mitrani@google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro@google.com>
Gerrit-Reviewer: Shelley Chen <shchen@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Tarun Tuli <taruntuli@google.com>
Gerrit-Attention: Eran Mitrani <mitrani@google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro@google.com>
Gerrit-MessageType: newpatchset