Re: [PATCH 0/2] make imx hdmi publicly used by dw hdmi compatible platform

2014-11-06 Thread Kuankuan.Yang
Hi Russell I'm working on Designware hdmi-audio, also add it as a standard ALSA device. Before I saw this email, I also planed to submit my patchs to upsteam. I'm very grateful if you can email those patchs to us. Best Regards. 于 2014年11月04日 22:29, Russell King - ARM Linux 写道: On Tue, Nov 04,

[PATCH v2 03/10] [media] Make use of the new media_bus_format definitions

2014-11-06 Thread Boris Brezillon
Replace references to the v4l2_mbus_pixelcode enum with the new media_bus_format enum in all common headers. Signed-off-by: Boris Brezillon boris.brezil...@free-electrons.com Acked-by: Sakari Ailus sakari.ai...@linux.intel.com --- include/media/v4l2-mediabus.h| 2 +-

[PATCH v2 00/10] [media] Make mediabus format subsystem neutral

2014-11-06 Thread Boris Brezillon
Hello, This patch series prepares the use of media bus formats outside of the V4L2 subsytem (my final goal is to use it in the Atmel HLCDC DRM driver where I have to configure my DPI/RGB bus according to the connected display). The series first defines a new enum with a neutral name

[PATCH v2 01/10] [media] Move mediabus format definition to a more standard place

2014-11-06 Thread Boris Brezillon
Rename mediabus formats and move the enum into a separate header file so that it can be used by DRM/KMS subsystem without any reference to the V4L2 subsystem. Old v4l2_mbus_pixelcode now points to media_bus_format. Signed-off-by: Boris Brezillon boris.brezil...@free-electrons.com Acked-by:

[PATCH v2 02/10] [media] v4l: Update subdev-formats doc with new MEDIA_BUS_FMT values

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Update the v4l documentation accordingly. Signed-off-by: Boris Brezillon

[PATCH v2 04/10] [media] i2c: Make use of media_bus_format enum

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definitions to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Replace all references to the old definitions in i2c drivers. Signed-off-by: Boris

[PATCH v2 07/10] [media] usb: Make use of media_bus_format enum

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all usb drivers. Signed-off-by: Boris Brezillon

[PATCH v2 09/10] gpu: ipu-v3: Make use of media_bus_format enum

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in the ipu-v3 driver. Signed-off-by: Boris Brezillon

[PATCH v2 05/10] [media] pci: Make use of media_bus_format enum

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Replace all references to the old definition in pci drivers. Signed-off-by: Boris

