On Wed, Mar 05, 2014 at 04:58:36PM +0900, DaeSeok Youn wrote:
Yes, it is not changed anything.
But I had a request of greg.
He want to resend my patches what he haven't applied to staging-next
branch because my patches had flushed from his own queue.
And I didn't know how to set the
Refactor this function to remove an extra indent level
Signed-off-by: Fred Akers kni...@botops.net
---
drivers/staging/comedi/range.c | 39 +++
1 file changed, 19 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/comedi/range.c
Testing for if (IS_ERR(padapter-cmdThread) 0) doesn't make sense.
The kthread_run() function returns error pointers on error pointers on
error so it should just be if (IS_ERR(padapter-cmdThread)).
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
diff --git
This patch fixes the DDC I2C bus property to use the common 'ddc-i2c-bus'
property name instead of 'ddc'. This is already documented in
Documentation/devicetree/bindings/staging/imx-drm/hdmi.txt
Signed-off-by: Philipp Zabel p.za...@pengutronix.de
---
drivers/staging/imx-drm/imx-hdmi.c | 2 +-
1
Hi,
this latest version of the imx-drm DT binding patches applies
on top of staging-next and also depends on the OF graph binding
patchset that moves the v4l2_of helpers to drivers/of.
Currently, the two patchsets are also available at:
git://git.pengutronix.de/git/pza/linux.git
This patch connects IPU and display encoder (VGA, LVDS)
device tree nodes, as well as parallel displays on the DISP0
and DISP1 outputs, using the OF graph bindings described in
Documentation/devicetree/bindings/media/video-interfaces.txt
The IPU ports correspond to the two display interfaces. The
This patch connects IPU and display encoder (HDMI, LVDS, MIPI)
device tree nodes, as well as parallel displays on the DISP0
and DISP1 outputs, using the OF graph bindings described in
Documentation/devicetree/bindings/media/video-interfaces.txt
The IPU ports correspond to the two display
This patch updates the device tree binding documentation for i.MX IPU/display
nodes using the OF graph bindings documented in
Documentation/devicetree/bindings/media/video-interfaces.txt.
Signed-off-by: Philipp Zabel p.za...@pengutronix.de
---
Changes since v4:
- changed DT compatible string to
This patch connects IPU and and parallel display device tree
nodes using the OF graph bindings described in
Documentation/devicetree/bindings/media/video-interfaces.txt
The IPU ports correspond to the two display interfaces. The
order of endpoints in the ports is arbitrary.
Since the imx-drm
The device tree bindings are updated regardless of the common display
framework and in the meantime the HDMI driver was included.
Signed-off-by: Philipp Zabel p.za...@pengutronix.de
---
drivers/staging/imx-drm/TODO | 5 -
1 file changed, 5 deletions(-)
diff --git
This patch fixes the Television Encoder node's DDC I2C bus property to
use the common property name of 'ddc-i2c-bus' instead of just 'ddc'.
Signed-off-by: Philipp Zabel p.za...@pengutronix.de
---
arch/arm/boot/dts/imx53-mba53.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
From: Philipp Zabel philipp.za...@gmail.com
This patch adds support to find the involved components connected to
the IPU display interface ports using the OF graph bindings documented
in Documentation/devicetree/bindings/media/video-interfaces.txt.
It makes use of the of_graph (formerly v4l2_of)
Thank you for your kind explanation.
I will do that when my patch is sent again.
regards,
Daeseok Youn.
2014-03-05 17:08 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com:
On Wed, Mar 05, 2014 at 04:58:36PM +0900, DaeSeok Youn wrote:
Yes, it is not changed anything.
But I had a request of
On Tue, Mar 04, 2014 at 09:10:41PM +0200, Valentina Manea wrote:
+int write_sysfs_attribute(const char *attr_path, const char *new_value,
+ size_t len)
+{
+ int fd;
+ int length;
+
+ if (attr_path == NULL || new_value == NULL || len == 0) {
Can any of
On Wed, Mar 05, 2014 at 10:20:52AM +0100, Philipp Zabel wrote:
+struct imx_drm_component {
+ struct device_node *of_node;
+ struct list_head list;
+};
+
The only thing this structure appears to be doing is ensuring that a
single component doesn't get added twice - is that correct?
On 2014-03-05 at 03:37:15 +0100, Daeseok Youn daeseok.y...@gmail.com wrote:
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com
---
drivers/staging/cxt1e1/sbecom_inline_linux.h |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git
On 2014-03-05 at 02:24:22 +0100, Daeseok Youn daeseok.y...@gmail.com wrote:
It doesn't need to assign name array address to np pointer.
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com
---
drivers/staging/cxt1e1/linux.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
On Tue, Mar 04, 2014 at 10:35:39PM -0800, Greg KH wrote:
On Tue, Mar 04, 2014 at 09:10:46PM +0200, Valentina Manea wrote:
Add a new list API from CCAN.
Why can't you just take the one from the kernel, as userspace is GPLv2
code, right?
And are you sure CC0 is a valid license that you can
On Wed, Mar 05, 2014 at 12:42:59PM +0300, Dan Carpenter wrote:
On Tue, Mar 04, 2014 at 09:10:41PM +0200, Valentina Manea wrote:
diff --git a/drivers/staging/usbip/userspace/libsrc/usbip_common.h
b/drivers/staging/usbip/userspace/libsrc/usbip_common.h
index 2cb81b3..565ac78 100644
---
These set off a static checker warning about warn: add curly braces?,
but actually the code it correct, it's just the indenting which is
wrong.
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
diff --git a/drivers/staging/rtl8712/ieee80211.c
b/drivers/staging/rtl8712/ieee80211.c
index
On 2014-03-04 08:43, Chase Southwood wrote:
This patch changes a handful of while loops to timeouts to prevent
infinite looping on hardware failure. A couple such loops are in a
function (s626_debi_transfer()) which is called from critical sections,
so comedi_timeout() is unusable for them, and
On 2014-03-04 08:44, Chase Southwood wrote:
This patch for s626.c propagates the errors from the newly introduced
calls to comedi_timeout() as far as possible.
Signed-off-by: Chase Southwood chase.southw...@yahoo.com
---
Compile tested only.
Looks good, but if you change patch 1 this one will
On 2014-03-05 08:11, Fred Akers wrote:
Refactor this function to remove an extra indent level
Signed-off-by: Fred Akers kni...@botops.net
Reviewed-by: Ian Abbott abbo...@mev.co.uk
--
-=( Ian Abbott @ MEV Ltd.E-mail: abbo...@mev.co.uk)=-
-=( Tel: +44 (0)161 477 1898 FAX: +44
On 2014-03-04 18:29, H Hartley Sweeten wrote:
This series continues the cleanup of the pcl812/816/818 drivers.
v2: address some issues pointed out by Ian Abbott
* remove interrupt only command support from the pcl816 driver
* reorder FIFO read in the pcl816 driver
H Hartley Sweeten
From: Maxin B. John maxin.j...@enea.com
cppcheck reported possible null pointer dereference in mdc_lib.c
[lustre/lustre/mdc/mdc_lib.c:233]: (error) Possible null pointer dereference
: op_data - otherwise it is redundant to check if op_data is null at line 226
Signed-off-by: Maxin B. John
From: Maxin B. John maxin.j...@enea.com
osc_obd can be NULL. cppcheck reported this:
[lustre/lustre/lov/lov_obd.c:283]: (error) Possible null pointer dereference:
osc_obd - otherwise it is redundant to check if osc_obd is null at line 295
Signed-off-by: Maxin B. John maxin.j...@enea.com
---
On 03/04/2014 06:54 PM, Greg KH wrote:
On Tue, Mar 04, 2014 at 07:58:05AM -0600, Ken Cox wrote:
+EXPORT_SYMBOL_GPL(proc_CreateType);
You have some _very_ generic symbols that you are now exporting.
Here's another funny one:
+int hexDumpWordsToBuffer(char *dest,
+int
On 03/04/2014 06:57 PM, Greg KH wrote:
On Tue, Mar 04, 2014 at 07:58:12AM -0600, Ken Cox wrote:
Index: upstream-staging/drivers/staging/unisys/MAINTAINERS
===
Are you using cvs? Please start using git...
I'm using quilt to
Am Mittwoch, den 05.03.2014, 10:05 + schrieb Russell King - ARM
Linux:
On Wed, Mar 05, 2014 at 10:20:52AM +0100, Philipp Zabel wrote:
+struct imx_drm_component {
+ struct device_node *of_node;
+ struct list_head list;
+};
+
The only thing this structure appears to be doing is
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Wednesday, March 5, 2014 12:57 AM
To: Haiyang Zhang; da...@davemloft.net; net...@vger.kernel.org
Cc: KY Srinivasan; o...@aepfle.de; linux-ker...@vger.kernel.org; driverdev-
de...@linuxdriverproject.org
Subject:
The C99 specification states in section 6.11.5:
The placement of a storage-class specifier other than at the beginning
of the declaration specifiers in a declaration is an obsolescent
feature.
Signed-off-by: Tobias Klauser tklau...@distanz.ch
---
drivers/staging/lustre/lustre/lov/lov_object.c |
On Wednesday, March 05, 2014 1:12 AM, Fred Akers wrote:
Refactor this function to remove an extra indent level
Signed-off-by: Fred Akers kni...@botops.net
---
drivers/staging/comedi/range.c | 39 +++
1 file changed, 19 insertions(+), 20 deletions(-)
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Wed, Mar 05, 2014 at 02:04:35PM +0100, Maxin B. John wrote:
From: Maxin B. John maxin.j...@enea.com
cppcheck reported possible null pointer dereference in mdc_lib.c
[lustre/lustre/mdc/mdc_lib.c:233]: (error) Possible null pointer dereference
: op_data - otherwise it is redundant to
To resolve a sparse should it be static warning, the declaration for the
pmcc4_OSSI_release version string was moved from its own file to linux.c, and
the resulting empty file was deleted.
Signed-off-by: Shaun Laing sh...@xresource.ca
---
drivers/staging/cxt1e1/Makefile |1 -
On Wed, Mar 05, 2014 at 12:29:31PM -0700, Shaun Laing wrote:
To resolve a sparse should it be static warning, the declaration for the
pmcc4_OSSI_release version string was moved from its own file to linux.c, and
the resulting empty file was deleted.
Signed-off-by: Shaun Laing
From: Ken Cox j...@dhcp-10-15-1-72.hsv.redhat.com
Add the Unisys s-Par maintainer entry to the MAINTAINERS file.
Add Greg Kroah-Hartman to the list of patch recipients in the TODO file
Signed-off-by: Ken Cox j...@redhat.com
---
MAINTAINERS| 7 +++
On Wed, Mar 05, 2014 at 01:48:59PM -0600, Ken Cox wrote:
From: Ken Cox j...@dhcp-10-15-1-72.hsv.redhat.com
Where are patches 1-8?
___
devel mailing list
de...@linuxdriverproject.org
On Wed, Mar 05, 2014 at 01:49:50PM -0600, Ken Cox wrote:
From: Ken Cox j...@dhcp-10-15-1-72.hsv.redhat.com
Add the Unisys s-Par maintainer entry to the MAINTAINERS file.
Add Greg Kroah-Hartman to the list of patch recipients in the TODO file
Signed-off-by: Ken Cox j...@redhat.com
---
On Wed, Mar 05, 2014 at 01:48:59PM -0600, Ken Cox wrote:
From: Ken Cox j...@dhcp-10-15-1-72.hsv.redhat.com
I don't think that's the email address you really want us to use for
your commit address...
___
devel mailing list
de...@linuxdriverproject.org
Em Tue, 11 Feb 2014 13:38:51 +0100
Laurent Pinchart laurent.pinch...@ideasonboard.com escreveu:
Hi Paul,
Thank you for the patch.
On Tuesday 11 February 2014 12:17:01 Paul Bolle wrote:
Commit d632dfefd36f ([media] v4l: omap4iss: Add support for OMAP4
camera interface - Build system)
Please disregard patches 9-11 sent previously. Corrected patches follow.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
A number of functions were used for debug at one time but that code has been
removed.
Signed-off-by: Ken Cox j...@redhat.com
---
drivers/staging/unisys/include/timskmodutils.h| 99 +---
drivers/staging/unisys/visorutil/visorkmodutils.c | 565 --
2 files changed, 1
Add the Unisys s-Par maintainer entry to the MAINTAINERS file.
Add Greg Kroah-Hartman to the list of patch recipients in the TODO file
Signed-off-by: Ken Cox j...@redhat.com
---
MAINTAINERS| 5 +
drivers/staging/unisys/MAINTAINERS | 6 --
This patch fixes a few 80+ char lines as reported by checkpatch
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 23 ---
1 file changed, 16 insertions(+), 7 deletions(-)
diff --git
This patchset fixes more 80+ char lines as reported by checkpatch
There are still many more.
--
1.8.1.4
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Btw, if you don't get any messages from me that means I have given your
patch the stamp of approval. So good job on your previous patchset. :)
On Wed, Mar 05, 2014 at 03:54:49PM -0500, Mark Hounschell wrote:
@@ -1613,7 +1616,8 @@ static void dgap_tty_uninit(struct board_t *brd)
* dgap_sniff
On Wed, Mar 05, 2014 at 03:54:50PM -0500, Mark Hounschell wrote:
@@ -2953,7 +2955,8 @@ static int dgap_tty_write(struct tty_struct *tty, const
unsigned char *buf, int
if (down_interruptible(dgap_TmpWriteSem))
return -EINTR;
- if
Hi Mauro,
Thank you for the review.
On Wednesday 05 March 2014 17:10:06 Mauro Carvalho Chehab wrote:
Em Tue, 11 Feb 2014 13:38:51 +0100 Laurent Pinchart escreveu:
On Tuesday 11 February 2014 12:17:01 Paul Bolle wrote:
Commit d632dfefd36f ([media] v4l: omap4iss: Add support for OMAP4
Hi Joe,
On Wednesday 05 March 2014 16:28:03 Joe Perches wrote:
On Thu, 2014-03-06 at 00:50 +0100, Laurent Pinchart wrote:
Please note that -DDEBUG is equivalent to '#define DEBUG', not to '#define
CONFIG_DEBUG'. 'DEBUG' needs to be defined for dev_dbg() to have any
effect.
Not quite. If
On Thu, 2014-03-06 at 01:48 +0100, Laurent Pinchart wrote:
Would you recommend to drop driver-specific Kconfig options related to
debugging and use CONFIG_DYNAMIC_DEBUG instead ?
For development, sure, if there's sufficient memory.
For embedded systems with limited memory, using
dynamic_debug
Hi Joe,
On Wednesday 05 March 2014 17:00:37 Joe Perches wrote:
On Thu, 2014-03-06 at 01:48 +0100, Laurent Pinchart wrote:
Would you recommend to drop driver-specific Kconfig options related to
debugging and use CONFIG_DYNAMIC_DEBUG instead ?
For development, sure, if there's sufficient
From: Haiyang Zhang haiya...@microsoft.com
Date: Tue, 4 Mar 2014 14:11:06 -0800
It moves the state setting for query into rndis_filter_receive_response().
All callbacks including query-complete and status-callback are synchronized
by channel-inbound_lock. This prevents pentential race between
Hi Joe,
On Wednesday 05 March 2014 17:35:42 Joe Perches wrote:
On Thu, 2014-03-06 at 02:27 +0100, Laurent Pinchart wrote:
On Wednesday 05 March 2014 17:00:37 Joe Perches wrote:
On Thu, 2014-03-06 at 01:48 +0100, Laurent Pinchart wrote:
Would you recommend to drop driver-specific Kconfig
On Thu, 2014-03-06 at 02:27 +0100, Laurent Pinchart wrote:
On Wednesday 05 March 2014 17:00:37 Joe Perches wrote:
On Thu, 2014-03-06 at 01:48 +0100, Laurent Pinchart wrote:
Would you recommend to drop driver-specific Kconfig options related to
debugging and use CONFIG_DYNAMIC_DEBUG
Hi John,
[I started by reverting commit 161d78555435 Revert Staging: rtl8812ae:
remove modules field of rate_control_ops]
After merging the wireless-next tree, today's linux-next build (x86_64
allmodconfig) failed like this:
In file included from drivers/staging/wlan-ng/p80211netdev.c:91:0:
On Wed, Mar 05, 2014 at 02:52:26PM -0600, Ken Cox wrote:
Add the Unisys s-Par maintainer entry to the MAINTAINERS file.
Add Greg Kroah-Hartman to the list of patch recipients in the TODO file
Signed-off-by: Ken Cox j...@redhat.com
---
MAINTAINERS| 5 +
On Thu, Mar 06, 2014 at 01:48:29AM +0100, Laurent Pinchart wrote:
Hi Joe,
On Wednesday 05 March 2014 16:28:03 Joe Perches wrote:
On Thu, 2014-03-06 at 00:50 +0100, Laurent Pinchart wrote:
Please note that -DDEBUG is equivalent to '#define DEBUG', not to '#define
CONFIG_DEBUG'. 'DEBUG'
On Thu, 2014-03-06 at 00:50 +0100, Laurent Pinchart wrote:
Please note that -DDEBUG is equivalent to '#define DEBUG', not to '#define
CONFIG_DEBUG'. 'DEBUG' needs to be defined for dev_dbg() to have any effect.
Not quite. If CONFIG_DYNAMIC_DEBUG is set, these
dev_dbg statements are compiled
On Thu, 2014-03-06 at 02:52 +0100, Laurent Pinchart wrote:
Hi again Laurent
I've thought about that, but it would require iss.h to be included before all
other headers. I've also thought about creating an iss-debug.h header to be
included first just to #define DEBUG, but decided to go for
On Tue, Mar 4, 2014 at 9:20 PM, Greg KH gre...@linuxfoundation.org wrote:
On Tue, Mar 04, 2014 at 09:16:39PM +0200, Valentina Manea wrote:
Signed-off-by: Valentina Manea valentina.mane...@gmail.com
---
drivers/staging/usbip/userspace/configure.ac | 2 +-
1 file changed, 1 insertion(+), 1
On Wed, Mar 5, 2014 at 12:15 PM, Dan Carpenter dan.carpen...@oracle.com wrote:
On Wed, Mar 05, 2014 at 12:42:59PM +0300, Dan Carpenter wrote:
On Tue, Mar 04, 2014 at 09:10:41PM +0200, Valentina Manea wrote:
diff --git a/drivers/staging/usbip/userspace/libsrc/usbip_common.h
On Thu, Mar 06, 2014 at 01:07:03AM -0500, Fred Akers wrote:
This check is unnecessary because range_table will always be
initialized to range_unknown by comedi_device_postconfig() for
drivers that do not initialize range_table or range_table_list
Signed-off-by: Fred Akers kni...@botops.net
Refactor this function to remove an extra indent level
Signed-off-by: Fred Akers kni...@botops.net
---
drivers/staging/comedi/range.c | 39 +++
1 file changed, 19 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/comedi/range.c
This check is unnecessary because range_table will always be
initialized to range_unknown by comedi_device_postconfig() for
drivers that do not initialize range_table or range_table_list
Signed-off-by: Fred Akers kni...@botops.net
---
drivers/staging/comedi/range.c | 4
1 file changed, 4
Hi,
Denis Carikli wrote:
If de-active and/or pixelclk-active properties were set in the
display-timings DT node, they were not used.
Instead the data-enable and the pixel data clock polarity
were hardcoded.
This change is needed for making the eukrea-cpuimx51
QVGA display work.
I
Thanks for review.
OK. I will try to change all of OS_kmalloc to kmalloc/kzalloc.
And I also check GFP_DMA flag in kmalloc/kzalloc.
Regards,
Daeseok Youn
2014-03-05 19:04 GMT+09:00 Tobias Klauser tklau...@distanz.ch:
On 2014-03-05 at 03:37:15 +0100, Daeseok Youn daeseok.y...@gmail.com wrote:
2014-03-05 19:13 GMT+09:00 Tobias Klauser tklau...@distanz.ch:
On 2014-03-05 at 02:24:22 +0100, Daeseok Youn daeseok.y...@gmail.com wrote:
It doesn't need to assign name array address to np pointer.
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com
---
drivers/staging/cxt1e1/linux.c |
This patch for hwdrv_apci1564.c fixes the register map defines for the
digital input registers such that they are all the real offsets to each
register, rather than a mix of real offsets and adders to those offsets.
Additionally, the defines have been renamed to assist with shortening
some lines
This patch for hwdrv_apci1564.c fixes the register map defines for the
timer registers such that they are all the real offsets to each register,
rather than a mix of real offsets and adders to those offsets.
Additionally, the defines have been renamed to assist with shortening
some lines in excess
This patch for hwdrv_apci1564.c fixes the register map defines for the
watchdog registers such that they are all the real offsets to each
register, rather than a mix of real offsets and adders to those offsets.
Additionally, the defines have been renamed to assist with shortening
some lines in
This patch for hwdrv_apci1564.c fixes the register map defines for the
digital output registers such that they are all the real offsets to each
register, rather than a mix of real offsets and adders to those offsets.
Additionally, the defines have been renamed to assist with shortening
some lines
On 2014-03-06 at 08:19:19 +0100, DaeSeok Youn daeseok.y...@gmail.com wrote:
2014-03-05 19:13 GMT+09:00 Tobias Klauser tklau...@distanz.ch:
On 2014-03-05 at 02:24:22 +0100, Daeseok Youn daeseok.y...@gmail.com
wrote:
It doesn't need to assign name array address to np pointer.
Ok.
I will use sizeof(name) for snprintf() call.
Thanks.
Daeseok Youn.
2014-03-06 16:33 GMT+09:00 Tobias Klauser tklau...@distanz.ch:
On 2014-03-06 at 08:19:19 +0100, DaeSeok Youn daeseok.y...@gmail.com wrote:
2014-03-05 19:13 GMT+09:00 Tobias Klauser tklau...@distanz.ch:
On 2014-03-05 at
74 matches
Mail list logo