This is an automated email from Gerrit.

"Jan Matyas <[email protected]>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/6667

-- gerrit

commit a379182aae4ab84187558d7c8a820d05e4f78048
Author: Jan Matyas <[email protected]>
Date:   Mon Nov 1 15:28:28 2021 +0100

    helper/log: Add macros for target-related errors/warnings/...
    
    This commit introduces macros for a target related log entries
    (error, warning, ...) which is a very common operation in OpenOCD:
    
    * LOG_TARGET_ERROR
    * LOG_TARGET_WARNING
    * LOG_TARGET_INFO
    * LOG_TARGET_DEBUG
    * LOG_TARGET_DEBUG_IO
    
    The goal is to have one macro for this common operation and to
    make such log entries look the same way - to make it more readable
    for humans as well easier for parsing via scripts.
    
    Change-Id: I6166565fc9040b03d3fca5c3aa44a1ccbcf96ad2
    Signed-off-by: Jan Matyas <[email protected]>

diff --git a/src/helper/log.h b/src/helper/log.h
index 34ff835b8..621d467b4 100644
--- a/src/helper/log.h
+++ b/src/helper/log.h
@@ -143,6 +143,23 @@ extern int debug_level;
 #define LOG_OUTPUT(expr ...) \
        log_printf(LOG_LVL_OUTPUT, __FILE__, __LINE__, __func__, expr)
 
+/* Output a log entry that is related to a given target */
+
+#define LOG_TARGET_DEBUG_IO(target, fmt_str, ...) \
+       LOG_DEBUG_IO("[%s] " fmt_str, target_name(target), ##__VA_ARGS__)
+
+#define LOG_TARGET_DEBUG(target, fmt_str, ...) \
+       LOG_DEBUG("[%s] " fmt_str, target_name(target), ##__VA_ARGS__)
+
+#define LOG_TARGET_INFO(target, fmt_str, ...) \
+       LOG_INFO("[%s] " fmt_str, target_name(target), ##__VA_ARGS__)
+
+#define LOG_TARGET_WARNING(target, fmt_str, ...) \
+       LOG_WARNING("[%s] " fmt_str, target_name(target), ##__VA_ARGS__)
+
+#define LOG_TARGET_ERROR(target, fmt_str, ...) \
+       LOG_ERROR("[%s] " fmt_str, target_name(target), ##__VA_ARGS__)
+
 /* general failures
  * error codes < 100
  */

-- 

Reply via email to