lkml.org 
[lkml]   [2017]   [Apr]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2] Staging: ks7010: style fix
    Date
    Fix the style of block comments.
    Move trailing */ to a separate line.
    Checkpatch was showing:
    WARNING: Block comments use a trailing */ on a separate line.

    Signed-off-by: Ilia Sergachev <ilia.sergachev@unibas.ch>
    ---
    Changes for v2:
    - improve the commit message

    drivers/staging/ks7010/ks_wlan_net.c | 43 ++++++++++++++++++++++--------------
    1 file changed, 27 insertions(+), 16 deletions(-)

    diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c
    index 121e1530fdba..7f6bd4d53d84 100644
    --- a/drivers/staging/ks7010/ks_wlan_net.c
    +++ b/drivers/staging/ks7010/ks_wlan_net.c
    @@ -230,7 +230,8 @@ static int ks_wlan_set_freq(struct net_device *dev,
    else {
    int channel = fwrq->m;
    /* We should do a better check than that,
    - * based on the card capability !!! */
    + * based on the card capability !!!
    + */
    if ((channel < 1) || (channel > 14)) {
    netdev_dbg(dev,
    "%s: New channel value of %d is invalid!\n",
    @@ -343,7 +344,8 @@ static int ks_wlan_get_essid(struct net_device *dev,

    /* for SLEEP MODE */
    /* Note : if dwrq->flags != 0, we should
    - * get the relevant SSID from the SSID list... */
    + * get the relevant SSID from the SSID list...
    + */
    if (priv->reg.ssid.size) {
    /* Get the current SSID */
    memcpy(extra, priv->reg.ssid.body, priv->reg.ssid.size);
    @@ -1137,7 +1139,8 @@ static int ks_wlan_get_range(struct net_device *dev,
    range->max_nwid = 0x0000;
    range->num_channels = 14;
    /* Should be based on cap_rid.country to give only
    - * what the current card support */
    + * what the current card support
    + */
    k = 0;
    for (i = 0; i < 13; i++) { /* channel 1 -- 13 */
    range->freq[k].i = i + 1; /* List index */
    @@ -1189,7 +1192,8 @@ static int ks_wlan_get_range(struct net_device *dev,

    /* Set an indication of the max TCP throughput
    * in bit/s that we can expect using this interface.
    - * May be use for QoS stuff... Jean II */
    + * May be use for QoS stuff... Jean II
    + */
    if (i > 2)
    range->throughput = 5000 * 1000;
    else
    @@ -1225,7 +1229,8 @@ static int ks_wlan_get_range(struct net_device *dev,

    /* Experimental measurements - boundary 11/5.5 Mb/s */
    /* Note : with or without the (local->rssi), results
    - * are somewhat different. - Jean II */
    + * are somewhat different. - Jean II
    + */
    range->avg_qual.qual = 50;
    range->avg_qual.level = 186; /* -70 dBm */
    range->avg_qual.noise = 0;
    @@ -1497,7 +1502,8 @@ static inline char *ks_wlan_translate_scan(struct net_device *dev,
    &(ap->ssid.body[0]));

    /* Rate : stuffing multiple values in a single event require a bit
    - * more of magic - Jean II */
    + * more of magic - Jean II
    + */
    current_val = current_ev + IW_EV_LCP_LEN;

    iwe.cmd = SIOCGIWRATE;
    @@ -1569,7 +1575,8 @@ static inline char *ks_wlan_translate_scan(struct net_device *dev,
    }

    /* The other data in the scan result are not really
    - * interesting, so for now drop it - Jean II */
    + * interesting, so for now drop it - Jean II
    + */
    return current_ev;
    }

    @@ -1596,7 +1603,8 @@ static int ks_wlan_get_scan(struct net_device *dev,

    if (priv->aplist.size == 0) {
    /* Client error, no scan results...
    - * The caller need to restart the scan. */
    + * The caller need to restart the scan.
    + */
    DPRINTK(2, "aplist 0\n");
    return -ENODATA;
    }
    @@ -1970,12 +1978,13 @@ static int ks_wlan_get_encode_ext(struct net_device *dev,
    return -EPERM;

    /* for SLEEP MODE */
    - /* WPA (not used ?? wpa_supplicant)
    - struct ks_wlan_private *priv = (struct ks_wlan_private *)dev->priv;
    - struct iw_encode_ext *enc;
    - enc = (struct iw_encode_ext *)extra;
    - int index = dwrq->flags & IW_ENCODE_INDEX;
    - WPA (not used ?? wpa_supplicant) */
    + /* WPA (not used ?? wpa_supplicant)
    + * struct ks_wlan_private *priv = (struct ks_wlan_private *)dev->priv;
    + * struct iw_encode_ext *enc;
    + * enc = (struct iw_encode_ext *)extra;
    + * int index = dwrq->flags & IW_ENCODE_INDEX;
    + * WPA (not used ?? wpa_supplicant)
    + */
    return 0;
    }

    @@ -2105,7 +2114,8 @@ static struct iw_statistics *ks_get_wireless_stats(struct net_device *dev)
    }

    /* Packets discarded in the wireless adapter due to wireless
    - * specific problems */
    + * specific problems
    + */
    wstats->discard.nwid = 0; /* Rx invalid nwid */
    wstats->discard.code = 0; /* Rx invalid crypt */
    wstats->discard.fragment = 0; /* Rx invalid frag */
    @@ -3351,7 +3361,8 @@ void send_packet_complete(void *arg1, void *arg2)
    }

    /* Set or clear the multicast filter for this adaptor.
    - This routine is not state sensitive and need not be SMP locked. */
    + * This routine is not state sensitive and need not be SMP locked.
    + */
    static
    void ks_wlan_set_multicast_list(struct net_device *dev)
    {
    --
    2.11.0
    \
     
     \ /
      Last update: 2017-04-26 10:40    [W:4.342 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site