We get 2 warnings when building kernel with W=1: drivers/staging/ks7010/ks7010_sdio.c:152:6: warning: no previous prototype for 'ks_wlan_hw_wakeup_request' [-Wmissing-prototypes] drivers/staging/ks7010/ks7010_sdio.c:255:5: warning: no previous prototype for 'ks_wlan_hw_power_save' [-Wmissing-prototypes] In fact, both functions are declared in drivers/staging/ks7010/ks_hostif.c, but should be declared in a header file, thus can be recognized in other file. So this patch adds the declarations into drivers/staging/ks7010/ks_wlan.h. Signed-off-by: Baoyou Xie <baoyou.xie@xxxxxxxxxx> --- drivers/staging/ks7010/ks_hostif.c | 3 --- drivers/staging/ks7010/ks_wlan.h | 2 ++ 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index aacb93e..49a81d2 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -25,9 +25,6 @@ extern int ks_wlan_hw_tx(struct ks_wlan_private *priv, void *p, void (*complete_handler) (void *arg1, void *arg2), void *arg1, void *arg2); -extern void ks_wlan_hw_wakeup_request(struct ks_wlan_private *priv); -extern int ks_wlan_hw_power_save(struct ks_wlan_private *priv); - /* macro */ #define inc_smeqhead(priv) \ ( priv->sme_i.qhead = (priv->sme_i.qhead + 1) % SME_EVENT_BUFF_SIZE ) diff --git a/drivers/staging/ks7010/ks_wlan.h b/drivers/staging/ks7010/ks_wlan.h index 936222b..54a1531 100644 --- a/drivers/staging/ks7010/ks_wlan.h +++ b/drivers/staging/ks7010/ks_wlan.h @@ -501,6 +501,8 @@ struct ks_wlan_private { extern int ks_wlan_net_start(struct net_device *dev); extern int ks_wlan_net_stop(struct net_device *dev); +void ks_wlan_hw_wakeup_request(struct ks_wlan_private *priv); +int ks_wlan_hw_power_save(struct ks_wlan_private *priv); void send_packet_complete(void *, void *); -- 2.7.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel