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 check my current working file alr
(#!##* 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, 201
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 a/drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c
b/drivers/staging/rt
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(+), 19 deletions(-)
diff --git
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 a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
b/drivers/staging
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 a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
b/d
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 happens on
> >> the s
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 story and as there are V
> -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-
> p...@vger.kernel.org; linux-ker...@vger.kernel.org
> Subject: Re: [Resend PATCH 1
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 nu
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
>
> drivers/pci/host/pci-hype
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 'its_fsl_mc_msi_clea
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
drivers/staging
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
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
>>> userspace.
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 af83f2a..5277f36 100644
--- a/drivers/s
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
index 64a9ebc..af83f2a 100644
--
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
b/drivers/staging/wlan-ng/prism2mgmt.c
index 16fb2d3..64a9ebc
On 11/02/17 21:08, Artur Lorincz wrote:
Fixed multiple line dereference for &cmd->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 a/drivers/staging/comedi/drivers/adl_pci9118.c
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: wlan
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 +-
drivers/staging/wlan-ng/p80211conv.h | 4 ++--
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 c558ad656c49..b641a59052ac 100644
--- a/
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 a/drivers/staging/comedi/drivers/addi_apci_3
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?
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 achiev
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...
h
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:
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 a/drivers/staging/comedi/drivers/s626.c
b/
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
> drivers/staging/rtl8712/rtl871x_ioctl_linux.c:1424
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 onl
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 a/drivers/staging/co
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 deletions(-)
diff --git
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 --git a/drivers/staging
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 a/drivers/staging/comedi/dr
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
> ---
> drivers/staging/greybus/arche-apb-ctrl
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
de...@linuxdriverproject.org
http://driverdev.linuxd
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
>
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
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(-)
diff --git a/drivers/sta
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
http://driverdev.linuxdriverproject.org/mailman/listinfo
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 a/drivers/staging/vc04_service
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 unsigned int [usertype]
*out_valu
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
___
43 matches
Mail list logo
