diff options
| author | Joe Komlodi <komlodi@google.com> | 2024-02-02 20:48:46 +0000 |
|---|---|---|
| committer | Philippe Mathieu-Daudé <philmd@linaro.org> | 2024-02-22 12:47:40 +0100 |
| commit | fcc8299e29816c9b6f8d9766254fce6e8a50ee52 (patch) | |
| tree | b75b5c9d0870fb98cd3e30f70da215a082e88dd9 | |
| parent | 61f406f37726833650d8c4441a4c1baf1f5b54db (diff) | |
| download | qemu-fcc8299e29816c9b6f8d9766254fce6e8a50ee52.zip qemu-fcc8299e29816c9b6f8d9766254fce6e8a50ee52.tar.gz qemu-fcc8299e29816c9b6f8d9766254fce6e8a50ee52.tar.bz2 | |
hw/i2c/smbus_slave: Add object path on error prints
The current logging doesn't tell us which specific smbus device is an
error state.
Signed-off-by: Joe Komlodi <komlodi@google.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240202204847.2062798-3-komlodi@google.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
| -rw-r--r-- | hw/i2c/smbus_slave.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/hw/i2c/smbus_slave.c b/hw/i2c/smbus_slave.c index 1300c9e..9f9afc2 100644 --- a/hw/i2c/smbus_slave.c +++ b/hw/i2c/smbus_slave.c @@ -25,11 +25,15 @@ #define DPRINTF(fmt, ...) \ do { printf("smbus(%02x): " fmt , dev->i2c.address, ## __VA_ARGS__); } while (0) #define BADF(fmt, ...) \ -do { fprintf(stderr, "smbus: error: " fmt , ## __VA_ARGS__); exit(1);} while (0) +do { g_autofree char *qom_path = object_get_canonical_path(OBJECT(dev)); \ + fprintf(stderr, "%s: smbus: error: " fmt , qom_path, ## __VA_ARGS__); \ + exit(1); } while (0) #else #define DPRINTF(fmt, ...) do {} while(0) #define BADF(fmt, ...) \ -do { fprintf(stderr, "smbus: error: " fmt , ## __VA_ARGS__);} while (0) +do { g_autofree char *qom_path = object_get_canonical_path(OBJECT(dev)); \ + fprintf(stderr, "%s: smbus: error: " fmt , qom_path, ## __VA_ARGS__); \ + } while (0) #endif enum { |