[PATCH v2] staging: iio: ad7606: use iio_device_{claim|release}_direct_mode()

Replace the code that guarantees the device stays in direct mode with iio_device_{claim|release}_direct_mode() which does same. Signed-off-by: Alison Schofield --- Changed in v2: - removed improper application of claim/release from intr handler - updated changelog

RE: [PATCH] staging: comedi: das1800: remove unused variable

On Tuesday, April 05, 2016 7:23 AM, Sudip Mukherjee wrote: > The variable unipolar was never used. > > Signed-off-by: Sudip Mukherjee > --- > > There may be a chance that reading from DAS1800_CONTROL_C is necessary > before reading from DAS1800_STATUS. If that is

[PATCH v5 6/6] drivers:hv: Separate out frame buffer logic when picking MMIO range

Simplify the logic that picks MMIO ranges by pulling out the logic related to trying to lay frame buffer claim on top of where the firmware placed the frame buffer. Signed-off-by: Jake Oshins --- drivers/hv/vmbus_drv.c | 80 ++

[PATCH v5 2/6] drivers:hv: Call vmbus_mmio_free() to reverse vmbus_mmio_allocate()

Existing code just called release_mem_region(). Adding a wrapper around it allows the more complex range tracking that is introduced later in this patch series. Signed-off-by: Jake Oshins --- drivers/hv/vmbus_drv.c | 15 +++

[PATCH v5 5/6] drivers:hv: Record MMIO range in use by frame buffer

Later in the boot sequence, we need to figure out which memory ranges can be given out to various paravirtual drivers. The hyperv_fb driver should, ideally, be placed right on top of the frame buffer, without some other device getting plopped on top of this range in the meantime. Recording this

[PATCH v5 4/6] drivers:hv: Track allocations of children of hv_vmbus in private resource tree

This patch changes vmbus_allocate_mmio() and vmbus_free_mmio() so that when child paravirtual devices allocate memory-mapped I/O space, they allocate it privately from a resource tree pointed at by hyperv_mmio and also by the public resource tree iomem_resource. This allows the region to be

[PATCH v5 1/6] drivers:hv: Lock access to hyperv_mmio resource tree

In existing code, this tree of resources is created in single-threaded code and never modified after it is created, and thus needs no locking. This patch introduces a semaphore for tree access, as other patches in this series introduce run-time modifications of this resource tree which can happen

[PATCH v5 0/6] drivers:hv: Ensure that bridge windows don't overlap

This version incorporates feedback from Bjorn Helgaas, folding two patches together. Hyper-V VMs expose paravirtual drivers through a mechanism called VMBus, which is managed by hv_vmbus.ko. For each parvirtual service instance, this driver exposes a new child device. Some of these child

[PATCH 2/8] Drivers: hv: vmbus: Introduce functions for estimating room in the ring buffer

Introduce separate functions for estimating how much can be read from and written to the ring buffer. Signed-off-by: K. Y. Srinivasan --- drivers/hv/ring_buffer.c | 25 - include/linux/hyperv.h | 27 +++ 2 files changed,

[PATCH 1/8] Drivers: hv: kvp: fix IP Failover

From: Vitaly Kuznetsov Hyper-V VMs can be replicated to another hosts and there is a feature to set different IP for replicas, it is called 'Failover TCP/IP'. When such guest starts Hyper-V host sends it KVP_OP_SET_IP_INFO message as soon as we finish negotiation procedure.

[PATCH 8/8] Drivers: hv: vmbus: handle various crash scenarios

From: Vitaly Kuznetsov <[mailto:vkuzn...@redhat.com]> Kdump keeps biting. Turns out CHANNELMSG_UNLOAD_RESPONSE is always delivered to the CPU which was used for initial contact or to CPU0 depending on host version. vmbus_wait_for_unload() doesn't account for the fact that in case we're crashing

[PATCH 4/8] Drivers: hv: vmbus: Use the new virt_xx barrier code

Use the virt_xx barriers that have been defined for use in virtual machines. Signed-off-by: K. Y. Srinivasan --- drivers/hv/ring_buffer.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/hv/ring_buffer.c b/drivers/hv/ring_buffer.c

[PATCH 7/8] Drivers: hv: vmbus: Implement APIs to support "in place" consumption of vmbus packets

Implement APIs for in-place consumption of vmbus packets. Currently, each packet is copied and processed one at a time and as part of processing each packet we potentially may signal the host (if it is waiting for room to produce a packet). These APIs help batched in-place processing of vmbus

[PATCH 5/8] Drivers: hv: vmbus: Export the vmbus_set_event() API

In preparation for moving some ring buffer functionality out of the vmbus driver, export the API for signaling the host. Signed-off-by: K. Y. Srinivasan --- drivers/hv/connection.c |1 + drivers/hv/hyperv_vmbus.h |2 -- include/linux/hyperv.h|1 + 3 files

