On Thu, Apr 14, 2022 at 11:18:38AM +0100, Colin Ian King wrote: > Don't populate the read-only array EWC11NHTCap on the stack but instead > make it static const. Also makes the object code a little smaller. Move > comment to clean up checkpatch warning. > > Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> > --- > drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c > index 358c35d9589c..b4cbe499ba37 100644 > --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c > +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c > @@ -480,7 +480,8 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u > } > memset(posHTCap, 0, *len); > if (pHT->ePeerHTSpecVer == HT_SPEC_VER_EWC) { > - u8 EWC11NHTCap[] = {0x00, 0x90, 0x4c, 0x33}; // For 11n EWC definition, 2007.07.17, by Emily > + /* For 11n EWC definition, 2007.07.17, by Emily */ This is not needed anymore, you know this... Just delete it please. thanks, greg k-h