[refactoring][SerialPrinter]: Remove atomic print lock from SerialPrinter

                       Eliminated the use of std::atomic<bool> print_lock and related locking logic from SerialPrinter.</bool>
                       Simplified the print() method to directly invoke PrintHandler::create without contention checks.
                       Updated method signature to be const-correct.

Signed-off-by: Goran Mišković <[email protected]>

Author: schkovich

include/SerialPrinter.hpp

@@ -47,9 +47,7 @@ namespace e5 {
              * @param message std::string to print
              * @return PICO_OK on success, or error code on failure
              */
-            uint32_t print(std::unique_ptr<std::string> message);
-
-            static std::atomic<bool> print_lock; ///< Global serialization and re-entrancy guard
+            uint32_t print(std::unique_ptr<std::string> message) const;
     };
 
 } // namespace e5

src/SerialPrinter.cpp

@@ -10,25 +10,15 @@
 #include "ContextManager.hpp"
 #include "MessageBuffer.hpp"
 #include "PrintHandler.hpp"
-#include <atomic>
 
 namespace e5 {
 
-    std::atomic<bool> SerialPrinter::print_lock{false};
-
     // Constructor implementation
     SerialPrinter::SerialPrinter(const AsyncCtx &ctx) : m_ctx(ctx) {}
 
     // Print method implementation for std::string
-    uint32_t SerialPrinter::print(std::unique_ptr<std::string> message) {
-        bool expected = false;
-        // Use class static member for serialization and reentrancy guard
-        if (!print_lock.compare_exchange_strong(expected, true, std::memory_order_acquire)) {
-            // Lock is already held, skip or handle contention (for test, just return error)
-            return PICO_ERROR_RESOURCE_IN_USE; // Indicate contention
-        }
+    uint32_t SerialPrinter::print(std::unique_ptr<std::string> message) const {
         PrintHandler::create(m_ctx, std::move(message));
-        print_lock.store(false, std::memory_order_release);
         return PICO_OK; // Return success code
     }