* [PATCH] video: sandbox: fix missing shim definition of sandbox_sdl_remove_display()
@ 2022-03-11 18:52 Vladimir Oltean
2022-03-12 5:02 ` Simon Glass
0 siblings, 1 reply; 2+ messages in thread
From: Vladimir Oltean @ 2022-03-11 18:52 UTC (permalink / raw)
To: u-boot; +Cc: Tom Rini, Simon Glass, Heinrich Schuchardt
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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] video: sandbox: fix missing shim definition of sandbox_sdl_remove_display()
2022-03-11 18:52 [PATCH] video: sandbox: fix missing shim definition of sandbox_sdl_remove_display() Vladimir Oltean
@ 2022-03-12 5:02 ` Simon Glass
0 siblings, 0 replies; 2+ messages in thread
From: Simon Glass @ 2022-03-12 5:02 UTC (permalink / raw)
To: Vladimir Oltean; +Cc: u-boot, Tom Rini, Heinrich Schuchardt
On Fri, 11 Mar 2022 at 11:52, Vladimir Oltean <vladimir.oltean@nxp.com> wrote:
>
> 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(+)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-03-12 5:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-11 18:52 [PATCH] video: sandbox: fix missing shim definition of sandbox_sdl_remove_display() Vladimir Oltean
2022-03-12 5:02 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.