[PATCH v2 10/10] [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the kernel

2014-11-06 Thread Boris Brezillon
Place v4l2_mbus_pixelcode in a #ifndef __KERNEL__ section so that kernel users don't have access to these definitions. We have to keep this definition for user-space users even though they're encouraged to move to the new media_bus_format enum. Signed-off-by: Boris Brezillon

[PATCH v2 08/10] staging: media: Make use of media_bus_format enum

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all media drivers residing in staging. Signed-off-by: Boris

[PATCH v2 06/10] [media] platform: Make use of media_bus_format enum

2014-11-06 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all platform drivers. Signed-off-by: Boris Brezillon

Re: [PATCH 0/2] make imx hdmi publicly used by dw hdmi compatible platform

2014-11-06 Thread Russell King - ARM Linux
On Thu, Nov 06, 2014 at 05:35:40PM +0800, Kuankuan.Yang wrote: I'm working on Designware hdmi-audio, also add it as a standard ALSA device. Before I saw this email, I also planed to submit my patchs to upsteam. I'm very grateful if you can email those patchs to us. I've attached the set of

[PATCH 1/4] staging: dgap: set tty's flags by tty_alloc_driver()

2014-11-06 Thread Daeseok Youn
tty's flags can be set by calling tty_alloc_driver(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c | 16 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index

[PATCH 2/4] staging: dgap: fix memory leak caused by double allocation of tty_structs

2014-11-06 Thread Daeseok Youn
The tty_struct of serial_driver and print_driver were getting allocated twice. One is allocated in tty_alloc_driver(), the other is in dgap_tty_register(). So remove these in dgap_tty_register(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c | 16

[PATCH 3/4] staging: dgap: remove useless variables for saving tty's major

2014-11-06 Thread Daeseok Youn
The board_t has a tty_struct(serial_driver and print_driver) that has a major number. When major number is compared in dgap_tty_open(), just use brd-serial_driver{print_driver}-major. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c |9 ++---

[PATCH 4/4] staging: dgap: introducing find_board_by_major()

2014-11-06 Thread Daeseok Youn
use find_board_by_major function instead of getting a brd from static arrary. Becasue tty's major number doesn't start zero and we can find a brd from dgap_board[]. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c | 28 +--- 1 files

Re: [PATCH v2 01/13] staging: comedi: drivers: introduce comedi_nscans_left()

2014-11-06 Thread Ian Abbott
On 05/11/14 17:31, H Hartley Sweeten wrote: Introduce a helper function to determine the number of scans left in the async command. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org ---

[PATCH V3 0/4] dw-hdmi: make imx hdmi publicly used by dw hdmi compatible platform

2014-11-06 Thread Andy Yan
We found freescale imx6 and rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly difference, such as phy pll configuration, register width(imx hdmi register is one byte, but rk3288 is 4 bytes width and can only access by

[PATCH V3 1/4] imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-06 Thread Andy Yan
imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly difference, such as phy pll configuration, register width, 4K support, clk useage, and the crtc mux configuration is also platform specific. To reuse the imx

[PATCH V3 2/4] dw-hdmi: move imx-hdmi to bridge/dw-hdmi

2014-11-06 Thread Andy Yan
the original imx hdmi driver is under staging/imx-drm, which depends on imx-drm, so move the imx hdmi drvier out to drm/bridge and rename imx-hdmi to dw-hdmi Signed-off-by: Andy Yan andy@rock-chips.com --- drivers/gpu/drm/bridge/Kconfig | 5 +

[PATCH V3 3/4] dw-hdmi: add support for multi byte register width access

2014-11-06 Thread Andy Yan
On rockchip rk3288, only word(32-bit) accesses are permitted for hdmi registers. Byte width access (writeb, readb) generate an imprecise external abort. Signed-off-by: Andy Yan andy@rock-chips.com --- drivers/gpu/drm/bridge/dw_hdmi.c | 53 1 file

[PATCH V3 4/4] dw-hdmi: convert dw-hdmi to drm_bridge mode

2014-11-06 Thread Andy Yan
From: ykk y...@rock-chips.com dw-hdmi is under drm/bridge, so it should be the bridge mode. hange off the encoder to dw_hdmi-imx.c, keep the connector birdge in dw_hdmi.c Signed-off-by: ykk y...@rock-chips.com Signed-off-by: Andy Yan andy@rock-chips.com --- drivers/gpu/drm/bridge/dw_hdmi.c

re: Staging: rtl8188eu: Use put_unaligned_le32

2014-11-06 Thread Dan Carpenter
Hello Vaishali Thakkar, The patch 2b365fa9d926: Staging: rtl8188eu: Use put_unaligned_le32 from Oct 31, 2014, leads to the following static checker warning: include/linux/unaligned/access_ok.h:44 put_unaligned_le32() warn: potential memory corrupting cast 4 vs 3 bytes

Re: [PATCH 1/4] [media] solo6x10: free vb2 buffers on stop_streaming

2014-11-06 Thread Dan Carpenter
On Wed, Oct 29, 2014 at 08:03:51PM +0400, Andrey Utkin wrote: This fixes warning from drivers/media/v4l2-core/videobuf2-core.c:2144 On my kernel that line is: q-start_streaming_called = 0; This changelog is useless. regards, dan carpenter

[PATCH] staging: unisys: Remove unnecessary spaces after casts in virthba.c

2014-11-06 Thread Ken Depro
From: Ken Depro kenneth.de...@unisys.com This patch removes unneeded spaces after casts within the virthba.c file. The checkpatch script was run after these changes, and no remove spaces after casts checks were generated. Later patches will fix the other checkpatch warnings/checks. Conflicts:

Re: [PATCH] staging:rtl8723au: core: Fix error reported by checkpatch.

2014-11-06 Thread Greg KH
Please specify the error in the subject in some way. On Thu, Nov 06, 2014 at 11:46:13AM +0530, Sanjeev Sharma wrote: This is a patch to the rtw_cmd.c file that fixes following error. ERROR: spaces required around that '' (ctx:WxV) ERROR: that open brace { should be on the previous line

Re: [PATCH] staging:rtl8723au: core: Fix Warning reported by checkpatch.

2014-11-06 Thread Greg KH
On Thu, Nov 06, 2014 at 12:06:36PM +0530, Sanjeev Sharma wrote: This is a patch to the rtw_cmd.c file that fixes following Warning by introducing temporary structure. WARNING: line over 80 characters Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com ---

Re: [PATCH] staging: unisys: Remove unnecessary spaces after casts in virthba.c

2014-11-06 Thread Dan Carpenter
On Thu, Nov 06, 2014 at 10:33:28AM -0500, Ken Depro wrote: - LOGINF(virtpcidev bus_no%ddevNo%d, virtpcidev-bus_no, -virtpcidev-device_no); - virthbainfo = (struct virthba_info *) scsihost-hostdata; + LOGINF(virtpcidev busNo%ddevNo%d, virtpcidev-busNo, +

Re: [PATCH V3 4/4] dw-hdmi: convert dw-hdmi to drm_bridge mode

2014-11-06 Thread Greg Kroah-Hartman
On Thu, Nov 06, 2014 at 07:26:16PM +0800, Andy Yan wrote: From: ykk y...@rock-chips.com We need a real name here, sorry. dw-hdmi is under drm/bridge, so it should be the bridge mode. hange off the encoder to dw_hdmi-imx.c, keep the connector birdge in dw_hdmi.c Signed-off-by: ykk

Re: Staging: rtl8188eu: Use put_unaligned_le32

2014-11-06 Thread Dan Carpenter
On Thu, Nov 06, 2014 at 09:18:44PM +0530, Vaishali Thakkar wrote: One more thing, as previously for me it was not showing any error could you please tell me what can be the reason behind it and how can I check my revised code?? [I am newbie to linux kernel so I it would be nice of you if you

Re: [PATCH] staging: unisys: Remove unnecessary spaces after casts in virthba.c

2014-11-06 Thread Greg KH
On Thu, Nov 06, 2014 at 10:33:28AM -0500, Ken Depro wrote: From: Ken Depro kenneth.de...@unisys.com This patch removes unneeded spaces after casts within the virthba.c file. The checkpatch script was run after these changes, and no remove spaces after casts checks were generated. Later

RE: [PATCH] staging: unisys: Remove unnecessary spaces after casts in virthba.c

2014-11-06 Thread Depro, Kenneth J
Dan, The CamelCase version is what is currently present in our source. I have not addressed/fixed those checks yet. With the members as busNo and deviceNo, the kernel builds cleanly. I'm not sure why they showed up as non-CamelCase in the diff. Ken -Original Message- From: Dan

RE: [PATCH] staging: unisys: Remove unnecessary spaces after casts in virthba.c

2014-11-06 Thread Depro, Kenneth J
Greg, When I executed the format-patch command, it added the conflict lines below. There was indeed a conflict in this file from yesterday (I had originally pulled/built from staging-next, but there was another pending patch of mine in staging-testing that was not in -next, which caused the

Re: [PATCH] staging: unisys: Remove unnecessary spaces after casts in virthba.c

2014-11-06 Thread Greg KH
A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Thu, Nov 06, 2014 at 10:22:30AM -0600, Depro, Kenneth J wrote: Greg, When I executed the format-patch command, it added the conflict lines below. There was indeed a conflict in this file

[PATCH v5 33/48] staging: nvec: Register with kernel power-off handler

2014-11-06 Thread Guenter Roeck
Register with kernel power-off handler instead of setting pm_power_off directly. Register with default priority since we don't know any better. Cc: Julian Andres Klode j...@jak-linux.org Cc: Marc Dietrich marvi...@gmx.de Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Acked-by: Greg

[PATCH 0/2] Drivers: hv: kvp, vss: improve kernel-userspace communication in failure case

2014-11-06 Thread Vitaly Kuznetsov
This series addresses two issues: - There is no timeout when communicating with userspace VSS daemon. - In case we fail to send a message to VSS or KVP userspace daemons we can report the failure to the host right away avoiding the timeout. Newly introduced 10 second timeout is something worth

[PATCH 2/2] Drivers: hv: kvp, vss: Fast propagation of userspace communication failure

2014-11-06 Thread Vitaly Kuznetsov
If we fail to send a message to userspace daemon with cn_netlink_send() there is no need to wait for userspace to reply as it is not going to happen. This happens when kvp or vss daemon is stopped after a successful handshake. Report HV_E_FAIL immediately and cancel the timeout job so host won't

[PATCH 1/2] Drivers: hv: vss: Introduce timeout for communication with userspace

2014-11-06 Thread Vitaly Kuznetsov
In contrast with KVP there is no timeout when communicating with userspace VSS daemon. In case it gets stuck performing freeze/thaw operation no message will be sent to the host so it will take very long (around 10 minutes) before backup fails. Introduce 10 second timeout using

[PATCH] staging: unisys: Fix CamelCase params in SignalInsert_withLock

2014-11-06 Thread Ken Depro
From: Ken Depro kenneth.de...@unisys.com This patch renames the following SignalInsert_withLock parameters to fix the CamelCase checks: pChannel to pchannel Queue to queue pSignal to psignal Signed-off-by: Ken Depro kenneth.de...@unisys.com --- drivers/staging/unisys/channels/chanstub.c |6

lustre: Checking the minimum size of buffers

2014-11-06 Thread Dan Carpenter
A couple weeks ago I found the bug 87ebccf97f54 ('staging: lustre: validate size in ll_setxattr()') by manual audit but now I have written a Smatch check to find bugs like that automatically. The results look to be fairly high quality generally except in lustre. These have a potential security

Re: [PATCH] staging: unisys: Fix CamelCase params in SignalInsert_withLock

2014-11-06 Thread Denis Kirjanov
On 11/6/14, Dan Carpenter dan.carpen...@oracle.com wrote: On Thu, Nov 06, 2014 at 12:50:10PM -0500, Ken Depro wrote: From: Ken Depro kenneth.de...@unisys.com This patch renames the following SignalInsert_withLock parameters to fix the CamelCase checks: pChannel to pchannel If you're going

Re: [PATCH] staging: unisys: Fix CamelCase params in SignalInsert_withLock

2014-11-06 Thread Dan Carpenter
On Thu, Nov 06, 2014 at 12:50:10PM -0500, Ken Depro wrote: From: Ken Depro kenneth.de...@unisys.com This patch renames the following SignalInsert_withLock parameters to fix the CamelCase checks: pChannel to pchannel If you're going to rename things, you may as well get rid of the p. (My

[PATCH] staging: unisys: add visorclientbus driver

2014-11-06 Thread Ken Cox
From: Benjamin Romer benjamin.ro...@unisys.com This patch adds the visorclientbus driver to the Unisys s-Par driver set. This driver is responsible for helping manage virtual devices like keyboards, mice, serial ports, displays, and any customized drivers for special-purpose guests.

RE: [PATCH] staging: unisys: Fix CamelCase params in SignalInsert_withLock

2014-11-06 Thread Depro, Kenneth J
If you're going to rename things, you may as well get rid of the p. And fix the function name as well... Denis, I had planned on fixing the function name in a separate patch since I thought that would constitute two separate work items and also generate a larger set of patches. From your

Re: [PATCH] staging: unisys: add visorclientbus driver

2014-11-06 Thread Greg KH
On Thu, Nov 06, 2014 at 12:53:32PM -0600, Ken Cox wrote: From: Benjamin Romer benjamin.ro...@unisys.com This patch adds the visorclientbus driver to the Unisys s-Par driver set. This driver is responsible for helping manage virtual devices like keyboards, mice, serial ports, displays, and

[PATCH 1/8] staging: unisys: visorchannel: Remove multiple blank lines

2014-11-06 Thread Bryan Thompson
Remove the instances of multiple blank lines in the visorchannel files. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- drivers/staging/unisys/visorchannel/globals.h |1 - .../unisys/visorchannel/visorchannel_funcs.c |1 - 2 files changed, 2 deletions(-) diff

[PATCH 4/8] staging: unisys: visorchannel: Remove unnecessary parentheses

2014-11-06 Thread Bryan Thompson
Remove extraneous parentheses around a variable in visorchannel_funcs.c Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 8/8] staging: unisys: visorchannel: Add braces to else arm of if/else statement

2014-11-06 Thread Bryan Thompson
Add braces to the else arm of an if/else block in visorchannel_funcs.c Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 6/8] staging: unisys: visorchannel: Adjust lines to contain a maximum of 80 characters

2014-11-06 Thread Bryan Thompson
Split multiple logging lines to get them closer to 80 characters per line. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git

[PATCH 2/8] staging: unisys: visorchannel: Fix alignment issues

2014-11-06 Thread Bryan Thompson
Use the appropriate whitespace for multiline statements in visorchannel_funcs.c Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git

[PATCH 5/8] staging: unisys: visorchannel: Place logical continuation at the end of a line

2014-11-06 Thread Bryan Thompson
Move the || logical continuation from the start of the second line of an if statement to the end of the first line. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH 7/8] staging: unisys: visorchannel: Rename goto label Away

2014-11-06 Thread Bryan Thompson
Rename the goto label used throughout visorchannel_funcs.c from Away to cleanup. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c | 54 ++-- 1 file changed, 27 insertions(+), 27 deletions(-) diff --git

[PATCH 3/8] staging: unisys: visorchannel: Remove space between cast and variable

2014-11-06 Thread Bryan Thompson
Remove the whitespace between a cast and the variable in visorchannel_funcs.c Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c | 22 ++-- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git

Re: [PATCH 2/5] drivers: serial: jsm: Add the Classic board implementation

2014-11-06 Thread Greg KH
On Mon, Nov 03, 2014 at 07:52:38PM +0100, Konrad Zapalowicz wrote: This commit adds the Digi Classic board implementation to the staging/jsm driver. The code here is taken from the staging/dgnc driver and modified to match the serial/jsm state. This work is mostly based on the changes that

Re: [PATCH] staging: unisys: add visorclientbus driver

2014-11-06 Thread Ken Cox
On 11/06/2014 01:05 PM, Greg KH wrote: On Thu, Nov 06, 2014 at 12:53:32PM -0600, Ken Cox wrote: From: Benjamin Romer benjamin.ro...@unisys.com This patch adds the visorclientbus driver to the Unisys s-Par driver set. This driver is responsible for helping manage virtual devices like

Re: [PATCH] staging: unisys: add visorclientbus driver

2014-11-06 Thread Greg KH
On Thu, Nov 06, 2014 at 01:17:56PM -0600, Ken Cox wrote: There are also other checkpatch errors in this patch, which makes me not want to take it at all, as you would now just be required to send more fixes for the file. So why not fix things up right the first time? The only checkpatch

Re: [PATCH] staging: unisys: Fix CamelCase params in SignalInsert_withLock

2014-11-06 Thread Denis Kirjanov
On 11/6/14, Depro, Kenneth J kenneth.de...@unisys.com wrote: If you're going to rename things, you may as well get rid of the p. And fix the function name as well... Denis, I had planned on fixing the function name in a separate patch since I thought that would constitute two separate work

[PATCH v2 1/4] [media] solo6x10: clean up properly in stop_streaming

2014-11-06 Thread Andrey Utkin
This fixes warning from drivers/media/v4l2-core/videobuf2-core.c, WARN_ON(atomic_read(q-owned_by_drv_count)). Signed-off-by: Andrey Utkin andrey.krieger.ut...@gmail.com --- drivers/media/pci/solo6x10/solo6x10-v4l2-enc.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff

Re: [PATCH] rtl8188eu: Fix several stylistic problems in rtw_led.*

2014-11-06 Thread Greg Kroah-Hartman
On Thu, Nov 06, 2014 at 11:56:05PM +0100, k...@konagma.com wrote: From: Krzysztof Konopko k...@konagma.com Several stylistics problems are reported by scripts/checkpatch.pl run on rtw_led.*: * FSF_MAILING_ADDRESS Free Software Foundation's mailing address should not be included in

[PATCH v2 0/4] serial: jsm: Add support for the Digi Classic adapters

2014-11-06 Thread Konrad Zapalowicz
The staging/dgnc and the serial/jsm drivers have the same origin. They come from the same codebase however the jsm is the community adopted version where the dgnc is kind of a direct submission from the Digi company. The recent commits to the jsm driver moved in the support for some of the Neo

[PATCH] rtl8188eu: Fix several stylistic problems in rtw_led.*

2014-11-06 Thread kris
From: Krzysztof Konopko k...@konagma.com Several stylistics problems are reported by scripts/checkpatch.pl run on rtw_led.*: * FSF_MAILING_ADDRESS Free Software Foundation's mailing address should not be included in the sample GPL notice. * LINE_SPACING Missing a blank line

[PATCH v2 2/4] drivers: serial: jsm: Enable support for Digi Classic adapters

2014-11-06 Thread Konrad Zapalowicz
This commit enables support for the Digi Classic adapters line in the jsm driver. This means changes in: - device probing code - device cleanup code - driver description (Kconfig) The init/cleanup code is based on the staging/dgnc driver. Signed-off-by: Konrad Zapalowicz

[PATCH v2 1/4] drivers: serial: jsm: Add the Classic board implementation

2014-11-06 Thread Konrad Zapalowicz
This commit adds the Digi Classic board implementation to the staging/jsm driver. The code here is taken from the staging/dgnc driver and modified to match the serial/jsm state. This work is mostly based on the changes that has been done to the code handling the Digi Neo cards with the

[PATCH v2 4/4] drivers: serial: jsm: Remove FSF address from the file documentation/header

2014-11-06 Thread Konrad Zapalowicz
This commit removes the address of Free Software Foundation from each of the mentioned file in order to suppress the checkpatch warning. Signed-off-by: Konrad Zapalowicz bergo.tor...@gmail.com --- drivers/tty/serial/jsm/jsm.h| 5 - drivers/tty/serial/jsm/jsm_cls.c| 5 -

Re: [PATCH 2/5] drivers: serial: jsm: Add the Classic board implementation

2014-11-06 Thread Konrad Zapalowicz
On 11/06, Greg KH wrote: On Mon, Nov 03, 2014 at 07:52:38PM +0100, Konrad Zapalowicz wrote: This commit adds the Digi Classic board implementation to the staging/jsm driver. The code here is taken from the staging/dgnc driver and modified to match the serial/jsm state. This work is

Re: [PATCH v2 3/4] staging: dgnc: Remove driver in favor of serial/jsm driver

2014-11-06 Thread Greg KH
On Fri, Nov 07, 2014 at 12:05:34AM +0100, Konrad Zapalowicz wrote: This commit removes the staging/dgnc driver in favor of the serial/jsm driver. This is because the serial/jsm driver now supports all of the hardware that has been supported by the staging/dgnc plus it offers better overall

[PATCH 2/3] rtl8188eu: Fix a typo in rtw_led.*

2014-11-06 Thread Krzysztof Konopko
A rather obvious typo in one of the identifier has been found. This patch fixes the typo and ensures any lines changed do not exceed 80 characters as indicated by scripts/checkpatch.pl Signed-off-by: Krzysztof Konopko k...@konagma.com --- drivers/staging/rtl8188eu/core/rtw_led.c| 6 --

[PATCH 1/3] rtl8188eu: Fix FSF_MAILING_ADDRESS in rtw_led.*

2014-11-06 Thread Krzysztof Konopko
rtw_led.* files include Free Software Foundation's mailing address in the sample GPL notice. This is not desired and picked when running scripts/checkpatch.pl. Signed-off-by: Krzysztof Konopko k...@konagma.com --- drivers/staging/rtl8188eu/core/rtw_led.c| 4

[PATCH 3/3] rtl8188eu: Remove leading spaces in rtw_led.c

2014-11-06 Thread Krzysztof Konopko
According to Linux coding convention leading spaces are not allowed. This patch removes leading spaces in rtw_led.c Signed-off-by: Krzysztof Konopko k...@konagma.com --- drivers/staging/rtl8188eu/core/rtw_led.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [PATCH v2 3/4] staging: dgnc: Remove driver in favor of serial/jsm driver

2014-11-06 Thread Konrad Zapalowicz
On 11/06, Greg KH wrote: On Fri, Nov 07, 2014 at 12:05:34AM +0100, Konrad Zapalowicz wrote: This commit removes the staging/dgnc driver in favor of the serial/jsm driver. This is because the serial/jsm driver now supports all of the hardware that has been supported by the staging/dgnc

Re: [PATCH V3 4/4] dw-hdmi: convert dw-hdmi to drm_bridge mode

2014-11-06 Thread Andy Yan
On 2014年11月06日 23:54, Greg Kroah-Hartman wrote: On Thu, Nov 06, 2014 at 07:26:16PM +0800, Andy Yan wrote: From: ykk y...@rock-chips.com We need a real name here, sorry. dw-hdmi is under drm/bridge, so it should be the bridge mode. hange off the encoder to dw_hdmi-imx.c, keep the connector

Re: [PATCH 1/2] imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-06 Thread Andy Yan
On 2014年11月05日 21:41, Philipp Zabel wrote: Hi Andy, I think separating the core from the SoC specific part is a good step into the right direction. Am Mittwoch, den 05.11.2014, 20:55 +0800 schrieb Andy Yan: imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS) use the interface

Re: [PATCH V2 1/2] imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-06 Thread Andy Yan
On 2014年11月05日 21:30, Zubair Lutfullah Kakakhel wrote: This one patch does too much to be reviewed easily. One patch is supposed to modify/add one thing at a time in the kernel. Separating platform specific code from imx-drm/imx-hdmi is one thing. Adding support for multi-byte register

Re: [PATCH v2 1/4] [media] solo6x10: clean up properly in stop_streaming

2014-11-06 Thread Dan Carpenter
On Fri, Nov 07, 2014 at 01:06:18AM +0400, Andrey Utkin wrote: This fixes warning from drivers/media/v4l2-core/videobuf2-core.c, WARN_ON(atomic_read(q-owned_by_drv_count)). Thanks! regards, dan carpenter ___ devel mailing list