Re: [PATCH] bcm2835-audio: Fix checkpatch errors

On Mon, Feb 13, 2017 at 8:03 PM, Greg Kroah-Hartman wrote: > On Mon, Feb 13, 2017 at 08:49:06AM +0800, Man Choy wrote: >> A quick question, I just updated my linux-next tree last night, >> checkout tag "next-20170210" and working on the cleanup. How do I >> double

Re: [PATCH v3 21/24] media: imx: Add MIPI CSI-2 Receiver subdev driver

(#!##* Thunderbird! resending text only) Hi Philipp, On 02/13/2017 01:20 AM, Philipp Zabel wrote: Hi Steve, On Thu, 2017-02-09 at 15:51 -0800, Steve Longerbeam wrote: On 02/09/2017 03:49 PM, Steve Longerbeam wrote: On 02/08/2017 03:42 PM, Russell King - ARM Linux wrote: On Wed, Feb 08,

[PATCH 04/15] staging: rtl8192u: blank lines aren't necessary before a close brace '}'

Fix checkpatch issues: "CHECK: Blank lines aren't necessary before a close brace '}'". Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 03/15] staging: rtl8192u: Adding space after enum and struct definition

Fixes checkpatch.pl warning: WARNING: missing space after struct definition WARNING: missing space after enum definition Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/rtl819x_HT.h | 38 - 1 file changed, 19 insertions(+),

[PATCH 02/15] staging: rtl8192u: Adding space after struct definition

Fixes checkpatch.pl warning: WARNING: missing space after struct definition Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 01/15] staging: rtl8192u: fixing white space issue

Fix the following checkpatch.pl error and warnings: WARNING: please, no space before tabs Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 2/2] x86/vdso: Add VCLOCK_HVCLOCK vDSO clock read method

On Mon, 13 Feb 2017, Andy Lutomirski wrote: > On Sun, Feb 12, 2017 at 11:49 PM, Dexuan Cui wrote: > >> From: Thomas Gleixner [mailto:t...@linutronix.de] > >> Sent: Saturday, February 11, 2017 02:02 > >> ... > >> That's important if the stuff happens cross CPU. If the update

Re: [PATCH 2/2] x86/vdso: Add VCLOCK_HVCLOCK vDSO clock read method

On Sun, Feb 12, 2017 at 11:49 PM, Dexuan Cui wrote: >> From: Thomas Gleixner [mailto:t...@linutronix.de] >> Sent: Saturday, February 11, 2017 02:02 >> ... >> That's important if the stuff happens cross CPU. If the update happens on >> the same CPU then this is a different

RE: [Resend PATCH 1/2 v3] pci-hyperv: properly handle pci bus remove

> -Original Message- > From: Bjorn Helgaas [mailto:bhelg...@google.com] > Sent: Saturday, February 11, 2017 9:35 AM > To: Long Li > Cc: KY Srinivasan ; Haiyang Zhang > ; de...@linuxdriverproject.org; linux- >

[PATCH] pci-hyperv: Use device serial number as PCI domain

This allows PCI domain numbers starts with 1, and also unique on the same VM. So names, such as VF NIC names, that include domain number as part of the name, can be shorter than that based on part of bus UUID previously. The new names will also stay same for VMs created with copied VHD and same

Re: [PATCH v2 3/5] pci: set msi_domain_ops as __ro_after_init

On Fri, Feb 10, 2017 at 05:37:56PM -0800, Jess Frazelle wrote: > Marked msi_domain_ops structs as __ro_after_init when called only during init. > This protects the data structure from accidental corruption. > > Suggested-by: Kees Cook > Signed-off-by: Jess Frazelle

[PATCH] staging: fsl-mc: Add missing header

Compiling the fsl-mc bus driver will yield a couple of static analysis errors: warning: symbol 'fsl_mc_msi_domain_alloc_irqs' was not declared warning: symbol 'fsl_mc_msi_domain_free_irqs' was not declared. warning: symbol 'its_fsl_mc_msi_init' was not declared. warning: symbol

Re: [PATCH 6/7] staging: r8712u: Fix Sparse warnings in rtl871x_ioctl_linux.c

On 02/13/2017 05:27 AM, David Laight wrote: From: Larry Finger Sent: 11 February 2017 03:30 Sparse reports the following: CHECK drivers/staging/rtl8712/rtl871x_ioctl_linux.c drivers/staging/rtl8712/rtl871x_ioctl_linux.c:1422:46: warning: restricted __le16 degrades to integer

Re: [PATCH 0/3 staging-next] android: Lowmemmorykiller task tree

