From: Johannes Schneider <johannes.schneider@leica-geosystems.com>
To: barebox@lists.infradead.org
Cc: thomas.haemmerle@leica-geosystems.com
Subject: [PATCH 6/6] video: imx-lcdif: drain write-combine buffer before LCDIF DMA reads pixels
Date: Tue, 2 Jun 2026 04:09:52 +0000 [thread overview]
Message-ID: <20260602040953.1060278-7-johannes.schneider@leica-geosystems.com> (raw)
In-Reply-To: <20260602040953.1060278-1-johannes.schneider@leica-geosystems.com>
From: Thomas Haemmerle <thomas.haemmerle@leica-geosystems.com>
The LCDIF framebuffer is allocated as Normal Non-Cacheable (write-combine)
memory. After the splash command calls gu_screen_blit(), which copies the
decoded PNG via memcpy() from a cached shadow buffer into the hardware
framebuffer, writes may still reside in the CPU write buffer and not yet
be visible to the LCDIF DMA engine. On hardware this manifests as
partial or corrupted rendering — the bottom portion of the splash image
renders black or shows garbage.
Implement fb_damage() to execute a DSB SY barrier after each blit,
ensuring all pending write-combine writes are committed to DRAM before
the LCDIF reads the pixel data.
Assisted-by: Claude:claude-sonnet-4-6
Signed-of-by: Thomas Haemmerle <thomas.haemmerle@leica-geosystems.com>
---
drivers/video/imx-lcdif.c | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/drivers/video/imx-lcdif.c b/drivers/video/imx-lcdif.c
index ae5976c771..8d9b40be0f 100644
--- a/drivers/video/imx-lcdif.c
+++ b/drivers/video/imx-lcdif.c
@@ -20,6 +20,7 @@
#include <of_graph.h>
#include <video/media-bus-format.h>
#include <video/vpl.h>
+#include <asm/system.h>
/* LCDIF V8 register map */
#define LCDC_V8_CTRL 0x00
@@ -262,9 +263,21 @@ static void lcdif_fb_disable(struct fb_info *info)
clk_disable_unprepare(priv->clk_axi);
}
+static void lcdif_fb_damage(struct fb_info *info, struct fb_rect *rect)
+{
+ /*
+ * The hardware framebuffer is Normal Non-Cacheable (write-combine).
+ * Writes from the shadow-buffer memcpy may sit in the CPU write buffer
+ * and not yet be visible to the LCDIF DMA engine. Execute DSB SY to
+ * drain the write buffer to DRAM before the LCDIF reads the pixels.
+ */
+ dsb();
+}
+
static struct fb_ops lcdif_fb_ops = {
.fb_enable = lcdif_fb_enable,
.fb_disable = lcdif_fb_disable,
+ .fb_damage = lcdif_fb_damage,
};
static int lcdif_probe(struct device *dev)
--
2.43.0
next prev parent reply other threads:[~2026-06-02 4:11 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-02 4:09 [PATCH 0/7] video: enable boot splash on i.MX8MP with LVDS panel Johannes Schneider
2026-06-02 4:09 ` [PATCH 1/6] clk: imx8mp: add 700 MHz rate entry for VIDEO_PLL1 Johannes Schneider
2026-06-02 7:04 ` Ahmad Fatoum
2026-06-02 4:09 ` [PATCH 2/6] pmdomain: imx8mp-blk-ctrl: add media blk-ctrl power domain support Johannes Schneider
2026-06-02 7:34 ` Ahmad Fatoum
2026-06-02 4:09 ` [PATCH 3/6] video: backlight-pwm: make power-supply and enable-gpio optional Johannes Schneider
2026-06-02 7:22 ` Ahmad Fatoum
2026-06-02 4:09 ` [PATCH 4/6] video: add i.MX8MP LCDIF2 V8 framebuffer driver Johannes Schneider
2026-06-02 7:16 ` Ahmad Fatoum
2026-06-04 3:34 ` SCHNEIDER Johannes
2026-06-02 4:09 ` [PATCH 5/6] video: add generic panel-lvds driver Johannes Schneider
2026-06-02 7:12 ` Ahmad Fatoum
2026-06-02 4:09 ` Johannes Schneider [this message]
2026-06-02 8:12 ` [PATCH 6/6] video: imx-lcdif: drain write-combine buffer before LCDIF DMA reads pixels Lucas Stach
2026-06-02 9:37 ` Ahmad Fatoum
2026-06-02 12:12 ` Lucas Stach
2026-06-02 15:17 ` Michael Grzeschik
2026-06-02 17:10 ` Michael Grzeschik
2026-06-04 3:42 ` SCHNEIDER Johannes
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260602040953.1060278-7-johannes.schneider@leica-geosystems.com \
--to=johannes.schneider@leica-geosystems.com \
--cc=barebox@lists.infradead.org \
--cc=thomas.haemmerle@leica-geosystems.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox