platform/surface: aggregator: Rename top-level request functions to avoid ambiguities
authorMaximilian Luz <luzmaximilian@gmail.com>
Tue, 20 Dec 2022 17:56:08 +0000 (18:56 +0100)
committerHans de Goede <hdegoede@redhat.com>
Thu, 2 Feb 2023 21:48:20 +0000 (22:48 +0100)
We currently have a struct ssam_request_sync and a function
ssam_request_sync(). While this is valid C, there are some downsides to
it.

One of these is that current Sphinx versions (>= 3.0) cannot
disambiguate between the two (see disucssion and pull request linked
below). It instead emits a "WARNING: Duplicate C declaration" and links
for the struct and function in the resulting documentation link to the
same entry (i.e. both to either function or struct documentation)
instead of their respective own entries.

While we could just ignore that and wait for a fix, there's also a point
to be made that the current naming can be somewhat confusing when
searching (e.g. via grep) or trying to understand the levels of
abstraction at play:

We currently have struct ssam_request_sync and associated functions
ssam_request_sync_[alloc|free|init|wait|...]() operating on this struct.
However, function ssam_request_sync() is one abstraction level above
this. Similarly, ssam_request_sync_with_buffer() is not a function
operating on struct ssam_request_sync, but rather a sibling to
ssam_request_sync(), both using the struct under the hood.

Therefore, rename the top level request functions:

  ssam_request_sync() -> ssam_request_do_sync()
  ssam_request_sync_with_buffer() -> ssam_request_do_sync_with_buffer()
  ssam_request_sync_onstack() -> ssam_request_do_sync_onstack()

Link: https://lore.kernel.org/all/085e0ada65c11da9303d07e70c510dc45f21315b.1656756450.git.mchehab@kernel.org/
Link: https://github.com/sphinx-doc/sphinx/pull/8313
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
Link: https://lore.kernel.org/r/20221220175608.1436273-2-luzmaximilian@gmail.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Documentation/driver-api/surface_aggregator/client.rst
drivers/hid/surface-hid/surface_hid.c
drivers/hid/surface-hid/surface_kbd.c
drivers/platform/surface/aggregator/bus.c
drivers/platform/surface/aggregator/controller.c
drivers/platform/surface/surface_acpi_notify.c
drivers/platform/surface/surface_aggregator_cdev.c
drivers/platform/surface/surface_aggregator_tabletsw.c
include/linux/surface_aggregator/controller.h
include/linux/surface_aggregator/device.h

index 9d7411223a848b29a84a92e697ccee70d51d4eb9..e100ab0a24cc4ac9aed5578b4d657ee579df47b1 100644 (file)
@@ -19,7 +19,7 @@
 .. |ssam_notifier_unregister| replace:: :c:func:`ssam_notifier_unregister`
 .. |ssam_device_notifier_register| replace:: :c:func:`ssam_device_notifier_register`
 .. |ssam_device_notifier_unregister| replace:: :c:func:`ssam_device_notifier_unregister`
-.. |ssam_request_sync| replace:: :c:func:`ssam_request_sync`
+.. |ssam_request_do_sync| replace:: :c:func:`ssam_request_do_sync`
 .. |ssam_event_mask| replace:: :c:type:`enum ssam_event_mask <ssam_event_mask>`
 
 
@@ -209,12 +209,12 @@ data received from it is converted from little-endian to host endianness.
             * with the SSAM_REQUEST_HAS_RESPONSE flag set in the specification
             * above.
             */
