[PATCH 4/8] rtlwifi: rtl8192cu: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8192cu/def.h | 4

[PATCH 6/8] rtlwifi: rtl8188ee: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8188ee/dm.h | 4

[PATCH 7/8] rtlwifi: rtl8192c: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8192c/dm_common.c | 4

[PATCH 8/8] rtlwifi: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/pci.c | 4 drivers/net/wireless/realtek/rtlwifi/pci.h | 4

[PATCH 2/8] rtlwifi: rtl8192se: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8192se/def.h | 4

[PATCH 3/8] rtlwifi: rtl8192ce: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8192ce/def.h | 4

[PATCH 0/8] Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
A number of these drivers were submitted to the kernel before checkpatch.pl was modified to state that that the address of the FSF should not be included. This set of patches remove this information from the sources. Signed-off-by: Larry Finger Larry Finger (8):

[PATCH 5/8] rtlwifi: rtl8723ae: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8723ae/dm.h | 4

[PATCH 1/8] rtlwifi: rtl8192de: Remove address of Free Software Foundation

2016-10-04 Thread Larry Finger
Since this driver was added to the kernel, the checkpatch script was modified to request that the address of the FSF not be included. Signed-off-by: Larry Finger --- drivers/net/wireless/realtek/rtlwifi/rtl8192de/def.h | 4

Re: [PATCH 1/7] selftest: sync: basic tests for sw_sync framework

2016-10-04 Thread Michael Ellerman
Emilio López writes: > Hi, > > El 27/09/16 a las 01:23, Michael Ellerman escribió: >> Emilio López writes: >>> El 22/09/16 a las 06:43, Michael Ellerman escribió: Emilio López writes:

[PATCH] staging: comedi: Align the * on block comments

2016-10-04 Thread Jakub Jedelsky
Block comments should align the * on each line as reported by checkpatch Signed-off-by: Jakub Jedelsky --- drivers/staging/comedi/drivers/cb_pcidda.c | 4 ++-- drivers/staging/comedi/drivers/mite.c | 2 +- drivers/staging/comedi/drivers/pcl818.c| 6 +++--- 3

RE: [PATCH 1/2 v3] pci-hyperv: properly handle pci bus remove

2016-10-04 Thread KY Srinivasan
> -Original Message- > From: Long Li > Sent: Monday, October 3, 2016 11:43 PM > To: KY Srinivasan ; Haiyang Zhang > ; Bjorn Helgaas > Cc: de...@linuxdriverproject.org; linux-...@vger.kernel.org; linux- >

[PATCH] staging: i4l: icn: fixed some errors and warnings caught by checkpatch.pl

