[PATCH] Staging: ks7010: Use preffered kernel types

2016-11-15 Thread Punit Vara
Replace uint8_t, uint16_t and uint32_t with preferred kernel types u8, u16 and u32 respectively suggested by checkpatch.pl Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 138 +-- drivers/staging/ks7010/michael

[PATCH v3] Staging: ks7010: Prefer using the BIT macro

2016-11-14 Thread Punit Vara
Replace all occurences of (1<<x) by BIT(x) in the file ks7010_sdio.h to get rid of checkpatch.pl "CHECK" output "Prefer using BIT macro". Signed-off-by: Punit Vara <punitv...@gmail.com> --- * Subject made more clear * Remove unnecessary indentation drivers/s

Re: [PATCH] Staging: ks7010: Prefer using the BIT macro

2016-11-14 Thread punit vara
Sorry for that. I have updated the patch and sent you version 2. If I am still wrong could you please share me link to the sample commit ? thanks, PV On Mon, Nov 14, 2016 at 12:30 PM, Greg KH <gre...@linuxfoundation.org> wrote: > On Mon, Nov 14, 2016 at 12:20:40PM +0530, Punit V

[PATCH v2] Subject: Staging: ks7010: Prefer using the BIT macro

2016-11-14 Thread Punit Vara
Replace all occurences of (1<<x) by BIT(x) in the file ks7010_sdio.h to get rid of checkpatch.pl "CHECK" output "Prefer using BIT macro". Signed-off-by: Punit Vara <punitv...@gmail.com> --- * remove unecessary indentation drivers/staging/ks7010/ks7010_sdio

[PATCH] Staging: ks7010: Prefer using the BIT macro

2016-11-13 Thread Punit Vara
Replace all occurances of (1<<x) by BIT(x) in the file ks7010_sdio.h to get rid of checkpatch.pl "CHECK" output "Prefer using BIT macro" Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/ks7010/ks7010_sdio.h | 16

Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-12-01 Thread punit vara
On Mon, Nov 9, 2015 at 3:53 PM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Mon, Nov 09, 2015 at 03:43:38PM +0530, punit vara wrote: >> On Mon, Nov 9, 2015 at 3:31 PM, glen lee <glen@atmel.com> wrote: >> > >> > >> > On 2015년 11월 09일

Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-12-01 Thread punit vara
On Wed, Dec 2, 2015 at 12:30 PM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Wed, Dec 02, 2015 at 08:39:10AM +0530, punit vara wrote: >> On Mon, Nov 9, 2015 at 3:53 PM, Sudip Mukherjee >> <sudipm.mukher...@gmail.com> wrote: >> > On Mon, Nov 09,

Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-09 Thread punit vara
On Mon, Nov 9, 2015 at 3:53 PM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Mon, Nov 09, 2015 at 03:43:38PM +0530, punit vara wrote: >> On Mon, Nov 9, 2015 at 3:31 PM, glen lee <glen@atmel.com> wrote: >> > >> > >> > On 2015년 11월 09일

Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-09 Thread punit vara
On Mon, Nov 9, 2015 at 3:31 PM, glen lee <glen@atmel.com> wrote: > > > On 2015년 11월 09일 18:05, punit vara wrote: >> >> On Mon, Nov 9, 2015 at 2:25 PM, glen lee <glen@atmel.com> wrote: >>> >>> On 2015년 11월 09일 17:18, Dan Carpenter wrote:

Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-09 Thread punit vara
On Mon, Nov 9, 2015 at 1:48 PM, Dan Carpenter <dan.carpen...@oracle.com> wrote: > On Mon, Nov 09, 2015 at 05:02:48PM +0900, glen lee wrote: >> Hi Punit Vara, >> >> I cannot find build errors on my build machines. >> >> According the log which you have

Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-09 Thread punit vara
On Mon, Nov 9, 2015 at 2:25 PM, glen lee <glen@atmel.com> wrote: > > On 2015년 11월 09일 17:18, Dan Carpenter wrote: >> >> On Mon, Nov 09, 2015 at 05:02:48PM +0900, glen lee wrote: >>> >>> Hi Punit Vara, >>> >>> I cannot find build erro

