[PATCH] staging: ks7010: ks7010_sdio.c: Fixing multiple assignments

2016-10-21 Thread Nick Rosbrook
Running checkpatch on ks7010_sdio.c shows two locations where multiple assignment statements are used. This patch modifies the assignments into single assignments. Signed-off-by: Nick Rosbrook --- drivers/staging/ks7010/ks7010_sdio.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-)

Re: [PATCH] vme: Convert documentation to reStructuredText, move under driver APIs

2016-10-21 Thread Jonathan Corbet
On Fri, 21 Oct 2016 22:15:27 +0100 Martyn Welch wrote: > Perform a relatively simple conversion of vme_api.txt to reStructuredText > and move under driver-api, which seems the most logical place for this > documentation. Makes sense to me, applied to the docs tree. Now if only the VME stuff had

[PATCH] vme: Convert documentation to reStructuredText, move under driver APIs

2016-10-21 Thread Martyn Welch
From: Martyn Welch Perform a relatively simple conversion of vme_api.txt to reStructuredText and move under driver-api, which seems the most logical place for this documentation. Signed-off-by: Martyn Welch --- Documentation/driver-api/index.rst| 1 + Documentation/{vme_api.t

Re: [PATCH v2 3/3] doc: add documentation for uio-hv-generic

2016-10-21 Thread Jonathan Corbet
On Mon, 17 Oct 2016 12:33:19 -0700 Stephen Hemminger wrote: > From: Stephen Hemminger > > Update UIO documentation to include basic information about > uio_hv_generic. I've applied this to the docs tree, thanks. jon ___ devel mailing list de...@linu

[PATCH] vme: vme_get_size potentially returning incorrect value on failure

2016-10-21 Thread Martyn Welch
The function vme_get_size returns the size of the window to the caller, however it doesn't check the return value of the call to vme_master_get. Return 0 on failure rather than anything else. Suggested-by: Dan Carpenter Signed-off-by: Martyn Welch --- drivers/vme/vme.c | 4 1 file changed

[PATCH] Staging:vt6656:main_usb: fix Block comments should align the * on each line

2016-10-21 Thread Nadim Almas
Block comments should align the * on each line as reported by checkpatch.pl Signed-off-by: Nadim Almas --- drivers/staging/vt6656/main_usb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/vt6656/main_usb.c b/drivers/staging/vt6656/main_usb.c index fe84

Re: [bug report] vme: Adding Fake VME driver

2016-10-21 Thread Martyn Welch
On Fri, Sep 30, 2016 at 02:37:01PM +0300, Dan Carpenter wrote: > Hello Martyn Welch, > > The patch 658bcdae9c67: "vme: Adding Fake VME driver" from Jul 7, > 2016, leads to the following static checker warning: > > drivers/vme/bridges/vme_fake.c:338 fake_master_set() > warn: 'spin_lock

[PATCH]: Staging: Greybus: Fix trailing */ in block comments

2016-10-21 Thread Rahul Krishnan
This patch fixes the following checkpath.pl warning WARNING: Block comments use a trailing */ on a separate line Signed-off-by: Rahul Krishnan --- drivers/staging/greybus/log.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/log.c b/drivers/stag

Re: [PATCH net-next] hv_netvsc: fix a race between netvsc_send() and netvsc_init_buf()

2016-10-21 Thread David Miller
From: Vitaly Kuznetsov Date: Wed, 19 Oct 2016 15:53:01 +0200 > Fix in commit 880988348270 ("hv_netvsc: set nvdev link after populating > chn_table") turns out to be incomplete. A crash in > netvsc_get_next_send_section() is observed on mtu change when the device > is under load. The race I identi

Re: [PATCH net-next] hv_netvsc: fix a race between netvsc_send() and netvsc_init_buf()

2016-10-21 Thread David Miller
From: Vitaly Kuznetsov Date: Fri, 21 Oct 2016 17:17:18 +0200 > David Miller writes: > >> From: Vitaly Kuznetsov >> Date: Fri, 21 Oct 2016 13:15:53 +0200 >> >>> David Miller writes: >>> From: Vitaly Kuznetsov Date: Thu, 20 Oct 2016 10:51:04 +0200 > Stephen Hemminger write

Re: [PATCH net-next] hv_netvsc: fix a race between netvsc_send() and netvsc_init_buf()

2016-10-21 Thread Vitaly Kuznetsov
David Miller writes: > From: Vitaly Kuznetsov > Date: Fri, 21 Oct 2016 13:15:53 +0200 > >> David Miller writes: >> >>> From: Vitaly Kuznetsov >>> Date: Thu, 20 Oct 2016 10:51:04 +0200 >>> Stephen Hemminger writes: > Do we need ACCESS_ONCE() here to avoid check/use issues?

Re: [PATCH net-next] hv_netvsc: fix a race between netvsc_send() and netvsc_init_buf()

2016-10-21 Thread David Miller
From: Vitaly Kuznetsov Date: Fri, 21 Oct 2016 13:15:53 +0200 > David Miller writes: > >> From: Vitaly Kuznetsov >> Date: Thu, 20 Oct 2016 10:51:04 +0200 >> >>> Stephen Hemminger writes: >>> Do we need ACCESS_ONCE() here to avoid check/use issues? >>> >>> I think we don't: this is

[PATCH 3/9] bus: fsl-mc: dpio: add APIs for DPIO objects

2016-10-21 Thread Stuart Yoder
From: Ioana Radulescu Add the command build/parse APIs for operating on DPIO objects through the DPAA2 Management Complex. Signed-off-by: Ioana Radulescu Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- drivers/bus/fsl-mc/Kconfig | 10 ++ drivers/bus/fsl-mc/Makefile

[PATCH 0/9] staging: fsl-mc: move bus driver out of staging, add dpio

2016-10-21 Thread Stuart Yoder
This patch series: A) addresses the final item in the staging TODO list for the fsl-mc bus driver-- adding a functional driver on top of the bus driver, and B) requests that the fsl-mc bus driver be moved out of staging. The proposed destination for the bus driver is drivers/bus. Proposed location

