[PATCH 04/10] staging: rtl8723bs: replace HT_CAP_AMPDU_FACTOR

Replace unique HT_CAP_AMPDU_FACTOR enum with kernel provided ieee80211_max_ampdu_length_exp enum. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/core/rtw_mlme.c | 4 ++-- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 12 ++--

[PATCH 02/10] staging: rtl8723bs: replace WIFI_REASON_CODE

Replace unique macros and WIFI_REASON_CODE enum with the kernel provided ieee80211_reasoncode. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 4 +-- drivers/staging/rtl8723bs/include/wifi.h | 36 --- 2 files changed, 2 insertions(+), 38

[PATCH 07/10] staging: rtl8723bs: replace RTW_IEEE80211_ADDBA_PARAM_BUF_SIZE_MASK

Replace unique macro RTW_IEEE80211_ADDBA_PARAM_BUF_SIZE_MASK with kernel provided IEEE80211_ADDBA_PARAM_BUF_SIZE_MASK. The other IEEE80211_ADDBA_* and IEEE_DELBA_* macros are duplicates already defined in linux/ieee80211.h, remove them. Signed-off-by: Ross Schmidt ---

[PATCH 01/10] staging: rtl8723bs: replace unique macros and WIFI_STATUS_CODE

Replace many unique macros and WIFI_STATUS_CODE enum with kernel provided ieee80211_statuscode. A duplicate WLAN_STATUS_ASSOC_DENIED_NOSHORT macro is also removed. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 54 +--

[PATCH] Staging: rtl8723bs/core: add blank line after declarations

Fix coding style issue. Add blank line after variable declarations at all the locations found by checkpatch.pl. Signed-off-by: Brother Matthew De Angelis --- drivers/staging/rtl8723bs/core/rtw_mlme.c | 7 +++ 1 file changed, 7 insertions(+) diff --git

[driver-core:driver-core-testing] BUILD SUCCESS 784b2c48ac12dcee27db001fb1a3c58c39380cb6

tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git driver-core-testing branch HEAD: 784b2c48ac12dcee27db001fb1a3c58c39380cb6 driver core: auxiliary bus: Fix auxiliary bus shutdown null auxdrv ptr elapsed time: 722m configs tested: 96 configs skipped: 2 The

[PATCH 10/10] staging: rtl8723bs: replace unique macros and ELEMENT_ID

Replace unique macros and ELEMENT_ID with kernel provided ieee80211_eid enum. In a several cases multiple macros or constants are replaced by one constant. WLAN_EID_HT_CAP, _HT_CAPABILITY_IE_, and EID_HTCapability are replace by WLAN_EID_HT_CAPABILITY. _WPA2_IE_ID_, EID_WPA2, and _RSN_IE_2_ are

[PATCH 03/10] staging: rtl8723bs: replace cap_* macros

Replace unique cap_* macros with kernel provided WLAN_CAPABILITY_* macros. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/core/rtw_ap.c| 2 +- drivers/staging/rtl8723bs/core/rtw_ieee80211.c | 6 +++--- drivers/staging/rtl8723bs/core/rtw_mlme.c | 4 ++--

[PATCH 09/10] staging: rtl8723bs: remove LIST_CONTAINOR

Remove unnecessary macro for container_of and call it directly. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/core/rtw_ap.c | 16 +++--- drivers/staging/rtl8723bs/core/rtw_cmd.c | 2 +- drivers/staging/rtl8723bs/core/rtw_mlme.c | 18 +++

[PATCH 05/10] staging: rtl8723bs: replace OP_MODE_* and HT_INFO_OPERATION_MODE_* macros

Replace unique OP_MODE_* and HT_INFO_OPERATION_MODE_* macro families with kernel provided IEEE80211_HT_OP_MODE_* macros. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/core/rtw_ap.c | 30 drivers/staging/rtl8723bs/include/wifi.h | 13 -- 2 files

[PATCH 08/10] staging: rtl8723bs: remove unused macros

Remove many macros from wifi.h because they are unused. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/include/wifi.h | 33 1 file changed, 33 deletions(-) diff --git a/drivers/staging/rtl8723bs/include/wifi.h b/drivers/staging/rtl8723bs/include/wifi.h

[PATCH 06/10] staging: rtl8723bs: remove WLAN_HT_CAP_SM_PS_* macros

The WLAN_HT_CAP_SM_PS_* macro family is already defined in linux/ieee80211.h, remove them. Signed-off-by: Ross Schmidt --- drivers/staging/rtl8723bs/include/wifi.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/staging/rtl8723bs/include/wifi.h

