[coreboot-gerrit] New patch to review for coreboot: veyron_rialto: Use VOP_MODE_NONE for display init.

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Jul 3 16:35:53 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10784

-gerrit

commit b6feb15e5a23f6f08f06951f6973eaac071a721c
Author: David Hendricks <dhendrix at chromium.org>
Date:   Thu Jun 25 18:03:12 2015 -0700

    veyron_rialto: Use VOP_MODE_NONE for display init.
    
    This uses VOP_MODE_NONE for display init on veyron_rialto and
    adds a mainboard_power_on_backlight() stub so that we can finally
    get rid of SKIP_DISPLAY_INIT_HACK.
    
    BUG=none
    BRANCH=none
    TEST=built for veyron_rialto
    
    Change-Id: Ia6b420a962fe266e773c804b8e5c68da35848753
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: a290c938c163759a3672c07d8ec7c0a38057b13d
    Original-Change-Id: Iec2d7f03857198a4d6f7490db1e3e19c74f18c43
    Original-Signed-off-by: David Hendricks <dhendrix at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/282048
---
 src/mainboard/google/veyron_rialto/Kconfig       | 8 --------
 src/mainboard/google/veyron_rialto/devicetree.cb | 1 +
 src/mainboard/google/veyron_rialto/mainboard.c   | 1 +
 3 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/mainboard/google/veyron_rialto/Kconfig b/src/mainboard/google/veyron_rialto/Kconfig
index 30c54f6..1886db9 100644
--- a/src/mainboard/google/veyron_rialto/Kconfig
+++ b/src/mainboard/google/veyron_rialto/Kconfig
@@ -63,14 +63,6 @@ config CONSOLE_SERIAL_UART_ADDRESS
 	depends on DRIVERS_UART
 	default 0xFF690000
 
-# FIXME(dhendrix): This is a gross hack intended to get us past
-# display init which currently hangs the machine. It will be removed
-# once we've re-factored the display init code to properly handle
-# various types of displays.
-config SKIP_DISPLAY_INIT_HACK
-	int
-	default 1
-
 config PMIC_BUS
 	int
 	default 0
diff --git a/src/mainboard/google/veyron_rialto/devicetree.cb b/src/mainboard/google/veyron_rialto/devicetree.cb
index 4c2ea8f..c40bbbb 100644
--- a/src/mainboard/google/veyron_rialto/devicetree.cb
+++ b/src/mainboard/google/veyron_rialto/devicetree.cb
@@ -21,5 +21,6 @@
 chip soc/rockchip/rk3288
 	device cpu_cluster 0 on end
 	register "vop_id" = "1"
+	register "vop_mode" = "VOP_MODE_NONE"
 	register "framebuffer_bits_per_pixel" = "16"
 end
diff --git a/src/mainboard/google/veyron_rialto/mainboard.c b/src/mainboard/google/veyron_rialto/mainboard.c
index 6e3e51b..52a27bb 100644
--- a/src/mainboard/google/veyron_rialto/mainboard.c
+++ b/src/mainboard/google/veyron_rialto/mainboard.c
@@ -108,4 +108,5 @@ void lb_board(struct lb_header *header)
 
 void mainboard_power_on_backlight(void)
 {
+	return;
 }



More information about the coreboot-gerrit mailing list