[PATCH 1/1] Staging: fbtft: fb_ssd1331: replaced C99 comments with C89

2015-02-27 Thread Tolga Ceylan
Replaced C99 '//' comments with C89 '/**/' Signed-off-by: Tolga Ceylan --- drivers/staging/fbtft/fb_ssd1331.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/fbtft/fb_ssd1331.c b/drivers/staging/fbtft/fb_ssd1331.c index da7464

Re: [PATCH 1/1] Staging: fbtft: fb_ssd1331: replaced C99 comments with C89

2015-02-27 Thread Tolga Ceylan
On Fri, Feb 27, 2015 at 9:37 PM, Tolga Ceylan wrote: > Replaced C99 '//' comments with C89 '/**/' > > Signed-off-by: Tolga Ceylan > --- > drivers/staging/fbtft/fb_ssd1331.c | 30 +++--- > 1 file changed, 15 insertions(+), 15 deletions(-) > > diff --git a/drivers/staging/f

[PATCH 1/1] Staging: fbtft: fb_ssd1331: replaced C99 comments with C89

2015-02-27 Thread Tolga Ceylan
Replaced C99 '//' comments with C89 '/**/' Signed-off-by: Tolga Ceylan --- drivers/staging/fbtft/fb_ssd1331.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/fbtft/fb_ssd1331.c b/drivers/staging/fbtft/fb_ssd1331.c index da7464

[PATCH 1/1] Staging: fbtft: fbtft-core: removed space before ','

2015-02-27 Thread Tolga Ceylan
Removed space before ',' Signed-off-by: Tolga Ceylan --- drivers/staging/fbtft/fbtft-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c index 37dcf7e..ac4287f 100644 --- a/drivers/staging/fbtft/fbtft

[PATCH 1/1] Staging: rtl8192u: r8192U_wx: removed unnecessary comments

2015-02-27 Thread Tolga Ceylan
1) Removed commented out include header statement 2) Removed unnecessary author/date comment Signed-off-by: Tolga Ceylan --- drivers/staging/rtl8192u/r8192U_wx.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.h b/drivers/staging/rtl8192u

[PATCH 1/3] scsi: storvsc: Increase the ring buffer size

2015-02-27 Thread K. Y. Srinivasan
Increase the default ring buffer size as this can significantly improve performance especially on high latency storage back-ends. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers

[PATCH 2/3] scsi: storvsc: Size the queue depth based on the ringbuffer size