[PATCH 6/9] bus: fsl-mc: dpio: add QBMan portal APIs for DPAA2

2016-10-21 Thread Stuart Yoder
From: Roy Pledge Add QBman APIs for frame queue and buffer pool operations. Signed-off-by: Roy Pledge Signed-off-by: Haiying Wang Signed-off-by: Stuart Yoder --- drivers/bus/fsl-mc/dpio/Makefile |2 +- drivers/bus/fsl-mc/dpio/qbman-portal.c | 1009

[PATCH 7/9] bus: fsl-mc: dpio: add the DPAA2 DPIO service interface

2016-10-21 Thread Stuart Yoder
From: Roy Pledge The DPIO service interface handles initialization of DPIO objects and exports APIs to be used by other DPAA2 object drivers to perform queuing and buffer management related operations. The service allows registration of callbacks when frames or notifications are received. Signe

[PATCH 2/9] bus: fsl-mc: dpio: add DPIO driver overview document

2016-10-21 Thread Stuart Yoder
add document describing the dpio driver and it's role, components and major interfaces Signed-off-by: Stuart Yoder --- Documentation/dpaa2/dpio-driver.txt | 135 1 file changed, 135 insertions(+) create mode 100644 Documentation/dpaa2/dpio-driver.txt diff -

[PATCH 9/9] bus: fsl-mc: dpio: add maintainer for DPIO

2016-10-21 Thread Stuart Yoder
From: Roy Pledge add Roy Pledge as maintainer of DPIO Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 797a08a..1aa991e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5097,6 +5097,12

[PATCH 8/9] bus: fsl-mc: dpio: add the DPAA2 DPIO object driver

2016-10-21 Thread Stuart Yoder
From: Roy Pledge The DPIO driver registers with the fsl-mc bus to handle bus-related events for DPIO objects. Key responsibility is mapping I/O regions, setting up interrupt handlers, and calling the DPIO service initialization during probe. Signed-off-by: Roy Pledge Signed-off-by: Haiying Wan

[PATCH 4/9] bus: fsl-mc: dpio: add frame descriptor and scatter/gather APIs

