Re: [PATCH 2/3] riscv: qemu: Enable EFI framebuffer

[Leo Liang <ycliang@andestech.com>]

Hi Lekkit,

On Sun, Sep 22, 2024 at 01:11:01PM +0300, lekkit@at.encryp.ch wrote:
> [EXTERNAL MAIL]
> 
> From: LekKit <50500857+LekKit@users.noreply.github.com>
> 
> Enable framebuffer for better virtual machine integration.
> Some guests need EFI FB to work properly.
> ---
>  board/emulation/qemu-riscv/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/board/emulation/qemu-riscv/Kconfig b/board/emulation/qemu-riscv/Kconfig
> index 9538c66e..012ac14a 100644
> --- a/board/emulation/qemu-riscv/Kconfig
> +++ b/board/emulation/qemu-riscv/Kconfig
> @@ -58,6 +58,8 @@ config BOARD_SPECIFIC_OPTIONS # dummy
>         imply E1000
>         imply PCI
>         imply NVME_PCI
> +       imply VIDEO
> +       imply VIDEO_SIMPLE

Enabling simplefb driver generates the following compile warning for 32 bit environment.
Could you take a look and fix that?

drivers/video/simplefb.c: In function 'simple_video_probe':
drivers/video/simplefb.c:30:15: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 3 has type 'fdt_addr_t' {aka 'unsigned int'} [-Wformat=]
   30 |         debug("%s: base=%llx, size=%llu\n", __func__, base, size);
      |               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~


drivers/video/simplefb.c:30:28: note: format string is defined here
   30 |         debug("%s: base=%llx, size=%llu\n", __func__, base, size);
      |                         ~~~^
      |                            |
      |                            long long unsigned int


Best regards,
Leo

>         imply PCIE_ECAM_GENERIC
>         imply DM_RNG
>         imply DM_RTC
> --
> 2.46.0
>