[PATCH 6/8] Drivers: hv: vmbus: Move some ring buffer functions to hyperv.h

In preparation for implementing APIs for in-place consumption of VMBUS packets, movve some ring buffer functionality into hyperv.h Signed-off-by: K. Y. Srinivasan --- drivers/hv/ring_buffer.c | 55 -- include/linux/hyperv.h |

[PATCH 3/8] Drivers: hv: vmbus: Use READ_ONCE() to read variables that are volatile

Use the READ_ONCE macro to access variabes that can change asynchronously. This is the recommended mechanism for dealing with "unsafe" compiler optimizations. Signed-off-by: K. Y. Srinivasan --- drivers/hv/ring_buffer.c |6 +++--- 1 files changed, 3 insertions(+), 3

[PATCH 0/8] Drivers: hv: Miscellaneous vmbus and util driver fixes

Cleanup the ringbuffer code and implement APIs for "in place" consumption. This patchset also includes some other miscellaneous fixes. K. Y. Srinivasan (6): Drivers: hv: vmbus: Introduce functions for estimating room in the ring buffer Drivers: hv: vmbus: Use READ_ONCE() to read variables

RE: [PATCH v4 3/7] drivers:hv: Use new vmbus_mmio_free() from client drivers.

> -Original Message- > From: Bjorn Helgaas [mailto:helg...@kernel.org] > Sent: Tuesday, April 5, 2016 11:00 AM > To: Jake Oshins > Cc: linux-...@vger.kernel.org; gre...@linuxfoundation.org; KY Srinivasan > ; linux-ker...@vger.kernel.org; >

RE: [PATCH v4 3/7] drivers:hv: Use new vmbus_mmio_free() from client drivers.

> -Original Message- > From: Bjorn Helgaas [mailto:helg...@kernel.org] > Sent: Tuesday, April 5, 2016 11:00 AM > To: Jake Oshins > Cc: linux-...@vger.kernel.org; gre...@linuxfoundation.org; KY Srinivasan > ; linux-ker...@vger.kernel.org; >

Re: [PATCH v10 3/3] staging/android: refactor SYNC IOCTLs

Hi Greg, Any comments on this? Thanks, Gustavo 2016-03-18 Gustavo Padovan : > From: Gustavo Padovan > > Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid > future API breaks and optimize buffer allocation.

Re: [PATCH v4 3/7] drivers:hv: Use new vmbus_mmio_free() from client drivers.

Hi Jake, On Fri, Apr 01, 2016 at 05:47:43PM -0700, Jake Oshins wrote: > This patch modifies all the callers of vmbus_mmio_allocate() > to call vmbus_mmio_free() instead of release_mem_region(). This changelog merely restates the C code. Presumably there's some important difference between

Re: [PATCH] staging: iio: ad7606: use iio_device_{claim|release}_direct_mode()

On Sun, Apr 03, 2016 at 10:09:13AM +0100, Jonathan Cameron wrote: > On 01/04/16 17:53, Alison Schofield wrote: > > Two instances are moved to the new claim/release API: > > > > In the first instance, the driver was using mlock followed by > > iio_buffer_enabled(). Replace that code with the new

[PATCH 1/2] Drivers: hv: balloon: don't crash when memory is added in non-sorted order

When we iterate through all HA regions in handle_pg_range() we have an assumption that all these regions are sorted in the list and the 'start_pfn >= has->end_pfn' check is enough to find the proper region. Unfortunately it's not the case with WS2016 where host can hot-add regions in a different

[PATCH 0/2] Drivers: hv: balloon: two memory hotplug fixes

Hi, I found an issue (crash) while testing memory hotplug on WS2016TP4 host. It should be easily reproducible with the following test sequence (4.6-rc2): 1) Start a VM with 4 vCPUs in 4 NUMA nodes, 4096Mb of RAM. 2) Enable memory auto onlining with 'echo online >

[PATCH 2/2] Drivers: hv: balloon: reset host_specified_ha_region

We set host_specified_ha_region = true on certain request but this is a global state which stays 'true' forever. We need to reset it when we receive a request where ha_region is not specified. I did not see any real issues, the bug was found by code inspection. Signed-off-by: Vitaly Kuznetsov

[PATCH 1/1] Drivers: hv: vmbus: Fix signaling logic in hv_need_to_signal_on_read()

On the consumer side, we have interrupt driven flow management of the producer. It is sufficient to base the signaling decision on the amount of space that is available to write after the read is complete. The current code samples the previous available space and uses this in making the signaling

[PATCH] staging: rts5208: ensure braces on all arms of if stmt

