Re: [PATCH] Staging: lustre: Fix warning detected by coccicheck

2015-09-26 Thread Sudip Mukherjee
On Sun, Sep 27, 2015 at 11:15:21AM +0530, Jayavant Kenjalkar wrote: > From: Jayavant Kenjalkar You do not need this From line here. This is needed only for those who can not configure their corporate mail servers. regards sudip ___ devel mailing list de

Re: [PATCH] Removed extra spaces, and error from checkpatch.pl

2015-09-26 Thread Sudip Mukherjee
On Sun, Sep 27, 2015 at 05:44:16AM +0530, Pinkesh Badjatiya wrote: > Removed trailing spaces. > Fixed some errors from checkpatch.pl verification. > No changes made to actual functional code. > > Signed-off-by: Pinkesh Badjatiya > --- And you have done more than one change in this patch. Please d

[PATCH] Staging: lustre: Fix warning detected by coccicheck

2015-09-26 Thread Jayavant Kenjalkar
From: Jayavant Kenjalkar Removing redundant semicolon as detected by coccicheck. Signed-off-by: Jayavant Kenjalkar --- drivers/staging/lustre/lustre/ptlrpc/niobuf.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ptlrpc/niobuf.c b/drivers/s

Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging

2015-09-26 Thread Mike Rapoport
Hi Sudip, On Fri, Sep 25, 2015 at 07:12:06PM +0530, Sudip Mukherjee wrote: > On Thu, Sep 24, 2015 at 02:58:31PM +0300, Tomi Valkeinen wrote: > > > > On 02/09/15 15:48, Sudip Mukherjee wrote: > > > > > > > Is there a public datasheet for the HW? > I could not find one in Silicon Motion website.

Re: [PATCH] Removed extra spaces, and error from checkpatch.pl

2015-09-26 Thread Jes Sorensen
Pinkesh Badjatiya writes: > Removed trailing spaces. > Fixed some errors from checkpatch.pl verification. > No changes made to actual functional code. > > Signed-off-by: Pinkesh Badjatiya > --- > drivers/staging/rtl8723au/include/rtw_cmd.h | 162 > ++-- > 1 file changed,

[PATCH] Removed extra spaces, and error from checkpatch.pl

2015-09-26 Thread Pinkesh Badjatiya
Removed trailing spaces. Fixed some errors from checkpatch.pl verification. No changes made to actual functional code. Signed-off-by: Pinkesh Badjatiya --- drivers/staging/rtl8723au/include/rtw_cmd.h | 162 ++-- 1 file changed, 80 insertions(+), 82 deletions(-) diff --gi

Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-26 Thread Joshua Clayton
On Saturday, September 26, 2015 11:24:06 PM punit vara wrote: > On Sat, Sep 26, 2015 at 11:15 PM, Larry Finger > wrote: > > On 09/26/2015 11:49 AM, Punit Vara wrote: > >> > >> This patch is to the rtl871x_ioctl_linux.c that fixes up following > >> warning reported by checkpatch.pl : > >> > >> - Co

Re: [PATCH V2] staging: rtl8723au: Remove unneeded endianness conversions

2015-09-26 Thread Larry Finger
On 09/24/2015 02:11 AM, Lars Svensson wrote: Fixing Sparse warnings in rtw_security.c. When checking crc, both actual and expected value was converted to cpu endianness before comparing, causing sparse warnings as below. Since the values are read from the buffer in correct byte order the extra co

Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-26 Thread punit vara
On Sat, Sep 26, 2015 at 11:15 PM, Larry Finger wrote: > On 09/26/2015 11:49 AM, Punit Vara wrote: >> >> This patch is to the rtl871x_ioctl_linux.c that fixes up following >> warning reported by checkpatch.pl : >> >> - Comparisons should place the constant on the right side of the test >> >> Signed

Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-26 Thread Larry Finger
On 09/26/2015 11:49 AM, Punit Vara wrote: This patch is to the rtl871x_ioctl_linux.c that fixes up following warning reported by checkpatch.pl : - Comparisons should place the constant on the right side of the test Signed-off-by: Punit Vara This warning is crap. WTF difference does it make??

Re: [PATCH 1/2] Staging: iio: cdc: Prefer using the BIT macro

2015-09-26 Thread Lars-Peter Clausen
On 09/25/2015 07:23 PM, Shraddha Barke wrote: > Replace bit shifting on 1 with the BIT(x) macro > > Signed-off-by: Shraddha Barke Acked-by: Lars-Peter Clausen ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org

Re: [PATCH 2/2] Staging: iio: cdc: Remove unnecessary dev_info

2015-09-26 Thread Lars-Peter Clausen
On 09/25/2015 07:23 PM, Shraddha Barke wrote: > Remove dev_info as the information can be obtained by other means > > Signed-off-by: Shraddha Barke Acked-by: Lars-Peter Clausen ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linu

[GIT PULL] Staging driver fixes for 4.3-rc3

2015-09-26 Thread Greg KH
The following changes since commit 6ff33f3902c3b1c5d0db6b1e2c70b6d76fba357f: Linux 4.3-rc1 (2015-09-12 16:35:56 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/ tags/staging-4.3-rc3 for you to fetch changes up to 74c600e36455a

[PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

2015-09-26 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c that fixes up following warning reported by checkpatch.pl : - Comparisons should place the constant on the right side of the test Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 16 1 file changed, 8 inser

Re: [x264-devel] Help to debug h264 headers (or video) generation in kernel driver

2015-09-26 Thread Andrey Utkin
On Sat, Sep 26, 2015 at 10:44 AM, Max Lapshin wrote: > Do you get some packets from this device or it is a bytestream? Oh hi a linux.org.ru buddy :) I get headerless binary data portions of known length, one portion for each encoded video frame. NAL headers generation is done on driver level in r

[PATCH] hyperv: Fix build failure with HYPERVISOR_GUEST=y && KEXEC_CORE=n

2015-09-26 Thread Ben Hutchings
Currently with this configuration I get: arch/x86/built-in.o: In function `hv_machine_crash_shutdown': mshyperv.c:(.text+0x29656): undefined reference to `native_machine_crash_shutdown' Fixes: b4370df2b1f5 ("Drivers: hv: vmbus: add special crash handler") Signed-off-by: Ben Hutchings Cc: Vitaly

Help to debug h264 headers (or video) generation in kernel driver

2015-09-26 Thread Andrey Utkin
I'm working on Linux kernel driver for multimedia grabber and H264 encoder based on Techwell/Intersil 5864 chip. Of course it will be open and pushed into upstream kernel. The device produces H264 encoded data, but it lacks any headers. The reference driver generates headers and glues frames toget