-           status = ssam_request_sync(ctrl, &rqst, &resp);
+           status = ssam_request_do_sync(ctrl, &rqst, &resp);
 
            /*
             * Alternatively use
             *
-            *   ssam_request_sync_onstack(ctrl, &rqst, &resp, sizeof(arg_le));
+            *   ssam_request_do_sync_onstack(ctrl, &rqst, &resp, sizeof(arg_le));
             *
             * to perform the request, allocating the message buffer directly
             * on the stack as opposed to allocation via kzalloc().
@@ -230,7 +230,7 @@ data received from it is converted from little-endian to host endianness.
            return status;
    }
 
-Note that |ssam_request_sync| in its essence is a wrapper over lower-level
+Note that |ssam_request_do_sync| in its essence is a wrapper over lower-level
 request primitives, which may also be used to perform requests. Refer to its
 implementation and documentation for more details.
 
index aa80d83a83d1bdd7a07a9a1fe5dbfc29edb6b1e6..61e5814b0ad7d4993ebad9fba54a6e903c6a2d1a 100644 (file)
@@ -80,7 +80,7 @@ static int ssam_hid_get_descriptor(struct surface_hid_device *shid, u8 entry, u8
 
                rsp.length = 0;
 
-               status = ssam_retry(ssam_request_sync_onstack, shid->ctrl, &rqst, &rsp,
+               status = ssam_retry(ssam_request_do_sync_onstack, shid->ctrl, &rqst, &rsp,
                                    sizeof(*slice));
                if (status)
                        return status;
@@ -131,7 +131,7 @@ static int ssam_hid_set_raw_report(struct surface_hid_device *shid, u8 rprt_id,
 
        buf[0] = rprt_id;
 
-       return ssam_retry(ssam_request_sync, shid->ctrl, &rqst, NULL);
+       return ssam_retry(ssam_request_do_sync, shid->ctrl, &rqst, NULL);
 }
 
 static int ssam_hid_get_raw_report(struct surface_hid_device *shid, u8 rprt_id, u8 *buf, size_t len)
@@ -151,7 +151,7 @@ static int ssam_hid_get_raw_report(struct surface_hid_device *shid, u8 rprt_id,
        rsp.length = 0;
        rsp.pointer = buf;
 
-       return ssam_retry(ssam_request_sync_onstack, shid->ctrl, &rqst, &rsp, sizeof(rprt_id));
+       return ssam_retry(ssam_request_do_sync_onstack, shid->ctrl, &rqst, &rsp, sizeof(rprt_id));
 }
 
 static u32 ssam_hid_event_fn(struct ssam_event_notifier *nf, const struct ssam_event *event)
index 42933bf3e925fba7b997df89c239479c70659a11..4fbce201db6a16ded942ee51ab8fbf9040474928 100644 (file)
@@ -49,7 +49,7 @@ static int ssam_kbd_get_descriptor(struct surface_hid_device *shid, u8 entry, u8
        rsp.length = 0;
        rsp.pointer = buf;
 
-       status = ssam_retry(ssam_request_sync_onstack, shid->ctrl, &rqst, &rsp, sizeof(entry));
+       status = ssam_retry(ssam_request_do_sync_onstack, shid->ctrl, &rqst, &rsp, sizeof(entry));
        if (status)
                return status;
 
@@ -75,7 +75,7 @@ static int ssam_kbd_set_caps_led(struct surface_hid_device *shid, bool value)
        rqst.length = sizeof(value_u8);
        rqst.payload = &value_u8;
 
-       return ssam_retry(ssam_request_sync_onstack, shid->ctrl, &rqst, NULL, sizeof(value_u8));
+       return ssam_retry(ssam_request_do_sync_onstack, shid->ctrl, &rqst, NULL, sizeof(value_u8));
 }
 
 static int ssam_kbd_get_feature_report(struct surface_hid_device *shid, u8 *buf, size_t len)
@@ -97,7 +97,7 @@ static int ssam_kbd_get_feature_report(struct surface_hid_device *shid, u8 *buf,
        rsp.length = 0;
        rsp.pointer = buf;
 
-       status = ssam_retry(ssam_request_sync_onstack, shid->ctrl, &rqst, &rsp, sizeof(payload));
+       status = ssam_retry(ssam_request_do_sync_onstack, shid->ctrl, &rqst, &rsp, sizeof(payload));
        if (status)
                return status;
 
index de539938896e25c3f9d3232641fdbf1f3fe11aa1..7004eb4a63a28583fbcfb978cadf4af6206b4976 100644 (file)
@@ -136,9 +136,9 @@ int ssam_device_add(struct ssam_device *sdev)
         * is always valid and can be used for requests as long as the client
         * device we add here is registered as child under it. This essentially
         * guarantees that the client driver can always expect the preconditions
-        * for functions like ssam_request_sync (controller has to be started
-        * and is not suspended) to hold and thus does not have to check for
-        * them.
+        * for functions like ssam_request_do_sync() (controller has to be
+        * started and is not suspended) to hold and thus does not have to check
+        * for them.
         *
         * Note that for this to work, the controller has to be a parent device.
         * If it is not a direct parent, care has to be taken that the device is
index 2c99f51ccd4ec81cb1af9c3d439a7a69267e9b4a..535581c0471c509cc6836aedc66b32a345b0561d 100644 (file)
@@ -1674,7 +1674,7 @@ int ssam_request_sync_submit(struct ssam_controller *ctrl,
 EXPORT_SYMBOL_GPL(ssam_request_sync_submit);
 
 /**
- * ssam_request_sync() - Execute a synchronous request.
+ * ssam_request_do_sync() - Execute a synchronous request.
  * @ctrl: The controller via which the request will be submitted.
  * @spec: The request specification and payload.
  * @rsp:  The response buffer.
@@ -1686,9 +1686,9 @@ EXPORT_SYMBOL_GPL(ssam_request_sync_submit);
  *
  * Return: Returns the status of the request or any failure during setup.
  */
