Re: [PATCH 4/9] staging: unisys: visorhba: simplify and enhance debugfs interface

2016-06-07 Thread Greg KH
On Thu, May 12, 2016 at 09:14:43AM -0400, David Kershner wrote: > From: Tim Sell > > debugfs info for each visorhba device is now presented by a file named of > the following form within the debugfs tree: > > visorhba/vbus:dev/info > > where is the vbus number,

[PATCH 2/5] Staging: comedi: das16: fix Block comment

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This fixes up a WARNING: 'Block comments use a trailing */ on a separate line'found by the checkpatch.pl tool. Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das16.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-)

[PATCH 3/5] Staging: comedi: das800: fix comment issue

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This fixes up a WARNING: 'Block comments use a trailing */ on a separate line'found by the checkpatch.pl tool Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das800.c | 102 1 file changed, 51

[PATCH 4/5] Staging: comedi: das800: Prefer unsigned int instead of unsigned

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This fixes up a WARNING: Prefer 'unsigned int' to bare use of 'unsigned' found by the checkpatch.pl tool. Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das800.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH 5/5] Staging: comedi: dmm32at: Prefer using the BIT macro

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This fixes all occurences of (1< --- drivers/staging/comedi/drivers/dmm32at.c | 86 1 file changed,

[PATCH 1/5] Staging: comedi: das16: fix blank line

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This fixes up a blank line after function/struct/union/enum check found by the checkpatch.pl tool Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das16.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 2/2] staging: comedi: das16: fixed coding style issue

2016-06-07 Thread Akshay Shipurkar
Fixed a multi-line comment coding style issue. Signed-off-by: Akshay Shipurkar --- drivers/staging/comedi/drivers/das16.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/das16.c

RE: [PATCH v3 04/30] staging: unisys: visorbus: remove unused module parameters

2016-06-07 Thread Binder, David Anthony
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Tuesday, June 07, 2016 9:23 AM > To: Kershner, David A > Cc: cor...@lwn.net; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; gre...@linuxfoundation.org; Arfvidson, Erik >

how to assign issue which require patch to my to do list

2016-06-07 Thread rspawnwolf
Dear sir, What the best place to assign issues which require patch to my to do list. Best ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH] Warnings in comedi_fops.c

2016-06-07 Thread Greg KH
On Wed, Jun 08, 2016 at 01:30:01AM +0530, Rithvik Patibandla wrote: > **Intro** Why this line? > The following patch fixes warnings thrown by checkpatch.pl script. The > warnings are: > "It is preferable to use WRITE_ONCE(, ) instead of > ACCESS_ONCE() = " > > "It is preferable to use

[PATCH] Warnings in comedi_fops.c

2016-06-07 Thread Rithvik Patibandla
**Intro** The following patch fixes warnings thrown by checkpatch.pl script. The warnings are: "It is preferable to use WRITE_ONCE(, ) instead of ACCESS_ONCE() = " "It is preferable to use READ_ONCE() instead of ACCESS_ONCE()" Signed-off-by: Rithvik Patibandla ---

RE: [PATCH 5/5] Staging: comedi: Prefer using the BIT macro issue in dmm32at.c

2016-06-07 Thread Hartley Sweeten
On Tuesday, June 07, 2016 4:51 AM, Ravishankar Karkala Mallikarjunayya wrote: > Subject: [PATCH 5/5] Staging: comedi: Prefer using the BIT macro issue in > dmm32at.c Please don't add random patches to a series. You started this series with 1/4, 2/4, 3/4 and 4/4 then added a 5/5. Also, please

[PATCH] Drivers: hv: utils: fix a race on userspace daemons registration

2016-06-07 Thread Vitaly Kuznetsov
Background: userspace daemons registration protocol for Hyper-V utilities drivers has two steps: 1) daemon writes its own version to kernel 2) kernel reads it and replies with module version at this point we consider the handshake procedure being completed and we do hv_poll_channel() transitioning

RE: [PATCH v3 11/30] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-07 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Tuesday, June 07, 2016 9:40 AM > To: Kershner, David A > Cc: cor...@lwn.net; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; gre...@linuxfoundation.org; Arfvidson, Erik; Sell, Timothy > C;

Re: [PATCH v3 11/30] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-07 Thread Neil Horman
On Tue, Jun 07, 2016 at 01:47:13PM +, Sell, Timothy C wrote: > > -Original Message- > > From: Neil Horman [mailto:nhor...@redhat.com] > > Sent: Tuesday, June 07, 2016 9:40 AM > > To: Kershner, David A > > Cc: cor...@lwn.net; t...@linutronix.de; mi...@redhat.com; > > h...@zytor.com;

Re: [PATCH v3 11/30] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-07 Thread Neil Horman
On Sat, Jun 04, 2016 at 01:27:11PM -0400, David Kershner wrote: > From: Tim Sell > > A kernel timer is now used as the vehicle to periodically call the > channel_interrupt function of registered visor drivers, instead of a > workqueue. > > This simplifies a lot of

Re: [PATCH v3 04/30] staging: unisys: visorbus: remove unused module parameters

2016-06-07 Thread Neil Horman
On Sat, Jun 04, 2016 at 01:27:04PM -0400, David Kershner wrote: > From: David Binder > > Removes unused module parameters from visorbus_main.c, in response to > findings by SonarQube. > > Signed-off-by: David Binder > Signed-off-by: David

[PATCH 5/5] Staging: comedi: Prefer using the BIT macro issue in dmm32at.c

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This patch Replace all occurences of (1< --- drivers/staging/comedi/drivers/dmm32at.c | 86

[PATCH 1/4] Staging: comedi: fix blank line issue in das16.c

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This is a patch to the das16.c file that fixes up a blank line after function/struct/union/enum check found by the checkpatch.pl tool Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das16.c | 1 + 1 file changed, 1 insertion(+)

[PATCH 3/4] Staging: comedi: fix comment issue fixed for das800.c

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This is a patch to the das800.c file that fixes up a WARNING: 'Block comments use a trailing */ on a separate line' found by the checkpatch.pl tool Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das800.c | 102

[PATCH 2/4] Staging: comedi: Block comment issue fixed for das16.c

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This is a patch to the das16.c file that fixes up a WARNING: 'Block comments use a trailing */ on a separate line' found by the checkpatch.pl tool. Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das16.c | 6 -- 1 file

[PATCH 4/4] Staging: comedi: Prefer unsigned int instead of unsigned in das800.c

2016-06-07 Thread Ravishankar Karkala Mallikarjunayya
This is a patch to the das800.c file that fixes up a WARNING: Prefer 'unsigned int' to bare use of 'unsigned' found by the checkpatch.pl tool. Signed-off-by: Ravishankar Karkala Mallikarjunayya --- drivers/staging/comedi/drivers/das800.c | 4 ++-- 1 file changed, 2

Re: [Linaro-mm-sig] [RFC][PATCH 0/6] ion: improved ABI

2016-06-07 Thread Chen Feng
The idea is good, define the heap ids in header file is inconvenient. But if we query the heaps information from user-space. It need to maintain this ids and name userspace one by one. The code may be complicated in different module user-space. In android, the gralloc and other lib will all use