[PATCH 2/2] pwm: Remove .can_sleep from struct pwm_chip

2017-01-04 Thread Thierry Reding
All PWM devices have been marked as "might sleep" since v4.5, there is no longer a need to differentiate on a per-chip basis. Signed-off-by: Thierry Reding --- drivers/pwm/pwm-atmel-hlcdc.c | 1 - drivers/pwm/pwm-atmel.c | 1 - drivers/pwm/pwm-bcm-kona.c| 1 -

[PATCH 1/2] pwm: Remove pwm_can_sleep()

2017-01-04 Thread Thierry Reding
The last user of this function has been removed, so it is no longer needed. Signed-off-by: Thierry Reding --- drivers/pwm/core.c | 12 include/linux/pwm.h | 7 --- 2 files changed, 19 deletions(-) diff --git a/drivers/pwm/core.c b/drivers/pwm/core.c

Re: [PATCH v2 04/19] ARM: dts: imx6-sabrelite: add OV5642 and OV5640 camera sensors

2017-01-04 Thread Vladimir Zapolskiy
Hi Steve, On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > Enables the OV5642 parallel-bus sensor, and the OV5640 MIPI CSI-2 sensor. > Both hang off the same i2c2 bus, so they require different (and non- > default) i2c slave addresses. > > The OV5642 connects to the parallel-bus mux input port

Re: [PATCH v2 05/19] ARM: dts: imx6-sabresd: add OV5642 and OV5640 camera sensors

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > Enables the OV5642 parallel-bus sensor, and the OV5640 MIPI CSI-2 sensor. > > The OV5642 connects to the parallel-bus mux input port on ipu1_csi0_mux. > > The OV5640 connects to the input port on the MIPI CSI-2 receiver on > mipi_csi. It is set

Re: [PATCH v2 09/19] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > Enables the ADV7180 decoder sensor. The ADV7180 connects to the > parallel-bus mux input on ipu1_csi0_mux. > > On the sabreauto, two analog video inputs are routed to the ADV7180, > composite on Ain1, and composite on Ain3. Those inputs are

Re: [PATCH net-next] net/hyperv: remove use of VLAN_TAG_PRESENT

2017-01-04 Thread Vitaly Kuznetsov
Michał Mirosław writes: > Signed-off-by: Michał Mirosław Can we have a non-empty description please? > --- > drivers/net/hyperv/hyperv_net.h | 2 +- > drivers/net/hyperv/netvsc_drv.c | 13 ++--- >

Re: [PATCH 1/2] staging: greybus: audio_topology: Fix spaces between operator and string

2017-01-04 Thread Johan Hovold
On Wed, Jan 04, 2017 at 03:07:31PM +0100, Emmanuil Chatzipetru wrote: > Fix coding style issue caught by checkpatch.pl related to the following > warning: > - "CHECK: spaces preferred around that '*' (ctx:VxV) " > > Signed-off-by: Emmanuil Chatzipetru Acked-by:

Re: [PATCH v2 12/19] media: imx: Add SMFC subdev driver

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > This is a media entity subdevice driver for the i.MX Sensor Multi-FIFO > Controller module. Video frames are received from the CSI and can > be routed to various sinks including the i.MX Image Converter for > scaling, color-space conversion, motion

