Re: [PATCH] staging: r8822be: Fix typo for CONFIG_RTLWIFI_DEBUG
Indeed, sorry I missed that as well. So what should we make of that #ifdef? The code inside it doesn't compile (anymore? I didn't find any development history for that patch except the original mail), as there is no definition of struct submit_ctx in the headers (for other rtl drivers - 8188eu, 8723bs - that struct lives in include/rtw_xmit.h). Is a comparable header simply missing? Regards, Andreas On 08/29/17 16:42, Greg KH wrote: > On Tue, Aug 29, 2017 at 09:10:10AM -0500, Larry Finger wrote: >> On 08/29/2017 06:30 AM, Andreas Ziegler wrote: >>> The debugging output in deinit_priv is guarded by an #ifdef using >>> CONFIG_RTL_DEBUG. This symbol does not exist and should be >>> CONFIG_RTLWIFI_DEBUG instead. >>> >>> Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> >> >> NACK. >> >> Yes, there is a problem; however, CONFIG_RTLWIFI_DEBUG is not the value that >> should be used. That one is reserved for the non-staging drivers in >> drivers/net/wireless/realtek/rtlwifi/. The correct symbol for r8822be is >> CONFIG_RTLWIFI_DEBUG_ST. > > Yeah, kbuild just blew up on this as well, I wonder why my local build > testing didn't see that :( > > Now dropped. > > greg k-h > smime.p7s Description: S/MIME Cryptographic Signature ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH] staging: r8822be: Fix typo for CONFIG_RTLWIFI_DEBUG
The debugging output in deinit_priv is guarded by an #ifdef using CONFIG_RTL_DEBUG. This symbol does not exist and should be CONFIG_RTLWIFI_DEBUG instead. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- drivers/staging/rtlwifi/halmac/rtl_halmac.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtlwifi/halmac/rtl_halmac.c b/drivers/staging/rtlwifi/halmac/rtl_halmac.c index 031bf2c..888ca43 100644 --- a/drivers/staging/rtlwifi/halmac/rtl_halmac.c +++ b/drivers/staging/rtlwifi/halmac/rtl_halmac.c @@ -386,7 +386,7 @@ static void deinit_priv(struct rtl_halmac *halmac) u32 count, size; count = HALMAC_FEATURE_ALL + 1; -#ifdef CONFIG_RTL_DEBUG +#ifdef CONFIG_RTLWIFI_DEBUG { struct submit_ctx *sctx; u32 i; @@ -405,7 +405,7 @@ static void deinit_priv(struct rtl_halmac *halmac) rtl_mfree((u8 *)sctx, sizeof(*sctx)); } } -#endif /* !CONFIG_RTL_DEBUG */ +#endif /* !CONFIG_RTLWIFI_DEBUG */ size = sizeof(*indicator) * count; kfree((u8 *)indicator); } -- 2.7.4 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: staging: r8822be: Add code for halmac sub-driver
Hi Ping-Ke, hi Larry, I noticed your commit in linux-next today using the in-tree script at scripts/checkkconfigsymbols.py [0]. The script showed that it uses CONFIG_RTL_DEBUG in an #ifdef statement - this symbol, however, does not exist, and should probably be CONFIG_RTLWIFI_DEBUG instead. I attached a patch which fixes this problem. Best regards, Andreas Ziegler [0]: I called the script like this: './scripts/checkkconfigsymbols.py --force -c 938a0447f094' From 6c713900ed877c01f94fb76630f9ffd32c73ec88 Mon Sep 17 00:00:00 2001 From: Andreas Ziegler <andreas.zieg...@fau.de> Date: Tue, 29 Aug 2017 12:50:37 +0200 Subject: [PATCH] staging: r8822be: Fix typo for CONFIG_RTLWIFI_DEBUG The debugging output in deinit_priv is guarded by an #ifdef using CONFIG_RTL_DEBUG. This symbol does not exist and should be CONFIG_RTLWIFI_DEBUG instead. Signed-off-by: Andreas Ziegler <andreas.zieg...@fau.de> --- drivers/staging/rtlwifi/halmac/rtl_halmac.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtlwifi/halmac/rtl_halmac.c b/drivers/staging/rtlwifi/halmac/rtl_halmac.c index 031bf2c..888ca43 100644 --- a/drivers/staging/rtlwifi/halmac/rtl_halmac.c +++ b/drivers/staging/rtlwifi/halmac/rtl_halmac.c @@ -386,7 +386,7 @@ static void deinit_priv(struct rtl_halmac *halmac) u32 count, size; count = HALMAC_FEATURE_ALL + 1; -#ifdef CONFIG_RTL_DEBUG +#ifdef CONFIG_RTLWIFI_DEBUG { struct submit_ctx *sctx; u32 i; @@ -405,7 +405,7 @@ static void deinit_priv(struct rtl_halmac *halmac) rtl_mfree((u8 *)sctx, sizeof(*sctx)); } } -#endif /* !CONFIG_RTL_DEBUG */ +#endif /* !CONFIG_RTLWIFI_DEBUG */ size = sizeof(*indicator) * count; kfree((u8 *)indicator); } -- 2.7.4 smime.p7s Description: S/MIME Cryptographic Signature ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel