[PATCH v2] staging: fbtft: differentiate between buffer and data types to fix sparse warning

2017-10-17 Thread Alfonso Lima Astor
[usertype] The solution is to add an extra parameter to the macro to differentiate between buffer type and data type. Cc: Thomas Petazzoni Signed-off-by: Alfonso Lima Astor --- changes for v2: Add maintainer to CC and add result of macro below '---' to ease review

[PATCH v2] staging: fbtft: differentiate between buffer and data types to fix sparse warning

2017-10-04 Thread Alfonso Lima Astor
[usertype] The solution is to add an extra parameter to the macro to differentiate between buffer type and data type. Cc: Thomas Petazzoni Signed-off-by: Alfonso Lima Astor --- changes for v2: Add maintainer to CC and add result of macro below '---' to ease review

[PATCH] staging: fbtft: diferenciate between buffer and data types to fix sparse warning

2017-09-27 Thread Alfonso Lima Astor
[usertype] The solution is to add an extra parameter to the macro to diferenciate between buffer type and data type. Signed-off-by: Alfonso Lima Astor --- drivers/staging/fbtft/fbtft-bus.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers

[PATCH] staging: wlan-ng: convert variable to le16 for assignment

2017-09-26 Thread Alfonso Lima Astor
/prism2sta.c:1450:29:got restricted __le16 [usertype] linkstatus Signed-off-by: Alfonso Lima Astor --- drivers/staging/wlan-ng/prism2sta.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c index c9df450

[PATCH v2] staging: wlan-ng: Fix endianness warnings

2017-07-18 Thread Alfonso Lima Astor
These variables were set to u16 and u32 although they always hold a little endian value. This patch fixes multiple sparse warnings like: drivers/staging/wlan-ng/prism2sta.c:375:46: warning: cast to restricted __le16 Signed-off-by: Alfonso Lima Astor --- v2: Resubmit patch that wouldn&#

Re: [PATCH] staging: rtl8188eu: Macro should be enclosed

2017-04-09 Thread Alfonso Lima
On Sat, Apr 01, 2017 at 09:24:58AM +0200, Sebastian Haas wrote: > Hi Alfonso, > > On Wed, Mar 29, 2017 at 07:50:11PM +0100, alfonsolimaas...@gmail.com wrote: > > diff --git a/drivers/staging/rtl8188eu/include/odm_debug.h > > b/drivers/staging/rtl8188eu/include/odm_debug.h > > index 687ff3e..fd92f

[PATCH] staging: rtl8188eu: Remove extra spaces

2017-03-30 Thread Alfonso Lima Astor
Coding style problem detected by checkpatch.pl ERROR: space prohibited before that ',' Signed-off-by: Alfonso Lima Astor --- drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 2 +- drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c| 2 +- 2 files changed, 2 insertions(+), 2

[PATCH v2] staging: rtl8188eu: Macros with complex values should be enclosed in parentheses

2017-03-29 Thread Alfonso Lima Astor
This macro is not used and also had a style error. I have run grep and compiled the module to be sure. Signed-off-by: Alfonso Lima Astor --- drivers/staging/rtl8188eu/include/rtw_ioctl.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_ioctl.h b

[PATCH] staging: rtl8188eu: Macro should be enclosed

2017-03-29 Thread Alfonso Lima Astor
Fix style error pointed out by checkpatch.pl: ERROR: Macros with complex values should be enclosed in parentheses I have compiled the module just to be sure. I haven't run it because I haven't changed any funcionality. Signed-off-by: Alfonso Lima Astor --- drivers/staging/rtl8188

[PATCH 1/1] staging: rtl8188eu: Macros with complex values should be enclosed in parentheses

2017-03-28 Thread Alfonso Lima Astor
Fix coding style issue Signed-off-by: Alfonso Lima Astor --- drivers/staging/rtl8188eu/include/rtw_ioctl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_ioctl.h b/drivers/staging/rtl8188eu/include/rtw_ioctl.h index a6b1c85