summaryrefslogtreecommitdiffstats
path: root/chipset_enable.c
diff options
context:
space:
mode:
authorStefan Tauner <stefan.tauner@alumni.tuwien.ac.at>2011-09-09 12:46:32 +0000
committerStefan Tauner <stefan.tauner@alumni.tuwien.ac.at>2011-09-09 12:46:32 +0000
commitec03fed4a1fdaf8cda3158539b9d7101e4e36675 (patch)
treee3982f2ce8401028028566c2f09ed38bc2e8338b /chipset_enable.c
parent7f07c256915aaf1dcc305a58a9c41e54b30127a6 (diff)
downloadflashrom-ec03fed4a1fdaf8cda3158539b9d7101e4e36675.zip
flashrom-ec03fed4a1fdaf8cda3158539b9d7101e4e36675.tar.gz
Revamp the warning of failing to set BIOS write enable in enable_flash_ich
- introduce a new variable 'wanted' that is used instead of 'new' - use 'new' for the actual value contained in BIOS_CNTL after we tried to write it - rephrase the warning which now also includes the old and new values besides the wanted one Corresponding to flashrom svn r1435. Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> Acked-by: Uwe Hermann <uwe@hermann-uwe.de>
Diffstat (limited to 'chipset_enable.c')
-rw-r--r--chipset_enable.c15
1 files changed, 8 insertions, 7 deletions
diff --git a/chipset_enable.c b/chipset_enable.c
index d013710..1c7eb31 100644
--- a/chipset_enable.c
+++ b/chipset_enable.c
@@ -256,7 +256,7 @@ static int enable_flash_piix4(struct pci_dev *dev, const char *name)
static int enable_flash_ich(struct pci_dev *dev, const char *name,
int bios_cntl)
{
- uint8_t old, new;
+ uint8_t old, new, wanted;
/*
* Note: the ICH0-ICH5 BIOS_CNTL register is actually 16 bit wide, but
@@ -280,15 +280,16 @@ static int enable_flash_ich(struct pci_dev *dev, const char *name,
if (old & (1 << 5))
msg_pinfo("WARNING: BIOS region SMM protection is enabled!\n");
- new = old | 1;
- if (new == old)
+ wanted = old | 1;
+ if (wanted == old)
return 0;
- rpci_write_byte(dev, bios_cntl, new);
+ rpci_write_byte(dev, bios_cntl, wanted);
- if (pci_read_byte(dev, bios_cntl) != new) {
- msg_pinfo("Setting register 0x%x to 0x%x on %s failed "
- "(WARNING ONLY).\n", bios_cntl, new, name);
+ if ((new = pci_read_byte(dev, bios_cntl)) != wanted) {
+ msg_pinfo("WARNING: Setting 0x%x from 0x%x to 0x%x on %s "
+ "failed. New value is 0x%x.\n",
+ bios_cntl, old, wanted, name, new);
return -1;
}
OpenPOWER on IntegriCloud