[PATCH v4 6/6] staging: ks7010: fix coding style issue of aligning comments properly

This patch fixes coding style issue of having block comments using a trailing */ on a separate line, warning as issued by checkpatch Signed-off-by: Chetan Sethi --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: -

[PATCH v4 5/6] staging: ks7010: fix coding style issue of using __func__ instead of __FUNCTION__

This patch fixes coding style issue of using __func__ instead of gcc specific __FUNCTION__, warning as issued by checkpatch Signed-off-by: Chetan Sethi --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified

[PATCH v4 4/6] staging: ks7010: fix coding style issue of using pr_notice instead of printk

This patch fixes coding style issue of using pr_notice instead of printk, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified description to

[PATCH v4 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

This patch fixes error of enclosing complex macro value in parentheses, error as issued by checkpatch Signed-off-by: Chetan Sethi --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject - incorporated review comment of correct

[PATCH v4 3/6] staging: ks7010: fix coding style issue of using tabs instead of spaces

This patch fixes coding style issue of using tabs instead of spaces at start of line, error as issued by checkpatch.pl Signed-off-by: Chetan Sethi --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified

[PATCH v4 1/6] staging: ks7010: fixed warning of avoiding line over 80 characters

This patch fixes warning of line over 80 characters, as issued by checkpatch.pl Signed-off-by: Chetan Sethi --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified description to exclude mention of patch number

Re: [PATCH v3 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

Hi Greg, > Always run checkpatch on your patches so you don't get a grumpy > maintainer telling you to run your patches through checkpatch! Since one change per patch has been included, initial patches in series will not be completely error/warning free, after running checkpatch. However it has

[PATCH] Staging: vt6656: Fix spaces and char limit

This is a patch to usbpipe.c to fix space and char limit warning found by checkpatch tool. Signed-off-by: Jitendra Khasdev --- drivers/staging/vt6656/usbpipe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/vt6656/usbpipe.c

[PATCH] speakup: synth: Fix checkpatch warning

Fix following checkpatch warning: WARNING: line over 80 characters +>attributes) < 0) total: 0 errors, 1 warnings, 485 lines checked Signed-off-by: Man Choy --- drivers/staging/speakup/synth.c | 4 ++-- 1 file

[PATCH 9/9] Staging: rtl8712: wlan_bssdef.h - style fix

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/wlan_bssdef.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8712/wlan_bssdef.h b/drivers/staging/rtl8712/wlan_bssdef.h

[PATCH 8/9] Staging: rtl8712: wifi.h - style fix

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/wifi.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8712/wifi.h b/drivers/staging/rtl8712/wifi.h index

[PATCH 7/9] Staging: rtl8712: rtl871x_recv.h - style fix

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_recv.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_recv.h

[PATCH 6/9] Staging: rtl8712: rtl871x_pwrctrl.h - style fix

Fixed style fo block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_pwrctrl.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_pwrctrl.h

[PATCH 5/9] Staging: rtl8712: rtl871x_mp_ioctl.h - style fix

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_mp_ioctl.h | 32 +++--- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_mp_ioctl.h

[PATCH 3/9] Staging: rtl8712: rtl871x_io.h - style fix

Fixed style of block comments. Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_io.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_io.h

[PATCH 4/9] Staging: rtl8712: rtl871x_mlme.h - style fix

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_mlme.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_mlme.h

[PATCH 1/9] Staging: rtl8712: rtl871x_cmd.h - block comments

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_cmd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8712/rtl871x_cmd.h b/drivers/staging/rtl8712/rtl871x_cmd.h index

[PATCH 0/9] drivers: staging: rtl8712 - style fix

Fixed style of block comments across whole of the rtl8712 driver Found using check patch Derek Robson (9): Staging: rtl8712: rtl871x_cmd.h - block comments Staging: rtl8712: rtl871x_event.h - style fix Staging: rtl8712: rtl871x_io.h - style fix Staging: rtl8712: rtl871x_mlme.h - style

[PATCH 2/9] Staging: rtl8712: rtl871x_event.h - style fix

Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/rtl8712/rtl871x_event.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_event.h

[PATCH] Staging: bcm2835-audio - style fix, block comments

Fixed block comments as found by checkpatch Signed-off-by: Derek Robson --- drivers/staging/bcm2835-audio/bcm2835-ctl.c | 3 ++- drivers/staging/bcm2835-audio/bcm2835-pcm.c | 3 ++- drivers/staging/bcm2835-audio/bcm2835.h | 5 - 3 files changed, 4 insertions(+), 7

Re: [PATCH V2] staging: vchiq_2835_arm: Make cache-line-size a required DT property

On Sat, 2017-02-25 at 19:15 +0100, Stefan Wahren wrote: > > > > What's the rule as to who exactly I should be including on > > patches?  > > get_maintainer.pl displays a very long list for stagging, so it > > isn't > > clear to me who should be getting this. > > At least the active maintainers

Re: [PATCH V2] staging: vchiq_2835_arm: Make cache-line-size a required DT property

> > What's the rule as to who exactly I should be including on patches? > get_maintainer.pl displays a very long list for stagging, so it isn't > clear to me who should be getting this. At least the active maintainers (bcm2835 + staging) and the relevant mailing lists. Usually i try to avoid

Re: [PATCH V2] staging: vchiq_2835_arm: Make cache-line-size a required DT property

On Sat, 2017-02-25 at 14:38 +0100, Stefan Wahren wrote: > [add Eric] > > > Michael Zoran hat am 18. Februar 2017 um > > 12:59 geschrieben: > > > > > > On Sat, 2017-02-18 at 03:22 -0800, Michael Zoran wrote: > > > The original github source allowed for the cache-line-size >

Re: [PATCH V2] staging: vchiq_2835_arm: Make cache-line-size a required DT property

[add Eric] > Michael Zoran hat am 18. Februar 2017 um 12:59 > geschrieben: > > > On Sat, 2017-02-18 at 03:22 -0800, Michael Zoran wrote: > > The original github source allowed for the cache-line-size property > > to be missing. Since recent firmwares also require this

[PATCH 1/3] staging: vchiq_arm: Remove unnecessary includes

Including those headers is unnecessary, so we better remove them. Signed-off-by: Stefan Wahren --- .../interface/vchiq_arm/vchiq_2835_arm.c |2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 3/3] staging: vchiq_arm: Don't define PAGE_SIZE

There is no need to define PAGE_SIZE in the vchiq driver so remove the define from vchiq_pagelist.h . Signed-off-by: Stefan Wahren --- .../interface/vchiq_arm/vchiq_pagelist.h |3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 2/3] staging: vchiq_arm: Include linux/mm.h

vchiq_2835_arm.c uses functions and macros like put_page() or PAGE_ALIGN(). So we better include the necessary header. Signed-off-by: Stefan Wahren --- .../interface/vchiq_arm/vchiq_2835_arm.c |1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] Staging: bcm2835-audio: Removed redundant check

This was reported by checkpatch.pl Signed-off-by: Alexandru Jercaianu --- drivers/staging/bcm2835-audio/bcm2835-pcm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/bcm2835-audio/bcm2835-pcm.c

Re: [Resend PATCH 2/2 v3] pci-hyperv: lock pci bus on device eject

On Fri, Feb 24, 2017 at 09:49:17PM +, Long Li wrote: > A PCI_EJECT message can arrive at the same time we are calling > pci_scan_child_bus in the workqueue for the previous PCI_BUS_RELATIONS > message or in create_root_hv_pci_bus(), in this case we could potentailly > modify the bus from