Re: [PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Anirudh Rayabharam
On Fri, Feb 12, 2021 at 03:43:10PM +0100, Greg KH wrote: > On Fri, Feb 12, 2021 at 08:00:25PM +0530, Anirudh Rayabharam wrote: > > On Thu, Feb 11, 2021 at 09:35:27PM +0100, Greg KH wrote: > > > On Fri, Feb 12, 2021 at 01:59:08AM +0530, Anirudh Rayabharam wrote: > > &

[PATCH v2] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Anirudh Rayabharam
:right side has type restricted __le32 wimax/i2400m/fw.c:196:32: warning: restricted __le32 degrades to integer wimax/i2400m/fw.c:196:47: warning: restricted __le32 degrades to integer wimax/i2400m/fw.c:196:66: warning: restricted __le32 degrades to integer Signed-off-by: Anirudh Rayabharam

Re: [PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Anirudh Rayabharam
On Thu, Feb 11, 2021 at 09:35:27PM +0100, Greg KH wrote: > On Fri, Feb 12, 2021 at 01:59:08AM +0530, Anirudh Rayabharam wrote: > > Fix sparse byte-order warnings in the i2400m_bm_cmd_prepare() > > function: > > > > wimax/i2400m/fw.c:194:36: warning: restricted __le32 de

[PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-11 Thread Anirudh Rayabharam
:right side has type restricted __le32 wimax/i2400m/fw.c:196:32: warning: restricted __le32 degrades to integer wimax/i2400m/fw.c:196:47: warning: restricted __le32 degrades to integer wimax/i2400m/fw.c:196:66: warning: restricted __le32 degrades to integer Signed-off-by: Anirudh Rayabharam

[PATCH] staging: rtl8192u/ieee80211: fix switch case indentation

2021-01-31 Thread Anirudh Rayabharam
Make switch and case to be at the same indent. This fixes the checkpatch error "switch and case should be at the same indent". Signed-off-by: Anirudh Rayabharam --- .../staging/rtl8192u/ieee80211/ieee80211_tx.c | 28 +-- 1 file changed, 14 insertions(+), 14 deletion

[PATCH v2] staging: wimax/i2400m: fix pointer declaration style

2021-01-30 Thread Anirudh Rayabharam
Fix a couple of pointer declarations where the pointer qualifier '*' is not attached to the variable name. This fixes the checkpatch error: "foo * bar" should be "foo *bar". Signed-off-by: Anirudh Rayabharam --- v2: Minor corrections in the commit message. --- dri

[PATCH] staging: wimax/i2400m: fix pointer declaration style

2021-01-30 Thread Anirudh Rayabharam
Fix a couple of pointer declarataions where the pointer qualifier '*' is not attached to the variable. This fixes the checkpatch error: "'foo * bar' should be 'foo *bar'". Signed-off-by: Anirudh Rayabharam --- drivers/staging/wimax/i2400m/rx.c | 4

Re: [PATCH v2] staging: rtl8723bs: core: fix line over 80 characters warning

2019-03-30 Thread Anirudh Rayabharam
On Wed, Mar 27, 2019 at 11:49:07PM +0530, Anirudh Rayabharam wrote: > Checkpatch.pl complains that these lines are over 80 characters. Use the > "psecuritypriv" pointer for consistency, remove unnecessary parantheses > and fix the alignment. > > This patch just cleans u

[PATCH v2] staging: rtl8723bs: core: fix line over 80 characters warning

2019-03-27 Thread Anirudh Rayabharam
Checkpatch.pl complains that these lines are over 80 characters. Use the "psecuritypriv" pointer for consistency, remove unnecessary parantheses and fix the alignment. This patch just cleans up a condition, it doesn't affect runtime. Signed-off-by: Anirudh Rayabharam --- v2:

[PATCH] staging: rtl8723bs: core: fix line over 80 characters warning

2019-03-26 Thread Anirudh Rayabharam
Shorten the expression by re-using the part that was already computed to fix the line over 80 characters warning reported by checkpatch.pl. Signed-off-by: Anirudh Rayabharam --- drivers/staging/rtl8723bs/core/rtw_ap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a