[PATCH] staging wlan-ng: fix type mismatch warnings in mkpdrlist()

2017-04-08 Thread Maciek Borzecki
ast to restricted __le16 drivers/staging/wlan-ng/prism2fw.c:625:21: warning: cast to restricted __le16 Signed-off-by: Maciek Borzecki --- drivers/staging/wlan-ng/hfa384x.h | 4 ++-- drivers/staging/wlan-ng/prism2fw.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/driv

Re: [PATCH] staging wlan-ng: mark PDA buf as __le16

2017-04-08 Thread Maciek Borzecki
On Sat, Apr 8, 2017 at 3:18 PM, Maciek Borzecki wrote: > On Sat, Apr 8, 2017 at 1:01 PM, Greg Kroah-Hartman > wrote: >> On Fri, Apr 07, 2017 at 02:47:55PM +0200, Maciek Borzecki wrote: >>> PDA buffer contains little-endian data, clearly mark it as such. >>

Re: [PATCH] staging wlan-ng: mark PDA buf as __le16

2017-04-08 Thread Maciek Borzecki
On Sat, Apr 8, 2017 at 1:01 PM, Greg Kroah-Hartman wrote: > On Fri, Apr 07, 2017 at 02:47:55PM +0200, Maciek Borzecki wrote: >> PDA buffer contains little-endian data, clearly mark it as such. >> >> Fixes sparse warnings: >> drivers/staging/wlan-ng/hfa384x_usb.

[PATCH] staging wlan-ng: mark PDA buf as __le16

2017-04-07 Thread Maciek Borzecki
:2382:44: warning: cast to restricted __le16 Signed-off-by: Maciek Borzecki --- drivers/staging/wlan-ng/hfa384x_usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/hfa384x_usb.c b/drivers/staging/wlan-ng/hfa384x_usb.c index

[PATCH] staging: wlan-ng: prism2sta: replace memcmp with ether_addr_equal

2015-10-03 Thread Maciek Borzecki
() Signed-off-by: Maciek Borzecki --- drivers/staging/wlan-ng/prism2sta.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c index 0329c521d17c5c0ee5baa237c81383021daefe24

[PATCH 0/3] [media] staging: lirc: mostly checkpatch cleanups

2015-09-04 Thread Maciek Borzecki
media tree. The first patch fixes minor warning with unnecessary brakes around single statement block. The second fixes non-tab indentation. The third patch does away with a custom dprintk() in favor of dev_dbg and pr_debug(). Maciek Borzecki (3): [media] staging: lirc: remove unnecessary b

[PATCH 3/3] [media] staging: lirc: lirc_serial: use dynamic debugs

2015-09-04 Thread Maciek Borzecki
... + printk(KERN_DEBUG LIRC_DRIVER_NAME ": " \ Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_serial.c | 32 ++-- 1 file changed, 10 insertions(+), 22 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_serial.c b/drivers/sta

[PATCH 2/3] [media] staging: lirc: fix indentation

2015-09-04 Thread Maciek Borzecki
Fix non-tab indentation. This resolves the following checkpatch problem: ERROR: code indent should use tabs where possible Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_sasem.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media

[PATCH 1/3] [media] staging: lirc: remove unnecessary braces

2015-09-04 Thread Maciek Borzecki
Remove unnecessary braces where appropriate. This removes the following checkpatch warnings: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_imon.c | 8 1 file changed, 4 insertions(+), 4 deletions

[PATCH v3 1/3] staging: media: lirc: remove unnecessary braces

2015-07-21 Thread Maciek Borzecki
Remove unnecessary braces where appropriate. This removes the following checkpatch warnings: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_imon.c | 8 1 file changed, 4 insertions(+), 4 deletions

[PATCH v3 0/3] staging: media: lirc: mostly checkpatch cleanups

2015-07-21 Thread Maciek Borzecki
block. The second fixes non-tab indentation. The third patch does away with a custom dprintk() in favor of dev_dbg and pr_debug(). Maciek Borzecki (3): staging: media: lirc: remove unnecessary braces staging: media: lirc: fix indentation staging: media: lirc: lirc_serial: use dynamic debugs

[PATCH v3 2/3] staging: media: lirc: fix indentation

2015-07-21 Thread Maciek Borzecki
Fix non-tab indentation. This resolves the following checkpatch problem: ERROR: code indent should use tabs where possible Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_sasem.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media

[PATCH v3 3/3] staging: media: lirc: lirc_serial: use dynamic debugs

2015-07-21 Thread Maciek Borzecki
... + printk(KERN_DEBUG LIRC_DRIVER_NAME ": " \ Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_serial.c | 33 ++-- 1 file changed, 10 insertions(+), 23 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_serial.c b/drivers/sta

Re: [PATCH v2] staging: media: lirc: fix various checkpatch warnings

2015-07-20 Thread Maciek Borzecki
On Tue, Jul 21, 2015 at 8:19 AM, Sudip Mukherjee wrote: > On Mon, Jul 20, 2015 at 06:10:04PM +0200, Maciek Borzecki wrote: >> Remove unnecessary braces where appropriate. Fix non-tab >> indentation. >> >> This removes the following checkpatch warnings: >> WARNING

[PATCH v2] staging: media: lirc: fix various checkpatch warnings

2015-07-20 Thread Maciek Borzecki
Remove unnecessary braces where appropriate. Fix non-tab indentation. This removes the following checkpatch warnings: WARNING: braces {} are not necessary for single statement blocks ERROR: code indent should use tabs where possible Signed-off-by: Maciek Borzecki --- drivers/staging/media

Re: [PATCH 1/2] staging: media: lirc: fix various checkpatch warnings

2015-07-19 Thread Maciek Borzecki
On Mon, Jul 20, 2015 at 7:38 AM, Sudip Mukherjee wrote: > On Fri, Jul 17, 2015 at 10:57:44PM +0200, Maciek Borzecki wrote: >> Remove unnecessary brakes where appropriate. > > brakes or braces? Right, I obviously meant braces. Unfortunate typo that the spellchecker is still fin

[PATCH 1/2] staging: media: lirc: fix various checkpatch warnings

2015-07-17 Thread Maciek Borzecki
Remove unnecessary brakes where appropriate. This removes the following checkpatch warning: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_imon.c | 8 drivers/staging/media/lirc/lirc_sasem.c | 4

[PATCH 2/2] staging: media: lirc: lirc_serial: use dynamic debugs

2015-07-17 Thread Maciek Borzecki
... + printk(KERN_DEBUG LIRC_DRIVER_NAME ": " \ Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc/lirc_serial.c | 25 +++-- 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_serial.c b/drivers/staging/

[PATCH 0/2] staging: lirc: mostly checkpatch cleanups

2015-07-17 Thread Maciek Borzecki
more 'invasive' than the first one. Maciek Borzecki (2): staging: media: lirc: fix various checkpatch warnings staging: media: lirc: lirc_serial: use dynamic debugs drivers/staging/media/lirc/lirc_imon.c | 8 drivers/staging/media/lirc/lirc_sasem.c | 4 ++-- drivers/sta