[PATCH 0/2] staging: rtl8723au: include/rtl8723a_hal.h: fix firmware header struct

2015-06-14 Thread Daniele Alessandrelli
o u8. The second patch also changes struct rt_8723a_firmware_hdr by making its other fields use the __leXX types in order to fix some sparse warnings. Daniele Alessandrelli (2): staging: rtl8723au: include/rtl8723a_hal.h: fix Rsvd1 size in fw header struct staging: rtl8723au: include/rtl8

[PATCH 2/2] staging: rtl8723au: include/rtl8723a_hal.h: use __leXX types in fw header struct

2015-06-14 Thread Daniele Alessandrelli
/rtl8723a_hal_init.c:277:13: warning: cast to restricted __le16 Signed-off-by: Daniele Alessandrelli --- drivers/staging/rtl8723au/include/rtl8723a_hal.h | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8723au/include/rtl8723a_hal.h b/drivers

[PATCH 1/2] staging: rtl8723au: include/rtl8723a_hal.h: fix Rsvd1 size in fw header struct

2015-06-14 Thread Daniele Alessandrelli
The size of field Rsvd1 in struct rt_8723a_firmware_hdr should be u8 otherwise we exceeds the boundaries of "LONG WORD 0". This patch fixes the issue. Signed-off-by: Daniele Alessandrelli --- drivers/staging/rtl8723au/include/rtl8723a_hal.h | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH 2/2] staging: rtl8723au: include/rtl8723a_hal.h: use __leXX types in fw header struct

2015-06-15 Thread Daniele Alessandrelli
On June 15, 2015 7:47:41 AM GMT+02:00, Sudip Mukherjee wrote: >On Sun, Jun 14, 2015 at 11:26:35PM +0200, Daniele Alessandrelli wrote: >> This patch makes field in struct rt_8723a_firmware_hdr use >endianness-aware >> types (__leXX), thus fixing the followi

[PATCHv2 0/2] staging: rtl8723au: include/rtl8723a_hal.h: fix firmware header struct

2015-06-15 Thread Daniele Alessandrelli
the __leXX types in order to fix some sparse warnings. Daniele Alessandrelli (2): staging: rtl8723au: include/rtl8723a_hal.h: fix Rsvd1 size in fw header struct staging: rtl8723au: include/rtl8723a_hal.h: use __leXX types in fw header struct drivers/staging/rtl8723au/include/rtl8723a_

[PATCHv2 1/2] staging: rtl8723au: include/rtl8723a_hal.h: fix Rsvd1 size in fw header struct

2015-06-15 Thread Daniele Alessandrelli
The size of field Rsvd1 in struct rt_8723a_firmware_hdr should be u8 otherwise we exceeds the boundaries of "LONG WORD 0". This patch fixes the issue. Signed-off-by: Daniele Alessandrelli Acked-by: Jes Sorensen --- drivers/staging/rtl8723au/include/rtl8723a_hal.h | 2 +- 1 file

[PATCHv2 2/2] staging: rtl8723au: include/rtl8723a_hal.h: use __leXX types in fw header struct

2015-06-15 Thread Daniele Alessandrelli
/rtl8723a_hal_init.c:277:13: warning: cast to restricted __le16 Signed-off-by: Daniele Alessandrelli --- drivers/staging/rtl8723au/include/rtl8723a_hal.h | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) ==Changelog== v2: corrected style of multi-line comment v1: initial

[PATCH 0/2] staging: ft1000-pcmcia: ft1000_hw.c: fix checkpatch warnings

2015-02-03 Thread Daniele Alessandrelli
cally, the function ft1000_read_dsp_timer() is introduced to replace a recurring block of code used for reading the DSP timer value. Daniele Alessandrelli (2): ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring ft1000-pcmcia: ft1000_hw.c: code refact

