On 09.05.2018 21:36, Ajay Singh wrote:
> On Wed, 9 May 2018 16:44:47 +0300
> Claudiu Beznea <claudiu.bez...@microchip.com> wrote:
> 
>> On 07.05.2018 11:43, Ajay Singh wrote:
>>> Fix checkpatch reported issue of line over 80 char in handle_key().
>>> Introduced new functions by spliting existing function to address
>>> the checkpatch issue.
>>>
>>> Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
>>> ---
>>>  drivers/staging/wilc1000/host_interface.c | 59
>>> +++++++++++++++++++------------ 1 file changed, 37 insertions(+),
>>> 22 deletions(-)
>>>
>>> diff --git a/drivers/staging/wilc1000/host_interface.c
>>> b/drivers/staging/wilc1000/host_interface.c index 4ba868c..29f9907
>>> 100644 --- a/drivers/staging/wilc1000/host_interface.c
>>> +++ b/drivers/staging/wilc1000/host_interface.c
>>> @@ -1498,12 +1498,45 @@ static s32
>>> handle_rcvd_gnrl_async_info(struct wilc_vif *vif, return result;
>>>  }
>>>  
>>> +static int wilc_pmksa_key_copy(struct wilc_vif *vif, struct
>>> key_attr *hif_key) +{
>>> +   int i;
>>> +   int ret;
>>> +   struct wid wid;
>>> +   u8 *key_buf;
>>> +
>>> +   key_buf = kmalloc((hif_key->attr.pmkid.numpmkid *
>>> PMKSA_KEY_LEN) + 1,
>>> +                     GFP_KERNEL);
>>> +   if (!key_buf)
>>> +           return -ENOMEM;
>>> +
>>> +   key_buf[0] = hif_key->attr.pmkid.numpmkid;
>>> +
>>> +   for (i = 0; i < hif_key->attr.pmkid.numpmkid; i++) {
>>> +           memcpy(key_buf + ((PMKSA_KEY_LEN * i) + 1),
>>> +                  hif_key->attr.pmkid.pmkidlist[i].bssid,
>>> ETH_ALEN);
>>> +           memcpy(key_buf + ((PMKSA_KEY_LEN * i) + ETH_ALEN +
>>> 1),
>>> +                  hif_key->attr.pmkid.pmkidlist[i].pmkid,
>>> PMKID_LEN);
>>> +   }
>>> +
>>> +   wid.id = (u16)WID_PMKID_INFO;
>>> +   wid.type = WID_STR;
>>> +   wid.val = (s8 *)key_buf;  
>>
>> Is this cast really needed?
>>
> 
> This patch is only to address line over 80 chars checkpatch issue.
> It is not good to club these change with type cast related
> modification. For removing unnecessary cast we can submit
> a separate patch series.
> These are my views. What do you think?
> 

I'm ok with this. I was thinking that since you introduced this new function
you may want to also address this, if any.

> 
>>> +   wid.size = (hif_key->attr.pmkid.numpmkid * PMKSA_KEY_LEN)
>>> + 1; +
>>> +   ret = wilc_send_config_pkt(vif, SET_CFG, &wid, 1,
>>> +                              wilc_get_vif_idx(vif));
>>> +
>>> +   kfree(key_buf);
>>> +
>>> +   return ret;
>>> +}
>>> +
>>>  static int handle_key(struct wilc_vif *vif, struct key_attr
>>> *hif_key) {
>>>     int result = 0;
>>>     struct wid wid;
>>>     struct wid wid_list[5];
>>> -   u8 i;
>>>     u8 *key_buf;
>>>     s8 s8idxarray[1];
>>>     struct host_if_drv *hif_drv = vif->hif_drv;
>>> @@ -1547,7 +1580,8 @@ static int handle_key(struct wilc_vif *vif,
>>> struct key_attr *hif_key) wilc_get_vif_idx(vif));
>>>                     kfree(key_buf);
>>>             } else if (hif_key->action & ADDKEY) {
>>> -                   key_buf =
>>> kmalloc(hif_key->attr.wep.key_len + 2, GFP_KERNEL);
>>> +                   key_buf =
>>> kmalloc(hif_key->attr.wep.key_len + 2,
>>> +                                     GFP_KERNEL);
>>>                     if (!key_buf) {
>>>                             result = -ENOMEM;
>>>                             goto out_wep;
>>> @@ -1715,26 +1749,7 @@ static int handle_key(struct wilc_vif *vif,
>>> struct key_attr *hif_key) break;
>>>  
>>>     case PMKSA:
>>> -           key_buf = kmalloc((hif_key->attr.pmkid.numpmkid *
>>> PMKSA_KEY_LEN) + 1, GFP_KERNEL);
>>> -           if (!key_buf)
>>> -                   return -ENOMEM;
>>> -
>>> -           key_buf[0] = hif_key->attr.pmkid.numpmkid;
>>> -
>>> -           for (i = 0; i < hif_key->attr.pmkid.numpmkid; i++)
>>> {
>>> -                   memcpy(key_buf + ((PMKSA_KEY_LEN * i) +
>>> 1), hif_key->attr.pmkid.pmkidlist[i].bssid, ETH_ALEN);
>>> -                   memcpy(key_buf + ((PMKSA_KEY_LEN * i) +
>>> ETH_ALEN + 1), hif_key->attr.pmkid.pmkidlist[i].pmkid, PMKID_LEN);
>>> -           }
>>> -
>>> -           wid.id = (u16)WID_PMKID_INFO;
>>> -           wid.type = WID_STR;
>>> -           wid.val = (s8 *)key_buf;
>>> -           wid.size = (hif_key->attr.pmkid.numpmkid *
>>> PMKSA_KEY_LEN) + 1; -
>>> -           result = wilc_send_config_pkt(vif, SET_CFG, &wid,
>>> 1,
>>> -
>>> wilc_get_vif_idx(vif)); -
>>> -           kfree(key_buf);
>>> +           result = wilc_pmksa_key_copy(vif, hif_key);
>>>             break;
>>>     }
>>>  
>>>   
> 
> 
> 
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to