On Tuesday 28 February 2017 07:21 PM, Sumantro wrote:
Fixed Warning rasied by checkpatch.pl

Hi,

Usually we prefer to have subject stating what change is done
and commit log stating why that change is done instead of
stating tool name. Though you can give credit to the tool
in the commit log after stating the reason behind the change.

Also, subject of a patch follows particular structure. You can
check that with 'git log --pretty=oneline --abbrev-commit
<file_name>'.

May be you can check this[1] for a reference.

Thanks!

[1] https://kernelnewbies.org/PatchPhilosophy


Signed-off-by: Sumantro <saggy.z...@gmail.com>
---
 drivers/staging/rtl8192e/rtl8192e/rtl_wx.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c 
b/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c
index 8ffb458..78eb871 100644
--- a/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c
+++ b/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c
@@ -475,12 +475,10 @@ static int _rtl92e_wx_set_scan(struct net_device *dev,
        return ret;
 }

-
 static int _rtl92e_wx_get_scan(struct net_device *dev,
                               struct iw_request_info *a,
                               union iwreq_data *wrqu, char *b)
 {
-
        int ret;
        struct r8192_priv *priv = rtllib_priv(dev);

@@ -654,7 +652,6 @@ static int _rtl92e_wx_set_wap(struct net_device *dev,

 }

-
 static int _rtl92e_wx_get_wap(struct net_device *dev,
                              struct iw_request_info *info,
                              union iwreq_data *wrqu, char *extra)
@@ -664,7 +661,6 @@ static int _rtl92e_wx_get_wap(struct net_device *dev,
        return rtllib_wx_get_wap(priv->rtllib, info, wrqu, extra);
 }

-
 static int _rtl92e_wx_get_enc(struct net_device *dev,
                              struct iw_request_info *info,
                              union iwreq_data *wrqu, char *key)
@@ -831,7 +827,6 @@ static int _rtl92e_wx_get_retry(struct net_device *dev,
 {
        struct r8192_priv *priv = rtllib_priv(dev);

-
        wrqu->retry.disabled = 0; /* can't be disabled */

        if ((wrqu->retry.flags & IW_RETRY_TYPE) ==
@@ -969,7 +964,6 @@ static int _rtl92e_wx_set_encode_ext(struct net_device *dev,
        priv->rtllib->wx_set_enc = 0;
        mutex_unlock(&priv->wx_mutex);
        return ret;
-
 }

 static int _rtl92e_wx_set_auth(struct net_device *dev,


_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to