From 99b84c33f977b8b16aa877a99cc3d105a0a65094 Mon Sep 17 00:00:00 2001
From: Jeremy Kerr <jk@codeconstruct.com.au>
Date: Thu, 30 May 2024 14:10:09 +0800
Subject: [PATCH] mctpd: use .DRAFT suffix on BusOwner1 interface

We'll be modifying this for the new Role-based interfaces which exist on
the link/interface objects, so use a temporary name for now.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
---
 docs/endpoint-recovery.md | 4 ++--
 docs/mctpd.md             | 2 +-
 src/mctpd.c               | 2 +-
 tests/mctp_test_utils.py  | 2 +-
 tests/test_mctpd.py       | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/docs/endpoint-recovery.md b/docs/endpoint-recovery.md
index d67ab4a..d5bd109 100644
--- a/docs/endpoint-recovery.md
+++ b/docs/endpoint-recovery.md
@@ -361,7 +361,7 @@ The general strategy for tracking endpoint lifecycles is [as follows]
    registered as an MCTP endpoint.
 
 2. The FRU data is decoded and `SetupEndpoint` on the
-   `au.com.codeconstruct.MCTP.BusOwner1` interface of the
+   `au.com.codeconstruct.MCTP.BusOwner1.DRAFT` interface of the
    `/au/com/codeconstruct/mctp1`
    object hosted by `mctpd` is invoked.
 
@@ -402,7 +402,7 @@ The general strategy for tracking endpoint lifecycles is [as follows]
    poll the drive CTEMP.
 
 9. `mctpd` [emits a `PropertiesChanged` signal][dbus-spec-standard-interfaces-properties]
-   from `/au/com/codeconstruct/mctp/networks/1/enpoints/9` with the following
+   from `/au/com/codeconstruct/mctp/networks/1/endpoints/9` with the following
    contents:
 
    ```
diff --git a/docs/mctpd.md b/docs/mctpd.md
index 2e1f967..3cc65da 100644
--- a/docs/mctpd.md
+++ b/docs/mctpd.md
@@ -13,7 +13,7 @@ As well as those standard interfaces, `mctpd` provides methods to add and
 configure MCTP endpoints. These are provided by the `au.com.codeconstruct.MCTP1`
 D-Bus interface.
 
-## Bus-owner methods: `au.com.codeconstruct.MCTP.BusOwner1` interface
+## Bus-owner methods: `au.com.codeconstruct.MCTP.BusOwner1.DRAFT` interface
 
 This interface exposes bus-owner level functions.
 
diff --git a/src/mctpd.c b/src/mctpd.c
index 5c9ec17..12b9c3c 100644
--- a/src/mctpd.c
+++ b/src/mctpd.c
@@ -41,7 +41,7 @@
 #define min(a, b) ((a) < (b) ? (a) : (b))
 
 #define MCTP_DBUS_PATH "/au/com/codeconstruct/mctp1"
-#define CC_MCTP_DBUS_IFACE_BUSOWNER "au.com.codeconstruct.MCTP.BusOwner1"
+#define CC_MCTP_DBUS_IFACE_BUSOWNER "au.com.codeconstruct.MCTP.BusOwner1.DRAFT"
 #define CC_MCTP_DBUS_IFACE_ENDPOINT "au.com.codeconstruct.MCTP.Endpoint1"
 #define CC_MCTP_DBUS_IFACE_TESTING "au.com.codeconstruct.MCTPTesting"
 #define MCTP_DBUS_NAME "au.com.codeconstruct.MCTP1"
diff --git a/tests/mctp_test_utils.py b/tests/mctp_test_utils.py
index 1b40bdc..ffd31cc 100644
--- a/tests/mctp_test_utils.py
+++ b/tests/mctp_test_utils.py
@@ -4,7 +4,7 @@ async def mctpd_mctp_obj(dbus):
             'au.com.codeconstruct.MCTP1',
             '/au/com/codeconstruct/mctp1'
         )
-    return await obj.get_interface('au.com.codeconstruct.MCTP.BusOwner1')
+    return await obj.get_interface('au.com.codeconstruct.MCTP.BusOwner1.DRAFT')
 
 async def mctpd_mctp_endpoint_obj(dbus, path):
     obj = await dbus.get_proxy_object(
diff --git a/tests/test_mctpd.py b/tests/test_mctpd.py
index 2ca90fa..8c2e550 100644
--- a/tests/test_mctpd.py
+++ b/tests/test_mctpd.py
@@ -13,7 +13,7 @@
 # - I: Interface
 MCTPD_C = 'au.com.codeconstruct.MCTP1'
 MCTPD_MCTP_P = '/au/com/codeconstruct/mctp1'
-MCTPD_MCTP_I = 'au.com.codeconstruct.MCTP.BusOwner1'
+MCTPD_MCTP_I = 'au.com.codeconstruct.MCTP.BusOwner1.DRAFT'
 MCTPD_ENDPOINT_I = 'au.com.codeconstruct.MCTP.Endpoint1'
 DBUS_OBJECT_MANAGER_I = 'org.freedesktop.DBus.ObjectManager'
 DBUS_PROPERTIES_I = 'org.freedesktop.DBus.Properties'