Re: [PATCH] staging: rtl8723bs: remove get_monotonic_boottime()

2018-06-19 Thread Quytelda Kahja
ime() is deprecated because it uses the > old 'timespec' structure. This replaces one of the last callers > with a call to ktime_get_boottime, which also simplifies it > a bit by avoiding a double conversion. > > Signed-off-by: Arnd Bergmann Reviewed-by: Quytelda Kahja > --- > d

[PATCH 01/11] staging: rtl8723bs: Clean up whitespace in 'PHY_GetTxPowerLimit()'.

2018-06-16 Thread Quytelda Kahja
Wrap lines longer than 80 characters where possible, delete double newlines, and fix alignment per the kernel coding style guidelines. Signed-off-by: Quytelda Kahja --- .../staging/rtl8723bs/hal/hal_com_phycfg.c| 110 ++ 1 file changed, 59 insertions(+), 51 deletions

[PATCH 10/11] staging: rtl8723bs: Merge workaround conditionals into single else-if.

2018-06-16 Thread Quytelda Kahja
The if conditionals used to work around wrong TX power limit indices can be condensed into a single if/else-if statement for more concise expression. Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions

[PATCH 07/11] staging: rtl8723bs: Move bandwidth index lookup to new function.

2018-06-16 Thread Quytelda Kahja
Factoring out the conditional lookup of bandwidth index into the power limit table into it's own function simplifies the logic of 'phy_get_tx_pwr_lmt()'. Signed-off-by: Quytelda Kahja --- .../staging/rtl8723bs/hal/hal_com_phycfg.c| 26 --- 1 file changed, 17 insertions(+), 9

[PATCH 08/11] staging: rtl8723bs: Fix spelling mistake in comment.

2018-06-16 Thread Quytelda Kahja
Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c index 6d8a07ac7bb3..dd097df86fa3 100644

[PATCH 03/11] staging: rtl8723bs: Fix camel-case names in phy_get_tx_pwr_lmt().

2018-06-16 Thread Quytelda Kahja
Change camel-case names to snake-case names; to avoid variable name conflicts, rename table index variables to idx_*. Signed-off-by: Quytelda Kahja --- .../staging/rtl8723bs/hal/hal_com_phycfg.c| 200 +- 1 file changed, 105 insertions(+), 95 deletions(-) diff --git

[PATCH 09/11] staging: rtl8723bs: Merge conditionals with similar bodies.

2018-06-16 Thread Quytelda Kahja
Two conditionals that set 'channel' based on 'band_type' in 'phy_get_tx_pwr_lmt()' can be simplified into one single-line conditional. Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

[PATCH 05/11] staging: rtl8723bs: Remove empty else-if conditional.

2018-06-16 Thread Quytelda Kahja
This else-if conditional block does nothing; remove it. Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c index

[PATCH 11/11] staging: rtl8723bs: Add missing curly braces on else statement.

2018-06-16 Thread Quytelda Kahja
Fix 'braces {} should be used on all arms of this statement' coding style problem in 'phy_get_tx_pwr_lmt()'. Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/hal

[PATCH 04/11] staging: rtl8723bs: Combine if statements with equivalent body.

2018-06-16 Thread Quytelda Kahja
Two if statements that carry out the same operation can be combined with a logical OR. Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b

[PATCH 06/11] staging: rtl8723bs: Move rate section index lookup to new function.

2018-06-16 Thread Quytelda Kahja
The rate section lookup is a large switch statement in the middle of 'phy_get_tx_pwr_lmt()'; refactor this statement into it's own function for increased readability. Signed-off-by: Quytelda Kahja --- .../staging/rtl8723bs/hal/hal_com_phycfg.c| 109 -- 1 file changed, 47

[PATCH 02/11] staging: rtl8723bs: Rename PHY_GetTxPowerLimit().

2018-06-16 Thread Quytelda Kahja
Rename camel-case 'PHY_GetTxPowerLimit()' to 'phy_get_tx_pwr_lmt()'. Signed-off-by: Quytelda Kahja --- drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 6 +++--- drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c| 2 +- drivers/staging/rtl8723bs/include/hal_com_phycfg.h | 2 +- 3 files

[PATCH 1/9] staging: rtl8723bs: Rename 'Hal8723BPwrSeq.{c, h}' to 'hal_pwr_seq.*'.

2018-05-22 Thread Quytelda Kahja
Camel-case naming is discouraged int the linux kernel coding style. Rename these files using snake case, and update the makefile to use the new names. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/rtl8723bs/Makefile | 2 +- .../s