[PATCH] vme: switch from 'pci_' to 'dma_' API

The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'ca91cx42_alloc_consistent()' and

Re: [PATCH] staging: greybus: audio: Add missing unlock in gbaudio_dapm_free_controls()

在 2020/12/5 2:02, Vaibhav Agarwal 写道: On Fri, Dec 4, 2020 at 2:10 PM Johan Hovold wrote: On Fri, Dec 04, 2020 at 10:13:50AM +0800, Wang Hai wrote: Add the missing unlock before return from function gbaudio_dapm_free_controls() in the error handling case. Fixes: 510e340efe0c ("staging:

[PATCH] staging: greybus: audio: Fix possible leak free widgets in gbaudio_dapm_free_controls

In gbaudio_dapm_free_controls(), if one of the widgets is not found, an error will be returned directly, which will cause the rest to be unable to be freed, resulting in leak. This patch fixes the bug. If if one of them is not found, just skip and free the others. Fixes: 510e340efe0c ("staging:

Re: [PATCH] Staging: rtl8723bs/core fix brace coding style issues in rtw_recv.c

On Fri, Dec 04, 2020 at 04:00:43PM -0600, Brother Matthew De Angelis wrote: > Fix all the brace code style warnings found by checkpatch.pl at the > following lines: > 748, 940, 1039, 1602, 1922, 1939. > At line 940 add a semi-colon to specify a line that does not execute > anything, as in

Re: [PATCH v3 3/6] ARM: dts: sun8i: v3s: Add node for system control

On 12/3/20 10:44 AM, Chen-Yu Tsai wrote: > On Fri, Dec 4, 2020 at 12:25 AM Martin Cerveny wrote: >> >> Hello. >> >> On Thu, 3 Dec 2020, Chen-Yu Tsai wrote: >> >>> Hi, >>> >>> On Mon, Nov 16, 2020 at 8:57 PM Martin Cerveny >>> wrote: Allwinner V3s has system control and SRAM C1 region

Re: [PATCH] Staging: rtl8723bs/core fix brace coding style issues in rtw_recv.c

On Sat, Dec 05, 2020 at 09:41:16AM +0100, Greg KH wrote: > On Fri, Dec 04, 2020 at 04:00:43PM -0600, Brother Matthew De Angelis wrote: > > Fix all the brace code style warnings found by checkpatch.pl at the > > following lines: > > 748, 940, 1039, 1602, 1922, 1939. > > At line 940 add a semi-colon

Re: [PATCH] staging: greybus: audio: Fix possible leak free widgets in gbaudio_dapm_free_controls

On Sat, Dec 5, 2020 at 4:02 PM Wang Hai wrote: > > In gbaudio_dapm_free_controls(), if one of the widgets is not found, an error > will be returned directly, which will cause the rest to be unable to be freed, > resulting in leak. > > This patch fixes the bug. If if one of them is not found, just

Re: [PATCH] Staging: rtl8723bs/core fix brace coding style issues in rtw_recv.c

On Sat, Dec 05, 2020 at 08:28:10AM -0600, YOUR NAME wrote: > On Sat, Dec 05, 2020 at 09:41:16AM +0100, Greg KH wrote: > > On Fri, Dec 04, 2020 at 04:00:43PM -0600, Brother Matthew De Angelis wrote: > > > Fix all the brace code style warnings found by checkpatch.pl at the > > > following lines: > >

[PATCH 2/2] Staging: rtl8723bs/core: fix brace coding style issue

Remove unnecessary braces identified by checkpatch.pl at lines 740, 1039,1602,1922,1939. Signed-off-by: Brother Matthew De Angelis --- drivers/staging/rtl8723bs/core/rtw_recv.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git

[PATCH 0/2] Fix coding style issues

Hello, this series fixes two separate coding style issues. 1. Delete an empty if statement. 2. Fix remaining unnecessary brace coding style issues. Brother Matthew De Angelis (2): Staging: rtl8723bs/core: delete empty if statement Staging: rtl8723bs/core: fix brace coding style issue

[PATCH 1/2] Staging: rtl8723bs/core: delete empty if statement

Delete an if statement that was not executing anything when true. This has the side effect of removing one checkpatch warning about braces not being necessary. Signed-off-by: Brother Matthew De Angelis --- drivers/staging/rtl8723bs/core/rtw_recv.c | 3 --- 1 file changed, 3 deletions(-) diff