* [PATCH 0/2] staging: rtl8192u: unused code cleanup @ 2022-10-26 3:26 Deepak R Varma 2022-10-26 3:27 ` [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation Deepak R Varma 2022-10-26 3:28 ` [PATCH 2/2] staging: rtl8192u: remove unused macro definition Deepak R Varma 0 siblings, 2 replies; 14+ messages in thread From: Deepak R Varma @ 2022-10-26 3:26 UTC (permalink / raw) To: outreachy, Greg Kroah-Hartman, linux-staging, linux-kernel Remove code that is unnecessary or unused. Deepak R Varma (2): staging: rtl8192u: remove unnecessary function implementation staging: rtl8192u: remove unused macro definition drivers/staging/rtl8192u/ieee80211/ieee80211.h | 5 ----- drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c | 6 ------ drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 --- 3 files changed, 14 deletions(-) -- 2.30.2 ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation 2022-10-26 3:26 [PATCH 0/2] staging: rtl8192u: unused code cleanup Deepak R Varma @ 2022-10-26 3:27 ` Deepak R Varma 2022-10-26 10:44 ` Greg Kroah-Hartman 2022-10-26 3:28 ` [PATCH 2/2] staging: rtl8192u: remove unused macro definition Deepak R Varma 1 sibling, 1 reply; 14+ messages in thread From: Deepak R Varma @ 2022-10-26 3:27 UTC (permalink / raw) To: outreachy, Greg Kroah-Hartman, linux-staging, linux-kernel Current implementation of function ieee80211_tkip_null simply returns back to the caller without any useful instruction executions. This makes the function call and its implementation unnecessary and should be removed. Signed-off-by: Deepak R Varma <drv@mailo.com> --- drivers/staging/rtl8192u/ieee80211/ieee80211.h | 3 --- drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c | 6 ------ drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 --- 3 files changed, 12 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h index 9cd4b1896745..00c07455cbb3 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h @@ -232,8 +232,6 @@ struct cb_desc { #define ieee80211_ccmp_null ieee80211_ccmp_null_rsl -#define ieee80211_tkip_null ieee80211_tkip_null_rsl - #define free_ieee80211 free_ieee80211_rsl #define alloc_ieee80211 alloc_ieee80211_rsl @@ -2256,7 +2254,6 @@ void ieee80211_ps_tx_ack(struct ieee80211_device *ieee, short success); void softmac_mgmt_xmit(struct sk_buff *skb, struct ieee80211_device *ieee); /* ieee80211_crypt_ccmp&tkip&wep.c */ -void ieee80211_tkip_null(void); int ieee80211_crypto_init(void); void ieee80211_crypto_deinit(void); diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c index 7b120b8cb982..9bfd24ad46b6 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c @@ -716,9 +716,3 @@ void ieee80211_crypto_tkip_exit(void) { ieee80211_unregister_crypto_ops(&ieee80211_crypt_tkip); } - -void ieee80211_tkip_null(void) -{ -// printk("============>%s()\n", __func__); - return; -} diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c index b94fe9b449b6..3f93939bc4ee 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c @@ -159,9 +159,6 @@ struct net_device *alloc_ieee80211(int sizeof_priv) ieee->last_packet_time[i] = 0; } -/* These function were added to load crypte module autoly */ - ieee80211_tkip_null(); - return dev; failed: -- 2.30.2 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation 2022-10-26 3:27 ` [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation Deepak R Varma @ 2022-10-26 10:44 ` Greg Kroah-Hartman 2022-10-26 12:12 ` Deepak R Varma 0 siblings, 1 reply; 14+ messages in thread From: Greg Kroah-Hartman @ 2022-10-26 10:44 UTC (permalink / raw) To: Deepak R Varma; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 26, 2022 at 08:57:48AM +0530, Deepak R Varma wrote: > Current implementation of function ieee80211_tkip_null simply returns > back to the caller without any useful instruction executions. This makes > the function call and its implementation unnecessary and should be > removed. > > Signed-off-by: Deepak R Varma <drv@mailo.com> > --- > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 3 --- > drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c | 6 ------ > drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 --- > 3 files changed, 12 deletions(-) > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > index 9cd4b1896745..00c07455cbb3 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > @@ -232,8 +232,6 @@ struct cb_desc { > > #define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > -#define ieee80211_tkip_null ieee80211_tkip_null_rsl > - > #define free_ieee80211 free_ieee80211_rsl > #define alloc_ieee80211 alloc_ieee80211_rsl > > @@ -2256,7 +2254,6 @@ void ieee80211_ps_tx_ack(struct ieee80211_device *ieee, short success); > void softmac_mgmt_xmit(struct sk_buff *skb, struct ieee80211_device *ieee); > > /* ieee80211_crypt_ccmp&tkip&wep.c */ > -void ieee80211_tkip_null(void); > > int ieee80211_crypto_init(void); > void ieee80211_crypto_deinit(void); > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c > index 7b120b8cb982..9bfd24ad46b6 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c > @@ -716,9 +716,3 @@ void ieee80211_crypto_tkip_exit(void) > { > ieee80211_unregister_crypto_ops(&ieee80211_crypt_tkip); > } > - > -void ieee80211_tkip_null(void) > -{ > -// printk("============>%s()\n", __func__); > - return; > -} > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c > index b94fe9b449b6..3f93939bc4ee 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c > @@ -159,9 +159,6 @@ struct net_device *alloc_ieee80211(int sizeof_priv) > ieee->last_packet_time[i] = 0; > } > > -/* These function were added to load crypte module autoly */ > - ieee80211_tkip_null(); > - Ah, but this was created on purpose, did you now break the module autoloading here? Is this built as a separate module or is it all one big module? You need to explain why this does not change functionality in your changelog text. thanks, greg k-h ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation 2022-10-26 10:44 ` Greg Kroah-Hartman @ 2022-10-26 12:12 ` Deepak R Varma 0 siblings, 0 replies; 14+ messages in thread From: Deepak R Varma @ 2022-10-26 12:12 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 26, 2022 at 12:44:36PM +0200, Greg Kroah-Hartman wrote: > On Wed, Oct 26, 2022 at 08:57:48AM +0530, Deepak R Varma wrote: > > Current implementation of function ieee80211_tkip_null simply returns > > back to the caller without any useful instruction executions. This makes > > the function call and its implementation unnecessary and should be > > removed. > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > --- > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 3 --- > > drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c | 6 ------ > > drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 --- > > 3 files changed, 12 deletions(-) > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > index 9cd4b1896745..00c07455cbb3 100644 > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > @@ -232,8 +232,6 @@ struct cb_desc { > > > > #define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > -#define ieee80211_tkip_null ieee80211_tkip_null_rsl > > - > > #define free_ieee80211 free_ieee80211_rsl > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > @@ -2256,7 +2254,6 @@ void ieee80211_ps_tx_ack(struct ieee80211_device *ieee, short success); > > void softmac_mgmt_xmit(struct sk_buff *skb, struct ieee80211_device *ieee); > > > > /* ieee80211_crypt_ccmp&tkip&wep.c */ > > -void ieee80211_tkip_null(void); > > > > int ieee80211_crypto_init(void); > > void ieee80211_crypto_deinit(void); > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c > > index 7b120b8cb982..9bfd24ad46b6 100644 > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c > > @@ -716,9 +716,3 @@ void ieee80211_crypto_tkip_exit(void) > > { > > ieee80211_unregister_crypto_ops(&ieee80211_crypt_tkip); > > } > > - > > -void ieee80211_tkip_null(void) > > -{ > > -// printk("============>%s()\n", __func__); > > - return; > > -} > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c > > index b94fe9b449b6..3f93939bc4ee 100644 > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c > > @@ -159,9 +159,6 @@ struct net_device *alloc_ieee80211(int sizeof_priv) > > ieee->last_packet_time[i] = 0; > > } > > > > -/* These function were added to load crypte module autoly */ > > - ieee80211_tkip_null(); > > - > > Ah, but this was created on purpose, did you now break the module > autoloading here? Is this built as a separate module or is it all one > big module? Looks like a one big module on the kernel I built post this change. I looked into the modules.dep file for dependencies. I was able to load the driver r8192u_usb using modprobe without any issues. Let me know if I should be looking at something else to validate your concern. > > You need to explain why this does not change functionality in your > changelog text. Okay. I will work on it and send in a revision. > > thanks, > > greg k-h > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-26 3:26 [PATCH 0/2] staging: rtl8192u: unused code cleanup Deepak R Varma 2022-10-26 3:27 ` [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation Deepak R Varma @ 2022-10-26 3:28 ` Deepak R Varma 2022-10-26 13:14 ` Greg Kroah-Hartman 1 sibling, 1 reply; 14+ messages in thread From: Deepak R Varma @ 2022-10-26 3:28 UTC (permalink / raw) To: outreachy, Greg Kroah-Hartman, linux-staging, linux-kernel Pre-processor macros that are defined but are never used should be cleaned up to avoid unexpected usage. Signed-off-by: Deepak R Varma <drv@mailo.com> --- drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h index 00c07455cbb3..0b3dda59d7c0 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h @@ -230,8 +230,6 @@ struct cb_desc { #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl - #define free_ieee80211 free_ieee80211_rsl #define alloc_ieee80211 alloc_ieee80211_rsl -- 2.30.2 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-26 3:28 ` [PATCH 2/2] staging: rtl8192u: remove unused macro definition Deepak R Varma @ 2022-10-26 13:14 ` Greg Kroah-Hartman 2022-10-26 13:44 ` Deepak R Varma 0 siblings, 1 reply; 14+ messages in thread From: Greg Kroah-Hartman @ 2022-10-26 13:14 UTC (permalink / raw) To: Deepak R Varma; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > Pre-processor macros that are defined but are never used should be > cleaned up to avoid unexpected usage. > > Signed-off-by: Deepak R Varma <drv@mailo.com> > --- > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > index 00c07455cbb3..0b3dda59d7c0 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > @@ -230,8 +230,6 @@ struct cb_desc { > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > - > #define free_ieee80211 free_ieee80211_rsl > #define alloc_ieee80211 alloc_ieee80211_rsl These #defines are a mess, please look into unwinding them as they should not be needed at all. thanks, greg k-h ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-26 13:14 ` Greg Kroah-Hartman @ 2022-10-26 13:44 ` Deepak R Varma 2022-10-26 13:55 ` Greg Kroah-Hartman 0 siblings, 1 reply; 14+ messages in thread From: Deepak R Varma @ 2022-10-26 13:44 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > Pre-processor macros that are defined but are never used should be > > cleaned up to avoid unexpected usage. > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > --- > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > index 00c07455cbb3..0b3dda59d7c0 100644 > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > @@ -230,8 +230,6 @@ struct cb_desc { > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > - > > #define free_ieee80211 free_ieee80211_rsl > > #define alloc_ieee80211 alloc_ieee80211_rsl > > These #defines are a mess, please look into unwinding them as they > should not be needed at all. Hello Greg, I would like to know what you mean by "unwind them". Is there a documentation or past commit that I can review to understand the expectations better? Thank you, ./drv > > thanks, > > greg k-h > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-26 13:44 ` Deepak R Varma @ 2022-10-26 13:55 ` Greg Kroah-Hartman 2022-10-27 19:33 ` Deepak R Varma 0 siblings, 1 reply; 14+ messages in thread From: Greg Kroah-Hartman @ 2022-10-26 13:55 UTC (permalink / raw) To: Deepak R Varma; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > Pre-processor macros that are defined but are never used should be > > > cleaned up to avoid unexpected usage. > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > --- > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > 1 file changed, 2 deletions(-) > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > - > > > #define free_ieee80211 free_ieee80211_rsl > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > These #defines are a mess, please look into unwinding them as they > > should not be needed at all. > > Hello Greg, > I would like to know what you mean by "unwind them". Is there a documentation or past > commit that I can review to understand the expectations better? Look at them and try to figure out why they are there, and then work to remove them entirely. A define like this is very odd in the kernel, it should not be needed at all, right? thanks, greg k-h ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-26 13:55 ` Greg Kroah-Hartman @ 2022-10-27 19:33 ` Deepak R Varma 2022-10-31 14:45 ` Deepak R Varma 0 siblings, 1 reply; 14+ messages in thread From: Deepak R Varma @ 2022-10-27 19:33 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 26, 2022 at 03:55:01PM +0200, Greg Kroah-Hartman wrote: > On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > > Pre-processor macros that are defined but are never used should be > > > > cleaned up to avoid unexpected usage. > > > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > > --- > > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > > 1 file changed, 2 deletions(-) > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > - > > > > #define free_ieee80211 free_ieee80211_rsl > > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > > > These #defines are a mess, please look into unwinding them as they > > > should not be needed at all. > > > > Hello Greg, > > I would like to know what you mean by "unwind them". Is there a documentation or past > > commit that I can review to understand the expectations better? > > Look at them and try to figure out why they are there, and then work to > remove them entirely. A define like this is very odd in the kernel, it > should not be needed at all, right? Hello Greg, I will look into these additional macros soon and send any further edits as a separate patch (set). Is the current patch set with 2 patches acceptable? Also, I am trying to get Coccinelle to work on my machine. Trying to work through strange issues. I will work on the macro unwinding task you suggested once a get the Coccinelle task completed. Thank you, ./drv > > thanks, > > greg k-h > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-27 19:33 ` Deepak R Varma @ 2022-10-31 14:45 ` Deepak R Varma 2022-10-31 14:55 ` Julia Lawall 0 siblings, 1 reply; 14+ messages in thread From: Deepak R Varma @ 2022-10-31 14:45 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: outreachy, linux-staging, linux-kernel On Fri, Oct 28, 2022 at 01:03:36AM +0530, Deepak Varma wrote: > On Wed, Oct 26, 2022 at 03:55:01PM +0200, Greg Kroah-Hartman wrote: > > On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > > > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > > > Pre-processor macros that are defined but are never used should be > > > > > cleaned up to avoid unexpected usage. > > > > > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > > > --- > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > > > 1 file changed, 2 deletions(-) > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > > - > > > > > #define free_ieee80211 free_ieee80211_rsl > > > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > > > > > These #defines are a mess, please look into unwinding them as they > > > > should not be needed at all. > > > > > > Hello Greg, > > > I would like to know what you mean by "unwind them". Is there a documentation or past > > > commit that I can review to understand the expectations better? > > > > Look at them and try to figure out why they are there, and then work to > > remove them entirely. A define like this is very odd in the kernel, it > > should not be needed at all, right? > > Hello Greg, > I will look into these additional macros soon and send any further edits as a > separate patch (set). Is the current patch set with 2 patches acceptable? > > Also, I am trying to get Coccinelle to work on my machine. Trying to work > through strange issues. I will work on the macro unwinding task you suggested > once a get the Coccinelle task completed. Hello Greg, Most of these macro defines appear to be unused in the module anywhere. I can simply delete the #defines for these and let the original function names be retained. The other way is to rename the functions by the defined value. So, we will have to make the code change to use the foo_rsl symbol names at all places. This will be a bigger change involving the API name change itself. I am unable to determine the initial intention as to why these #defines were added. Can you please suggest what would be the recommended way for the clean up of these unused macros? Thank you, ./drv > > Thank you, > ./drv > > > > > thanks, > > > > greg k-h > > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-31 14:45 ` Deepak R Varma @ 2022-10-31 14:55 ` Julia Lawall 2022-10-31 15:19 ` Deepak R Varma 0 siblings, 1 reply; 14+ messages in thread From: Julia Lawall @ 2022-10-31 14:55 UTC (permalink / raw) To: Deepak R Varma; +Cc: Greg Kroah-Hartman, outreachy, linux-staging, linux-kernel On Mon, 31 Oct 2022, Deepak R Varma wrote: > On Fri, Oct 28, 2022 at 01:03:36AM +0530, Deepak Varma wrote: > > On Wed, Oct 26, 2022 at 03:55:01PM +0200, Greg Kroah-Hartman wrote: > > > On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > > > > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > > > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > > > > Pre-processor macros that are defined but are never used should be > > > > > > cleaned up to avoid unexpected usage. > > > > > > > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > > > > --- > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > > > > 1 file changed, 2 deletions(-) > > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > > > - > > > > > > #define free_ieee80211 free_ieee80211_rsl > > > > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > > > > > > > These #defines are a mess, please look into unwinding them as they > > > > > should not be needed at all. > > > > > > > > Hello Greg, > > > > I would like to know what you mean by "unwind them". Is there a documentation or past > > > > commit that I can review to understand the expectations better? > > > > > > Look at them and try to figure out why they are there, and then work to > > > remove them entirely. A define like this is very odd in the kernel, it > > > should not be needed at all, right? > > > > Hello Greg, > > I will look into these additional macros soon and send any further edits as a > > separate patch (set). Is the current patch set with 2 patches acceptable? > > > > Also, I am trying to get Coccinelle to work on my machine. Trying to work > > through strange issues. I will work on the macro unwinding task you suggested > > once a get the Coccinelle task completed. > > Hello Greg, > Most of these macro defines appear to be unused in the module anywhere. > I can simply delete the #defines for these and let the original function > names be retained. > The other way is to rename the functions by the defined value. So, we will have > to make the code change to use the foo_rsl symbol names at all places. This will > be a bigger change involving the API name change itself. I'm not sure to understand. In the case of #define abc def If abc is never used, it would seem that you could just remove the macro definition. If abc is used, one might consider whether the renaming is worth it, or whether the abc's should be changed to def. Or maybe def is a very simple function, that just calls some standard kernel function like kfree, in which can you could get rid of both abc and def everywhere and just use kfree. It is often better to use standard functions, because it makes it easier for people to understand immediately what is going on. julia > > I am unable to determine the initial intention as to why these #defines were > added. Can you please suggest what would be the recommended way for the clean up > of these unused macros? > > Thank you, > ./drv > > > > > Thank you, > > ./drv > > > > > > > > thanks, > > > > > > greg k-h > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-31 14:55 ` Julia Lawall @ 2022-10-31 15:19 ` Deepak R Varma 2022-10-31 15:37 ` Julia Lawall 0 siblings, 1 reply; 14+ messages in thread From: Deepak R Varma @ 2022-10-31 15:19 UTC (permalink / raw) To: Julia Lawall; +Cc: Greg Kroah-Hartman, outreachy, linux-staging, linux-kernel On Mon, Oct 31, 2022 at 03:55:59PM +0100, Julia Lawall wrote: > > > On Mon, 31 Oct 2022, Deepak R Varma wrote: > > > On Fri, Oct 28, 2022 at 01:03:36AM +0530, Deepak Varma wrote: > > > On Wed, Oct 26, 2022 at 03:55:01PM +0200, Greg Kroah-Hartman wrote: > > > > On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > > > > > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > > > > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > > > > > Pre-processor macros that are defined but are never used should be > > > > > > > cleaned up to avoid unexpected usage. > > > > > > > > > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > > > > > --- > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > > > > > 1 file changed, 2 deletions(-) > > > > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > > > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > > > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > > > > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > > > > - > > > > > > > #define free_ieee80211 free_ieee80211_rsl > > > > > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > > > > > > > > > These #defines are a mess, please look into unwinding them as they > > > > > > should not be needed at all. > > > > > > > > > > Hello Greg, > > > > > I would like to know what you mean by "unwind them". Is there a documentation or past > > > > > commit that I can review to understand the expectations better? > > > > > > > > Look at them and try to figure out why they are there, and then work to > > > > remove them entirely. A define like this is very odd in the kernel, it > > > > should not be needed at all, right? > > > > > > Hello Greg, > > > I will look into these additional macros soon and send any further edits as a > > > separate patch (set). Is the current patch set with 2 patches acceptable? > > > > > > Also, I am trying to get Coccinelle to work on my machine. Trying to work > > > through strange issues. I will work on the macro unwinding task you suggested > > > once a get the Coccinelle task completed. > > > > Hello Greg, > > Most of these macro defines appear to be unused in the module anywhere. > > I can simply delete the #defines for these and let the original function > > names be retained. > > The other way is to rename the functions by the defined value. So, we will have > > to make the code change to use the foo_rsl symbol names at all places. This will > > be a bigger change involving the API name change itself. > > I'm not sure to understand. In the case of My apologies for not being clear in my message. > > #define abc def > > If abc is never used, it would seem that you could just remove the macro > definition. The module uses abc at all places in the code. It gets simply replaced by abc_rsl symbol name at compile time. I am unable to determine the utility of this compile time conversion. So, to clean it up, we can continue to use abc in the code (by simply removing the #define line) or make code changes to use abc_rsl (makes the #define line redundant and be deleted). Hope this helps. My question: should we use abc or def in the code? The only hint I have form the code comment is this line: // added for kernel conflict > > If abc is used, one might consider whether the renaming is worth it, or > whether the abc's should be changed to def. Or maybe def is a very simple > function, that just calls some standard kernel function like kfree, in > which can you could get rid of both abc and def everywhere and just use > kfree. > > It is often better to use standard functions, because it makes it easier > for people to understand immediately what is going on. Thank you so much for the explanation. Since the initial intention is not clear to me, I am unable to decide the go forward name for these functions. May I request to look at one of the macro implementations and make suggestion? > > julia > > > > > > I am unable to determine the initial intention as to why these #defines were > > added. Can you please suggest what would be the recommended way for the clean up > > of these unused macros? > > > > Thank you, > > ./drv > > > > > > > > Thank you, > > > ./drv > > > > > > > > > > > thanks, > > > > > > > > greg k-h > > > > > > > > > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-31 15:19 ` Deepak R Varma @ 2022-10-31 15:37 ` Julia Lawall 2022-10-31 16:08 ` Deepak R Varma 0 siblings, 1 reply; 14+ messages in thread From: Julia Lawall @ 2022-10-31 15:37 UTC (permalink / raw) To: Deepak R Varma; +Cc: Greg Kroah-Hartman, outreachy, linux-staging, linux-kernel On Mon, 31 Oct 2022, Deepak R Varma wrote: > On Mon, Oct 31, 2022 at 03:55:59PM +0100, Julia Lawall wrote: > > > > > > On Mon, 31 Oct 2022, Deepak R Varma wrote: > > > > > On Fri, Oct 28, 2022 at 01:03:36AM +0530, Deepak Varma wrote: > > > > On Wed, Oct 26, 2022 at 03:55:01PM +0200, Greg Kroah-Hartman wrote: > > > > > On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > > > > > > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > > > > > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > > > > > > Pre-processor macros that are defined but are never used should be > > > > > > > > cleaned up to avoid unexpected usage. > > > > > > > > > > > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > > > > > > --- > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > > > > > > 1 file changed, 2 deletions(-) > > > > > > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > > > > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > > > > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > > > > > > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > > > > > - > > > > > > > > #define free_ieee80211 free_ieee80211_rsl > > > > > > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > > > > > > > > > > > These #defines are a mess, please look into unwinding them as they > > > > > > > should not be needed at all. > > > > > > > > > > > > Hello Greg, > > > > > > I would like to know what you mean by "unwind them". Is there a documentation or past > > > > > > commit that I can review to understand the expectations better? > > > > > > > > > > Look at them and try to figure out why they are there, and then work to > > > > > remove them entirely. A define like this is very odd in the kernel, it > > > > > should not be needed at all, right? > > > > > > > > Hello Greg, > > > > I will look into these additional macros soon and send any further edits as a > > > > separate patch (set). Is the current patch set with 2 patches acceptable? > > > > > > > > Also, I am trying to get Coccinelle to work on my machine. Trying to work > > > > through strange issues. I will work on the macro unwinding task you suggested > > > > once a get the Coccinelle task completed. > > > > > > Hello Greg, > > > Most of these macro defines appear to be unused in the module anywhere. > > > I can simply delete the #defines for these and let the original function > > > names be retained. > > > The other way is to rename the functions by the defined value. So, we will have > > > to make the code change to use the foo_rsl symbol names at all places. This will > > > be a bigger change involving the API name change itself. > > > > I'm not sure to understand. In the case of > > My apologies for not being clear in my message. > > > > #define abc def > > > > If abc is never used, it would seem that you could just remove the macro > > definition. > > The module uses abc at all places in the code. It gets simply replaced by > abc_rsl symbol name at compile time. I am unable to determine the utility of > this compile time conversion. So, to clean it up, we can continue to use abc > in the code (by simply removing the #define line) or make code changes to use > abc_rsl (makes the #define line redundant and be deleted). > > Hope this helps. > My question: should we use abc or def in the code? The only hint I have form the > code comment is this line: > // added for kernel conflict > > > > > If abc is used, one might consider whether the renaming is worth it, or > > whether the abc's should be changed to def. Or maybe def is a very simple > > function, that just calls some standard kernel function like kfree, in > > which can you could get rid of both abc and def everywhere and just use > > kfree. > > > > It is often better to use standard functions, because it makes it easier > > for people to understand immediately what is going on. > > Thank you so much for the explanation. Since the initial intention is not clear > to me, I am unable to decide the go forward name for these functions. > > May I request to look at one of the macro implementations and make suggestion? Ah, ok I see, these macros replace both the names at the uses of eg free_ieee80211, but also the name at the point of the definition. But the name free_ieee80211 isn't used elsewhere in the kernel source tree, so it would seem that changing the name is useless. Maybe the purpose was to link this code with some other code that also uses the name free_ieee80211, but that other code doesn't seem to be part of the kernel. At least, you could propose a patch that removes these definitions and explains clearly what the impact is, and see if the maintainers of the driver complain. julia > > > > > julia > > > > > > > > > > I am unable to determine the initial intention as to why these #defines were > > > added. Can you please suggest what would be the recommended way for the clean up > > > of these unused macros? > > > > > > Thank you, > > > ./drv > > > > > > > > > > > Thank you, > > > > ./drv > > > > > > > > > > > > > > thanks, > > > > > > > > > > greg k-h > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] staging: rtl8192u: remove unused macro definition 2022-10-31 15:37 ` Julia Lawall @ 2022-10-31 16:08 ` Deepak R Varma 0 siblings, 0 replies; 14+ messages in thread From: Deepak R Varma @ 2022-10-31 16:08 UTC (permalink / raw) To: Julia Lawall; +Cc: Greg Kroah-Hartman, outreachy, linux-staging, linux-kernel On Mon, Oct 31, 2022 at 04:37:38PM +0100, Julia Lawall wrote: > > > On Mon, 31 Oct 2022, Deepak R Varma wrote: > > > On Mon, Oct 31, 2022 at 03:55:59PM +0100, Julia Lawall wrote: > > > > > > > > > On Mon, 31 Oct 2022, Deepak R Varma wrote: > > > > > > > On Fri, Oct 28, 2022 at 01:03:36AM +0530, Deepak Varma wrote: > > > > > On Wed, Oct 26, 2022 at 03:55:01PM +0200, Greg Kroah-Hartman wrote: > > > > > > On Wed, Oct 26, 2022 at 07:14:43PM +0530, Deepak R Varma wrote: > > > > > > > On Wed, Oct 26, 2022 at 03:14:23PM +0200, Greg Kroah-Hartman wrote: > > > > > > > > On Wed, Oct 26, 2022 at 08:58:44AM +0530, Deepak R Varma wrote: > > > > > > > > > Pre-processor macros that are defined but are never used should be > > > > > > > > > cleaned up to avoid unexpected usage. > > > > > > > > > > > > > > > > > > Signed-off-by: Deepak R Varma <drv@mailo.com> > > > > > > > > > --- > > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 2 -- > > > > > > > > > 1 file changed, 2 deletions(-) > > > > > > > > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > > > index 00c07455cbb3..0b3dda59d7c0 100644 > > > > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h > > > > > > > > > @@ -230,8 +230,6 @@ struct cb_desc { > > > > > > > > > #define ieee80211_unregister_crypto_ops ieee80211_unregister_crypto_ops_rsl > > > > > > > > > #define ieee80211_get_crypto_ops ieee80211_get_crypto_ops_rsl > > > > > > > > > > > > > > > > > > -#define ieee80211_ccmp_null ieee80211_ccmp_null_rsl > > > > > > > > > - > > > > > > > > > #define free_ieee80211 free_ieee80211_rsl > > > > > > > > > #define alloc_ieee80211 alloc_ieee80211_rsl > > > > > > > > > > > > > > > > These #defines are a mess, please look into unwinding them as they > > > > > > > > should not be needed at all. > > > > > > > > > > > > > > Hello Greg, > > > > > > > I would like to know what you mean by "unwind them". Is there a documentation or past > > > > > > > commit that I can review to understand the expectations better? > > > > > > > > > > > > Look at them and try to figure out why they are there, and then work to > > > > > > remove them entirely. A define like this is very odd in the kernel, it > > > > > > should not be needed at all, right? > > > > > > > > > > Hello Greg, > > > > > I will look into these additional macros soon and send any further edits as a > > > > > separate patch (set). Is the current patch set with 2 patches acceptable? > > > > > > > > > > Also, I am trying to get Coccinelle to work on my machine. Trying to work > > > > > through strange issues. I will work on the macro unwinding task you suggested > > > > > once a get the Coccinelle task completed. > > > > > > > > Hello Greg, > > > > Most of these macro defines appear to be unused in the module anywhere. > > > > I can simply delete the #defines for these and let the original function > > > > names be retained. > > > > The other way is to rename the functions by the defined value. So, we will have > > > > to make the code change to use the foo_rsl symbol names at all places. This will > > > > be a bigger change involving the API name change itself. > > > > > > I'm not sure to understand. In the case of > > > > My apologies for not being clear in my message. > > > > > > #define abc def > > > > > > If abc is never used, it would seem that you could just remove the macro > > > definition. > > > > The module uses abc at all places in the code. It gets simply replaced by > > abc_rsl symbol name at compile time. I am unable to determine the utility of > > this compile time conversion. So, to clean it up, we can continue to use abc > > in the code (by simply removing the #define line) or make code changes to use > > abc_rsl (makes the #define line redundant and be deleted). > > > > Hope this helps. > > My question: should we use abc or def in the code? The only hint I have form the > > code comment is this line: > > // added for kernel conflict > > > > > > > > If abc is used, one might consider whether the renaming is worth it, or > > > whether the abc's should be changed to def. Or maybe def is a very simple > > > function, that just calls some standard kernel function like kfree, in > > > which can you could get rid of both abc and def everywhere and just use > > > kfree. > > > > > > It is often better to use standard functions, because it makes it easier > > > for people to understand immediately what is going on. > > > > Thank you so much for the explanation. Since the initial intention is not clear > > to me, I am unable to decide the go forward name for these functions. > > > > May I request to look at one of the macro implementations and make suggestion? > > Ah, ok I see, these macros replace both the names at the uses of eg > free_ieee80211, but also the name at the point of the definition. > > But the name free_ieee80211 isn't used elsewhere in the kernel source > tree, so it would seem that changing the name is useless. > > Maybe the purpose was to link this code with some other code that also > uses the name free_ieee80211, but that other code doesn't seem to be part > of the kernel. > > At least, you could propose a patch that removes these definitions and > explains clearly what the impact is, and see if the maintainers of the > driver complain. Thank you so much. I will send in the proposed patch shortly. ./drv > > julia > > > > > > > > > julia > > > > > > > > > > > > > > I am unable to determine the initial intention as to why these #defines were > > > > added. Can you please suggest what would be the recommended way for the clean up > > > > of these unused macros? > > > > > > > > Thank you, > > > > ./drv > > > > > > > > > > > > > > Thank you, > > > > > ./drv > > > > > > > > > > > > > > > > > thanks, > > > > > > > > > > > > greg k-h > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2022-10-31 16:08 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-10-26 3:26 [PATCH 0/2] staging: rtl8192u: unused code cleanup Deepak R Varma 2022-10-26 3:27 ` [PATCH 1/2] staging: rtl8192u: remove unnecessary function implementation Deepak R Varma 2022-10-26 10:44 ` Greg Kroah-Hartman 2022-10-26 12:12 ` Deepak R Varma 2022-10-26 3:28 ` [PATCH 2/2] staging: rtl8192u: remove unused macro definition Deepak R Varma 2022-10-26 13:14 ` Greg Kroah-Hartman 2022-10-26 13:44 ` Deepak R Varma 2022-10-26 13:55 ` Greg Kroah-Hartman 2022-10-27 19:33 ` Deepak R Varma 2022-10-31 14:45 ` Deepak R Varma 2022-10-31 14:55 ` Julia Lawall 2022-10-31 15:19 ` Deepak R Varma 2022-10-31 15:37 ` Julia Lawall 2022-10-31 16:08 ` Deepak R Varma
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).