Added braces on if arm of if statement where else arm already needs braces as suggested for clarity in Documentation/CodingStyle (several) Signed-off-by: Nicholas Sim --- drivers/staging/rts5208/sd.c | 16 1 file changed, 8 insertions(+), 8 deletions(-)

[PATCH] staging: wlan-ng: rewrite NULL comparison

It is not necessary to compare explicitly to NULL. Rewrite if condition as (!dev) or (dev) as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/wlan-ng/hfa384x_usb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

[PATCH] staging: rts5208: ensure braces on all arms of if stmt

Added braces on if arm of if statement where else arm already needs braces as suggested for clarity in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/rts5208/ms.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH] staging: rtl8192u: rewrite NULL comparison for pointers

When testing pointers, it is not necessary to explicitly compare to NULL. Rewrite if condition as (!ptr) or (ptr) as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 6 +++--- 1 file changed, 3

[PATCH] staging: rtl8192u: add blank line after declarations

Add a blank line after function/struct/union/enum declarations for readability, as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 14 ++ 1 file changed, 14 insertions(+) diff --git

[PATCH] staging: rtl8192u: remove blank lines after braces (opening)

Remove unneeded blank lines appearing after opening braces as suggested by checkpatch.pl Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c

Re: Staging: unisys/verisonic: Correct double unlock

On Tue, Apr 05, 2016 at 03:49:57PM +, Sell, Timothy C wrote: > > -Original Message- > > From: Neil Horman [mailto:nhor...@redhat.com] > > Sent: Tuesday, April 05, 2016 10:58 AM > > To: Sell, Timothy C > > Cc: Iban Rodriguez; Kershner, David A; Greg Kroah-Hartman; Benjamin > > Romer;

[PATCH] staging: rtl8192u: remove blank lines before braces (closing)

Remove unneeded blank lines occuring before closing braces Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 35 -- 1 file changed, 35 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c

RE: Staging: unisys/verisonic: Correct double unlock

> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Tuesday, April 05, 2016 10:58 AM > To: Sell, Timothy C > Cc: Iban Rodriguez; Kershner, David A; Greg Kroah-Hartman; Benjamin > Romer; *S-Par-Maintainer; de...@driverdev.osuosl.org; linux- > ker...@vger.kernel.org

[PATCH 4/7] drivers:hv: Reverse order of resources in hyperv_mmio

From: Jake Oshins A patch later in this series allocates child nodes in this resource tree. For that to work, this tree needs to be sorted in ascending order. Signed-off-by: Jake Oshins Signed-off-by: K. Y. Srinivasan ---

[PATCH 3/7] drivers:hv: Use new vmbus_mmio_free() from client drivers.

From: Jake Oshins This patch modifies all the callers of vmbus_mmio_allocate() to call vmbus_mmio_free() instead of release_mem_region(). Signed-off-by: Jake Oshins Signed-off-by: K. Y. Srinivasan --- Greg, please apply

[PATCH 5/7] drivers:hv: Track allocations of children of hv_vmbus in private resource tree

From: Jake Oshins This patch changes vmbus_allocate_mmio() and vmbus_free_mmio() so that when child paravirtual devices allocate memory-mapped I/O space, they allocate it privately from a resource tree pointed at by hyperv_mmio and also by the public resource tree

[PATCH 7/7] drivers:hv: Separate out frame buffer logic when picking MMIO range

From: Jake Oshins Simplify the logic that picks MMIO ranges by pulling out the logic related to trying to lay frame buffer claim on top of where the firmware placed the frame buffer. Signed-off-by: Jake Oshins Signed-off-by: K. Y. Srinivasan

[PATCH 6/7] drivers:hv: Record MMIO range in use by frame buffer

From: Jake Oshins Later in the boot sequence, we need to figure out which memory ranges can be given out to various paravirtual drivers. The hyperv_fb driver should, ideally, be placed right on top of the frame buffer, without some other device getting plopped on top of

[PATCH 2/7] drivers:hv: Make a function to free mmio regions through vmbus

From: Jake Oshins This patch introduces a function that reverses everything done by vmbus_allocate_mmio(). Existing code just called release_mem_region(). Future patches in this series require a more complex sequence of actions, so this function is introduced to wrap those

[PATCH 1/7] drivers:hv: Lock access to hyperv_mmio resource tree

From: Jake Oshins In existing code, this tree of resources is created in single-threaded code and never modified after it is created, and thus needs no locking. This patch introduces a semaphore for tree access, as other patches in this series introduce run-time

[PATCH 0/7] drivers: hv: Ensure that bridge windows don't overlap

Greg, please apply this set to 4.6 tree. Hyper-V VMs expose paravirtual drivers through a mechanism called VMBus, which is managed by hv_vmbus.ko. For each parvirtual service instance, this driver exposes a new child device. Some of these child devices need memory address space, into which

[PATCH] staging: android: make function static

