[PATCH] staging: rtl8188eu: unused macro read_next_pair removed

2015-10-07 Thread Ivan Safonov
Unused macro read_next_pair removed from rtl8188eu driver. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index 9c7e626..fa461b4 100

[PATCH V2] staging: dgnc: Remove dgnc_ports_state_show

2015-10-07 Thread Salah Triki
dgnc_ports_state_show is removed, since it exposes ports status which help an adversary to plan an attack. Signed-off-by: Salah Triki --- drivers/staging/dgnc/dgnc_sysfs.c | 21 - 1 file changed, 21 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_sysfs.c b/drivers/stag

[PATCH] staging: rtl8188eu: ARRAY_SIZE instead of sizeof/sizeof used

2015-10-07 Thread Ivan Safonov
Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index fa461b4..ff5b15e 100644 --- a/drivers/staging/rtl8188eu/hal/bb_cf

Re: [PATCH 0/3] staging: sm750fb: remove sii164 DVI controller support

2015-10-07 Thread Sudip Mukherjee
On Tue, Oct 06, 2015 at 04:49:13PM +0100, Mike Rapoport wrote: > Currently the sii164 DVI controller support is unused anywhere in the rest of > the driver, and, anyway it does not belong to framebuffer device driver. > These patches remove the sii164 DVI controller support. I am not sure if we can

Re: [PATCH] unused macro read_next_pair removed

2015-10-07 Thread Sudip Mukherjee
On Thu, Oct 08, 2015 at 08:26:40AM +0700, Ivan Safonov wrote: > Unused macro read_next_pair removed. > > Signed-off-by: Ivan Safonov > --- Your subject is not clear. It should have been: [PATCH] staging: rtl8188eu: unused macro read_next_pair removed same for your next patch also. regards sudip

[PATCH 14/14] staging: wilc1000: remove wilc_wlan.c included in wilc_wfi_cfgoperations.c

2015-10-07 Thread Tony Cho
This patch removes "wilc_wlan.c" from the wilc_wfi_cfgoperation.c file and adds wilc_wlan.o into Makefile to compile it because there is few benefits. This patch also adds "wilc_wfi_netdevice.h" in the wilc_wlan.c file to avoid the compile errors. Signed-off-by: Tony Cho --- drivers/staging/wilc

[PATCH 11/14] staging: wilc1000: remove typedef from tenuConnDisconnEvent

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the enum tenuConnDisconnEvent and renames it to conn_event. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 6 +++--- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +- 2 files changed,

[PATCH 12/14] staging: wilc1000: remove typedef from tenuScanConnTimer

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the enum tenuScanConnTimer and renames it to scan_conn_timer. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/

[PATCH 13/14] staging: wilc1000: host_infterface.h: remove unused enum tenuWILC_StaFlag

2015-10-07 Thread Tony Cho
This patch removes unused enum tenuWILC_StaFlag from the host_interface.h. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 20 1 file changed, 20 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.h b/drivers/staging/wilc1000/host_int

[PATCH 05/14] staging: wilc1000: remove typedef from tstrHiddenNetworkInfo

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the struct tstrHiddenNetworkInfo with related comments and renames it to hidden_net_info. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) d

[PATCH 07/14] staging: wilc1000: remove unused struct cfg_param_t

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes unused the struct cfg_param_t. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.h b/drivers/staging/wilc1000/host_inte

[PATCH 06/14] staging: wilc1000: remove unused struct WILC_WFIDrvHandle

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes unused the struct WILC_WFIDrvHandle with related commnets. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.h b/drivers

[PATCH 09/14] staging: wilc1000: remove typedef from tenuScanEvent

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the enum tenuScanEvent and rename it to scan_event. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- drivers/staging/wilc1000/host_interface.h | 15 +-- drivers/stagi

[PATCH 08/14] staging: wilc1000: remove typedef from tstrStatistics

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the struct tstrStatistics and rename it to statistics. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- drivers/staging/wilc1000/host_interface.h | 7 +++ drivers/staging/

