diff options
author | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2011-06-11 18:16:50 +0000 |
---|---|---|
committer | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2011-06-11 18:16:50 +0000 |
commit | f9a8da595d0961a7c5c899117d8e8dc7bdafaf42 (patch) | |
tree | 1f4f899d4669f36734fe0e78852a489f9ec748d0 | |
parent | c0207069488ea2c1d6bb39dc02ad5370f82f1067 (diff) | |
download | flashrom-f9a8da595d0961a7c5c899117d8e8dc7bdafaf42.tar.gz flashrom-f9a8da595d0961a7c5c899117d8e8dc7bdafaf42.tar.bz2 flashrom-f9a8da595d0961a7c5c899117d8e8dc7bdafaf42.zip |
enable_flash_ich: warn if SMM BIOS Write Protection is detected in BIOS_CNTL
Corresponding to flashrom svn r1332.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
-rw-r--r-- | chipset_enable.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/chipset_enable.c b/chipset_enable.c index b5385dc4..afb82aa2 100644 --- a/chipset_enable.c +++ b/chipset_enable.c @@ -264,8 +264,17 @@ static int enable_flash_ich(struct pci_dev *dev, const char *name, (old & (1 << 0)) ? "en" : "dis"); msg_pdbg("BIOS_CNTL is 0x%x\n", old); - new = old | 1; + /* + * Quote from the 6 Series datasheet (Document Number: 324645-004): + * "Bit 5: SMM BIOS Write Protect Disable (SMM_BWP) + * 1 = BIOS region SMM protection is enabled. + * The BIOS Region is not writable unless all processors are in SMM." + * In earlier chipsets this bit is reserved. */ + if (old & (1 << 5)) { + msg_pinfo("WARNING: BIOS region SMM protection is enabled!\n"); + } + new = old | 1; if (new == old) return 0; |