[coreboot-gerrit] Change in coreboot[master]: vx900: decode the whole ROM image

Lubomir Rintel (Code Review) gerrit at coreboot.org
Tue Oct 31 15:25:31 CET 2017


Lubomir Rintel has uploaded this change for review. ( https://review.coreboot.org/22262


Change subject: vx900: decode the whole ROM image
......................................................................

vx900: decode the whole ROM image

Fixes supports for flash ROMs larger than 512K, such as the 1M one in
HP t5550 Thin Client.

Change-Id: I4d6287e130809c33dfbd40bce7913a95b4b3a9c7
Signed-off-by: Lubomir Rintel <lkundrak at v3.sk>
---
M src/northbridge/via/vx900/Kconfig
A src/northbridge/via/vx900/bootblock.c
2 files changed, 50 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/22262/1

diff --git a/src/northbridge/via/vx900/Kconfig b/src/northbridge/via/vx900/Kconfig
index 4b1e6cc..bb5e06f 100644
--- a/src/northbridge/via/vx900/Kconfig
+++ b/src/northbridge/via/vx900/Kconfig
@@ -41,4 +41,8 @@
 	string
 	default "1106,7122"
 
+config BOOTBLOCK_NORTHBRIDGE_INIT
+	string
+	default "northbridge/via/vx900/bootblock.c"
+
 endif
diff --git a/src/northbridge/via/vx900/bootblock.c b/src/northbridge/via/vx900/bootblock.c
new file mode 100644
index 0000000..11123b6
--- /dev/null
+++ b/src/northbridge/via/vx900/bootblock.c
@@ -0,0 +1,46 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2017 Lubomir Rintel <lkundrak at v3.sk>
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include <arch/io.h>
+
+#if CONFIG_ROM_SIZE == 0x80000
+#  define ROM_DECODE_MAP 0x00
+#elif CONFIG_ROM_SIZE == 0x100000
+#  define ROM_DECODE_MAP 0x40
+#elif CONFIG_ROM_SIZE == 0x180000
+#  define ROM_DECODE_MAP 0x60
+#elif CONFIG_ROM_SIZE == 0x200000
+#  define ROM_DECODE_MAP 0x70
+#elif CONFIG_ROM_SIZE == 0x280000
+#  define ROM_DECODE_MAP 0x78
+#elif CONFIG_ROM_SIZE == 0x300000
+#  define ROM_DECODE_MAP 0x7c
+#elif CONFIG_ROM_SIZE == 0x380000
+#  define ROM_DECODE_MAP 0x7e
+#elif CONFIG_ROM_SIZE == 0x400000
+#  define ROM_DECODE_MAP 0x7f
+#else
+#  error "Bad CONFIG_ROM_SIZE"
+#endif
+
+static void bootblock_northbridge_init(void)
+{
+	u8 reg;
+
+	pci_io_read_config8(PCI_DEV(0, 0x11, 0), 0x41);
+	reg |= ROM_DECODE_MAP;
+	pci_io_write_config8(PCI_DEV(0, 0x11, 0), 0x41, reg);
+}

-- 
To view, visit https://review.coreboot.org/22262
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4d6287e130809c33dfbd40bce7913a95b4b3a9c7
Gerrit-Change-Number: 22262
Gerrit-PatchSet: 1
Gerrit-Owner: Lubomir Rintel <lkundrak at v3.sk>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171031/13861f78/attachment.html>


More information about the coreboot-gerrit mailing list