[PATCH v2] sandbox: fix a compilation error

Tom Rini trini at konsulko.com
Tue Jun 13 19:33:32 CEST 2023


On Tue, Jun 13, 2023 at 07:05:52PM +0300, Sergei Antonov wrote:
> On Tue, 13 Jun 2023 at 17:58, Simon Glass <sjg at chromium.org> wrote:
> > The problem with this is it then cannot be built on non-sandbox
> > boards. I think v1 was OK.
> 
> Before the patch as well as after the patch:
> if CONFIG_SANDBOX_SDL
>   declaration of sandbox_sdl_set_bpp()
> else
>   definition of sandbox_sdl_set_bpp() returning error
> end
> This patch only moves code to another header file in arch/sandbox/include/asm.
> How can a non-sandbox build fail because of this patch?

Right, test/dm/video.c is for sandbox-only.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230613/231beac8/attachment.sig>


More information about the U-Boot mailing list