[PATCH] staging: fbtft: fbtft-core.c: EXPORT_SYMBOL should immediately follow its function

2017-04-28 Thread Shiva Kerdel
This patch fixes the following checkpatch.pl warning - EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fbtft/fbtft-core.c | 25 +++-- 1 file changed, 15 insertions(+), 10 deletions(-)

[PATCH] staging: fbtft: fbtft-core.c: EXPORT_SYMBOL should immediately follow its function

2017-04-28 Thread Shiva Kerdel
This patch fixes the following checkpatch.pl warning - EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Shiva Kerdel --- drivers/staging/fbtft/fbtft-core.c | 25 +++-- 1 file changed, 15 insertions(+), 10 deletions(-) diff --git a/drivers

Re: [PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t'

2017-03-14 Thread Shiva Kerdel
. Thanks, S. Kerdel On 03/12/17 04:00, Tobin C. Harding wrote: On Sun, Mar 12, 2017 at 03:29:59AM +0100, Shiva Kerdel wrote: Fix prefer kernel type 'u8' over 'uint8_t' checks. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 4 +- drivers/staging/

Re: [PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t'

2017-03-14 Thread Shiva Kerdel
. Thanks, S. Kerdel On 03/12/17 04:00, Tobin C. Harding wrote: On Sun, Mar 12, 2017 at 03:29:59AM +0100, Shiva Kerdel wrote: Fix prefer kernel type 'u8' over 'uint8_t' checks. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 4 +- drivers/staging/ks7010/ks_hostif.h | 114

[PATCH 2/3] Staging: ks7010: ks_hostif.*: Use preferred 'u16' kernel type over 'uint16_t'

2017-03-11 Thread Shiva Kerdel
Fix prefer kernel type 'u16' over 'uint16_t' checks. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 20 +++--- drivers/staging/ks7010/ks_hostif.h | 134 ++--- 2 files changed, 77 insertions(+), 77 deletions(-) diff

[PATCH 2/3] Staging: ks7010: ks_hostif.*: Use preferred 'u16' kernel type over 'uint16_t'

2017-03-11 Thread Shiva Kerdel
Fix prefer kernel type 'u16' over 'uint16_t' checks. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 20 +++--- drivers/staging/ks7010/ks_hostif.h | 134 ++--- 2 files changed, 77 insertions(+), 77 deletions(-) diff --git a/drivers/staging

[PATCH 3/3] Staging: ks7010: ks_hostif.*: Use preferred 'u32' kernel type over 'uint32_t'

2017-03-11 Thread Shiva Kerdel
Fix prefer kernel type 'u32' over 'uint32_t' checks. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 18 +- drivers/staging/ks7010/ks_hostif.h | 30 +++--- 2 files changed, 24 insertions(+), 24 deletions(-) diff

[PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t'

2017-03-11 Thread Shiva Kerdel
Fix prefer kernel type 'u8' over 'uint8_t' checks. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 4 +- drivers/staging/ks7010/ks_hostif.h | 114 +-- drivers/staging/ks7010/ks_wlan_net.c | 2 +- 3 files chang

[PATCH 3/3] Staging: ks7010: ks_hostif.*: Use preferred 'u32' kernel type over 'uint32_t'

2017-03-11 Thread Shiva Kerdel
Fix prefer kernel type 'u32' over 'uint32_t' checks. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 18 +- drivers/staging/ks7010/ks_hostif.h | 30 +++--- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers

[PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t'

2017-03-11 Thread Shiva Kerdel
Fix prefer kernel type 'u8' over 'uint8_t' checks. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 4 +- drivers/staging/ks7010/ks_hostif.h | 114 +-- drivers/staging/ks7010/ks_wlan_net.c | 2 +- 3 files changed, 60 insertions(+), 60

[PATCH] Staging: ks7010: ks_*: Braces should be used on all arms of these statements

2017-02-17 Thread Shiva Kerdel
Braces should be used on all arms of these statements (CHECK).. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 6 -- drivers/staging/ks7010/ks_wlan_net.c | 42 +++- 2 files changed, 31 insertions(+), 17 del

[PATCH] Staging: ks7010: ks_*: Braces should be used on all arms of these statements

2017-02-17 Thread Shiva Kerdel
Braces should be used on all arms of these statements (CHECK).. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 6 -- drivers/staging/ks7010/ks_wlan_net.c | 42 +++- 2 files changed, 31 insertions(+), 17 deletions(-) diff --git

[PATCH] Staging: ks7010: ks_*: Use the BIT macro for bitwise checks

2017-02-16 Thread Shiva Kerdel
Changed bit swifting operators to BIT macros (preferred), This change will also solve the preferred space surrounding the operator checks. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.h | 20 ++-- drivers/staging/ks7010/ks_wlan.h

[PATCH] Staging: ks7010: ks_*: Use the BIT macro for bitwise checks

2017-02-16 Thread Shiva Kerdel
Changed bit swifting operators to BIT macros (preferred), This change will also solve the preferred space surrounding the operator checks. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.h | 20 ++-- drivers/staging/ks7010/ks_wlan.h | 30

[PATCH] Staging: ks7010: Add required and preferred spaces around operators

2017-02-16 Thread Shiva Kerdel
Spaces should be added around operators to improve readability and are required in some cases. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 2 +- drivers/staging/ks7010/ks_hostif.h | 64 +- drivers/staging/

[PATCH] Staging: ks7010: Add required and preferred spaces around operators

2017-02-16 Thread Shiva Kerdel
Spaces should be added around operators to improve readability and are required in some cases. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 2 +- drivers/staging/ks7010/ks_hostif.h | 64 +- drivers/staging/ks7010/ks_wlan.h

[PATCH 2/2] Staging: ks7010: ks*: Remove redundant blank lines

2017-02-16 Thread Shiva Kerdel
Multiple blank lines shouldn't be used. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks7010_sdio.c | 1 - drivers/staging/ks7010/ks_wlan_net.c | 2 -- 2 files changed, 3 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/

[PATCH 2/2] Staging: ks7010: ks*: Remove redundant blank lines

2017-02-16 Thread Shiva Kerdel
Multiple blank lines shouldn't be used. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks7010_sdio.c | 1 - drivers/staging/ks7010/ks_wlan_net.c | 2 -- 2 files changed, 3 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c index

[PATCH 1/2] Staging: ks7010: ks*: Add missing blank lines after declarations

2017-02-16 Thread Shiva Kerdel
A blank line should be added after function/struct/union/enum declarations. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks7010_sdio.h | 1 + drivers/staging/ks7010/ks_wlan.h | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/staging/ks7010/ks7010_

[PATCH 1/2] Staging: ks7010: ks*: Add missing blank lines after declarations

2017-02-16 Thread Shiva Kerdel
A blank line should be added after function/struct/union/enum declarations. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks7010_sdio.h | 1 + drivers/staging/ks7010/ks_wlan.h | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/staging/ks7010/ks7010_sdio.h b/drivers

[PATCH] Staging: ks7010: ks_*: Removed blank lines before and after braces.

2017-02-10 Thread Shiva Kerdel
Removing unnecessary blank lines around braces to solve CHECKS. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/ks7010/ks_hostif.c | 11 --- drivers/staging/ks7010/ks_wlan.h | 1 - drivers/staging/ks7010/ks_wlan_net.c | 5 - 3 files changed, 17 del

[PATCH] Staging: ks7010: ks_*: Removed blank lines before and after braces.

2017-02-10 Thread Shiva Kerdel
Removing unnecessary blank lines around braces to solve CHECKS. Signed-off-by: Shiva Kerdel --- drivers/staging/ks7010/ks_hostif.c | 11 --- drivers/staging/ks7010/ks_wlan.h | 1 - drivers/staging/ks7010/ks_wlan_net.c | 5 - 3 files changed, 17 deletions(-) diff --git

Re: [PATCH v2] Staging: dgnc: dgnc_*.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-06 Thread Shiva Kerdel
On Tue, Dec 06, 2016 at 09:59:58AM +0100, Shiva Kerdel wrote: In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. But, udelay does something different than usleep, are you sure you should be giving up the cpu at this point in time? Are you

Re: [PATCH v2] Staging: dgnc: dgnc_*.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-06 Thread Shiva Kerdel
On Tue, Dec 06, 2016 at 09:59:58AM +0100, Shiva Kerdel wrote: In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. But, udelay does something different than usleep, are you sure you should be giving up the cpu at this point in time? Are you

[PATCH v2] Staging: dgnc: dgnc_*.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-06 Thread Shiva Kerdel
In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. usleep_range gives a much better chance of coalescing processor wakeups. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- Changes for v2: - Squashed the two commits to one patch. d

[PATCH v2] Staging: dgnc: dgnc_*.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-06 Thread Shiva Kerdel
In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. usleep_range gives a much better chance of coalescing processor wakeups. Signed-off-by: Shiva Kerdel --- Changes for v2: - Squashed the two commits to one patch. drivers/staging/dgnc

[PATCH 2/2] Staging: dgnc: dgnc_neo.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-05 Thread Shiva Kerdel
In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. usleep_range gives a much better chance of coalescing processor wakeups. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/dgnc/dgnc_neo.c | 10 +- 1 file chan

[PATCH 2/2] Staging: dgnc: dgnc_neo.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-05 Thread Shiva Kerdel
In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. usleep_range gives a much better chance of coalescing processor wakeups. Signed-off-by: Shiva Kerdel --- drivers/staging/dgnc/dgnc_neo.c | 10 +- 1 file changed, 5 insertions(+), 5

[PATCH 1/2] Staging: dgnc: dgnc_cls.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-05 Thread Shiva Kerdel
In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. usleep_range gives a much better chance of coalescing processor wakeups. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/dgnc/dgnc_cls.c | 6 +++--- 1 file changed, 3 inse

[PATCH 1/2] Staging: dgnc: dgnc_cls.c: Use usleep_range over udelay to improve coalescing processor wakeups

2016-12-05 Thread Shiva Kerdel
In most cases, usleep_range is better than udelay, as the precise wakeup from udelay is unnecessary. usleep_range gives a much better chance of coalescing processor wakeups. Signed-off-by: Shiva Kerdel --- drivers/staging/dgnc/dgnc_cls.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH] Staging: rtl8712: rtl871x_*: Removed unnecessary else statements after a break or return

2016-11-21 Thread Shiva Kerdel
The indent code blocks of the else statements were unnecessary and are better written without them. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 4 ++-- drivers/staging/rtl8712/rtl871x_mlme.c| 4 ++-- 2 files changed, 4 insertions

[PATCH] Staging: rtl8712: rtl871x_*: Removed unnecessary else statements after a break or return

2016-11-21 Thread Shiva Kerdel
The indent code blocks of the else statements were unnecessary and are better written without them. Signed-off-by: Shiva Kerdel --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 4 ++-- drivers/staging/rtl8712/rtl871x_mlme.c| 4 ++-- 2 files changed, 4 insertions(+), 4 deletions

[PATCH] Staging: wlan-ng: prism2mgmt: Fixed operators spacing style issues

2016-11-18 Thread Shiva Kerdel
Fixed spaces around operators to fix their coding style issues. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/wlan-ng/prism2mgmt.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/w

[PATCH] Staging: wlan-ng: prism2mgmt: Fixed operators spacing style issues

2016-11-18 Thread Shiva Kerdel
Fixed spaces around operators to fix their coding style issues. Signed-off-by: Shiva Kerdel --- drivers/staging/wlan-ng/prism2mgmt.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c

[PATCH][RESPIN] Staging: fsl-mc: include: mc: Kernel type 'int' preferred over 's16'

2016-11-15 Thread Shiva Kerdel
After following a discussion about the used integer types Dan Carpenter pointed out that 'int' types should be used over the current change to 's16'. The reason for this is to have an upper bound instead of overflowing the 's16' so we could still remove devices. Signed-off-by: Shiva Kerdel <

[PATCH][RESPIN] Staging: fsl-mc: include: mc: Kernel type 'int' preferred over 's16'

2016-11-15 Thread Shiva Kerdel
After following a discussion about the used integer types Dan Carpenter pointed out that 'int' types should be used over the current change to 's16'. The reason for this is to have an upper bound instead of overflowing the 's16' so we could still remove devices. Signed-off-by: Shiva Kerdel

Re: [PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-14 Thread Shiva Kerdel
-Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Monday, November 14, 2016 4:06 AM To: Stuart Yoder <stuart.yo...@nxp.com> Cc: Shiva Kerdel <sh...@exdev.nl>; de...@driverdev.osuosl.org; gre...@linuxfoundation.org; linux- ker...@vger.kernel.org;

Re: [PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-14 Thread Shiva Kerdel
-Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Monday, November 14, 2016 4:06 AM To: Stuart Yoder Cc: Shiva Kerdel ; de...@driverdev.osuosl.org; gre...@linuxfoundation.org; linux- ker...@vger.kernel.org; Nipun Gupta ; tred...@nvidia.com; Laurentiu

[PATCH v5 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- Changes for v2: - None Changes for v3: - None Changes for v4: - corrected patch version number. Ch

[PATCH v5 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- Changes for v2: - None Changes for v3: - None Changes for v4: - corrected patch version number. Changes for v5: - version tag was missing. drivers/staging

[PATCH v5 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- Changes for v2: - corrected an error in the log message, wrote 's32' instead of 's16'. Changes for v3: - added

[PATCH v5 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- Changes for v2: - corrected an error in the log message, wrote 's32' instead of 's16'. Changes for v3: - added the missing annotates. Changes for v4: - corrected

[PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- Changes for v2: - corrected an error in the log message, wrote 's32' instead of 's16'. Changes for v3: - added

[PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- Changes for v2: - corrected an error in the log message, wrote 's32' instead of 's16'. Changes for v3: - added the missing annotates. Changes for v4: - corrected

[PATCH 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- Changes for v2: - None Changes for v3: - None Changes for v4: - corrected patch version number. drivers/st

[PATCH 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-11 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- Changes for v2: - None Changes for v3: - None Changes for v4: - corrected patch version number. drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1

[PATCH v2 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fs

[PATCH v2 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- Changes for v2: - corrected an error in the log message, wrote 's32' instead of 's16'. Changes for v3: - added

[PATCH v2 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc

[PATCH v2 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- Changes for v2: - corrected an error in the log message, wrote 's32' instead of 's16'. Changes for v3: - added the missing annotate of v2. --- drivers/staging/fsl

Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
I corrected the log message because I wrote 's32' instead of 's16' in this patch (confused with the other one that I sent too). Thank you for your guidance, Shiva Kerdel On 11/10/16 15:20, Greg KH wrote: On Thu, Nov 10, 2016 at 03:09:07PM +0100, Shiva Kerdel wrote: Follow the kernel type

Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
I corrected the log message because I wrote 's32' instead of 's16' in this patch (confused with the other one that I sent too). Thank you for your guidance, Shiva Kerdel On 11/10/16 15:20, Greg KH wrote: On Thu, Nov 10, 2016 at 03:09:07PM +0100, Shiva Kerdel wrote: Follow the kernel type

[PATCH v2 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/st

[PATCH v2 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging

[PATCH v2 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fs

[PATCH v2 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc

[PATCH 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fs

[PATCH 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc

[PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> Acked-by: Stuart Yoder <stuart.yo...@nxp.com> --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/st

[PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int16_t'. Signed-off-by: Shiva Kerdel Acked-by: Stuart Yoder --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging

Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
clear to me at the beginning. I hope that you could still use these patches and I am trying to avoid misunderstandings like these. Do I still need to resend them and is there anything left for me to do with these? Best regards, Shiva Kerdel On 11/10/16 13:24, Greg KH wrote: On Tue, Nov 08

Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'

2016-11-10 Thread Shiva Kerdel
clear to me at the beginning. I hope that you could still use these patches and I am trying to avoid misunderstandings like these. Do I still need to resend them and is there anything left for me to do with these? Best regards, Shiva Kerdel On 11/10/16 13:24, Greg KH wrote: On Tue, Nov 08

[PATCH] Staging: xgifb: vgatypes: Replace decimal permissions with 4 digit octal

2016-11-08 Thread Shiva Kerdel
Replaced decimal permissions with 4 digit octal to solve checkpatch errors. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/xgifb/XGI_main_26.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/s

[PATCH] Staging: xgifb: vgatypes: Replace decimal permissions with 4 digit octal

2016-11-08 Thread Shiva Kerdel
Replaced decimal permissions with 4 digit octal to solve checkpatch errors. Signed-off-by: Shiva Kerdel --- drivers/staging/xgifb/XGI_main_26.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/staging/xgifb/XGI_main_26.c

[PATCH v2 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-08 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc/includ

[PATCH v2 2/2] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-08 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc/include/mc.h index a781a36

[PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'

2016-11-08 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging/

[PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'

2016-11-08 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging/fsl-mc/include/mc-bus.h

[PATCH 2/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u16' preferred over 'uint16_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u16' over 'uint16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/bus/dpmcp.h | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpmc

[PATCH 3/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u8' preferred over 'uint8_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u8' over 'uint8_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/bus/dpmcp.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/stagi

[PATCH 2/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u16' preferred over 'uint16_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u16' over 'uint16_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/bus/dpmcp.h | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/staging/fsl

[PATCH 3/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u8' preferred over 'uint8_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u8' over 'uint8_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/bus/dpmcp.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/staging/fsl-mc/bus/dpmcp.h

[PATCH 1/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u32' preferred over 'uint32_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u32' over 'uint32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/bus/dpmcp.h | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpmc

[PATCH 6/6] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging/

[PATCH 5/6] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc/includ

[PATCH 4/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u64' preferred over 'uint64_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u64' over 'uint64_t'. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/bus/dpmcp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/staging/fsl-mc/bus/d

[PATCH 1/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u32' preferred over 'uint32_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u32' over 'uint32_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/bus/dpmcp.h | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/staging

[PATCH 6/6] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging/fsl-mc/include/mc-bus.h

[PATCH 5/6] Staging: fsl-mc: include: mc: Kernel type 's32' preferred over 'int32_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 's32' over 'int32_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/include/mc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/include/mc.h b/drivers/staging/fsl-mc/include/mc.h index f6e720e

[PATCH 4/6] Staging: fsl-mc: bus: dpmcp: Kernel type 'u64' preferred over 'uint64_t'

2016-11-07 Thread Shiva Kerdel
Follow the kernel type preferrences of using 'u64' over 'uint64_t'. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/bus/dpmcp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/staging/fsl-mc/bus/dpmcp.h index 854621a

[PATCH] Staging: fsl-mc: Changed allocation types into the types that the kernel prefers

2016-11-06 Thread Shiva Kerdel
Fixed reports from checkpatch that pointed out that the code was not using the preferred types of the kernel. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/fsl-mc/bus/dpmcp.h | 80 - drivers/staging/fsl-mc/include/mc-bus.h | 4 +- d

[PATCH] Staging: fsl-mc: Changed allocation types into the types that the kernel prefers

2016-11-06 Thread Shiva Kerdel
Fixed reports from checkpatch that pointed out that the code was not using the preferred types of the kernel. Signed-off-by: Shiva Kerdel --- drivers/staging/fsl-mc/bus/dpmcp.h | 80 - drivers/staging/fsl-mc/include/mc-bus.h | 4 +- drivers/staging/fsl-mc

[PATCH] Staging: speakup: Remove unnecessary space after cast

2016-11-06 Thread Shiva Kerdel
The affected files have been modified to remove redundant spaces after casts to solve checkpatch.pl checks. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/speakup/main.c | 42 ++--- drivers/staging/speakup/selection.c | 2 +- d

[PATCH] Staging: speakup: Remove unnecessary space after cast

2016-11-06 Thread Shiva Kerdel
The affected files have been modified to remove redundant spaces after casts to solve checkpatch.pl checks. Signed-off-by: Shiva Kerdel --- drivers/staging/speakup/main.c | 42 ++--- drivers/staging/speakup/selection.c | 2 +- drivers/staging/speakup

[PATCH] Staging: vme: vme_pio2: Prefer using the BIT macro

2016-11-05 Thread Shiva Kerdel
Replace all occurences of (1 << x) by BIT(x) in the file vme_pio2.h to get rid of checkpatch.pl "check" output "Prefer using the BIT macro". Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/vme/devices/vme_pio2.h | 86 +-

[PATCH] Staging: vme: vme_pio2: Prefer using the BIT macro

2016-11-05 Thread Shiva Kerdel
Replace all occurences of (1 << x) by BIT(x) in the file vme_pio2.h to get rid of checkpatch.pl "check" output "Prefer using the BIT macro". Signed-off-by: Shiva Kerdel --- drivers/staging/vme/devices/vme_pio2.h | 86 +- 1 file cha

[PATCH] Staging: rtl8723au: rtw_ieee80211: Fixed operators spacing style issues

2016-08-05 Thread Shiva Kerdel
Fixed spaces around operators to fix their coding style issues. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 80 +- 1 file changed, 40 insertions(+), 40 deletions(-) diff --git a/drivers/staging/rtl8723a

[PATCH] Staging: rtl8723au: rtw_ieee80211: Fixed operators spacing style issues

2016-08-05 Thread Shiva Kerdel
Fixed spaces around operators to fix their coding style issues. Signed-off-by: Shiva Kerdel --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 80 +- 1 file changed, 40 insertions(+), 40 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ieee80211.c b

[PATCH] Staging: rtl8723au: rtw_xmit: fixed coding style issues

2016-08-02 Thread Shiva Kerdel
Fixed some coding style issues that were detected as errors. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/rtl8723au/core/rtw_xmit.c | 26 ++ 1 file changed, 10 insertions(+), 16 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_xmi

[PATCH] Staging: rtl8723au: rtw_xmit: fixed coding style issues

2016-08-02 Thread Shiva Kerdel
Fixed some coding style issues that were detected as errors. Signed-off-by: Shiva Kerdel --- drivers/staging/rtl8723au/core/rtw_xmit.c | 26 ++ 1 file changed, 10 insertions(+), 16 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c b/drivers/staging

[PATCH] Staging: rtl8723au: rtw_xmit: Fixed operators spacing style issues

2016-08-02 Thread Shiva Kerdel
Fixed spaces around operators to fix their coding style issues. Signed-off-by: Shiva Kerdel <sh...@exdev.nl> --- drivers/staging/rtl8723au/core/rtw_xmit.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_xmi

[PATCH] Staging: rtl8723au: rtw_xmit: Fixed operators spacing style issues

2016-08-02 Thread Shiva Kerdel
Fixed spaces around operators to fix their coding style issues. Signed-off-by: Shiva Kerdel --- drivers/staging/rtl8723au/core/rtw_xmit.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c b/drivers/staging