[PATCH 3/9] staging: rtl8723bs: Rename 'Hal8723BPhyCfg.h' using snake case.

2018-05-22 Thread Quytelda Kahja
Camel-case is discouraged in the linux kernel coding style. Rename this header file using snake case instead. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- .../rtl8723bs/include/{Hal8723BPhyCfg.h => hal_phy_cfg.h} | 0 drivers/staging/rtl8723bs/include/rtl87

[PATCH 6/9] staging: rtl8723bs: Fix camel-case in 'struct rt_firmware'.

2018-05-22 Thread Quytelda Kahja
Change the members of 'struct rt_firmware' to be snake case instead of camel-case, per the kernel coding style guide. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c| 16 drivers/staging/rtl8723bs/include/rtl8723b

[PATCH 7/9] staging: rtl8723bs: Fix spelling/grammar errors in comment.

2018-05-22 Thread Quytelda Kahja
Fix the spelling/grammar errors in the comment block describing the 'Function' member of 'struct rt_firmware_hdr'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/rtl8723bs/include/rtl8723b_hal.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH 9/9] staging: rtl8723bs: Fix grammar error in comment.

2018-05-22 Thread Quytelda Kahja
Fix a grammatical error in the comment describing 'struct rt_firmware_hdr'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/rtl8723bs/include/rtl8723b_hal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/i

[PATCH 4/9] staging: rtl8723bs: Clean up whitespace in 'rtl8723_hal.h'.

2018-05-22 Thread Quytelda Kahja
Make alignment and whitespace more consistent within the file 'rtl8723_hal.h' and with the kernel coding style guidelines. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- .../staging/rtl8723bs/include/rtl8723b_hal.h | 261 ++ 1 file changed, 144 insertions(+

[PATCH 5/9] staging: rtl8723bs: Fix camel-case in IS_FW_HEADER_EXIST_8723B().

2018-05-22 Thread Quytelda Kahja
Change the parameter of the macro to the snake case 'fw_hdr' instead of '_pFwHdr'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/rtl8723bs/include/rtl8723b_hal.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723bs/i

[PATCH 8/9] staging: rtl8723bs: Fix camel-case in 'struct rt_firmware_hdr'.

2018-05-22 Thread Quytelda Kahja
Replace camel-case member names with snake-case names per the linux kernel coding style guidelines. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 14 .../staging/rtl8723bs/include/rtl8723b_hal.h | 32 +---

[PATCH v2 75/75] staging: ks7010: Replace memcpy() with ether_addr_copy().

2018-03-31 Thread Quytelda Kahja
ether_addr_copy() is the function for copying a hardware address, so replace the manual memcpy() operation with ether_addr_copy(). Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2 69/75] staging: ks7010: Remove extra blank line between functions.

2018-03-31 Thread Quytelda Kahja
Remove an extra blank line indicated by checkpatch. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks7010_sdio.c | 3 ++- drivers/staging/ks7010/ks_hostif.c | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

[PATCH v2 65/75] staging: ks7010: Remove trailing _t from 'struct pmk_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct pmk_t' with 'struct pmk'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks

[PATCH v2 61/75] staging: ks7010: Remove trailing _t from 'struct wpa_key_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct wpa_key_t' with 'struct wpa_key'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 74/75] staging: ks7010: Replace memcmp() with ether_addr_equal().

2018-03-31 Thread Quytelda Kahja
ether_addr_equal() is the function for comparing HW addresses, so remove the manual memcmp operation and replace it with ether_addr_equals(). Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.c | 4 ++-- 1 file changed, 2 insertions(+), 2 del

[PATCH v2 53/75] staging: ks7010: Remove trailing _t from 'struct wps_ie_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct wps_ie_t' with 'struct wps_ie'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/

[PATCH v2 58/75] staging: ks7010: Remove trailing _t from 'struct power_save_status_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct power_save_status_t' with 'struct power_save_status'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org&

[PATCH v2 70/75] staging: ks7010: Rename ks_wlan_set_multicast_list()

2018-03-31 Thread Quytelda Kahja
All of the net_device_ops callbacks are named after their counterparts in the kernel's 'struct net_device_ops', except ks_wlan_set_multicast_list(). Rename it to ks_wlan_set_rx_mode() for greater consistency. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/

[PATCH v2 63/75] staging: ks7010: Remove trailing _t from 'struct wpa_status_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct wpa_status_t' with 'struct wpa_status'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 73/75] staging: ks7010: Remove unnecessary casts in 'struct ks_wlan_handler_def'.

