On Thu, Nov 24, 2016 at 03:10:29PM -0700, Jason Litzinger wrote:
> Fix checkpatch warnings regarding the use of symbolic permissions.
>
> Where the MOST_CHANNEL_ATTR macro is used, convert to octal permissions
> over symbolic.
>
> Where _ATTR is used directly, replace with _ATTR_RW/_ATTR_WO and
>
Fix to avoid possible memory leak if the decoder initialization
got failed.Free the allocated memory for file handle object
before return in case decoder initialization fails.
Signed-off-by: Shailendra Verma
---
drivers/staging/media/davinci_vpfe/vpfe_video.c |3 +++
1 file changed, 3 insert
Fixed checkpatch.pl errors related to "space prohibited after that '*'
or '&'" in ks_wlan_net.c file.
Signed-off-by: Yamanappagouda Patil
---
drivers/staging/ks7010/ks_wlan_net.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/ks7010/ks_wlan_net.c
On Fri, Nov 25, 2016 at 12:13:04AM +0100, Lino Sanfilippo wrote:
> --- a/include/linux/pci_ids.h
> +++ b/include/linux/pci_ids.h
> @@ -2015,6 +2015,24 @@
> #define PCI_SUBDEVICE_ID_CCD_OV4S0xE888
> #define PCI_SUBDEVICE_ID_CCD_OV8S0xE998
>
> +#define PCI_VENDOR_ID_ALACRITECH
Hi,
>
> On Fri, Nov 25, 2016 at 12:13:04AM +0100, Lino Sanfilippo wrote:
> > --- a/include/linux/pci_ids.h
> > +++ b/include/linux/pci_ids.h
> > @@ -2015,6 +2015,24 @@
> > #define PCI_SUBDEVICE_ID_CCD_OV4S 0xE888
> > #define PCI_SUBDEVICE_ID_CCD_OV8S 0xE998
> >
> > +#define PCI_VENDOR_ID_ALA
On 24/11/16 20:31, Saber Rezvani wrote:
From c3dd43f3d43e7bec8f7f1140519ffc1f3b727b01 Mon Sep 17 00:00:00 2001
From: Saber
Date: Thu, 24 Nov 2016 23:15:21 +0330
Subject: [PATCH 1/3] staging: comedi: s626: use preferred kernel type u8
All that extra header stuff above will have to be removed.
On 24/11/16 20:32, Saber Rezvani wrote:
From 946376d81bd841696316123586028914448523d1 Mon Sep 17 00:00:00 2001
From: Saber
Date: Thu, 24 Nov 2016 23:25:47 +0330
Subject: [PATCH 2/3] staging: comedi: s626: use preferred kernel type u16
Fix the checkpatch.pl issue:
CHECK: Prefer kernel type 'u16'
On 24/11/16 20:33, Saber Rezvani wrote:
From 5a907ea23f93f4358a91eb75915ef4b6134a9034 Mon Sep 17 00:00:00 2001
From: Saber
Date: Thu, 24 Nov 2016 23:35:52 +0330
Subject: [PATCH 3/3] staging: comedi: s626: use preferred kernel type u32
Fix the checkpatch.pl issue:
CHECK: Prefer kernel type 'u32'
On Fri, Nov 18, 2016 at 11:48:42AM -0500, James Simmons wrote:
> diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h
> b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h
> index 93bff1b..fc180b8 100644
> --- a/drivers/staging/lustre/include/linux/libcfs/libcfs_pri
Hi Laura,
Thanks for what you are doing on ION.
As soon that will be ok I will submit again the driver for sti platform.
After reading your mail I have the feeling that solves cache,
devicetree and legacy support problems will be difficult.
Since "Unix Device Memory Allocator" claims that it wil
To get prepared to CPU offlining support we need co change the way how we
unbind clockevent devices. As one CPU may go online/offline multiple times
we need to bind it in hv_synic_init() and unbind it in hv_synic_cleanup().
The is an additional corner case: when we unload the module completely we
n
It should never happen, but let's get prepared to receiving a confirmation
for VMBus unload on an offlined CPU. As we allocate all structures for all
present CPUs now it's safe.
Signed-off-by: Vitaly Kuznetsov
---
drivers/hv/channel_mgmt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-
"kernel BUG at drivers/hv/channel_mgmt.c:350!" is observed when hv_vmbus
module is unloaded. BUG_ON() was introduced in commit 85d9aa705184
("Drivers: hv: vmbus: add an API vmbus_hvsock_device_unregister()") as
vmbus_free_channels() codepath was apparently forgotten.
Fixes: 85d9aa705184 ("Drivers:
Some time ago we forbade CPU offlining for Hyper-V and this was sufficient
if you boot with all CPUs onlined. Turns out, people may want to limit the
number online CPUs by passing 'maxcpus=' kernel parameter and we hit a
crash in Hyper-V code in this case. After some thinking, I think we may not
on
Initializing hv_context.percpu_list in hv_synic_alloc() helps to prevent a
crash in percpu_channel_enq() when not all CPUs were online during
initialization and it naturally belongs there.
Signed-off-by: Vitaly Kuznetsov
---
drivers/hv/hv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(
It may happen that not all CPUs are online when we do hv_synic_alloc() and
in case more CPUs come online later we may try accessing these allocated
structures.
Signed-off-by: Vitaly Kuznetsov
---
drivers/hv/hv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/hv/h
To make it possible to online/offline CPUs switch to cpuhp infrastructure
for doing hv_synic_init()/hv_synic_cleanup().
Signed-off-by: Vitaly Kuznetsov
---
drivers/hv/hv.c | 15 +++
drivers/hv/hyperv_vmbus.h | 4 ++--
drivers/hv/vmbus_drv.c| 19 +++
3 f
Since commit e513229b4c38 ("Drivers: hv: vmbus: prevent cpu offlining on
newer hypervisors") cpu offlining was disabled. It is still true that we
can't offline CPUs which have VMBus channels bound to them but we may have
'free' CPUs (e.v. we booted with maxcpus= parameter and onlined CPUs after
VMB
On Fri, Nov 18, 2016 at 01:07:51PM -0800, Laura Abbott wrote:
> I'd like to keep using linaro-mm-sig as a mailing list for Ion
> discussion as well as drivers-devel.
You could add it to MAINTAINERS so people thinking about ION know it
exists.
regards,
dan carpenter
__
-permissions-usage/20161125-194750
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
wget
https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross
-O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the
On Sat, Nov 19, 2016 at 12:35:43PM -0500, Walt Feasel wrote:
> Make suggested checkpatch modification for
> CHECK: spaces preferred around that '+,|,-,/'
>
> Signed-off-by: Walt Feasel
> ---
> v2 makes changes to correct for email format patch submission
>
> drivers/staging/speakup/synth.c | 12
Signed-off-by: wenhungyang
---
drivers/staging/most/mostcore/core.c | 40 ++--
1 file changed, 20 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/most/mostcore/core.c
b/drivers/staging/most/mostcore/core.c
index 4c580d1..9286c36 100644
--- a/drivers/s
Hi Lino,
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Lino-Sanfilippo/net-ethernet-slicoss-add-slicoss-gigabit-ethernet-driver/20161125-190558
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1
Align parameters to open parenthesis. Also remove one blank line in
sequence of two.
Signed-off-by: Dawid Kurek
---
drivers/staging/gdm724x/gdm_lte.h | 14 +++---
drivers/staging/gdm724x/gdm_tty.h | 1 -
drivers/staging/gdm724x/netlink_k.h | 3 ++-
3 files changed, 9 insertions(+),
On Sun, Nov 20, 2016 at 05:01:52PM -0800, Vijai Kumar K wrote:
> + *Please insert all the queue initialization using _init_queue below
Add a space after the '*'.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://dri
On Wed, Nov 23, 2016 at 06:01:45PM -0500, James Simmons wrote:
> From: Jinshan Xiong
>
> Early launch page LRU work in osc_io_rw_iter_init();
> Change the page LRU shrinking policy by OSC attributes;
> Delete the contented lock osc_object::oo_seatbelt
>
The cli_name() stuff should be in a separ
Litzinger
---
changelog:
v2) Fix missing semicolon on hunk with _ATTR_WO.
Tested/built/loaded linux-next(next-20161125)
Test-built staging-next from staging tree
drivers/staging/most/mostcore/core.c | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/drivers
On Nov 24, 2016, at 04:12, Dan Carpenter wrote:
>
> We should free "desc" before returning NULL.
>
> Signed-off-by: Dan Carpenter
Reviewed-by: Andreas Dilger
> diff --git a/drivers/staging/lustre/lustre/ptlrpc/client.c
> b/drivers/staging/lustre/lustre/ptlrpc/client.c
> index ac959ef..80474
On Nov 18, 2016, at 09:48, James Simmons wrote:
>
> Remove the zero comparisions in the libcfs headers.
>
> Signed-off-by: James Simmons
> ---
> .../lustre/include/linux/libcfs/libcfs_crypto.h|2 +-
> .../lustre/include/linux/libcfs/libcfs_fail.h |4 +-
> .../lustre/include/linux
fix drivers/staging/most/mostcore/core.c coding style
1. move static from MACRO to function name
2. change symbolic permission such as 'S_IRUGO' to number
>From 3e1dba83090c58251aae1dc7aca431bb11632a98 Mon Sep 17 00:00:00 2001
From: wenhungyang
Date: Wed, 23 Nov 2016 21:55:36 +0800
Subject: [c15
Ignore patch version...comments below
Will resend it
On 24.11.2016 19:38, Boyan Vladinov wrote:
Fixes sysfs entries user/group modes and coding style warnings
found by checkpatch.pl tool. Also use the IIO_DEVICE_ATTR_[RO|RW] and
macros to create device attributes
Signed-off-by: Boyan Vladinov
On Nov 24, 2016, at 04:10, Dan Carpenter wrote:
>
> We want sizeof(struct lstcon_node) but instead we're getting the sizeof
> a pointer.
>
> Fixes: 8d78f0f2ba76 ("staging: lustre: lnet: cleanup some of the > 80 line
> issues")
> Signed-off-by: Dan Carpenter
Reviewed-by: Andreas Dilger
>
>
32 matches
Mail list logo
