[PATCH] video: sandbox: fix missing shim definition of sandbox_sdl_remove_display()

From: Vladimir Oltean <vladimir.oltean@nxp.com>
To: u-boot@lists.denx.de
Cc: Tom Rini <trini@konsulko.com>, Simon Glass <sjg@chromium.org>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>
Subject: [PATCH] video: sandbox: fix missing shim definition of sandbox_sdl_remove_display()
Date: Fri, 11 Mar 2022 20:52:28 +0200	[thread overview]
Message-ID: <20220311185228.2471858-1-vladimir.oltean@nxp.com> (raw)

When CONFIG_SANDBOX_SDL=n, sandbox_sdl_set_bpp() from drivers/video/sandbox_sdl.c
calls sandbox_sdl_remove_display() in arch/sandbox/cpu/sdl.c, but this
isn't compiled in. A shim definition is missing, leading to a
compilation warning (missing function prototype) and a linkage bug.

Fixes: 8657ad43f353 ("sandbox: video: Add BMP tests for 32bpp and 8bpp modes")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
I see Simon's email on the list from January 10 saying:

| I see that I broke it...it needs a static inline for
| sandbox_sdl_remove_display().

https://lore.kernel.org/all/CAPnjgZ0yqn_MjZ3uTLCrRp1-ifskKJgTmqmA4x31JZqVYn=Qgg@mail.gmail.com/

but I'm looking at the current 'master' and 'next' branches and that bug
is still there, huh. Am I missing something?

 arch/sandbox/include/asm/sdl.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/sandbox/include/asm/sdl.h b/arch/sandbox/include/asm/sdl.h
index e271a849af10..56dcb84803d3 100644
--- a/arch/sandbox/include/asm/sdl.h
+++ b/arch/sandbox/include/asm/sdl.h
@@ -94,6 +94,11 @@ static inline int sandbox_sdl_init_display(int width, int height, int log2_bpp,
 	return -ENODEV;
 }
 
+static inline int sandbox_sdl_remove_display(void)
+{
+	return -ENODEV;
+}
+
 static inline int sandbox_sdl_sync(void *lcd_base)
 {
 	return -ENODEV;
-- 
2.25.1

