diff options
| author | Zheng Bao <zheng.bao@amd.com> | 2010-11-05 07:59:06 +0000 |
|---|---|---|
| committer | Zheng Bao <Zheng.Bao@amd.com> | 2010-11-05 07:59:06 +0000 |
| commit | 27770c93d49cfd5f5cdec7094259ae600ca4bcca (patch) | |
| tree | 008f642a7bded1dd1460b7fabc9edd4a8f662e10 /src/superio | |
| parent | 0e40e6ba4d087ec89dc6029475523ef033896822 (diff) | |
| download | coreboot-27770c93d49cfd5f5cdec7094259ae600ca4bcca.tar.xz | |
According to the description in datasheet of f71889,
"To enable configuration, the entry key 0x87 must be written to
the index port"
"
-o 4e 87
-o 4e 87 (enable configuration)
-o 4e aa (disable configuration)
"
This piece of text appears in most of the datasheet of fintek superio.
It doesnt say it quite clear, but it seems that the 0x87 should
be written twice. I tried on f81865, which is not in the coreboot tree
yet. If the 0x87 is only written once, you can only R/W the index/data
port once. All the subsequent RW will fail. Writing twice will be ok.
Plus, in the superiotool, the function enter_conf_mode_winbond_fintek_ite_8787
also write 8787.
The fintek superio chips seem to enable the UART automatically when the
power is on. So I didnt find it failed to access.
Signed-off-by: Zheng Bao <zheng.bao@amd.com>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6025 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'src/superio')
4 files changed, 4 insertions, 0 deletions
diff --git a/src/superio/fintek/f71805f/f71805f_early_serial.c b/src/superio/fintek/f71805f/f71805f_early_serial.c index de30cbb48b..dd82e8352b 100644 --- a/src/superio/fintek/f71805f/f71805f_early_serial.c +++ b/src/superio/fintek/f71805f/f71805f_early_serial.c @@ -27,6 +27,7 @@ static void pnp_enter_conf_state(device_t dev) { u16 port = dev >> 8; outb(0x87, port); + outb(0x87, port); } static void pnp_exit_conf_state(device_t dev) diff --git a/src/superio/fintek/f71859/f71859_early_serial.c b/src/superio/fintek/f71859/f71859_early_serial.c index c27b17937e..14d22a6310 100755 --- a/src/superio/fintek/f71859/f71859_early_serial.c +++ b/src/superio/fintek/f71859/f71859_early_serial.c @@ -27,6 +27,7 @@ static void pnp_enter_conf_state(device_t dev) { u16 port = dev >> 8; outb(0x87, port); + outb(0x87, port); } static void pnp_exit_conf_state(device_t dev) diff --git a/src/superio/fintek/f71863fg/f71863fg_early_serial.c b/src/superio/fintek/f71863fg/f71863fg_early_serial.c index 7eeb06aef0..d410ef6031 100644 --- a/src/superio/fintek/f71863fg/f71863fg_early_serial.c +++ b/src/superio/fintek/f71863fg/f71863fg_early_serial.c @@ -27,6 +27,7 @@ static void pnp_enter_conf_state(device_t dev) { u16 port = dev >> 8; outb(0x87, port); + outb(0x87, port); } static void pnp_exit_conf_state(device_t dev) diff --git a/src/superio/fintek/f71889/f71889_early_serial.c b/src/superio/fintek/f71889/f71889_early_serial.c index 3435bf55f7..29042ae817 100644 --- a/src/superio/fintek/f71889/f71889_early_serial.c +++ b/src/superio/fintek/f71889/f71889_early_serial.c @@ -26,6 +26,7 @@ static void pnp_enter_conf_state(device_t dev) { u16 port = dev >> 8; outb(0x87, port); + outb(0x87, port); } static void pnp_exit_conf_state(device_t dev) |