diff options
| author | Simon Glass <sjg@chromium.org> | 2021-01-21 06:10:54 +0300 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2021-03-13 01:41:35 +0300 |
| commit | 7bd06587decafabb56f68de3ae87adb4c49ca8db (patch) | |
| tree | 79186153ce0f6339b3f172bdffcd300e502e64db /include/log.h | |
| parent | 9ad7a6c25c7142a46fe4b811c13bc3280c4bb27f (diff) | |
| download | u-boot-7bd06587decafabb56f68de3ae87adb4c49ca8db.tar.xz | |
log: Add return-checking macros for 0 being success
The existing log_ret() and log_msg_ret() macros consider an error to be
less than zero. But some function may return a positive number to indicate
a different kind of failure. Add macros to check for that also.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'include/log.h')
| -rw-r--r-- | include/log.h | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/include/log.h b/include/log.h index c0453d2f97..6ef891d4d2 100644 --- a/include/log.h +++ b/include/log.h @@ -316,10 +316,30 @@ void __assert_fail(const char *assertion, const char *file, unsigned int line, __ret); \ __ret; \ }) + +/* + * Similar to the above, but any non-zero value is consider an error, not just + * values less than 0. + */ +#define log_retz(_ret) ({ \ + int __ret = (_ret); \ + if (__ret) \ + log(LOG_CATEGORY, LOGL_ERR, "returning err=%d\n", __ret); \ + __ret; \ + }) +#define log_msg_retz(_msg, _ret) ({ \ + int __ret = (_ret); \ + if (__ret) \ + log(LOG_CATEGORY, LOGL_ERR, "%s: returning err=%d\n", _msg, \ + __ret); \ + __ret; \ + }) #else /* Non-logging versions of the above which just return the error code */ #define log_ret(_ret) (_ret) #define log_msg_ret(_msg, _ret) ((void)(_msg), _ret) +#define log_retz(_ret) (_ret) +#define log_msg_retz(_msg, _ret) ((void)(_msg), _ret) #endif /** * enum log_rec_flags - Flags for a log record */ |