[PATCH 1/1] sandbox: make SDL window resizable

[Heinrich Schuchardt <xypron.glpk@gmx.de>]

On 05.10.20 03:41, Simon Glass wrote:
> Hi Heinrich,
>
> On Mon, 28 Sep 2020 at 19:11, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>>
>> Without resizing the SDL window showed by
>>
>>     ./u-boot -D -l
>>
>> is not legible on a high resolution screen.
>>
>> Start with a maximized window and allow resizing.
>>
>> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>> ---
>>  arch/sandbox/cpu/sdl.c | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>
> Have you troubled the --double_lcd option?

That looks better.

This option is not documented in doc/arch/sandbox.rst.

>
> Does this have any effect on performance?

With --double_lcd you are already doing a resize operation. I could not
observe any performance loss for a maximized window. The BitBlt
operations in X11 are well accelerated by current GPUs.

>
> I actually don't like this as it makes the text hard to read, and full
> screen is annoying. I wonder if this should be a flag?

I understand that you do not want the window maximized. But is anything
wrong about making the window resizable (SDL_WINDOW_RESIZABLE)?

Shall I update the patch accordingly?

Best regards

Heinrich

>
>> diff --git a/arch/sandbox/cpu/sdl.c b/arch/sandbox/cpu/sdl.c
>> index 7dc3dab32e..911247123f 100644
>> --- a/arch/sandbox/cpu/sdl.c
>> +++ b/arch/sandbox/cpu/sdl.c
>> @@ -127,7 +127,9 @@ int sandbox_sdl_init_display(int width, int height, int log2_bpp,
>>         sdl.pitch = sdl.width * sdl.depth / 8;
>>         SDL_Window *screen = SDL_CreateWindow("U-Boot", SDL_WINDOWPOS_UNDEFINED,
>>                                               SDL_WINDOWPOS_UNDEFINED,
>> -                                             sdl.vis_width, sdl.vis_height, 0);
>> +                                             sdl.vis_width, sdl.vis_height,
>> +                                             SDL_WINDOW_MAXIMIZED |
>> +                                             SDL_WINDOW_RESIZABLE);
>>         if (!screen) {
>>                 printf("Unable to initialise SDL screen: %s\n",
>>                        SDL_GetError());
>> --
>> 2.28.0
>>
>
> Regards,
> Simon
>