Hi Phillip,
On Mon May 13 2013 20:03:38 Phillip Norisez wrote:
> 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 a
From: Lad, Prabhakar
Both synchronous and asynchronous tvp514x subdevice probing is supported by
this patch.
Signed-off-by: Lad, Prabhakar
Cc: Guennadi Liakhovetski
Cc: Laurent Pinchart
Cc: Hans Verkuil
Cc: Sakari Ailus
Cc: Mauro Carvalho Chehab
---
drivers/media/i2c/tvp514x.c | 23
From: Lad, Prabhakar
This patch series adds support for vpif capture and display
driver to support asynchronously register subdevices.
The first two patches adds asynchronous probing for adv7343
and tvp514x respectively.
Need for this support:
Currently bridge device drivers register devices for
From: Lad, Prabhakar
Both synchronous and asynchronous adv7343 subdevice probing is supported by
this patch.
Signed-off-by: Lad, Prabhakar
Cc: Guennadi Liakhovetski
Cc: Laurent Pinchart
Cc: Hans Verkuil
Cc: Sakari Ailus
Cc: Mauro Carvalho Chehab
---
drivers/media/i2c/adv7343.c | 16
From: Lad, Prabhakar
Add support for asynchronous subdevice probing, using the v4l2-async API.
The legacy synchronous mode is still supported too, which allows to
gradually update drivers and platforms.
Signed-off-by: Lad, Prabhakar
Cc: Guennadi Liakhovetski
Cc: Hans Verkuil
Cc: Laurent Pinch
From: Lad, Prabhakar
Add support for asynchronous subdevice probing, using the v4l2-async API.
The legacy synchronous mode is still supported too, which allows to
gradually update drivers and platforms.
Signed-off-by: Prabhakar Lad
Cc: Guennadi Liakhovetski
Cc: Hans Verkuil
Cc: Laurent Pincha
Hello,
On 2013-05-13 07:48, Wei Yongjun wrote:
From: Wei Yongjun
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
Acked-by: Marek Szyprowski
---
drivers/media/v4l2-core/videobuf2-core.c | 4
From: Lad, Prabhakar
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
---
drivers/media/pci/bt8xx/bttv-driver.c |6 --
drivers/media/pci/cx18/cx18-av-core.c |4
From: Lad, Prabhakar
This patch removes check for EPERM in vidioc_g/s_register
as this check is already performed by core.
Signed-off-by: Lad, Prabhakar
---
drivers/media/usb/pvrusb2/pvrusb2-hdw.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/drivers/media/usb/pvrus
From: Lad, Prabhakar
This patch removes check for EPERM in dbg_g/s_register
as this check is already performed by core.
Signed-off-by: Lad, Prabhakar
---
drivers/media/dvb-frontends/au8522_decoder.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/media/dvb-f
From: Lad, Prabhakar
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
---
drivers/media/i2c/ad9389b.c |4
drivers/media/i2c/adv7183.c |4
drivers/med
From: Lad, Prabhakar
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
dbg_g/s_register
media: usb:
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:
> > >
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
u
> -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: Intro
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 Ma
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-
> > ker...@lists.in
From: Vladimir Barinov
Add subdev video ops for ADV7180 video decoder. This makes decoder usable on
the soc-camera drivers.
Signed-off-by: Vladimir Barinov
Signed-off-by: Sergei Shtylyov
---
This patch is against the 'media_tree.git' repo.
Changes from version 2:
- set the field format depe
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 versi
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
On Mon, May 13, 2013 at 1:18 PM, Inki Dae wrote:
>
>
> 2013/5/13 Rob Clark
>>
>> On Mon, May 13, 2013 at 8:21 AM, Inki Dae 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 si
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
> Cc: Laurent Pinchart
> Cc: Sakari Ailus
Thanks!
A
On Wed, May 8, 2013 at 1:23 PM, Geert Uytterhoeven 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: Geert Uytt
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
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
Hi Wei,
On Mon, May 13, 2013 at 01:48:45PM +0800, Wei Yongjun wrote:
> From: Wei Yongjun
>
> 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
> ---
> drivers/media/v4l2-core/videobuf2-core.c
On Monday 13 May 2013 22:00:01 Wei Yongjun wrote:
> From: Wei Yongjun
>
> 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 th
From: Wei Yongjun
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 v4l2_device_register() to 'ret'
and return i
On Mon, May 13, 2013 at 8:21 AM, Inki Dae 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 cases,
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,
> >
> > bu
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
> -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;
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
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.
The
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 ma
> -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';
> 'Kyung
Signed-off-by: Leonid Kegulskiy
---
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
--- a/drivers/media/usb/hdpvr/hdpvr-vi
Signed-off-by: Leonid Kegulskiy
---
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
b/drivers/media/usb/hdpvr/hdpvr-control.c
in
Signed-off-by: Leonid Kegulskiy
---
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
--- a/drivers/media/usb/hdpvr/hdpvr-core.c
+++ b/driver
Signed-off-by: Leonid Kegulskiy
---
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 deletions(-)
diff --git a/dr
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: Re
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,
Hi Paul,
On Mon, May 13, 2013 at 3:35 PM, Paul Bolle 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 trivia
From: Lad, Prabhakar
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 patch fixes this dependency by
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 tha
Hi Paul,
On Mon, May 13, 2013 at 3:11 PM, Paul Bolle 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 welcome.
>>
>>
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; linux-media@vge
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 &&
2013/5/13 Wei Yongjun :
> From: Wei Yongjun
>
> 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
> ---
> v1 -> v2: move config->num_inputs check to the beginning of this function
> ---
> drivers/me
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
---
drivers/media/rc/gpio-ir-recv.c |2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/medi
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
Cc: Pelagicore AB
---
drivers/media/platform/timblogiw.c |4
1 file changed, 4 deletions(-)
diff --git a/drivers/media/platform
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
---
drivers/media/platform/soc_camera/sh_mobile_csi2.c |8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/drive
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
---
.../platform/soc_camera/soc_camera_platform.c |7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/media/platform
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 detect
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
Cc: Laurent Pinchart
Cc: Sakari Ailus
---
drivers/media/platform/omap3isp/isp.c |2 --
1 file changed, 2 deletions(-)
diff --git a/
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
fu
> -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;
From: Wei Yongjun
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
---
v1 -> v2: move config->num_inputs check to the beginning of this function
---
drivers/media/platform/blackfin/bfin_capture.c | 8 +
This patch adds managed versions of initialization
and cleanup functions for media entity.
Signed-off-by: Andrzej Hajda
Reviewed-by: Sylwester Nawrocki
Signed-off-by: Kyungmin Park
---
v2:
- added missing struct device forward declaration
---
drivers/media/media-entity.c | 70 +++
This patch adds managed versions of initialization
and cleanup functions for v4l2 control handler.
Signed-off-by: Andrzej Hajda
Reviewed-by: Sylwester Nawrocki
Signed-off-by: Kyungmin Park
---
v2:
- added missing struct device forward declaration,
- corrected few comments
---
d
This patch adds managed versions of initialization
functions for v4l2 subdevices.
Signed-off-by: Andrzej Hajda
Reviewed-by: Sylwester Nawrocki
Signed-off-by: Kyungmin Park
---
v2:
- changes of v4l2-ctrls.h moved to proper patch
---
drivers/media/v4l2-core/v4l2-common.c | 10 +++
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 routin
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 wrote:
> On Sat, May 11, 2013 at 10:2
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 wrote:
> On Thu, May 9, 2013 at 1:11 PM, a b wrote:
>> Hi,
>>
>> I am seeing occasional issues when using
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 migh
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
> --- a/drivers/media/platform/blackfin/bf
66 matches
Mail list logo