[PATCH 04/14] staging: wilc1000: remove typedef from tstrFoundNetworkInfo

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the struct tstrFoundNetworkInfo and rename it to found_net_info. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 10/14] staging: wilc1000: rename typedef from tenuCfgParam

2015-10-07 Thread Tony Cho
From: Leo Kim This patch rename typedef from the enum tenuCfgParamand rename it to cfg_param. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_inte

[PATCH 03/14] staging: wilc1000: remove typedef from tenuHostIFstate

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the enum tenuHostIFstate and rename it to host_if_state. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wilc10

[PATCH 01/14] staging: wilc1000: remove typedef from tstrWILC_UsrScanReq

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the struct tstrWILC_UsrScanReq with related comments and renames it to user_scan_req. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff -

[PATCH 02/14] staging: wilc1000: remove typedef from tstrWILC_UsrConnReq

2015-10-07 Thread Tony Cho
From: Leo Kim This patch removes typedef from the struct tstrWILC_UsrConnReq with the related comments and renames it to user_conn_req. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) d

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

2015-10-07 Thread Laura Abbott
On 10/7/15 11:36 AM, Rob Herring wrote: On Wed, Oct 7, 2015 at 5:36 AM, Andrew wrote: On 2015-10-07 02:01, Laura Abbott wrote: On 10/6/15 3:35 PM, Rob Herring wrote: On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott wrote: From: Laura Abbott This adds a base set of devicetree bindings for

[PATCH] ARRAY_SIZE instead of sizeof/sizeof used

2015-10-07 Thread Ivan Safonov
ARRAY_SIZE instead of sizeof/sizeof used. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index fa461b4..ff5b15e 10064

[PATCH 13/14] staging: comedi: dt3000: remove #if 0'ed out code

2015-10-07 Thread H Hartley Sweeten
Replace the disabled code in dt3k_dio_config() with a comment in case it's actually needed. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/stag

[PATCH 12/14] staging: comedi: dt3000: tidy up memory subdevice init

2015-10-07 Thread H Hartley Sweeten
Remove the unnecessary 'len_chanlist' initialization. This member is only used by subdevices that support async commands. For aesthetics, reorder the initialization a bit. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 5 ++

[PATCH 14/14] staging: comedi: dt3000: update the MODULE_DESCRIPTION

2015-10-07 Thread H Hartley Sweeten
Change the MODULE_DESCRIPTION to something more ueeful than the generic "Comedi low-level driver". Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/stag

[PATCH 10/14] staging: comedi: dt3000: rename dt3k_ai_insn()

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename this subdevice (*insn_read) function to follow the normal naming in comedi drivers. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff -

[PATCH 11/14] staging: comedi: dt3000: tidy up digital i/o subdevice init

2015-10-07 Thread H Hartley Sweeten
Remove the unnecessary 'len_chanlist' initialization. This member is only used by subdevices that support async commands. For aesthetics, reorder the initialization a bit. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 7 ++

[PATCH 04/14] staging: comedi: dt3000: rename the SUBS_* defines

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename these defines to avoid any confusion with the similar comedi defines. Move them closer to the register they are associated with. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 31 +++---

[PATCH 03/14] staging: comedi: dt3000: tidy up the command mailbox defines

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename these defines so they are associated with the register. Move the defines closer to the register define. Define some macros for the "completion" and "cmd" bits. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt30

[PATCH 07/14] staging: comedi: dt3000: remove proc subdevice

2015-10-07 Thread H Hartley Sweeten
This subdevice is not allocated or defined in the driver. Remove the disabled subdevice initialization. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/co

[PATCH 06/14] staging: comedi: dt3000: tidy up remaining defines

