summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/soc/amd/picasso/Kconfig30
-rw-r--r--src/soc/amd/picasso/Makefile.inc1
-rw-r--r--src/soc/amd/picasso/graphics.c22
-rw-r--r--src/soc/amd/picasso/include/soc/cpu.h5
4 files changed, 56 insertions, 2 deletions
diff --git a/src/soc/amd/picasso/Kconfig b/src/soc/amd/picasso/Kconfig
index 27901ba600..e23cfd8f87 100644
--- a/src/soc/amd/picasso/Kconfig
+++ b/src/soc/amd/picasso/Kconfig
@@ -194,15 +194,41 @@ config VERSTAGE_ADDR
config VGA_BIOS_ID
string
- default "1002,15d8"
+ default "1002,15d8,c1"
help
The default VGA BIOS PCI vendor/device ID should be set to the
- result of the map_oprom_vendev() function in northbridge.c.
+ result of the map_oprom_vendev_rev() function in northbridge.c.
config VGA_BIOS_FILE
string
default "3rdparty/amd_blobs/picasso/PicassoGenericVbios.bin"
+config VGA_BIOS_SECOND
+ def_bool y
+
+config VGA_BIOS_SECOND_ID
+ string
+ default "1002,15dd,c4"
+ help
+ Because Dali and Picasso need different video BIOSes, but have the
+ same vendor/device IDs, we need an alternate method to determine the
+ correct video BIOS. In map_oprom_vendev_rev(), we look at the cpuid
+ and decide which rom to load.
+
+ Even though the hardware has the same vendor/device IDs, the vBIOS
+ contains a *different* device ID, confusing the situation even more.
+
+config VGA_BIOS_SECOND_FILE
+ string
+ default "3rdparty/amd_blobs/picasso/Raven2GenericVbios.bin"
+
+config CHECK_REV_IN_OPROM_NAME
+ bool
+ default y
+ help
+ Select this in the platform BIOS or chipset if the option rom has a
+ revision that needs to be checked when searching CBFS.
+
config S3_VGA_ROM_RUN
bool
default n
diff --git a/src/soc/amd/picasso/Makefile.inc b/src/soc/amd/picasso/Makefile.inc
index 7b80b21c78..03ce272d4d 100644
--- a/src/soc/amd/picasso/Makefile.inc
+++ b/src/soc/amd/picasso/Makefile.inc
@@ -69,6 +69,7 @@ ramstage-y += psp.c
ramstage-y += fsp_params.c
ramstage-y += config.c
ramstage-y += update_microcode.c
+ramstage-y += graphics.c
all-y += reset.c
diff --git a/src/soc/amd/picasso/graphics.c b/src/soc/amd/picasso/graphics.c
new file mode 100644
index 0000000000..047559cbf3
--- /dev/null
+++ b/src/soc/amd/picasso/graphics.c
@@ -0,0 +1,22 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <console/console.h>
+#include <device/pci_rom.h>
+#include <soc/cpu.h>
+#include <soc/soc_util.h>
+
+void map_oprom_vendev_rev(u32 *vendev, u8 *rev)
+{
+ if (*vendev != PICASSO_VBIOS_VID_DID)
+ return;
+
+ /* Check if the RV2 video bios needs to be used instead of the RV1/PCO one */
+ if (soc_is_raven2()) {
+ printk(BIOS_NOTICE, "Using RV2 VBIOS.\n");
+ *vendev = RAVEN2_VBIOS_VID_DID;
+ *rev = RAVEN2_VBIOS_REV;
+ } else {
+ printk(BIOS_NOTICE, "Using RV1/PCO VBIOS.\n");
+ *rev = PICASSO_VBIOS_REV;
+ }
+}
diff --git a/src/soc/amd/picasso/include/soc/cpu.h b/src/soc/amd/picasso/include/soc/cpu.h
index fd9c5fee48..e79dd87faf 100644
--- a/src/soc/amd/picasso/include/soc/cpu.h
+++ b/src/soc/amd/picasso/include/soc/cpu.h
@@ -12,4 +12,9 @@ void check_mca(void);
#define RAVEN2_A0_CPUID 0x00820f00
#define RAVEN2_A1_CPUID 0x00820f01
+#define PICASSO_VBIOS_VID_DID 0x100215d8
+#define PICASSO_VBIOS_REV 0xc1
+#define RAVEN2_VBIOS_VID_DID 0x100215dd /* VID/DID in RV2 VBIOS header */
+#define RAVEN2_VBIOS_REV 0xc4
+
#endif /* __PICASSO_CPU_H__ */