net: hisilicon: mark symbols static where possible
authorBaoyou Xie <baoyou.xie@linaro.org>
Sun, 25 Sep 2016 09:16:44 +0000 (17:16 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Sep 2016 06:23:39 +0000 (02:23 -0400)
We get 2 warnings when building kernel with W=1:
drivers/net/ethernet/hisilicon/hisi_femac.c:943:5: warning: no previous prototype for 'hisi_femac_drv_suspend' [-Wmissing-prototypes]
drivers/net/ethernet/hisilicon/hisi_femac.c:960:5: warning: no previous prototype for 'hisi_femac_drv_resume' [-Wmissing-prototypes]

In fact, these two functions are only used in the file in which they are
declared and don't need a declaration, but can be made static.
so this patch marks these functions with 'static'.

Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hisi_femac.c

index ca68e22..ced1859 100644 (file)
@@ -940,8 +940,8 @@ static int hisi_femac_drv_remove(struct platform_device *pdev)
 }
 
 #ifdef CONFIG_PM
-int hisi_femac_drv_suspend(struct platform_device *pdev,
-                          pm_message_t state)
+static int hisi_femac_drv_suspend(struct platform_device *pdev,
+                                 pm_message_t state)
 {
        struct net_device *ndev = platform_get_drvdata(pdev);
        struct hisi_femac_priv *priv = netdev_priv(ndev);
@@ -957,7 +957,7 @@ int hisi_femac_drv_suspend(struct platform_device *pdev,
        return 0;
 }
 
-int hisi_femac_drv_resume(struct platform_device *pdev)
+static int hisi_femac_drv_resume(struct platform_device *pdev)
 {
        struct net_device *ndev = platform_get_drvdata(pdev);
        struct hisi_femac_priv *priv = netdev_priv(ndev);