2015-10-07 Thread H Hartley Sweeten
Move the remaining defines and prefix them with 'DPR_' for consistency. Define a macro to set the analog input trigger bits to fix the checkpatch.pl issues about: CHECK: spaces preferred around that '<<' (ctx:VxV) CHECK: Prefer using the BIT macro Signed-off-by: H Hartley Sweeten Cc: Ian Abbott

[PATCH 01/14] staging: comedi: dt3000: tidy up multi-line comments

2015-10-07 Thread H Hartley Sweeten
Reformat the multi-line comments in the kernel CodingStyle. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 91 + 1 file changed, 46 insertions(+), 45 deletions(-) diff --git a/drivers/staging

[PATCH 08/14] staging: comedi: dt3000: refactor 'adbits' boardinfo

2015-10-07 Thread H Hartley Sweeten
All the board supported by this driver have analog inputs. The input resolution is either 12-bit or 16-bit. Replace the 'adbits' member of the boardinfo with a bit-field flag 'ai_is_16bits' and just set if for the 16-bit boards. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Har

[PATCH 09/14] staging: comedi: dt3000: the dt3002 board does not have analog outputs

2015-10-07 Thread H Hartley Sweeten
According to the boardinfo, the dt3002 board does not have analog outputs. The rest of the board have two 12-bit analog output channels. Replace the 'dachan' and 'dabits' members of the boardinfo with a bit- field flag 'has_ao'. Use the new member to conditionally initialize the analog output subd

[PATCH] unused macro read_next_pair removed

2015-10-07 Thread Ivan Safonov
Unused macro read_next_pair removed. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index 9c7e626..fa461b4 100644 --- a/drivers/stag

[PATCH 05/14] staging: comedi: dt3000: rename the interrupt flag defines

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename these defines so they have association with the register. Move them closer to the register define and use the BIT macro to define them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt3000.c | 24 +++

[PATCH 02/14] staging: comedi: dt3000: rename dual-ported RAM location defines

2015-10-07 Thread H Hartley Sweeten
Rename these CamelCase defines and fix the checkpatch.pl issues: CHECK: Avoid CamelCase: CHECK: spaces preferred around that '*' (ctx:VxV) CHECK: spaces preferred around that '+' (ctx:VxV) For aesthetics, move the defines to after the includes. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott

[PATCH 00/14] staging: comedi: dt3000: cleanup driver

