[V2,1/3] staging: vchiq_core: Make vchiq_dump_service_state static
Commit Message
The function vchiq_dump_service_state() is only used by vchiq_dump_state()
within vchiq_core.c. So move the definition of vchiq_dump_state() below
vchiq_dump_service_state() in order to make it static.
Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
---
.../interface/vchiq_arm/vchiq_core.c | 169 +++++++++---------
.../interface/vchiq_arm/vchiq_core.h | 3 -
2 files changed, 85 insertions(+), 87 deletions(-)
--
2.34.1
Comments
Hi Stefan,
Thank you for the patch.
On Sun, Oct 29, 2023 at 01:48:35PM +0100, Stefan Wahren wrote:
> The function vchiq_dump_service_state() is only used by vchiq_dump_state()
> within vchiq_core.c. So move the definition of vchiq_dump_state() below
> vchiq_dump_service_state() in order to make it static.
>
> Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
I don't remember suggesting this, but that's fine, it looks like a good
change :-)
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> .../interface/vchiq_arm/vchiq_core.c | 169 +++++++++---------
> .../interface/vchiq_arm/vchiq_core.h | 3 -
> 2 files changed, 85 insertions(+), 87 deletions(-)
>
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
> index 39b857da2d42..94073f92651a 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
> @@ -3428,90 +3428,8 @@ vchiq_dump_shared_state(void *dump_context, struct vchiq_state *state,
> return 0;
> }
>
> -int vchiq_dump_state(void *dump_context, struct vchiq_state *state)
> -{
> - char buf[80];
> - int len;
> - int i;
> - int err;
> -
> - len = scnprintf(buf, sizeof(buf), "State %d: %s", state->id,
> - conn_state_names[state->conn_state]);
> - err = vchiq_dump(dump_context, buf, len + 1);
> - if (err)
> - return err;
> -
> - len = scnprintf(buf, sizeof(buf), " tx_pos=%x(@%pK), rx_pos=%x(@%pK)",
> - state->local->tx_pos,
> - state->tx_data + (state->local_tx_pos & VCHIQ_SLOT_MASK),
> - state->rx_pos,
> - state->rx_data + (state->rx_pos & VCHIQ_SLOT_MASK));
> - err = vchiq_dump(dump_context, buf, len + 1);
> - if (err)
> - return err;
> -
> - len = scnprintf(buf, sizeof(buf), " Version: %d (min %d)",
> - VCHIQ_VERSION, VCHIQ_VERSION_MIN);
> - err = vchiq_dump(dump_context, buf, len + 1);
> - if (err)
> - return err;
> -
> - if (VCHIQ_ENABLE_STATS) {
> - len = scnprintf(buf, sizeof(buf),
> - " Stats: ctrl_tx_count=%d, ctrl_rx_count=%d, error_count=%d",
> - state->stats.ctrl_tx_count, state->stats.ctrl_rx_count,
> - state->stats.error_count);
> - err = vchiq_dump(dump_context, buf, len + 1);
> - if (err)
> - return err;
> - }
> -
> - len = scnprintf(buf, sizeof(buf),
> - " Slots: %d available (%d data), %d recyclable, %d stalls (%d data)",
> - ((state->slot_queue_available * VCHIQ_SLOT_SIZE) -
> - state->local_tx_pos) / VCHIQ_SLOT_SIZE,
> - state->data_quota - state->data_use_count,
> - state->local->slot_queue_recycle - state->slot_queue_available,
> - state->stats.slot_stalls, state->stats.data_stalls);
> - err = vchiq_dump(dump_context, buf, len + 1);
> - if (err)
> - return err;
> -
> - err = vchiq_dump_platform_state(dump_context);
> - if (err)
> - return err;
> -
> - err = vchiq_dump_shared_state(dump_context,
> - state,
> - state->local,
> - "Local");
> - if (err)
> - return err;
> - err = vchiq_dump_shared_state(dump_context,
> - state,
> - state->remote,
> - "Remote");
> - if (err)
> - return err;
> -
> - err = vchiq_dump_platform_instances(dump_context);
> - if (err)
> - return err;
> -
> - for (i = 0; i < state->unused_service; i++) {
> - struct vchiq_service *service = find_service_by_port(state, i);
> -
> - if (service) {
> - err = vchiq_dump_service_state(dump_context, service);
> - vchiq_service_put(service);
> - if (err)
> - return err;
> - }
> - }
> - return 0;
> -}
> -
> -int vchiq_dump_service_state(void *dump_context, struct vchiq_service *service)
> +static int
> +vchiq_dump_service_state(void *dump_context, struct vchiq_service *service)
> {
> char buf[80];
> int len;
> @@ -3606,6 +3524,89 @@ int vchiq_dump_service_state(void *dump_context, struct vchiq_service *service)
> return err;
> }
>
> +int vchiq_dump_state(void *dump_context, struct vchiq_state *state)
> +{
> + char buf[80];
> + int len;
> + int i;
> + int err;
> +
> + len = scnprintf(buf, sizeof(buf), "State %d: %s", state->id,
> + conn_state_names[state->conn_state]);
> + err = vchiq_dump(dump_context, buf, len + 1);
> + if (err)
> + return err;
> +
> + len = scnprintf(buf, sizeof(buf), " tx_pos=%x(@%pK), rx_pos=%x(@%pK)",
> + state->local->tx_pos,
> + state->tx_data + (state->local_tx_pos & VCHIQ_SLOT_MASK),
> + state->rx_pos,
> + state->rx_data + (state->rx_pos & VCHIQ_SLOT_MASK));
> + err = vchiq_dump(dump_context, buf, len + 1);
> + if (err)
> + return err;
> +
> + len = scnprintf(buf, sizeof(buf), " Version: %d (min %d)",
> + VCHIQ_VERSION, VCHIQ_VERSION_MIN);
> + err = vchiq_dump(dump_context, buf, len + 1);
> + if (err)
> + return err;
> +
> + if (VCHIQ_ENABLE_STATS) {
> + len = scnprintf(buf, sizeof(buf),
> + " Stats: ctrl_tx_count=%d, ctrl_rx_count=%d, error_count=%d",
> + state->stats.ctrl_tx_count, state->stats.ctrl_rx_count,
> + state->stats.error_count);
> + err = vchiq_dump(dump_context, buf, len + 1);
> + if (err)
> + return err;
> + }
> +
> + len = scnprintf(buf, sizeof(buf),
> + " Slots: %d available (%d data), %d recyclable, %d stalls (%d data)",
> + ((state->slot_queue_available * VCHIQ_SLOT_SIZE) -
> + state->local_tx_pos) / VCHIQ_SLOT_SIZE,
> + state->data_quota - state->data_use_count,
> + state->local->slot_queue_recycle - state->slot_queue_available,
> + state->stats.slot_stalls, state->stats.data_stalls);
> + err = vchiq_dump(dump_context, buf, len + 1);
> + if (err)
> + return err;
> +
> + err = vchiq_dump_platform_state(dump_context);
> + if (err)
> + return err;
> +
> + err = vchiq_dump_shared_state(dump_context,
> + state,
> + state->local,
> + "Local");
> + if (err)
> + return err;
> + err = vchiq_dump_shared_state(dump_context,
> + state,
> + state->remote,
> + "Remote");
> + if (err)
> + return err;
> +
> + err = vchiq_dump_platform_instances(dump_context);
> + if (err)
> + return err;
> +
> + for (i = 0; i < state->unused_service; i++) {
> + struct vchiq_service *service = find_service_by_port(state, i);
> +
> + if (service) {
> + err = vchiq_dump_service_state(dump_context, service);
> + vchiq_service_put(service);
> + if (err)
> + return err;
> + }
> + }
> + return 0;
> +}
> +
> int vchiq_send_remote_use(struct vchiq_state *state)
> {
> if (state->conn_state == VCHIQ_CONNSTATE_DISCONNECTED)
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
> index 161358db457c..ea8d58844775 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
> @@ -507,9 +507,6 @@ vchiq_bulk_transfer(struct vchiq_instance *instance, unsigned int handle, void *
> extern int
> vchiq_dump_state(void *dump_context, struct vchiq_state *state);
>
> -extern int
> -vchiq_dump_service_state(void *dump_context, struct vchiq_service *service);
> -
> extern void
> vchiq_loud_error_header(void);
>
@@ -3428,90 +3428,8 @@ vchiq_dump_shared_state(void *dump_context, struct vchiq_state *state,
return 0;
}
-int vchiq_dump_state(void *dump_context, struct vchiq_state *state)
-{
- char buf[80];
- int len;
- int i;
- int err;
-
- len = scnprintf(buf, sizeof(buf), "State %d: %s", state->id,
- conn_state_names[state->conn_state]);
- err = vchiq_dump(dump_context, buf, len + 1);
- if (err)
- return err;
-
- len = scnprintf(buf, sizeof(buf), " tx_pos=%x(@%pK), rx_pos=%x(@%pK)",
- state->local->tx_pos,
- state->tx_data + (state->local_tx_pos & VCHIQ_SLOT_MASK),
- state->rx_pos,
- state->rx_data + (state->rx_pos & VCHIQ_SLOT_MASK));
- err = vchiq_dump(dump_context, buf, len + 1);
- if (err)
- return err;
-
- len = scnprintf(buf, sizeof(buf), " Version: %d (min %d)",
- VCHIQ_VERSION, VCHIQ_VERSION_MIN);
- err = vchiq_dump(dump_context, buf, len + 1);
- if (err)
- return err;
-
- if (VCHIQ_ENABLE_STATS) {
- len = scnprintf(buf, sizeof(buf),
- " Stats: ctrl_tx_count=%d, ctrl_rx_count=%d, error_count=%d",
- state->stats.ctrl_tx_count, state->stats.ctrl_rx_count,
- state->stats.error_count);
- err = vchiq_dump(dump_context, buf, len + 1);
- if (err)
- return err;
- }
-
- len = scnprintf(buf, sizeof(buf),
- " Slots: %d available (%d data), %d recyclable, %d stalls (%d data)",
- ((state->slot_queue_available * VCHIQ_SLOT_SIZE) -
- state->local_tx_pos) / VCHIQ_SLOT_SIZE,
- state->data_quota - state->data_use_count,
- state->local->slot_queue_recycle - state->slot_queue_available,
- state->stats.slot_stalls, state->stats.data_stalls);
- err = vchiq_dump(dump_context, buf, len + 1);
- if (err)
- return err;
-
- err = vchiq_dump_platform_state(dump_context);
- if (err)
- return err;
-
- err = vchiq_dump_shared_state(dump_context,
- state,
- state->local,
- "Local");
- if (err)
- return err;
- err = vchiq_dump_shared_state(dump_context,
- state,
- state->remote,
- "Remote");
- if (err)
- return err;
-
- err = vchiq_dump_platform_instances(dump_context);
- if (err)
- return err;
-
- for (i = 0; i < state->unused_service; i++) {
- struct vchiq_service *service = find_service_by_port(state, i);
-
- if (service) {
- err = vchiq_dump_service_state(dump_context, service);
- vchiq_service_put(service);
- if (err)
- return err;
- }
- }
- return 0;
-}
-
-int vchiq_dump_service_state(void *dump_context, struct vchiq_service *service)
+static int
+vchiq_dump_service_state(void *dump_context, struct vchiq_service *service)
{
char buf[80];
int len;
@@ -3606,6 +3524,89 @@ int vchiq_dump_service_state(void *dump_context, struct vchiq_service *service)
return err;
}
+int vchiq_dump_state(void *dump_context, struct vchiq_state *state)
+{
+ char buf[80];
+ int len;
+ int i;
+ int err;
+
+ len = scnprintf(buf, sizeof(buf), "State %d: %s", state->id,
+ conn_state_names[state->conn_state]);
+ err = vchiq_dump(dump_context, buf, len + 1);
+ if (err)
+ return err;
+
+ len = scnprintf(buf, sizeof(buf), " tx_pos=%x(@%pK), rx_pos=%x(@%pK)",
+ state->local->tx_pos,
+ state->tx_data + (state->local_tx_pos & VCHIQ_SLOT_MASK),
+ state->rx_pos,
+ state->rx_data + (state->rx_pos & VCHIQ_SLOT_MASK));
+ err = vchiq_dump(dump_context, buf, len + 1);
+ if (err)
+ return err;
+
+ len = scnprintf(buf, sizeof(buf), " Version: %d (min %d)",
+ VCHIQ_VERSION, VCHIQ_VERSION_MIN);
+ err = vchiq_dump(dump_context, buf, len + 1);
+ if (err)
+ return err;
+
+ if (VCHIQ_ENABLE_STATS) {
+ len = scnprintf(buf, sizeof(buf),
+ " Stats: ctrl_tx_count=%d, ctrl_rx_count=%d, error_count=%d",
+ state->stats.ctrl_tx_count, state->stats.ctrl_rx_count,
+ state->stats.error_count);
+ err = vchiq_dump(dump_context, buf, len + 1);
+ if (err)
+ return err;
+ }
+
+ len = scnprintf(buf, sizeof(buf),
+ " Slots: %d available (%d data), %d recyclable, %d stalls (%d data)",
+ ((state->slot_queue_available * VCHIQ_SLOT_SIZE) -
+ state->local_tx_pos) / VCHIQ_SLOT_SIZE,
+ state->data_quota - state->data_use_count,
+ state->local->slot_queue_recycle - state->slot_queue_available,
+ state->stats.slot_stalls, state->stats.data_stalls);
+ err = vchiq_dump(dump_context, buf, len + 1);
+ if (err)
+ return err;
+
+ err = vchiq_dump_platform_state(dump_context);
+ if (err)
+ return err;
+
+ err = vchiq_dump_shared_state(dump_context,
+ state,
+ state->local,
+ "Local");
+ if (err)
+ return err;
+ err = vchiq_dump_shared_state(dump_context,
+ state,
+ state->remote,
+ "Remote");
+ if (err)
+ return err;
+
+ err = vchiq_dump_platform_instances(dump_context);
+ if (err)
+ return err;
+
+ for (i = 0; i < state->unused_service; i++) {
+ struct vchiq_service *service = find_service_by_port(state, i);
+
+ if (service) {
+ err = vchiq_dump_service_state(dump_context, service);
+ vchiq_service_put(service);
+ if (err)
+ return err;
+ }
+ }
+ return 0;
+}
+
int vchiq_send_remote_use(struct vchiq_state *state)
{
if (state->conn_state == VCHIQ_CONNSTATE_DISCONNECTED)
@@ -507,9 +507,6 @@ vchiq_bulk_transfer(struct vchiq_instance *instance, unsigned int handle, void *
extern int
vchiq_dump_state(void *dump_context, struct vchiq_state *state);
-extern int
-vchiq_dump_service_state(void *dump_context, struct vchiq_service *service);
-
extern void
vchiq_loud_error_header(void);