Re: [PATCH 2/2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Johan Hovold
On Wed, Jan 04, 2017 at 03:07:32PM +0100, Emmanuil Chatzipetru wrote: > Fix coding style issue caught by checkpatch.pl related to the following > warning: > - CHECK: WARNING: line over 80 characters > > Signed-off-by: Emmanuil Chatzipetru > --- >

Re: [PATCH v2 11/19] media: imx: Add CSI subdev driver

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > This is a media entity subdevice for the i.MX Camera > Serial Interface module. > > Signed-off-by: Steve Longerbeam > --- [snip] > diff --git a/drivers/staging/media/imx/imx-csi.c >

Re: [PATCH v2 13/19] media: imx: Add IC subdev drivers

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > This is a set of three media entity subdevice drivers for the i.MX > Image Converter. The i.MX IC module contains three independent > "tasks": > > - Pre-processing Encode task: video frames are routed directly from > the CSI and can be scaled,

[PATCH 2/2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - CHECK: WARNING: line over 80 characters Signed-off-by: Emmanuil Chatzipetru --- drivers/staging/greybus/log.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/2] staging: greybus: audio_topology: Fix spaces between operator and string

2017-01-04 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - "CHECK: spaces preferred around that '*' (ctx:VxV) " Signed-off-by: Emmanuil Chatzipetru --- drivers/staging/greybus/audio_topology.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH v2 15/19] media: imx: Add MIPI CSI-2 Receiver subdev driver

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > Adds MIPI CSI-2 Receiver subdev driver. This subdev is required > for sensors with a MIPI CSI2 interface. > > Signed-off-by: Steve Longerbeam > --- > drivers/staging/media/imx/Makefile| 1 + >

Re: [PATCH v2 10/19] media: Add i.MX media core driver

2017-01-04 Thread Vladimir Zapolskiy
Hi Steve, On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > Add the core media driver for i.MX SOC. > > Signed-off-by: Steve Longerbeam > --- > Documentation/devicetree/bindings/media/imx.txt | 205 + v2 was sent before getting Rob's review comments, but

Re: [PATCH v2 14/19] media: imx: Add Camera Interface subdev driver

2017-01-04 Thread Vladimir Zapolskiy
On 01/03/2017 10:57 PM, Steve Longerbeam wrote: > This is the camera interface driver that provides the v4l2 > user interface. Frames can be received from various sources: > > - directly from SMFC for capturing unconverted images directly from > camera sensors. > > - from the IC pre-process

Re: [PATCH] staging: gdm724x: add forced casts in endian converters to fix sparse warnings

2017-01-04 Thread Eric S. Stone
On Wed, Jan 04, 2017 at 09:59:27AM +0100, Greg KH wrote: > On Wed, Jan 04, 2017 at 09:54:53AM +0100, Greg KH wrote: > > On Tue, Jan 03, 2017 at 10:44:17PM -0800, Eric S. Stone wrote: > > > On Tue, Jan 03, 2017 at 04:30:58PM +0100, Greg KH wrote: > > > > On Wed, Dec 28, 2016 at 10:08:43PM -0800,

[PATCH] hv: use substraction to update ring buffer index

2017-01-04 Thread Long Li
From: Long Li The ring buffer code uses %= to calculate index. For x86/64, %= compiles to div, more than 10 times slower than sub. Replace div with sub for this data heavy code path. Signed-off-by: Long Li --- drivers/hv/ring_buffer.c | 9 ++---

Re: Designated initializers, struct randomization and addressing?