2015-10-07 Thread H Hartley Sweeten
This series fixes all the checkpatch.pl issues and cleans up the driver a bit. There are still some issues with this driver but I have not been able to locate a datasheet... :-( H Hartley Sweeten (14): staging: comedi: dt3000: tidy up multi-line comments staging: comedi: dt3000: rename dual-p

[PATCH 03/10] tools: hv: report ENOSPC errors in hv_fcopy_daemon

2015-10-07 Thread K. Y. Srinivasan
From: Olaf Hering Currently some "Unspecified error 0x80004005" is reported on the Windows side if something fails. Handle the ENOSPC case and return ERROR_DISK_FULL, which allows at least Copy-VMFile to report a meaning full error. Signed-off-by: Olaf Hering Signed-off-by: K. Y. Srinivasan --

[PATCH 10/10] drivers:hv: Define the channel type for Hyper-V PCI Express pass-through

2015-10-07 Thread K. Y. Srinivasan
From: Jake Oshins This defines the channel type for PCI front-ends in Hyper-V VMs. Signed-off-by: Jake Oshins Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c |3 +++ include/linux/hyperv.h| 11 +++ 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a

[PATCH 01/10] Drivers: hv: util: Increase the timeout for util services

2015-10-07 Thread K. Y. Srinivasan
Util services such as KVP and FCOPY need assistance from daemon's running in user space. Increase the timeout so we don't prematurely terminate the transaction in the kernel. Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_fcopy.c |3 ++- drivers/hv/hv_kvp.c |3 ++- drivers/h

[PATCH 02/10] Drivers: hv: utils: run polling callback always in interrupt context

2015-10-07 Thread K. Y. Srinivasan
From: Olaf Hering All channel interrupts are bound to specific VCPUs in the guest at the point channel is created. While currently, we invoke the polling function on the correct CPU (the CPU to which the channel is bound to) in some cases we may run the polling function in a non-interrupt context

[PATCH 06/10] Drivers: hv: utils: use memdup_user in hvt_op_write

2015-10-07 Thread K. Y. Srinivasan
From: Olaf Hering Use memdup_user to handle OOM. Fixes: 14b50f80c32d ('Drivers: hv: util: introduce hv_utils_transport abstraction') Signed-off-by: Olaf Hering Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_utils_transport.c |9 - 1 files changed, 4 insertions(+), 5 deletions

[PATCH 04/10] tools: hv: remove repeated HV_FCOPY string

2015-10-07 Thread K. Y. Srinivasan
From: Olaf Hering HV_FCOPY is already used as identifier in syslog. Signed-off-by: Olaf Hering Signed-off-by: K. Y. Srinivasan --- tools/hv/hv_fcopy_daemon.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/hv/hv_fcopy_daemon.c b/tools/hv/hv_fcopy_daemon.c ind

[PATCH 05/10] Drivers: hv: util: catch allocation errors

2015-10-07 Thread K. Y. Srinivasan
From: Olaf Hering Catch allocation errors in hvutil_transport_send. Fixes: 14b50f80c32d ('Drivers: hv: util: introduce hv_utils_transport abstraction') Signed-off-by: Olaf Hering Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_utils_transport.c |9 ++--- 1 files changed, 6 inserti

[PATCH 07/10] drivers/hv: cleanup synic msrs if vmbus connect failed

2015-10-07 Thread K. Y. Srinivasan
From: Denis V. Lunev Before vmbus_connect() synic is setup per vcpu - this means hypervisor receives writes at synic msr's and probably allocate hypervisor resources per synic setup. If vmbus_connect() failed for some reason it's neccessary to cleanup synic setup by call hv_synic_cleanup() at ea

[PATCH 08/10] drivers:hv: Export a function that maps Linux CPU num onto Hyper-V proc num

2015-10-07 Thread K. Y. Srinivasan
From: Jake Oshins This patch exposes the mapping between Linux CPU number and Hyper-V virtual processor number. This is necessary because the hypervisor needs to know which virtual processors to target when making a mapping in the Interrupt Redirection Table in the I/O MMU. Signed-off-by: Jake

[PATCH 09/10] drivers:hv: Export the API to invoke a hypercall on Hyper-V

2015-10-07 Thread K. Y. Srinivasan
From: Jake Oshins This patch exposes the function that hv_vmbus.ko uses to make hypercalls. This is necessary for retargeting an interrupt when it is given a new affinity. Since we are exporting this API, rename the API as it will be visible outside the hv.c file. Signed-off-by: Jake Oshins S

[PATCH 00/10] Drivers: hv: Miscellaneous fixes.

2015-10-07 Thread K. Y. Srinivasan
Miscellaneous fixes. Denis V. Lunev (1): drivers/hv: cleanup synic msrs if vmbus connect failed Jake Oshins (3): drivers:hv: Export a function that maps Linux CPU num onto Hyper-V proc num drivers:hv: Export the API to invoke a hypercall on Hyper-V drivers:hv: Define the channel type

[PATCH 2/3] staging: comedi: dt9812: rename F020_MASK_DACxCN_DACxEN

2015-10-07 Thread H Hartley Sweeten
Rename this CamelCase define. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt9812.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/dt9812.c b/drivers/staging/comedi/drivers/d

[PATCH 3/3] staging: comedi: dt9812: reword comments to fix checkpatch.pl issues

2015-10-07 Thread H Hartley Sweeten
Reword the block comments to fix the checkpatch.pl issues about: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line For consistency, also reword the previous comments. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Har

[PATCH 1/3] staging: comedi: dt9812: tidy up multi-line comment

2015-10-07 Thread H Hartley Sweeten
Reformat the multi-line comment in the kernel CodingStyle. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/dt9812.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/comedi/driv

[PATCH 0/3] staging: comedi: dt9812: minor cleanup

2015-10-07 Thread H Hartley Sweeten
Fix the checkpatch.pl issues in this driver. H Hartley Sweeten (3): staging: comedi: dt9812: tidy up multi-line comment staging: comedi: dt9812: rename F020_MASK_DACxCN_DACxEN staging: comedi: dt9812: reword comments to fix checkpatch.pl issues drivers/staging/comedi/drivers/dt9812.c | 38

[PATCH] staging: comedi: fl512: usleep_range is preferred over udelay

2015-10-07 Thread H Hartley Sweeten
Fix checkpatch.pl issue: CHECK: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.txt Replace the udelay() with usleep_range() with a reasonable upper limit. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/dr

[PATCH] staging: comedi: gsc_hpdi: use preferred kernel types

2015-10-07 Thread H Hartley Sweeten
Fix the checkpatch.pl issues about: CHECK: Prefer kernel type 'u32' over 'uint32_t' CHECK: Prefer kernel type 'u8' over 'uint8_t' Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/gsc_hpdi.c | 28 ++-- 1 file chan

[PATCH 13/17] staging: comedi: icp_multi: fix clock comment CodingStyle

2015-10-07 Thread H Hartley Sweeten
Reword the block comment to fix a CodingStyle issue. For aesthetics, also rename the icp_multi_ao_eoc() function. It's testing for the analog output to be 'ready' not for an 'end-of-conversion'. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/

[PATCH 16/17] staging: comedi: icp_multi: remove board reset during (*detach)

2015-10-07 Thread H Hartley Sweeten
The icp_multi_reset() function currnelt does the following during the (*auto_attach) and (*detach) of this driver: 1) disables all interrupts and clears any pending requests 2) resets the analog output channels to 0V (0..5V range) 3) sets all digital output channels to 0 Interrupts are not