2018-03-31 Thread Quytelda Kahja
The casts used when initializing members of this data structure mirror the types the variables already have. Remove the casts. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_wlan_net.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH v2 55/75] staging: ks7010: Remove trailing _t from 'struct local_aplist_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct local_aplist_t' with 'struct local_aplist'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> ---

[PATCH v2 68/75] staging: ks7010: Replace manual array copy with ether_addr_copy().

2018-03-31 Thread Quytelda Kahja
Copying the dummy HW address into the struct net_device doesn't need to be done byte by byte; use ether_addr_copy() instead. Additionally, dev->dev_addr is not eight bytes long. ether_setup() sets the dev->addr_len to ETH_ALEN (defined as 6) in the net core code. Signed-off-by: Quytelda

[PATCH v2 67/75] staging: ks7010: Remove trailing _t from 'struct michael_mic_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct michael_mic_t' with 'struct michael_mic'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 64/75] staging: ks7010: Remove trailing _t from 'struct pmk_list_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct pmk_list_t' with 'struct pmk_list'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 71/75] staging: ks7010: Remove dummy address set.

2018-03-31 Thread Quytelda Kahja
Setting a dummy address during the driver probe is not necessary. The dev_addr field is already zeroed out from alloc_etherdev(). Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_wlan_net.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/s

[PATCH v2 54/75] staging: ks7010: Remove trailing _t from 'struct local_ap_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct local_ap_t' with 'struct local_ap'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 40/75] staging: ks7010: Remove trailing _t from 'struct association_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct association_request_t' with 'struct association_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 72/75] staging: ks7010: Change 'device_open_status' to a bool.

2018-03-31 Thread Quytelda Kahja
The 'device_open_status' member of 'struct ks_wlan_private' is only ever set to zero or one, so it makes more sense for it to be a bool instead of an int. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_wlan.h | 2 +- drivers/staging/ks7010/ks_wlan

[PATCH v2 59/75] staging: ks7010: Remove trailing _t from 'struct sleep_status_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct sleep_status_t' with 'struct sleep_status'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> ---

[PATCH v2 66/75] staging: ks7010: Remove trailing _t from 'struct wps_status_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct wps_status_t' with 'struct wps_status'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 62/75] staging: ks7010: Remove trailing _t from 'struct mic_failure_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct mic_failure_t' with 'struct mic_failure'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 60/75] staging: ks7010: Remove trailing _t from 'struct scan_ext_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct scan_ext_t' with 'struct scan_ext'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 46/75] staging: ks7010: Remove unused 'struct hostif_phy_information_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_phy_information_confirm_t' is not used in this driver. Remove 'struct hostif_phy_information_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 12 1 file changed, 12 deletions(-) diff --git a/drivers/s

[PATCH v2 52/75] staging: ks7010: Remove trailing _t from 'struct rsn_ie_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct rsn_ie_t' with 'struct rsn_ie'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/

[PATCH v2 56/75] staging: ks7010: Remove trailing _t from 'struct local_gain_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct local_gain_t' with 'struct local_gain'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 41/75] staging: ks7010: Remove trailing _t from 'struct association_response_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct association_response_t' with 'struct association_response'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 48/75] staging: ks7010: Remove unused 'struct hostif_sleep_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_sleep_confirm_t' is not used in this driver. Remove 'struct hostif_sleep_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/d

[PATCH v2 35/75] staging: ks7010: Remove unused 'struct hostif_infrastructure_set_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_infrastructure_set_confirm_t' is not used in this driver. Remove 'struct hostif_infrastructure_set_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/

[PATCH v2 57/75] staging: ks7010: Remove trailing _t from 'struct local_eeprom_sum_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct local_eeprom_sum_t' with 'struct local_eeprom_sum'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org&

[PATCH v2 51/75] staging: ks7010: Remove trailing _t from 'struct hostt_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostt_t' with 'struct hostt'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/

[PATCH v2 50/75] staging: ks7010: Remove unused 'struct hostif_mic_failure_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_mic_failure_confirm_t' is not used in this driver. Remove 'struct hostif_mic_failure_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_ho

[PATCH v2 33/75] staging: ks7010: Remove unused 'struct hostif_ps_adhoc_set_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_ps_adhoc_set_confirm_t' is not used in this driver. Remove 'struct hostif_ps_adhoc_set_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_ho

[PATCH v2 49/75] staging: ks7010: Remove trailing _t from 'struct hostif_mic_failure_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_mic_failure_request_t' with 'struct hostif_mic_failure_request'. Signed-off-by: Quytelda Ka