[PATCH 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-02-03 Thread Daniele Alessandrelli
Add new function ft1000_read_dsp_timer() replacing recurring code block for reading DSP timer. Such code refactoring solves all remaining "line over 80 characters" warnings reported by checkpatch.pl. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_

[PATCH 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-02-03 Thread Daniele Alessandrelli
Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 521

[PATCHv2 0/2] staging: ft1000-pcmcia: ft1000_hw.c: fix checkpatch warnings

2015-02-10 Thread Daniele Alessandrelli
cally, the function ft1000_read_dsp_timer() is introduced to replace a recurring block of code used for reading the DSP timer value. This updated version of the patchset should apply cleanly to Greg's Tree. Daniele Alessandrelli (2): ft1000-pcmcia: ft1000_hw.c: fix style issues not

[PATCHv2 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-02-10 Thread Daniele Alessandrelli
Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 455

[PATCHv2 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-02-10 Thread Daniele Alessandrelli
Add new function ft1000_read_dsp_timer() replacing recurring code block for reading DSP timer. Such code refactoring solves all remaining "line over 80 characters" warnings reported by checkpatch.pl. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_

Re: [PATCHv2 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-02-11 Thread Daniele Alessandrelli
I'm sorry. I just realized I sent the wrong patch. This one is embarrassingly wrong :/ I'm going to resend the whole patchset tonight or tomorrow. Please excuse me for the mess :/ ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxd

[PATCHv3 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-02-11 Thread Daniele Alessandrelli
Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 455

[PATCHv3 0/2] staging: ft1000-pcmcia: ft1000_hw.c: fix checkpatch warnings

2015-02-11 Thread Daniele Alessandrelli
cally, the function ft1000_read_dsp_timer() is introduced to replace a recurring block of code used for reading the DSP timer value. This updated version of the patchset should apply cleanly to Greg's Tree. Daniele Alessandrelli (2): ft1000-pcmcia: ft1000_hw.c: fix style issues not

[PATCHv3 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-02-11 Thread Daniele Alessandrelli
Add new function ft1000_read_dsp_timer() replacing recurring code block for reading DSP timer. Such code refactoring solves all remaining "line over 80 characters" warnings reported by checkpatch.pl. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_

[PATCH] staging: rtl8723au: rtl8723a_hal_init.c: remove unnecessary braces

2015-02-22 Thread Daniele Alessandrelli
Fix all checkpatch "braces {} are not necessary" warnings for rtl8723au/hal/rtl8723a_hal_init.c Signed-off-by: Daniele Alessandrelli --- drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 40 --- 1 file changed, 14 insertions(+), 26 deletions(-) diff --git

[PATCHv4 0/2] staging: ft1000-pcmcia: ft1000_hw.c: fix all checkpatch warnings

2015-03-11 Thread Daniele Alessandrelli
cally, the function ft1000_read_dsp_timer() is introduced to replace a recurring block of code used for reading the DSP timer value. This updated version of the patchset should apply cleanly to Greg's staging-testing tree. Daniele Alessandrelli (2): ft1000-pcmcia: ft1000_hw.c: fix s

[PATCHv4 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-03-11 Thread Daniele Alessandrelli
Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 199

[PATCHv4 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-03-11 Thread Daniele Alessandrelli
Add new function ft1000_read_dsp_timer() replacing recurring code block for reading DSP timer. Such code refactoring solves all remaining "line over 80 characters" warnings reported by checkpatch.pl. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_

[PATCHv5 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-03-16 Thread Daniele Alessandrelli
Add new function ft1000_read_dsp_timer() replacing recurring code block for reading DSP timer. Such code refactoring solves all remaining "line over 80 characters" warnings reported by checkpatch.pl. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_

[PATCHv5 0/2] staging: ft1000-pcmcia: ft1000_hw.c: fix all checkpatch warnings

2015-03-16 Thread Daniele Alessandrelli
cally, the function ft1000_read_dsp_timer() is introduced to replace a recurring block of code used for reading the DSP timer value. This updated version of the patchset should apply cleanly to Greg's staging-testing tree. Thanks to Greg and Giedrius for the feedback they provided so

[PATCHv5 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-03-16 Thread Daniele Alessandrelli
Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 189