On 02/10/2017 11:27 AM, Michal Hocko wrote: > [I have only now see this cover - it answers some of the questions I've > had to specific patches. It would be really great if you could use git > send-email to post patch series - it just does the right thing(tm)] > > On Thu 09-02-17 14:21:40, peter

Re: [PATCH 1/3 staging-next] android: Collect statistics from lowmemorykiller

On 02/10/2017 10:15 AM, Michal Hocko wrote: > On Fri 10-02-17 10:05:34, peter enderborg wrote: >> On 02/10/2017 08:59 AM, Michal Hocko wrote: > [...] >>> The approach was wrong from the day 1. Abusing slab shrinkers >>> is just a bad place to stick this logic. This all belongs to the >>>

[PATCHv6 3/3] staging: wlan-ng: realign else if continuation

realign else if continuation Signed-off-by: Maksymilian Piechota --- drivers/staging/wlan-ng/prism2mgmt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c index

[PATCHv6 2/3] staging: wlan-ng: move else if statement to a single line

move else if statement to a single line Signed-off-by: Maksymilian Piechota --- drivers/staging/wlan-ng/prism2mgmt.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c

[PATCHv6 1/3] staging: wlan-ng: move logical continuations at the end of line

move logical continuations at the end of line Signed-off-by: Maksymilian Piechota --- drivers/staging/wlan-ng/prism2mgmt.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c

Re: [PATCH 01/01] comedi: fixed multiple line dereference

On 11/02/17 21:08, Artur Lorincz wrote: Fixed multiple line dereference for >scan_begin_arg. Signed-off-by: Artur Lorincz --- drivers/staging/comedi/drivers/adl_pci9118.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git

[PATCHv6 0/3] staging: wlan-ng: align else if statement to coding standard

PATCHv0: - Spaces replaced with tabs Maksymilian Piechota (1): staging: wlan-ng: This patch fixes the checkpatch.pl warning: else if statement still did not fulfill coding standard requirements. PATCHv2: - else if statement moved to single line Maksymilian Piechota (1): staging:

[PATCH] wlan-ng/p80211conv.c: Fix warnings regarding types.

Fix several warnings in p80211conv.c emitted when using make C=1 by turning the type member in wlan_ethhdr and wlan_snap into __be16 and changing le16_to_cpu into ntohs. Signed-off-by: Anthony Brandon --- drivers/staging/wlan-ng/p80211conv.c | 2 +-

[PATCH] Staging: wlan-ng: Fixed code style issue:

Statements should start on a tabstop Signed-off-by: YU Bo --- drivers/staging/wlan-ng/prism2mgmt.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c index

Re: [patch 01/01] comedi: made comedi_lrange struct constant

On 11/02/17 20:26, Artur Lorincz wrote: Added the const type qualifier to the comedi_lrange structure. Signed-off-by: Artur Lorincz --- drivers/staging/comedi/drivers/addi_apci_3501.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] bcm2835-audio: Fix checkpatch errors

On Mon, Feb 13, 2017 at 08:49:06AM +0800, Man Choy wrote: > A quick question, I just updated my linux-next tree last night, > checkout tag "next-20170210" and working on the cleanup. How do I > double check my current working file already been updated by someone > else to avoid the duplicate work?

Re: [PATCH v2] Staging: comedi: drivers: comedi_test: Add auto-configuration capability

On 11/02/17 10:37, Cheah Kok Cheong wrote: Currently this module needs to be manually configured by COMEDI userspace tool before the test waveform can be read by a COMEDI compatible application. This patch adds auto-configuration capability and makes it the default loading option. This is

Re: [PATCH 3/20] staging: rtl8192u: Prefer using the BIT macro

On Mon, Feb 13, 2017 at 02:45:04PM +0530, SIMRAN SINGHAL wrote: > Ok, I will send this patch again with the proper label. What are you referring to? Please always properly quote your emails so we know what you are talking about. Remember, some of us get over a thousand emails a day...

RE: [PATCH 5/7] staging: r8712u: Fix Sparse endian warning in rtl871x_recv.c

From: Larry Finger > Sent: 11 February 2017 03:30 > Sparse reports the following: > CHECK drivers/staging/rtl8712/rtl871x_recv.c > drivers/staging/rtl8712/rtl871x_recv.c:657:21: warning: incorrect type in > assignment (different base > types) > drivers/staging/rtl8712/rtl871x_recv.c:657:21:

Re: [PATCH v2 1/1] staging: comedi: s626: usleep_range is preferred over udelay

On 10/02/17 18:14, Saber Rezvani wrote: Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/s626.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

RE: [PATCH 6/7] staging: r8712u: Fix Sparse warnings in rtl871x_ioctl_linux.c

From: Larry Finger > Sent: 11 February 2017 03:30 > Sparse reports the following: > CHECK drivers/staging/rtl8712/rtl871x_ioctl_linux.c > drivers/staging/rtl8712/rtl871x_ioctl_linux.c:1422:46: warning: restricted > __le16 degrades to integer >

