qcacmn: Fix -Wmissing-prototypes in ce_service_srng.c
We want to enable the compiler's -Wmissing-prototypes switch, but
there is existing code in ce_service_srng.c that is generating
warnings. Fix these warnings by making the following functions
static:
ce_send_nolock_srng
ce_sendlist_send_srng
ce_recv_buf_enqueue_srng
ce_recv_entries_done_nolock_srng
ce_send_entries_done_nolock_srng
ce_completed_recv_next_nolock_srng
ce_revoke_recv_next_srng
ce_completed_send_next_nolock_srng
ce_cancel_send_next_srng
ce_check_int_watermark_srng
ce_get_desc_size_srng
ce_srng_src_ring_setup
ce_srng_dest_ring_setup
ce_srng_status_ring_setup
ce_ring_setup_srng
Change-Id: Iea59d7b65179ad2a0c6a45af6acf94663b41304c
CRs-Fixed: 1104119
diff --git a/hif/src/ce/ce_service_srng.c b/hif/src/ce/ce_service_srng.c
index 1ef125c..3839ab9 100644
--- a/hif/src/ce/ce_service_srng.c
+++ b/hif/src/ce/ce_service_srng.c
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2016 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2016-2017 The Linux Foundation. All rights reserved.
*
* Permission to use, copy, modify, and/or distribute this software for
* any purpose with or without fee is hereby granted, provided that the
@@ -77,7 +77,7 @@
(uint32_t)(((dma_addr) >> 32) & 0xFF);\
} while (0)
-int
+static int
ce_send_nolock_srng(struct CE_handle *copyeng,
void *per_transfer_context,
qdf_dma_addr_t buffer,
@@ -154,7 +154,7 @@
return status;
}
-int
+static int
ce_sendlist_send_srng(struct CE_handle *copyeng,
void *per_transfer_context,
struct ce_sendlist *sendlist, unsigned int transfer_id)
@@ -234,7 +234,7 @@
*
* Return: 0 if the buffer is enqueued
*/
-int
+static int
ce_recv_buf_enqueue_srng(struct CE_handle *copyeng,
void *per_recv_context, qdf_dma_addr_t buffer)
{
@@ -295,7 +295,7 @@
* Guts of ce_recv_entries_done.
* The caller takes responsibility for any necessary locking.
*/
-unsigned int
+static unsigned int
ce_recv_entries_done_nolock_srng(struct hif_softc *scn,
struct CE_state *CE_state)
{
@@ -309,7 +309,7 @@
* Guts of ce_send_entries_done.
* The caller takes responsibility for any necessary locking.
*/
-unsigned int
+static unsigned int
ce_send_entries_done_nolock_srng(struct hif_softc *scn,
struct CE_state *CE_state)
{
@@ -335,7 +335,7 @@
* Guts of ce_completed_recv_next.
* The caller takes responsibility for any necessary locking.
*/
-int
+static int
ce_completed_recv_next_nolock_srng(struct CE_state *CE_state,
void **per_CE_contextp,
void **per_transfer_contextp,
@@ -415,7 +415,7 @@
return status;
}
-QDF_STATUS
+static QDF_STATUS
ce_revoke_recv_next_srng(struct CE_handle *copyeng,
void **per_CE_contextp,
void **per_transfer_contextp, qdf_dma_addr_t *bufferp)
@@ -429,7 +429,7 @@
* Guts of ce_completed_send_next.
* The caller takes responsibility for any necessary locking.
*/
-int
+static int
ce_completed_send_next_nolock_srng(struct CE_state *CE_state,
void **per_CE_contextp,
void **per_transfer_contextp,
@@ -486,7 +486,7 @@
}
/* NB: Modelled after ce_completed_send_next */
-QDF_STATUS
+static QDF_STATUS
ce_cancel_send_next_srng(struct CE_handle *copyeng,
void **per_CE_contextp,
void **per_transfer_contextp,
@@ -524,13 +524,14 @@
{
}
-bool ce_check_int_watermark_srng(struct CE_state *CE_state, unsigned int *flags)
+static bool ce_check_int_watermark_srng(struct CE_state *CE_state,
+ unsigned int *flags)
{
/*TODO*/
return false;
}
-uint32_t ce_get_desc_size_srng(uint8_t ring_type)
+static uint32_t ce_get_desc_size_srng(uint8_t ring_type)
{
switch (ring_type) {
case CE_RING_SRC:
@@ -577,7 +578,7 @@
(void *)ring_params->msi_addr, ring_params->msi_data);
}
-void ce_srng_src_ring_setup(struct hif_softc *scn, uint32_t ce_id,
+static void ce_srng_src_ring_setup(struct hif_softc *scn, uint32_t ce_id,
struct CE_ring_state *src_ring)
{
struct hal_srng_params ring_params = {0};
@@ -608,7 +609,7 @@
&ring_params);
}
-void ce_srng_dest_ring_setup(struct hif_softc *scn, uint32_t ce_id,
+static void ce_srng_dest_ring_setup(struct hif_softc *scn, uint32_t ce_id,
struct CE_ring_state *dest_ring,
struct CE_attr *attr)
{
@@ -636,7 +637,7 @@
&ring_params);
}
-void ce_srng_status_ring_setup(struct hif_softc *scn, uint32_t ce_id,
+static void ce_srng_status_ring_setup(struct hif_softc *scn, uint32_t ce_id,
struct CE_ring_state *status_ring)
{
struct hal_srng_params ring_params = {0};
@@ -661,7 +662,7 @@
ce_id, 0, &ring_params);
}
-void ce_ring_setup_srng(struct hif_softc *scn, uint8_t ring_type,
+static void ce_ring_setup_srng(struct hif_softc *scn, uint8_t ring_type,
uint32_t ce_id, struct CE_ring_state *ring,
struct CE_attr *attr)
{
@@ -680,7 +681,8 @@
break;
}
}
-struct ce_ops ce_service_srng = {
+
+static struct ce_ops ce_service_srng = {
.ce_get_desc_size = ce_get_desc_size_srng,
.ce_ring_setup = ce_ring_setup_srng,
.ce_sendlist_send = ce_sendlist_send_srng,