zephyr: Fix sys_io.cc to respect Kconfig

Functions like console_init, console_getchar, and console_putchar all depend on CONFIG_CONSOLE_GETCHAR Kconfig. We shouldn't assume it'll always be there. BUG: b/380001331 Change-Id: Ice6e93c244134c51ef9110ed3cb24a7835e6f07f Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/263834 Lint: Lint 🤖 <[email protected]> Docs-Not-Needed: Yuval Peress <[email protected]> Reviewed-by: Ted Pudlik <[email protected]> Pigweed-Auto-Submit: Yuval Peress <[email protected]> Commit-Queue: Auto-Submit <[email protected]>
google · Feb 1, 2025 · 05504be · 05504be
1 parent 471eba4
commit 05504be
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/pw_sys_io_zephyr/sys_io.cc b/pw_sys_io_zephyr/sys_io.cc
@@ -20,15 +20,17 @@
 #include <zephyr/usb/usb_device.h>
 
 static int sys_io_init(void) {
-  int err;
+  int err = 0;
 
   if (IS_ENABLED(CONFIG_PIGWEED_SYS_IO_USB)) {
     err = usb_enable(nullptr);
     if (err) {
       return err;
     }
   }
-  err = console_init();
+  if (IS_ENABLED(CONFIG_CONSOLE_GETCHAR)) {
+    err = console_init();
+  }
   return err;
 }
 
@@ -48,6 +50,9 @@ Status ReadByte(std::byte* dest) {
 }
 
 Status TryReadByte(std::byte* dest) {
+  if (!IS_ENABLED(CONFIG_CONSOLE_GETCHAR)) {
+    return Status::Unimplemented();
+  }
   if (dest == nullptr) {
     return Status::InvalidArgument();
   }
@@ -64,6 +69,9 @@ Status TryReadByte(std::byte* dest) {
 }
 
 Status WriteByte(std::byte b) {
+  if (!IS_ENABLED(CONFIG_CONSOLE_GETCHAR)) {
+    return Status::Unimplemented();
+  }
   return console_putchar(static_cast<char>(b)) < 0
              ? Status::FailedPrecondition()
              : OkStatus();