2015-02-27 Thread K. Y. Srinivasan
Size the queue depth based on the ringbuffer size. Also accomodate for the fact that we could have multiple channels (ringbuffers) per adaptor. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_drv.c | 27 --- 1 files changed, 16 insertions(

[PATCH 3/3] scsi: storvsc: Always send on the selected outgoing channel

2015-02-27 Thread K. Y. Srinivasan
The current code always sent packets without data on the primary channel. Properly distribute sending of packets with no data amongst all available channels. I would like to thank Long Li for noticing this problem. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_dr

[PATCH 0/3] scsi: storvsc: Some enhancements

2015-02-27 Thread K. Y. Srinivasan
While testing on some high latency storage backends, to get the advertised IOPS, we have to increase the size of the ringbuffer. Address this issue. K. Y. Srinivasan (3): scsi: storvsc: Increase the ring buffer size Scsi: storvsc: Size the queue depth based on the ringbuffer size scsi: storv

Re:HELLO ARE YOU TODAY ????

2015-02-27 Thread Ms Jamal Jurdi
Good day, I am Ms Jamal Jurdi Lebanese origin., I serve under the United Nations department of safety and security (UNDSS) in Syria, I and my second in command came across some money that was parked in a trunk box and the money is worth over $ 13 million, in one of President Bashar al-Assa

Re: [PATCH RFC 0/3] Drivers: hv: utils: re-implement the kernel/userspace communication layer

2015-02-27 Thread Radim Krčmář
2015-02-27 17:14+0100, Vitaly Kuznetsov: > This series converts kvp/vss daemons to use misc char devices instead of > netlink for userspace/kernel communication and then updates fcopy to be > consistent with kvp/vss. > > Userspace/kernel communication via netlink has a number of issues: > - It is

[PATCH 4/5] staging: rtl8723au: The RF on an 8723au is always a 6052

2015-02-27 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c| 34 +- drivers/staging/rtl8723au/hal/usb_halinit.c| 12 drivers/staging/rtl8723au/include/Hal8723APhyCfg.h | 9 -- drivers/staging/rtl8723au/include/

[PATCH 2/5] staging: rtl8723au: Be more consistent in checking for 2 TX paths

2015-02-27 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/HalDMOutSrc8723A_CE.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/HalDMOutSrc8723A_CE.c b/drivers/staging/rtl8723au/hal/HalDMOutSrc8723A_CE

[PATCH 5/5] staging: rtl8723au: MAX_AGGR_NUM is not used

2015-02-27 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c| 1 - drivers/staging/rtl8723au/include/Hal8723APhyCfg.h | 3 --- 2 files changed, 4 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c b/drivers/staging/rtl8723au/hal

[PATCH 0/5] staging: rtl8723au: Another set of cleanups

2015-02-27 Thread Jes . Sorensen
From: Jes Sorensen Hi, Here is another 5 cleanups for the rtl8723au driver, to go on top of the set I posted yesterday. Cheers, Jes Jes Sorensen (5): staging: rtl8723au: No need for two copies of rf_type staging: rtl8723au: Be more consistent in checking for 2 TX paths staging: rtl8723a

[PATCH 1/5] staging: rtl8723au: No need for two copies of rf_type

2015-02-27 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 12 +++- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 7 --- drivers/staging/rtl8723au/include/odm.h | 16 3 files changed, 3 insertions(+), 32 deletions(-) diff

[PATCH 3/5] staging: rtl8723au: Remove rf type from struct hal_version

2015-02-27 Thread Jes . Sorensen
From: Jes Sorensen This gets rid of yet another duplicated copy of the RF type Signed-off-by: Jes Sorensen --- .../staging/rtl8723au/hal/HalDMOutSrc8723A_CE.c| 19 +- drivers/staging/rtl8723au/hal/hal_com.c| 40 -- drivers/staging/rtl8723au/hal/rtl87

Re: [PATCH RFC 1/3] Drivers: hv: kvp: convert userspace/kernel communication to using char device

2015-02-27 Thread Radim Krčmář
2015-02-27 17:14+0100, Vitaly Kuznetsov: > Re-implement the communication using misc char device. Use ioctl to do > kernel/userspace version negotiation (doesn't make much sense at this moment > as we're breaking backwards compatibility but can be used in future). The main question is whether we w

Re: [PATCH v5 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-27 Thread Pavel Machek
On Fri 2015-02-27 09:20:01, Krzysztof Kozlowski wrote: > Use power_supply_put() to decrement the power supply's device reference > counter. > > Signed-off-by: Krzysztof Kozlowski > Reviewed-by: Bartlomiej Zolnierkiewicz > Reviewed-by: Sebastian Reichel > Acked-by: Robert Jarzmik Acked-by: Pav

[PATCH] Staging: vt6655: replace C99 style comments

2015-02-27 Thread Matteo Semenzato
From: Matteo Semenzato This patch fixes the following error: ERROR: do not use C99 // comments Signed-off-by: Matteo Semenzato --- drivers/staging/vt6655/key.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6655/key.h b/drivers/staging/vt6655/key.h index

[PATCH] Staging: vt6655: remove C99 style comments.

2015-02-27 Thread Matteo Semenzato
From: Matteo Semenzato This patch fixes the following error: ERROR: do not use C99 // comments Signed-off-by: Matteo Semenzato --- drivers/staging/vt6655/device_main.c | 35 ++- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/drivers/staging/vt66

[PATCH RESEND 11/15] hv: channel: match var type to return type of wait_for_completion

2015-02-27 Thread K. Y. Srinivasan
From: Nicholas Mc Guire return type of wait_for_completion_timeout is unsigned long not int, this patch changes the type of t from int to unsigned long. Signed-off-by: Nicholas Mc Guire Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel.c |3 ++- 1 files changed, 2 insertions(+), 1 de

[PATCH RESEND 12/15] hv: channel_mgmt: match var type to return type of wait_for_completion

2015-02-27 Thread K. Y. Srinivasan
From: Nicholas Mc Guire return type of wait_for_completion_timeout is unsigned long not int, this patch changes the type of t from int to unsigned long. Signed-off-by: Nicholas Mc Guire Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c |3 ++- 1 files changed, 2 insertions(+),

[PATCH RESEND 10/15] hv: vmbus_open(): reset the channel state on ENOMEM

2015-02-27 Thread K. Y. Srinivasan
From: Dexuan Cui Without this patch, the state is put to CHANNEL_OPENING_STATE, and when the driver is loaded next time, vmbus_open() will fail immediately due to newchannel->state != CHANNEL_OPEN_STATE. CC: "K. Y. Srinivasan" Signed-off-by: Dexuan Cui Reviewed-by: Jason Wang Signed-off-by: K

[PATCH RESEND 04/15] Drivers: hv: vmbus: teardown hv_vmbus_con workqueue and vmbus_connection pages on shutdown

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov We need to destroy hv_vmbus_con on module shutdown, otherwise the following crash is sometimes observed: [ 76.569845] hv_vmbus: Hyper-V Host Build:9600-6.3-17-0.17039; Vmbus version:3.0 [ 82.598859] BUG: unable to handle kernel paging request at a0003480 [ 8

[PATCH RESEND 03/15] Drivers: hv: vmbus: avoid double kfree for device_obj

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov On driver shutdown device_obj is being freed twice: 1) In vmbus_free_channels() 2) vmbus_device_release() (which is being triggered by device_unregister() in vmbus_device_unregister(). This double kfree leads to the following sporadic crash on driver unload: [ 23.4698

[PATCH RESEND 09/15] hv: vmbus_post_msg: retry the hypercall on some transient errors

2015-02-27 Thread K. Y. Srinivasan
From: Dexuan Cui I got HV_STATUS_INVALID_CONNECTION_ID on Hyper-V 2008 R2 when keeping running "rmmod hv_netvsc; modprobe hv_netvsc; rmmod hv_utils; modprobe hv_utils" in a Linux guest. Looks the host has some kind of throttling mechanism if some kinds of hypercalls are sent too frequently. Witho

[PATCH RESEND 08/15] hv: hv_util: move vmbus_open() to a later place

2015-02-27 Thread K. Y. Srinivasan
From: Dexuan Cui Before the line vmbus_open() returns, srv->util_cb can be already running and the variables, like util_fw_version, are needed by the srv->util_cb. So we have to make sure the variables are initialized before the vmbus_open(). CC: "K. Y. Srinivasan" Reviewed-by: Vitaly Kuznetso

[PATCH RESEND 01/15] Drivers: hv: vmbus: prevent cpu offlining on newer hypervisors

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov When an SMP Hyper-V guest is running on top of 2012R2 Server and secondary cpus are sent offline (with echo 0 > /sys/devices/system/cpu/cpu$cpu/online) the system freeze is observed. This happens due to the fact that on newer hypervisors (Win8, WS2012R2, ...) vmbus channel

[PATCH RESEND 02/15] Drivers: hv: vmbus: rename channel work queues

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov All channel work queues are named 'hv_vmbus_ctl', this makes them indistinguishable in ps output and makes it hard to link to the corresponding vmbus device. Rename them to hv_vmbus_ctl/N and make vmbus device names match, e.g. now vmbus_1 device is served by hv_vmbus_ctl/1

[PATCH RESEND 05/15] drivers: hv: vmbus: Teardown synthetic interrupt controllers on module unload

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov SynIC has to be switched off when we unload the module, otherwise registered memory pages can get corrupted after (as Hyper-V host still writes there) and we see the following crashes for random processes: [ 89.116774] BUG: Bad page map in process sh pte:4989c716 pmd:36

[PATCH RESEND 14/15] Drivers: hv: vmbus: Fix a bug in the error path in vmbus_open()

2015-02-27 Thread K. Y. Srinivasan
Correctly rollback state if the failure occurs after we have handed over the ownership of the buffer to the host. Signed-off-by: K. Y. Srinivasan Cc: sta...@vger.kernel.org --- drivers/hv/channel.c |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/hv/channel.

[PATCH RESEND 13/15] hv: hv_balloon: match var type to return type of wait_for_completion

2015-02-27 Thread K. Y. Srinivasan
From: Nicholas Mc Guire return type of wait_for_completion_timeout is unsigned long not int, this patch changes the type of t from int to unsigned long. Signed-off-by: Nicholas Mc Guire Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_balloon.c |3 ++- 1 files changed, 2 insertions(+), 1

[PATCH RESEND 07/15] Drivers: hv: vmbus: Teardown clockevent devices on module unload

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov Newly introduced clockevent devices made it impossible to unload hv_vmbus module as clockevents_config_and_register() takes additional reverence to the module. To make it possible again we do the following: - avoid setting dev->owner for clockevent devices; - implement hv_s

[PATCH RESEND 06/15] clockevents: export clockevents_unbind_device instead of clockevents_unbind

2015-02-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov It looks like clockevents_unbind is being exported by mistake as: - it is static; - it is not listed in include/linux/clockchips.h; - EXPORT_SYMBOL_GPL(clockevents_unbind) follows clockevents_unbind_device() implementation. I think clockevents_unbind_device should be exp

[PATCH RESEND 15/15] Drivers: hv: vmbus: Add support for the NetworkDirect GUID

2015-02-27 Thread K. Y. Srinivasan
NetworkDirect is a service that supports guest RDMA. Define the GUID for this service. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c |2 ++ include/linux/hyperv.h| 10 ++ 2 files changed, 12 insertions(+), 0 deletions(-) diff --git a/drivers/hv/channel_mgmt.c b

[PATCH RESEND 00/15] Drivers: hv: vmbus: Miscellaneous fixes/enhancements

2015-02-27 Thread K. Y. Srinivasan
Windows hosts starting with Ws2012 R2 permit re-establishing the vmbus connection from the guest. This patch-set includes patches from Vitaly to cleanup the VMBUS unload path so we can potentially reload the driver. Also included here are some bug fixes from Dexuan and me. Dexuan Cui (3): hv: hv

Re: [PATCH v2] android: binder: fix binder mmap failures

2015-02-27 Thread John Stultz
On Fri, Feb 27, 2015 at 9:44 AM, Andrey Ryabinin wrote: > binder_update_page_range() initializes only addr and size > fields in 'struct vm_struct tmp_area;' and passes it to > map_vm_area(). > > Before 71394fe50146 ("mm: vmalloc: add flag preventing guard hole allocation") > this was because map_v

Re: [PATCH] android: binder: fix binder mmap failures

2015-02-27 Thread John Stultz
On Fri, Feb 27, 2015 at 8:30 AM, Andrey Ryabinin wrote: > binder_update_page_range() initializes only addr and size > fields in 'struct vm_struct tmp_area;' and passes it to > map_vm_area(). > > Before 71394fe50146 ("mm: vmalloc: add flag preventing guard hole allocation") > this was because map_v

[PATCH v2] android: binder: fix binder mmap failures

2015-02-27 Thread Andrey Ryabinin
binder_update_page_range() initializes only addr and size fields in 'struct vm_struct tmp_area;' and passes it to map_vm_area(). Before 71394fe50146 ("mm: vmalloc: add flag preventing guard hole allocation") this was because map_vm_area() didn't use any other fields in vm_struct except addr and si

Re: [PATCH] android: binder: fix binder mmap failures

2015-02-27 Thread Andrey Ryabinin
On 02/27/2015 08:26 PM, John Stultz wrote: > On Fri, Feb 27, 2015 at 8:30 AM, Andrey Ryabinin > wrote: >> binder_update_page_range() initializes only addr and size >> fields in 'struct vm_struct tmp_area;' and passes it to >> map_vm_area(). >> >> Before 71394fe50146 ("mm: vmalloc: add flag preven

Re: [PATCH] Staging: rtl8188eu: fix coding style errors

2015-02-27 Thread Larry Finger
On 02/27/2015 10:44 AM, Matteo Semenzato wrote: From: Matteo Semenzato This patch fixes the following errors: ERROR: space prohibited before that ',' (ctx:WxW) ERROR: space prohibited before that ',' (ctx:WxW) ERROR: switch and case should be at the same indent Signed-off-by: Matteo Semenzato

Re: [PATCH] Staging: rtl8188eu: fix style errors

2015-02-27 Thread Greg KH
On Fri, Feb 27, 2015 at 05:49:15PM +0100, Matteo Semenzato wrote: > From: Matteo Semenzato > > This patch fixes the following errors: > ERROR: code indent should use tabs where possible > ERROR: space prohibited before that Again, multiple things means multiple patches. _

Re: [PATCH] Staging: rtl8188eu: fix coding style errors

2015-02-27 Thread Greg KH
On Fri, Feb 27, 2015 at 05:44:09PM +0100, Matteo Semenzato wrote: > From: Matteo Semenzato > > This patch fixes the following errors: > ERROR: space prohibited before that ',' (ctx:WxW) > ERROR: space prohibited before that ',' (ctx:WxW) > ERROR: switch and case should be at the same indent That

[PATCH] Staging: rtl8188eu: fix style errors

2015-02-27 Thread Matteo Semenzato
From: Matteo Semenzato This patch fixes the following errors: ERROR: code indent should use tabs where possible ERROR: space prohibited before that Signed-off-by: Matteo Semenzato --- drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 4 ++-- drivers/staging/rtl8188eu/hal/odm.c

[PATCH] Staging: rtl8188eu: fix coding style errors

2015-02-27 Thread Matteo Semenzato
From: Matteo Semenzato This patch fixes the following errors: ERROR: space prohibited before that ',' (ctx:WxW) ERROR: space prohibited before that ',' (ctx:WxW) ERROR: switch and case should be at the same indent Signed-off-by: Matteo Semenzato --- drivers/staging/rtl8188eu/core/rtw_efuse.c

[PATCH] android: binder: fix binder mmap failures

2015-02-27 Thread Andrey Ryabinin
binder_update_page_range() initializes only addr and size fields in 'struct vm_struct tmp_area;' and passes it to map_vm_area(). Before 71394fe50146 ("mm: vmalloc: add flag preventing guard hole allocation") this was because map_vm_area() didn't use any other fields in vm_struct except addr and si

[PATCH RFC 2/3] Drivers: hv: vss: convert userspace/kernel communication to using char device

2015-02-27 Thread Vitaly Kuznetsov
Userspace/kernel communication via netlink has a number of issues: - It is hard for userspace to figure out if the kernel part was loaded or not and this fact can change as there is a way to enable/disable the service from host side. Racy daemon startup is also a problem. - When the userspace d

[PATCH RFC 1/3] Drivers: hv: kvp: convert userspace/kernel communication to using char device

2015-02-27 Thread Vitaly Kuznetsov
Userspace/kernel communication via netlink has a number of issues: - It is hard for userspace to figure out if the kernel part was loaded or not and this fact can change as there is a way to enable/disable the service from host side. Racy daemon startup is also a problem. - When the userspace d

[PATCH RFC 0/3] Drivers: hv: utils: re-implement the kernel/userspace communication layer

2015-02-27 Thread Vitaly Kuznetsov
This series converts kvp/vss daemons to use misc char devices instead of netlink for userspace/kernel communication and then updates fcopy to be consistent with kvp/vss. Userspace/kernel communication via netlink has a number of issues: - It is hard for userspace to figure out if the kernel part w

[PATCH RFC 3/3] Drivers: hv: fcopy: make it consistent with vss/kvp

2015-02-27 Thread Vitaly Kuznetsov
Re-implement fcopy in a consistent with "Drivers: hv: vss/kvp: convert userspace/kernel communication to using char device" way. In particular: - Implement "state machine" for the driver instead of 3 separate syncronization variables ('fcopy_transaction.active', 'fcopy_transaction.read_sema', '

[PATCH] staging: comedi: adv_pci1710: fix AI INSN_READ for non-zero channel

2015-02-27 Thread Ian Abbott
Reading of analog input channels by the `INSN_READ` comedi instruction is broken for all except channel 0. `pci171x_ai_insn_read()` calls `pci171x_ai_read_sample()` with the wrong value for the third parameter. It is supposed to be the current index in a channel list (which is always of length 1 i

RE: [PATCH net-next] hyperv: Implement netvsc_get_channels() ethool op

2015-02-27 Thread Haiyang Zhang
> -Original Message- > From: Andrew Schwartzmeyer [mailto:and...@schwartzmeyer.com] > Sent: Thursday, February 26, 2015 7:27 PM > To: Haiyang Zhang > Cc: KY Srinivasan; de...@linuxdriverproject.org; net...@vger.kernel.org; > linux-ker...@vger.kernel.org; hall5...@vandals.uidaho.edu; > kei

[PATCH 3/3] staging: comedi: adl_pci6208: support old PLX device ID

2015-02-27 Thread Ian Abbott
ADLINK's MS Windows drivers for the PCI-6208/6216 boards include the following line in the DDInstall secion of the INF file: %String6208%=DriverInstall6208.NT,PCI\VEN_10B5&DEV_9050&SUBSYS_6208 That's for a PLX PCI 9050/9052 PCI interface chip with custom subvendor and subdevice ID. The "%Str

[PATCH 1/3] staging: comedi: adl_pci6208: combine PCI-6208 and PCI-6216 support

2015-02-27 Thread Ian Abbott
This module's PCI device table has separate PCI device IDs for PCI-6208 and PCI-6216, but in reality, both boards and their cPCI and PCIe variants seem to have the same PCI device ID: 0x6208. The PCI subdevice ID doesn't seem to help either. It shouldn't do any harm to claim 16 AO channels for al

[PATCH 2/3] staging: comedi: adl_pci6208: remove multiple board type support

2015-02-27 Thread Ian Abbott
This driver module now only supports a single board type, so remove the infrastructure for describing multiple board types. The comedi "auto_attach" handler, `pci6208_auto_attach()` doesn't need to set the comedi device's `board_name` or `board_ptr` members. The former is automatically pointed to

[PATCH 0/3] staging: comedi: adl_pci6208: combine board types

2015-02-27 Thread Ian Abbott
The driver uses the wrong PCI device ID for the PCI-6216. In reality, it has the same device ID as the PCI-6208 (and the subdevice ID's don't help either). This results in only the first 8 digital output channels of the PCI-6216 being useable. This series of patches treats the PCI-6208 and PCI-6

[PATCH v2] staging: dgnc: Warn on macro with flow control statements

2015-02-27 Thread Salah Triki
This fixes the following checkpatch.pl warnings: WARNING: Macros with flow control statements should be avoided. Signed-off-by: Salah Triki --- drivers/staging/dgnc/dgnc_sysfs.c | 38 -- 1 file changed, 12 insertions(+), 26 deletions(-) diff --git a/drivers/s

[PATCH v5 16/20] x86/olpc/xo15/sci: Use newly added power_supply_put API

2015-02-27 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo15-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deleti

[PATCH v5 19/20] mfd: ab8500: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Acked-by: Linus Walleij Acked-by: Lee Jones Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/mfd/ab8500-sysctrl.c | 2 +

[PATCH v5 14/20] power_supply: charger-manager: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/charger-manager.c | 70 +++--

[PATCH v5 13/20] power_supply: Increment power supply use counter when obtaining references

2015-02-27 Thread Krzysztof Kozlowski
Increment the power_supply.use_cnt usage counter on: - power_supply_get_by_phandle() - power_supply_get_by_name() and decrement it on power_supply_put() call. This helps tracking of valid usage of power supply instance by consumers. The usage counter itself also allows safe calling of power_supp

[PATCH v5 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Robert Jarzmik --- arch/arm/mach-pxa/raumfeld.c | 4 +++- 1 file changed, 3 insertions(+), 1 dele

[PATCH v5 17/20] power_supply: 88pm860x_charger: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/88pm860x_charger.c | 11 +-- 1 file changed, 9 insertions(

[PATCH v5 09/20] power_supply: bq2415x_charger: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: P

[PATCH v5 15/20] x86/olpc/xo1/sci: Use newly added power_supply_put API

2015-02-27 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo1-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletio

[PATCH v5 02/20] power_supply: Move run-time configuration to separate structure

2015-02-27 Thread Krzysztof Kozlowski
Add new structure 'power_supply_config' for holding run-time initialization data like of_node, supplies and private driver data. The power_supply_register() function is changed so all power supply drivers need updating. When registering the power supply this new 'power_supply_config' should be us

[PATCH v5 18/20] power_supply: bq2415x_charger: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter (increased by power_supply_get_by_name() or power_supply_get_by_phandle()). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drive

[PATCH v5 12/20] power_supply: Add power_supply_put for decrementing device reference counter

2015-02-27 Thread Krzysztof Kozlowski
The power_supply_get_by_phandle() and power_supply_get_by_name() use function class_find_device() for obtaining the reference to power supply. Each use of class_find_device() increases the power supply's device reference counter. However the reference counter was not decreased by users of this API

[PATCH v5 10/20] power_supply: charger-manager: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: P

[PATCH v5 07/20] mfd: ab8500: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: P

[PATCH v5 03/20] power_supply: Add API for safe access of power supply function attrs

2015-02-27 Thread Krzysztof Kozlowski
Add simple wrappers for accessing power supply's function attributes: - get_property -> power_supply_get_property - set_property -> power_supply_set_property - property_is_writeable -> power_supply_property_is_writeable - external_power_changed -> power_supply_external_power_changed This API a

[PATCH v5 01/20] power_supply: Add driver private data

2015-02-27 Thread Krzysztof Kozlowski
Allow drivers to store private data inside power_supply structure for later usage in power supply operations. Usage of driver private data is necessary to access driver's state container object from power supply calls (like get_property()) if struct 'power_supply' is a stored there as a pointer, f

[PATCH v5 08/20] power_supply: apm_power: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: P

[PATCH v5 00/20] power_supply: Allow safe usage of power supply

2015-02-27 Thread Krzysztof Kozlowski
Hi, The patchset fixes invalid memory accesses in certain race scenarios by moving ownership of struct power_supply to the core. All drivers are modified. TLDR for driver and subsystem maintainers = Two patches of patchset change power_supply_register() f

[PATCH v5 06/20] power_supply: ab8500: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: P

[PATCH v5 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property - set_property -> power_supply_set_property - property_is_writeable -

[PATCH v5 05/20] power_supply: 88pm860x_charger: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property - set_property -> power_supply_set_property Signed-off-by: Krzysztof