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

[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 changed, 4 insertions(+), 2 deletio

[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 insertions(+), 2 deletions(-)

[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(+) diff --git a/drivers/staging

[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 1 file change

[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 1 file changed, 43 insertions(+), 43 deletions(-) diff --git a/drivers/staging/comedi/drivers/dmm32at.c b/drivers/staging/comedi/drivers/dmm32at.c index 958c0d4..3c4722a 1

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 Kershner > Reviewed-by: Tim Sell > --- > drivers/stag

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 things by making periodic_work.

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; gre.

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; hof...@osadl.org;

[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 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 f

[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 --- drivers/staging/comedi/comed

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 READ_ONC

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 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 > ; Sell, Timothy C ; > hof...@osadl

[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 b/drivers/staging/comedi/drivers/das16.c index fd8e0b7..9bf1f29

[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 a/drivers/staging/comedi/drivers/das16.c b/drivers/st

[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 --git a/drivers/staging/comedi/

[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, 43 insertions(+), 43 deletions(-) diff --git a/drivers/staging/comedi/drivers/dmm32at.c b/drivers/staging/comedi/drivers/dmm32at.c index 958c0d4..4ca6104 100644 --

[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(-) diff --git a/drivers/staging/c

[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 insertions(+), 51 deletions(-)

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, and is the relative device

Re: [PATCH] Staging: unisys: visorhba: visorhba_main: fixed a coding style issue

2016-06-07 Thread Greg KH
On Sat, May 21, 2016 at 12:30:46AM +0530, Rumesh Hapuarachchi wrote: > fixed checkpatch.pl warning about 'Prefer 'unsigned int' to bare use of > 'unsigned' > > Signed-off-by: Rumesh Hapuarahcchi > --- > drivers/staging/unisys/visorhba/visorhba_main.c | 6 +++--- > 1 file changed, 3 insertions(+