[PATCH] fixed spelling error in TODO file for dgnc driver

2016-12-21 Thread Scott Matheina
fixed a missing letter in the TODO file 'unneeded' Signed-off-by: Scott Matheina --- drivers/staging/dgnc/TODO | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/TODO b/drivers/staging/dgnc/TODO index 0e0825b..6c91bbd 100644 --- a/drivers/staging/dgnc/TODO

Re: [PATCH] indention and space fixes to align with linux style

2016-12-21 Thread Scott Matheina
On 12/20/2016 09:23 PM, Joe Perches wrote: On Tue, 2016-12-20 at 20:58 -0600, Scott Matheina wrote: Fixed indention and space issues to align the code with the linux style guide. [] diff --git a/drivers/staging/iio/addac/adt7316.c b/drivers/staging/iio/addac/adt7316.c [] @@ -176,16 +176,1

[PATCH v2 0/2] staging: greybus: loopback fixes and updates

2016-12-21 Thread Bryan O'Donoghue
Two simple patches here resulting from using greybus on gbsim and developing support for async. I found a bug in the user-space tool and while doing that decided to update the kernel thread to be better behaved when waiting for test completions. V2: Use available bundle pointer for dev_dbg printou

[PATCH v2 1/2] staging: greybus: loopback: use gb_loopback_async_wait_all don't spin

2016-12-21 Thread Bryan O'Donoghue
Currently the greybus-loopback thread logic spins around waiting for send_count == iteration_max which on real hardware doesn't make a difference to us but in simulation is excruciatingly slow, anti-social and bad manners. Use the existing gb_loopback_async_wait_all() function to gate continuing wh

[PATCH v2 2/2] staging: greybus: loopback_test: Fix race preventing test completion

2016-12-21 Thread Bryan O'Donoghue
commit 9250c0ee2626 ("greybus: Loopback_test: use poll instead of inotify") changes the flow of determining when to break out of a loop polling for loopback test completion. The clause is_complete() which determines if all tests are complete - as used is subject to a race condition where one of th

RE: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread KY Srinivasan
> -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Wednesday, December 21, 2016 10:03 AM > To: Christoph Hellwig > Cc: Paolo Bonzini ; Roman Kagan > ; Radim Krčmář ; KY > Srinivasan ; Vitaly Kuznetsov > ; k...@vger.kernel.org; Denis V . Lunev > ; H

[patch 05/10] staging/lustre/libcfs: Convert to hotplug state machine

2016-12-21 Thread Thomas Gleixner
From: Anna-Maria Gleixner Install the callbacks via the state machine. No functional change. Signed-off-by: Anna-Maria Gleixner Signed-off-by: Sebastian Andrzej Siewior Cc: de...@driverdev.osuosl.org Cc: Andreas Dilger Cc: Greg Kroah-Hartman Cc: Oleg Drokin Cc: r...@linutronix.de Cc: lustre

RE: [PATCH 10/15] hyperv: uapi-fy PostMessage and SignalEvent hypercall structures

2016-12-21 Thread KY Srinivasan
> -Original Message- > From: Roman Kagan [mailto:rka...@virtuozzo.com] > Sent: Tuesday, December 20, 2016 7:56 AM > To: Paolo Bonzini ; Radim Krčmář > ; KY Srinivasan ; Vitaly > Kuznetsov > Cc: Thomas Gleixner ; Ingo Molnar > ; H. Peter Anvin ; x...@kernel.org; > Haiyang Zhang ; k...@vge

RE: [PATCH 03/15] hyperv: use standard bitops

2016-12-21 Thread KY Srinivasan
> -Original Message- > From: Roman Kagan [mailto:rka...@virtuozzo.com] > Sent: Tuesday, December 20, 2016 7:56 AM > To: Paolo Bonzini ; Radim Krčmář > ; KY Srinivasan ; Vitaly > Kuznetsov > Cc: Thomas Gleixner ; Ingo Molnar > ; H. Peter Anvin ; x...@kernel.org; > Haiyang Zhang ; k...@vge

RE: [PATCH 06/15] hyperv: avoid unnecessary vmalloc

2016-12-21 Thread KY Srinivasan
> -Original Message- > From: Roman Kagan [mailto:rka...@virtuozzo.com] > Sent: Tuesday, December 20, 2016 7:56 AM > To: Paolo Bonzini ; Radim Krčmář > ; KY Srinivasan ; Vitaly > Kuznetsov > Cc: Thomas Gleixner ; Ingo Molnar > ; H. Peter Anvin ; x...@kernel.org; > Haiyang Zhang ; k...@vge

