Re: [PATCH] staging:iio: moved platform_data into include/linux/iio

2014-08-19 Thread Lars-Peter Clausen
On 08/20/2014 08:44 AM, sanjeev sharma wrote: Hi, This was the action item(TO-DO). IMO, it make sense to move into include/linux/iio because IIO complete subsystem may take some time. The code that is in staging is not supposed to 'leak' outside of staging. So either you move the driver as a

Re: [PATCH] staging:iio: moved platform_data into include/linux/iio

2014-08-19 Thread Lars-Peter Clausen
On 08/20/2014 08:41 AM, Sanjeev Sharma wrote: This is a patch to the iio which will move all platform_data into include/linux/iio. It should be moved together with the driver. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxd

[PATCH] staging:iio: moved platform_data into include/linux/iio

2014-08-19 Thread Sanjeev Sharma
This is a patch to the iio which will move all platform_data into include/linux/iio. Signed-off-by: Sanjeev Sharma --- drivers/staging/iio/adc/ad7192.c| 3 +- drivers/staging/iio/adc/ad7192.h| 47 - drivers/staging/iio/adc/ad7280a.c | 3

Re: [hyperv] BUG at drivers/hv/channel.c:462 while changing MTU

2014-08-19 Thread Sitsofe Wheeler
On Tue, Aug 19, 2014 at 09:29:44PM +, Haiyang Zhang wrote: > > > -Original Message- > > From: Sitsofe Wheeler [mailto:sits...@gmail.com] > > Sent: Tuesday, August 19, 2014 8:37 AM > > To: KY Srinivasan > > Cc: Greg Kroah-Hartman; Haiyang Zhang; de...@linuxdriverproject.org; > > linux-k

RE: [hyperv] BUG at drivers/hv/channel.c:462 while changing MTU

2014-08-19 Thread Haiyang Zhang
> -Original Message- > From: Sitsofe Wheeler [mailto:sits...@gmail.com] > Sent: Tuesday, August 19, 2014 8:37 AM > To: KY Srinivasan > Cc: Greg Kroah-Hartman; Haiyang Zhang; de...@linuxdriverproject.org; > linux-ker...@vger.kernel.org; Jason Wang; Daniel Borkmann; David S. > Miller; net..

[PATCH net-next] hyperv: Add handling of IP header with option field in netvsc_set_hash()

2014-08-19 Thread Haiyang Zhang
In case that the IP header has optional field at the end, this patch will get the port numbers after that field, and compute the hash. Signed-off-by: Haiyang Zhang Reviewed-by: K. Y. Srinivasan --- drivers/net/hyperv/netvsc_drv.c | 16 1 files changed, 12 insertions(+), 4 d

[PATCH] staging: lustre: lustre: ptlrpc: Placing inline keyword

2014-08-19 Thread Hema Prathaban
This patch fixes the following error using checkpatch.pl ERROR: inline keyword should sit between storage class and type Signed-off-by: Hema Prathaban --- drivers/staging/lustre/lustre/ptlrpc/sec_config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/

Re: [PATCH 2/2] Drivers: scsi: storvsc: Force discovery of LUNs that may have been removed.

2014-08-19 Thread Christoph Hellwig
On Sat, Aug 16, 2014 at 08:09:48PM -0700, K. Y. Srinivasan wrote: > The host asks the guest to scan when a LUN is removed or added. > The only way a guest can identify the removed LUN is when an I/O is > attempted on a removed LUN - the SRB status code indicates that the LUN > is invalid. We curren

[PATCH] staging: lustre: lnet: socklnd_cb.c Fix sparse non-static symbol warning

2014-08-19 Thread Phong Tran
This patch fixes some non static function declarations that cause sparse warning. Tested by compilation. Signed-off-by: Phong Tran --- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/l

