Re: [PATCH 3/3] ANDROID: binder: fix proc->tsk check.

2017-07-30 Thread Amit Pundir
On 28 July 2017 at 17:26, Martijn Coenen wrote: > Commit c4ea41ba195d ("binder: use group leader instead of open thread")' > was incomplete and didn't update a check in binder_mmap(), causing all > mmap() calls into the binder driver to fail. > Fixes Android WiFi/BT regression

Re: [greybus-dev] [PATCH 10/11] greybus: usb: constify hc_driver structures

2017-07-30 Thread Viresh Kumar
On 28-07-17, 22:41, Julia Lawall wrote: > The hc_driver structure is only passed as the first argument to > usb_create_hcd, which is declared as const. Thus the hc_driver structure > itself can be const. > > Done with the help of Coccinelle. > > Signed-off-by: Julia Lawall

[PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-30 Thread Harinath Nampally
This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be supported by this driver and current design doesn't have the

[PATCH] hyperv: netvsc: Neaten netvsc_send_pkt by using a temporary

2017-07-30 Thread Joe Perches
Repeated dereference of nvmsg.msg.v1_msg.send_rndis_pkt can be shortened by using a temporary. Do so. No change in object code. Signed-off-by: Joe Perches --- drivers/net/hyperv/netvsc.c | 24 ++-- 1 file changed, 10 insertions(+), 14 deletions(-) diff

[PATCH] staging: ccree: Use sizeof(variable) in memory allocs

2017-07-30 Thread Simon Sandström
Fixes 9 checkpatch.pl warnings of type "Prefer kmalloc(sizeof(variable)...) over kmalloc(sizeof(type)...)" in staging/ccree. Signed-off-by: Simon Sandström --- drivers/staging/ccree/ssi_aead.c| 4 ++-- drivers/staging/ccree/ssi_cipher.c | 5 ++---

[staging:staging-testing 202/216] include/linux/msi.h:196:21: fatal error: asm/msi.h: No such file or directory

2017-07-30 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: 5551ad1e9fad2cf06288c6e93cb95f879b8cdf5e commit: 03274850279c8da86358adfdad1422a850b066ac [202/216] staging: fsl-mc: allow the driver compile multi-arch config: powerpc-allyesconfig (attached as

Re: [PATCH v1 3/6] staging: unisys: Switch to use new generic UUID API

2017-07-30 Thread Greg Kroah-Hartman
On Sun, Jul 30, 2017 at 08:26:48PM +0300, Andy Shevchenko wrote: > On Sun, Jul 30, 2017 at 6:32 PM, Greg Kroah-Hartman > wrote: > > On Wed, Jul 26, 2017 at 01:01:41PM +0300, Andy Shevchenko wrote: > >> On Wed, 2017-07-19 at 21:28 +0300, Andy Shevchenko wrote: > >> >

[staging:staging-testing 202/216] include/linux/msi.h:225:10: error: unknown type name 'msi_alloc_info_t'

2017-07-30 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: 5551ad1e9fad2cf06288c6e93cb95f879b8cdf5e commit: 03274850279c8da86358adfdad1422a850b066ac [202/216] staging: fsl-mc: allow the driver compile multi-arch config: sparc64-allmodconfig (attached as

Re: [PATCH v1 3/6] staging: unisys: Switch to use new generic UUID API

2017-07-30 Thread Andy Shevchenko
On Sun, Jul 30, 2017 at 6:32 PM, Greg Kroah-Hartman wrote: > On Wed, Jul 26, 2017 at 01:01:41PM +0300, Andy Shevchenko wrote: >> On Wed, 2017-07-19 at 21:28 +0300, Andy Shevchenko wrote: >> > There are new types and helpers that are supposed to be used in new >> >

Re: [PATCH v1 3/6] staging: unisys: Switch to use new generic UUID API

2017-07-30 Thread Greg Kroah-Hartman
On Wed, Jul 26, 2017 at 01:01:41PM +0300, Andy Shevchenko wrote: > On Wed, 2017-07-19 at 21:28 +0300, Andy Shevchenko wrote: > > There are new types and helpers that are supposed to be used in new > > code. > > > > As a preparation to get rid of legacy types and API functions do > > the

Re: [PATCH] staging: pi433: Use matching enum types calling rf69_set_packet_format

2017-07-30 Thread Greg Kroah-Hartman
On Sat, Jul 29, 2017 at 11:29:13AM +0200, Elia Geretto wrote: > This patch fixes the following four warnings found using sparse: > > drivers/staging/pi433/pi433_if.c:211:9: warning: mixing different enum types > drivers/staging/pi433/pi433_if.c:211:9: int enum optionOnOff versus >

Re: [PATCH v2] staging: octeon: fix line over 80 characters

2017-07-30 Thread Greg KH
On Wed, Jul 26, 2017 at 07:40:39PM +0300, catalin...@gmail.com wrote: > From: John Smith > > ethernet-rx.c: > > fix WARNING: line over 80 characters > > The code was restructured a bit, a helper function > was added to cvm_oct_poll. > > Signed-off-by: John Smith

Re: [PATCH 18/20] staging: lustre: llite: Remove filtering of seclabel xattr

2017-07-30 Thread Greg Kroah-Hartman
On Thu, Jul 27, 2017 at 08:35:33PM +0100, James Simmons wrote: > > > From: Robin Humble > > > > The security.capability xattr is used to implement File > > Capabilities in recent Linux versions. Capabilities are a > > fine grained approach to granting executables

[PATCH] staging: ccree: Fix format/argument mismatches

2017-07-30 Thread Joe Perches
By default, debug logging is disabled by CC_DEBUG not being defined. Convert SSI_LOG_DEBUG to use no_printk instead of an empty define to validate formats and arguments. Fix fallout. Miscellanea: o One of the conversions now uses %pR instead of multiple uses of %pad Signed-off-by: Joe Perches

Re: [PATCH 2/3] staging: ccree: Convert to devm_ioremap_resource for map, unmap

2017-07-30 Thread Gilad Ben-Yossef
On Thu, Jul 27, 2017 at 10:48 PM, Dan Carpenter wrote: > On Thu, Jul 27, 2017 at 05:27:33PM +0300, Gilad Ben-Yossef wrote: >> + new_drvdata->cc_base = devm_ioremap_resource(_dev->dev, >> + req_mem_cc_regs); >> + if

[PATCH v2 6/6] staging: ccree: simplify resource release on error

2017-07-30 Thread Gilad Ben-Yossef
The resource release on probe/init error was being handled in an awkward manner and possibly leaking memory on certain (unlikely) error path. Fix it by simplifying the error resource release and making it easier to track. Reported-by: Dan Carpenter Signed-off-by: Gilad

[PATCH v2 4/6] staging: ccree: Convert to devm_ioremap_resource for map, unmap

2017-07-30 Thread Gilad Ben-Yossef
From: Suniel Mahesh It is recommended to use managed function devm_ioremap_resource(), which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace request_mem_region(), ioremap() and corresponding error handling with

[PATCH v2 5/6] staging: ccree: Use platform_get_irq and devm_request_irq

2017-07-30 Thread Gilad Ben-Yossef
From: Suniel Mahesh It is recommended to use managed function devm_request_irq(), which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace platform_get_resource(), request_irq() and corresponding error handling with

[PATCH v2 3/6] staging: ccree: Replace kzalloc with devm_kzalloc

2017-07-30 Thread Gilad Ben-Yossef
From: Suniel Mahesh It is recommended to use managed function devm_kzalloc, which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace kzalloc with devm_kzalloc. (b) drop kfree(), because memory allocated with devm_kzalloc() is

[PATCH v2 2/6] staging: ccree: kmalloc by sizeof var not type

2017-07-30 Thread Gilad Ben-Yossef
Change places where we alloc memory by sizeof type to sizeof var. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c| 4 ++-- drivers/staging/ccree/ssi_cipher.c | 4 ++-- drivers/staging/ccree/ssi_driver.c | 2 +-

[PATCH v2 1/6] staging: ccree: fix split strings

2017-07-30 Thread Gilad Ben-Yossef
Fix strings in log messages being split across lines and the resulting alignment issues when being fixed. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_aead.c| 9 ++-- drivers/staging/ccree/ssi_buffer_mgr.c | 86 ++---

[PATCH v2 0/6] ccree style fixes and simplification

2017-07-30 Thread Gilad Ben-Yossef
Various style code fixes and simplification from myself and Suniel. Changes from v2: - Rebase on top of current staging-next. - Added resource release on error simplification patch suggested by Dan C. as part of the review. I made sure the code applies on top both current staging-next (commit

Re: [PATCH 1/1] staging: pi433: fix problem with division in rf69_set_deviation

2017-07-30 Thread Marcus Wolf
Hi Greg, hi Dan the toolingproblem is a little bit more complicated. Since a mainline kernel isn't directly able to boot on the RasPi, for testing I work with a Raspbian kernel. Since at the beginning, I never thought of submitting the driver, I started with a side-build project with SVN - that's