summaryrefslogtreecommitdiff
path: root/tests/device/i2c-test.c
diff options
context:
space:
mode:
authorMichael Büchler <michael.buechler@posteo.net>2020-08-20 16:06:26 +0200
committerFelix Held <felix-coreboot@felixheld.de>2020-08-21 21:51:33 +0000
commit6c5f47b50ce8a65ccf3039e109af3740572c5345 (patch)
tree088765a2c0fde20ded8a8986ec346d2251402439 /tests/device/i2c-test.c
parent1594e8ff9c94f49605d356d8761dd818c3bc2905 (diff)
superio/ite: delay PWRGD3 during resume
According to the IT8728F datasheet it is possible to add an extra delay between 3VSBSW# being set and PWRGD3 being set during resume from Suspend-to-RAM. This is enabled in the special function selection register, the default being 0. This is also useful for the IT8720F although this chip does not have the PWRGD3 output. On the corresponding pin it has PWROK2, which the setting then seems to apply to. The datasheet for the IT8720F marks the corresponding bit as reserved, but the vendor BIOS of an Acer Aspire M3800 sets it anyway. Without setting the bit, coreboot fails to resume from S3. Oscilloscope measurements have shown that setting the bit increases the delay between 3VSBSW# being set and PWROK2 being set from around 1 us to 140 ms. The actual use of PWROK2 on the board design is unclear - the only destination it seems to reach is a pin header near the SuperIO marked as "GPIO1". Signed-off-by: Michael Büchler <michael.buechler@posteo.net> Change-Id: I51cbf2470dc2b840a647a20090acb5a0cf4f4025 Reviewed-on: https://review.coreboot.org/c/coreboot/+/44639 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Diffstat (limited to 'tests/device/i2c-test.c')
0 files changed, 0 insertions, 0 deletions