Re: [PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Philipp Zabel
Am Dienstag, den 19.08.2014, 18:26 +0300 schrieb Yannis Damigos: > This is a patch to the ipuv3-crtc.c file that fixes up two "lines > over 80 characters" warnings found by the checkpatch.pl tool, > keeping the alignment of the x/y/width/height parameters for > readability. > > Signed-off-by: Yann

[PATCH] staging: dgnc: remove DPR Macros and related codes.

2014-08-19 Thread Seunghun Lee
In dgnc_drivers.h, DPR macro and DPR_* macros are defined but do nothing. So remove them and related codes. CC: Lidza Louina CC: Mark Hounschell Signed-off-by: Seunghun Lee --- drivers/staging/dgnc/dgnc_cls.c| 51 drivers/staging/dgnc/dgnc_driver.c | 19 +-- drivers/staging/d

[PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Yannis Damigos
This is a patch to the ipuv3-crtc.c file that fixes up two "lines over 80 characters" warnings found by the checkpatch.pl tool, keeping the alignment of the x/y/width/height parameters for readability. Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 ++--- 1 file c

[PATCH 5/6] staging: comedi: ni_labpc: break lines after operators

2014-08-19 Thread Ian Abbott
For consistency, break lines before operators, not after. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/ni_labpc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging/comedi/drivers/ni_labpc.c

[PATCH 0/6] staging: comedi: ni_labpc: split module in two

2014-08-19 Thread Ian Abbott
The "ni_labpc" module is a Comedi driver for the National Instruments Lab-PC series of ISA data acquistion boards, and also provides common code for the "ni_labpc_pci" and "ni_labpci_cs" modules (for PCI boards and PCMCIA cards). Split out the common code into a new module "ni_labpc_common", leavi

[PATCH 2/6] staging: comedi: ni_labpc: use braces on all arms of `if` statement

2014-08-19 Thread Ian Abbott
Fix checkpatch issues "CHECK: braces {} should be used on all arms of this statement". Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/ni_labpc.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging

[PATCH 4/6] staging: comedi: ni_labpc: Logical continuations should be on the previous line

2014-08-19 Thread Ian Abbott
Fix checkpatch issues "CHECK: Logical continuations should be on the previous line". Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/ni_labpc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging/come

[PATCH 6/6] staging: comedi: ni_labpc: split common code to ni_labpc_common

2014-08-19 Thread Ian Abbott
The "ni_labpc" module is a Comedi driver for the National Instruments Lab-PC series of ISA data acquistion boards, and also provides common code for the "ni_labpc_pci" and "ni_labpci_cs" modules (for PCI boards and PCMCIA cards). Split out the common code into a new module "ni_labpc_common", leavi

[PATCH 1/6] staging: comedi: ni_labpc: remove unnecessary braces in labpc_common_attach()

2014-08-19 Thread Ian Abbott
Correct checkpatch issue "WARNING: braces {} are not necessary for any arm of this statement". Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/ni_labpc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/sta

[PATCH 3/6] staging: comedi: ni_labpc: Alignment should match open parenthesis

2014-08-19 Thread Ian Abbott
Fix checkpatch issues "CHECK: Alignment should match open parenthesis". Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/ni_labpc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging/comedi/drivers/ni_l

Re: [PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Russell King - ARM Linux
On Tue, Aug 19, 2014 at 05:36:10PM +0300, Yannis Damigos wrote: > diff --git a/drivers/staging/imx-drm/ipuv3-crtc.c > b/drivers/staging/imx-drm/ipuv3-crtc.c > index 720868b..d6657a0 100644 > --- a/drivers/staging/imx-drm/ipuv3-crtc.c > +++ b/drivers/staging/imx-drm/ipuv3-crtc.c > @@ -201,9 +201,10

[PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Yannis Damigos
This is a patch to the ipuv3-crtc.c file that fixes up two "lines over 80 characters" warnings found by the checkpatch.pl tool, keeping the alignment of the x/y/width/height parameters for readability. Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 10 ++ 1 file

[no subject]

2014-08-19 Thread Yannis Damigos
I updated the patch to keep the alignment of the x/y/width/height parameters for readability. regards, Yannis ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Philipp Zabel
Am Dienstag, den 19.08.2014, 16:59 +0300 schrieb Yannis Damigos: > This is a patch to the ipuv3-crtc.c file that fixes up two "lines > over 80 characters" warnings found by the checkpatch.pl tool. > > Signed-off-by: Yannis Damigos > --- > drivers/staging/imx-drm/ipuv3-crtc.c | 9 + > 1 f

[PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Yannis Damigos
This is a patch to the ipuv3-crtc.c file that fixes up two "lines over 80 characters" warnings found by the checkpatch.pl tool. Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/imx-

[PATCH 8/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_parse_of

2014-08-19 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/staging/imx-drm/imx-drm-core.c b/drivers/stagi

[PATCH 7/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-08-19 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3 inser

[PATCH 2/8] imx-drm: Do not decrement endpoint node refcount in the loop

2014-08-19 Thread Philipp Zabel
In preparation for the following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel ---

[PATCH 5/8] of: Add of_graph_get_port_by_id function

2014-08-19 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel --- drivers/of/base.c| 30 ++ include/linux/of_graph.h | 7 +++ 2 files changed, 37 insertions(+) diff --git a/drivers/of/base.c b/d

[PATCH 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-08-19 Thread Philipp Zabel
In preparation for a following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel ---

[PATCH 6/8] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-08-19 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.c index 16150a0..024fa77 100

[PATCH 3/8] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-08-19 Thread Philipp Zabel
Decrementing the reference count of the previous endpoint node allows to use the of_graph_get_next_endpoint function in a for_each_... style macro. Prior to this patch, all current users of this function that actually pass a non-NULL prev parameter should be changed to not decrement the passed prev

[PATCH 4/8] of: Add for_each_endpoint_of_node helper macro

2014-08-19 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel --- include/linux/of_graph.h | 4 1 file changed, 4 insertions(+) diff --

[PATCH 0/5] Add of-graph helpers to loop over endpoints and find ports by id

2014-08-19 Thread Philipp Zabel
Hi, this series converts all existing users of of_graph_get_next_endpoint that pass a non-NULL prev argument to the function and decrement its refcount themselves to stop doing that. The of_node_put is moved into of_graph_get_next_endpoint instead. This allows to add a for_each_endpoint_of_node he

Re: [PATCH 1/1] Fix some code style issues

2014-08-19 Thread Greg Kroah-Hartman
On Tue, Aug 19, 2014 at 02:57:11PM +0300, Yannis Damigos wrote: > Signed-off-by: Yannis Damigos Your subject: line needs a lot of work. What part of the kernel are you changing? Then you need some description in the body of the changelog saying _why_ you made this change, and what you changed.

[hyperv] BUG at drivers/hv/channel.c:462 while changing MTU

2014-08-19 Thread Sitsofe Wheeler
Now we get to the issue I've been meaning to report: changing the MTU of a Hyper-V network interface while traffic is flowing through it triggers a BUG. I've seen this happen on a variety of kernels but the trace below is from 3.17.0-rc1. Steps to reproduce: (Guests were customised Fedora 20 cloud

Re: i8k: Don't revert affinity in i8k_smm

2014-08-19 Thread Con Kolivas
On Mon, 18 Aug 2014 07:32:15 PM Guenter Roeck wrote: > On Tue, Aug 19, 2014 at 09:19:55AM +1000, Con Kolivas wrote: > > As a followup to this discussion: > > > > On Tue, 15 Jul 2014 08:01:13 PM Sam Asadi wrote: > > > Commit f36fdb9f0266 (i8k: Force SMM to run on CPU 0) adds support > > > for multi

[PANIC, hyperv] kernel BUG at drivers/hv/channel.c:504!

2014-08-19 Thread Sitsofe Wheeler
While booting a Hyper-V 3.17.0-rc1 guest on a 2012 R2 host a BUG was triggered in drivers/hv/channel.c:504 which in turn caused a panic: [ 11.367472] Non-volatile memory driver v1.3 [ 11.395587] Linux agpgart interface v0.103 [ 11.425034] loop: module loaded [ 11.446046] Loading iSCSI tran

[PATCH 1/1] Fix some code style issues

2014-08-19 Thread Yannis Damigos
Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/imx-drm/ipuv3-crtc.c b/drivers/staging/imx-drm/ipuv3-crtc.c index 720868b..c4c6455 100644 --- a/drivers/staging/imx-drm/ipuv3-crtc.c

Re: BUG: unable to handle kernel paging request at ffff8801f3febe63 (netvsc_select_queue)

2014-08-19 Thread Sitsofe Wheeler
On Tue, Aug 19, 2014 at 10:57:30AM +0200, Daniel Borkmann wrote: > On 08/19/2014 10:15 AM, Sitsofe Wheeler wrote: > >After a variety of issues on Hyper-V (host is running Windows 2012 R2) I > >updated to the latest kernel (3.17-rc1 > >7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9), turned on a bunch of

[PATCH] staging: comedi: amplc_pci230: fix DACOUT write

2014-08-19 Thread Ian Abbott
Commit 4f9c63fe5333b27ab23ed399830c7977f6970744 ("staging: comedi: amplc_pci230: refactor iobase addresses") removed some parentheses (presumably to keep the line withing 80 chars) in `pci230_ao_write_nofifo()` when writing to the DACOUT1 or DACOUT2 registers, but it removed the wrong parentheses.

[linux-next PATCH] staging: unisys/virthba/virthba.c print u64 with %llu

2014-08-19 Thread Honggang Li
drivers/staging/unisys/virthba/virthba.c:520:2: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 4 has type ‘u64’ [-Wformat=] drivers/staging/unisys/virthba/virthba.c:749:2: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘u64’ [-Wformat=] driv

[linux-next PATCH] print u64 with %llu

2014-08-19 Thread Honggang Li
There are printk format issue and coding style issue for the lines changed by this patch. This patch only fix the format issue. Please take those coding style errors as false positives for this patch. Honggang Li (1): staging: unisys/virthba/virthba.c print u64 with %llu drivers/staging/unisys

[linux-next PATCH] print u64 with %llu

2014-08-19 Thread Honggang Li
There are printk format issue and coding style issue for the lines changed by this patch. This patch only fix the format issue. Please take those coding style errors as false positives for this patch. [real@name linux-next]$ scripts/checkpatch.pl outgoing2/0001-staging-unisys-virthba-virthba.c-pr

Re: BUG: unable to handle kernel paging request at ffff8801f3febe63 (netvsc_select_queue)

2014-08-19 Thread Daniel Borkmann
On 08/19/2014 10:15 AM, Sitsofe Wheeler wrote: After a variety of issues on Hyper-V (host is running Windows 2012 R2) I updated to the latest kernel (3.17-rc1 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9), turned on a bunch of kernel validation options and booted which has resulted in a BUG being tri

RE: [PATCH] Input: hyperv-keyboard - implement Type Clipboard Text

2014-08-19 Thread Dexuan Cui
> -Original Message- > From: Dmitry Torokhov > Sent: Tuesday, August 19, 2014 2:52 AM > To: Dexuan Cui > On Mon, Aug 18, 2014 at 03:54:51AM +, Dexuan Cui wrote: > > > -Original Message- > > > From: Dmitry Torokhov > > > Sent: Saturday, August 16, 2014 0:58 AM > > > To: Dexuan Cu

[PATCH] Fix checkpatch warning: Missing a blank line after declarations: drivers/staging/android/sync.c

2014-08-19 Thread Dipak Zope
Signed-off-by: Dipak Zope --- drivers/staging/android/sync.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index e7b2e02..0d37495 100644 --- a/drivers/staging/android/sync.c +++ b/drivers/staging/android/sync.c @@ -705,6 +705,7

BUG: unable to handle kernel paging request at ffff8801f3febe63 (netvsc_select_queue)

2014-08-19 Thread Sitsofe Wheeler
After a variety of issues on Hyper-V (host is running Windows 2012 R2) I updated to the latest kernel (3.17-rc1 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9), turned on a bunch of kernel validation options and booted which has resulted in a BUG being triggered (IP claims to be at netvsc_select_queue),