Re: [PATCH v18 12/12] drm: bridge/dw_hdmi: add rockchip rk3288 support

2014-12-10 Thread Daniel Kurtz
Hi Andy, This driver adds HDMI to rockchip/drm. The fact that rockchip's hdmi uses dw_hdmi is an implementation detail. I do not think that the names used for rk3288-hdmi should include "dw" in them. See inline for what I mean... On Thu, Dec 4, 2014 at 10:34 PM, Andy Yan wrote: > Rockchip RK3

Re: [PATCH V3 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread Jeremiah Mahler
K. Y. Srinivasan, On Wed, Dec 10, 2014 at 05:13:00PM -0800, K. Y. Srinivasan wrote: > Correctly compute the local (gpadl) handle. This description is still too sparse for me. How was it computed before and why was this incorrect? Pretend like you are trying to explain your patch to someone who

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Joe Perches
On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: > checkpatch makes an exception to the 80-colum rule for quotes strings, and > Documentation/CodingStyle recommends not splitting quotes strings across lines > because it breaks the ability to grep for the string. Fixing these. [] > diff

Re: Fwd: improve support for usb board camera

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 07:35:48PM -0500, Kristopher Borer wrote: > Hello, > > I am looking to improve support for a USB board camera. It currently > works fairly well, but we occasionally cause kernel panics while using > OpenCV on Ubuntu 14.04. What kernel version is that, and what is the oops

[socfpga-nex:for_next-fpgamgr-without-overlays-v4.3 8/9] drivers/staging/fpga/socfpga.c:655:25: sparse: symbol 'socfpga_fpga_ops' was not declared. Should it be static?

2014-12-10 Thread kbuild test robot
tree: git://git.rocketboards.org/linux-socfpga-next for_next-fpgamgr-without-overlays-v4.3 head: 7920447fc75922e9a72db640cddf890d75a30e0d commit: 99aa2d008ce7a2a925368bfddd53ae83d307fb38 [8/9] staging: fpga manager: add driver for socfpga fpga manager reproduce: # apt-get install sparse g

[PATCH socfpga-nex] staging: fpga manager: socfpga_fpga_ops can be static

2014-12-10 Thread kbuild test robot
drivers/staging/fpga/socfpga.c:655:25: sparse: symbol 'socfpga_fpga_ops' was not declared. Should it be static? Signed-off-by: Fengguang Wu --- socfpga.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fpga/socfpga.c b/drivers/staging/fpga/socfpga.c index

Fwd: improve support for usb board camera

2014-12-10 Thread Kristopher Borer
Hello, I am looking to improve support for a USB board camera. It currently works fairly well, but we occasionally cause kernel panics while using OpenCV on Ubuntu 14.04. Here is the device: e-consystems.com/5mp-usb-cameraboard.asp We do not need an NDA. We can provide hardware, test scripts

[PATCH v5 3/3] staging: media: lirc: lirc_zilog.c: missing newline in dev_err()

2014-12-10 Thread Luis de Bethencourt
Missing newline character at the end of string passed to dev_err() Signed-off-by: Luis de Bethencourt --- drivers/staging/media/lirc/lirc_zilog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc/lirc_zilog.c b/drivers/staging/media/lirc/lirc_zilog.c

[PATCH v5 2/3] staging: media: lirc: lirc_zilog.c: keep consistency in dev functions

2014-12-10 Thread Luis de Bethencourt
The previous patch switched some dev functions to move the string to a second line. Doing this for all similar functions because it makes the driver easier to read if all similar lines use the same criteria. Signed-off-by: Luis de Bethencourt --- drivers/staging/media/lirc/lirc_zilog.c | 57

[PATCH v5 1/3] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
checkpatch makes an exception to the 80-colum rule for quotes strings, and Documentation/CodingStyle recommends not splitting quotes strings across lines because it breaks the ability to grep for the string. Fixing these. WARNING: quoted string split across lines Signed-off-by: Luis de Bethencour

RE: [PATCH 2/2] Drivers: hv: balloon: Fix the deadlock issue in the memory hot-add code

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Michal Hocko [mailto:mho...@suse.cz] > Sent: Tuesday, December 9, 2014 2:56 AM > To: Yasuaki Ishimatsu > Cc: KY Srinivasan; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; linux..

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
On Wed, Dec 10, 2014 at 04:01:04PM -0800, Joe Perches wrote: > On Wed, 2014-12-10 at 23:57 +, Luis de Bethencourt wrote: > > On Wed, Dec 10, 2014 at 03:39:09PM -0800, Joe Perches wrote: > > > On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: > > > > diff --git a/drivers/staging/medi

[PATCH V3 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread K. Y. Srinivasan
Correctly compute the local (gpadl) handle. I would like to thank Michael Brown for seeing this bug. Signed-off-by: K. Y. Srinivasan Reported-by: Michael Brown --- Changes in V2: Added the Reported-by tag. Changes in V3: Cleaned up the commit log. drivers/hv/channel.c |4 ++-- 1 files ch

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Joe Perches
On Wed, 2014-12-10 at 23:57 +, Luis de Bethencourt wrote: > On Wed, Dec 10, 2014 at 03:39:09PM -0800, Joe Perches wrote: > > On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: > > > diff --git a/drivers/staging/media/lirc/lirc_zilog.c > > > b/drivers/staging/media/lirc/lirc_zilog.c

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
On Wed, Dec 10, 2014 at 03:39:09PM -0800, Joe Perches wrote: > On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: > > checkpatch makes an exception to the 80-colum rule for quotes strings, and > > Documentation/CodingStyle recommends not splitting quotes strings across > > lines > > bec

RE: [PATCH 3/3] Drivers: hv: hv_balloon: Don't post pressure status from interrupt context

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Sitsofe Wheeler [mailto:sits...@gmail.com] > Sent: Wednesday, December 10, 2014 12:50 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > jasow...@redhat.c

RE: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Long Li > Sent: Wednesday, December 10, 2014 2:28 PM > To: Martin K. Petersen > Cc: KY Srinivasan; Haiyang Zhang; jbottom...@parallels.com; linux- > s...@vger.kernel.org; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org > Subject: RE: [PATCH] scsi

[PATCH v4 2/2] staging: media: lirc: lirc_zilog.c: keep consistency in dev functions

2014-12-10 Thread Luis de Bethencourt
The previous patch switched some dev functions to move the string to a second line. Doing this for all similar functions because it makes the driver easier to read if all similar lines use the same criteria. Signed-off-by: Luis de Bethencourt --- drivers/staging/media/lirc/lirc_zilog.c | 57

[PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
checkpatch makes an exception to the 80-colum rule for quotes strings, and Documentation/CodingStyle recommends not splitting quotes strings across lines because it breaks the ability to grep for the string. Fixing these. WARNING: quoted string split across lines Signed-off-by: Luis de Bethencour

RE: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Long Li
> -Original Message- > From: Sitsofe Wheeler [mailto:sits...@gmail.com] > Sent: Wednesday, December 10, 2014 12:58 PM > To: Long Li > Cc: KY Srinivasan; Haiyang Zhang; jbottom...@parallels.com; linux- > s...@vger.kernel.org; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org > Su

RE: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Long Li
> > "Long" == Long Li writes: > > Long> MSFT targets currently claim SPC-2 compliance while they implement > Long> post SPC-2 features. With this patch we can correctly handle > Long> WRITE_SAME_16 issues. > > Handle the issues or handle WRITE SAME(10/16)? With this patch, the SCSI layer wi

Re: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Martin K. Petersen
> "Long" == Long Li writes: Long> MSFT targets currently claim SPC-2 compliance while they implement Long> post SPC-2 features. With this patch we can correctly handle Long> WRITE_SAME_16 issues. Handle the issues or handle WRITE SAME(10/16)? + {"Msft", "Virtual Disk", "1.0", BLIST_TR

Re: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Sitsofe Wheeler
On Wed, Dec 10, 2014 at 12:38:25AM -0800, Long Li wrote: > MSFT targets currently claim SPC-2 compliance while they implement > post SPC-2 features. With this patch we can correctly handle > WRITE_SAME_16 issues. > > This patch fixes an issue where the flag is setup too late in drive > initializat

Re: [PATCH 3/3] Drivers: hv: hv_balloon: Don't post pressure status from interrupt context

2014-12-10 Thread Sitsofe Wheeler
On Mon, Dec 08, 2014 at 06:04:35AM +, KY Srinivasan wrote: > > Greg has not committed these patches yet. One of the patches changes the > balloon floor. > This means that the guest will not be ballooned down below the floor. Is this > what you are > seeing? In our testing we did not see anyt

[PATCH V2 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread K. Y. Srinivasan
Fix a bug in vmbus_establish_gpadl(). I would like to thank Michael Brown for seeing this bug. In this version, I have added the Reported-by tag. Signed-off-by: K. Y. Srinivasan Reported-by: Michael Brown --- drivers/hv/channel.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) d

RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Wednesday, December 10, 2014 11:00 AM > To: KY Srinivasan > Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; > mc...@ipxe.org > Subjec

Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 11:42:43AM -0800, K. Y. Srinivasan wrote: > Fix a bug in vmbus_establish_gpadl(). I would like to > thank Michael Brown for seeing this bug. Then use the "Reported-by:" tag with his name in this patch please. thanks, greg k-h _

RE: [PATCH 4/5] Tools: hv: address compiler warnings for hv_fcopy_daemon.c

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, December 9, 2014 7:48 AM > To: KY Srinivasan > Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; Dexuan Cui > Subject: [PATCH 4/5] Tools: hv: address compiler warnin

RE: [PATCH 5/5] Tools: hv: do not add redundant '/' in hv_start_fcopy()

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, December 9, 2014 7:48 AM > To: KY Srinivasan > Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; Dexuan Cui > Subject: [PATCH 5/5] Tools: hv: do not add redundant '/

RE: [PATCH 1/5] Tools: hv: add mising fcopyd to the Makefile

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, December 9, 2014 7:48 AM > To: KY Srinivasan > Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; Dexuan Cui > Subject: [PATCH 1/5] Tools: hv: add mising fcopyd to th

RE: [PATCH 3/5] Tools: hv: address compiler warnings for hv_kvp_daemon.c

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, December 9, 2014 7:48 AM > To: KY Srinivasan > Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; Dexuan Cui > Subject: [PATCH 3/5] Tools: hv: address compiler warnin

RE: [PATCH 2/5] Tools: hv: remove unused bytes_written from kvp_update_file()

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, December 9, 2014 7:48 AM > To: KY Srinivasan > Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; Dexuan Cui > Subject: [PATCH 2/5] Tools: hv: remove unused bytes_wri

RE: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, December 9, 2014 5:06 AM > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > jasow...@r

[PATCH 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread K. Y. Srinivasan
Fix a bug in vmbus_establish_gpadl(). I would like to thank Michael Brown for seeing this bug. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c index 433f72a..c76ffbe

RE: [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup

2014-12-10 Thread KY Srinivasan
> -Original Message- > From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On > Behalf Of gre...@linuxfoundation.org > Sent: Wednesday, December 10, 2014 6:48 AM > To: Vitaly Kuznetsov > Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux- > ker...@vger.kernel.org > Sub

[PATCH] staging: vt6656: whitespace checkpatch warning fix.

2014-12-10 Thread Krzysztof Adamski
This fixes following checkpatch.pl error: ERROR: space prohibited before that ',' (ctx:WxW) Signed-off-by: Krzysztof Adamski --- drivers/staging/vt6656/rxtx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rxtx.c b/drivers/staging/vt6656/rxtx.c inde

[PATCH 2/2] staging: lustre: Fix trailing statement error in libcfs_string.c

2014-12-10 Thread Matthew Tyler
This patch fixes a trailing statement warning found by checkpatch.pl Signed-off-by: Matthew Tyler --- drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c b/drivers/st

[PATCH 1/2] staging: lustre: Fix minor style error in libcfs_string.c

2014-12-10 Thread Matthew Tyler
This patch adds spacing around an '=' operator Signed-off-by: Matthew Tyler --- drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c b/drivers/staging/lustre/lustre/libcfs/li

Re: [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup

2014-12-10 Thread gre...@linuxfoundation.org
On Wed, Dec 10, 2014 at 10:22:40AM +0100, Vitaly Kuznetsov wrote: > Dexuan Cui writes: > > >> -Original Message- > >> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > >> Sent: Tuesday, December 9, 2014 23:48 PM > >> To: KY Srinivasan > >> Cc: Haiyang Zhang; de...@linuxdriverproject.o

Re: [PATCH] staging: lustre: Fix the warning messages about casting without __user macro

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 07:09:59AM +, Al Viro wrote: > On Tue, Dec 09, 2014 at 10:56:12PM -0800, Shalin Mehta wrote: > > From: Shalin Mehta > > > > This issue is showed up while compiling with sparse. The iov_base in struct > > iovec struct explicitly declares that the assigned value should

Re: [PATCH] staging: rts5208: Removed the new typedefs

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 07:24:16PM +0530, Anjana Sasindran wrote: > This patch fix the checkpatch.pl warning: > > WARNING: do not add new typedefs > > Signed-off-by: Anjana Sasindran > --- > drivers/staging/rts5208/rtsx_sys.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --g

[PATCH] staging: rts5208: Removed the new typedefs

2014-12-10 Thread Anjana Sasindran
This patch fix the checkpatch.pl warning: WARNING: do not add new typedefs Signed-off-by: Anjana Sasindran --- drivers/staging/rts5208/rtsx_sys.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rts5208/rtsx_sys.h b/drivers/staging/rts5208/rtsx_sys.h index 0b

[PATCH] staging: fpga manager: fix array_size.cocci warnings

2014-12-10 Thread kbuild test robot
drivers/staging/fpga/altera.c:368:33-34: WARNING: Use ARRAY_SIZE Use ARRAY_SIZE instead of dividing sizeof array with sizeof an element Semantic patch information: This makes an effort to find cases where ARRAY_SIZE can be used such as where there is a division of sizeof the array by the sizeo

[PATCH] staging: fpga manager: fix platform_no_drv_owner.cocci warnings

2014-12-10 Thread kbuild test robot
drivers/staging/fpga/altera.c:768:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci CC: Alan Tull Signed-off-by: Fengguang Wu --- altera.c |1 - 1 file

[PATCH v2] staging: writeboost: Add dm-writeboost

2014-12-10 Thread Akira Hayakawa
This patch adds dm-writeboost to staging tree. dm-writeboost is a log-structured SSD-caching driver. It caches data in log-structured way on the cache device so that the performance is maximized. The merit of putting this driver in staging tree is to make it possible to get more feedback from use

Re: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Vitaly Kuznetsov
Dexuan Cui writes: >> -Original Message- >> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] >> Sent: Tuesday, December 9, 2014 21:06 PM >> To: Dexuan Cui >> Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- >> de...@linuxdriverproject.org; o...@aepfle.de; a...@c

[PATCH v2] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Dexuan Cui
In the case the host only injects an IPv6 address, the dhcp_enabled flag is true (it's only for IPv4 according to Hyper-V host team), but we still need to proceed to parse the IPv6 information. Cc: Vitaly Kuznetsov Cc: K. Y. Srinivasan Signed-off-by: Dexuan Cui --- v2: removed the distro-speci

RE: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Dexuan Cui
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Wednesday, December 10, 2014 17:08 PM > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > jasow...

Re: [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup

2014-12-10 Thread Vitaly Kuznetsov
Dexuan Cui writes: >> -Original Message- >> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] >> Sent: Tuesday, December 9, 2014 23:48 PM >> To: KY Srinivasan >> Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- >> ker...@vger.kernel.org; Dexuan Cui >> Subject: [PATCH 0/5] Tools:

Re: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Vitaly Kuznetsov
Dexuan Cui writes: > Thanks, > -- Dexuan > >> -Original Message- >> From: Dexuan Cui >> Sent: Wednesday, December 10, 2014 15:34 PM >> To: 'Vitaly Kuznetsov' >> Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- >> de...@linuxdriverproject.org; o...@aepfle.de; a...@c