Fix AtomGL PR review feedback

Author: drlinux

dcs_lcd_color.h

@@ -43,20 +43,16 @@ static inline uint8_t rgba8888_get_alpha(uint32_t color)
 
 static inline uint16_t rgba8888_color_to_rgb565(uint32_t color)
 {
-    uint8_t r = (color >> 24) & 0xFF;
-    uint8_t g = (color >> 16) & 0xFF;
-    uint8_t b = (color >> 8) & 0xFF;
+    uint8_t r = color >> 24;
+    uint8_t g = color >> 16;
+    uint8_t b = color >> 8;
 
     return (((uint16_t) (r >> 3)) << 11) | (((uint16_t) (g >> 2)) << 5) | ((uint16_t) b >> 3);
 }
 
 static inline uint16_t display_color_to_rgb565(uint32_t color)
 {
-    uint8_t r = (color >> 24) & 0xFF;
-    uint8_t g = (color >> 16) & 0xFF;
-    uint8_t b = (color >> 8) & 0xFF;
-
-    return (((uint16_t) (r >> 3)) << 11) | (((uint16_t) (g >> 2)) << 5) | ((uint16_t) b >> 3);
+    return rgba8888_color_to_rgb565(color);
 }
 
 static inline uint16_t rgb565_color_to_surface(uint16_t color16)

display_items.c

@@ -35,9 +35,8 @@ struct Surface
     int width;
     int height;
     void *buffer;
-    uint32_t fg_color; // RGBA8888 little-endian byte order with the
-                       // alpha byte cleared; ORed with the per-pixel
-                       // alpha in epd_draw_pixel.
+    uint32_t fg_color; // RGB bytes in 0x00BBGGRR order; alpha is cleared
+                       // so epd_draw_pixel can append per-pixel alpha.
 };
 
 #define BPP 4

display_task.c

@@ -49,6 +49,8 @@ static bool try_pre_ack_render_cmd(Message *message, Context *ctx)
     if (cmd != globalcontext_make_atom(ctx->global, "\x6" "update")
             && cmd != globalcontext_make_atom(ctx->global, "\xD" "update_region")
             && cmd != globalcontext_make_atom(ctx->global, "\xF" "draw_rgb565_raw")
+            && cmd != globalcontext_make_atom(ctx->global, "\x16" "draw_rgb565_rle_base64")
+            && cmd != globalcontext_make_atom(ctx->global, "\x1D" "draw_rgb565_rle_base64_scaled")
             && cmd != globalcontext_make_atom(ctx->global,
                     "\xB" "draw_buffer")) {
         return false;

docs/display-drivers.md

@@ -386,8 +386,9 @@ supported drivers:
 ### update_region
 
 Partially updates a rectangular region of the display without redrawing the entire
-screen. Useful for incremental UI updates like progress bars or dynamic text fields
-where a full-screen redraw is unnecessary.
+screen. This is a compatibility workaround for partial refreshes until a richer
+display-list damage tracker is available. It is useful for incremental UI updates
+like progress bars or dynamic text fields where a full-screen redraw is unnecessary.
 
 ```elixir
 # Update only a 200×100 region at (50, 40)
@@ -398,7 +399,8 @@ where a full-screen redraw is unnecessary.
 
 Draws raw RGB565 binary pixel data directly to the display. Each pixel is 2 bytes
 in little-endian RGB565 format. The binary must contain exactly `width × height × 2`
-bytes.
+bytes. This is a low-level direct-buffer path for preformatted RGB565 data; the
+regular image tuple API remains the preferred route for encoded image assets.
 
 ```elixir
 # Draw a 100×100 pre-formatted RGB565 image at (10, 10)

rgb_lcd_display_driver.c

@@ -2,6 +2,7 @@
  * This file is part of AtomGL.
  *
  * Copyright 2026 AtomGL contributors
+ * Copyright 2026 Ibrahim YILMAZ <ibrahim@drlinux.org>
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -528,11 +529,12 @@ static void do_update_region(Context *ctx, int x0, int y0, int width, int height
                 memcpy(driver->framebuffers[work_fb], active_fb, fb_bytes);
             }
             if (render_items_to_framebuffer(driver, work_fb, x0, y0, width, height, items, len)) {
-            esp_err_t err = switch_to_framebuffer(driver, work_fb);
-            if (err != ESP_OK) {
-                ESP_LOGE(TAG, "region framebuffer switch failed: %s", esp_err_to_name(err));
-            } else {
-                mirror_region_from_active_to_inactive_framebuffers(driver, x0, y0, width, height);
+                esp_err_t err = switch_to_framebuffer(driver, work_fb);
+                if (err != ESP_OK) {
+                    ESP_LOGE(TAG, "region framebuffer switch failed: %s", esp_err_to_name(err));
+                } else {
+                    mirror_region_from_active_to_inactive_framebuffers(driver, x0, y0, width, height);
+                }
             }
         }
         display_items_delete(items, len);