[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 <thomas.petazz...@free-electrons.com> Signed-off-by: Alfonso Lima Astor <alfonsolimaas...@gmail.com> --- changes for v2: Add maintainer t

[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 <thomas.petazz...@free-electrons.com> Signed-off-by: Alfonso Lima Astor <alfonsolimaas...@gmail.com> --- changes for v2: Add maintainer t

[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 <alfonsolimaas...@gmail.com> --- drivers/staging/fbtft/fbtft-bus.c | 24 1 file changed, 12 insertions(+), 12 del

[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 <alfonsolimaas...@gmail.com> --- 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/stagin

[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 <alfonsolimaas...@gmail.com>

[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 <alfonsolimaas...@gmail.com> --- drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 2 +- drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c| 2 +- 2 files chan

[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 <alfonsolimaas...@gmail.com> --- drivers/staging/rtl8188eu/include/rtw_ioctl.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/s

[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 <alfonsolimaas...@gmail.com> --- d

[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 <alfonsolimaas...@gmail.com> --- 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/i