Re: [PATCH V3 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-24 Thread Sudip Mukherjee
On Thu, Jul 23, 2015 at 01:24:50PM +0300, Dan Carpenter wrote: On Thu, Jul 23, 2015 at 01:10:57PM +0300, Dan Carpenter wrote: In this specific case, writing it as if (ret != 0) caused the bug. If we had written it as if (ret) return ret; then there are no zeroes so wouldn't have been any

Re: [PATCH v9 4/7] staging: fpga manager: add sysfs interface document

2015-07-24 Thread Pavel Machek
Hi! +What:/sys/class/fpga_manager/fpga/state +Date:July 2015 +KernelVersion: 4.2 +Contact: Alan Tull at...@opensource.altera.com +Description: Read fpga manager state as a string. fpga-FPGA. + Valid states may vary by manufacturer;

Re: [PATCH V3 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-24 Thread Dan Carpenter
On Fri, Jul 24, 2015 at 11:57:01AM +0530, Sudip Mukherjee wrote: This is also ok, the function is supposed to return ret or-ed with the relevant flags based on the scan position. It is considered error if 0 is returned (without any flag). Yeah. You're right. I looked through my list again

Re: [PATCH 3/5] staging: rtl8712: remove duplicate struct

2015-07-24 Thread Sudip Mukherjee
On Thu, Jul 23, 2015 at 09:57:50PM -0700, Joshua Clayton wrote: struct ndis_wlan_bssid_ex is a dopelganger of struct wlan_bssid_ex, and is used about a third as often Switch all instances to wlan_bssid_ex This also gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by: Joshua

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

2015-07-24 Thread Wolfram Sang
Hi Andrey, On Mon, Jul 20, 2015 at 11:35:43PM +0300, Andrey Danin wrote: 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 danind...@mail.ru Still doesn't work for me and I think

Re: [PATCH 3/5] staging: rtl8712: remove duplicate struct

2015-07-24 Thread Joshua Clayton
On Friday, July 24, 2015 02:02:56 PM Sudip Mukherjee wrote: On Thu, Jul 23, 2015 at 09:57:50PM -0700, Joshua Clayton wrote: struct ndis_wlan_bssid_ex is a dopelganger of struct wlan_bssid_ex, and is used about a third as often Switch all instances to wlan_bssid_ex This also gets rid

Re: [PATCH v9 4/7] staging: fpga manager: add sysfs interface document

2015-07-24 Thread Pavel Machek
On Fri 2015-07-24 07:39:15, atull wrote: On Fri, 24 Jul 2015, Pavel Machek wrote: Hi Pavel, Thanks for your your feedback in cleaning up these docs. Hi! +What:/sys/class/fpga_manager/fpga/state +Date:July 2015 +KernelVersion: 4.2 +Contact: Alan

Re: [PATCH 3/5] staging: rtl8712: remove duplicate struct

2015-07-24 Thread Dan Carpenter
On Fri, Jul 24, 2015 at 05:39:29AM -0700, Joshua Clayton wrote: On Friday, July 24, 2015 02:02:56 PM Sudip Mukherjee wrote: On Thu, Jul 23, 2015 at 09:57:50PM -0700, Joshua Clayton wrote: struct ndis_wlan_bssid_ex is a dopelganger of struct wlan_bssid_ex, and is used about a third as

Re: [PATCH 5/5] staging: rtl8712: style fix:

2015-07-24 Thread Joshua Clayton
Dan, On Friday, July 24, 2015 01:52:27 PM Dan Carpenter wrote: Write a better subject line. On Thu, Jul 23, 2015 at 09:53:18PM -0700, Joshua Clayton wrote: change instances SupportedRates to compliant and sane rates This change in no way harms readability, and brings several lines under

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

2015-07-24 Thread Andrey Danin
On 24.07.2015 12:27, Wolfram Sang wrote: Still doesn't work for me and I think I understand why. Do you run your I2C controller in slave mode only? Yes. That might work, but using it in master/slave mode simultanously won't work yet as I see it: * After every transfer (as master), clocks

Re: [PATCH 5/5] staging: rtl8712: style fix:

2015-07-24 Thread Dan Carpenter
Write a better subject line. On Thu, Jul 23, 2015 at 09:53:18PM -0700, Joshua Clayton wrote: change instances SupportedRates to compliant and sane rates This change in no way harms readability, and brings several lines under the 80 character limit. Yeah, but it does a some other stuff as

Re: [PATCH 1/5] staging: rtl8712: fix buggy size calculation

2015-07-24 Thread Dan Carpenter
On Thu, Jul 23, 2015 at 09:38:33PM -0700, Joshua Clayton wrote: r8712_get_ndis_wlan_bssid_ex_sz has a 6 * sizeof(unsigned long) where the underlying struct has a 6 * unsigned char. Simplify the calculation by just subtracting the variable part from the size of the struct. This also gets rid

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

2015-07-24 Thread Wolfram Sang
At the begin of my work on this patchset I even denied clock disable call if slave is registered (to minimize code that can affect transfer). I hacked something like this, but it seems it was not enough. If only slave mode is used, then this logic is not needed. This is not sufficent. We

[PATCH RESEND] staging: rtl8188eu: kill unused hal_data_8188e::fw_ractrl flag

2015-07-24 Thread Jakub Sitnicki
Flag is never set. Remove it and the code that is dead because of it. Signed-off-by: Jakub Sitnicki jsitni...@gmail.com --- drivers/staging/rtl8188eu/hal/odm.c | 11 -- drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c | 21 --

[PATCH 0/6] staging: unisys: module autoload support

2015-07-24 Thread Benjamin Romer
This patch series adds support for automatically loading the unisys drivers from modules. Prarit Bhargava (6): drivers, staging, unisys, cleanup channel_guid.h staging: unisys: add UUID strings to channel_guid.h drivers, staging, unisys Add modalias files to visorbus devices drivers,

[PATCH 8/9] Staging: most: add MOST driver's hdm-usb module

2015-07-24 Thread Christian Gromm
This patch adds the hdm-usb module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles the USB interface of the MOST network interface controller. This patch is needed in order to use the USB peripheral interface of the network interface

[PATCH 7/9] Staging: most: add MOST driver's hdm-i2c module

2015-07-24 Thread Christian Gromm
This patch adds the hdm-i2c module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles the I2C interface of the MOST network interface controller. This patch is needed in order to use the I2C peripheral interface of the network interface

[PATCH 6/9] Staging: most: add MOST driver's hdm-dim2 module

2015-07-24 Thread Christian Gromm
This patch adds the hdm-dim2 module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles the MediaLB interface of the MOST network interface controller. This patch is needed in order to use the MediaLB peripheral interface of the network

[PATCH RESEND] staging: rtl8188eu: don't duplicate ieee80211 WLAN_EID_* constants

2015-07-24 Thread Jakub Sitnicki
linux/ieee80211.h already defines constants for information element IDs. Include it where needed, resolve discrepancies in naming, and remove the duplicated definitions. While at it, wrap a line that was too long and remove extra parentheses in an expression that mixes only equality and logical

[PATCH v2] staging: unisys: Add s-Par visorhba

2015-07-24 Thread Benjamin Romer
From: David Kershner david.kersh...@unisys.com This driver create a host bus adapter device when s-Par sends a device create message to create a storage adapter on the visorbus. When the message is received by visorbus, the visorhba_probe function is called and the hba device is created and

[PATCH 5/7] staging: unisys: visornic - check visorchannel_signalinsert/remove failures

2015-07-24 Thread Benjamin Romer
From: Tim Sell timothy.s...@unisys.com Logic to check for failures of visorchannel_signalinsert() and visorchannel_signalremove() were added, and a new sent_post_failed counter tracks the number of times we failed to post a rcv buffer to the IO partition. Signed-off-by: Tim Sell

[PATCH 6/7] staging: unisys: visornic - consolidate+simplify xmit watermark checks

2015-07-24 Thread Benjamin Romer
From: Tim Sell timothy.s...@unisys.com The code that tests for high and low xmit watermarks was consolidated and simplified. The output behavior should be identical, with the exception of an off-by-one error being corrected in the tests done when the counters overflowed. Note that the literals

[PATCH 2/7] staging: unisys: add simple error-check into visornic receive path

2015-07-24 Thread Benjamin Romer
From: Tim Sell timothy.s...@unisys.com Add an error check here for use with dynamic debugging. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Tim Sell timothy.s...@unisys.com --- drivers/staging/unisys/visornic/visornic_main.c | 6 +++--- 1 file changed, 3 insertions(+),

[PATCH 7/7] staging: unisys: visornic - prevent NETDEV WATCHDOG timeouts after IO recovery

2015-07-24 Thread Benjamin Romer
From: Tim Sell timothy.s...@unisys.com After IO partition recovery, it was possible to get into a situation where a visornic device would repeatedly report: NETDEV WATCHDOG: eth0 (): transmit queue 0 timed out The actual problem would affect any visornic device that was rapidly transmitting

[PATCH 1/7] staging: unisys: visornic: fix serialization mechanism around usage atomic

2015-07-24 Thread Benjamin Romer
From: Neil Horman nhor...@redhat.com Missed this in my initial review. The usage counter that guards against kthread task is horribly racy. The atomic is self consistent, but theres nothing that prevents the service_resp_queue function from re-incrementing it immediately after the check in

[PATCH 3/6] drivers, staging, unisys Add modalias files to visorbus devices

2015-07-24 Thread Benjamin Romer
From: Prarit Bhargava pra...@redhat.com This patch adds modalias files that export the device UUID type to sysfs so that udev can autoload the appropriate device driver on demand. Note that is required a minor name change to the channel device sysfs files which are currently named

[PATCH 2/6] staging: unisys: add UUID strings to channel_guid.h

2015-07-24 Thread Benjamin Romer
From: Prarit Bhargava pra...@redhat.com Define additional strings for module loading code ease of use. Signed-off-by: Prarit Bhargava pra...@redhat.com --- drivers/staging/unisys/include/channel_guid.h | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 1/6] drivers, staging, unisys, cleanup channel_guid.h

2015-07-24 Thread Benjamin Romer
From: Prarit Bhargava pra...@redhat.com This file has a lot of dead comments and needs to be cleaned up. Signed-off-by: Prarit Bhargava pra...@redhat.com --- drivers/staging/unisys/include/channel_guid.h | 20 +--- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git

[PATCH 6/6] drivers, staging, unisys Add visorhid module autoloading code

2015-07-24 Thread Benjamin Romer
From: Prarit Bhargava pra...@redhat.com This patch adds an module alias and a MODULE_DEVICE_TABLE to autoload the visorhid driver when an appropriate device is created by the visorbus. Additional code cleanups to match the other visor* drivers. Signed-off-by: Prarit Bhargava pra...@redhat.com

[PATCH 4/6] drivers, staging, unisys Add visorbus module autoloading code

2015-07-24 Thread Benjamin Romer
From: Prarit Bhargava pra...@redhat.com This patch adds an module alias and a MODULE_DEVICE_TABLE to autoload the visorhba driver when an appropriate device is created by the visorbus. Signed-off-by: Prarit Bhargava pra...@redhat.com --- drivers/staging/unisys/visorhba/visorhba_main.c | 2 ++ 1

[PATCH 5/6] drivers, staging, unisys Add visornic module autoloading code

2015-07-24 Thread Benjamin Romer
From: Prarit Bhargava pra...@redhat.com This patch adds an module alias and a MODULE_DEVICE_TABLE to autoload the visornic driver when an appropriate device is created by the visorbus. Signed-off-by: Prarit Bhargava pra...@redhat.com --- drivers/staging/unisys/visornic/visornic_main.c | 2 ++ 1

[PATCH 0/9] Staging: most: Patchset to add Microchip's MOST driver to kernel 4.1.2

2015-07-24 Thread Christian Gromm
This patchset adds the components of the Media Oriented Systems Transport (MOST) driver to the kernel's driver staging area and adapts the build system accordingly. By including this driver Linux applications will be able to access a MOST network: The Automotive Information Backbone and the

[PATCH 1/9] Staging: most: add MOST driver's core module

2015-07-24 Thread Christian Gromm
This patch adds the core module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles the configuration interface in sysfs, the buffer management and the data routing. Signed-off-by: Christian Gromm christian.gr...@microchip.com --- This

[PATCH 4/9] Staging: most: add MOST driver's aim-sound module

2015-07-24 Thread Christian Gromm
This patch adds the aim-sound module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles user space interaction by means of ALSA devices. This patch is needed in order to have access to MOST synchronous data through ALSA devices.

[PATCH 2/9] Staging: most: add MOST driver's aim-cdev module

2015-07-24 Thread Christian Gromm
This patch adds the aim-cdev module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles user space interaction by means of character devices. This patch is needed in order to have access to MOST data through character devices. Signed-off-by:

[PATCH 5/9] Staging: most: add MOST driver's aim-v4l2 module

2015-07-24 Thread Christian Gromm
This patch adds the aim-v4l2 module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles user space interaction by means of V4L2. This patch is needed in order to have access to MOST isochronous AVP data through V4L2 devices. Signed-off-by:

[PATCH net-next] hv_netvsc: Add structs and handlers for VF messages

2015-07-24 Thread Haiyang Zhang
This patch adds data structures and handlers for messages related to SRIOV Virtual Function. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Reviewed-by: K. Y. Srinivasan k...@microsoft.com --- drivers/net/hyperv/hyperv_net.h | 29 ++ drivers/net/hyperv/netvsc.c

Re: [PATCH 2/7] staging: unisys: add simple error-check into visornic receive path

2015-07-24 Thread Greg KH
On Fri, Jul 24, 2015 at 12:00:20PM -0400, Benjamin Romer wrote: From: Tim Sell timothy.s...@unisys.com Add an error check here for use with dynamic debugging. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Tim Sell timothy.s...@unisys.com ---

Re: [PATCH 1/9] Staging: most: add MOST driver's core module

2015-07-24 Thread Greg KH
On Fri, Jul 24, 2015 at 04:11:48PM +0200, Christian Gromm wrote: This patch adds the core module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles the configuration interface in sysfs, the buffer management and the data routing. snip I

Re: [PATCH 0/9] Staging: most: Patchset to add Microchip's MOST driver to kernel 4.1.2

2015-07-24 Thread Greg KH
On Fri, Jul 24, 2015 at 04:11:47PM +0200, Christian Gromm wrote: This patchset adds the components of the Media Oriented Systems Transport (MOST) driver to the kernel's driver staging area and adapts the build system accordingly. By including this driver Linux applications will be able to

Re: [PATCH RESEND] staging: rtl8188eu: don't duplicate ieee80211 WLAN_EID_* constants

2015-07-24 Thread Greg Kroah-Hartman
On Fri, Jul 24, 2015 at 05:12:38PM +0200, Jakub Sitnicki wrote: linux/ieee80211.h already defines constants for information element IDs. Include it where needed, resolve discrepancies in naming, and remove the duplicated definitions. While at it, wrap a line that was too long and remove

Re: [PATCH 6/7] staging: unisys: visornic - consolidate+simplify xmit watermark checks

2015-07-24 Thread Greg KH
On Fri, Jul 24, 2015 at 12:00:24PM -0400, Benjamin Romer wrote: From: Tim Sell timothy.s...@unisys.com The code that tests for high and low xmit watermarks was consolidated and simplified. The output behavior should be identical, with the exception of an off-by-one error being corrected in

Re: [PATCH 1/7] staging: unisys: visornic: fix serialization mechanism around usage atomic

2015-07-24 Thread Greg KH
On Fri, Jul 24, 2015 at 12:00:19PM -0400, Benjamin Romer wrote: From: Neil Horman nhor...@redhat.com Missed this in my initial review. The usage counter that guards against kthread task is horribly racy. The atomic is self consistent, but theres nothing that prevents the service_resp_queue

Re: [PATCH 2/6] staging: comedi: usbduxsigma: don't clobber ao_timer in command test

2015-07-24 Thread Bernd Porr
Reviewed-by: Bernd Porr m...@berndporr.me.uk Ian Abbott wrote: `devpriv-ao_timer` is used while an asynchronous command is running on the AO subdevice. It also gets modified by the subdevice's `cmdtest` handler for checking new asynchronous commands, `usbduxsigma_ao_cmdtest()`, which is not

RE: [PATCH 0/6] staging: comedi: usbduxsigma: fix some problems in command handling

2015-07-24 Thread Hartley Sweeten
On Thursday, July 23, 2015 8:47 AM, Ian Abbott wrote: Fix some minor problems in the testing of asynchronous commands for the AI and AO subdevices and remove some redundant code. The main problem is that the testing of a new command can affect the operation of an already running command,

Re: [PATCH 5/6] staging: comedi: usbduxsigma: remove unused convert timing for AO

2015-07-24 Thread Bernd Porr
Reviewed-by: Bernd Porr m...@berndporr.me.uk Ian Abbott wrote: The `cmdtest` and `cmd` handlers for the AO subdevice (`usbduxsigma_ao_cmdtest()` and `usbduxsigma_ao_cmd()`) support scan timing of commands with all channels updated every scan period. There is some disabled code to use convert

Re: [PATCH 6/6] staging: comedi: usbduxsigma: round down AO scan_begin_arg at step 4.

2015-07-24 Thread Bernd Porr
Reviewed-by: Bernd Porr m...@berndporr.me.uk Ian Abbott wrote: The return value of the `cmdtest` handler for a subdevice checks the prospective new command in various steps and returns the step number at which any problem was detected, or 0 if no problem was detected. It is allowed to modify

Re: [PATCH 4/6] staging: comedi: usbduxsigma: round down AI scan_begin_arg at step 4.

2015-07-24 Thread Bernd Porr
Reviewed-by: Bernd Porr m...@berndporr.me.uk Ian Abbott wrote: The return value of the `cmdtest` handler for a subdevice checks the prospective new command in various steps and returns the step number at which any problem was detected, or 0 if no problem was detected. It is allowed to modify

Re: [PATCH 3/6] staging: comedi: usbduxsigma: remove AI scan_begin_src == TRIG_FOLLOW

2015-07-24 Thread Bernd Porr
Reviewed-by: Bernd Porr m...@berndporr.me.uk Ian Abbott wrote: The AI subdevice `cmdtest` handler `usbduxsigma_ai_cmdtest()` ensures that `cmd-scan_begin_src == TRIG_TIMER` by the end of step 2 of the command checking code, so assume that this is the case for step 3 onwards and remove the

[PATCH 3/9] Staging: most: add MOST driver's aim-network module

2015-07-24 Thread Christian Gromm
This patch adds the aim-network module of the MOST driver to the kernel's driver staging area. This module is part of the MOST driver and handles user space interaction by means of network devices. This patch is needed in order to have access to MOST Ethernet Packets (MEP) through a networking

[PATCH 3/7] staging: unisys: Process more than one response per check

2015-07-24 Thread Benjamin Romer
From: David Kershner david.kersh...@unisys.com When s-Par is in polling mode it checks every 2 ms to see if there is a response from the IO service partition in the queue. Currently it just reads one entry per 2 ms, this needs to be changed so it drains the queue on each check. Signed-off-by:

[PATCH 0/7] staging: unisys: visornic testing fix series

2015-07-24 Thread Benjamin Romer
This patch series fixes more technical issues and bugs found in visornic testing. David Kershner (1): staging: unisys: Process more than one response per check Neil Horman (1): staging: unisys: visornic: fix serialization mechanism around usage atomic Tim Sell (5): staging: unisys:

[PATCH 4/7] staging: unisys: visornic - ensure proper net locking in tx reset logic

2015-07-24 Thread Benjamin Romer
From: Tim Sell timothy.s...@unisys.com visornic tx reset handling is done asynchronously via a workqueue in visornic_timeout_reset(). As a result, it needs to use rtnl_lock() / rtnl_unlock() to lock against possible simultaneous close() of the network device. (I consulted the bnx2 driver as a

Re: [PATCH 1/7] staging: unisys: visornic: fix serialization mechanism around usage atomic

2015-07-24 Thread Neil Horman
On Fri, Jul 24, 2015 at 01:35:51PM -0700, Greg KH wrote: On Fri, Jul 24, 2015 at 12:00:19PM -0400, Benjamin Romer wrote: From: Neil Horman nhor...@redhat.com Missed this in my initial review. The usage counter that guards against kthread task is horribly racy. The atomic is self

[PATCH] staging: unisys: avoid format string parsing

2015-07-24 Thread Kees Cook
This makes sure the kthread name can't be parsed as a format string. Signed-off-by: Kees Cook keesc...@chromium.org --- drivers/staging/unisys/visornic/visornic_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys/visornic/visornic_main.c

Re: [PATCH 3/5] staging: rtl8712: remove duplicate struct

2015-07-24 Thread Greg Kroah-Hartman
On Thu, Jul 23, 2015 at 09:57:50PM -0700, Joshua Clayton wrote: struct ndis_wlan_bssid_ex is a dopelganger of struct wlan_bssid_ex, and is used about a third as often Switch all instances to wlan_bssid_ex This also gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by: Joshua

Re: [PATCH 1/6] staging: comedi: usbduxsigma: don't clobber ai_timer in command test

2015-07-24 Thread Bernd Porr
Reviewed-by: Bernd Porr m...@berndporr.me.uk Ian Abbott wrote: `devpriv-ai_timer` is used while an asynchronous command is running on the AI subdevice. It also gets modified by the subdevice's `cmdtest` handler for checking new asynchronous commands (`usbduxsigma_ai_cmdtest()`), which is not

[PATCH] staging/xgifb: fix dumpVGAReg compile error if DEBUG is set

2015-07-24 Thread Peter Huewe
If DEBUG is set dumpVGAReg is called and tries to access XGISR which is defined as (xgifb_info-dev_info.P3c4) which is not known within this function. - add as parameter to dumpVGAReg so xgifb_info becomes known Signed-off-by: Peter Huewe peterhu...@gmx.de ---

Re: [PATCH v9 4/7] staging: fpga manager: add sysfs interface document

2015-07-24 Thread atull
On Fri, 24 Jul 2015, Pavel Machek wrote: Hi Pavel, Thanks for your your feedback in cleaning up these docs. Hi! +What: /sys/class/fpga_manager/fpga/state +Date: July 2015 +KernelVersion: 4.2 +Contact: Alan Tull at...@opensource.altera.com