This patch replaces WILC_WFI_set_cqm_rssi_config with
set_cqm_rssi_config to avoid CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperati
This patch replaces WILC_WFI_frame_register with
wilc_mgmt_frame_register to avoid CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/linux_wlan.c | 8
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 9 -
drivers/staging/wilc1000/wilc_wfi_cfgoper
This patch removes blank line after an open brace '{' found by checkpatch.pl.
CHECK: Blank lines aren't necessary after an open brace '{'
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1000/
This patch replaces PRINT_D with netdev_err which is used at error
checking.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/st
This patch replaces WILC_WFI_set_power_mgmt with set_power_mgmt to avoid
CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/d
This patch replaces WILC_WFI_DeInitHostInt with wilc_deinit_host_int to
avoid CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/linux_wlan.c | 6 +++---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +-
drivers/staging/wilc1000/wilc_wfi_cfgoperations.h | 2 +
This patch removes blank line before a close brace '{' found by checkpatch.pl
CHECK: Blank lines aren't necessary before a close brace '}'
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1000
This patch removes function declaration of WILC_WFI_frame_register in
linux_wlan.c file, then adds it in wilc_wfi_cfgoperations.h file.
The compilation warning occurs because it is assigned to incorrect
pointer type of second parameter of WILC_WFI_frame_register.
It is assigned with struct wireles
This patch replaces PRINT_D with netdev_info in wilc_free_wiphy function.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/staging
This patch removes useless comment that is containing "Bug"
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/host_interface.c | 14 --
drivers/staging/wilc1000/linux_mon.c | 2 --
drivers/staging/wilc1000/linux_wlan.c | 5 ++---
drivers/
This patch replaces s32RetStatus with result to avoid CamelCase in
wilc_msgqueue.c
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_msgqueue.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_msgqueue.c
b/drivers/s
This patch removes braces for single if statement blocks found by
checkpatch.pl
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff -
This patch removes #if 1, which is encapsulated some codes.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4
drivers/staging/wilc1000/wilc_wlan.c | 2 --
drivers/staging/wilc1000/wilc_wlan_cfg.c | 2 --
3 files changed, 8 deletions
This patch changes date type of s32Error as int.
The return type of WILC_WFI_InitHostInt and WILC_WFI_DeInitHostInt is int
so that data type of this variable is changed by int.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4 ++--
1 file changed, 2 insertion
This patch replaces WILC_WFI_InitHostInt with wilc_init_host_int to
avoid CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/linux_wlan.c | 2 +-
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +-
drivers/staging/wilc1000/wilc_wfi_cfgoperations.h | 2 +-
3 fil
This patch replaces WILC_WFI_WiphyRegister with wilc_create_wiphy to
avoid CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/linux_wlan.c | 2 +-
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4 ++--
drivers/staging/wilc1000/wilc_wfi_cfgoperations.h | 2 +-
3
This patch fixes NULL comparison style found by checkpatch.pl.
CHECK: Comparison to NULL could be written "!net"
CHECK: Comparison to NULL could be written "!net->ieee80211_ptr"
CHECK: Comparison to NULL could be written "!net->ieee80211_ptr->wiphy"
Signed-off-by: Chaehyun Lim
---
drivers/stagi
This patch replaces WILC_WFI_WiphyFree with wilc_free_wiphy to avoid
CamelCase.
Signed-off-by: Chaehyun Lim
---
drivers/staging/wilc1000/linux_wlan.c | 2 +-
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +-
drivers/staging/wilc1000/wilc_wfi_cfgoperations.h | 2 +-
3 files ch
On 09/19/2015 01:13 PM, punit vara wrote:
[PATCH] Staging: comedi: Fix block comment warning
[PATCH] Staging: comedi: Coding style warnings fix for block comments
___
devel mailing list
[email protected]
http://driverdev.linuxdriverproj
>
> On Fri, Sep 18, 2015 at 11:51:09AM -0400, Doug Ledford wrote:
> > On 09/16/2015 02:22 AM, Dan Carpenter wrote:
> > > __get_txreq() returns an ERR_PTR() but this checks for NULL so it
> > > would oops on failure.
> > >
> > > Signed-off-by: Dan Carpenter
> >
> > Thanks, applied.
>
> Applied to
On Fri, Sep 18, 2015 at 10:22 PM, Punit Vara wrote:
> This patch is to comedidev.h file that fixes up following warnings
> reported by checkpatch.pl :
>
> -Block comments use a trailing */ on a separate line
> -Block comments use * on subsequent lines
>
> Signed-off-by: Punit Vara
> ---
> driver
[PATCH] Staging: comedi: Fix block comment according to preferred style
[PATCH] Staging: comedi: Fix block comment warning
which would be the better subject for my patch ? anybody can suggest please ?
regards,
punitvara
___
devel mailing list
de...@lin
Remove debug messages related to fbtft_par_dbg(DEBUG_INIT_DISPLAY.. )
as this info can be obtained using kernel function tracer
Signed-off-by: Shraddha Barke
---
drivers/staging/fbtft/fb_agm1264k-fl.c | 2 --
drivers/staging/fbtft/fb_bd663474.c| 4
drivers/staging/fbtft/fb_hx8347d.c
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Sat, Sep 19, 2015 at 09:57:43PM +0530, punit vara wrote:
> For this patch can I put subject like
>
> [PATCH] Staging: comedi: Fix trailing */ and * comment warning
>
> ??
Yes, that would be much
For this patch can I put subject like
[PATCH] Staging: comedi: Fix trailing */ and * comment warning
??
Kindly suggest me as this is my first patch and i am trying to
contribute to linux kernel .
On Fri, Sep 18, 2015 at 10:22 PM, Punit Vara wrote:
> This patch is to comedidev.h file that fixes
This patch is to comedidev.h file that fixes up following warnings
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
-Block comments use * on subsequent lines
Signed-off-by: Punit Vara
---
drivers/staging/comedi/comedidev.h | 62 ---
ALLIANCE & LEICESTER LOANS
Do you need a loan? Arrangements to borrow up to £ 100,000,000.00, choose
between 1 to 25 years repayment period, choose between monthly and annual
repayment plan, flexible loan terms and conditions. All this plan and more by
contacting us
Dr. Richard Jose
Head of Fi
On Fri, Sep 18, 2015 at 01:06:42PM -0600, BryanSPaul wrote:
> fixed block comment use * on subsequent lines fix
>
> ---
Same problem like the previous patch. Missing Signed-off-by.
regards
sudip
___
devel mailing list
[email protected]
http:/
On Fri, Sep 18, 2015 at 12:21:07PM -0600, BryanSPaul wrote:
> altered to compare constants from left
>
> ---
Missing Signed-off-by.
regards
sudip
___
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinf
ALLIANCE & LEICESTER LOANS
Do you need a loan? Arrangements to borrow up to £ 100,000,000.00, choose
between 1 to 25 years repayment period, choose between monthly and annual
repayment plan, flexible loan terms and conditions. All this plan and more by
contacting us
Dr. Richard Jose
Head of Fi
30 matches
Mail list logo