3a9410
From 94a416e1b67c72c69c19944937acc619503484b3 Mon Sep 17 00:00:00 2001
3a9410
Message-Id: <94a416e1b67c72c69c19944937acc619503484b3@dist-git>
3a9410
From: Boris Fiuczynski <fiuczy@linux.ibm.com>
3a9410
Date: Fri, 13 May 2022 12:31:03 +0200
3a9410
Subject: [PATCH] util: make reuse of ccw device address format constant
3a9410
3a9410
Signed-off-by: Boris Fiuczynski <fiuczy@linux.ibm.com>
3a9410
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
3a9410
(cherry picked from commit b41163005c0874b1d4809523227cd1921f4852d2)
3a9410
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2165011
3a9410
Signed-off-by: Thomas Huth <thuth@redhat.com>
3a9410
---
3a9410
 src/conf/device_conf.h  | 1 -
3a9410
 src/qemu/qemu_command.c | 2 +-
3a9410
 src/util/virccw.c       | 2 +-
3a9410
 src/util/virccw.h       | 2 ++
3a9410
 4 files changed, 4 insertions(+), 3 deletions(-)
3a9410
3a9410
diff --git a/src/conf/device_conf.h b/src/conf/device_conf.h
3a9410
index 60d90bbf19..950d333e2e 100644
3a9410
--- a/src/conf/device_conf.h
3a9410
+++ b/src/conf/device_conf.h
3a9410
@@ -202,7 +202,6 @@ int virDomainDeviceCCWAddressParseXML(xmlNodePtr node,
3a9410
                                       virCCWDeviceAddress *addr);
3a9410
 bool virDomainDeviceCCWAddressEqual(virCCWDeviceAddress *addr1,
3a9410
                                     virCCWDeviceAddress *addr2);
3a9410
-#define VIR_CCW_DEVICE_ADDRESS_FMT "%x.%x.%04x"
3a9410
 
3a9410
 int virDomainDeviceDriveAddressParseXML(xmlNodePtr node,
3a9410
                                         virDomainDeviceDriveAddress *addr);
3a9410
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
3a9410
index d23af97e0c..a7855d3370 100644
3a9410
--- a/src/qemu/qemu_command.c
3a9410
+++ b/src/qemu/qemu_command.c
3a9410
@@ -613,7 +613,7 @@ qemuBuildDeviceAddressProps(virJSONValue *props,
3a9410
         return 0;
3a9410
 
3a9410
     case VIR_DOMAIN_DEVICE_ADDRESS_TYPE_CCW: {
3a9410
-        g_autofree char *devno = g_strdup_printf("%x.%x.%04x",
3a9410
+        g_autofree char *devno = g_strdup_printf(VIR_CCW_DEVICE_ADDRESS_FMT,
3a9410
                                                  info->addr.ccw.cssid,
3a9410
                                                  info->addr.ccw.ssid,
3a9410
                                                  info->addr.ccw.devno);
3a9410
diff --git a/src/util/virccw.c b/src/util/virccw.c
3a9410
index 409287b380..5a19d3a112 100644
3a9410
--- a/src/util/virccw.c
3a9410
+++ b/src/util/virccw.c
3a9410
@@ -25,5 +25,5 @@
3a9410
 char*
3a9410
 virCCWDeviceAddressAsString(virCCWDeviceAddress *addr)
3a9410
 {
3a9410
-    return g_strdup_printf("%x.%x.%04x", addr->cssid, addr->ssid, addr->devno);
3a9410
+    return g_strdup_printf(VIR_CCW_DEVICE_ADDRESS_FMT, addr->cssid, addr->ssid, addr->devno);
3a9410
 }
3a9410
diff --git a/src/util/virccw.h b/src/util/virccw.h
3a9410
index 21a03406fa..127359c299 100644
3a9410
--- a/src/util/virccw.h
3a9410
+++ b/src/util/virccw.h
3a9410
@@ -22,6 +22,8 @@
3a9410
 
3a9410
 #include "internal.h"
3a9410
 
3a9410
+#define VIR_CCW_DEVICE_ADDRESS_FMT "%x.%x.%04x"
3a9410
+
3a9410
 typedef struct _virCCWDeviceAddress virCCWDeviceAddress;
3a9410
 struct _virCCWDeviceAddress {
3a9410
     unsigned int cssid;
3a9410
-- 
3a9410
2.39.1
3a9410