summaryrefslogtreecommitdiff
path: root/src/superio/fintek
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2021-01-21 14:40:59 -0800
committerPatrick Georgi <pgeorgi@google.com>2021-01-25 08:55:44 +0000
commitac80610b510e5e1d270b711bbc9632e3ab30f5de (patch)
treee3b0d9732968a8c37a0a6af5c3977cc226b466da /src/superio/fintek
parentce66cd3e496b3df5775822942fa1540f3f8f9b41 (diff)
trogdor: Explicitly initialize display pins in bootblock
This patch adds explicit initializations for the remaining named display (power) control GPIOs to the bootblock GPIO init code. These pins are usually mapped to pins that are already configured to pull-downs on power-on reset so this wasn't really required, but we have already moved them around so often that you never know when EEs might one day move them to a pin with a different power-on reset configuration, so it's better to be explicit. In one particular case, GPIO(67) (used by CoachZ rev1+ but not by anything else for the EN_PP3300_DX_EDP pin) is not actually a pull-down on boot, even though that is claimed by the datasheet. This is likely due to the fact that it can serve as the SPI_HOLD pin for the boot flash QSPI bus, so even though our board's boot flash doesn't really use that pin, it seems that the boot ROM still configures it as such. Signed-off-by: Julius Werner <jwerner@chromium.org> Change-Id: I533baa962d2dfc87cfa510f442ed2e8912e0e5b0 Reviewed-on: https://review.coreboot.org/c/coreboot/+/49810 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: mturney mturney <mturney@codeaurora.org> Reviewed-by: Douglas Anderson <dianders@chromium.org>
Diffstat (limited to 'src/superio/fintek')
0 files changed, 0 insertions, 0 deletions