[coreboot-gerrit] Change in ...coreboot[master]: libpayload: Don't try to use invalid row count

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/30092 )

Change subject: libpayload: Don't try to use invalid row count
......................................................................

libpayload: Don't try to use invalid row count

console->scroll_up() was hanging when console->rows is 0.  This
was happening on delan if no screen was attached.  If there are no
rows, just return.

BUG=b:119234919
TEST=Boot delan with no flat panel.  System boots to OS

Change-Id: Ib022d3c6fc0c9cf360809dca28761a50c787304a
Signed-off-by: Martin Roth 
Reviewed-on: https://review.coreboot.org/c/30092
Tested-by: build bot (Jenkins) 
Reviewed-by: Raul Rangel 
---
M payloads/libpayload/drivers/video/video.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Raul Rangel: Looks good to me, approved

diff --git a/payloads/libpayload/drivers/video/video.c b/payloads/libpayload/drivers/video/video.c
index 71140f0..b932a63 100644
--- a/payloads/libpayload/drivers/video/video.c
+++ b/payloads/libpayload/drivers/video/video.c
@@ -87,6 +87,9 @@
    	cursory++;
    }

+	if (console->rows <= 0)
+		return;
+
    while(cursory >= console->rows) {
    	console->scroll_up();
    	cursory--;