Re: [jes:rtl8723au-devel 206/499] WARNING: __func__ should be used instead of gcc specific __FUNCTION__

2014-06-26 Thread Jes Sorensen
Fengguang Wu fengguang...@intel.com writes: tree: git://git.kernel.org/pub/scm/linux/kernel/git/jes/linux.git rtl8723au-devel head: 07068767ab5cc9eb549dc85cb858e00c384ade4c commit: 068bf12cd5cc11f85a5b4d5423e629b640e12df7 [206/499] staging: rtl8723au: hal/usb_ops_linux.c: Various

[PATCH 03/10] staging: rtl8723au: rtw_sta_mgt.c: Use ether_addr_{copy, equal}()

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_sta_mgt.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_sta_mgt.c

[PATCH 00/10] staging: rtl8723au: sparse fixes and some cleanups

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Greg, A couple of sparse fixes (a fix from Clement Calmels and one reported by Fengguang Wu) as well as some cleanups removing dead code. Cheers, Jes Clément Calmels (1): staging: rtl8723au: fix sparse warning Jes Sorensen (9): staging:

[PATCH 09/10] staging: rtl8723au: Remove no-op rtl8723a_sreset_linked_status_check()

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 2 -- drivers/staging/rtl8723au/hal/rtl8723a_sreset.c | 4 drivers/staging/rtl8723au/include/rtl8723a_sreset.h | 1 - 3 files changed,

[PATCH 02/10] staging: rtl8723au: usb_ops_linux.c: Add blank line between declaration and usage

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com This fixes a minor sparse warning. Reported-by: Fengguang Wu fengguang...@intel.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/hal/usb_ops_linux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 08/10] staging: rtl8723au: sreset_priv-dbg_trigger_point is never set

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com No point carrying a bunch of code around that is never going to get called. Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_sreset.c | 7 --- drivers/staging/rtl8723au/hal/rtl8723a_sreset.c | 14

[PATCH 04/10] staging: rtl8723au: rtw_sta_mgt.c: One copy of the broadcast address should suffice

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_sta_mgt.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_sta_mgt.c

[PATCH 06/10] staging: rtl8723au: Remove unused rtw_sreset_get_wifi_status()

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_sreset.c| 29 -- drivers/staging/rtl8723au/include/rtw_sreset.h | 1 - 2 files changed, 30 deletions(-) diff --git

[PATCH 07/10] staging: rtl8723au: Wifi_Error_Status isn't used for anything

2014-06-26 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_sreset.c | 11 --- drivers/staging/rtl8723au/hal/rtl8723a_sreset.c | 1 - drivers/staging/rtl8723au/hal/usb_ops_linux.c| 11 ---

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

2014-06-26 Thread Olaf Hering
On Mon, Jun 23, Haiyang Zhang wrote: Yes, there is a link down/up event from the host, we currently call netif_carrier_off() / netif_carrier_on() with these events. Will hotplug scripts be triggered by netif_carrier_off/on? Where are the scripts located at (SLES)? In

[PATCH] staging: unisys: remove unused macros

2014-06-26 Thread Paul Bolle
The macros CONFIG_SPAR_GUEST, GUESTDRIVERBUILD, and NOAUTOVERSION are defined in Makefiles for the Unisys SPAR driver. They are never used. Remove the lines that define these macros. Signed-off-by: Paul Bolle pebo...@tiscali.nl --- Compile tested only, on top of next-20140626, by fiddling

Re: [PATCH 1/4] Staging: unisys: remove references to __DATE__ and __TIME__