[PATCH] Staging: comedi: comedidev.h: Drop old style zero-length array

2016-12-21 Thread Cheah Kok Cheong
According to Documentation/Changes, the minimum gcc version required to compile the kernel is 3.2 (this is probably outdated too). Signed-off-by: Cheah Kok Cheong --- drivers/staging/comedi/comedidev.h | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/staging/com

RE: [PATCH 02/15] hyperv: uapi-fy synic event flags definitions

2016-12-21 Thread KY Srinivasan
> -Original Message- > From: Roman Kagan [mailto:rka...@virtuozzo.com] > Sent: Tuesday, December 20, 2016 7:56 AM > To: Paolo Bonzini ; Radim Krčmář > ; KY Srinivasan ; Vitaly > Kuznetsov > Cc: Thomas Gleixner ; Ingo Molnar > ; H. Peter Anvin ; x...@kernel.org; > Haiyang Zhang ; k...@vge

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Stephen Hemminger
On Wed, 21 Dec 2016 09:58:36 -0800 Christoph Hellwig wrote: > On Wed, Dec 21, 2016 at 09:50:49AM -0800, Stephen Hemminger wrote: > > Lastly, there is licensing issues on headers. It would be good to have any > > userspace ABI headers licensed with a more liberal license so that BSD and > > DPDK

RE: [PATCH 00/15] hyperv: more stuff to uapi + cleanup

2016-12-21 Thread KY Srinivasan
> -Original Message- > From: Roman Kagan [mailto:rka...@virtuozzo.com] > Sent: Tuesday, December 20, 2016 7:56 AM > To: Paolo Bonzini ; Radim Krčmář > ; KY Srinivasan ; Vitaly > Kuznetsov > Cc: Thomas Gleixner ; Ingo Molnar > ; H. Peter Anvin ; x...@kernel.org; > Haiyang Zhang ; k...@vge

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Christoph Hellwig
On Wed, Dec 21, 2016 at 09:50:49AM -0800, Stephen Hemminger wrote: > Lastly, there is licensing issues on headers. It would be good to have any > userspace ABI headers licensed with a more liberal license so that BSD and > DPDK drivers > could use them directly. Right now each one reinvents. Micr

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Paolo Bonzini
On 21/12/2016 18:50, Stephen Hemminger wrote: > The other problem with the hyperv headers is they were initially done with > only the Linux driver usage in mind. This made perfect sense at the time, > the problem is that they mix internal state with protocol definitions. Yes, and this was partly

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Stephen Hemminger
On Wed, 21 Dec 2016 07:43:48 -0800 Christoph Hellwig wrote: > On Wed, Dec 21, 2016 at 04:39:18PM +0100, Paolo Bonzini wrote: > > That said, there are precedents in using UAPI this way for PV > > interfaces. See for example include/uapi/linux/virtio*.h and > > arch/x86/include/uapi/asm/kvm_para.h

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Paolo Bonzini
On 21/12/2016 16:43, Christoph Hellwig wrote: > On Wed, Dec 21, 2016 at 04:39:18PM +0100, Paolo Bonzini wrote: >> That said, there are precedents in using UAPI this way for PV >> interfaces. See for example include/uapi/linux/virtio*.h and >> arch/x86/include/uapi/asm/kvm_para.h. > > We have al

[PATCH] rtlwifi: rtl_usb: Fix missing entry in USB driver's private data

2016-12-21 Thread Larry Finger
These drivers need to be able to reference "struct ieee80211_hw" from the driver's private data, and vice versa. The USB driver failed to store the address of ieee80211_hw in the private data. Although this bug has been present for a long time, it was not exposed until commit ba9f93f82aba ("rtlwifi

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Christoph Hellwig
On Wed, Dec 21, 2016 at 04:39:18PM +0100, Paolo Bonzini wrote: > That said, there are precedents in using UAPI this way for PV > interfaces. See for example include/uapi/linux/virtio*.h and > arch/x86/include/uapi/asm/kvm_para.h. We have all kinds of historical examples, but most of them turned i

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Paolo Bonzini
On 21/12/2016 15:26, Christoph Hellwig wrote: > On Wed, Dec 21, 2016 at 03:59:20PM +0300, Roman Kagan wrote: >> That's fine by me. >> >> I guess the series should then start with a complete move >> arch/x86/include/uapi/asm/hyperv.h -> >> arch/x86/include/asm/hyperv_proto.h, and the remaining pat

[PATCH] staging: greybus: Fix spaces on a single definition statement

2016-12-21 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - "CHECK: spaces preferred around that '*' (ctx:VxV) " Signed-off-by: Emmanuil Chatzipetru --- drivers/staging/greybus/svc_watchdog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drive

