* [PATCH] net: sxgbe: make local functions static
@ 2014-04-17 10:13 Jingoo Han
2014-04-18 11:35 ` Byungho An
2014-04-20 22:20 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Jingoo Han @ 2014-04-17 10:13 UTC (permalink / raw)
To: netdev, 'David Miller'
Cc: 'Byungho An', 'Girish K S',
'Siva Reddy Kallam', 'Vipul Pandya',
'Jingoo Han'
Make local functions static, because these are used only in this
file.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
index a72688e..dbb0abb 100644
--- a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
+++ b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
@@ -425,8 +425,8 @@ dmamem_err:
* @rx_rsize: ring size
* Description: this function initializes the DMA RX descriptor
*/
-void free_rx_ring(struct device *dev, struct sxgbe_rx_queue *rx_ring,
- int rx_rsize)
+static void free_rx_ring(struct device *dev, struct sxgbe_rx_queue *rx_ring,
+ int rx_rsize)
{
dma_free_coherent(dev, rx_rsize * sizeof(struct sxgbe_rx_norm_desc),
rx_ring->dma_rx, rx_ring->dma_rx_phy);
@@ -519,8 +519,8 @@ error:
* @tx_rsize: ring size
* Description: this function initializes the DMA TX descriptor
*/
-void free_tx_ring(struct device *dev, struct sxgbe_tx_queue *tx_ring,
- int tx_rsize)
+static void free_tx_ring(struct device *dev, struct sxgbe_tx_queue *tx_ring,
+ int tx_rsize)
{
dma_free_coherent(dev, tx_rsize * sizeof(struct sxgbe_tx_norm_desc),
tx_ring->dma_tx, tx_ring->dma_tx_phy);
@@ -1218,11 +1218,10 @@ static int sxgbe_release(struct net_device *dev)
return 0;
}
-
/* Prepare first Tx descriptor for doing TSO operation */
-void sxgbe_tso_prepare(struct sxgbe_priv_data *priv,
- struct sxgbe_tx_norm_desc *first_desc,
- struct sk_buff *skb)
+static void sxgbe_tso_prepare(struct sxgbe_priv_data *priv,
+ struct sxgbe_tx_norm_desc *first_desc,
+ struct sk_buff *skb)
{
unsigned int total_hdr_len, tcp_hdr_len;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH] net: sxgbe: make local functions static
2014-04-17 10:13 [PATCH] net: sxgbe: make local functions static Jingoo Han
@ 2014-04-18 11:35 ` Byungho An
2014-04-20 22:20 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Byungho An @ 2014-04-18 11:35 UTC (permalink / raw)
To: 'Jingoo Han', netdev, 'David Miller'
Cc: 'Girish K S', 'Siva Reddy Kallam',
'Vipul Pandya'
Jingoo Han wrote:
>
> Make local functions static, because these are used only in this
> file.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Byungho An <bh74.an@samsung.com>
> ---
> drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c | 15 +++++++--------
> 1 file changed, 7 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
> b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
> index a72688e..dbb0abb 100644
> --- a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
> +++ b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
> @@ -425,8 +425,8 @@ dmamem_err:
> * @rx_rsize: ring size
> * Description: this function initializes the DMA RX descriptor
> */
> -void free_rx_ring(struct device *dev, struct sxgbe_rx_queue *rx_ring,
> - int rx_rsize)
> +static void free_rx_ring(struct device *dev, struct sxgbe_rx_queue
> *rx_ring,
> + int rx_rsize)
> {
> dma_free_coherent(dev, rx_rsize * sizeof(struct sxgbe_rx_norm_desc),
> rx_ring->dma_rx, rx_ring->dma_rx_phy);
> @@ -519,8 +519,8 @@ error:
> * @tx_rsize: ring size
> * Description: this function initializes the DMA TX descriptor
> */
> -void free_tx_ring(struct device *dev, struct sxgbe_tx_queue *tx_ring,
> - int tx_rsize)
> +static void free_tx_ring(struct device *dev, struct sxgbe_tx_queue
> *tx_ring,
> + int tx_rsize)
> {
> dma_free_coherent(dev, tx_rsize * sizeof(struct sxgbe_tx_norm_desc),
> tx_ring->dma_tx, tx_ring->dma_tx_phy);
> @@ -1218,11 +1218,10 @@ static int sxgbe_release(struct net_device *dev)
>
> return 0;
> }
> -
> /* Prepare first Tx descriptor for doing TSO operation */
> -void sxgbe_tso_prepare(struct sxgbe_priv_data *priv,
> - struct sxgbe_tx_norm_desc *first_desc,
> - struct sk_buff *skb)
> +static void sxgbe_tso_prepare(struct sxgbe_priv_data *priv,
> + struct sxgbe_tx_norm_desc *first_desc,
> + struct sk_buff *skb)
> {
> unsigned int total_hdr_len, tcp_hdr_len;
>
> --
> 1.7.10.4
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] net: sxgbe: make local functions static
2014-04-17 10:13 [PATCH] net: sxgbe: make local functions static Jingoo Han
2014-04-18 11:35 ` Byungho An
@ 2014-04-20 22:20 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-04-20 22:20 UTC (permalink / raw)
To: jg1.han; +Cc: netdev, bh74.an, ks.giri, siva.kallam, vipul.pandya
From: Jingoo Han <jg1.han@samsung.com>
Date: Thu, 17 Apr 2014 19:13:53 +0900
> Make local functions static, because these are used only in this
> file.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-04-20 22:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-17 10:13 [PATCH] net: sxgbe: make local functions static Jingoo Han
2014-04-18 11:35 ` Byungho An
2014-04-20 22:20 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).