[PATCH 14/17] staging: comedi: icp_multi: remove counter subdevice

2015-10-07 Thread H Hartley Sweeten
The support functions for this subdevice are not complete. Since the counter is the last subdevice just remove it and it's allocation. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 29 ++--- 1 fil

[PATCH 17/17] staging: comedi: icp_multi: update the MODULE_DESCRIPTION

2015-10-07 Thread H Hartley Sweeten
Change the MODULE_DESCRIPTION to something more useful than the generic "Comedi low-level driver". Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s

[PATCH 15/17] staging: comedi: icp_multi: tidy up subdevice init

2015-10-07 Thread H Hartley Sweeten
For aesthetics, add some whitespace to the subdevice initialization. Remove the unnecessary initialization of the 'len_chanlist' for each subdevice. That member is only used by subdevices that support async commands. Also, remove the initialzation of the dev->read_subdev. That member is also only

[PATCH 10/17] staging: comedi: icp_multi: remove useless interrupt disable code

2015-10-07 Thread H Hartley Sweeten
This driver does not use interrupts and all the interrupt source are disabled by icp_multi_reset() when the board is first attached. Remove the unnecessary, and useless, disable and clearing of interrupts in the analog input and output (*insn_read) and (*insn_write) functions. This also removes t

[PATCH 11/17] staging: comedi: icp_multi: remove check_channel_list()

2015-10-07 Thread H Hartley Sweeten
This function would only be needed in the driver supported async commands. It's #if 0'ed out anyway so just remove it. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 33 -- 1 file changed, 33 d

[PATCH 12/17] staging: comedi: icp_multi: remove private data member 'DacCmdStatus'

2015-10-07 Thread H Hartley Sweeten
This private data member isn't really needed. The two functions that use it, icp_multi_ao_insn_write() and icp_multi_reset(), mask any previous value before setting the bits. The masking just clears the variable. Refactor the code to not use the private data member. This also removes the need for

