Changeset ca645a2 in mainline for kernel/genarch/src/fb/fb.c

Timestamp:

2019-01-02T10:41:50Z (5 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

7acd787

Parents:

e8d3165e

Message:

Slightly improve KFB scrolling speed

File:

• kernel/genarch/src/fb/fb.c (modified) (5 diffs)

kernel/genarch/src/fb/fb.c

@@ -76 +76 @@
             (instance)->cols + (col))
 
+#define BB_NEXT_COL(pos) (++(pos))
+
 #define GLYPH_POS(instance, glyph, y) \
         ((glyph) * (instance)->glyphbytes + (y) * (instance)->glyphscanline)
@@ -270 +272 @@
                                 unsigned int x;
                                 unsigned int col;
+                                size_t bb_pos = BB_POS(instance, 0, row);
+                                size_t bb_pos1 = BB_POS(instance, 0, row + 1);
 
                                 for (col = 0, x = 0; col < instance->cols;
@@ -276 +280 @@
 
                                         if (row < instance->rows - 1) {
-                                                if (instance->backbuf[BB_POS(instance, col, row)] ==
-                                                    instance->backbuf[BB_POS(instance, col, row + 1)])
-                                                        continue;
-
-                                                glyph = instance->backbuf[BB_POS(instance, col, row + 1)];
+                                                if (instance->backbuf[bb_pos] ==
+                                                    instance->backbuf[bb_pos1])
+                                                        goto skip;
+
+                                                glyph = instance->backbuf[bb_pos1];
                                         } else
                                                 glyph = 0;
@@ -287 +291 @@
                                             &instance->glyphs[GLYPH_POS(instance, glyph, yd)],
                                             instance->glyphscanline);
+                                skip:
+                                        BB_NEXT_COL(bb_pos);
+                                        BB_NEXT_COL(bb_pos1);
                                 }
                         }
@@ -373 +380 @@
                         unsigned int x;
                         unsigned int col;
+                        size_t bb_pos = BB_POS(instance, 0, row);
 
                         for (col = 0, x = 0; col < instance->cols;
                             col++, x += FONT_WIDTH) {
                                 uint16_t glyph =
-                                    instance->backbuf[BB_POS(instance, col, row)];
+                                    instance->backbuf[bb_pos];
                                 void *dst = &instance->addr[FB_POS(instance, x, y + yd)];
                                 void *src = &instance->glyphs[GLYPH_POS(instance, glyph, yd)];
                                 memcpy(dst, src, instance->glyphscanline);
+                                BB_NEXT_COL(bb_pos);
                         }
                 }