[PATCH v2 10/75] staging: ks7010: Remove unused 'struct hostif_mib_get_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_mib_get_confirm_t' is not used in this driver. Remove 'struct hostif_mib_get_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 23/75] staging: ks7010: Remove unused 'struct rsn_t'.

2018-03-31 Thread Quytelda Kahja
'struct rsn_t' is not used in this driver. Remove 'struct rsn_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/drivers/staging/ks7010/ks_hostif.h

[PATCH v2 14/75] staging: ks7010: Remove unused 'struct hostif_power_mgmt_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_power_mgmt_confirm_t' is not used in this driver. Remove 'struct hostif_power_mgmt_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 37/75] staging: ks7010: Remove trailing _t from 'struct hostif_adhoc_set2_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_adhoc_set2_request_t' with 'struct hostif_adhoc_set2_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 42/75] staging: ks7010: Remove unused 'struct hostif_associate_indication_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_associate_indication_t' is not used in this driver. Remove 'struct hostif_associate_indication_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/

[PATCH v2 28/75] staging: ks7010: Remove unused 'struct hostif_connect_indication_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_connect_indication_t' is not used in this driver. Remove 'struct hostif_connect_indication_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 15/75] staging: ks7010: Remove trailing _t from 'struct hostif_start_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_start_request_t' with 'struct hostif_start_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 32/75] staging: ks7010: Remove trailing _t from 'struct hostif_ps_adhoc_set_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_ps_adhoc_set_request_t' with 'struct hostif_ps_adhoc_set_request'. Signed-off-by: Quytelda Ka

[PATCH v2 24/75] staging: ks7010: Remove trailing _t from 'struct erp_params_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct erp_params_t' with 'struct erp_params'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 34/75] staging: ks7010: Remove trailing _t from 'struct hostif_infrastructure_set_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_infrastructure_set_request_t' with 'struct hostif_infrastructure_set_request'. Signed-off-by: Quytelda Ka

[PATCH v2 20/75] staging: ks7010: Remove trailing _t from 'struct ds_parms_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct ds_parms_t' with 'struct ds_parms'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 38/75] staging: ks7010: Remove unused 'struct hostif_adhoc_set_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_adhoc_set_confirm_t' is not used in this driver. Remove 'struct hostif_adhoc_set_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 29/75] staging: ks7010: Remove trailing _t from 'struct hostif_stop_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_stop_request_t' with 'struct hostif_stop_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 18/75] staging: ks7010: Remove trailing _t from 'struct rate_set8_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct rate_set8_t' with 'struct rate_set8'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 47/75] staging: ks7010: Remove trailing _t from 'struct hostif_sleep_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_sleep_request_t' with 'struct hostif_sleep_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 44/75] staging: ks7010: Remove unused 'struct hostif_bss_scan_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_bss_scan_confirm_t' is not used in this driver. Remove 'struct hostif_bss_scan_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 45/75] staging: ks7010: Remove trailing _t from 'struct hostif_phy_information_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_phy_information_request_t' with 'struct hostif_phy_information_request'. Signed-off-by: Quytelda Ka

[PATCH v2 43/75] staging: ks7010: Remove trailing _t from 'struct hostif_bss_scan_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_bss_scan_request_t' with 'struct hostif_bss_scan_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 36/75] staging: ks7010: Remove trailing _t from 'struct hostif_adhoc_set_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_adhoc_set_request_t' with 'struct hostif_adhoc_set_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 30/75] staging: ks7010: Remove unused 'struct hostif_stop_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_stop_confirm_t' is not used in this driver. Remove 'struct hostif_stop_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/d

[PATCH v2 39/75] staging: ks7010: Remove unused 'struct last_associate_t'.

2018-03-31 Thread Quytelda Kahja
'struct last_associate_t' is not used in this driver. Remove 'struct last_associate_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/drivers/staging/

[PATCH v2 21/75] staging: ks7010: Remove trailing _t from 'struct cf_parms_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct cf_parms_t' with 'struct cf_parms'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 31/75] staging: ks7010: Remove trailing _t from 'struct hostif_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_request_t' with 'struct hostif_request'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> ---

[PATCH v2 06/75] staging: ks7010: Remove unused 'struct hostif_data_indication_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_data_indication_t' is not used in this driver. Remove 'struct hostif_data_indication_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 13/75] staging: ks7010: Remove trailing _t from 'struct hostif_power_mgmt_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_power_mgmt_request_t' with 'struct hostif_power_mgmt_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 25/75] staging: ks7010: Remove trailing _t from 'struct rate_set16_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct rate_set16_t' with 'struct rate_set16'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 17/75] staging: ks7010: Remove trailing _t from 'struct ssid_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct ssid_t' with 'struct ssid'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks

