samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacld-3.0: Rename HDD variable pNumChannels

Browse Source 
The Linux Coding Style frowns upon mixed-case names and so-called
Hungarian notation, so rename HDD local variable pNumChannels to be
compliant.

Change-Id: If1b27fd188eebd48a3155d57effe27b5c6b00559
CRs-Fixed: 2407329
This commit is contained in:
Jeff Johnson
2019-02-26 18:09:53 -08:00
committed by nshrivas
parent 8a2e1da1d9
commit c3eac6ccd6
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
core/hdd/src/wlan_hdd_ioctl.c
View File

@@ -1189,7 +1189,7 @@ hdd_parse_sendactionframe(struct hdd_adapter *adapter, const char *command,
* @command: Pointer to input channel list
* @channel_list: Pointer to local output array to record
* channel list
* @pNumChannels: Pointer to number of roam scan channels
* @num_channels: Pointer to number of roam scan channels
*
* This function parses the channel list passed in the format
* SETROAMSCANCHANNELS<space><Number of channels><space>Channel 1<space>
@@ -1205,7 +1205,7 @@ hdd_parse_sendactionframe(struct hdd_adapter *adapter, const char *command,
*/
static int
hdd_parse_channellist(const uint8_t *command, uint8_t *channel_list,
uint8_t *pNumChannels)
uint8_t *num_channels)
{
const uint8_t *in_ptr = command;
int tempInt;
@@ -1238,11 +1238,11 @@ hdd_parse_channellist(const uint8_t *command, uint8_t *channel_list,
(tempInt <= 0) || (tempInt > CFG_VALID_CHANNEL_LIST_LEN))
return -EINVAL;
*pNumChannels = tempInt;
*num_channels = tempInt;
hdd_debug("Number of channels are: %d", *pNumChannels);
hdd_debug("Number of channels are: %d", *num_channels);
for (j = 0; j < (*pNumChannels); j++) {
for (j = 0; j < (*num_channels); j++) {
/*
* in_ptr pointing to the beginning of first space after number
* of channels
@@ -1251,7 +1251,7 @@ hdd_parse_channellist(const uint8_t *command, uint8_t *channel_list,
/* no channel list after the number of channels argument */
if (NULL == in_ptr) {
if (0 != j) {
*pNumChannels = j;
*num_channels = j;
return 0;
} else {
return -EINVAL;
@@ -1268,7 +1268,7 @@ hdd_parse_channellist(const uint8_t *command, uint8_t *channel_list,
*/
if ('\0' == *in_ptr) {
if (0 != j) {
*pNumChannels = j;
*num_channels = j;
return 0;
} else {
return -EINVAL;