RE: [PATCH v3] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-28 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Friday, November 28, 2014 14:47 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan;

Re: [PATCH 1/2] mfd: rtsx: add func to split u32 into register

2014-11-28 Thread Dan Carpenter
On Fri, Nov 28, 2014 at 02:10:36AM +, 敬锐 wrote: eg, in sd_ops.c the cmd.arg is constructed bit by bit, we can put the right byte to the right register by shift, so the endian check is not need. I looked at drivers/mmc/core/sd_ops.c and cmd.arg seems to be cpu endian. The new function

Re: [PATCH v13 07/12] drm: bridge/dw_hdmi: add support for multi-byte register width access

2014-11-28 Thread Andy Yan
Hi Zabel: On 2014年11月27日 00:34, Philipp Zabel wrote: Am Mittwoch, den 26.11.2014, 21:32 +0800 schrieb Andy Yan: On rockchip rk3288, only word(32-bit) accesses are permitted for hdmi registers. Byte width accesses (writeb, readb) generate an imprecise external abort. Signed-off-by: Andy Yan

Re: [PATCH v13 08/12] drm: bridge/dw_hdmi: add mode_valid support

2014-11-28 Thread Andy Yan
Hi Philipp: On 2014年11月27日 00:23, Philipp Zabel wrote: Am Mittwoch, den 26.11.2014, 21:33 +0800 schrieb Andy Yan: some platform may not support all the display mode, add mode_valid interface check it also add drm_connector_register which add a debugfs interface for dump display modes and edid

Re: [PATCH v2] staging: octeon: Fix checkpatch warnings

2014-11-28 Thread Dan Carpenter
On Thu, Nov 27, 2014 at 05:18:10PM +0100, Luis de Bethencourt wrote: Fixing 80 character limit warnings in octeon/ethernet-rx.c Signed-off-by: Luis de Bethencourt l...@debethencourt.com --- drivers/staging/octeon/ethernet-rx.c | 51 +--- 1 file changed, 35

Re: [PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-28 Thread Dan Carpenter
On Thu, Nov 27, 2014 at 05:42:06PM +0530, Athira Lekshmi C V wrote: C V is my initals . Can I just go with the name Athira Lekshmi ? Sign it like you would sign a legal document. In American, that's normally Firstname Lastname but if you normally sign a legal document with C V on the end in

Re: [PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-28 Thread Athira Lekshmi C V
Hello I sign the documents as Athira Lekshmi. Thanking You Athira Lekshmi ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH] Drivers:staging:rtl8723au:core: Fixed checkpatch error

2014-11-28 Thread Jes Sorensen
Athira Lekshmi andnlnb...@gmail.com writes: Fixed the checkpatch error: ERROR: spaces required around that '' Signed-off-by: Athira Lekshmi andnlnb...@gmail.com --- drivers/staging/rtl8723au/core/rtw_cmd.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] Drivers:staging:rtl8192e: Fixed checkpatch warning

2014-11-28 Thread Athira Lekshmi
Fixed checkpatch warning: WARNING: space prohibited before semicolon Signed-off-by: Athira Lekshmi andnlnb...@gmail.com --- drivers/staging/rtl8192e/rtllib_rx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192e/rtllib_rx.c

Re: [PATCH v2] staging: octeon: Fix checkpatch warnings

2014-11-28 Thread Dan Carpenter
On Fri, Nov 28, 2014 at 02:30:34PM +0100, Luis de Bethencourt wrote: On Fri, Nov 28, 2014 at 01:29:28PM +0300, Dan Carpenter wrote: On Thu, Nov 27, 2014 at 05:18:10PM +0100, Luis de Bethencourt wrote: Fixing 80 character limit warnings in octeon/ethernet-rx.c Signed-off-by: Luis de

Re: [PATCH v2] staging: octeon: Fix checkpatch warnings

2014-11-28 Thread Luis de Bethencourt
On Fri, Nov 28, 2014 at 05:14:32PM +0300, Dan Carpenter wrote: On Fri, Nov 28, 2014 at 02:30:34PM +0100, Luis de Bethencourt wrote: On Fri, Nov 28, 2014 at 01:29:28PM +0300, Dan Carpenter wrote: On Thu, Nov 27, 2014 at 05:18:10PM +0100, Luis de Bethencourt wrote: Fixing 80 character

Re: [PATCH 1/2] mfd: rtsx: add func to split u32 into register

2014-11-28 Thread Dan Carpenter
Oh. You're right. I'm sorry for the noise. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

RE: [HPDD-discuss] [PATCH] staging: lustre: fix sparse warnings related to lock context imbalance

2014-11-28 Thread Patrick Farrell
Dan, I disagree about the change suggested here. In this particular code, 'object_attr' is distinct from 'attr', as in a 'setattr' call on an inode. 'cl_object' is a distinct thing from an inode/file on disk, and specifying it is the objects attr is helpful in understanding there is not a