2017-01-04 Thread Kees Cook
On Wed, Jan 4, 2017 at 8:55 AM, Stephen Hemminger wrote: > On Tue, 3 Jan 2017 22:35:26 -0800 > Kees Cook wrote: > >> For randstruct and constify, the automatic selection is done on >> structures with only function pointers. (Additional

[PATCH v2] hv: retry infinitely on hypercall transient failures

2017-01-04 Thread Long Li
From: Long Li Hyper-v host guarantees that a hypercall will finish in reasonable time. Retry infinitely on transient failures to avoid returning error to upper layer. Signed-off-by: Long Li --- drivers/hv/connection.c | 30

[PATCH 1/2] drivers/staging/rtl8188eu/core/rtw_ap.c un-necessary parenthesis

2017-01-04 Thread Scott Matheina
Fixed coding style issue, un-necessary parenthesis present Signed-off-by: Scott Matheina --- drivers/staging/rtl8188eu/core/rtw_ap.c | 106 1 file changed, 53 insertions(+), 53 deletions(-) diff --git

[PATCH 2/2] drivers/staging/rtl8188eu/core/rtw_ap.c Alignment should match open parenthesis

2017-01-04 Thread Scott Matheina
Fixed style issue: Alignment should match open parenthesis throughout file Signed-off-by: Scott Matheina --- drivers/staging/rtl8188eu/core/rtw_ap.c | 38 - 1 file changed, 18 insertions(+), 20 deletions(-) diff --git

[PATCH 0/2] Fixes coding style

2017-01-04 Thread Scott Matheina
The patch series addresses some coding style issues contained within the file. Scott Matheina (2): drivers/staging/rtl8188eu/core/rtw_ap.c un-necessary parenthesis drivers/staging/rtl8188eu/core/rtw_ap.c Alignment should match open parenthesis drivers/staging/rtl8188eu/core/rtw_ap.c |

[PATCH] Staging: ks7010: style fix, long lines

2017-01-04 Thread Derek Robson
Debug code had very long lines. Reworked code to use several prints rather than one big print. Signed-off-by: Derek Robson --- drivers/staging/ks7010/ks_hostif.c | 32 +--- 1 file changed, 25 insertions(+), 7 deletions(-) diff --git

[PATCH] staging: wilc1000: Connect to highest RSSI value for required SSID

2017-01-04 Thread Aditya Shankar
Connect to the highest rssi with the required SSID in the shadow table if the connection criteria is based only on the SSID. For the first matching SSID, an index to the table is saved. Later the index is updated if matching SSID has a higher RSSI value than the last saved index. However if

[PATCH] Staging: lustre: lustre: lmv: Compress return logic into one line.

2017-01-04 Thread Gustavo A. R. Silva
Simplify return logic to avoid unnecessary variable assignments. These issues were detected using Coccinelle and the following semantic patch: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: Gustavo A. R. Silva ---

Re: [PATCH v2] hv: retry infinitely on hypercall transient failures

2017-01-04 Thread Greg KH
On Wed, Jan 04, 2017 at 06:12:20PM -0800, Long Li wrote: > From: Long Li > > Hyper-v host guarantees that a hypercall will finish in reasonable time. > Retry infinitely on transient failures to avoid returning error to upper > layer. Again, never retry "forever", always

Re: [PATCH v2 05/19] ARM: dts: imx6-sabresd: add OV5642 and OV5640 camera sensors

2017-01-04 Thread Fabio Estevam
On Tue, Jan 3, 2017 at 6:57 PM, Steve Longerbeam wrote: > + camera: ov5642@3c { > + compatible = "ovti,ov5642"; > + pinctrl-names = "default"; > + pinctrl-0 = <_ov5642>; > + clocks = < IMX6QDL_CLK_CKO>; > +

Re: [PATCH v2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Johan Hovold
On Wed, Jan 04, 2017 at 04:14:45PM +0100, Emmanuil Chatzipetru wrote: > Fix coding style issue caught by checkpatch.pl related to the following > warning: > - CHECK: WARNING: line over 80 characters This commit message should be expanded to explain how this is done (e.g. "by dropping a

Re: [PATCH v3 1/2] staging: greybus: audio_topology: Fix spaces between operator and string

2017-01-04 Thread Johan Hovold
On Wed, Jan 04, 2017 at 05:08:18PM +0100, Emmanuil Chatzipetru wrote: > Fix coding style issue caught by checkpatch.pl related to the following > warning: > - "CHECK: spaces preferred around that '*' (ctx:VxV) " > > Signed-off-by: Emmanuil Chatzipetru Acked-by:

Re: [PATCH v3 2/2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Johan Hovold
On Wed, Jan 04, 2017 at 05:08:19PM +0100, Emmanuil Chatzipetru wrote: > Fix coding style issue caught by checkpatch.pl related to the following > warning: > - CHECK: WARNING: line over 80 characters > This is done by dropping a redundant cast and by replacing the format > specifier > in

Re: [PATCH v2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Emmanuil Chatzipetru
Hello, On Wed, Jan 04, 2017 at 04:26:35PM +0100, Johan Hovold wrote: > On Wed, Jan 04, 2017 at 04:14:45PM +0100, Emmanuil Chatzipetru wrote: > > Fix coding style issue caught by checkpatch.pl related to the following > > warning: > > - CHECK: WARNING: line over 80 characters > > This commit

[PATCH v3 2/2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - CHECK: WARNING: line over 80 characters This is done by dropping a redundant cast and by replacing the format specifier in dev_err(); to "%zu" instead of "%d", in order to silence the warnings of the

[PATCH v3 1/2] staging: greybus: audio_topology: Fix spaces between operator and string

2017-01-04 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - "CHECK: spaces preferred around that '*' (ctx:VxV) " Signed-off-by: Emmanuil Chatzipetru --- drivers/staging/greybus/audio_topology.c | 2 +- 1 file changed, 1 insertion(+), 1

[RFC PATCH] scsi: scsi_transport_fc: Create a lightweight option for Virtual FC Hosts.

2017-01-04 Thread Cathy Avery
This patch represents an attempt to resurrect the conversation based on the submission of patch: [PATCH 1/1] scsi: storvsc: Support manual scan of FC hosts on Hyper-V K. Y. Srinivasan kys at microsoft.com Sat Mar 12 21:52:48 UTC 2016

[PATCH v2] staging: greybus: log: Fix line over 80 characters.

2017-01-04 Thread Emmanuil Chatzipetru
Fix coding style issue caught by checkpatch.pl related to the following warning: - CHECK: WARNING: line over 80 characters While at it, drop the redundant cast in the comparison. Signed-off-by: Emmanuil Chatzipetru --- v2: - As Johan suggested, is better to

Re: [PATCH 1/2] pwm: Remove pwm_can_sleep()

2017-01-04 Thread kbuild test robot
Hi Thierry, [auto build test ERROR on pwm/for-next] [also build test ERROR on v4.10-rc2 next-20170104] [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/Thierry-Reding/pwm-Remove-pwm_can_sleep

Re: [PATCH] staging: gdm724x: add forced casts in endian converters to fix sparse warnings

2017-01-04 Thread Greg KH
On Wed, Jan 04, 2017 at 09:54:53AM +0100, Greg KH wrote: > On Tue, Jan 03, 2017 at 10:44:17PM -0800, Eric S. Stone wrote: > > On Tue, Jan 03, 2017 at 04:30:58PM +0100, Greg KH wrote: > > > On Wed, Dec 28, 2016 at 10:08:43PM -0800, Eric S. Stone wrote: > > > > The modified functions do explicit

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

2017-01-04 Thread Eva Rachel Retuya
On Fri, Dec 30, 2016 at 08:16:02PM +, Jonathan Cameron wrote: > On 11/12/16 02:47, Eva Rachel Retuya wrote: > > 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

[PATCH v2] staging: gdm724x: use different endian converters to fix sparse warnings

2017-01-04 Thread Eric S. Stone
These functions require non __bitwise annotated types for both input and output. Changing the endian converters used from ones that operate on __bitwise annotated types, to those that operate on u16/u32, fixes these sparse warnings: CHECK drivers/staging/gdm724x/gdm_endian.c

Re: [PATCH] staging: gdm724x: add forced casts in endian converters to fix sparse warnings

2017-01-04 Thread Greg KH
On Tue, Jan 03, 2017 at 10:44:17PM -0800, Eric S. Stone wrote: > On Tue, Jan 03, 2017 at 04:30:58PM +0100, Greg KH wrote: > > On Wed, Dec 28, 2016 at 10:08:43PM -0800, Eric S. Stone wrote: > > > The modified functions do explicit endian checking and conversion. The > > > added forced casts fix

Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file

2017-01-04 Thread Greg KH
On Tue, Jan 03, 2017 at 09:26:46PM +, Stuart Yoder wrote: > > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Tuesday, January 03, 2017 10:48 AM > > To: Stuart Yoder > > Cc: de...@driverdev.osuosl.org; a...@arndb.de; Roy

Re: [PATCH] Staging: fbtft: hx8340bn: Style fix, octal not decimal permissions

2017-01-04 Thread Andy Shevchenko
On Wed, Jan 4, 2017 at 3:20 AM, Derek Robson wrote: > Fixed coding style issue, found with Checkpatch.pl > changed permissions to octal style. You missed SoB. > @@ -37,7 +37,7 @@ > "3 3 17 8 4 7 05 7 6 0 3 1 6 0 0 " > > static bool emulate; >

Re: [PATCH v2] staging: gdm724x: use different endian converters to fix sparse warnings

2017-01-04 Thread Greg KH
On Wed, Jan 04, 2017 at 12:25:55AM -0800, Eric S. Stone wrote: > These functions require non __bitwise annotated types for both input > and output. Changing the endian converters used from ones that operate > on __bitwise annotated types, to those that operate on u16/u32, fixes > these sparse

Re: [PATCH net-next] net/hyperv: remove use of VLAN_TAG_PRESENT

2017-01-04 Thread Stephen Hemminger
On Wed, 4 Jan 2017 01:07:58 +0100 (CET) Michał Mirosław wrote: > Signed-off-by: Michał Mirosław I have a cleaner way of handling this in the receive path for hyperv. Rather than passing vlan_tci, pass the vlan info

Re: [PATCH v2 3/4] hv_util: use do_adjtimex() to update system time

2017-01-04 Thread Stephen Hemminger
On Wed, 4 Jan 2017 18:24:38 +0100 Vitaly Kuznetsov wrote: > With TimeSync version 4 protocol support we started updating system time > continuously through the whole lifetime of Hyper-V guests. Every 5 seconds > there is a time sample from the host which triggers

Re: [PATCH v2 4/4] hv_util: improve time adjustment accuracy by disabling interrupts

2017-01-04 Thread Stephen Hemminger
On Wed, 4 Jan 2017 18:24:39 +0100 Vitaly Kuznetsov wrote: > If we happen to receive interrupts during hv_set_host_time() execution > our adjustments may get inaccurate. Make the whole function atomic. > Unfortunately, we can's call do_settimeofday64() with interrupts >

[PATCH v2] staging: rtl8188eu: remove unused function _linked_rx_signal_strehgth_display

2017-01-04 Thread Luca Ceresoli
Not referenced anymore since commit 9fe7b29c6cc1 ("staging: rtl8188eu: remove unused field bRxRSSIDisplay in struct adapter"). Cc: Greg Kroah-Hartman Cc: Larry Finger Cc: Ivan Safonov Cc: de...@driverdev.osuosl.org

[PATCH v2 3/4] hv_util: use do_adjtimex() to update system time

2017-01-04 Thread Vitaly Kuznetsov
With TimeSync version 4 protocol support we started updating system time continuously through the whole lifetime of Hyper-V guests. Every 5 seconds there is a time sample from the host which triggers do_settimeofday[64](). While the time from the host is very accurate such adjustments may cause

[PATCH v2 4/4] hv_util: improve time adjustment accuracy by disabling interrupts

2017-01-04 Thread Vitaly Kuznetsov
If we happen to receive interrupts during hv_set_host_time() execution our adjustments may get inaccurate. Make the whole function atomic. Unfortunately, we can's call do_settimeofday64() with interrupts disabled as some cross-CPU work is being done but this call happens very rarely.

[PATCH v2 2/4] hv_util: switch to using timespec64

2017-01-04 Thread Vitaly Kuznetsov
do_settimeofday() is deprecated, use do_settimeofday64() instead. Signed-off-by: Vitaly Kuznetsov --- drivers/hv/hv_util.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/hv/hv_util.c b/drivers/hv/hv_util.c index e770774..94719eb 100644 ---

[PATCH v2 1/4] timekeeping: export do_adjtimex() to modules

2017-01-04 Thread Vitaly Kuznetsov
While do_adjtimex() is available to userspace via adjtimex syscall it is not available to modules which may want to implement in-kernel 'NTP clients'. Hyper-V hv_utils is going to be the first one. Signed-off-by: Vitaly Kuznetsov --- kernel/time/timekeeping.c | 1 + 1 file

Re: Designated initializers, struct randomization and addressing?

2017-01-04 Thread Stephen Hemminger
On Tue, 3 Jan 2017 22:35:26 -0800 Kees Cook wrote: > For randstruct and constify, the automatic selection is done on > structures with only function pointers. (Additional structures can be > added via a compiler attribute marking.) > > See is_pure_ops_struct(): Is there

staging/fbtft: Backward Device Tree compatibility in a drm version

2017-01-04 Thread Noralf Trønnes
Hi, I'm working on a drivers/gpu/drm version of drivers/staging/fbtft which are drivers for tiny, usually SPI connected, displays. Now I'm wondering if I can be backwards compatible and support Device Trees written for the fbtft drivers. The main obstacle as I understand it, is the init property

[PATCH v2 0/4] hv_util: adjust system time smoothly

2017-01-04 Thread Vitaly Kuznetsov
Changes since v1: - do do_settimeofday64() when ICTIMESYNCFLAG_SYNC flag is present in the request (Alex Ng) - add pr_debug() for the case when do_adjtimex() fails (Alex Ng) Original description: With TimeSync version 4 protocol support we started updating system time continuously through the

Re: Designated initializers, struct randomization and addressing?

2017-01-04 Thread Julia Lawall
On Wed, 4 Jan 2017, Stephen Hemminger wrote: > On Tue, 3 Jan 2017 22:35:26 -0800 > Kees Cook wrote: > > > For randstruct and constify, the automatic selection is done on > > structures with only function pointers. (Additional structures can be > > added via a compiler

[PATCH 5/5] staging:r8188eu: remove unused WIFI_MP_*STATE and WIFI_MP_CTX* definitions

2017-01-04 Thread Ivan Safonov
fw_state member of struct mlme_priv never obtain WIFI_MP_STATE value, so code only for (fw_state == WIFI_MP_STATE) is dead. Remove it, WIFI_MP_*STATE and WIFI_MP_CTX* definitions. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 3 --

[PATCH 4/5] staging:r8188eu: eleminate recovery attemp using skb_clone after netdev_alloc_skb fail

2017-01-04 Thread Ivan Safonov
It is wrong to create new skb using skb_clone instead netdev_alloc_skb, because buffer data will be changed later. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c | 19 +++ 1 file changed, 3 insertions(+), 16 deletions(-)

Re: [PATCH] staging: greybus: add host device function pointer checks

2017-01-04 Thread Johan Hovold
On Tue, Jan 03, 2017 at 01:46:09PM -0600, Jason Hrycay wrote: > On Mon, Jan 02, 2017 at 02:54:37PM +0100, Johan Hovold wrote: > > On Tue, Dec 20, 2016 at 02:49:27PM -0600, Jason Hrycay wrote: > > > Add sanity checks for cport_quiesce and cport_clear before invoking the > > > callbacks as these

[PATCH 1/5] staging:r8188eu: remove RTW_STATUS_CODE()

2017-01-04 Thread Ivan Safonov
RTW_STATUS_CODE() does not used. Remove it. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/osdep_service.h | 2 -- drivers/staging/rtl8188eu/os_dep/osdep_service.c | 12 2 files changed, 14 deletions(-) diff --git

[PATCH 3/5] staging:r8188eu: remove (NDEV|ADPT)_(FMT|ARG) definitions

2017-01-04 Thread Ivan Safonov
(NDEV|ADPT)_(FMT|ARG) definitions does not used. Remove it. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/osdep_service.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/osdep_service.h

[PATCH 2/5] staging:r8188eu: remove rtw_update_mem_stat definition

2017-01-04 Thread Ivan Safonov
rtw_update_mem_stat definition does not used. Remove it. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/osdep_service.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/osdep_service.h

[PATCH 04/14] staging: comedi: daqboard2000: check firmware length

2017-01-04 Thread Ian Abbott
Firmware files for DAQBoard/2000 have a header, which is skipped, followed by a sequence of FPGA configuration bytes to be programmed in pairs. The FPGA configuration bytes start with the sequence 0xff, 0x20. Make the firmware loading callback function `daqboard2000_load_firmware()` return an

[PATCH 03/14] staging: comedi: daqboard2000: use type 'u16' for CPLD data and status

2017-01-04 Thread Ian Abbott
The CPLD status and data registers used to load firmware are 16 bits wide. Use the type `u16` to represent data and status values instead of `int`. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/daqboard2000.c | 8 1 file changed, 4 insertions(+), 4

[PATCH 02/14] staging: comedi: daqboard2000: define macros for CPLD registers

2017-01-04 Thread Ian Abbott
The Daqboard/2000 uses a write-only data register and a read-only status register in a pre-programmed CPLD device to program the main firmware on the board. Both registers are at offset 0x1000 from PCI BAR 2. Define macros for the register offsets. Rename the existing macros for the status

[PATCH 01/14] staging: comedi: daqboard2000: use macros from "plx9080.h"

2017-01-04 Thread Ian Abbott
The Daqboard/2000 uses a PLX PCI-9080 chip to interface with the PCI bus. The "daqboard2000" driver uses the PCI-9080 "CNTRL" register to perform various tasks, but defines its own macros for the register values. Use the macros from "plx9080.h" instead. The various functions that change the

[PATCH 05/14] staging: comedi: daqboard2000: replace daqboard2000_poll_cpld()

2017-01-04 Thread Ian Abbott
`daqboard2000_poll_cpld()` waits for a specified status bit in the CPLD status register to be set, giving up after 50 tries over a period of about 5 milliseconds. It returns 1 if the status bit is set, otherwise 0. It is only ever called to check the "INIT" status bit. Replace it with new

[PATCH 07/14] staging: comedi: daqboard2000: check result of FPGA programming

2017-01-04 Thread Ian Abbott
According to an old, GPL'ed Linux driver at , after programming the FPGA, the General Purpose Input (USERI) of the PLX PCI-9080 should go high shortly after a valid FPGA bitstream has been loaded.

[PATCH 08/14] staging: comedi: daqboard2000: check CPLD status before writing firmware data

2017-01-04 Thread Ian Abbott
According to an old GPL'ed driver at , The CPLD status register can be checked to make sure that it is ready to accept the next 16-bit word of FPGA firmware data, but that doesn't work on older

[PATCH 14/14] staging: comedi: daqboard2000: use pci_id_table 'driver_data'

2017-01-04 Thread Ian Abbott
The driver's COMEDI "auto-attach" handler `db2k_auto_attach()` calls `db2k_find_boardinfo()` to find an element of our board information array `db2k_boardtypes[]` that matches the probed PCI device. The driver's PCI device table matches several boards in the DaqBoard/2000 series that match a

[PATCH 12/14] staging: comedi: daqboard2000: support 4 AO channels

2017-01-04 Thread Ian Abbott
The driver supports DaqBoard/2000 and DaqBoard/2001. DaqBoard/2000 has 2 AO channels, but DaqBoard/2001 has 4 AO channels. The driver currently only supports 2 AO channels, but supporting 4 channels is just a case of setting the `n_chan` member of the COMEDI subdevice to 4 instead of 2. Add a

[PATCH 11/14] staging: comedi: daqboard2000: use designated initializers

2017-01-04 Thread Ian Abbott
Replace the undesignated initializers for each element of `db2k_boardtypes[]` with an equivalent designated initializer for ease of future maintenance. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/daqboard2000.c | 10 -- 1 file changed, 8

[PATCH 06/14] staging: comedi: daqboard2000: change daqboard2000_write_cpld() return value

2017-01-04 Thread Ian Abbott
`daqboard2000_write_cpld()` currently returns 1 on success, or 0 on failure. Change it to return 0 on success, or `-EIO` on failure. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/daqboard2000.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-)

[PATCH 09/14] staging: comedi: daqboard2000: remove unused 'card' member

2017-01-04 Thread Ian Abbott
The `card` member of `struct daqboard2000_private` and the enumerated constant `card_daqboard_2000` are not used. Remove them. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/daqboard2000.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 00/14] staging: comedi: daqboard2000: some clean-up

2017-01-04 Thread Ian Abbott
I've had these clean-up patches for the daqboard2000 driver sitting around for a few months, with a view to adding support for more boards in the same series, and adding support for extra subdevice types. I never got around to doing that, but I did manage to add support for a couple of extra DAC

[PATCH 13/14] staging: comedi: daqboard2000: change COMEDI device names

2017-01-04 Thread Ian Abbott
The COMEDI device name strings are currently set to "ids2" for the DaqBoard/2000, and to "ids4" for the DaqBoard/2001. Change them to "daqboard2000" and "daqboard2001" respectively. (The COMEDI driver name string is also "daqboard2000".) Signed-off-by: Ian Abbott ---

[PATCH 10/14] staging: comedi: daqboard2000: use shorter, consistent prefix

2017-01-04 Thread Ian Abbott
Use a consistent prefix of `db2k_` or `DB2K_` for identifiers. The existing prefixes `DAQBOARD2000_` and `daqboard2000_` are a bit on the lengthy side. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/daqboard2000.c | 214 -- 1 file

[PATCH] Retry infinitely for hypercall

2017-01-04 Thread Long Li
From: Long Li Hyper-v host guarantees that a hypercall will succeed. Retry infinitely to avoid returning transient failures to upper layer. Signed-off-by: Long Li --- drivers/hv/connection.c | 17 - 1 file changed, 8 insertions(+),

Re: [PATCH] Retry infinitely for hypercall

2017-01-04 Thread Greg KH
On Wed, Jan 04, 2017 at 02:39:31PM -0800, Long Li wrote: > From: Long Li > > Hyper-v host guarantees that a hypercall will succeed. Retry infinitely to > avoid returning transient failures to upper layer. Please wrap your changelog at the proper column. And what happens

Re: [PATCH] Retry infinitely for hypercall

2017-01-04 Thread Dan Carpenter
Fix the subsystem prefix in the subject. On Wed, Jan 04, 2017 at 02:39:31PM -0800, Long Li wrote: > From: Long Li > > Hyper-v host guarantees that a hypercall will succeed. Retry infinitely to > avoid returning transient failures to upper layer. > > Signed-off-by: Long

RE: [PATCH] Retry infinitely for hypercall

2017-01-04 Thread Long Li
> -Original Message- > From: Greg KH [mailto:g...@kroah.com] > Sent: Wednesday, January 4, 2017 12:51 PM > To: Long Li > Cc: KY Srinivasan ; Haiyang Zhang > ; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org

RE: [PATCH] Retry infinitely for hypercall

2017-01-04 Thread Long Li
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Wednesday, January 4, 2017 1:48 PM > To: Long Li > Cc: KY Srinivasan ; Haiyang Zhang > ; de...@linuxdriverproject.org; linux- >