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
This commit is contained in:
@@ -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
|
* Permission to use, copy, modify, and/or distribute this software for
|
||||||
* any purpose with or without fee is hereby granted, provided that the
|
* any purpose with or without fee is hereby granted, provided that the
|
||||||
@@ -77,7 +77,7 @@
|
|||||||
(uint32_t)(((dma_addr) >> 32) & 0xFF);\
|
(uint32_t)(((dma_addr) >> 32) & 0xFF);\
|
||||||
} while (0)
|
} while (0)
|
||||||
|
|
||||||
int
|
static int
|
||||||
ce_send_nolock_srng(struct CE_handle *copyeng,
|
ce_send_nolock_srng(struct CE_handle *copyeng,
|
||||||
void *per_transfer_context,
|
void *per_transfer_context,
|
||||||
qdf_dma_addr_t buffer,
|
qdf_dma_addr_t buffer,
|
||||||
@@ -154,7 +154,7 @@ ce_send_nolock_srng(struct CE_handle *copyeng,
|
|||||||
return status;
|
return status;
|
||||||
}
|
}
|
||||||
|
|
||||||
int
|
static int
|
||||||
ce_sendlist_send_srng(struct CE_handle *copyeng,
|
ce_sendlist_send_srng(struct CE_handle *copyeng,
|
||||||
void *per_transfer_context,
|
void *per_transfer_context,
|
||||||
struct ce_sendlist *sendlist, unsigned int transfer_id)
|
struct ce_sendlist *sendlist, unsigned int transfer_id)
|
||||||
@@ -234,7 +234,7 @@ ce_sendlist_send_srng(struct CE_handle *copyeng,
|
|||||||
*
|
*
|
||||||
* Return: 0 if the buffer is enqueued
|
* Return: 0 if the buffer is enqueued
|
||||||
*/
|
*/
|
||||||
int
|
static int
|
||||||
ce_recv_buf_enqueue_srng(struct CE_handle *copyeng,
|
ce_recv_buf_enqueue_srng(struct CE_handle *copyeng,
|
||||||
void *per_recv_context, qdf_dma_addr_t buffer)
|
void *per_recv_context, qdf_dma_addr_t buffer)
|
||||||
{
|
{
|
||||||
@@ -295,7 +295,7 @@ ce_recv_buf_enqueue_srng(struct CE_handle *copyeng,
|
|||||||
* Guts of ce_recv_entries_done.
|
* Guts of ce_recv_entries_done.
|
||||||
* The caller takes responsibility for any necessary locking.
|
* The caller takes responsibility for any necessary locking.
|
||||||
*/
|
*/
|
||||||
unsigned int
|
static unsigned int
|
||||||
ce_recv_entries_done_nolock_srng(struct hif_softc *scn,
|
ce_recv_entries_done_nolock_srng(struct hif_softc *scn,
|
||||||
struct CE_state *CE_state)
|
struct CE_state *CE_state)
|
||||||
{
|
{
|
||||||
@@ -309,7 +309,7 @@ ce_recv_entries_done_nolock_srng(struct hif_softc *scn,
|
|||||||
* Guts of ce_send_entries_done.
|
* Guts of ce_send_entries_done.
|
||||||
* The caller takes responsibility for any necessary locking.
|
* The caller takes responsibility for any necessary locking.
|
||||||
*/
|
*/
|
||||||
unsigned int
|
static unsigned int
|
||||||
ce_send_entries_done_nolock_srng(struct hif_softc *scn,
|
ce_send_entries_done_nolock_srng(struct hif_softc *scn,
|
||||||
struct CE_state *CE_state)
|
struct CE_state *CE_state)
|
||||||
{
|
{
|
||||||
@@ -335,7 +335,7 @@ void *ce_debug_cmplsn_context_srng; /* completed send next context */
|
|||||||
* Guts of ce_completed_recv_next.
|
* Guts of ce_completed_recv_next.
|
||||||
* The caller takes responsibility for any necessary locking.
|
* The caller takes responsibility for any necessary locking.
|
||||||
*/
|
*/
|
||||||
int
|
static int
|
||||||
ce_completed_recv_next_nolock_srng(struct CE_state *CE_state,
|
ce_completed_recv_next_nolock_srng(struct CE_state *CE_state,
|
||||||
void **per_CE_contextp,
|
void **per_CE_contextp,
|
||||||
void **per_transfer_contextp,
|
void **per_transfer_contextp,
|
||||||
@@ -415,7 +415,7 @@ done:
|
|||||||
return status;
|
return status;
|
||||||
}
|
}
|
||||||
|
|
||||||
QDF_STATUS
|
static QDF_STATUS
|
||||||
ce_revoke_recv_next_srng(struct CE_handle *copyeng,
|
ce_revoke_recv_next_srng(struct CE_handle *copyeng,
|
||||||
void **per_CE_contextp,
|
void **per_CE_contextp,
|
||||||
void **per_transfer_contextp, qdf_dma_addr_t *bufferp)
|
void **per_transfer_contextp, qdf_dma_addr_t *bufferp)
|
||||||
@@ -429,7 +429,7 @@ ce_revoke_recv_next_srng(struct CE_handle *copyeng,
|
|||||||
* Guts of ce_completed_send_next.
|
* Guts of ce_completed_send_next.
|
||||||
* The caller takes responsibility for any necessary locking.
|
* The caller takes responsibility for any necessary locking.
|
||||||
*/
|
*/
|
||||||
int
|
static int
|
||||||
ce_completed_send_next_nolock_srng(struct CE_state *CE_state,
|
ce_completed_send_next_nolock_srng(struct CE_state *CE_state,
|
||||||
void **per_CE_contextp,
|
void **per_CE_contextp,
|
||||||
void **per_transfer_contextp,
|
void **per_transfer_contextp,
|
||||||
@@ -486,7 +486,7 @@ ce_completed_send_next_nolock_srng(struct CE_state *CE_state,
|
|||||||
}
|
}
|
||||||
|
|
||||||
/* NB: Modelled after ce_completed_send_next */
|
/* NB: Modelled after ce_completed_send_next */
|
||||||
QDF_STATUS
|
static QDF_STATUS
|
||||||
ce_cancel_send_next_srng(struct CE_handle *copyeng,
|
ce_cancel_send_next_srng(struct CE_handle *copyeng,
|
||||||
void **per_CE_contextp,
|
void **per_CE_contextp,
|
||||||
void **per_transfer_contextp,
|
void **per_transfer_contextp,
|
||||||
@@ -524,13 +524,14 @@ ce_per_engine_handler_adjust_srng(struct CE_state *CE_state,
|
|||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
|
||||||
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*/
|
/*TODO*/
|
||||||
return false;
|
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) {
|
switch (ring_type) {
|
||||||
case CE_RING_SRC:
|
case CE_RING_SRC:
|
||||||
@@ -577,7 +578,7 @@ static void ce_srng_msi_ring_params_setup(struct hif_softc *scn, uint32_t ce_id,
|
|||||||
(void *)ring_params->msi_addr, ring_params->msi_data);
|
(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 CE_ring_state *src_ring)
|
||||||
{
|
{
|
||||||
struct hal_srng_params ring_params = {0};
|
struct hal_srng_params ring_params = {0};
|
||||||
@@ -608,7 +609,7 @@ void ce_srng_src_ring_setup(struct hif_softc *scn, uint32_t ce_id,
|
|||||||
&ring_params);
|
&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_ring_state *dest_ring,
|
||||||
struct CE_attr *attr)
|
struct CE_attr *attr)
|
||||||
{
|
{
|
||||||
@@ -636,7 +637,7 @@ void ce_srng_dest_ring_setup(struct hif_softc *scn, uint32_t ce_id,
|
|||||||
&ring_params);
|
&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 CE_ring_state *status_ring)
|
||||||
{
|
{
|
||||||
struct hal_srng_params ring_params = {0};
|
struct hal_srng_params ring_params = {0};
|
||||||
@@ -661,7 +662,7 @@ void ce_srng_status_ring_setup(struct hif_softc *scn, uint32_t ce_id,
|
|||||||
ce_id, 0, &ring_params);
|
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,
|
uint32_t ce_id, struct CE_ring_state *ring,
|
||||||
struct CE_attr *attr)
|
struct CE_attr *attr)
|
||||||
{
|
{
|
||||||
@@ -680,7 +681,8 @@ void ce_ring_setup_srng(struct hif_softc *scn, uint8_t ring_type,
|
|||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
struct ce_ops ce_service_srng = {
|
|
||||||
|
static struct ce_ops ce_service_srng = {
|
||||||
.ce_get_desc_size = ce_get_desc_size_srng,
|
.ce_get_desc_size = ce_get_desc_size_srng,
|
||||||
.ce_ring_setup = ce_ring_setup_srng,
|
.ce_ring_setup = ce_ring_setup_srng,
|
||||||
.ce_sendlist_send = ce_sendlist_send_srng,
|
.ce_sendlist_send = ce_sendlist_send_srng,
|
||||||
|
|||||||
Reference in New Issue
Block a user