[PATCH 08/17] staging: comedi: icp_multi: absorb setup_channel_list()

2015-10-07 Thread H Hartley Sweeten
This function is only called in one place and the 'n_chan' parameter is always 1. This makes the for() loop a bit silly. Absorb the function into the caller and simplify the code. This also removes the need for the private data member 'AdcCmdStatus'. Signed-off-by: H Hartley Sweeten Cc: Ian Abbo

[PATCH 09/17] staging: comedi: icp_multi: remove interrupt support

2015-10-07 Thread H Hartley Sweeten
The interrupt handler in this driver doesn't do anything other than read the interrupt status register. Since I can't locate a datasheet for the board, remove the useless interrupt support code. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/d

[PATCH 05/17] staging: comedi: icp_multi: rename static variable 'range_analog'

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename this static variable so it has namespace associated with the driver. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/

[PATCH 06/17] staging: comedi: icp_multi: remove unnecessary block comment

2015-10-07 Thread H Hartley Sweeten
This comment doesn't add any value to the code. Remove it. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/comedi/drivers/icp_multi.c b/drivers/stagin

[PATCH 07/17] staging: comedi: icp_multi: remove unused members from private data

2015-10-07 Thread H Hartley Sweeten
These members are either not used at all or they are set but never used. Just remove them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/comedi/dri

[PATCH 04/17] staging: comedi: icp_multi: tidy up the interrupt enable/status register bits

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename these bit defines so they are associated with the registers and use the BIT macro to define them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 60 ++ 1 file changed, 28

[PATCH 02/17] staging: comedi: icp_multi: tidy up the ADC command/status register bits

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename these bit defines so they are associated with the register and use the BIT macro to define them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 22 ++ 1 file changed, 10 insertio

[PATCH 03/17] staging: comedi: icp_multi: tidy up the DAC command/status register bits

2015-10-07 Thread H Hartley Sweeten
For aesthetics, rename these bit defines so they are associated with the register and use the BIT macro to define them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 22 ++ 1 file changed, 10 insertio

[PATCH 01/17] staging: comedi: icp_multi: tidy up multi-line comments

2015-10-07 Thread H Hartley Sweeten
Reformat the multi-line comments in the kernel CodingStyle. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/icp_multi.c | 81 +++--- 1 file changed, 41 insertions(+), 40 deletions(-) diff --git a/drivers/staging

[PATCH 00/17] staging: comedi: icp_multi: cleanup driver

2015-10-07 Thread H Hartley Sweeten
Fix the checkpatch.pl issues and tidy up this driver a bit. H Hartley Sweeten (17): staging: comedi: icp_multi: tidy up multi-line comments staging: comedi: icp_multi: tidy up the ADC command/status register bits staging: comedi: icp_multi: tidy up the DAC command/status register bits stag

Re: [PATCH] rtl8188eu: unused macros read_next_pair removed, ARRAY_SIZE instead of sizeof/sizeof inserted

2015-10-07 Thread Mike Rapoport
On Wed, Oct 07, 2015 at 11:01:00PM +0700, Ivan Safonov wrote: > Some rtl8188eu driver cleanups: > Unused macros read_next_pair removed > ARRAY_SIZE instead of sizeof/sizeof inserted > Some checkpath fixes Please split into three different patches, each one doing one logical change. > > Sig

[PATCH 2/2] staging: fbtft: add support for C-Berry28 display

2015-10-07 Thread Dennis Menschel
This patch adds support for the C-Berry28, a small 2.8" color display with a resolution of 320x240 pixels from admatec for the Raspberry Pi. It uses the Sitronix ST7789V display controller along with a custom init sequence and custom gamma curves. Signed-off-by: Dennis Menschel --- drivers/stagi

[PATCH 1/2] staging: fbtft: add support for ST7789V display controller

