Re: [PATCH v2 2/2] staging: iio: ad7606: move out of staging

2016-12-09 Thread kbuild test robot
Hi Eva, [auto build test WARNING on iio/togreg] [also build test WARNING on next-20161209] [cannot apply to v4.9-rc8] [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/Eva-Rachel-Retuya/staging

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 22:35:05 + Haiyang Zhang wrote: > > > > > > > > Emulated NIC is already excluded in start of netvc notifier handler. > > > > > > > > static int netvsc_netdev_event(struct notifier_block *this, > > > >unsigned long event, void *ptr) > > > > {

Re: [PATCH 10/14] rtlwifi: Add BTC_TRACE_STRING to new btcoex

2016-12-09 Thread Markus Böhme
On 12/09/2016 09:50 AM, Kalle Valo wrote: > Dan Carpenter writes: > >> On Thu, Dec 08, 2016 at 02:50:49PM +0300, Dan Carpenter wrote: >>> On Thu, Dec 08, 2016 at 01:43:42PM +0200, Kalle Valo wrote: But it would make me very happy if someone would add a similar grouping functionality to

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Haiyang Zhang
> -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Friday, December 9, 2016 5:05 PM > To: Haiyang Zhang > Cc: Greg KH ; KY Srinivasan > ; o...@aepfle.de; linux-ker...@vger.kernel.org; > bjorn.helg...@gmail.com; a...@canonical.com; de...@linuxdriver

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 21:53:49 + Haiyang Zhang wrote: > > -Original Message- > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Friday, December 9, 2016 4:45 PM > > To: Haiyang Zhang > > Cc: Greg KH ; KY Srinivasan > > ; o...@aepfle.de; linux-ker...@vger.kernel.or

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Haiyang Zhang
> -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Friday, December 9, 2016 4:45 PM > To: Haiyang Zhang > Cc: Greg KH ; KY Srinivasan > ; o...@aepfle.de; linux-ker...@vger.kernel.org; > bjorn.helg...@gmail.com; a...@canonical.com; de...@linuxdriver

Re: [PATCH v2 2/2] staging: iio: ad7606: move out of staging

2016-12-09 Thread kbuild test robot
Hi Eva, [auto build test WARNING on iio/togreg] [also build test WARNING on next-20161209] [cannot apply to v4.9-rc8] [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/Eva-Rachel-Retuya/staging

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 21:31:25 + Haiyang Zhang wrote: > > -Original Message- > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Friday, December 9, 2016 3:30 PM > > To: Haiyang Zhang > > Cc: Greg KH ; KY Srinivasan > > ; o...@aepfle.de; linux-ker...@vger.kernel.or

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Haiyang Zhang
> -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Friday, December 9, 2016 3:30 PM > To: Haiyang Zhang > Cc: Greg KH ; KY Srinivasan > ; o...@aepfle.de; linux-ker...@vger.kernel.org; > bjorn.helg...@gmail.com; a...@canonical.com; de...@linuxdriver

Re: [PATCH] uio-hv-generic: store physical addresses instead of virtual

2016-12-09 Thread Arnd Bergmann
On Friday, December 9, 2016 9:28:44 AM CET Stephen Hemminger wrote: > On Fri, 9 Dec 2016 12:44:40 +0100 > Arnd Bergmann wrote: > > Fixes: 95096f2fbd10 ("uio-hv-generic: new userspace i/o driver for VMBus") > > Signed-off-by: Arnd Bergmann > > Thanks, the code was inherited from outside, and on

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 20:09:49 + Haiyang Zhang wrote: > > -Original Message- > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Friday, December 9, 2016 1:21 PM > > To: Greg KH > > Cc: KY Srinivasan ; o...@aepfle.de; Haiyang Zhang > > ; linux-ker...@vger.kernel.or

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Haiyang Zhang
> -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Friday, December 9, 2016 1:21 PM > To: Greg KH > Cc: KY Srinivasan ; o...@aepfle.de; Haiyang Zhang > ; linux-ker...@vger.kernel.org; > bjorn.helg...@gmail.com; a...@canonical.com; de...@linuxdriver

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 08:31:22 +0100 Greg KH wrote: > On Fri, Dec 09, 2016 at 12:05:53AM +, KY Srinivasan wrote: > > > > > > > -Original Message- > > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > > Sent: Thursday, December 8, 2016 7:56 AM > > > To: KY Srinivasan > > > Cc:

Re: [PATCH] uio-hv-generic: store physical addresses instead of virtual

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 12:44:40 +0100 Arnd Bergmann wrote: > gcc warns about the newly added driver when phys_addr_t is wider than > a pointer: > > drivers/uio/uio_hv_generic.c: In function 'hv_uio_mmap': > drivers/uio/uio_hv_generic.c:71:17: error: cast to pointer from integer of > different size

[PATCH] staging: vc04_services: Fix NULL ptr sparse warnings

2016-12-09 Thread Mike Kofron
In calls to queue_message() in vchiq_core.c, the "void *context" parameter is set as 0 rather than NULL. This patch amends each call to use the proper NULL pointer. The following sparse warnings are fixed: drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c:1623:23: warning: Using plai

RE: [PATCH v3 1/9] staging: fsl-mc: move bus driver out of staging

2016-12-09 Thread Stuart Yoder
> > > Where is the device freed? I see you trying to do some "odd" stuff in > > > fsl_mc_device_remove() by deleting and then putting a device structure. > > > I can't find a "release()" callback anywhere for your bus, where is it? > > > > > > What happens when the reference count falls to 0 for y

Re: [PATCH 0/4] staging-greybus: Fine-tuning for four functions

2016-12-09 Thread Greg Kroah-Hartman
On Fri, Dec 09, 2016 at 03:32:38PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 9 Dec 2016 15:25:35 +0100 > > A few update suggestions were taken into account > from static source code analysis. Again, nope, sorry, not going to even look at these... ___

[PATCH 4/4] staging: greybus: power_supply: Use kcalloc() in gb_power_supplies_setup()

2016-12-09 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 9 Dec 2016 15:04:24 +0100 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kcalloc". This issue was detected by using the Coccinelle software

[PATCH 3/4] staging: greybus: light: Check return value of a kstrndup() call in gb_lights_light_config()

2016-12-09 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 9 Dec 2016 14:36:16 +0100 A return value was not checked after a call of the function "kstrndup". This issue was detected by using the Coccinelle software. Add a bit of exception handling. Fixes: 2870b52bae4c81823ffcb3ed2b0626fb39d64f48 ("greybus: lights: add lig

[PATCH 2/4] staging: greybus: light: Use kcalloc() in two functions

2016-12-09 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 9 Dec 2016 13:46:25 +0100 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "kcalloc". This issue was detected by using the Coccinelle software. *

[PATCH 1/4] staging: greybus: camera: One function call less in gb_camera_configure_streams() after error detection

2016-12-09 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Dec 2016 18:25:13 +0100 The kfree() function was called in one case by the gb_camera_configure_streams() function during error handling even if the passed variable contained a null pointer. This issue was detected by using the Coccinelle software. Adjust a jump

[PATCH 0/4] staging-greybus: Fine-tuning for four functions

2016-12-09 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 9 Dec 2016 15:25:35 +0100 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): One function call less in gb_camera_configure_streams() after error detection light: Use kcalloc() in two functions Check return

[PATCH] Staging: ks7010: ks7010_sdio.h: Fixed coding style error

2016-12-09 Thread Manoj Sawai
Error - Complex macro not in parentheses and trailing whitespace Signed-off-by: Manoj Sawai --- drivers/staging/ks7010/ks7010_sdio.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.h b/drivers/staging/ks7010/ks7010_sdio.h index 0f5fd848

[PATCH v2 2/2] staging: iio: ad7606: move out of staging

2016-12-09 Thread Eva Rachel Retuya
Move the ad7606 driver from staging/iio/adc to iio/adc. Also, update the corresponding Makefile and Kconfig associated with the change. Signed-off-by: Eva Rachel Retuya --- drivers/iio/adc/Kconfig| 34 ++ drivers/iio/adc/Makefile

[PATCH v2 1/2] staging: iio: ad7606: replace range/range_available with corresponding scale

2016-12-09 Thread Eva Rachel Retuya
Eliminate the non-standard attributes in_voltage_range and in_voltage_range_available. Implement in_voltage_scale_available in place of these attributes and update the SCALE accordingly. The array scale_avail is introduced to hold the available scale values. Signed-off-by: Eva Rachel Retuya --- C

[PATCH v2 0/2] staging: iio: ad7606: move driver out of staging

2016-12-09 Thread Eva Rachel Retuya
Address the last remaining TODO [1] for this driver and move it from staging into mainline. [1] https://marc.info/?l=linux-iio&m=147689684332118&w=2 Change in v2: * Address the incorrect way of implementing the scale. (Pointed out by Lars) Eva Rachel Retuya (2): staging: iio: ad7606: replace

[PATCH] uio-hv-generic: store physical addresses instead of virtual

2016-12-09 Thread Arnd Bergmann
gcc warns about the newly added driver when phys_addr_t is wider than a pointer: drivers/uio/uio_hv_generic.c: In function 'hv_uio_mmap': drivers/uio/uio_hv_generic.c:71:17: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] virt_to_phys((void *)info->mem[mi].

Re: [PATCH 10/14] rtlwifi: Add BTC_TRACE_STRING to new btcoex

2016-12-09 Thread Kalle Valo
Dan Carpenter writes: > On Thu, Dec 08, 2016 at 02:50:49PM +0300, Dan Carpenter wrote: >> On Thu, Dec 08, 2016 at 01:43:42PM +0200, Kalle Valo wrote: >> > But it would make me very happy if someone would add a similar grouping >> > functionality to dyndbg to make it easy to enable a set of debug

Re: [PATCH 10/14] rtlwifi: Add BTC_TRACE_STRING to new btcoex

2016-12-09 Thread Kalle Valo
Dan Carpenter writes: > I don't have a problem with the ath debug printks. Larry asked me for > examples of better debug functions and the ath code is an example. > Literally, any existing debug functions are better than the > BTC_TRACE_STRING() stuff. Sure, I agree with that. My point was just