-int ssam_request_sync(struct ssam_controller *ctrl,
-                     const struct ssam_request *spec,
-                     struct ssam_response *rsp)
+int ssam_request_do_sync(struct ssam_controller *ctrl,
+                        const struct ssam_request *spec,
+                        struct ssam_response *rsp)
 {
        struct ssam_request_sync *rqst;
        struct ssam_span buf;
@@ -1722,10 +1722,10 @@ int ssam_request_sync(struct ssam_controller *ctrl,
        ssam_request_sync_free(rqst);
        return status;
 }
-EXPORT_SYMBOL_GPL(ssam_request_sync);
+EXPORT_SYMBOL_GPL(ssam_request_do_sync);
 
 /**
- * ssam_request_sync_with_buffer() - Execute a synchronous request with the
+ * ssam_request_do_sync_with_buffer() - Execute a synchronous request with the
  * provided buffer as back-end for the message buffer.
  * @ctrl: The controller via which the request will be submitted.
  * @spec: The request specification and payload.
@@ -1738,17 +1738,17 @@ EXPORT_SYMBOL_GPL(ssam_request_sync);
  * SSH_COMMAND_MESSAGE_LENGTH() macro can be used to compute the required
  * message buffer size.
  *
- * This function does essentially the same as ssam_request_sync(), but instead
- * of dynamically allocating the request and message data buffer, it uses the
- * provided message data buffer and stores the (small) request struct on the
- * heap.
+ * This function does essentially the same as ssam_request_do_sync(), but
+ * instead of dynamically allocating the request and message data buffer, it
+ * uses the provided message data buffer and stores the (small) request struct
+ * on the heap.
  *
  * Return: Returns the status of the request or any failure during setup.
  */
-int ssam_request_sync_with_buffer(struct ssam_controller *ctrl,
-                                 const struct ssam_request *spec,
-                                 struct ssam_response *rsp,
-                                 struct ssam_span *buf)
+int ssam_request_do_sync_with_buffer(struct ssam_controller *ctrl,
+                                    const struct ssam_request *spec,
+                                    struct ssam_response *rsp,
+                                    struct ssam_span *buf)
 {
        struct ssam_request_sync rqst;
        ssize_t len;
@@ -1772,7 +1772,7 @@ int ssam_request_sync_with_buffer(struct ssam_controller *ctrl,
 
        return status;
 }
-EXPORT_SYMBOL_GPL(ssam_request_sync_with_buffer);
+EXPORT_SYMBOL_GPL(ssam_request_do_sync_with_buffer);
 
 
 /* -- Internal SAM requests. ------------------------------------------------ */
@@ -1864,7 +1864,7 @@ static int __ssam_ssh_event_request(struct ssam_controller *ctrl,
        result.length = 0;
        result.pointer = &buf;
 
-       status = ssam_retry(ssam_request_sync_onstack, ctrl, &rqst, &result,
+       status = ssam_retry(ssam_request_do_sync_onstack, ctrl, &rqst, &result,
                            sizeof(params));
 
        return status < 0 ? status : buf;
index 50500e562963d6027553aabf6678c7e9fd23bf73..897cdd9c3aae8114194f3ff65ed43070e8487924 100644 (file)
@@ -590,7 +590,7 @@ static acpi_status san_rqst(struct san_data *d, struct gsb_buffer *buffer)
                return san_rqst_fixup_suspended(d, &rqst, buffer);
        }
 
-       status = __ssam_retry(ssam_request_sync_onstack, SAN_REQUEST_NUM_TRIES,
+       status = __ssam_retry(ssam_request_do_sync_onstack, SAN_REQUEST_NUM_TRIES,
                              d->ctrl, &rqst, &rsp, SAN_GSB_MAX_RQSX_PAYLOAD);
 
        if (!status) {
index 492c82e691827ff75806fbc47c517056f7d2b934..07f0ed658369b5fcaf92c3b77ce5d07e6e8bf3e1 100644 (file)
@@ -302,8 +302,8 @@ static long ssam_cdev_request(struct ssam_cdev_client *client, struct ssam_cdev_
                 * theoretical maximum (SSH_COMMAND_MAX_PAYLOAD_SIZE) of the
                 * underlying protocol (note that nothing remotely this size
                 * should ever be allocated in any normal case). This size is
-                * validated later in ssam_request_sync(), for allocation the
-                * bound imposed by u16 should be enough.
+                * validated later in ssam_request_do_sync(), for allocation
+                * the bound imposed by u16 should be enough.
                 */
                spec.payload = kzalloc(spec.length, GFP_KERNEL);
                if (!spec.payload) {
@@ -342,7 +342,7 @@ static long ssam_cdev_request(struct ssam_cdev_client *client, struct ssam_cdev_
        }
 
        /* Perform request. */
-       status = ssam_request_sync(client->cdev->ctrl, &spec, &rsp);
+       status = ssam_request_do_sync(client->cdev->ctrl, &spec, &rsp);
        if (status)
                goto out;
 
index 6147aa8879391dfaecd4c2c02b4c6839734cd6d8..9fed800c7cc098b02447e110abfc1ff1ffa376b1 100644 (file)
@@ -382,7 +382,7 @@ static int ssam_pos_get_sources_list(struct ssam_tablet_sw *sw, struct ssam_sour
        rsp.length = 0;
        rsp.pointer = (u8 *)sources;
 
-       status = ssam_retry(ssam_request_sync_onstack, sw->sdev->ctrl, &rqst, &rsp, 0);
+       status = ssam_retry(ssam_request_do_sync_onstack, sw->sdev->ctrl, &rqst, &rsp, 0);
        if (status)
                return status;
 
index 8932bc0bae1872536afe1bdcb13515b91630a117..cb7980805920a24e90556a4d5554ef285bd04866 100644 (file)
@@ -207,17 +207,17 @@ static inline int ssam_request_sync_wait(struct ssam_request_sync *rqst)
        return rqst->status;
 }
 
-int ssam_request_sync(struct ssam_controller *ctrl,
-                     const struct ssam_request *spec,
-                     struct ssam_response *rsp);
+int ssam_request_do_sync(struct ssam_controller *ctrl,
+                        const struct ssam_request *spec,
+                        struct ssam_response *rsp);
 
-int ssam_request_sync_with_buffer(struct ssam_controller *ctrl,
-                                 const struct ssam_request *spec,
-                                 struct ssam_response *rsp,
-                                 struct ssam_span *buf);
+int ssam_request_do_sync_with_buffer(struct ssam_controller *ctrl,
+                                    const struct ssam_request *spec,
+                                    struct ssam_response *rsp,
+                                    struct ssam_span *buf);
 
 /**
- * ssam_request_sync_onstack - Execute a synchronous request on the stack.
+ * ssam_request_do_sync_onstack - Execute a synchronous request on the stack.
  * @ctrl: The controller via which the request is submitted.
  * @rqst: The request specification.
  * @rsp:  The response buffer.
@@ -227,7 +227,7 @@ int ssam_request_sync_with_buffer(struct ssam_controller *ctrl,
  * fully initializes it via the provided request specification, submits it,
  * and finally waits for its completion before returning its status. This
  * helper macro essentially allocates the request message buffer on the stack
- * and then calls ssam_request_sync_with_buffer().
+ * and then calls ssam_request_do_sync_with_buffer().
  *
  * Note: The @payload_len parameter specifies the maximum payload length, used
  * for buffer allocation. The actual payload length may be smaller.
@@ -235,12 +235,12 @@ int ssam_request_sync_with_buffer(struct ssam_controller *ctrl,
  * Return: Returns the status of the request or any failure during setup, i.e.
  * zero on success and a negative value on failure.
  */
-#define ssam_request_sync_onstack(ctrl, rqst, rsp, payload_len)                        \
+#define ssam_request_do_sync_onstack(ctrl, rqst, rsp, payload_len)             \
        ({                                                                      \
                u8 __data[SSH_COMMAND_MESSAGE_LENGTH(payload_len)];             \
                struct ssam_span __buf = { &__data[0], ARRAY_SIZE(__data) };    \
                                                                                \
-               ssam_request_sync_with_buffer(ctrl, rqst, rsp, &__buf);         \
+               ssam_request_do_sync_with_buffer(ctrl, rqst, rsp, &__buf);      \
        })
 
 /**
@@ -349,7 +349,7 @@ struct ssam_request_spec_md {
  * zero on success and negative on failure. The ``ctrl`` parameter is the
  * controller via which the request is being sent.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_N(name, spec...)                              \
@@ -366,7 +366,7 @@ struct ssam_request_spec_md {
                rqst.length = 0;                                                \
                rqst.payload = NULL;                                            \
                                                                                \
-               return ssam_request_sync_onstack(ctrl, &rqst, NULL, 0);         \
+               return ssam_request_do_sync_onstack(ctrl, &rqst, NULL, 0);      \
        }
 
 /**
@@ -389,7 +389,7 @@ struct ssam_request_spec_md {
  * parameter is the controller via which the request is sent. The request
  * argument is specified via the ``arg`` pointer.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_W(name, atype, spec...)                       \
@@ -406,8 +406,8 @@ struct ssam_request_spec_md {
                rqst.length = sizeof(atype);                                    \
                rqst.payload = (u8 *)arg;                                       \
                                                                                \
-               return ssam_request_sync_onstack(ctrl, &rqst, NULL,             \
-                                                sizeof(atype));                \
+               return ssam_request_do_sync_onstack(ctrl, &rqst, NULL,          \
+                                                   sizeof(atype));             \
        }
 
 /**
@@ -430,7 +430,7 @@ struct ssam_request_spec_md {
  * the controller via which the request is sent. The request's return value is
  * written to the memory pointed to by the ``ret`` parameter.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_R(name, rtype, spec...)                       \
@@ -453,7 +453,7 @@ struct ssam_request_spec_md {
                rsp.length = 0;                                                 \
                rsp.pointer = (u8 *)ret;                                        \
                                                                                \
-               status = ssam_request_sync_onstack(ctrl, &rqst, &rsp, 0);       \
+               status = ssam_request_do_sync_onstack(ctrl, &rqst, &rsp, 0);    \
                if (status)                                                     \
                        return status;                                          \
                                                                                \
@@ -491,7 +491,7 @@ struct ssam_request_spec_md {
  * request argument is specified via the ``arg`` pointer. The request's return
  * value is written to the memory pointed to by the ``ret`` parameter.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_WR(name, atype, rtype, spec...)               \
@@ -514,7 +514,7 @@ struct ssam_request_spec_md {
                rsp.length = 0;                                                 \
                rsp.pointer = (u8 *)ret;                                        \
                                                                                \
-               status = ssam_request_sync_onstack(ctrl, &rqst, &rsp, sizeof(atype)); \
+               status = ssam_request_do_sync_onstack(ctrl, &rqst, &rsp, sizeof(atype)); \
                if (status)                                                     \
                        return status;                                          \
                                                                                \
@@ -550,7 +550,7 @@ struct ssam_request_spec_md {
  * parameter is the controller via which the request is sent, ``tid`` the
  * target ID for the request, and ``iid`` the instance ID.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_MD_N(name, spec...)                           \
@@ -567,7 +567,7 @@ struct ssam_request_spec_md {
                rqst.length = 0;                                                \
                rqst.payload = NULL;                                            \
                                                                                \
-               return ssam_request_sync_onstack(ctrl, &rqst, NULL, 0);         \
+               return ssam_request_do_sync_onstack(ctrl, &rqst, NULL, 0);      \
        }
 
 /**
@@ -592,7 +592,7 @@ struct ssam_request_spec_md {
  * ``tid`` the target ID for the request, and ``iid`` the instance ID. The
  * request argument is specified via the ``arg`` pointer.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_MD_W(name, atype, spec...)                    \
@@ -609,7 +609,7 @@ struct ssam_request_spec_md {
                rqst.length = sizeof(atype);                                    \
                rqst.payload = (u8 *)arg;                                       \
                                                                                \
-               return ssam_request_sync_onstack(ctrl, &rqst, NULL,             \
+               return ssam_request_do_sync_onstack(ctrl, &rqst, NULL,          \
                                                 sizeof(atype));                \
        }
 
@@ -635,7 +635,7 @@ struct ssam_request_spec_md {
  * the target ID for the request, and ``iid`` the instance ID. The request's
  * return value is written to the memory pointed to by the ``ret`` parameter.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_MD_R(name, rtype, spec...)                    \
@@ -658,7 +658,7 @@ struct ssam_request_spec_md {
                rsp.length = 0;                                                 \
                rsp.pointer = (u8 *)ret;                                        \
                                                                                \
-               status = ssam_request_sync_onstack(ctrl, &rqst, &rsp, 0);       \
+               status = ssam_request_do_sync_onstack(ctrl, &rqst, &rsp, 0);    \
                if (status)                                                     \
                        return status;                                          \
                                                                                \
@@ -698,7 +698,7 @@ struct ssam_request_spec_md {
  * The request argument is specified via the ``arg`` pointer. The request's
  * return value is written to the memory pointed to by the ``ret`` parameter.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_MD_WR(name, atype, rtype, spec...)            \
@@ -722,7 +722,7 @@ struct ssam_request_spec_md {
                rsp.length = 0;                                                 \
                rsp.pointer = (u8 *)ret;                                        \
                                                                                \
-               status = ssam_request_sync_onstack(ctrl, &rqst, &rsp, sizeof(atype)); \
+               status = ssam_request_do_sync_onstack(ctrl, &rqst, &rsp, sizeof(atype)); \
                if (status)                                                     \
                        return status;                                          \
                                                                                \
index 4da20b7a0ee5edeff70de4022fc6c9a122c5b369..1545e5567b152215f1920550e877aa7b06360c54 100644 (file)
@@ -456,7 +456,7 @@ static inline int ssam_device_register_clients(struct ssam_device *sdev)
  * device of the request and by association the controller via which the
  * request is sent.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_CL_N(name, spec...)                   \
@@ -490,7 +490,7 @@ static inline int ssam_device_register_clients(struct ssam_device *sdev)
  * which the request is sent. The request's argument is specified via the
  * ``arg`` pointer.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_CL_W(name, atype, spec...)            \
@@ -524,7 +524,7 @@ static inline int ssam_device_register_clients(struct ssam_device *sdev)
  * the request is sent. The request's return value is written to the memory
  * pointed to by the ``ret`` parameter.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_CL_R(name, rtype, spec...)            \
@@ -560,7 +560,7 @@ static inline int ssam_device_register_clients(struct ssam_device *sdev)
  * specified via the ``arg`` pointer. The request's return value is written to
  * the memory pointed to by the ``ret`` parameter.
  *
- * Refer to ssam_request_sync_onstack() for more details on the behavior of
+ * Refer to ssam_request_do_sync_onstack() for more details on the behavior of
  * the generated function.
  */
 #define SSAM_DEFINE_SYNC_REQUEST_CL_WR(name, atype, rtype, spec...)            \