2015-10-07 Thread Dennis Menschel
This patch adds support for the Sitronix ST7789V display controller. The controller is intended for small color displays with a resolution of up to 320x240 pixels. Signed-off-by: Dennis Menschel --- drivers/staging/fbtft/Kconfig | 10 ++ drivers/staging/fbtft/Makefile | 1 + drivers/

[PATCH 0/2] staging: fbtft: add support for ST7789V and C-Berry28

2015-10-07 Thread Dennis Menschel
This set of patches extends the fbtft driver by the following two things, - support for the ST7789V display controller from Sitronix and - support for a concrete display which uses this controller, namely the C-Berry28, a 2.8" color display from admatec for the Raspberry Pi. Dennis Menschel (2)

[PATCH 11/11] Staging: rtl8712: rtl8712_hal.h: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_hal.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl8712_hal.h | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff

[PATCH 07/11] Staging: rtl8712: recv_linux.c: Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the recv_linux.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/recv_linux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/driver

[PATCH 10/11] Staging: rtl8712: rtl8712_gp_bitdef.h: Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_gp_bitdef.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl8712_gp_bitdef.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

[PATCH 08/11] Staging: rtl8712: rtl8712_cmd.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_cmd.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl8712_cmd.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff -

[PATCH 09/11] Staging: rtl8712: rtl8712_cmdctrl_bitdef.h: Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the rtl8712_cmdctrl_bitdef.h file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl8712_cmdctrl_bitdef.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletio

[PATCH 05/11] Staging: rtl8712: mlme_linux.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the mlme_linux.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/mlme_linux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/driver

[PATCH 06/11] Staging: rtl8712: os_intfs.c : Coding style warning fix for block comment

2015-10-07 Thread Punit Vara
This is patch to the os_intfs.c file that fixes up following warning reported by checkpatch.pl : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/os_intfs.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/st

[PATCH 04/11] Staging: rtl8712: ieee80211.h: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the ieee80211.h file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/ieee80211.h | 48 ---

[PATCH 03/11] Staging: rtl8712: ieee80211.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the ieee80211.c file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/ieee80211.c | 7 --- 1 file changed, 4 inser

[PATCH 02/11] Staging: rtl8712: hal_init.c: Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the hal_init.c file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/hal_init.c | 12 1 file changed, 8 i

[PATCH 01/11] Staging: rtl8712: drv_types.h:Coding style warnings fix for block comments

2015-10-07 Thread Punit Vara
This is patch to the drv_types.h file that fixes up following warning reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/drv_types.h | 11 ++- 1 file changed, 6

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

2015-10-07 Thread Andrew
On 2015-10-07 21:36, Rob Herring wrote: On Wed, Oct 7, 2015 at 5:36 AM, Andrew wrote: On 2015-10-07 02:01, Laura Abbott wrote: On 10/6/15 3:35 PM, Rob Herring wrote: On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott wrote: From: Laura Abbott This adds a base set of devicetree bindings fo

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

2015-10-07 Thread Rob Herring
On Wed, Oct 7, 2015 at 5:36 AM, Andrew wrote: > On 2015-10-07 02:01, Laura Abbott wrote: >> >> On 10/6/15 3:35 PM, Rob Herring wrote: >>> >>> On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott >>> wrote: From: Laura Abbott This adds a base set of devicetree bindings for the Io

[PATCH 4/4] staging: comedi: ii_pci20kc: update the MODULE_DESCRIPTION

2015-10-07 Thread H Hartley Sweeten
Change the MODULE_DESCRIPTION to something more useful than the generic "Comedi low-level driver". Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/ii_pci20kc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/

[PATCH 3/4] staging: comedi: ii_pci20kc: use comedi_offset_munge()

2015-10-07 Thread H Hartley Sweeten
For aesthetics, use the helper function to handle the 2's complement to offset binary munging of the analog input data. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/ii_pci20kc.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletio

[PATCH 0/4] staging: comedi: ii_pci20kc: cleanup driver

