Attention is currently required from: Patrick Rudolph, Christian Walter.

Angel Pons uploaded patch set #3 to this change.

View Change

mb/prodrive/hermes: Prevent SGPIO cross-powering 5V rail

The PCH's SGPIO pads are connected to a buffer chip which is powered from
the always-on +3V3_AUX rail. For some reason, if the SGPIO pads remain in
their native function when a Poseidon system shuts down, a bit of voltage
from the +3V3_AUX-powered buffer chip will leak into the +5V rail through
the SATA backplane. Simply pulling the SGPIO pads low before powering off
the system stops the +5V rail from being cross-powered.

This issue has only been observed in S5, but it is very likely other sleep
states are affected as well. Thus, always pull the SGPIO pins low when the
power supply will be switched off, i.e. ACPI S3 or deeper.

TEST=Obtain a Poseidon system, verify that the +5V rail is cross-powered
after going to S5. We measured 0.17V on our system, but voltages as
high as 0.6V were measured on other systems. Verify that unplugging
the SGPIO cable going to the SATA backplane results in the +5V rail
voltage dropping to 0V, which indicates that the voltage leakage is
exclusively coming from the SGPIO and SATA backplane. Finally, make
sure that the +5V rail voltage drops to 0V after going into ACPI S5
with this patch applied and the SGPIO cable connected.

Change-Id: Ic872903d5fcdd1c17e02b4c06d5ba29889fbc27d
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
---
A src/mainboard/prodrive/hermes/smihandler.c
1 file changed, 51 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/16/66616/3

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic872903d5fcdd1c17e02b4c06d5ba29889fbc27d
Gerrit-Change-Number: 66616
Gerrit-PatchSet: 3
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Attention: Christian Walter <christian.walter@9elements.com>
Gerrit-MessageType: newpatchset