Hi Sachin,
On Tue, Jun 18, 2013 at 11:42 AM, Sachin Kamat wrote:
> Hi Arun,
>
> On 18 June 2013 11:25, Arun Kumar K wrote:
>> Hi Sachin,
>>
>>
>> On Tue, Jun 18, 2013 at 10:56 AM, Sachin Kamat
>> wrote:
>>> On 18 June 2013 10:21, Arun Kumar K wrote:
Hi Kamil,
Thank you for the
Guennadi,
Can you review/ack this? This patch series really should be merged for 3.11
to prevent problems with the debug ioctls being unable to discover and access
subdevices in the v4l2-core code.
I've tested this on my renesas board.
If you prefer, you can also take care of this patch yourself
Hi Arun,
On 18 June 2013 11:25, Arun Kumar K wrote:
> Hi Sachin,
>
>
> On Tue, Jun 18, 2013 at 10:56 AM, Sachin Kamat
> wrote:
>> On 18 June 2013 10:21, Arun Kumar K wrote:
>>> Hi Kamil,
>>>
>>> Thank you for the review.
>>>
>>>
> #define IS_MFCV6(dev)(dev->variant->versio
Hi Sachin,
On Tue, Jun 18, 2013 at 10:56 AM, Sachin Kamat wrote:
> On 18 June 2013 10:21, Arun Kumar K wrote:
>> Hi Kamil,
>>
>> Thank you for the review.
>>
>>
#define IS_MFCV6(dev)(dev->variant->version >= 0x60 ? 1 :
>>> 0)
+#define IS_MFCV7(dev)(dev
> -Original Message-
> From: Russell King - ARM Linux [mailto:li...@arm.linux.org.uk]
> Sent: Tuesday, June 18, 2013 3:21 AM
> To: Inki Dae
> Cc: Maarten Lankhorst; linux-fbdev; Kyungmin Park; DRI mailing list; Rob
> Clark; myungjoo.ham; YoungJun Cho; Daniel Vetter; linux-arm-
> ker...@li
On 18 June 2013 10:21, Arun Kumar K wrote:
> Hi Kamil,
>
> Thank you for the review.
>
>
>>> #define IS_MFCV6(dev)(dev->variant->version >= 0x60 ? 1 :
>> 0)
>>> +#define IS_MFCV7(dev)(dev->variant->version >= 0x70 ? 1 :
>> 0)
>>
>> According to this, MFC v7 is also
From: Wei Yongjun
Add the missing unlock before return from function coda_stop_streaming()
in the error handling case.
Signed-off-by: Wei Yongjun
---
drivers/media/platform/coda.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c
Hi Kamil,
Thank you for the review.
>> #define IS_MFCV6(dev)(dev->variant->version >= 0x60 ? 1 :
> 0)
>> +#define IS_MFCV7(dev)(dev->variant->version >= 0x70 ? 1 :
> 0)
>
> According to this, MFC v7 is also detected as MFC v6. Was this intended?
Yes this was int
Hi Sylwester,
>
> -The machine code (plat-s5p and arch/arm/mach-*) must select following options
> +The machine code (arch/arm/plat-samsung and arch/arm/mach-*) must select
> +following options:
After the recent platform code cleanup the below entries are not found
in arch/arm/mach-* (checked in
Hi Sylwester,
Just a couple of nits inline.
On 17 June 2013 22:12, Sylwester Nawrocki wrote:
> This change is mainly to update the driver's path changed from
> drivers/media/platform/s5p-fimc to drivers/media/platform/exynos4-is/.
> While at it, remove non-existent files rule, move the whole ent
The following changes since commit
e049ca5e854263c821a15c0e25fe2ae202c365e1:
[media] staging/media: lirc_imon: fix leaks in imon_probe()
(2013-06-17 15:52:20 -0300)
are available in the git repository at:
git://linuxtv.org/mkrufky/tuners mxl111sf
for you to fetch changes up to 0fca4f2af6a
The following changes since commit
e049ca5e854263c821a15c0e25fe2ae202c365e1:
[media] staging/media: lirc_imon: fix leaks in imon_probe()
(2013-06-17 15:52:20 -0300)
are available in the git repository at:
git://linuxtv.org/mkrufky/tuners r820t
for you to fetch changes up to a02dfce109f6dc
eek... I'd rather merge it than ack it -- why don't I see it in patchwork?
I'll try to merge it tonight or tomorrow.
Cheers,
Mike
On Thu, Jun 13, 2013 at 2:09 AM, Hans Verkuil wrote:
> On Thu May 23 2013 08:50:43 Hans Verkuil wrote:
>> Remove the silly redefines of pr_err/info/debug.
>>
>> Thi
Loan Syndicacion
Am AFG Guaranty Trust Bank, zu strukturieren wir Kreditlinien treffen Sie
unsere
Kunden spezifischen geschäftlichen Anforderungen und einen deutlichen
Mehrwert für unsere
Kunden Unternehmen.
eine Division der AFG Finance und Private Bank plc.
Wenn Sie erwägen, eine große Akqui
Hi Sylwester,
On Sun, Jun 16, 2013 at 11:16:44PM +0200, Sylwester Nawrocki wrote:
> Hi,
>
> On 06/12/2013 06:44 AM, Prabhakar Lad wrote:
> >On Tue, Jun 11, 2013 at 4:20 PM, Sakari Ailus wrote:
> >>media_entity_cleanup() frees the links array which will be accessed by
> >>media_entity_remove_link
Hi Sylwester,
A bit late, but for the whole series,
Acked-by: Laurent Pinchart
As discussed privately earlier today, sorry for not having handled the patches
myself. I will gather media controller related patches in the future.
On Monday 17 June 2013 14:12:18 Sylwester Nawrocki wrote:
> Hi Ma
Hi Hans,
Thanks for the patch.
On Wednesday 12 June 2013 17:01:02 Hans Verkuil wrote:
> From: Hans Verkuil
>
> 'parent' was renamed to 'dev_parent'. Clarify how/when this should be used.
>
> Signed-off-by: Hans Verkuil
Acked-by: Laurent Pinchart
with a small comment below.
> ---
> Docume
Hi Hans,
Thanks for the patch.
On Wednesday 12 June 2013 17:00:58 Hans Verkuil wrote:
> From: Hans Verkuil
>
> This driver did not yet support struct v4l2_device, so add it. This
> make it possible to replace the deprecated parent field with the
> v4l2_dev field, allowing the eventual removal o
Hi Guennadi,
Thank you for the patch. Overall it looks pretty good, please see below for
some small comments. If you've been wondering why your stock of comas is so
low, wonder no more: I've found them :-D
On Monday 17 June 2013 08:04:10 Guennadi Liakhovetski wrote:
> Add documentation for the
On Tue, Jun 18, 2013 at 02:19:04AM +0900, Inki Dae wrote:
> It seems like that all pages of the scatterlist should be mapped with
> DMA every time DMA operation is started (or drm_xxx_set_src_dma_buffer
> function call), and the pages should be unmapped from DMA again every
> time DMA operation is
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 Jun 17 19:00:22 CEST 2013
git branch: test
git hash: 3080f8c77f277eb87397d639581ebea859f9ea41
gcc versi
Hi Linus,
Please pull from:
git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media
v4l_for_linus
For a series of fixes for Kernel 3.10. There are some usual driver fixes (mostly
on s5p/exynos playform drivers), plus some fixes at V4L2 core.
Thank you!
Mauro
Latest commit at the br
Hi Prabhakar,
Thank you for the patches.
On Monday 17 June 2013 20:50:40 Prabhakar Lad wrote:
> From: "Lad, Prabhakar"
>
> This patch series cleans the VPIF driver, uses devm_* api wherever
> required and uses module_platform_driver() to simplify the code.
>
> This patch series applies on http
This change is mainly to update the driver's path changed from
drivers/media/platform/s5p-fimc to drivers/media/platform/exynos4-is/.
While at it, remove non-existent files rule, move the whole entry to
the Samsung drivers section and add the patch tracking system URL.
Signed-off-by: Sylwester Naw
Update the documentation with the driver's path changed in
commit 56fa1a6a6a7da91e7ece8b01b0ae8adb2926e434
[media] s5p-fimc: Change the driver directory to exynos4-is
Signed-off-by: Sylwester Nawrocki
Signed-off-by: Kyungmin Park
---
Documentation/video4linux/fimc.txt | 21 +++
Since SOC_EXYNOS4412 Kconfig symbol has been removed, it is enough to
check for SOC_EXYNOS4212 for both SoCs from Exynos4x12 series.
Cc: linux-media@vger.kernel.org
Cc: Mauro Carvalho Chehab
Cc: Sylwester Nawrocki
Signed-off-by: Tomasz Figa
Signed-off-by: Kyungmin Park
---
drivers/media/platf
ARCH_EXYNOS is going to be excluded from PLAT_S5P, so it must be checked
separately in Exynos-related Kconfig entries.
Cc: linux-media@vger.kernel.org
Cc: Mauro Carvalho Chehab
Cc: Sylwester Nawrocki
Cc: Kamil Debski
Cc: Tomasz Stanislawski
Cc: Jeongtae Park
Signed-off-by: Tomasz Figa
Signed
On Mon, Jun 17, 2013 at 04:42:37PM +0100, Russell King - ARM Linux wrote:
> On Tue, Jun 18, 2013 at 12:03:31AM +0900, Inki Dae wrote:
> > 2013/6/17 Russell King - ARM Linux
> > Exactly right. But that is not definitely my point. Could you please see
> > the below simple example?:
> > (Presume that
On Tue, Jun 18, 2013 at 12:03:31AM +0900, Inki Dae wrote:
> 2013/6/17 Russell King - ARM Linux
> Exactly right. But that is not definitely my point. Could you please see
> the below simple example?:
> (Presume that CPU and DMA share a buffer and the buffer is mapped with user
> space as cachable)
Hi Guennadi,
Thanks for the patch.
On Mon, Jun 17, 2013 at 11:34 AM, Guennadi Liakhovetski
wrote:
> Add documentation for the V4L2 clock and V4L2 asynchronous probing APIs
> to v4l2-framework.txt.
>
> Signed-off-by: Guennadi Liakhovetski
Reviewed-by: Lad, Prabhakar
Regards,
--Prabhakar Lad
-
Hi Guennadi,
Thanks for the patch.
On Sat, Jun 15, 2013 at 12:38 AM, Guennadi Liakhovetski
wrote:
> Typical video devices like camera sensors require an external clock source.
> Many such devices cannot even access their hardware registers without a
> running clock. These clock sources should be
From: "Lad, Prabhakar"
This patch removes unwanted header include of mach/hardware.h
and along side sorts the header inclusion alphabetically.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif.c | 10 --
1 file changed, 4 insertions(+),
From: "Lad, Prabhakar"
Ideally the freeing of irq's and the global variables needs to be
done in the remove() rather than module_exit(), this patch moves
the freeing up of irq's and freeing the memory allocated to channel
objects to remove() callback of struct platform_driver.
Signed-off-by: Lad
From: "Lad, Prabhakar"
This patch uses module_platform_driver() to simplify the code.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif_capture.c | 28 +
1 file changed, 1 insertion(+), 27 deletions(-)
diff --git a/driv
From: "Lad, Prabhakar"
use devm_request_irq() instead of request_irq(). This ensures
more consistent error values and simplifies error paths.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif_capture.c | 36 +
1 file cha
From: "Lad, Prabhakar"
For vpif capture driver each IRQ resource contains a single IRQ
so drop the second loop.
Signed-off-by: Lad, Prabhakar
---
drivers/media/platform/davinci/vpif_capture.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/drivers/media
From: "Lad, Prabhakar"
Ideally the freeing of irq's and the global variables needs to be
done in the remove() rather than module_exit(), this patch moves
the freeing up of irq's and freeing the memory allocated to channel
objects to remove() callback of struct platform_driver.
Signed-off-by: Lad
From: "Lad, Prabhakar"
This patch removes unnecessary braces around defines.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/media/platform/davinci/vpif.c
From: "Lad, Prabhakar"
For vpif display driver each IRQ resource contains a single IRQ
so drop the second loop.
Signed-off-by: Lad, Prabhakar
---
drivers/media/platform/davinci/vpif_display.c | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/drivers/medi
From: "Lad, Prabhakar"
use devm_request_irq() instead of request_irq(). This ensures
more consistent error values and simplifies error paths.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif_display.c | 35 +++--
1 file cha
From: "Lad, Prabhakar"
This patch uses module_platform_driver() to simplify the code.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif_display.c | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/drivers/med
From: "Lad, Prabhakar"
Use devm_ioremap_resource instead of reques_mem_region()/ioremap().
This ensures more consistent error values and simplifies error paths.
Signed-off-by: Lad, Prabhakar
Acked-by: Laurent Pinchart
---
drivers/media/platform/davinci/vpif.c | 27 --
From: "Lad, Prabhakar"
This patch series cleans the VPIF driver, uses devm_* api wherever
required and uses module_platform_driver() to simplify the code.
This patch series applies on http://git.linuxtv.org/hverkuil/media_tree.git/
shortlog/refs/heads/for-v3.11 and is tested on OMAP-L138 EVM.
C
Hello,
On 6/14/13 9:15 AM, Hans de Goede wrote:
So how about taking current master and releasing that as a 1.0.0 release ?
I'm fine with preparing a 1.0.0 release.
But I'm under the impression that the libdvbv5 interface is not polished
enough to build a shared library from it. So I'd like t
This patch splits the global workbuf into a global tempbuf and a per-context
workbuf, adds the codec mode aux register, and restores the work buffer
pointer on commands. With the new firmware, there is only a single set of
read/write pointers which need to be restored between context switches.
This
Add a bitstream ringbuffer using kfifo. Queued source buffers are to be copied
into the bitstream ringbuffer immediately and marked as done, if possible.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda.c | 134 +-
drivers/media/platform/coda.h
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda.c | 50 +--
1 file changed, 48 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c
index 1f3bd43..856a93e 100644
--- a/drivers/media/platform/
This patch enables decoding of h.264 and mpeg4 streams on CODA7541.
Queued output buffers are immediately copied into the bitstream
ringbuffer. A device_run can be scheduled whenever there is either
enough compressed bitstream data, or the CODA is in stream end mode.
Each successful device_run, da
This sets up IRAM areas used as temporary memory for the different
hardware units depending on the frame size.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda.c | 145 +++---
drivers/media/platform/coda.h | 11 +++-
2 files changed, 146 insertion
Add coda_prepare_encode() and coda_finish_encode() functions. They are called
from coda_device_run() and coda_irq_handler(), respectively, before and after
the hardware picture run. This should make the following decoder support patch
easier to read, which will add the coda_prepare/finish_decode()
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c
index c4566c4..90f3386 100644
--- a/drivers/media/platform/coda.c
+++ b/drivers/media/platfor
The following patch series depends on the CODA patches queued in Kamil's branch
and on the "mem2mem: add support for hardware buffered queue" patch I've posted
earlier. It should allow decoding h.264 high profile 1080p streams on i.MX53
with the current CODA7541 firmware version 1.4.50.
regards
P
The encoder only ever needs two buffers, but we'll have to increase
CODA_MAX_FRAMEBUFFERS for the decoder.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/cod
Hi Arun,
I have read your patches. They seem alright, I back comments made by Hans
and Sylwester. I have one question, which follows inline.
Best wishes,
--
Kamil Debski
Linux Kernel Developer
Samsung R&D Institute Poland
> -Original Message-
> From: Arun Kumar K [mailto:arun...@samsung
Reverse-engineered driver for cheapo video digitizer, made from observations of
Windows XP driver. The protocol is not yet completely understood, so far we
don't provide any controls, only support a single format out of three and don't
support the audio device.
Signed-off-by: Lubomir Rintel
Acked
On Wed, 2013-06-12 at 09:49 +0200, Hans Verkuil wrote:
...
> > > > +static int usbtv_queryctrl(struct file *file, void *priv,
> > > > + struct v4l2_queryctrl *ctrl)
> > > > +{
> > > > + return -EINVAL;
> > > > +}
> > >
> > > Drop this ioctl. If it doesn't do any
On Mon, Jun 17, 2013 at 10:04:45PM +0900, Inki Dae wrote:
> It's just to implement a thin sync framework coupling cache operation. This
> approach is based on dma-buf for more generic implementation against android
> sync driver or KDS.
>
> The described steps may be summarized as:
> lock ->
Op 17-06-13 15:04, Inki Dae schreef:
>
>> -Original Message-
>> From: Maarten Lankhorst [mailto:maarten.lankho...@canonical.com]
>> Sent: Monday, June 17, 2013 8:35 PM
>> To: Inki Dae
>> Cc: dri-de...@lists.freedesktop.org; linux-fb...@vger.kernel.org; linux-
>> arm-ker...@lists.infradead.o
> -Original Message-
> From: Maarten Lankhorst [mailto:maarten.lankho...@canonical.com]
> Sent: Monday, June 17, 2013 8:35 PM
> To: Inki Dae
> Cc: dri-de...@lists.freedesktop.org; linux-fb...@vger.kernel.org; linux-
> arm-ker...@lists.infradead.org; linux-media@vger.kernel.org;
> dan...@f
Hi Mauro,
Only one patch in this pull request, but it fixes pretty serious bug,
that took me long time to debug. Please queue for 3.10, if possible.
The following changes since commit af44ad5edd1eb6ca92ed5be48e0004e1f04bf219:
[media] soc_camera: error dev remove and v4l2 call (2013-06-08 21:51
Hi Mauro,
This includes corrections of the media entity links handling and resolves
potential issues when media entity drivers are in different kernel modules.
It allows to keep all entities that belong to same media graph in correct
state, when one of an entity's driver module gets unloaded.
Th
On Mon 17 June 2013 00:09:59 Sakari Ailus wrote:
> Hi Hans,
>
> On Wed, Jun 12, 2013 at 02:57:21PM +0200, Hans Verkuil wrote:
> > On Wed 12 June 2013 14:26:27 Sakari Ailus wrote:
> > > Hi Hans,
> > >
> > > Thanks for the RFC!
> > >
> > > On Wed, Jun 12, 2013 at 10:35:07AM +0200, Hans Verkuil wro
Op 17-06-13 13:15, Inki Dae schreef:
> This patch adds a buffer synchronization framework based on DMA BUF[1]
> and reservation[2] to use dma-buf resource, and based on ww-mutexes[3]
> for lock mechanism.
>
> The purpose of this framework is not only to couple cache operations,
> and buffer access
This patch adds a buffer synchronization framework based on DMA BUF[1]
and reservation[2] to use dma-buf resource, and based on ww-mutexes[3]
for lock mechanism.
The purpose of this framework is not only to couple cache operations,
and buffer access control to CPU and DMA but also to provide easy-
On Fri 14 June 2013 21:08:10 Guennadi Liakhovetski wrote:
> v11 of the V4L2 clock helper and asynchronous probing patch set.
> Functionally identical to v10, only differences are a couple of comment
> lines and one renamed struct field - as requested by respectable
> reviewers :)
>
> Only patch
On Mon 17 June 2013 11:43:13 Lubomir Rintel wrote:
> Reverse-engineered driver for cheapo video digitizer, made from observations
> of
> Windows XP driver. The protocol is not yet completely understood, so far we
> don't provide any controls, only support a single format out of three and
> don't
Reverse-engineered driver for cheapo video digitizer, made from observations of
Windows XP driver. The protocol is not yet completely understood, so far we
don't provide any controls, only support a single format out of three and don't
support the audio device.
Signed-off-by: Lubomir Rintel
Cc: M
Hi Sylwester,
On Mon, Jun 17, 2013 at 2:34 PM, Sylwester Nawrocki
wrote:
> On 06/17/2013 06:25 AM, Arun Kumar K wrote:
>> On Sat, Jun 15, 2013 at 1:01 AM, Sylwester Nawrocki
>> wrote:
>>> On 06/14/2013 03:21 PM, Arun Kumar K wrote:
On Fri, Jun 14, 2013 at 3:23 PM, Sylwester Nawrocki
On 16-06-13 10:15, Duval Mickael wrote:
It is true that only scans once! It can therefore be quite limited at
this fr_All file and remove fr_Cherbourg and fr_Bordeaux.
Allright, per your request I will merge Cherbourg and Bordeaux into
fr-All (if there are missing frequencies) and I'll use your
Signed-off-by: Sylwester Nawrocki
---
Hi Arun,
Perhaps you find this patch useful for your VP8 controls works.
It's not complete and I didn't test it, except ensuring it compiles.
Thanks,
Sylwester
---
Documentation/video4linux/v4l2-controls.txt | 21 ++--
drivers/media/v4l2-c
Hi Arun,
On 06/17/2013 06:25 AM, Arun Kumar K wrote:
> On Sat, Jun 15, 2013 at 1:01 AM, Sylwester Nawrocki
> wrote:
>> On 06/14/2013 03:21 PM, Arun Kumar K wrote:
>>> On Fri, Jun 14, 2013 at 3:23 PM, Sylwester Nawrocki
>>> wrote:
On 06/14/2013 11:26 AM, Arun Kumar K wrote:
>>> + sta
Hi Sylwester,
On Mon, Jun 17, 2013 at 2:46 AM, Sylwester Nawrocki
wrote:
> Hi,
>
>
> On 06/12/2013 06:44 AM, Prabhakar Lad wrote:
>>
>> On Tue, Jun 11, 2013 at 4:20 PM, Sakari Ailus wrote:
>>>
>>> media_entity_cleanup() frees the links array which will be accessed by
>>> media_entity_remove_link
72 matches
Mail list logo
