From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/media/i2c/ad9389b.c | 2 ++
1 file changed, 2 insertions(+)
Hi Wei,
Thanks for the patch.
On Mon, May 13, 2013 at 11:27 AM, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENODEV in the subdevice register error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei
Hi Wei
Thanks for the patch.
On Mon, May 13, 2013 at 11:27 AM, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENODEV in the subdevice register error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei
On Mon May 13 2013 08:00:10 Wei Yongjun wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
Acked-by: Hans Verkuil
Hi Wei Yongjun,
drivers/media/platform/blackfin/bfin_capture.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/platform/blackfin/bfin_capture.c
b/drivers/media/platform/blackfin/bfin_capture.c
index 0e55b08..2d1e032 100644
---
Op 09-05-13 09:33, Inki Dae schreef:
Hi all,
This post introduces a new helper framework based on dma fence. And the
purpose of this post is to collect other opinions and advices before RFC
posting.
First of all, this helper framework, called fence helper, is in progress
yet so might not
Hey Ezequiel,
Thank you for taking the time to look at this, it really is appreciated.
If you need anything else just let me know.
Thanks!!
On Sat, May 11, 2013 at 2:28 PM, Ezequiel Garcia elezegar...@gmail.com wrote:
On Thu, May 9, 2013 at 1:11 PM, a b genericgroupm...@gmail.com wrote:
Hi,
Hi Ezequiel,
Sorry, just saw your suggestion RE: keep_buffers, i will definitely
try this out and let you know how it goes.
Will probably give it a few days worth of runs to see if it re-occurs.
Thanks again!
On Sat, May 11, 2013 at 3:40 PM, Ezequiel Garcia elezegar...@gmail.com wrote:
On Sat,
This is the 2nd version of managed initializations for media/v4l2.
There are small changes documented in separate patches.
Additionally to advertise this solution I suggest to look at all *_remove
functions in drivers/media/i2c/ tree. After conversion to devm_* versions
most of the *_remove
This patch adds managed versions of initialization
and cleanup functions for v4l2 control handler.
Signed-off-by: Andrzej Hajda a.ha...@samsung.com
Reviewed-by: Sylwester Nawrocki s.nawro...@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
v2:
- added missing struct
This patch adds managed versions of initialization
functions for v4l2 subdevices.
Signed-off-by: Andrzej Hajda a.ha...@samsung.com
Reviewed-by: Sylwester Nawrocki s.nawro...@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
v2:
- changes of v4l2-ctrls.h moved to
This patch adds managed versions of initialization
and cleanup functions for media entity.
Signed-off-by: Andrzej Hajda a.ha...@samsung.com
Reviewed-by: Sylwester Nawrocki s.nawro...@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
v2:
- added missing struct device
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: move config-num_inputs check to the beginning of this function
-Original Message-
From: Maarten Lankhorst [mailto:maarten.lankho...@canonical.com]
Sent: Monday, May 13, 2013 5:01 PM
To: Inki Dae
Cc: Rob Clark; Daniel Vetter; DRI mailing list; linux-arm-
ker...@lists.infradead.org; linux-media@vger.kernel.org; linux-fbdev;
Kyungmin Park;
Hi Andrzej,
On Mon May 13 2013 10:34:46 Andrzej Hajda wrote:
This patch adds managed versions of initialization
functions for v4l2 subdevices.
I figured out what is bothering me about this patch: the fact that it is
tied to the v4l2_i2c_subdev_init/v4l2_subdev_init functions. Normally devm
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no
driver is bound) removes the need to set driver data field to
NULL.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Sakari Ailus sakari.ai...@iki.fi
---
This RFC looks at adding support for motion detection to V4L2. This is the main
missing piece that prevents the go7007 and solo6x10 drivers from being moved
into mainline from the staging directory.
This is the second version of this RFC. The changes are:
- Use a new event to signal motion
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no
driver is bound) removes the need to set driver data field to
NULL.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/media/platform/soc_camera/sh_mobile_csi2.c |8 +---
1 file changed, 1 insertion(+), 7
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no
driver is bound) removes the need to set driver data field to
NULL.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
.../platform/soc_camera/soc_camera_platform.c |7 ---
1 file changed, 7 deletions(-)
diff --git
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no
driver is bound) removes the need to set driver data field to
NULL.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Pelagicore AB i...@pelagicore.com
---
drivers/media/platform/timblogiw.c |4
1 file changed, 4
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no
driver is bound) removes the need to set driver data field to
NULL.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/media/rc/gpio-ir-recv.c |2 --
1 file changed, 2 deletions(-)
diff --git
2013/5/13 Wei Yongjun weiyj...@gmail.com:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: move
On Tue, 2013-03-12 at 09:14 +, Sekhar Nori wrote:
drivers/media/platform/davinci/Kconfig uses selects where
it should be using 'depends on'. This results in warnings of
the following sort when doing randconfig builds.
warning: (VIDEO_DM6446_CCDC VIDEO_DM355_CCDC VIDEO_ISIF
Op 13-05-13 11:21, Inki Dae schreef:
-Original Message-
From: Maarten Lankhorst [mailto:maarten.lankho...@canonical.com]
Sent: Monday, May 13, 2013 5:01 PM
To: Inki Dae
Cc: Rob Clark; Daniel Vetter; DRI mailing list; linux-arm-
ker...@lists.infradead.org;
Hi Paul,
On Mon, May 13, 2013 at 3:11 PM, Paul Bolle pebo...@tiscali.nl wrote:
On Tue, 2013-03-12 at 09:14 +, Sekhar Nori wrote:
[Snip]
This patch has only been build tested; I have tried to not break
any existing assumptions. I do not have the setup to test video,
so any test reports
Prabhakar,
On Mon, 2013-05-13 at 15:27 +0530, Prabhakar Lad wrote:
Good catch! the dependency can be dropped now.
Great.
Are you planning to post a patch for it or shall I do it ?
I don't mind submitting that trivial patch.
However, it's probably better if you do that. I can only state that
From: Lad, Prabhakar prabhakar.cse...@gmail.com
from commit 3778d05036cc7ddd983ae2451da579af00acdac2
[media: davinci: kconfig: fix incorrect selects]
VIDEO_VPFE_CAPTURE was removed but there was a negative
dependancy for building the DM365 VPFE MC based capture driver
(VIDEO_DM365_VPFE), This
Hi Paul,
On Mon, May 13, 2013 at 3:35 PM, Paul Bolle pebo...@tiscali.nl wrote:
Prabhakar,
On Mon, 2013-05-13 at 15:27 +0530, Prabhakar Lad wrote:
Good catch! the dependency can be dropped now.
Great.
Are you planning to post a patch for it or shall I do it ?
I don't mind submitting that
On Wed, May 08, 2013 at 12:37:29PM +0200, Laurent Pinchart wrote:
Hi Prabhakar,
On Wednesday 08 May 2013 10:19:57 Prabhakar Lad wrote:
On Wed, May 8, 2013 at 7:32 AM, Laurent Pinchart wrote:
On Tuesday 07 May 2013 15:10:36 Prabhakar Lad wrote:
On Mon, May 6, 2013 at 8:29 PM, Prabhakar
Hi Hans,
This series of patches replace the previous series sent on Apr 25.
This adds the check you requested for ret byte count in get_video_info().
Thank you,
-Leo.
Leonid Kegulskiy (4):
[media] hdpvr: Removed unnecessary get_video_info() call from
hdpvr_device_init()
[media] hdpvr:
Signed-off-by: Leonid Kegulskiy l...@lumanate.com
---
drivers/media/usb/hdpvr/hdpvr-control.c | 25 ++
drivers/media/usb/hdpvr/hdpvr-video.c | 54 +++
drivers/media/usb/hdpvr/hdpvr.h |2 +-
3 files changed, 37 insertions(+), 44
Signed-off-by: Leonid Kegulskiy l...@lumanate.com
---
drivers/media/usb/hdpvr/hdpvr-core.c |8
1 file changed, 8 deletions(-)
diff --git a/drivers/media/usb/hdpvr/hdpvr-core.c
b/drivers/media/usb/hdpvr/hdpvr-core.c
index 8247c19..cb69405 100644
---
Signed-off-by: Leonid Kegulskiy l...@lumanate.com
---
drivers/media/usb/hdpvr/hdpvr-video.c |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/hdpvr/hdpvr-video.c
b/drivers/media/usb/hdpvr/hdpvr-video.c
index d9eb8e1..2d02b49 100644
---
Signed-off-by: Leonid Kegulskiy l...@lumanate.com
---
drivers/media/usb/hdpvr/hdpvr-control.c |5 +
drivers/media/usb/hdpvr/hdpvr-video.c | 10 +++---
2 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/drivers/media/usb/hdpvr/hdpvr-control.c
-Original Message-
From: Maarten Lankhorst [mailto:maarten.lankho...@canonical.com]
Sent: Monday, May 13, 2013 6:52 PM
To: Inki Dae
Cc: 'Rob Clark'; 'Daniel Vetter'; 'DRI mailing list'; linux-arm-
ker...@lists.infradead.org; linux-media@vger.kernel.org; 'linux-fbdev';
'Kyungmin
Op 13-05-13 13:24, Inki Dae schreef:
and can be solved with userspace locking primitives. No need for the
kernel to get involved.
Yes, that is how we have synchronized buffer between CPU and DMA device
until now without buffer synchronization mechanism. I thought that it's best
to make user
Currently the event API allows for a payload of up to 64 bytes. Sometimes we
would like to pass larger payloads to userspace such as metadata associated
with a particular video stream.
A typical example of that would be object detection events.
This RFC describes one approach for doing this.
Hello,
I agree with the content of the patch, but I disagree with the commit message.
From the commit message it seems that you fixed a bug about the error code,
but the aim of this patch is to uniform the code style. I suggest something
like: uniform code style in sta2x11_vip_init_one()
On
-Original Message-
From: linux-fbdev-ow...@vger.kernel.org [mailto:linux-fbdev-
ow...@vger.kernel.org] On Behalf Of Maarten Lankhorst
Sent: Monday, May 13, 2013 8:41 PM
To: Inki Dae
Cc: 'Rob Clark'; 'Daniel Vetter'; 'DRI mailing list'; linux-arm-
ker...@lists.infradead.org;
On 05/13/2013 08:19 PM, Federico Vaga wrote:
Hello,
I agree with the content of the patch, but I disagree with the commit
message.
From the commit message it seems that you fixed a bug about the error code,
but the aim of this patch is to uniform the code style. I suggest something
On Monday 13 May 2013 20:40:33 Wei Yongjun wrote:
On 05/13/2013 08:19 PM, Federico Vaga wrote:
Hello,
I agree with the content of the patch, but I disagree with the commit
message.
From the commit message it seems that you fixed a bug about the error
code,
but the aim of this
On Mon, May 13, 2013 at 8:21 AM, Inki Dae inki@samsung.com wrote:
In that case you still wouldn't give userspace control over the fences. I
don't see any way that can end well.
What if userspace never signals? What if userspace gets killed by oom
killer. Who keeps track of that?
In all
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The orig code will release all the resources if v4l2_device_register()
failed and return 0. But what we need in this case is to return an
negative error code to let the caller known we are failed.
So the patch save the return value of
On Monday 13 May 2013 22:00:01 Wei Yongjun wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The orig code will release all the resources if v4l2_device_register()
failed and return 0. But what we need in this case is to return an
negative error code to let the caller known we are
Hi Wei,
On Mon, May 13, 2013 at 01:48:45PM +0800, Wei Yongjun wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -EINVAL in the get kernel address error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
Hi all,
On Wed, May 08, 2013 at 09:32:17AM +0200, Andrzej Hajda wrote:
On 07.05.2013 17:07, Laurent Pinchart wrote:
On Tuesday 07 May 2013 02:11:27 Kim, Milo wrote:
On Monday, May 06, 2013 6:34 PM Andrzej Hajda wrote:
This RFC proposes generic API for exposing flash subdevices via LED
Hi Mauro,
On Wed, Apr 17, 2013 at 07:43:00AM -0300, Mauro Carvalho Chehab wrote:
and a camera anymore. The OMAP2 were used on some Nokia phones.
They used to maintain that code, but now that they moved to the dark
side of the moon, they lost their interests on it. So, it may not
be easily
On Wed, May 8, 2013 at 1:23 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote:
Gcc 4.3 doesn't understand binary constanrs (0b*):
drivers/media/radio/radio-si476x.c:862:20: error: invalid suffix b1000
on integer constant
Hence use a hexadecimal constant (0x*) instead.
Signed-off-by:
Hi Sachin,
On Mon, May 13, 2013 at 02:47:54PM +0530, Sachin Kamat wrote:
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no
driver is bound) removes the need to set driver data field to
NULL.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laurent Pinchart
On Mon, May 13, 2013 at 1:18 PM, Inki Dae inki@samsung.com wrote:
2013/5/13 Rob Clark robdcl...@gmail.com
On Mon, May 13, 2013 at 8:21 AM, Inki Dae inki@samsung.com wrote:
In that case you still wouldn't give userspace control over the fences.
I
don't see any way that can end
Hans,
I fear that in my ignorance of V4L2, I have backed my client into a corner, so
to speak. I am developing embedded Linux firmware for boards intended to
driver video sensors within a medical device. As such, tried and true versions
of everything on board are preferred, even if they are
This message is generated daily by a cron job that builds media_tree for
the kernels and architectures in the list below.
Results of the daily build of media_tree:
date: Mon May 13 19:00:28 CEST 2013
git branch: test
git hash: 02615ed5e1b2283db2495af3cf8f4ee172c77d80
gcc
From: Vladimir Barinov vladimir.bari...@cogentembedded.com
Add subdev video ops for ADV7180 video decoder. This makes decoder usable on
the soc-camera drivers.
Signed-off-by: Vladimir Barinov vladimir.bari...@cogentembedded.com
Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com
Hi,
On Monday 13 of May 2013 20:24:01 Inki Dae wrote:
-Original Message-
From: Maarten Lankhorst [mailto:maarten.lankho...@canonical.com]
Sent: Monday, May 13, 2013 6:52 PM
To: Inki Dae
Cc: 'Rob Clark'; 'Daniel Vetter'; 'DRI mailing list'; linux-arm-
Hi Sascha,
On Monday 13 May 2013 12:46:04 Sascha Hauer wrote:
On Wed, May 08, 2013 at 12:37:29PM +0200, Laurent Pinchart wrote:
Hi Prabhakar,
On Wednesday 08 May 2013 10:19:57 Prabhakar Lad wrote:
On Wed, May 8, 2013 at 7:32 AM, Laurent Pinchart wrote:
On Tuesday 07 May 2013
-Original Message-
From: Rob Clark [mailto:robdcl...@gmail.com]
Sent: Tuesday, May 14, 2013 2:58 AM
To: Inki Dae
Cc: linux-fbdev; DRI mailing list; Kyungmin Park; myungjoo.ham; YoungJun
Cho; linux-arm-ker...@lists.infradead.org; linux-media@vger.kernel.org
Subject: Re: Introduce a
All of the firmwares I've tested, including 0x1e, will inevitably crash
before recording for even 10 minutes. There must be a race condition of
IR RX vs. video-encoding in the firmware, because if you disable IR receiver
polling, then the firmware is stable again. I'd guess that most people don't
On Tue, May 14, 2013 at 12:59:27AM +0200, Laurent Pinchart wrote:
Hi Sascha,
On Monday 13 May 2013 12:46:04 Sascha Hauer wrote:
On Wed, May 08, 2013 at 12:37:29PM +0200, Laurent Pinchart wrote:
Hi Prabhakar,
On Wednesday 08 May 2013 10:19:57 Prabhakar Lad wrote:
On Wed, May 8,
From: Lad, Prabhakar prabhakar.cse...@gmail.com
This patch series cleanups the check for EPERM in dbg_g/s_register
and vidioc_g/s_register.
Lad, Prabhakar (4):
media: i2c: remove duplicate checks for EPERM in dbg_g/s_register
media: dvb-frontends: remove duplicate checks for EPERM in
From: Lad, Prabhakar prabhakar.cse...@gmail.com
This patch removes check for EPERM in dbg_g/s_register of subdevice
drivers as this check is already performed by core.
Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com
---
drivers/media/i2c/ad9389b.c |4
From: Lad, Prabhakar prabhakar.cse...@gmail.com
This patch removes check for EPERM in dbg_g/s_register
as this check is already performed by core.
Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com
---
drivers/media/dvb-frontends/au8522_decoder.c |4
1 files changed, 0
From: Lad, Prabhakar prabhakar.cse...@gmail.com
This patch removes check for EPERM in vidioc_g/s_register
as this check is already performed by core.
Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com
---
drivers/media/usb/pvrusb2/pvrusb2-hdw.c |2 --
1 files changed, 0 insertions(+),
From: Lad, Prabhakar prabhakar.cse...@gmail.com
This patch removes check for EPERM in dbg_g/s_register and
vidioc_g/s_register as this check is already performed by core.
Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com
---
drivers/media/pci/bt8xx/bttv-driver.c |6 --
63 matches
Mail list logo