Re: [PATCH v5 2/6] efi: stub: Change _debug_uart_putc function to inline

[Kever Yang]

On 2025/5/20 19:36, Lukasz Czechowski wrote:

Update definition of _debug_uart_putc to static inline.
This will allow to avoid compilation warnings about unused code
after introduction of patch changing debug uart functions to
dummies if CONFIG_DEBUG_UART is not set.
This also matches the instructions in include/debug_uart.h and
provides consistency with implementations for other platforms.

Signed-off-by: Lukasz Czechowski <lukasz.czechow...@thaumatec.com>

Reviewed-by: Kever Yang <kever.y...@rock-chips.com>

Thanks,
- Kever

---
  lib/efi/efi_stub.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/efi/efi_stub.c b/lib/efi/efi_stub.c
index 40fc29d9adf7..a083c7f1e9b4 100644
--- a/lib/efi/efi_stub.c
+++ b/lib/efi/efi_stub.c
@@ -83,7 +83,7 @@ void puts(const char *str)
                putc(*str++);
  }

-static void _debug_uart_putc(int ch)

+static inline void _debug_uart_putc(int ch)
  {
        putc(ch);
  }