Re: [PATCH V2 0/6] iio: mxs-lradc: Clean up and add datasheet names

2015-07-20 Thread Dan Carpenter
On Mon, Jul 20, 2015 at 12:06:11AM +0200, Marek Vasut wrote: > On Sunday, July 19, 2015 at 10:49:47 PM, Dan Carpenter wrote: > > On Sun, Jul 19, 2015 at 11:33:43PM +0300, Dan Carpenter wrote: > > > I didn't necessarily read the thread that way, but the thread was > > > long... > > > > I read the t

Re: Generic VME UIO driver

2015-07-20 Thread Martyn Welch
On 08/07/15 16:02, Dmitry Kalinkin wrote: On 08 Jul 2015, at 16:22, Martyn Welch wrote: On 06/07/15 18:24, Dmitry Kalinkin wrote: Some functionality was dropped as it was not good practice (such as receiving VME interrupts in user space, it's not really doable if the slave card is Release

Re: [PATCH V2 0/6] iio: mxs-lradc: Clean up and add datasheet names

2015-07-20 Thread Marek Vasut
On Monday, July 20, 2015 at 09:00:32 AM, Dan Carpenter wrote: > On Mon, Jul 20, 2015 at 12:06:11AM +0200, Marek Vasut wrote: > > On Sunday, July 19, 2015 at 10:49:47 PM, Dan Carpenter wrote: > > > On Sun, Jul 19, 2015 at 11:33:43PM +0300, Dan Carpenter wrote: > > > > I didn't necessarily read the t

RE: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed during probe

2015-07-20 Thread Dexuan Cui
> From: KY Srinivasan > Sent: Friday, July 17, 2015 23:33 > > From: Dexuan Cui > > Sent: Friday, July 17, 2015 3:01 AM > > > From: K. Y. Srinivasan > > > Sent: Friday, July 17, 2015 3:17 > > > Subject: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be > > processed > > > during probe > >

[PATCH] staging: comedi: drivers: pcl816.c remove leading space

2015-07-20 Thread Chandra S Gorentla
Checkpatch.pl warning - suspect code indent for conditional statements - is corrected Signed-off-by: Chandra S Gorentla --- drivers/staging/comedi/drivers/pcl816.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/pcl816.c b/drivers/staging/comed

RE: [PATCH 3/5] Drivers: hv_vmbus: Fix signal to host condition

2015-07-20 Thread Dexuan Cui
> -Original Message- > From: deve On Behalf of K. Y. Srinivasan > Sent: Monday, July 20, 2015 11:37 > > From: Christopher Oo > > Fixes a bug where previously hv_ringbuffer_read would pass in the old > number of bytes available to read instead of the expected old read index > when calculat

RE: [PATCH 5/5] Drivers: hv: vmbus: Implement a clocksource based on the TSC page

2015-07-20 Thread Dexuan Cui
> From: devel [mailto:driverdev-devel-bounces...linuxdriverproject.org] On > Behalf > Of K. Y. Srinivasan > Sent: Monday, July 20, 2015 11:37 > > The current Hyper-V clock source is based on the per-partition reference > counter > and this counter is being accessed via s synthetic MSR - > HV_X64

[PATCH 1/2] staging: lustre: obdclass: Make variables static.

2015-07-20 Thread Cihangir Akturk
obd_device_list_sops and obd_device_list_fops are not referenced outside of linux-module.c, so make them static. Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

[PATCH 2/2] staging: lustre: obdclass: Add const qualifier to structure declerations.

2015-07-20 Thread Cihangir Akturk
This patch fixes the following sparse warnings: WARNING: struct seq_operations should normally be const WARNING: struct file_operations should normally be const Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2 insertions(+

[PATCH 0/2] staging: lustre: obdclass: Miscellaneous fixes

2015-07-20 Thread Cihangir Akturk
This patchset contains sparse and checkpatch.pl fixes. Cihangir Akturk (2): staging: lustre: obdclass: Make variables static. staging: lustre: obdclass: Add const qualifier to structure declerations. drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2

Re: [PATCH v3 01/46] usb: gadget: encapsulate endpoint claiming mechanism

2015-07-20 Thread Krzysztof Opasiak
On 07/15/2015 08:31 AM, Robert Baldyga wrote: So far it was necessary for usb functions to set ep->driver_data in endpoint obtained from autoconfig to non-null value, to indicate that endpoint is claimed by function (in autoconfig it was checked if endpoint has set this field to non-null value,

[PATCH 0/3] Drivers: hv: vmbus: Add sysfs attributes to show CPU binding of channels

2015-07-20 Thread K. Y. Srinivasan
Add sysfs attributes to track the binding of CPUs to channels. This patchset also adds a script to list VMBUS devices. Dexuan Cui (3): Drivers: hv: vmbus: add a sysfs attr to show the binding of channel/VP tools: hv: add a python script lsvmbus to list VMBus devices Drivers: hv: vmbus:

[PATCH 1/3] Drivers: hv: vmbus: add a sysfs attr to show the binding of channel/VP

2015-07-20 Thread K. Y. Srinivasan
From: Dexuan Cui This is useful to analyze performance issue. Signed-off-by: Dexuan Cui Signed-off-by: K. Y. Srinivasan --- drivers/hv/vmbus_drv.c | 38 ++ 1 files changed, 38 insertions(+), 0 deletions(-) diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv

[PATCH 3/3] Drivers: hv: vmbus: document the VMBus sysfs files

2015-07-20 Thread K. Y. Srinivasan
From: Dexuan Cui The 4 sysfs files should be stable ABIs to the user space. Signed-off-by: Dexuan Cui Signed-off-by: K. Y. Srinivasan --- Documentation/ABI/stable/sysfs-bus-vmbus | 29 + MAINTAINERS |1 + 2 files changed, 30 inser

Re: [PATCH v3 26/46] usb: gadget: pch_ud: add ep capabilities support

2015-07-20 Thread Krzysztof Opasiak
On 07/15/2015 08:32 AM, Robert Baldyga wrote: Convert endpoint configuration to new capabilities model. Signed-off-by: Robert Baldyga --- drivers/usb/gadget/udc/pch_udc.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/drivers/usb/gadget/udc/pch_udc.c b/

[PATCH 2/3] tools: hv: add a python script lsvmbus to list VMBus devices

2015-07-20 Thread K. Y. Srinivasan
From: Dexuan Cui By default lsvmbus lists all the devices in the VMBus. With -v or -vv, more information is printed, including the VMBus Rel_ID, class ID, device ID and which channel is bound to which virtual processor, etc. Signed-off-by: Dexuan Cui Signed-off-by: K. Y. Srinivasan --- tools/

[PATCH v2] staging: media: lirc: fix various checkpatch warnings

2015-07-20 Thread Maciek Borzecki
Remove unnecessary braces where appropriate. Fix non-tab indentation. This removes the following checkpatch warnings: WARNING: braces {} are not necessary for single statement blocks ERROR: code indent should use tabs where possible Signed-off-by: Maciek Borzecki --- drivers/staging/media/lirc

[PATCH v6 1/2] staging: rtl8192u: remove bool comparisons

2015-07-20 Thread Luis de Bethencourt
Remove explicit true/false comparisons to bool variables. Signed-off-by: Luis de Bethencourt --- drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c | 4 ++-- drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +- drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c | 4 ++-- dri

[PATCH v6 2/2] staging: rtl8192u: remove unneeded bool

2015-07-20 Thread Luis de Bethencourt
bool Reval is set to match the value of bHalfWirelessN24GMode just to this. The value can be returned directly. Removing uneeded bool. Signed-off-by: Luis de Bethencourt Suggested-by: Joe Perches Suggested-by: Franks Klaver --- drivers/staging/rtl8192u/r8192U_core.c | 9 + 1 file chang

RE: staging drivers

2015-07-20 Thread Weiny, Ira
> On 07/17/2015 02:14 AM, Greg Kroah-Hartman (gre...@linuxfoundation.org) > wrote: > > > > It's up to the IB maintainer if they are willing to let it be in > > stable as-is. > > I wouldn't call it stable as-is. However, that doesn't mean I think it > *must* go to staging. It's a first cut at a t

Re: [PATCH] staging: ion: ion_cma_heap: Don't directly use dma_common_get_sgtable

2015-07-20 Thread Robin Murphy
Hi Laura, On 17/07/15 17:50, Laura Abbott wrote: On 07/17/2015 08:21 AM, Robin Murphy wrote: Hi Tixy, On 17/07/15 12:01, Jon Medhurst (Tixy) wrote: Use dma_get_sgtable rather than dma_common_get_sgtable so a device's dma_ops aren't bypassed. This is essential in situations where a device uses

RE: [PATCH 5/5] Drivers: hv: vmbus: Implement a clocksource based on the TSC page

2015-07-20 Thread KY Srinivasan
> -Original Message- > From: Dexuan Cui > Sent: Monday, July 20, 2015 5:14 AM > To: KY Srinivasan; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; vkuzn...@redhat.com; jasow...@redhat.com > Subject: RE: [PATC

Re: [PATCH v6 1/2] staging: rtl8192u: remove bool comparisons

2015-07-20 Thread Dan Carpenter
On Mon, Jul 20, 2015 at 06:35:42PM +0200, Luis de Bethencourt wrote: > Remove explicit true/false comparisons to bool variables. > > Signed-off-by: Luis de Bethencourt > --- Put a note here under the cut off why you are redoing this patch. regards, dan carpenter ___

RE: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed during probe

2015-07-20 Thread KY Srinivasan
> -Original Message- > From: Dexuan Cui > Sent: Monday, July 20, 2015 2:39 AM > To: KY Srinivasan; da...@davemloft.net; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow...@redhat.com; vkuzn...@redhat.com > S

Re: [PATCH v6 1/2] staging: rtl8192u: remove bool comparisons

2015-07-20 Thread Luis de Bethencourt
On Mon, Jul 20, 2015 at 09:54:56PM +0300, Dan Carpenter wrote: > On Mon, Jul 20, 2015 at 06:35:42PM +0200, Luis de Bethencourt wrote: > > Remove explicit true/false comparisons to bool variables. > > > > Signed-off-by: Luis de Bethencourt > > --- > > Put a note here under the cut off why you are

Re: [PATCH] staging: fix to match 80-char rule

2015-07-20 Thread Mario Bambagini
On Sun, Jul 19, 2015 at 12:28:01PM -0700, Joe Perches wrote: > On Sun, 2015-07-19 at 20:41 +0200, Mario Bambagini wrote: > > Fixed comment line to match the 80-char rules. > > The line contained 81 characters. > > The file is fully compliant with the code style, now. > > > > Signed-off-by: Mario B

[PATCH v3 2/4] staging/nvec: reimplement on top of tegra i2c driver

2015-07-20 Thread Andrey Danin
Remove i2c controller related code and use tegra i2c driver in slave mode. Update nvec documentation. Signed-off-by: Andrey Danin --- Changes for v3: - resolve conflict: 'nvec != NULL' changed to '!nvec' Changes for v2: - remove extra new line - keep old functions to simplify review - move nvec_

[PATCH v3 0/4] arm: tegra: implement NVEC driver using tegra i2c.

2015-07-20 Thread Andrey Danin
This version (v3) is for pushing tegra i2c driver to i2c tree. NVEC driver will be reworked later to use i2c core slave framework. NVEC driver contains code to manage tegra i2c controller in slave mode. I2C slave support was implemented in linux kernel. The goal of this patch serie is to implement

[PATCH v3 3/4] staging/nvec: remove old code

2015-07-20 Thread Andrey Danin
Signed-off-by: Andrey Danin --- No changes for v3 No changes for v2 Signed-off-by: Andrey Danin --- drivers/staging/nvec/nvec.c | 211 1 file changed, 211 deletions(-) diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c index 7da

[PATCH v3 4/4] dt: paz00: define nvec as child of i2c bus

2015-07-20 Thread Andrey Danin
NVEC driver was reimplemented to use tegra i2c. Use common i2c bindings for NVEC node. Signed-off-by: Andrey Danin --- No changes for v3: Changes for v2: - swap reg and request-gpios properties - use nvec-slave instead of nvec to keep ABI compatibility - place doc in separate patch Signed-off-b

[PATCH v3 1/4] i2c: tegra: implement slave mode

2015-07-20 Thread Andrey Danin
Initialization code is based on NVEC driver. There is a HW bug in AP20 that was also mentioned in kernel sources for Toshiba AC100. Signed-off-by: Andrey Danin --- Changes for v3: - handle 10-bit clients properly Changes for v2: - remove hack from tegra_i2c_clock_disable - replace slave status

[PATCH V2 0/5] Drivers: hv: vmbus: Miscellaneous improvements and fixes

2015-07-20 Thread K. Y. Srinivasan
In addition to a bug fix and some improvements to the way we distribute channel load amongst available CPUs, this patch set also includes an implementation of a clocksource based on the TSC page that Hyper-V supports. In this version, I have addressed comments from Dexuan Cui. Christopher Oo (1):

[PATCH V2 5/5] Drivers: hv: vmbus: Implement a clocksource based on the TSC page

2015-07-20 Thread K. Y. Srinivasan
The current Hyper-V clock source is based on the per-partition reference counter and this counter is being accessed via s synthetic MSR - HV_X64_MSR_TIME_REF_COUNT. Hyper-V has a more efficient way of computing the per-partition reference counter value that does not involve reading a synthetic MSR

[PATCH V2 2/5] Drivers: hv: vmbus: Further improve CPU affiliation logic

2015-07-20 Thread K. Y. Srinivasan
From: Dexuan Cui Keep track of CPU affiliations of sub-channels within the scope of the primary channel. This will allow us to better distribute the load amongst available CPUs. Signed-off-by: Dexuan Cui Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c | 20 ++--

[PATCH V2 3/5] Drivers: hv_vmbus: Fix signal to host condition

2015-07-20 Thread K. Y. Srinivasan
From: Christopher Oo Fixes a bug where previously hv_ringbuffer_read would pass in the old number of bytes available to read instead of the expected old read index when calculating when to signal to the host that the ringbuffer is empty. Since the previous write size is already saved, also change

[PATCH V2 4/5] drivers/hv: Migrate to new 'set-state' interface

2015-07-20 Thread K. Y. Srinivasan
From: Viresh Kumar Migrate hv driver to the new 'set-state' interface provided by clockevents core, the earlier 'set-mode' interface is marked obsolete now. This also enables us to implement callbacks for new states of clockevent devices, for example: ONESHOT_STOPPED. Cc: "K. Y. Srinivasan" Cc

[PATCH V2 1/5] Drivers: hv: vmbus: Improve the CPU affiliation for channels

2015-07-20 Thread K. Y. Srinivasan
The current code tracks the assigned CPUs within a NUMA node in the context of the primary channel. So, if we have a VM with a single NUMA node with 8 VCPUs, we may end up unevenly distributing the channel load. Fix the issue by tracking affiliations globally. Signed-off-by: K. Y. Srinivasan --

Re: [PATCH v3 2/4] staging/nvec: reimplement on top of tegra i2c driver

2015-07-20 Thread Stephen Warren
On 07/20/2015 02:35 PM, Andrey Danin wrote: Remove i2c controller related code and use tegra i2c driver in slave mode. Update nvec documentation. diff --git a/Documentation/devicetree/bindings/nvec/nvidia,nvec.txt b/Documentation/devicetree/bindings/nvec/nvidia,nvec.txt I would expect this

Re: [PATCH v3 4/4] dt: paz00: define nvec as child of i2c bus

2015-07-20 Thread Stephen Warren
On 07/20/2015 02:35 PM, Andrey Danin wrote: NVEC driver was reimplemented to use tegra i2c. Use common i2c bindings for NVEC node. diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts + nvec: nvec@45 { + compatible = "nv

Re: [PATCH v6 1/2] staging: rtl8192u: remove bool comparisons

2015-07-20 Thread Frans Klaver
On 20 July 2015 21:42:39 CEST, Luis de Bethencourt wrote: >On Mon, Jul 20, 2015 at 09:54:56PM +0300, Dan Carpenter wrote: >> On Mon, Jul 20, 2015 at 06:35:42PM +0200, Luis de Bethencourt wrote: >> > Remove explicit true/false comparisons to bool variables. >> > >> > Signed-off-by: Luis de Bethen

Re: [PATCH V3 1/2] staging: rtl8188eu: core: Fix space style errors

2015-07-20 Thread Sudip Mukherjee
On Mon, Jul 20, 2015 at 08:55:43AM +0200, Guillaume Bienkowski wrote: > Signed-off-by: Guillaume Bienkowski > --- you have not given anything in the commit message. Please mention which type of space style issue you have corrected in the commit message. Same for your next 2/2 patch also. There is

Re: [PATCH 1/2] staging: lustre: obdclass: Make variables static.

2015-07-20 Thread Sudip Mukherjee
On Mon, Jul 20, 2015 at 02:15:35PM +0300, Cihangir Akturk wrote: > obd_device_list_sops and obd_device_list_fops are not > referenced outside of linux-module.c, so make them static. And they are not modified also, so const ? regards sudip ___ devel maili

Re: [PATCH 1/2] staging: lustre: obdclass: Make variables static.

2015-07-20 Thread Sudip Mukherjee
On Tue, Jul 21, 2015 at 11:39:49AM +0530, Sudip Mukherjee wrote: > On Mon, Jul 20, 2015 at 02:15:35PM +0300, Cihangir Akturk wrote: > > obd_device_list_sops and obd_device_list_fops are not > > referenced outside of linux-module.c, so make them static. > And they are not modified also, so const ? Y

Re: [PATCH v2] staging: media: lirc: fix various checkpatch warnings

2015-07-20 Thread Sudip Mukherjee
On Mon, Jul 20, 2015 at 06:10:04PM +0200, Maciek Borzecki wrote: > Remove unnecessary braces where appropriate. Fix non-tab > indentation. > > This removes the following checkpatch warnings: > WARNING: braces {} are not necessary for single statement blocks > > ERROR: code indent should use tabs

Re: [PATCH v2] staging: media: lirc: fix various checkpatch warnings

2015-07-20 Thread Maciek Borzecki
On Tue, Jul 21, 2015 at 8:19 AM, Sudip Mukherjee wrote: > On Mon, Jul 20, 2015 at 06:10:04PM +0200, Maciek Borzecki wrote: >> Remove unnecessary braces where appropriate. Fix non-tab >> indentation. >> >> This removes the following checkpatch warnings: >> WARNING: braces {} are not necessary for s

Re: [PATCH v3 4/4] dt: paz00: define nvec as child of i2c bus

2015-07-20 Thread Andrey Danin
On 21.07.2015 1:19, Stephen Warren wrote: On 07/20/2015 02:35 PM, Andrey Danin wrote: NVEC driver was reimplemented to use tegra i2c. Use common i2c bindings for NVEC node. diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts +nvec: nvec@45 { +

[PATCH] sm750fb: coding style fixes in _accel.h,__help.h

2015-07-20 Thread Vinay Simha BN
WARNING: line over 80 characters comment block , odd statement fixes Signed-off-by: Vinay Simha BN --- drivers/staging/sm750fb/sm750_accel.h | 30 -- drivers/staging/sm750fb/sm750_help.h | 23 +-- 2 files changed, 21 insertions(+), 32 deletions(-)