RE: [PATCH 1/7] staging: rtl8712: Fix some Sparse endian messages

From: Larry Finger > Sent: 11 February 2017 03:30 > Sparse reports the following: > > CHECK drivers/staging/rtl8712/rtl8712_xmit.c > drivers/staging/rtl8712/rtl8712_xmit.c:564:42: warning: cast from restricted > __le32 ... I think you ought to do separate patches for the changes that are

Re: [PATCH 1/1] staging: comedi: ni_pcidio: blank line issues

On 10/02/17 19:28, Saber Rezvani wrote: Fix the checkpatch.pl issue: CHECK: Please use a blank line after function/struct/union/enum declarations Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/ni_pcidio.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH 1/1] staging: comedi: ni_pcidio.c: Spaces preferred around operators

On 10/02/17 19:00, Saber Rezvani wrote: Fix the checkpatch.pl issue: CHECK: spaces preferred around that '|' (ctx:VxV) Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/ni_pcidio.c | 40 +++--- 1 file changed, 20 insertions(+), 20

Re: [PATCH] Staging: media: bcm2048: Fixed coding style issue.

On 02/12/2017 10:45 PM, Ran Algawi wrote: > Fixed a coding style issue, where an octal value was needed insted of decimal. > > Signed-off-by: Ran Algawi > --- > drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff

Re: [PATCH v2 1/1] staging: comedi: dyna_pci10xx: usleep_range is preferred over udelay

On 10/02/17 18:11, Saber Rezvani wrote: Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/dyna_pci10xx.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [PATCH 1/2] staging: greybus: fix "line over 80 characters" coding style issues

On Thu, Feb 09, 2017 at 05:30:11PM +0100, Gioh Kim wrote: > This patch fixes only obvious lines. > There are still more issues. Please make this commit message self-contained (e.g. mention what you are doing here and why). > Signed-off-by: Gioh Kim > --- >

Re: [PATCH 2/2] staging: greybus: fix symbolic permission coding style issues

On Thu, Feb 09, 2017 at 05:30:12PM +0100, Gioh Kim wrote: > Fix "Octal permissions are preffered than symbolic ones" issues. > > Signed-off-by: Gioh Kim Acked-by: Johan Hovold ___ devel mailing list

RE: [PATCH 2/2] x86/vdso: Add VCLOCK_HVCLOCK vDSO clock read method

On Mon, 13 Feb 2017, Dexuan Cui wrote: > > From: Thomas Gleixner [mailto:t...@linutronix.de] > > Sent: Saturday, February 11, 2017 02:02 > > ... > > That's important if the stuff happens cross CPU. If the update happens on > > the same CPU then this is a different story and as there are VMexits >

Re: [PATCH v3 21/24] media: imx: Add MIPI CSI-2 Receiver subdev driver

Hi Steve, On Thu, 2017-02-09 at 15:51 -0800, Steve Longerbeam wrote: > > On 02/09/2017 03:49 PM, Steve Longerbeam wrote: > > > > > > On 02/08/2017 03:42 PM, Russell King - ARM Linux wrote: > >> On Wed, Feb 08, 2017 at 03:23:53PM -0800, Steve Longerbeam wrote: > Actually, this exact function

[PATCH v2 1/1] Staging: vc04_services: Fix the "space prohibited" code style errors

This patch fixes the following code style errors: ERROR: space prohibited after/before that open/closed parenthesis Signed-off-by: Mandel Benjamin --- drivers/staging/vc04_services/interface/vchi/vchi_common.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

Re: [PATCH 3/20] staging: rtl8192u: Prefer using the BIT macro

Ok, I will send this patch again with the proper label. But do I have to send all the patches again of this patch series? Thanks, Simran Singhal ___ devel mailing list de...@linuxdriverproject.org

[PATCH v2 0/1] Staging: vc04_services: Fix the "wrong indent" code style errors

This patch fixes the following code style errors: ERROR: code indent should use tabs where possible Signed-off-by: Mandel Benjamin --- drivers/staging/vc04_services/interface/vchi/vchi_common.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] staging: nvec: Fix incorrect type of i2c address

From: Franck Demathieu The i2c address is unsigned according to the dt-bindings. Fix sparse issue (-Wtypesign): drivers/staging/nvec/nvec.c:781:35: warning: incorrect type in argument 3 (different signedness) drivers/staging/nvec/nvec.c:781:35:expected

Re: [PATCH] staging: greybus: arpc.h: remove duplicate line.

On Fri, Feb 10, 2017 at 11:16:37PM -0500, Nathan Howard wrote: > Fix checkpatch.pl issue of the form: > "CHECK: Please don't use multiple blank lines". Don't take checkpatch too seriously. Using double blank lines as a separator is perfectly fine. Thanks, Johan