Re: [PATCH] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-08 Thread punit vara
On Sun, Nov 8, 2015 at 2:21 AM, Dan Carpenter wrote: > I haven't looked at it, sorry. You'll have to figure it out or if you > complain to the original person who broken the build he can figure it > out. You are still missing a Fixes tag. > > regards, > dan carpenter

Re: [PATCH] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-08 Thread punit vara
On Sun, Nov 8, 2015 at 2:21 AM, Dan Carpenter wrote: > I haven't looked at it, sorry. You'll have to figure it out or if you > complain to the original person who broken the build he can figure it > out. You are still missing a Fixes tag. > > regards, > dan carpenter

[PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-08 Thread Punit Vara
isr_uh_routine -removes unnecessary lines to declare *wilc Patch fixes 702c0e50f and 2c1d05d10 tags. Signed-off-by: Punit Vara <punitv...@gmail.com> --- -Fixes tag added suggested by Dan carpenter. -Remove declaration of autovariable with same type and same name suggested by Joe Perches d

[PATCH] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-07 Thread Punit Vara
WILC_SDIO) to defination isr_uh_routine Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/wilc1000/linux_wlan.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index 0

Re: [PATCH] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq

2015-11-07 Thread punit vara
On Sat, Nov 7, 2015 at 7:24 PM, Dan Carpenter <dan.carpen...@oracle.com> wrote: > On Sat, Nov 07, 2015 at 05:01:49PM +0530, Punit Vara wrote: >> int wilc1000_wlan_init(struct net_device *dev, perInterface_wlan_t *p_nic) >> { >> @@ -1791,6 +1794,7 @@ int wilc_net

[PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread Punit Vara
This patch is to the linux_wlan.c file that fixes declaration of *wilc to remove following error while building it. make drivers/staging/wilc1000/linux_wlan.o drivers/staging/wilc1000/linux_wlan.c:1824:24: error: ‘wilc’ undeclared Signed-off-by: Punit Vara <punitv...@gmail.com> --- d

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread punit vara
On Sat, Nov 7, 2015 at 3:29 AM, Punit Vara <punitv...@gmail.com> wrote: > This patch is to the linux_wlan.c file that fixes declaration of *wilc > to remove following error while building it. > > make drivers/staging/wilc1000/linux_wlan.o > drivers/staging/wilc1000/linux_

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread punit vara
On Sat, Nov 7, 2015 at 3:39 AM, punit vara <punitv...@gmail.com> wrote: > On Sat, Nov 7, 2015 at 3:29 AM, Punit Vara <punitv...@gmail.com> wrote: >> This patch is to the linux_wlan.c file that fixes declaration of *wilc >> to remove following error while building it.

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread punit vara
On Sat, Nov 7, 2015 at 3:47 AM, punit vara <punitv...@gmail.com> wrote: > On Sat, Nov 7, 2015 at 3:39 AM, punit vara <punitv...@gmail.com> wrote: >> On Sat, Nov 7, 2015 at 3:29 AM, Punit Vara <punitv...@gmail.com> wrote: >>> This patch is to the linux_wlan.c

[RESEND PATCH v2] Staging: wilc1000: Prefer eth_broadcast_addr over memset()

2015-10-28 Thread Punit Vara
This patch is to the host_interface.c that fixes up following warning by checkpatch: -prefer eth_broadcast_addr() over memset() Signed-off-by: Punit Vara <punitv...@gmail.com> --- Applied on updated staging tree. drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 ins

[PATCH v2] Staging: wilc1000: Prefer eth_broadcast_addr over memset()

2015-10-27 Thread Punit Vara
This patch is to the host_interface.c that fixes up following warning by checkpatch: -prefer eth_broadcast_addr() over memset() Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset()

2015-10-26 Thread Punit Vara
This patch is to the host_interface.c that fixes up following warning by checkpatch.pl: -Prefer eth_broadcast_addr() over memset() Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH 1/3] staging: wilc1000: Remove reference preceded by free

2015-10-25 Thread punit vara
On Mon, Oct 26, 2015 at 1:01 AM, punit vara <punitv...@gmail.com> wrote: > On Mon, Oct 26, 2015 at 12:42 AM, Greg KH <gre...@linuxfoundation.org> wrote: >> On Sun, Oct 25, 2015 at 04:01:23AM +0530, Punit Vara wrote: >>> This patch is to the wilc_wfi_cfgoperations.c fi

Re: [PATCH 1/3] staging: wilc1000: Remove reference preceded by free

2015-10-25 Thread punit vara
On Mon, Oct 26, 2015 at 12:42 AM, Greg KH <gre...@linuxfoundation.org> wrote: > On Sun, Oct 25, 2015 at 04:01:23AM +0530, Punit Vara wrote: >> This patch is to the wilc_wfi_cfgoperations.c file that fixes up >> following error reported by coccicheck: >> >> ERROR: r

[PATCH 2/3] Staging: wilc1000: Remove comparision of field address to NULL

2015-10-24 Thread Punit Vara
This is patch to the linux_wlan file that fixes up following error reported by coccicheck : ERROR: test of a variable/field address Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/wilc1000/linux_wlan.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

[PATCH 0/3] Staging: wilc1000: Remove coccicheck warnings and error

2015-10-24 Thread Punit Vara
Punit Vara (3): staging: wilc1000: Remove reference preceded by free Staging: wilc1000: Remove comparision of field address to NULL Staging: wilc1000: Remove boolean comparision drivers/staging/wilc1000/host_interface.c | 4 ++-- drivers/staging/wilc1000/linux_wlan.c

[PATCH 1/3] staging: wilc1000: Remove reference preceded by free

2015-10-24 Thread Punit Vara
ree whenever params->seq_len <=0 remove it and place kfree inside (params->seq_len) >0 condition. Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --

[PATCH 3/3] Staging: wilc1000: Remove boolean comparision

2015-10-24 Thread Punit Vara
This patch is to the host_interface.c file that fixes up following warning reported by coccicheck: WARNING: Comparison to bool Boolean tests do not need explicit comparison to true or false Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/wilc1000/host_interface

Re: [PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-24 Thread punit vara
On Fri, Oct 23, 2015 at 2:04 AM, Dan Carpenter wrote: > First fetch the changes, then check them out. > > $ git fetch git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git > staging-next > $ git checkout FETCH_HEAD > > regards, > dan carpenter > Thank you

Re: [PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-22 Thread punit vara
On Mon, Oct 19, 2015 at 8:51 PM, punit vara <punitv...@gmail.com> wrote: > On Mon, Oct 19, 2015 at 8:14 AM, Larry Finger <larry.fin...@lwfinger.net> > wrote: >> On 10/18/2015 09:00 PM, Greg KH wrote: >>> >>> On Sun, Oct 18, 2015 at 12:02:53PM +0530, pu

Re: [PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-22 Thread punit vara
On Fri, Oct 23, 2015 at 12:52 AM, Dan Carpenter <dan.carpen...@oracle.com> wrote: > On Fri, Oct 23, 2015 at 12:34:36AM +0530, punit vara wrote: >> I am working on linux-next .I have tried to rebase my local branch on >> your staging tree but I am getting too many confl

Re: [PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-19 Thread punit vara
On Mon, Oct 19, 2015 at 8:14 AM, Larry Finger <larry.fin...@lwfinger.net> wrote: > On 10/18/2015 09:00 PM, Greg KH wrote: >> >> On Sun, Oct 18, 2015 at 12:02:53PM +0530, punit vara wrote: >>> >>> One more thing I would like suggestion how can start switchin

Re: [PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-18 Thread punit vara
On Sat, Oct 17, 2015 at 10:46 AM, Greg KH <gre...@linuxfoundation.org> wrote: > On Wed, Oct 14, 2015 at 11:55:55PM +0530, Punit Vara wrote: >> Put constant on the right side of a test.Problem found using checkpatch: >> >> Warning:Comparisons should place the constant on

[PATCH RESEND 0/4] Staging: rtl8712: Fix coding style warnings

2015-10-14 Thread Punit Vara
should place the constant on the right side of the test v2: Fixed API suggested by : Larry Finger larry.fin...@lwfinger.net Improved patch description suggested by Julia julia.law...@lip6.fr Punit Vara (4): Staging: rtl8712: Fix warning prefer eth_broadcast_addr() over memset() Staging

[PATCH RESEND 3/4] Staging: rtl8712: Coding style warnings fix for block comments

2015-10-14 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warnings reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/r

[PATCH RESEND 1/4] Staging: rtl8712: Fix warning prefer eth_broadcast_addr() over memset()

2015-10-14 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c that fixes up following warning by checkpatch.pl: -Prefer eth_broadcast_addr() over memset() Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-14 Thread Punit Vara
Put constant on the right side of a test.Problem found using checkpatch: Warning:Comparisons should place the constant on the right side of the test Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 16 1 file chan

[PATCH v2 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-14 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warning reported by checkpatch.pl : -Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() bssid and pnetwork->network.MacAddress both are 6 byte array which aligned with u16 Signed-off-by: Punit V

Re: [PATCH v2 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-12 Thread punit vara
On Tue, Oct 13, 2015 at 12:16 AM, Punit Vara <punitv...@gmail.com> wrote: > This patch is to the rtl871x_ioctl_linux.c file that fixes up following > warning reported by checkpatch.pl : > > -Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() >

[PATCH v2 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-12 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warning reported by checkpatch.pl : -Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() bssid and pnetwork->network.MacAddress both are 6 byte array which aligned with u16 Signed-off-by: Punit V

Re: [PATCH 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-11 Thread punit vara
On Sun, Oct 11, 2015 at 6:07 AM, kbuild test robot wrote: > Hi Punit, > > [auto build test WARNING on v4.3-rc4 -- if it's inappropriate base, please > ignore] > > reproduce: > # apt-get install sparse > make ARCH=x86_64 allmodconfig > make C=1

Re: [PATCH 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-10 Thread punit vara
On Sat, Oct 10, 2015 at 10:37 PM, Julia Lawall <julia.law...@lip6.fr> wrote: > > > On Sat, 10 Oct 2015, Punit Vara wrote: > >> This patch is to the rtl871x_ioctl_linux.c file that fixes up following > > The entire above line doesn't give any information (except the fi

[PATCH 1/4] Staging: rtl8712: Fix warning prefer eth_broadcast_addr() over memset()

2015-10-10 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c that fixes up following warning by checkpatch.pl: -Prefer eth_broadcast_addr() over memset() Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[PATCH 0/4] Staging: rtl8712: Fix coding style warnings

2015-10-10 Thread Punit Vara
should place the constant on the right side of the test Punit Vara (4): Staging: rtl8712: Fix warning prefer eth_broadcast_addr() over memset() Staging: rtl8712: Use ether_addr_equal() over memcmp() Staging: rtl8712: Coding style warnings fix for block comments Staging: rtl8712: fix

[PATCH 3/4] Staging: rtl8712: Coding style warnings fix for block comments

2015-10-10 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warnings reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/r

[PATCH 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-10 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warnings reported by checkpatch.pl: -Comparisons should place the constant on the right side of the test Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl_linux.

[PATCH 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-10 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warning reported by checkpatch.pl : -Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() bssid has datatype u8 and pnetwork->network.MacAddress has data type unsigned char. Signed-off-by: Punit V

Re: [PATCH 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-10 Thread punit vara
On Sat, Oct 10, 2015 at 11:04 PM, Larry Finger <larry.fin...@lwfinger.net> wrote: > On 10/10/2015 11:58 AM, Punit Vara wrote: >> >> This patch is to the rtl871x_ioctl_linux.c file that fixes up following >> warning reported by checkpatch.pl : >

Re: [PATCH 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-10 Thread punit vara
On Sun, Oct 11, 2015 at 1:20 AM, Julia Lawall <julia.law...@lip6.fr> wrote: > > > On Sun, 11 Oct 2015, punit vara wrote: > >> On Sat, Oct 10, 2015 at 11:04 PM, Larry Finger >> <larry.fin...@lwfinger.net> wrote: >> > On 10/10/2015 11:58

[PATCH 08/11] Staging: rtl8712: rtl8712_cmd.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_cmd.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_cmd.c | 12 1 file changed, 8 inse

[PATCH 10/11] Staging: rtl8712: rtl8712_gp_bitdef.h: Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_gp_bitdef.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_gp_bitdef.h | 3 ++- 1 file changed, 2 inse

[PATCH 07/11] Staging: rtl8712: recv_linux.c: Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the recv_linux.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/recv_linux.c | 3 ++- 1 file changed, 2 insertions(+), 1 de

[PATCH 06/11] Staging: rtl8712: os_intfs.c : Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the os_intfs.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/os_intfs.c | 3 ++- 1 file changed, 2 insertions(+), 1 de

[PATCH 09/11] Staging: rtl8712: rtl8712_cmdctrl_bitdef.h: Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_cmdctrl_bitdef.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_cmdctrl_bitdef.h | 3 ++- 1 file chan

[PATCH 05/11] Staging: rtl8712: mlme_linux.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the mlme_linux.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/mlme_linux.c | 3 ++- 1 file changed, 2 insertions(+), 1 de

[PATCH 01/11] Staging: rtl8712: drv_types.h:Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the drv_types.h file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/drv_types.

[PATCH 03/11] Staging: rtl8712: ieee80211.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the ieee80211.c file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/ieee80211.c | 7

[PATCH 02/11] Staging: rtl8712: hal_init.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the hal_init.c file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/hal_init.

[PATCH 04/11] Staging: rtl8712: ieee80211.h: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the ieee80211.h file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/ieee80211.

[PATCH 11/11] Staging: rtl8712: rtl8712_hal.h: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_hal.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_hal.h | 14 +- 1 file changed, 9 inse

Re: [PATCH 3/3] Staging: rts5208: Coding style warnings fix for block comments

2015-09-30 Thread punit vara
On Tue, Sep 29, 2015 at 5:55 AM, Greg KH <gre...@linuxfoundation.org> wrote: > On Mon, Sep 21, 2015 at 11:29:50PM +0530, Punit Vara wrote: >> This patch is to rtsx_chip.h that fixes up following warning reported >> by checkpatch.pl : >> >> -Block comments us

Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-28 Thread punit vara
On Mon, Sep 28, 2015 at 4:48 PM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Sat, Sep 26, 2015 at 12:45:46PM -0500, Larry Finger wrote: >> On 09/26/2015 11:49 AM, Punit Vara wrote: >> >This patch is to the rtl871x_ioctl_linux.c that fixes up follow

[PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-26 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c that fixes up following warning reported by checkpatch.pl : - Comparisons should place the constant on the right side of the test Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 16 -

Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-26 Thread punit vara
On Sat, Sep 26, 2015 at 11:15 PM, Larry Finger <larry.fin...@lwfinger.net> wrote: > On 09/26/2015 11:49 AM, Punit Vara wrote: >> >> This patch is to the rtl871x_ioctl_linux.c that fixes up following >> warning reported by checkpatch.pl : >> >> - Comparisons

Re: [PATCH] Staging: rtl8712: rtl871x_io.h: Coding style warning fix for block comments

2015-09-24 Thread punit vara
On Thu, Sep 24, 2015 at 11:25 AM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Thu, Sep 24, 2015 at 01:22:33AM +0530, Punit Vara wrote: >> This patch is to the rtl871x_io.h file that fixes up following warning >> reported by checkpatch.pl : >> >>

Re: [PATCH 13/13] Staging: rtl8712: Coding style warning fix for block comment

2015-09-24 Thread punit vara
On Thu, Sep 24, 2015 at 9:55 AM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Wed, Sep 23, 2015 at 11:22:09PM +0530, punit vara wrote: >> On Wed, Sep 23, 2015 at 10:25 PM, punit vara <punitv...@gmail.com> wrote: >> > Thank you very much sir I will definite

[PATCH 3/6] Staging: rtl8712: usb_intf.c: Coding style warnings fix for block comments

2015-09-24 Thread Punit Vara
This patch is to the usb_intf.c file that fixes up following warnings generated by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/usb_intf.c | 12 1 file changed, 8 insertions

[PATCH 2/6] Staging: rtl8712: usb_halinit.c: Coding style warning fix for block comment

2015-09-24 Thread Punit Vara
This patch is to the usb_halinit.c file that fixes up following warnings generated by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/usb_halinit.c | 3 ++- 1 file changed, 2 insertions

[PATCH 4/6] Staging: rtl8712: wifi.h: Coding style warning fix for block comment

2015-09-24 Thread Punit Vara
This patch is to the wifi.h file that fixes up following warnings generated by checkpatch.pl tool : -Block comments use a trailing */ on a separate line -Block comments use a * on subsequent lines Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/wifi.

[PATCH 5/6] Staging: rtl8712: wlan_bssdef.h: Coding style warnings fix for block comments

2015-09-24 Thread Punit Vara
This patch is to the wlan_bssdef.h file that fixes up following warnings generated by checkpatch.pl : -Block comments use a trailing */ on a separate line -Block comments use a * on subsequent lines Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/wlan_bssdef.

[PATCH 6/6] Staging: rtl8712: xmit_linux.c: Coding style warning fix for block comment

2015-09-24 Thread Punit Vara
This patch is to the xmit_linux.c file that fixes up following warnings generated by checkpatch.pl tool : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/xmit_linux.c | 3 ++- 1 file changed, 2 insertions

[PATCH 1/6] Staging: rtl8712: rtl871x_xmit.h: Coding style warnings fix for block comments

2015-09-24 Thread Punit Vara
This patch is to the rtl871x_xmit.h file that fixes up following warnings generated by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_xmit.h | 12 1 file changed, 8 inse

Re: [PATCH] Staging: rtl8712: rtl871x_mp.h: Coding style warnings fix for block comment

2015-09-24 Thread punit vara
On Thu, Sep 24, 2015 at 4:52 PM, Dan Carpenter <dan.carpen...@oracle.com> wrote: > On Thu, Sep 24, 2015 at 03:57:22PM +0530, Sudip Mukherjee wrote: >> On Thu, Sep 24, 2015 at 12:00:01PM +0300, Dan Carpenter wrote: >> > On Thu, Sep 24, 2015 at 01:54:22AM

Re: [PATCH 13/13] Staging: rtl8712: Coding style warning fix for block comment

2015-09-23 Thread punit vara
On Wed, Sep 23, 2015 at 12:55 PM, Jaime Arrocha <j...@innercoder.com> wrote: > > On 09/23/2015 12:31 AM, Sudip Mukherjee wrote: >> >> On Wed, Sep 23, 2015 at 02:03:56AM +0530, Punit Vara wrote: >>> >>> This patch is to rtl8712_gp_bitdef.h file th

Re: [PATCH 13/13] Staging: rtl8712: Coding style warning fix for block comment

2015-09-23 Thread punit vara
On Wed, Sep 23, 2015 at 9:04 PM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > On Wed, Sep 23, 2015 at 07:32:52PM +0530, punit vara wrote: >> On Wed, Sep 23, 2015 at 12:55 PM, Jaime Arrocha <j...@innercoder.com> wrote: >> > >> > On 09/

[PATCH] Staging: rtl8712: rtl8712_recv.h: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl8712_recv.h that fixes up following warnings generated by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_recv.

Re: [PATCH 13/13] Staging: rtl8712: Coding style warning fix for block comment

2015-09-23 Thread punit vara
Thank you very much sir I will definitely follow that. I have seen so many people fixing patches over here. Now I know process of submitting patches.I am able to fix several patches created by checkpatch.pl . But can you suggest me any driver to fix API or another small TODO task. I have seen so

[PATCH] Staging: rtl8712: rtl8712_syscfg_bitdef.h: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl8712_syscfg_bitdef.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_syscfg_bitdef.

Re: [PATCH 13/13] Staging: rtl8712: Coding style warning fix for block comment

2015-09-23 Thread punit vara
On Wed, Sep 23, 2015 at 10:25 PM, punit vara <punitv...@gmail.com> wrote: > Thank you very much sir I will definitely follow that. I have seen so > many people fixing patches over here. Now I know process of submitting > patches.I am able to fix several patches created by checkpatch

[PATCH] Staging: rtl8712: rtl8712_xmit.c: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is the rtl8712_xmit.c that fixes up following warnings reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_xmit.c | 18 -- 1 file changed, 12 inse

[PATCH] Staging: rtl8712: rtl871x_cmd.c: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_cmd.c 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 <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_cmd.

[PATCH] Staging: rtl8712: rtl871x_cmd.h: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_cmd.h file that fixes up following warnings reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_cmd.h

[PATCH] Staging: rtl8712: rtl871x_ht.h: Coding style warning fix for block comment

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_ht.h that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ht.h | 6 -- 1 file changed, 4 insertions(+), 2 del

[PATCH] Staging: rtl8712: rtl871x_ioctl.h: Coding style warning fix for block comment

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_ioctl.h that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl.h | 6 -- 1 file changed, 4 insertions

[PATCH] Staging: rtl8712: rtl871x_ioctl_set.c: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_ioctl_set.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_ioctl_set.c | 12 1 file chan

[PATCH] Staging: rtl8712: rtl8712_spec.h: Coding Style warning fix for block comment

2015-09-23 Thread Punit Vara
This patch is to the rtl8712_spec.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl8712_spec.h | 8 +--- 1 file changed, 5 insertions

[PATCH] Staging: rtl8712: rtl871x_cmd.h: Fix typo error "ture"

2015-09-23 Thread Punit Vara
This patch is to the rtl8712_cmd.h that fixes up typo error in comment for word "true". Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_cmd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8712/rtl871x_

[PATCH] Staging: rtl8712: rtl871x_io.h: Coding style warning fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_io.h file that fixes up following warning reported by checkpatch.pl : Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_io.h | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/staging/r

[PATCH] Staging: rtl8712: rtl871x_mlme.c: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_mlme.c file that fixes up following warning reported by checkpatch.pl: - Block comments use * on subsequent lines - Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_mlme.

[PATCH] Staging: rtl8712: rtl871x_recv.c: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_recv.c 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 <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_recv.

[PATCH] Staging: rtl8712: rtl871x_recv.h: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_recv.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 <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_recv.

[PATCH] Staging: rtl8712: rtl871x_led.h: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_led.h that fixes up following warnings reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_led.h | 15 ++- 1 file changed, 10 inse

[PATCH] Staging: rtl8712: rtl871x_security.c: Coding style warning fix for block commment

2015-09-23 Thread Punit Vara
This is patch to the rtl871x_security.c that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_security.c | 4 +--- 1 file changed, 1 insertion(+), 3 del

[PATCH] Staging: rtl8712: rtl871x_mlme.h: Coding style warnings fix for block comments

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_mlme.h file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_mlme.

[PATCH] Staging: rtl8712: rtl871x_mp.h: Coding style warnings fix for block comment

2015-09-23 Thread Punit Vara
This is a patch to the rtl871x_mp.h that fixes up following warnings reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_mp.h | 3 ++- 1 file changed, 2 insertions(+), 1 de

[PATCH] Staging: rtl8712: rtl871x_sta_mgt.c: Coding style warning fix for block comment

2015-09-23 Thread Punit Vara
This is a patch to the rtl871x_sta_mgt.c file that fixes up following warning reported by checkpatch.pl: -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_sta_mgt.c | 3 ++- 1 file changed, 2 insertions

[PATCH] Staging: rtl8712: rtl871x_mp.c: Coding style warning fix for block comment

2015-09-23 Thread Punit Vara
This patch is to the rtl871x_mp.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_mp.c | 3 ++- 1 file changed, 2 insertions(+), 1 de

[PATCH] Staging: rtl8712: Coding Style warnings fix for block comments

2015-09-23 Thread Punit Vara
This is patch to the rtl871x_pwrctrl.c file that fixes following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line -Block comments use * on subsequent lines Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/rtl871x_pwrctrl.

[PATCH] Staging: rtl8712: ieee80211.h: Coding style warnings fix for block comments

2015-09-22 Thread Punit Vara
This patch is to the ieee80211.h that fixes up following warnings reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara <punitv...@gmail.com> --- drivers/staging/rtl8712/ieee80211.

  1   2   >