2014-06-26 Thread Ken Cox
On 06/25/2014 10:57 PM, Greg KH wrote: On Wed, Jun 25, 2014 at 11:48:23AM -0500, Ken Cox wrote: --snip-- .id_table = virthba_id_table, .probe = virthba_probe, .remove = virthba_remove, @@ -1413,9 +1413,6 @@ info_proc_read(struct file *file, char __user *buf, size_t

Re: [PATCH RFC v2 3/8] component: add support for component match array

2014-06-26 Thread Philipp Zabel
Hi Russell, On Tue, Jun 24, 2014 at 9:29 PM, Russell King rmk+ker...@arm.linux.org.uk wrote: [...] +/* + * Add a component to be matched. + * + * The match array is first created or extended if necessary. + */ +void component_match_add(struct device *dev, struct component_match

Re: [PATCH 1/4] Staging: unisys: remove references to __DATE__ and __TIME__

2014-06-26 Thread Greg KH
On Thu, Jun 26, 2014 at 05:37:35AM -0500, Ken Cox wrote: On 06/25/2014 10:57 PM, Greg KH wrote: On Wed, Jun 25, 2014 at 11:48:23AM -0500, Ken Cox wrote: --snip-- .id_table = virthba_id_table, .probe = virthba_probe, .remove = virthba_remove, @@ -1413,9 +1413,6 @@

[PATCH] staging: wlan-ng/hfa384x_usb.c: add blank line after declarations

2014-06-26 Thread Cheng-Wei Lee
This patch fixes the following checkpatch.pl issues in hfa384x_usb.c: WARNING: Missing a blank line after declarations Signed-off-by: Quentin Lee lee.rhaps...@gmail.com --- drivers/staging/wlan-ng/hfa384x_usb.c |1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH RFC v2 3/8] component: add support for component match array

2014-06-26 Thread Russell King - ARM Linux
On Thu, Jun 26, 2014 at 02:34:17PM +0200, Philipp Zabel wrote: Hi Russell, On Tue, Jun 24, 2014 at 9:29 PM, Russell King rmk+ker...@arm.linux.org.uk wrote: [...] +/* + * Add a component to be matched. + * + * The match array is first created or extended if necessary. + */ +void

[PATCH v2 0/2] Staging: unisys: Remove references to __DATE__ and __TIME__

2014-06-26 Thread Ken Cox
This series removes all references to __DATE__ and __TIME__ from the unisys drivers and also removes BROKEN from the Kconfig so the drivers can be built. ___ devel mailing list de...@linuxdriverproject.org

[PATCH v2 1/2] Staging: unisys: remove references to __DATE__ and __TIME__

2014-06-26 Thread Ken Cox
The use of __DATE__ and __TIME__ is no longer allowed in the kernel so this commit removes those. They were once useful when the drivers were being built externally, but now that the drivers are in the kernel the use of the macros is redundant since the kernel already has the same information

[PATCH v2 2/2] Staging: unisys Remove BROKEN from Kconfig to allow compilation

2014-06-26 Thread Ken Cox
The unisys drivers now properly check to make sure they are running on the s-Par platform before they will initialize. This was fixed in commit fcd0157ece so it is safe to allow the unisys drivers to be built. This has been tested in the same qemu environment that originally produced the panic

Re: Question about patches for bcm

2014-06-26 Thread Matthias Beyer
On 26-06-2014 10:58:20, Kevin McKinney wrote: On Thu, Jun 26, 2014 at 10:42 AM, Matthias Beyer m...@beyermatthias.de wrote: Hi, I have a question related to some patches I send in. (I send them to de...@driverdev.osuosl.org, but they show up in the archives of

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

2014-06-26 Thread Haiyang Zhang
-Original Message- From: Olaf Hering [mailto:o...@aepfle.de] Sent: Thursday, June 26, 2014 4:46 AM To: Haiyang Zhang Cc: Greg KH; net...@vger.kernel.org; jasow...@redhat.com; driverdev- de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; da...@davemloft.net Subject: Re:

Re: Question about patches for bcm

2014-06-26 Thread Matthias Beyer
Hi, your email client sends HTML-Emails. Can you switch this off? Maybe that's also the reason why your emails don't show up in the archives[0]! I will send my patches tomorrow, no time now. Regards, Matthias [0]:

Re: Question about patches for bcm

2014-06-26 Thread Kevin McKinney
On Thu, Jun 26, 2014 at 11:14 AM, Matthias Beyer m...@beyermatthias.de wrote: Hi, your email client sends HTML-Emails. Can you switch this off? Maybe that's also the reason why your emails don't show up in the archives[0]! Yeh, sorry about that. During the day I use gmail instead of mutt and

[PATCH v2 9/16] staging: comedi: ni_stc.h: the boardinfo 'name' should be first

2014-06-26 Thread H Hartley Sweeten
By convention, the first member of the boardinfo is a 'const char *' to the board name, especially for those comedi drivers that rely on the comedi core to set dev-board_ptr to the entry matching the board name when doing a legacy (*attach) to a comedi driver. Modify the ni_board_struct

Re: [PATCH 00/16] staging: comedi: remove some cruft from the NI drivers

2014-06-26 Thread Ian Abbott
On 2014/06/20 07:10 PM, H Hartley Sweeten wrote: The National Instruments MIO drivers, ni_pcimio, ni_mio_cs, and ni_atmio, all include the ni_mio_common.c source file to provide the common functionality for the drivers. Including source files is a bit ugly. Remove some cruft from the NI

[PATCH v2 13/18] staging: comedi: ni_65xx: update the MODULE_DESCRIPTION

2014-06-26 Thread H Hartley Sweeten
Update the MODULE_DESCRIPTION to better describe the driver. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gr...@linuxfoundation.org --- drivers/staging/comedi/drivers/ni_65xx.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v2 03/18] staging: comedi: ni_65xx: cleanup recurring register map

2014-06-26 Thread H Hartley Sweeten
Convert the inline functions used to calculate the offsets to the recurring port registers and rename them to remove the CamelCase. Define all the recurring registers. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman

[PATCH v2 14/18] staging: comedi: ni_65xx: tidy ni_65xx_intr_insn_bits() declaration

2014-06-26 Thread H Hartley Sweeten
For aesthetics, tidy up the whitespace of this function declarations to follow the form in the rest of the driver. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gr...@linuxfoundation.org ---

[PATCH v2 16/18] staging: comedi: ni_65xx: use the subdevice 'io_bits' to handle the 'invert_outputs'

2014-06-26 Thread H Hartley Sweeten
Some of the boards supported by this driver have output ports that are inverted from the comedi view of the output state. For these boards the read values from the output ports needs to be inverted before being modified and inverted again before being written back in the (*insn_bits) operation.

[PATCH v2 09/18] staging: comedi: ni_65xx: tidy up the port-chan conversions

2014-06-26 Thread H Hartley Sweeten
Introduce some convienence macros to handle the port to channel, channel to port, and channel mask calculations based on the 8 channels per port of the hardware. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman

[PATCH v2 11/18] staging: comedi: ni_65xx: tidy up the comedi_driver declaration

2014-06-26 Thread H Hartley Sweeten
For aesthetics, add some whitespace to the comedi_driver declaration. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gr...@linuxfoundation.org --- drivers/staging/comedi/drivers/ni_65xx.c | 8 1 file changed, 4

[PATCH v2 01/18] staging: comedi: ni_65xx: filter interval register is 32-bit

2014-06-26 Thread H Hartley Sweeten
According to the register programming manual, the filter interval register is 32-bit. Fix the writes to this register. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gr...@linuxfoundation.org ---

[PATCH v2 17/18] staging: comedi: ni_65xx: fix digital output reset during attach

2014-06-26 Thread H Hartley Sweeten
During the attach of this driver, the digital output ports are all initialized to a known state. Some of the boards supported by this driver have output ports that are inverted from the comedi view of the output state. For these boards the values written to the ports needs to be inverted.

[PATCH v2 15/18] staging: comedi: ni_65xx: remove 'output_bits' from private data

2014-06-26 Thread H Hartley Sweeten
Remove the need for the 'output_bits' in the private data by just reading the current state of the data port when updating the output channels in the (*insn_bits) function. Add a local variable to handle the inverting of the hardware values when the boardinfo indicates that the outputs are

[PATCH v2 18/18] staging: comedi: ni_65xx: factor input filter disable out of (*auto_attach)

2014-06-26 Thread H Hartley Sweeten
For aesthetics, factor the code that disables the input filters out of ni_65xx_auto_attach(). Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gr...@linuxfoundation.org --- drivers/staging/comedi/drivers/ni_65xx.c | 31

[PATCH v2 06/18] staging: comedi: ni_65xx: remove the need for the subdevice private data

2014-06-26 Thread H Hartley Sweeten
There is only one member in the subdevice private data, an unsigned value that is the 'base_port' that the subdevice uses to access the port registers. Just cast the appropriate value into s-private instead of allocating the private data for each subdevice. The casts are a bit of a nusance but it

Re: Anybody working on ced1401?

2014-06-26 Thread Greg KH
On Thu, Jun 26, 2014 at 09:36:17AM +0200, Alois Schloegl wrote: On 2014-06-18 13:33, Kristina Martšenko wrote: Hi Alois, I'm helping Greg do a bit of cleanup in the staging tree. I noticed that nobody seems to have worked towards moving ced1401 out of staging in over a year. Are there

[PATCH] Tools: hv: fix file overwriting of hv_fcopy_daemon

2014-06-26 Thread Yue Zhang
From: Yue Zhang yue...@microsoft.com hv_fcopy_daemon fails to overwrite a file if the target file already exits. Add O_TRUNC flag on opening. Signed-off-by: Yue Zhang yue...@microsoft.com --- tools/hv/hv_fcopy_daemon.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

RE: [PATCH] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-06-26 Thread Dexuan Cui
MS-TFS: 157532 What is this line for? Hi Greg, This line is for our internal bug repository. We have an automated system to correlate bugs with fixes so that our test team knows when a bug fix has been accepted upstream and they need to write a new test case for it. The

[PATCH v2] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-06-26 Thread Dexuan Cui
Currently the VSC has no chance to notify the VSP of the dirty rectangle on VM panic because the notification work is done in a workqueue, and in panic() the kernel typically ends up in an infinite loop, and a typical kernel config has CONFIG_PREEMPT_VOLUNTARY=y and CONFIG_PREEMPT is not set, so a

RE: [PATCH] Tools: hv: fix file overwriting of hv_fcopy_daemon

2014-06-26 Thread KY Srinivasan
-Original Message- From: Yue Zhang [mailto:yue...@microsoft.com] Sent: Thursday, June 26, 2014 2:09 PM To: KY Srinivasan; Haiyang Zhang; driverdev-devel@linuxdriverproject.org; linux-ker...@vger.kernel.org; o...@aepfle.de; jasow...@redhat.com; a...@canonical.com Cc: Dexuan Cui;

Re: [PATCH v14 04/10] imx-drm: use defines for clock polarity settings

2014-06-26 Thread Russell King - ARM Linux
On Wed, Jun 25, 2014 at 11:44:47AM +0200, Denis Carikli wrote: On 06/25/2014 06:48 AM, Sascha Hauer wrote: +#define ENABLE_POL_LOW 0 +#define ENABLE_POL_HIGH1 Adding defines without a proper namespace (IPU_) outside a driver private header file is not nice. Anyway,

RE: [PATCH] Tools: hv: fix file overwriting of hv_fcopy_daemon

2014-06-26 Thread Yue Zhang (OSTC DEV)
-Original Message- From: KY Srinivasan -Original Message- From: Yue Zhang [mailto:yue...@microsoft.com] hv_fcopy_daemon fails to overwrite a file if the target file already exits. Add O_TRUNC flag on opening. Signed-off-by: Yue Zhang yue...@microsoft.com ---

Re: Anybody working on line6?

2014-06-26 Thread Greg KH
On Tue, Jun 24, 2014 at 10:12:03PM +0800, Stefan Hajnoczi wrote: On Tue, Jun 24, 2014 at 9:52 PM, Greg KH gre...@linuxfoundation.org wrote: On Tue, Jun 24, 2014 at 11:54:10AM +0800, Stefan Hajnoczi wrote: On Tue, Jun 24, 2014 at 4:23 AM, Kristina Martšenko kristina.martse...@gmail.com

Re: ACPI Support for HP Pavillion laptops

2014-06-26 Thread Greg KH
On Wed, Jun 25, 2014 at 08:19:52AM +0530, Chetal Patil wrote: Hi, I recently joined the Linux Driver Development mailing list and would like to contribute to the Linux Driver Project. Specifically, I would like to contribute by working on ACPI Support for HP Pavillion laptops (Referred list

Re: [PATCH 1/3] staging: rtl8188eu: core: Remove unnecessary conditionals

2014-06-26 Thread Greg KH
On Sun, Jun 22, 2014 at 01:01:50PM +0200, Thomas Vegas wrote: The variable 'pxmitbuf' is already set to NULL when entering the function. We can remove the first conditional. When _rtw_queue_empty() returns false, 'pxmitbuf' if not NULL. We can merge the second and third conditional.

Re: [PATCH 2/6] Staging: bcm: Indentation fixes

2014-06-26 Thread Greg KH
On Mon, Jun 23, 2014 at 01:41:05PM -0700, j...@joshtriplett.org wrote: On Mon, Jun 23, 2014 at 11:42:29AM +0200, Matthias Beyer wrote: This patch fixes some indentation errors, where multi-line statements where not indented. Signed-off-by: Matthias Beyer m...@beyermatthias.de This

Re: [PATCH] staging: winbond: remove driver

2014-06-26 Thread Greg KH
On Mon, Jun 23, 2014 at 06:41:29PM +0300, Kristina Martšenko wrote: The driver hasn't been cleaned up and nobody is working to do so, so remove it. Signed-off-by: Kristina Martšenko kristina.martse...@gmail.com Cc: Pavel Machek pa...@ucw.cz This patch didn't apply to my tree for some odd

Re: [PATCH] staging: wlan-ng/hfa384x_usb.c: add blank line after declarations

2014-06-26 Thread Greg Kroah-Hartman
On Thu, Jun 26, 2014 at 10:43:55PM +0800, Cheng-Wei Lee wrote: This patch fixes the following checkpatch.pl issues in hfa384x_usb.c: WARNING: Missing a blank line after declarations Signed-off-by: Quentin Lee lee.rhaps...@gmail.com --- drivers/staging/wlan-ng/hfa384x_usb.c |1 + 1