[PATCH] staging: greybus: Fix static array of pointers declaration as static const.

2016-12-21 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - "WARNING: char * array declaration might be better as static const" Signed-off-by: Emmanuil Chatzipetru --- drivers/staging/greybus/svc_watchdog.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH] staging: greybus: replace printk() with pr_err()

2016-12-21 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - "WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ." Signed-off-by: Emmanuil Chatzipetru --- drivers/staging/greybus/svc_watchdog.c |

????????????????????

2016-12-21 Thread ???
???.xls Description: application/msexcel ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] Staging: vt6656: checkpatch: fix identifier name in wcmd.h

2016-12-21 Thread Tobias Heineken
This is a patch to the wcmd.h file that fixes up two identifier name warnings found by the checkpatch.pl tool at lines 54 and 56 Signed-off-by: Tobias Heineken Signed-off-by: Florian Schleicher CC: linux-ker...@i4.cs.fau.de --- drivers/staging/vt6656/wcmd.h | 4 ++-- 1 file changed, 2 insertion

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Roman Kagan
On Wed, Dec 21, 2016 at 06:26:54AM -0800, Christoph Hellwig wrote: > On Wed, Dec 21, 2016 at 03:59:20PM +0300, Roman Kagan wrote: > > That's fine by me. > > > > I guess the series should then start with a complete move > > arch/x86/include/uapi/asm/hyperv.h -> > > arch/x86/include/asm/hyperv_proto

Re: rtlwifi: Fix kernel oops introduced with commit e49656147359

2016-12-21 Thread Kalle Valo
Larry Finger wrote: > With commit e49656147359 {"rtlwifi: Use dev_kfree_skb_irq instead of > kfree_skb"), the method used to free an skb was changed because the > kfree_skb() was inside a spinlock. What was forgotten is that kfree_skb() > guards against a NULL value for the argument. Routine dev_k

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Christoph Hellwig
On Wed, Dec 21, 2016 at 03:59:20PM +0300, Roman Kagan wrote: > That's fine by me. > > I guess the series should then start with a complete move > arch/x86/include/uapi/asm/hyperv.h -> > arch/x86/include/asm/hyperv_proto.h, and the remaining patches have to > change the latter instead of the former

Re: [PATCH 03/15] hyperv: use standard bitops

2016-12-21 Thread Roman Kagan
On Wed, Dec 21, 2016 at 01:00:44PM +0100, Olaf Hering wrote: > On Tue, Dec 20, Roman Kagan wrote: > > Reverting commit 22356585712d ("staging: hv: use sync_bitops when > interacting with the hypervisor") is save because ... > > > - sync_set_bit(channel->monitor_bit, > > +

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Roman Kagan
On Wed, Dec 21, 2016 at 04:18:58AM -0800, Christoph Hellwig wrote: > On Wed, Dec 21, 2016 at 09:29:39AM +0300, Roman Kagan wrote: > > QEMU in particular. We're planning to implement VMBus devices in QEMU > > and would like to have the definitions shared with the Linux guest > > drivers for Hyper-V

Re: [PATCH 12/15] hyperv: move VMBus connection ids to uapi

2016-12-21 Thread Christoph Hellwig
On Wed, Dec 21, 2016 at 09:29:39AM +0300, Roman Kagan wrote: > QEMU in particular. We're planning to implement VMBus devices in QEMU > and would like to have the definitions shared with the Linux guest > drivers for Hyper-V. And that's everything but a userspace API. The way to go for protocol c

Re: [PATCH 03/15] hyperv: use standard bitops

2016-12-21 Thread Olaf Hering
On Tue, Dec 20, Roman Kagan wrote: Reverting commit 22356585712d ("staging: hv: use sync_bitops when interacting with the hypervisor") is save because ... > - sync_set_bit(channel->monitor_bit, > + set_bit(channel->monitor_bit, Olaf signature.asc Description: PGP s

Re: [PATCH 02/15] hyperv: uapi-fy synic event flags definitions

2016-12-21 Thread kbuild test robot
Hi Roman, [auto build test ERROR on linus/master] [also build test ERROR on v4.9 next-20161221] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Roman-Kagan/hyperv-more-stuff-to-uapi-cleanup

Re: [PATCH] staging: greybus: add host device function pointer checks

2016-12-21 Thread Bryan O'Donoghue
On 20/12/16 20:49, Jason Hrycay wrote: > Add sanity checks for cport_quiesce and cport_clear before invoking the > callbacks as these function pointers are not required during the host > device registration. This follows the logic implemented elsewhere for > various other function pointers. > > Si