linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Claudiu Beznea <Claudiu.Beznea@microchip.com>
To: Ajay Singh <ajay.kathat@microchip.com>, <linux-wireless@vger.kernel.org>
Cc: <devel@driverdev.osuosl.org>, <gregkh@linuxfoundation.org>,
	<ganesh.krishna@microchip.com>, <venkateswara.kaja@microchip.com>,
	<aditya.shankar@microchip.com>
Subject: Re: [PATCH v2 12/14] staging: wilc1000: rename Handle_ScanDone() to avoid camelCase
Date: Mon, 22 Jan 2018 13:07:50 +0200	[thread overview]
Message-ID: <422e8a89-21f8-e52b-d061-4a6f44e7108e@microchip.com> (raw)
In-Reply-To: <1516616547-24654-13-git-send-email-ajay.kathat@microchip.com>



On 22.01.2018 12:22, Ajay Singh wrote:
> Fix "Avoid camelCase" issue reported by checkpatch.pl
> script.
Same here as in patch 02 of this series.

> 
> Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
> Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>
> ---
>  drivers/staging/wilc1000/host_interface.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c
> index 0eaf6c6..b34243c 100644
> --- a/drivers/staging/wilc1000/host_interface.c
> +++ b/drivers/staging/wilc1000/host_interface.c
> @@ -267,7 +267,7 @@ static struct wilc_vif *join_req_vif;
>  
>  static void *host_int_parse_join_bss_param(struct network_info *info);
>  static int host_int_get_ipaddress(struct wilc_vif *vif, u8 *ip_addr, u8 idx);
> -static s32 Handle_ScanDone(struct wilc_vif *vif, enum scan_event enuEvent);
> +static s32 handle_scan_done(struct wilc_vif *vif, enum scan_event enuEvent);
>  static void host_if_work(struct work_struct *work);
>  
>  /*!
> @@ -847,7 +847,7 @@ static s32 handle_scan(struct wilc_vif *vif, struct scan_attr *scan_info)
>  ERRORHANDLER:
>  	if (result) {
>  		del_timer(&hif_drv->scan_timer);
> -		Handle_ScanDone(vif, SCAN_EVENT_ABORTED);
> +		handle_scan_done(vif, SCAN_EVENT_ABORTED);
>  	}
>  
>  	kfree(scan_info->ch_freq_list);
> @@ -863,8 +863,8 @@ static s32 handle_scan(struct wilc_vif *vif, struct scan_attr *scan_info)
>  	return result;
>  }
>  
> -static s32 Handle_ScanDone(struct wilc_vif *vif,
> -			   enum scan_event enuEvent)
> +static s32 handle_scan_done(struct wilc_vif *vif,
> +			    enum scan_event enuEvent)
>  {
>  	s32 result = 0;
>  	u8 u8abort_running_scan;
> @@ -1460,7 +1460,7 @@ static s32 Handle_RcvdGnrlAsyncInfo(struct wilc_vif *vif,
>  
>  			if (hif_drv->usr_scan_req.scan_result) {
>  				del_timer(&hif_drv->scan_timer);
> -				Handle_ScanDone(vif, SCAN_EVENT_ABORTED);
> +				handle_scan_done(vif, SCAN_EVENT_ABORTED);
>  			}
>  
>  			strDisconnectNotifInfo.reason = 0;
> @@ -1508,7 +1508,7 @@ static s32 Handle_RcvdGnrlAsyncInfo(struct wilc_vif *vif,
>  			   (hif_drv->usr_scan_req.scan_result)) {
>  			del_timer(&hif_drv->scan_timer);
>  			if (hif_drv->usr_scan_req.scan_result)
> -				Handle_ScanDone(vif, SCAN_EVENT_ABORTED);
> +				handle_scan_done(vif, SCAN_EVENT_ABORTED);
>  		}
>  	}
>  
> @@ -2525,7 +2525,7 @@ static void host_if_work(struct work_struct *work)
>  		if (!wilc_wlan_get_num_conn_ifcs(wilc))
>  			wilc_chip_sleep_manually(wilc);
>  
> -		Handle_ScanDone(msg->vif, SCAN_EVENT_DONE);
> +		handle_scan_done(msg->vif, SCAN_EVENT_DONE);
>  
>  		if (msg->vif->hif_drv->remain_on_ch_pending)
>  			Handle_RemainOnChan(msg->vif,
> @@ -2567,7 +2567,7 @@ static void host_if_work(struct work_struct *work)
>  		break;
>  
>  	case HOST_IF_MSG_SCAN_TIMER_FIRED:
> -		Handle_ScanDone(msg->vif, SCAN_EVENT_ABORTED);
> +		handle_scan_done(msg->vif, SCAN_EVENT_ABORTED);
>  		break;
>  
>  	case HOST_IF_MSG_CONNECT_TIMER_FIRED:
> 

  reply	other threads:[~2018-01-22 11:07 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-22 10:22 [PATCH v2 00/14] fix to remove unnecessary parenthesis,typedef and avoid camelCase Ajay Singh
2018-01-22 10:22 ` [PATCH v2 01/14] staging: wilc1000: remove unnecessary use of parentheses Ajay Singh
2018-01-22 10:22 ` [PATCH v2 02/14] staging: wilc1000: removed unnecessary defined enums typedef Ajay Singh
2018-01-22 11:06   ` Claudiu Beznea
2018-01-22 11:22     ` Dan Carpenter
2018-01-22 11:32       ` Claudiu Beznea
2018-01-22 10:22 ` [PATCH v2 03/14] staging: wilc1000: fix alignments to match open parenthesis Ajay Singh
2018-01-22 10:22 ` [PATCH v2 04/14] staging: wilc1000: rename host_int_ParseJoinBssParam() and it's variable using camelCase Ajay Singh
2018-01-22 11:13   ` Claudiu Beznea
2018-01-22 10:22 ` [PATCH v2 05/14] staging: wilc1000: rename camelCase used in Handle_DelAllSta() and its variable Ajay Singh
2018-01-22 10:22 ` [PATCH v2 06/14] staging: wilc1000: rename strWIDList variable to wid_list to avoid camelCase Ajay Singh
2018-01-22 10:22 ` [PATCH v2 07/14] staging: wilc1000: rename u32WidsCount variable " Ajay Singh
2018-01-22 10:22 ` [PATCH v2 08/14] staging: wilc1000: rename pu8CurrByte " Ajay Singh
2018-01-22 10:22 ` [PATCH v2 09/14] staging: wilc1000: rename pstrHostIFconnectAttr argument " Ajay Singh
2018-01-22 10:22 ` [PATCH v2 10/14] staging: wilc1000: rename ptstrJoinBssParam variable " Ajay Singh
2018-01-22 10:22 ` [PATCH v2 11/14] staging: wilc1000: rename strConnectInfo " Ajay Singh
2018-01-22 10:22 ` [PATCH v2 12/14] staging: wilc1000: rename Handle_ScanDone() " Ajay Singh
2018-01-22 11:07   ` Claudiu Beznea [this message]
2018-01-22 10:22 ` [PATCH v2 13/14] staging: wilc1000: rename Handle_Connect() " Ajay Singh
2018-01-22 11:08   ` Claudiu Beznea
2018-01-29 18:13   ` kbuild test robot
2018-01-30 13:59     ` Ajay Singh
2018-01-30 14:40       ` Dan Carpenter
2018-01-30 16:10         ` Ajay Singh
2018-01-30 14:36     ` Dan Carpenter
2018-01-22 10:22 ` [PATCH v2 14/14] staging: wilc1000: rename Handle_ConnectTimeout() and Handle_Key() Ajay Singh
2018-01-22 11:10 ` [PATCH v2 00/14] fix to remove unnecessary parenthesis,typedef and avoid camelCase Claudiu Beznea
2018-01-22 11:15   ` Claudiu Beznea
2018-01-22 11:54 ` Claudiu Beznea

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=422e8a89-21f8-e52b-d061-4a6f44e7108e@microchip.com \
    --to=claudiu.beznea@microchip.com \
    --cc=aditya.shankar@microchip.com \
    --cc=ajay.kathat@microchip.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=ganesh.krishna@microchip.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=venkateswara.kaja@microchip.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).