The only user of ion_handle_put() is within the file ion.c, so we can safely make it static. Just to be double sure, checked the function ion_handle_get() which is static. So we can make ion_handle_put() static. Signed-off-by: Sudip Mukherjee ---

Re: Staging: unisys/verisonic: Correct double unlock

On Mon, Apr 04, 2016 at 09:40:13PM +, Sell, Timothy C wrote: > > -Original Message- > > From: Neil Horman [mailto:nhor...@redhat.com] > > Sent: Monday, April 04, 2016 10:35 AM > > To: Sell, Timothy C > > Cc: Iban Rodriguez; Kershner, David A; Greg Kroah-Hartman; Benjamin > > Romer;

Re: [PATCH] Staging: android: timed_gpio: fixed bare use of 'unsigned'

On Tue, Apr 05, 2016 at 07:02:54PM +0530, Shyam Saini wrote: > hi, > Sorry for duplicate work, actually i was following http://kernelnewbies.org/ > FirstKernelPatch . > I updated the repository, then i watched this video  and finally submitted the > duplicate patch. > > As you said, I should work

Re: [PATCH 08/27] staging: lustre: avoid to use bio->bi_vcnt directly

The lloop driver should be removed entirely - use the loop driver instead. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 08/27] staging: lustre: avoid to use bio->bi_vcnt directly

On Tue, Apr 05, 2016 at 07:56:53PM +0800, Ming Lei wrote: > Signed-off-by: Ming Lei A bit more of a commit message is always nice :) Acked-by: Greg Kroah-Hartman ___ devel mailing list

[PATCH 08/27] staging: lustre: avoid to use bio->bi_vcnt directly

Signed-off-by: Ming Lei --- drivers/staging/lustre/lustre/llite/lloop.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/lloop.c b/drivers/staging/lustre/lustre/llite/lloop.c index b725fc1..67323db 100644 ---

[PATCH 00/27] block: cleanup direct access on .bi_vcnt & .bi_io_vec

Hi Guys, It is always not a good practice to access bio->bi_vcnt and bio->bi_io_vec from drivers directly. Also this kind of direct access will cause trouble when converting to multipage bvecs. The 1st patch introduces the following 4 bio helpers which can be used inside drivers for avoiding

[PATCH 2/2] staging: dgnc: remove redundant NULL check in

There were already checking NULL about channel_t / un_t before calling dgnc_maxcps_room(). Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_tty.c | 15 ++- 1 file changed, 2 insertions(+), 13 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c

[PATCH 1/2] staging: dgnc: return -ENOMEM when kzalloc failed

The kzalloc can be failed when memory is not enough to allocate. When kzalloc failed, it need to return error code with ENOMEM Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_tty.c | 8 1 file changed, 8 insertions(+) diff --git

[PATCH 07/14] staging: wilc1000: remove unused hif_drv in wilc_del_station

This patch removes unused hif_drv in wilc_del_station. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 14/14] staging: wilc1000: remove unused hif_drv in wilc_add_beacon

This patch removes unused hif_drv in wilc_add_beacon. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 13/14] staging: wilc1000: remove unused hif_drv in wilc_del_beacon

This patch removes unused hif_drv in wilc_del_beacon. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 12/14] staging: wilc1000: remove unused hif_drv in wilc_set_pmkid_info

This patch removes unused hif_drv in wilc_set_pmkid_info. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 09/14] staging: wilc1000: remove unused hif_drv in wilc_remain_on_channel

This patch removes unused hif_drv in wilc_remain_on_channel. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 10/14] staging: wilc1000: remove unused hif_drv in wilc_set_mac_chnl_num

This patch removes unused hif_drv in wilc_set_mac_chnl_num. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 08/14] staging: wilc1000: remove unused hif_drv in wilc_add_station

This patch removes unused hif_drv in wilc_add_station. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 11/14] staging: wilc1000: remove unused hif_drv in host_int_get_assoc_res_info

This patch removes unused hif_drv in host_int_get_assoc_res_info. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 06/14] staging: wilc1000: remove unused hif_drv in wilc_del_allstation

This patch removes unused hif_drv in wilc_del_allstation. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 05/14] staging: wilc1000: remove unused hif_drv in wilc_edit_station

This patch removes unused hif_drv in wilc_edit_station. There is no need to checku null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 03/14] staging: wilc1000: remove unused hif_drv in wilc_setup_multicast_filter

This patch removes unused hif_drv in wilc_setup_multicast_filter. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 01/14] staging: wilc1000: remove unused hif_drv in host_int_get_ipaddress

This patch removes unused hif_drv in host_int_get_ipaddress. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 04/14] staging: wilc1000: remove unused hif_drv in wilc_set_power_mgmt

This patch removes unused hif_drv in wilc_set_power_mgmt. There is no need to check null and print debug log. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git