2016-10-04 Thread Harman Kalra
2 errors "open brace { should be on the previous line" are fixed. Warning "Prefer "pr_debug over printk(KERN_DEBUG... " fixed Few line more tahn 80 character warnings fixed Signed-off-by: Harman Kalra --- drivers/staging/i4l/icn/icn.c |3 +--

RE: [PATCH 2/2 v3] pci-hyperv: lock pci bus on device eject

2016-10-04 Thread KY Srinivasan
> -Original Message- > From: Long Li > Sent: Monday, October 3, 2016 11:43 PM > To: KY Srinivasan ; Haiyang Zhang > ; Bjorn Helgaas > Cc: de...@linuxdriverproject.org; linux-...@vger.kernel.org; linux- >

Re: [patch] staging: ion: use two separate locks for heaps and clients in ion_device

2016-10-04 Thread Laura Abbott
On 09/30/2016 01:18 AM, Xu YiPing wrote: ion_alloc may get into slow path to get free page, the call stack: __alloc_pages_slowpath ion_page_pool_alloc_pages alloc_buffer_page ion_system_heap_allocate ion_buffer_create <-- hold ion_device->lock ion_alloc after that, kernel invokes low-memory

[PATCH 00/10] staging: most: fix driver modules

2016-10-04 Thread Christian Gromm
This patch set is needed to fix up issues of the driver. Andrey Shvetsov (10): staging: most: core: remove member add_link staging: most: core: remove read option from remove_link staging: most: core: remove processing of deprecated names staging: most: core: update examples on how to

[PATCH 09/10] staging: most: hdm-dim2: remove tracing of mac address

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov This patch removes tracing of the MAC address from the DIM2 HDM as it is already done in the networking AIM. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm ---

[PATCH 08/10] staging: most: hdm-usb: remove filtering of

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov networking state In case the networking interface goes down and up again, the USB HDM does not report the state the MOST networking interface controller is in. This might lead to nonfunctional network. Since the networking AIM already takes care of

[PATCH 10/10] staging: most: hdm-usb: fix mbo buffer leak

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov This patch fixes an MBO leak by replacing the proprietary free_anchored_buffers() function with the usb_kill_anchored_urbs() function of the USB subsystem and guarantees that the mbo->complete() completion function is being called for each URB.

[PATCH 02/10] staging: most: core: remove read option from remove_link

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov The attribute "remove_link" returns the latest link that has been removed of a certain channel. Since this piece information isn't particulary useful this patch is going to remove it. Signed-off-by: Andrey Shvetsov

[PATCH 03/10] staging: most: core: remove processing of deprecated names

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov The USB HDM does not use the "@" character for channel names anymore. This patch cleans up the code processing such names and adapts the corresponding examples on how to use the properties "add_link" and "remove_link". Signed-off-by: Andrey Shvetsov

[PATCH 05/10] staging: most: aim-network: fix startup scenario.

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov In case the networking interface (running on MediaLB) is being started before the MOST network is, it remains disabled even after the MOST network has transitioned to active mode. This patch removes the dependency on the MOST link status to keep the

[PATCH 07/10] staging: most: aim-network: avoid calling netdev_info()

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov This patch removes the needless call of function netdev_info() from function most_nd_setup(). Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm ---

[PATCH 06/10] staging: most: aim-network: setup mac address before ifup has finished

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov The networking AIM implements a non-standard behavior as it starts the networking interface with an invalid MAC address and changes it by the time a valid one is available. This patch introduces a completion object to set the MAC address of the

[PATCH 01/10] staging: most: core: remove member add_link

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov This patch removes the unused field add_link of struct most_aim_obj. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm --- drivers/staging/most/mostcore/core.c | 2 -- 1 file

[PATCH 04/10] staging: most: core: update examples on how to link channels

2016-10-04 Thread Christian Gromm
From: Andrey Shvetsov This patch updates the comments with examples on how to use the "add_link" and "remove_link" properties. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm ---

[PATCH 00/13] staging: unisys: Convert client_bus_info to debugfs

2016-10-04 Thread David Kershner
Move /sys/bus/visorbus/devices/visorbus/client_bus_info to debugfs In a 9/28/2016 patch inspection that fixed code within the function that emits this client_bus_info sysfs entry, GregKH pointed out that this shouldn't actually be a sysfs entry at all, and in fact belongs in debugfs. This

[PATCH 11/13] staging: unisys: visorbus: visorbus_private rename 'local' var to dest

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch renames the poorly named 'local' var to 'dest' in the visorbus_private.h file Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner ---

[PATCH 07/13] staging: unisys: include: channel.h remove unused pound defines

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch removes all the unused pound defines currently in channel.h. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/include/channel.h | 49

[PATCH 01/13] staging: unisys: visorbus: controlvmchannel.h remove unused pound defines

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch removes all unused pound defines in controlvmchannel.h. Adds a comment on the now unused error codes for our spar firmware to avoid new error messages conflicting with previously defined messages. Signed-off-by: Erik Arfvidson

RE: [PATCH 1/2 v2] pci-hyperv: properly handle pci bus remove

2016-10-04 Thread KY Srinivasan
> -Original Message- > From: Bjorn Helgaas [mailto:helg...@kernel.org] > Sent: Tuesday, September 27, 2016 12:30 PM > To: Long Li > Cc: KY Srinivasan ; Haiyang Zhang > ; Bjorn Helgaas ; >

[PATCH 13/13] staging: unisys: visorbus: convert client_bus_info sysfs to debugfs

2016-10-04 Thread David Kershner
From: Tim Sell Previously, the sysfs entry (assuming traditional sysfs mountpoint): /sys/bus/visorbus/devices/visorbus/client_bus_info violated kernel conventions by printing more than one item. This along with the fact that the data emitted was diagnostic data

[PATCH 08/13] staging: unisys: visorbus: vmcallinterface.h remove unused pound defines

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch removes all the unused pound defines currently in vmcallinterface.h. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner ---

[PATCH 03/13] staging: unisys: visorbus: controlvmchannel.h fix spacing

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch fixes space and tab aligned comments in controlvmchannel.h. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/controlvmchannel.h | 42

[PATCH 12/13] staging: unisys: visorbus: move DEVICE_ATTR_RO defs as-per conventions

2016-10-04 Thread David Kershner
From: Tim Sell The kernel convention is for DEVICE_ATTR_RO() definitions to occur immediately after the functions that implement the attribute behaviors. DEVICE_ATTR_RO definitions in visorbus_main.c have been moved to comply with this convention. Signed-off-by: Tim

[PATCH 10/13] staging: unisys: visorbus: visorchannel.c rename 'local' variable to dest

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch renames poorly named 'local' variable to 'dest'. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchannel.c | 10 +- 1

[PATCH 06/13] staging: unisys: include: iochannel.h remove unused pound defines

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch removes all the unused pound defines currently in iochannel.h. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/include/iochannel.h | 28

[PATCH 09/13] staging: unisys: visorbus: vbuschannel.h remove unused pound defines

2016-10-04 Thread David Kershner
From: Erik Arfvidson this patch removes all the unused pound defines currently in vbuschannel.h Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/vbuschannel.h | 11

[PATCH 05/13] staging: unisys: visorbus: remove unused initializations

2016-10-04 Thread David Kershner
From: sameer wadgaonkar Removed initializations from visorchipset_open() and visorchipset_init(), and moved the memset in initialize_controlvm_payload_info() down in case of early returns from the function. Signed-off-by: Sameer Wadgaonkar

[PATCH 02/13] staging: unisys: visorbus: controlvmchannel.h fix comments

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch simply fixes comments to mantain a more consistent and shorter comments. githubissue: Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner ---

[PATCH 04/13] staging: unisys: include: include.h remove unused pound defines

2016-10-04 Thread David Kershner
From: Erik Arfvidson This patch removes all the unused pound defines currently in include.h. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/include/channel.h | 14

Re: [PATCH] staging: greybus: fix block comment

2016-10-04 Thread Johan Hovold
On Mon, Oct 03, 2016 at 11:45:15PM +0900, Hisao Tanabe wrote: > This patch fix the coding style problem of block comment detected by the > checkpatch.pl. > > WARNING: Block comments use a trailing */ on a separate line > > Signed-off-by: Hisao Tanabe > --- >

Re: [PATCH 14/15] staging: wlang-ng: fix line style warnings in prism2sta.c

2016-10-04 Thread Joe Perches
On Mon, 2016-10-03 at 20:12 +0200, Sergio Paracuellos wrote: > El 2016年10月03日 a las 18:16, Joe Perches escribió: > > Perhaps better as hw->scanresults = kmemdup(inf, sizeof(*inf), > > GFP_ATOMIC); > I agree. But because all the code is full of sizeof(struct xxx) I though > it would be bettter

Re: [PATCH] [media] : Removing warnings caught by checkpatch.pl

2016-10-04 Thread Laurent Pinchart
Hi Andrey, On Monday 03 Oct 2016 22:52:11 Andrey Utkin wrote: > On Sun, Oct 02, 2016 at 02:30:45AM +0530, Harman Kalra wrote: > > static int iss_video_queue_setup(struct vb2_queue *vq, > > > > -unsigned int *count, unsigned int *num_planes, > > -