Re: [PATCH v13 0/12] dw-hdmi: convert imx hdmi to bridge/dw_hdmi

2014-11-28 Thread Philipp Zabel
Am Freitag, den 28.11.2014, 17:57 +0800 schrieb Andy Yan: Hi Philipp: [...] Very sorry about this trouble. Because I have no imx board, we do all the test on RK3288 board. I had sent a mail with a debug patch to you directly yesterday, hope it will helpful. If you have received the

Re: [HPDD-discuss] [PATCH] staging: lustre: fix sparse warnings related to lock context imbalance

2014-11-28 Thread Dan Carpenter
On Fri, Nov 28, 2014 at 03:45:24PM +, Patrick Farrell wrote: Dan, I disagree about the change suggested here. In this particular code, 'object_attr' is distinct from 'attr', as in a 'setattr' call on an inode. 'cl_object' is a distinct thing from an inode/file on disk, and specifying

[PATCH] staging:drivers:staging:rtl8188eu:hal:rtl8188e_hal_init.c: Added a blank line after declaration

2014-11-28 Thread Anjana Sasindran
This patch fix a checkpatch.pl warning WARNING:Missing blank line after declaration Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com --- drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH v2 4/9] staging: panel: Use defined value or checking module params state

2014-11-28 Thread Greg Kroah-Hartman
On Thu, Nov 27, 2014 at 08:50:55PM +0100, Mariusz Gorski wrote: On Thu, Nov 27, 2014 at 07:57:06AM -0800, Greg Kroah-Hartman wrote: On Thu, Nov 27, 2014 at 07:24:17AM -0800, Greg Kroah-Hartman wrote: On Thu, Nov 27, 2014 at 02:26:59PM +0100, Mariusz Gorski wrote: On Wed, Nov 26, 2014 at

Re: [PATCH v2 4/9] staging: panel: Use defined value or checking module params state

2014-11-28 Thread Mariusz Gorski
On Fri, Nov 28, 2014 at 12:32:48PM -0800, Greg Kroah-Hartman wrote: On Thu, Nov 27, 2014 at 08:50:55PM +0100, Mariusz Gorski wrote: On Thu, Nov 27, 2014 at 07:57:06AM -0800, Greg Kroah-Hartman wrote: On Thu, Nov 27, 2014 at 07:24:17AM -0800, Greg Kroah-Hartman wrote: On Thu, Nov 27, 2014

Re: [PATCH v2 4/9] staging: panel: Use defined value or checking module params state

2014-11-28 Thread Greg Kroah-Hartman
On Fri, Nov 28, 2014 at 09:57:06PM +0100, Mariusz Gorski wrote: I tried to apply the 3rd, but it didn't apply due to patches I applied in your first set of 4 patches. Does that help? Yes, thanks. I've resent the patchset yesterday as v3: https://lkml.org/lkml/2014/11/27/805 It

Re: [PATCH] staging: lustre: fix sparse warnings related to lock context imbalance

2014-11-28 Thread Greg KH
On Thu, Nov 27, 2014 at 07:34:10PM +0100, Loïc Pefferkorn wrote: Hello Greg, After some investigation, I think that removing these wrappers is not going to improve the code readability: On Wed, Nov 26, 2014 at 12:54:43PM -0800, Greg KH wrote: On Wed, Nov 26, 2014 at 05:15:48PM +0100,

[GIT PULL] Staging driver fixes for 3.18-rc7

2014-11-28 Thread Greg KH
The following changes since commit 206c5f60a3d902bc4b56dab2de3e88de5eb06108: Linux 3.18-rc4 (2014-11-09 14:55:29 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/ tags/staging-3.18-rc7 for you to fetch changes up to

[PATCH] staging: ion : ion_cma_heap: fix bug for highmem cma

2014-11-28 Thread Zeng Tao
when cma is located in highmem, virt_to_page will not work the right way, use pfn_to_page instead. Signed-off-by: Zeng Tao prime.z...@huawei.com --- drivers/staging/android/ion/ion_cma_heap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] Staging: ft1000 : ft1000-pcmcia: replace __attribute ((__packed__) with __packed macro per coding style guidelines in ft1000_dnld.c

2014-11-28 Thread Geoff Darst
Replace two instances of __attribute ((__packed__) with __packed macro to address the warning found by the checkpatch.pl tool. Signed-off-by: Geoff Darst geof...@comcast.net --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

Re: [PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-28 Thread Athira Lekshmi C V
Hello I have resend the patch signing off as Athira Lekshmi. Thanking You Athira Lekshmi ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-28 Thread Athira Lekshmi
Fixed the checkpatch warning: Missing a blank line after declarations Signed-off-by: Athira Lekshmi andnlnb...@gmail.com --- drivers/staging/octeon/ethernet-rx.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/octeon/ethernet-rx.c b/drivers/staging/octeon/ethernet-rx.c