2016-10-21 Thread Stuart Yoder
From: Roy Pledge Add global definitions for DPAA2 frame descriptors and scatter gather entries. Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- include/linux/fsl/dpaa2-fd.h | 415 +++ 1 file changed, 415 insertions(+) create mode 100644 inclu

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

2016-10-21 Thread Stuart Yoder
Move the source files out of staging into their final locations: -include files in drivers/staging/fsl-mc/include go to include/linux/fsl -irq-gic-v3-its-fsl-mc-msi.c goes to drivers/irqchip -source in drivers/staging/fsl-mc/bus goes to drivers/bus/fsl-mc -README.txt, providing and overview

[PATCH 5/9] bus: fsl-mc: dpio: add global dpaa2 definitions

2016-10-21 Thread Stuart Yoder
From: Roy Pledge Create header for global dpaa2 definitions. Add definitions for dequeue results. Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- include/linux/fsl/dpaa2-global.h | 203 +++ 1 file changed, 203 insertions(+) create mode 100644 in

[PATCH] Staging:wilc1000:host_interface: fixed 80 character line limit coding style issue

2016-10-21 Thread Nadim Almas
Fixed coding style issue Signed-off-by: Nadim Almas --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 0c5dea7..6e6540f 100644 --- a/d

[PATCH] Staging:wlan-ng:hfa384x: fixed 80 character line limit coding style issue

2016-10-21 Thread Nadim Almas
Fixed coding style issue Signed-off-by: Nadim Almas --- drivers/staging/wlan-ng/hfa384x.h | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wlan-ng/hfa384x.h b/drivers/staging/wlan-ng/hfa384x.h index 6337b1d..cca2652 100644 --- a/drivers/staging/wlan-ng

Re: [PATCH net-next] hv_netvsc: fix a race between netvsc_send() and netvsc_init_buf()

2016-10-21 Thread Vitaly Kuznetsov
David Miller writes: > From: Vitaly Kuznetsov > Date: Thu, 20 Oct 2016 10:51:04 +0200 > >> Stephen Hemminger writes: >> >>> Do we need ACCESS_ONCE() here to avoid check/use issues? >>> >> >> I think we don't: this is the only place in the function where we read >> the variable so we'll get no

Re: [PATCH 4/6] gpio: cbc-presence: Add CBC presence detect as GPIO driver

2016-10-21 Thread Linus Walleij
On Fri, Oct 7, 2016 at 5:20 PM, Pantelis Antoniou wrote: > From: Georgi Vlaev > > This driver exports the CB FPGA presence detect bits from a > single 32bit CB register as GPIOs. > > Signed-off-by: Georgi Vlaev > Signed-off-by: Guenter Roeck > [Ported from Juniper kernel] > Signed-off-by: Pant

Re: [PATCH 3/6] gpio: gpio-cbc: Document bindings of CBC FPGA GPIO block

2016-10-21 Thread Linus Walleij
On Fri, Oct 7, 2016 at 5:20 PM, Pantelis Antoniou wrote: > From: Georgi Vlaev > > Add device tree bindings document for the GPIO driver of > Juniper's CBC FPGA. > > Signed-off-by: Georgi Vlaev > [Ported from Juniper kernel] > Signed-off-by: Pantelis Antoniou > --- > .../devicetree/bindings/gp

Re: [PATCH 2/6] gpio: Add support for PTX1K CBC FPGA spare GPIOs

2016-10-21 Thread Linus Walleij
On Fri, Oct 7, 2016 at 5:20 PM, Pantelis Antoniou wrote: > From: Georgi Vlaev > > Add support for the GPIO block in Juniper's CBC FPGA. > > A number of GPIOs exported by different kind of boards > is supported. > > Signed-off-by: Georgi Vlaev > Signed-off-by: Guenter Roeck > [Ported from Junip

Re: [PATCH] Staging: lustre: lov: check return value of lov_sub_get()

2016-10-21 Thread Greg Kroah-Hartman
On Fri, Oct 21, 2016 at 07:33:52AM +0800, Jinshan Xiong wrote: > Check return value of lov_sub_get() in lov_io_read_ahead(). > > Signed-off-by: Jinshan Xiong No reported-by: tag? ___ devel mailing list de...@linuxdriverproject.org http://driverdev.lin