Skip to contentSkip to conversation

Staging: rtl8188eu: Remove unused extern declarations.

PATCH In review Vatika Harlalka 2 participants 4 messages Mar 19, 2015 Staging

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Staging: rtl8188eu: Remove unused extern declarations.

These externs are defined but not used anywhere in the code.

---
 drivers/staging/rtl8188eu/include/rtw_security.h | 4 ----
 1 file changed, 4 deletions(-)
drivers/staging/rtl8188eu/include/rtw_security.h +0 -4
index 66d60aa..a00d1e6 100644
@@ -245,10 +245,6 @@ struct mic_data {
 };
 
 extern const u32 Te0[256];
-extern const u32 Te1[256];
-extern const u32 Te2[256];
-extern const u32 Te3[256];
-extern const u32 Te4[256];
 extern const u32 Td0[256];
 extern const u32 Td1[256];
 extern const u32 Td2[256];
Signature · 1 line
1.9.1
maintainer
#1 lore raw
Quoted 6 lines · Vatika Harlalka

These externs are defined but not used anywhere in the code.

---
 drivers/staging/rtl8188eu/include/rtw_security.h | 4 ----
 1 file changed, 4 deletions(-)

You sent 3 patches for the same file, but no clue as to which patches should be applied in which order :(

Please resend all of them as a patch series, properly numbered, so that I have a chance to get it correct.

thanks,

greg k-h

maintainer
#2 lore raw
Quoted 10 lines · Greg KH
Quoted 6 lines · Vatika Harlalka

These externs are defined but not used anywhere in the code.

---
 drivers/staging/rtl8188eu/include/rtw_security.h | 4 ----
 1 file changed, 4 deletions(-)

You sent 3 patches for the same file, but no clue as to which patches should be applied in which order :(

Should have said "driver", two patches were for the same file, which triggered this worry. Please resend all 3.

thanks,

greg k-h

#3 lore raw

Hey! :)
I've resent them as a patchset in order.
Sorry for the confusion caused!