[PATCH v2 26/75] staging: ks7010: Remove trailing _t from 'struct ap_info_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct ap_info_t' with 'struct ap_info'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 22/75] staging: ks7010: Remove trailing _t from 'struct ibss_parms_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct ibss_parms_t' with 'struct ibss_parms'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 19/75] staging: ks7010: Remove trailing _t from 'struct fh_parms_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct fh_parms_t' with 'struct fh_parms'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 27/75] staging: ks7010: Remove trailing _t from 'struct link_ap_info_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct link_ap_info_t' with 'struct link_ap_info'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> ---

[PATCH v2 12/75] staging: ks7010: Remove unused 'struct hostif_mib_set_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_mib_set_confirm_t' is not used in this driver. Remove 'struct hostif_mib_set_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/ks7010/ks_hosti

[PATCH v2 07/75] staging: ks7010: Remove trailing _t from 'struct channel_list_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct channel_list_t' with 'struct channel_list'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> ---

[PATCH v2 16/75] staging: ks7010: Remove unused 'struct hostif_start_confirm_t'.

2018-03-31 Thread Quytelda Kahja
'struct hostif_start_confirm_t' is not used in this driver. Remove 'struct hostif_start_confirm_t'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/d

[PATCH v2 01/75] staging: ks7010: Use the ARRAY_SIZE() macro to calculate array sizes.

2018-03-31 Thread Quytelda Kahja
This macro, provided in 'linux/kernel.h', will calculate the size more succinctly than a division operation. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_wlan_net.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/s

[PATCH v2 09/75] staging: ks7010: Remove trailing _t from 'struct hostif_mib_value_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_mib_value_t' with 'struct hostif_mib_value'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org&

[PATCH v2 08/75] staging: ks7010: Remove trailing _t from 'struct hostif_mib_get_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_mib_get_request_t' with 'struct hostif_mib_get_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 11/75] staging: ks7010: Remove trailing _t from 'struct hostif_mib_set_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_mib_set_request_t' with 'struct hostif_mib_set_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH v2 02/75] staging: ks7010: Remove trailing _t from 'struct wpa_suite_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct wpa_suite_t' with 'struct wpa_suite'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 04/75] staging: ks7010: Remove trailing _t from 'struct pmk_cache_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct pmk_cache_t' with 'struct pmk_cache'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 03/75] staging: ks7010: Remove trailing _t from 'struct rsn_mode_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct rsn_mode_t' with 'struct rsn_mode'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH v2 05/75] staging: ks7010: Remove trailing _t from 'struct hostif_data_request_t'.

2018-03-31 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct hostif_data_request_t' with 'struct hostif_data_request'. Signed-off-by: Quytelda Kahja <quyte..

[PATCH 75/76] staging: ks7010: Replace memcmp() with ether_addr_equal().

2018-03-30 Thread Quytelda Kahja
ether_addr_equal() is the function for comparing HW addresses, so remove the manual memcmp operation and replace it with ether_addr_equals(). Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks_hostif.c | 4 ++-- 1 file changed, 2 insertions(+), 2 del

[PATCH 72/72] staging: ks7010: Remove extra blank line between functions.

2018-03-30 Thread Quytelda Kahja
Remove an extra blank line indicated by checkpatch. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks7010_sdio.c | 3 ++- drivers/staging/ks7010/ks_hostif.c | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

[PATCH 65/76] staging: ks7010: Remove trailing _t from 'struct pmk_t'.

2018-03-30 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct pmk_t' with 'struct pmk'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks

[PATCH 63/76] staging: ks7010: Remove trailing _t from 'struct wpa_status_t'.

2018-03-30 Thread Quytelda Kahja
The "_t" suffix is not needed for structure names in this driver, and is a reflection of an older typedef system that is no longer in place. Replace all occurences of 'struct wpa_status_t' with 'struct wpa_status'. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- driver

[PATCH 70/76] staging: ks7010: Remove extra blank line between functions.

2018-03-30 Thread Quytelda Kahja
Remove an extra blank line indicated by checkpatch. Signed-off-by: Quytelda Kahja <quyte...@tamalin.org> --- drivers/staging/ks7010/ks7010_sdio.c | 3 ++- drivers/staging/ks7010/ks_hostif.c | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

  1   2   3   >