2015-10-07 Thread H Hartley Sweeten
Fix a minor checkpatch.pl issue and cleanup this driver a bit. H Hartley Sweeten (4): staging: comedi: ii_pci20kc: prefer using the BIT macro staging: comedi: ii_pci20kc: only bipolar ao ranges are 2's complement staging: comedi: ii_pci20kc: use comedi_offset_munge() staging: comedi: ii_pc

[PATCH 2/4] staging: comedi: ii_pci20kc: only bipolar ao ranges are 2's complement

2015-10-07 Thread H Hartley Sweeten
According the the PCI-2006M user's manual, bipolar ranges use 2's complement coding and unipolar ranges are straight binary. Fix ii20k_ao_insn_write() to use the correct coding based on the range. For aesthetics, use the comedi_offset_munge() helper to handle the munging of the data. Signed-off-b

[PATCH 1/4] staging: comedi: ii_pci20kc: prefer using the BIT macro

2015-10-07 Thread H Hartley Sweeten
As suggested by checkpatch.pl, use the BIT macro to define the register bits. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/ii_pci20kc.c | 70 ++--- 1 file changed, 35 insertions(+), 35 deletions(-) diff --git

Re: [PATCH v11 0/4] FPGA Manager Framework

2015-10-07 Thread Moritz Fischer
Hi Greg, Alan On Wed, Oct 7, 2015 at 6:09 PM, Greg KH wrote: > On Wed, Oct 07, 2015 at 04:36:25PM +0100, at...@opensource.altera.com wrote: >> From: Alan Tull >> >> Hi Greg, >> >> I'm resending v11 >> >> The changes requested for v12 are minor. We can fix them upstream in >> some small patches.

Re: [PATCH 1/1] drivers/hv: cleanup synic msrs if vmbus connect failed

2015-10-07 Thread Vitaly Kuznetsov
"Denis V. Lunev" writes: > From: Andrey Smetanin > > Before vmbus_connect() synic is setup per vcpu - this means > hypervisor receives writes at synic msr's and probably allocate > hypervisor resources per synic setup. > > If vmbus_connect() failed for some reason it's neccessary to cleanup > sy

Re: [PATCH v11 0/4] FPGA Manager Framework

2015-10-07 Thread Greg KH
On Wed, Oct 07, 2015 at 04:36:25PM +0100, at...@opensource.altera.com wrote: > From: Alan Tull > > Hi Greg, > > I'm resending v11 > > The changes requested for v12 are minor. We can fix them upstream in > some small patches. Thanks for everybody's support and help in reviewing > these. Looks

Re: [PATCH 00/13] staging: comedi: mf6x4: cleanup driver

2015-10-07 Thread Ian Abbott
On 07/10/15 17:37, Hartley Sweeten wrote: On Wednesday, October 07, 2015 2:20 AM, Ian Abbott wrote: On 06/10/15 19:11, H Hartley Sweeten wrote: Fix some minor checkpatch.pl issues and tidy up this driver a bit. H Hartley Sweeten (13): staging: comedi: mf6x4: prefer using the BIT macro

RE: [PATCH 00/13] staging: comedi: mf6x4: cleanup driver

2015-10-07 Thread Hartley Sweeten
On Wednesday, October 07, 2015 2:20 AM, Ian Abbott wrote: > On 06/10/15 19:11, H Hartley Sweeten wrote: >> Fix some minor checkpatch.pl issues and tidy up this driver a bit. >> >> H Hartley Sweeten (13): >>staging: comedi: mf6x4: prefer using the BIT macro >>staging: comedi: mf6x4: remove u

[PATCH] rtl8188eu: unused macros read_next_pair removed, ARRAY_SIZE instead of sizeof/sizeof inserted

2015-10-07 Thread Ivan Safonov
Some rtl8188eu driver cleanups: Unused macros read_next_pair removed ARRAY_SIZE instead of sizeof/sizeof inserted Some checkpath fixes Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 30 +++--- 1 file changed, 11 insertions(+), 19 deletions(

  1   2   >