On Mon, 29 Sep 2014 23:41:09 +0300
Laurent Pinchart wrote:
> Hi Boris,
>
> Thank you for the patch.
>
> On Monday 29 September 2014 16:02:39 Boris Brezillon wrote:
> > Rename mediabus formats and move the enum into a separate header file so
> > that it can be used by DRM/KMS subsystem without a
On Mon, Sep 29, 2014 at 08:44:28PM +0200, Johannes Stezenbach wrote:
> On Mon, Sep 29, 2014 at 03:30:18PM -0300, Mauro Carvalho Chehab wrote:
> > Em Mon, 29 Sep 2014 19:44:30 +0200
> > Johannes Stezenbach escreveu:
> >
> > > Disregarding your mails from the "em28xx breaks after hibernate"
> > > t
Hi,
The problem can also come from register address or bit offset because
depending source used (GStreamer for 2.6.22 or new coda kernel driver
3.6 or datasheet), registers do not have same addresses or do not even
exists !
On 09/30/2014 08:00 AM, Gaëtan Carlier wrote:
Hi Philipp and Fabio,
Am Montag, den 29.09.2014, 18:10 -0400 schrieb Greg Kroah-Hartman:
> On Mon, Sep 29, 2014 at 08:03:34PM +0200, Philipp Zabel wrote:
> > Decrementing the reference count of the previous endpoint node allows to
> > use the of_graph_get_next_endpoint function in a for_each_... style macro.
> > All cur
On Tue, Sep 30, 2014 at 09:37:57AM +0200, Boris Brezillon wrote:
> On Mon, 29 Sep 2014 23:41:09 +0300
> Laurent Pinchart wrote:
[...]
> > Incidentally, patch 2/5 in this series is missing a documentation update ;-)
>
> Yep, regarding this patch, I wonder if it's really necessary to add
> new form
I referring to the following bug:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1362358
I've run some tests and after increasing verbosity for uvcvideo, I get:
EOF on empty payload
this seems consistent with the zero size frames returned by the driver.
After VIDIOC_DQBUF | VIDIOC_QBUF, I
Em Tue, 30 Sep 2014 09:38:10 +0200
Johannes Stezenbach escreveu:
> On Mon, Sep 29, 2014 at 08:44:28PM +0200, Johannes Stezenbach wrote:
> > On Mon, Sep 29, 2014 at 03:30:18PM -0300, Mauro Carvalho Chehab wrote:
> > > Em Mon, 29 Sep 2014 19:44:30 +0200
> > > Johannes Stezenbach escreveu:
> > >
>
On Tue, 30 Sep 2014 10:39:53 +0200
Thierry Reding wrote:
> On Tue, Sep 30, 2014 at 09:37:57AM +0200, Boris Brezillon wrote:
> > On Mon, 29 Sep 2014 23:41:09 +0300
> > Laurent Pinchart wrote:
> [...]
> > > Incidentally, patch 2/5 in this series is missing a documentation update
> > > ;-)
> >
>
This patch adds JPEG encoding and decoding support for CODA7541,
using the BIT processor. Separate JPEG encoder and decoder video
devices are created due to different supported pixel formats.
The hardware can not change subsampling on the fly, but encode
and decode 4:2:2 subsampled JPEG images from
Add JPEG specific sequence initialization registers and bit definitions.
Signed-off-by: Lucas Stach
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda/coda_regs.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/media/platform/coda/coda_regs.h
b/drivers/media/platfo
Storing the buffer position in the bitstream with the buffer metadata
allows to later use that information to drop metadata for skipped buffers
and to determine whether bitstream padding has to be applied.
This patch also renames struct coda_timestamp to struct coda_buffer_meta
to make clear that
With bitstream padding, it is possible to decode a single JPEG in the bitstream
immediately. This allows us to only ever queue a single JPEG into the bitstream
buffer, except to increase payload over 512 bytes or to back out of hold state.
This is a measure to decrease JPEG decoder latency.
Signed
Before starting a PIC_RUN, pad the bitstream with 0xff until 256 bytes
past the next multiple of 256 bytes, if the buffer to be decoded is the
last buffer in the bitstream.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda/coda-bit.c | 20
1 file changed, 20 inser
For small frame sizes, allocating 1 MiB per compressed buffer is a waste of
space. On the other hand, incompressible 1080p data can produce JPEGs larger
than 1 MiB at higher quality settings. Allow userspace to set the compressed
buffer size and clamp the value to a sensible range.
Also set the ini
Hi,
These patches add JPEG encoding and decoding support for CODA7541 (i.MX5).
The encoder video device is split into one video device per codec, so that
each video device can register only the relevant controls. The H.264/MPEG4
decoder is kept as one video device, but the JPEG decoder video devic
This patch adds support for the three-plane YUV422P format with one luma plane
and two horizontally subsampled chroma planes.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda/coda-bit.c| 14 +-
drivers/media/platform/coda/coda-common.c | 13 +
2 files chan
This patch splits the H.264/MPEG4 encoder specific controls out of the main
control setup function. This way each video device registers only relevant
controls.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda/coda-common.c | 66 ++-
1 file changed, 39 ins
Depending on the devtype, the coda driver will now spawn a number of video
devices, one separate encoder for each format, and one multi-format decoder.
Each video device descriptor determines the name, callback ops, and input and
output formats on the corresponding video device. This also simplifi
We'll use this information to decide whether to request the JPEG IRQ later.
Signed-off-by: Philipp Zabel
---
drivers/media/platform/coda/coda-common.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/drivers/media/platform/coda/coda-common.c
b/drivers/me
Ok,
so I've set a workaround in guvcview, it now uses the length filed if
bytesused is set to zero.
Anyway I think this violates the v4l2 api:
http://linuxtv.org/downloads/v4l-dvb-apis/buffer.html
bytesused - ..., Drivers must set this field when type refers to an
input stream, ...
without this v
Le 2014-09-30 04:50, Paulo Assis a écrit :
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1362358
I've run some tests and after increasing verbosity for uvcvideo, I get:
EOF on empty payload
this seems consistent with the zero size frames returned by the driver.
After VIDIOC_DQBUF | VIDI
On 09/30/14 11:57, Philipp Zabel wrote:
> Hi,
>
> These patches add JPEG encoding and decoding support for CODA7541 (i.MX5).
> The encoder video device is split into one video device per codec, so that
> each video device can register only the relevant controls. The H.264/MPEG4
> decoder is kept a
lockdep reports a potential circular dependecy deadlock when registering input
device.
Unlock mutex rc_dev->lock prior to calling ir_raw_event_register to avoid the
circular
dependency since that function also calls input_register_device and rc_open.
===
Yes,
but disabling libv4l2 and accessing the driver directly, has the same
effect. Also like I said, gspca works fine, even through libv4l2.
So I'm pretty sure this is related to uvcvideo.
My libv4l2 version is 1.2.0 and the reported problems with libv4l2
I've received relate to version 1.4.0 (debi
Hi,
On 09/29/2014 11:23 PM, Nicolas Dufresne wrote:
> This was initially reported to GStreamer project:
> https://bugzilla.gnome.org/show_bug.cgi?id=737521
>
> We track this down to be a regression introduced in v4l2-utils from version
> 1.4.0. In recent GStreamer we make sure the buffer.length
Hi Laurent,
Here is an initial review of this driver:
On 09/29/14 22:27, Laurent Pinchart wrote:
> Xilinx platforms have no hardwired video capture or video processing
> interface. Users create capture and memory to memory processing
> pipelines in the FPGA fabric to suit their particular needs,
Read the original pixelformat from dest_fmt, before overwriting dest_fmt with
the new src_fmt.
Signed-off-by: Hans de Goede
---
lib/libv4l2/libv4l2.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/libv4l2/libv4l2.c b/lib/libv4l2/libv4l2.c
index fe513d7..22ed984 1006
We need convert_mmap_frame_size well before we need the convert-mmap buffer
itself, and thus well before we call v4l2_ensure_convert_mmap_buf.
This fixes querybuf returning a length of 0 for fake buffers.
Signed-off-by: Hans de Goede
---
lib/libv4l2/libv4l2-priv.h | 1 +
lib/libv4l2/libv4l2.c
So that we always do the alignment, this is necessary because there are
several code paths where dest_fmt gets set to a fmt which has not been
passed through libv4lconvert_try_fmt, and thus is not aligned.
Also call v4lconvert_fixup_fmt when aligment has changed the height / width,
so that bytespe
Hi Hans,
Am Dienstag, den 30.09.2014, 15:43 +0200 schrieb Hans Verkuil:
> On 09/30/14 11:57, Philipp Zabel wrote:
> > Hi,
> >
> > These patches add JPEG encoding and decoding support for CODA7541 (i.MX5).
> > The encoder video device is split into one video device per codec, so that
> > each vide
On 09/30/14 16:20, Philipp Zabel wrote:
> Hi Hans,
>
> Am Dienstag, den 30.09.2014, 15:43 +0200 schrieb Hans Verkuil:
>> On 09/30/14 11:57, Philipp Zabel wrote:
>>> Hi,
>>>
>>> These patches add JPEG encoding and decoding support for CODA7541 (i.MX5).
>>> The encoder video device is split into one
Hi,
> From: Hans Verkuil [mailto:hverk...@xs4all.nl]
> Sent: Tuesday, September 30, 2014 4:25 PM
> To: Philipp Zabel
> Cc: Kamil Debski; Mauro Carvalho Chehab; Hans Verkuil; linux-
> me...@vger.kernel.org; ker...@pengutronix.de
> Subject: Re: [PATCH 00/10] CODA7 JPEG support
>
> On 09/30/14 16:20
On 09/30/2014 12:14 PM, Mauro Carvalho Chehab wrote:
Em Tue, 30 Sep 2014 09:38:10 +0200
Johannes Stezenbach escreveu:
On Mon, Sep 29, 2014 at 08:44:28PM +0200, Johannes Stezenbach wrote:
On Mon, Sep 29, 2014 at 03:30:18PM -0300, Mauro Carvalho Chehab wrote:
Ah, OK. I'll try to test with
CC: Andre Roth
Signed-off-by: Gregor Jasny
---
TODO.libdvbv5| 1 +
lib/include/libdvbv5/desc_service_location.h | 107 ---
lib/libdvbv5/Makefile.am | 3 +-
lib/libdvbv5/compat-soname.c |
Signed-off-by: Gregor Jasny
---
TODO.libdvbv5 | 2 ++
1 file changed, 2 insertions(+)
create mode 100644 TODO.libdvbv5
diff --git a/TODO.libdvbv5 b/TODO.libdvbv5
new file mode 100644
index 000..21091c4
--- /dev/null
+++ b/TODO.libdvbv5
@@ -0,0 +1,2 @@
+- On next SONAME bump
+ - clean up dv
CC: Andre Roth
Signed-off-by: Gregor Jasny
---
doxygen_libdvbv5.cfg | 1 -
lib/include/libdvbv5/desc_service_list.h | 119 ---
lib/libdvbv5/Makefile.am | 2 -
lib/libdvbv5/compat-soname.c | 8 ++
lib/li
Hello,
this patch series removes the service_list and service_location
descriptors. It breaks API compatibility by removing the public
header files but maintains ABI compatibility by providing stub
implementations.
Thanks,
Gregor
Gregor Jasny (3):
libdvbv5: Add todo file
libdvbv5: remove ser
Hello,
On 27/09/14 13:54, Mauro Carvalho Chehab wrote:
> Em Sat, 27 Sep 2014 12:57:11 +0200
> Gregor Jasny escreveu:
>> As far as I understand the service_location feature should work but is
>> an extension to the standard. Does it harm if we keep it until we have
>> something better in place to
Hi,
On 09/30/2014 05:20 PM, Gregor Jasny wrote:
Hello,
On 27/09/14 13:54, Mauro Carvalho Chehab wrote:
Em Sat, 27 Sep 2014 12:57:11 +0200
Gregor Jasny escreveu:
As far as I understand the service_location feature should work but is
an extension to the standard. Does it harm if we keep it unt
Hello,
After updating the kernel to 3.14.15 I am seeing these messages:
[273684.964081] saa7146: saa7146 (0): saa7146_wait_for_debi_done_sleep
timed out while waiting for registers getting programmed
[273690.020061] saa7146: saa7146 (0): saa7146_wait_for_debi_done_sleep
timed out while waiting
Hi Boris,
On Tuesday 30 September 2014 11:44:23 Boris Brezillon wrote:
> On Tue, 30 Sep 2014 10:39:53 +0200 Thierry Reding wrote:
> > On Tue, Sep 30, 2014 at 09:37:57AM +0200, Boris Brezillon wrote:
> >> On Mon, 29 Sep 2014 23:41:09 +0300 Laurent Pinchart wrote:
> >
> > [...]
> >
> >>> Incidental
Commit e93e7fd9f5a3fffec7792dbcc4c3574653effda7 ("v4l2: uvcvideo: Allow
using larger buffers") reworked the buffer size sanity check at buffer
completion time to use the frame size instead of the allocated buffer
size. However, it introduced two bugs in doing so:
- it assigned the allocated buffer
On Wednesday 01 October 2014 00:42:51 Laurent Pinchart wrote:
> Commit e93e7fd9f5a3fffec7792dbcc4c3574653effda7 ("v4l2: uvcvideo: Allow
> using larger buffers") reworked the buffer size sanity check at buffer
> completion time to use the frame size instead of the allocated buffer
> size. However, i
Hi Paulo,
Thank you for investigation this.
On Tuesday 30 September 2014 13:56:15 Paulo Assis wrote:
> Ok,
> so I've set a workaround in guvcview, it now uses the length filed if
> bytesused is set to zero.
> Anyway I think this violates the v4l2 api:
> http://linuxtv.org/downloads/v4l-dvb-apis/b
Em Tue, 30 Sep 2014 17:17:05 +0200
Gregor Jasny escreveu:
> Hello,
>
> this patch series removes the service_list and service_location
> descriptors. It breaks API compatibility by removing the public
> header files but maintains ABI compatibility by providing stub
> implementations.
>
> Thanks
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: Wed Oct 1 04:00:20 CEST 2014
git branch: test
git hash: cf3167cf1e969b17671a4d3d956d22718a8ceb85
gcc versi
This is a preparation for deleting the otherwise useless i2c_clients
that are allocated for all the i2c master adapters.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-cards.c | 24 +---
1 file changed, 13 insertions(+),
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/media/usb/cx231xx/cx231xx-i2c.c
b/drivers/media/usb/cx231xx/cx231xx-i2c.c
index a30d400..178fa48 100644
--- a/drivers/media/usb/cx231xx/cx231xx-i2c.c
++
This is a preparation for deleting the otherwise useless i2c_clients
that are allocated for all the i2c master adapters.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 17 +++--
drivers/media/usb/cx231xx/cx231xx.h |
This series changes cx231xx driver to use standard muxed i2c busses.
Everything works as before (tested with Hauppauge WinTV-930C-HD).
Also the scanning is changed to these new busses, but still does not work (as
before).
Change scanning to read 1 byte instead of 0 only works for one bus.
V2: T
For each i2c master there is a i2c_client allocated that could be
deleted now that its only two users have been changed to use their
own i2c_client.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 7 ---
drivers/media/usb/cx231xx
Get used i2c port from bus_nr and status of port_3 switch.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/cx231xx/cx231xx-i2c.c
b/drivers/media/usb/cx231xx/cx231xx-i
Replace numbers by the constants of same value and same meaning.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-cards.c | 62 +++
1 file changed, 31 insertions(+), 31 deletions(-)
diff --git a/drivers/media/usb/
If remembering is not stable enough, this must be changed to query from the
register when needed.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-avcore.c | 3 +++
drivers/media/usb/cx231xx/cx231xx.h| 1 +
2 files changed, 4 insertions(+)
diff --git a/drivers/m
Change to I2C_0 ... I2C_2 for the master ports
and add I2C_1_MUX_1 and I2C_1_MUX_3 for the muxed ones.
V2: Renamed mux adapters to seperate them from master adapters.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx.h | 9 +
1 file changed, 5 insertions(+), 4 del
This is a preparation for mapping I2C_1_MUX_1 and I2C_1_MUX_3 later to the
seperate
muxed i2c adapters.
Map mux adapters to I2C_1 for now.
Add local variables for i2c_adapters in dvb_init to get line lengths
shorter.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-car
change almost all instances of I2C_1 to I2C_1_MUX_3
Only these cases are changed to I2C_1_MUX_1:
* All that have dont_use_port_3 set.
* CX231XX_BOARD_HAUPPAUGE_EXETER, old code did explicitly not switch to port3.
* eeprom access for 930C
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/
The scanning itself just fails (as before this series) but now the correct
busses are scanned.
V2: Changed to symbolic names where muxed adapters can be seen directly.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-core.c | 6 ++
drivers/media/usb/cx231xx/cx231xx-
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/Kconfig| 1 +
drivers/media/usb/cx231xx/cx231xx-core.c | 5
drivers/media/usb/cx231xx/cx231xx-i2c.c | 44 +++-
drivers/media/usb/cx231xx/cx231xx.h | 4 +++
4 files changed, 53 insert
All switching should be done by i2c mux adapters.
Drop explicit dont_use_port_3 flag.
Drop info message about switch.
Only the removed code in start_streaming is questionable:
It did switch the port_3 flag without accessing i2c in between.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Pa
The new file has a different name: dvb-demod-si2165-d.fw
Count blocks instead of reading count from extra header.
Calculate CRC during upload and compare result to what chip calcuated.
Use 0x01 instead of real patch version, because this is only used to
check if something was uploaded but not to c
Store it with a different name based on hardware revision.
V2: change firmware filename to lower case.
Signed-off-by: Matthias Schwarzott
---
Documentation/dvb/get_dvb_firmware | 20
1 file changed, 4 insertions(+), 16 deletions(-)
diff --git a/Documentation/dvb/get_dvb_fi
On 01.10.2014 07:58, Matthias Schwarzott wrote:
> The new file has a different name: dvb-demod-si2165-d.fw
>
> Count blocks instead of reading count from extra header.
> Calculate CRC during upload and compare result to what chip calcuated.
> Use 0x01 instead of real patch version, because this is
63 matches
Mail list logo
