From cfeeb7889d5346e2abaf9b198375df62c58b098f Mon Sep 17 00:00:00 2001
From: William Wilgus <wilgus.william@gmail.com>
Date: Mon, 21 Mar 2022 18:54:52 -0400
Subject: Lcd save function pointer to frame buffer get_address_fn before loops

Calling multiple levels of indirection in a loop slows things down

Really these need to be rewritten to take a start and end address
like most of the rest of the codebase

But this is safer without having test hardware in hand

Change-Id: Idae7b92ee779d020ed7fcc9334e2d5a9c710e64d
---
 firmware/target/arm/rk27xx/lcdif-rk27xx.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'firmware/target/arm/rk27xx')

diff --git a/firmware/target/arm/rk27xx/lcdif-rk27xx.c b/firmware/target/arm/rk27xx/lcdif-rk27xx.c
index e6af0d978a..618b476480 100644
--- a/firmware/target/arm/rk27xx/lcdif-rk27xx.c
+++ b/firmware/target/arm/rk27xx/lcdif-rk27xx.c
@@ -198,9 +198,10 @@ static void create_llp(int x, int y, int width, int height)
 
     width = width>>1;
 
+    void* (*fbaddr)(int x, int y) = FB_CURRENTVP_BUFFER->get_address_fn;
     /* build LLPs */
     for (i=0; i<height; i++)
-        llp_setup((void *)FBADDR(x,y+i),
+        llp_setup((void *)fbaddr(x,y+i),
                   (void*)(LCD_BUFF+((i%4)*4*width)),
                   &(scr_llp[i]),
                   width);
-- 
cgit v1.2.3