Re: go7007 question

2012-09-10 Thread volokh
On Sat, Sep 08, 2012 at 10:23:31PM -0400, Adam Rosi-Kessel wrote: On Fri, Sep 07, 2012 at 06:18:31PM +0400, vol...@telros.ru wrote: On Thu, Sep 06, 2012 at 11:10:14PM +0400, Volokh Konstantin wrote: On Mon, Sep 03, 2012 at 02:37:16PM -0400, Adam Rosi-Kessel wrote: [469.928881]

Re: go7007 question

2012-09-10 Thread volokh
On Sun, Sep 09, 2012 at 08:21:55AM -0400, Adam Rosi-Kessel wrote: On Sat, Sep 08, 2012 at 10:23:31PM -0400, Adam Rosi-Kessel wrote: On Fri, Sep 07, 2012 at 06:18:31PM +0400, vol...@telros.ru wrote: On Thu, Sep 06, 2012 at 11:10:14PM +0400, Volokh Konstantin wrote: e On Mon, Sep 03, 2012

Re: [PATCH v5] media: v4l2-ctrls: add control for dpcm predictor

2012-09-10 Thread Sakari Ailus
Hi Prabhakar, On Mon, Sep 10, 2012 at 11:27:55AM +0530, Prabhakar Lad wrote: ... + row id=v4l2-dpcm-predictor + entry spanname=descr Differential pulse-code modulation (DPCM) compression can + be used to compress the samples into fewer bits than they would otherwise

Re: [PATCH v3 03/16] media: coda: fix IRAM/AXI handling for i.MX53

2012-09-10 Thread Richard Zhao
On Fri, Aug 31, 2012 at 10:10:57AM +0200, Philipp Zabel wrote: This uses the ARCH_MXC specific iram_alloc API to allocate a work buffer in the SoC's on-chip SRAM and sets up the AXI_SRAM_USE register. In the future, the allocation will be converted to use the genalloc API. Signed-off-by:

Re: [PATCH 01/10] staging: media: go7007: Some additional code for TW2804 driver functionality

2012-09-10 Thread Dan Carpenter
This patch got corrupted somewhere so it doesn't apply. There are some minor style issues as well. On Wed, Aug 22, 2012 at 02:45:10PM +0400, Volokh Konstantin wrote: - using new v4l2 framework controls - function for reading volatile controls via i2c bus - separate V4L2_CID_ ctrls into each

Re: [PATCH v3 03/16] media: coda: fix IRAM/AXI handling for i.MX53

2012-09-10 Thread Philipp Zabel
Hi Richard, thank you for your comments. Am Montag, den 10.09.2012, 16:20 +0800 schrieb Richard Zhao: On Fri, Aug 31, 2012 at 10:10:57AM +0200, Philipp Zabel wrote: This uses the ARCH_MXC specific iram_alloc API to allocate a work buffer in the SoC's on-chip SRAM and sets up the

Re: [PATCH] Support for Asus MyCinema U3100Mini Plus

2012-09-10 Thread Antti Palosaari
On 09/10/2012 12:58 PM, Oliver Schinagl wrote: Changed the address as recommended, which after reading 7bit and 8bit addressing makes perfect sense (drop the r/w bit and get the actual address). static struct fc2580_config af9035_fc2580_config = { - .i2c_addr = 0xac, + .i2c_addr =

[PATCH] media: v4l2-ctrl: add a helper fucntion to modify the menu

2012-09-10 Thread Prabhakar Lad
From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com Cc: Hans Verkuil hans.verk...@cisco.com Cc: Sakari Ailus sakari.ai...@iki.fi Cc: Sylwester Nawrocki s.nawro...@samsung.com Cc: Laurent Pinchart

[PATCH v6] media: v4l2-ctrls: add control for dpcm predictor

2012-09-10 Thread Prabhakar Lad
From: Lad, Prabhakar prabhakar@ti.com add V4L2_CID_DPCM_PREDICTOR control of type menu, which determines the dpcm predictor. The predictor can be either simple or advanced. Signed-off-by: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com Acked-by: Hans

Re: [PATCH] media: v4l2-ctrl: add a helper fucntion to modify the menu

2012-09-10 Thread Hans Verkuil
On Mon September 10 2012 13:57:36 Prabhakar Lad wrote: From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com Cc: Hans Verkuil hans.verk...@cisco.com Cc: Sakari Ailus sakari.ai...@iki.fi Cc:

Re: [PATCH] media: v4l2-ctrl: add a helper fucntion to modify the menu

2012-09-10 Thread Manjunath Hadli
Hi Hans, Thanks for the review. On Monday 10 September 2012 05:43 PM, Hans Verkuil wrote: On Mon September 10 2012 13:57:36 Prabhakar Lad wrote: From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com

Re: [PATCH] media: v4l2-ctrl: add a helper fucntion to modify the menu

2012-09-10 Thread Prabhakar Lad
On Monday 10 September 2012 06:06 PM, Manjunath Hadli wrote: Hi Hans, Thanks for the review. On Monday 10 September 2012 05:43 PM, Hans Verkuil wrote: On Mon September 10 2012 13:57:36 Prabhakar Lad wrote: From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad, Prabhakar

Re: [PATCH v2] drivers/media/pci/cx25821/cx25821-video-upstream-ch2.c: fix error return code

2012-09-10 Thread Peter Senna Tschudin
You're replacing kmemdup for kstrdup, which is great, but that's not anywhere in the commit message. Sorry for that. I'm not sure if you should re-send, but you should definitely try to have better commit messages in the future! I'll kindly ask to ignore the V2 of this patch. I'll send

[PATCH] drivers/media/pci/cx25821/cx25821-video-upstream-ch2.c: Replace kmemdup for kstrdup

2012-09-10 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Replace kmemdup for kstrdup and cleaning up the code. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- It depends on the patch http://patchwork.linuxtv.org/patch/14231/ tmp/cx25821-video-upstream-ch2.c | 27

Re: [PATCH 3/5] dvb_frontend: do not allow statistic IOCTLs when sleeping

2012-09-10 Thread Mauro Carvalho Chehab
Em 15-08-2012 21:28, Antti Palosaari escreveu: Demodulator cannot perform statistic IOCTLs when it is not tuned. Return -EAGAIN in such case. Signed-off-by: Antti Palosaari cr...@iki.fi --- drivers/media/dvb-core/dvb_frontend.c | 34 +- 1 file changed, 25

Re: [PATCH] Support for Asus MyCinema U3100Mini Plus

2012-09-10 Thread Oliver Schinagl
On 10-09-12 13:46, Antti Palosaari wrote: On 09/10/2012 12:58 PM, Oliver Schinagl wrote: Changed the address as recommended, which after reading 7bit and 8bit addressing makes perfect sense (drop the r/w bit and get the actual address). static struct fc2580_config af9035_fc2580_config = { -

Re: [PATCH 4/5] DocBook: update ioctl error codes

2012-09-10 Thread Mauro Carvalho Chehab
Em 15-08-2012 21:28, Antti Palosaari escreveu: ENOTTY is now returned for unimplemented ioctl by dvb-frontend. Old EOPNOTSUPP ENOSYS could be still returned by some drivers as well as other non standard error codes. EAGAIN is returned in case of device is in state where it cannot perform

Re: [PATCH 3/5] dvb_frontend: do not allow statistic IOCTLs when sleeping

2012-09-10 Thread Antti Palosaari
On 09/10/2012 05:27 PM, Mauro Carvalho Chehab wrote: Em 15-08-2012 21:28, Antti Palosaari escreveu: Demodulator cannot perform statistic IOCTLs when it is not tuned. Return -EAGAIN in such case. Signed-off-by: Antti Palosaari cr...@iki.fi --- drivers/media/dvb-core/dvb_frontend.c | 34

Re: (still) NULL pointer crashes with nuvoton_cir driver

2012-09-10 Thread Mauro Carvalho Chehab
Em 17-08-2012 12:04, Luis Henriques escreveu: (Adding Mauro to CC has he is the maintainer) I'm actually waiting for Jarod's comments on it, as he wrote this driver and the similar ones. Jarod? Regards, Mauro On Thu, Aug 16, 2012 at 10:09:32AM +0200, Matthijs Kooijman wrote: Hi folks,

Re: [RFC PATCH v6] media: add v4l2 subdev driver for S5K4ECGX sensor

2012-09-10 Thread Sylwester Nawrocki
Hi, On 09/09/2012 06:01 PM, Francesco Lavra wrote: +static int s5k4ecgx_load_firmware(struct v4l2_subdev *sd) +{ +const struct firmware *fw; +int err, i, regs_num; +struct i2c_client *client = v4l2_get_subdevdata(sd); +u16 val; +u32 addr, crc, crc_file, addr_inc = 0; +

[RFC API] Capture Overlay API ambiguities

2012-09-10 Thread Hans Verkuil
Hi all, While working on making bttv compliant with the V4L2 API I managed to resolve all v4l2-compliance errors except for two: fail: v4l2-test-formats.cpp(339): !fmt.width || !fmt.height test VIDIOC_G_FBUF: FAIL fail: v4l2-test-formats.cpp(607): Video

[PATCH v4 0/16] Initial i.MX5/CODA7 support for the CODA driver

2012-09-10 Thread Philipp Zabel
These patches contain initial firmware loading and encoding support for the CODA7 series VPU contained in i.MX51 and i.MX53 SoCs, and fix some multi-instance issues. Patches 13 and 14, touching files in arch/arm/*, are included for illustration purposes. Changes since v3: - Remove iram_vaddr

[PATCH v4 01/16] media: coda: firmware loading for 64-bit AXI bus width

2012-09-10 Thread Philipp Zabel
Add support for loading a raw firmware with 16-bit chars ordered in little-endian 64-bit words, corresponding to the memory access pattern of CODA7 and above: When writing the boot code into the code download register, the chars have to be reordered back. Signed-off-by: Philipp Zabel

[PATCH v4 03/16] media: coda: fix IRAM/AXI handling for i.MX53

2012-09-10 Thread Philipp Zabel
This uses the ARCH_MXC specific iram_alloc API to allocate a work buffer in the SoC's on-chip SRAM and sets up the AXI_SRAM_USE register. In the future, the allocation will be converted to use the genalloc API. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v3: - Remove

[PATCH v4 04/16] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-09-10 Thread Philipp Zabel
Some codecs running on CODA need internal framebuffers for reference and reconstructed frames. Allocate them separately, and do not use the input vb2_buffers: those will be handed off to userspace regularly, and there is no way to signal to the CODA which of the registered framebuffers are off

[PATCH v4 05/16] media: coda: ignore coda busy status in coda_job_ready

2012-09-10 Thread Philipp Zabel
job_ready is supposed to signal whether a context is ready to be added to the job queue, not whether the CODA is ready to run it immediately. Calling v4l2_m2m_job_finish at the end of coda_irq_handler already guarantees that the coda is ready when v4l2-mem2mem eventually tries to run the next

[PATCH v4 10/16] media: coda: fix sizeimage setting in try_fmt

2012-09-10 Thread Philipp Zabel
VIDIOC_TRY_FMT would incorrectly return bytesperline * height, instead of width * height * 3 / 2. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/coda.c

[PATCH v4 16/16] media: coda: support 1024 px height on CODA7, set max frame size to 1080p

2012-09-10 Thread Philipp Zabel
Increases the maximum encoded frame buffer size to 1 MiB. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda.c | 11 +-- drivers/media/platform/coda.h |3 ++- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git

[PATCH v4 02/16] media: coda: add i.MX53 / CODA7541 platform support

2012-09-10 Thread Philipp Zabel
Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda.c | 31 +++ 1 file changed, 31 insertions(+) diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index d4a5dd0..8ec2ff4 100644 ---

[PATCH v4 15/16] media: coda: set up buffers to be sized as negotiated with s_fmt

2012-09-10 Thread Philipp Zabel
This fixes a failure in vb2_qbuf in user pointer mode where __qbuf_userptr checks if the buffer queued by userspace is large enough. The failure would happen if coda_queue_setup was called with empty fmt (and thus set the expected buffer size to the maximum resolution), and userspace queues

[PATCH v4 14/16] ARM i.MX5: Add CODA7 to device tree for i.MX51 and i.MX53

2012-09-10 Thread Philipp Zabel
Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- arch/arm/boot/dts/imx51.dtsi |6 ++ arch/arm/boot/dts/imx53.dtsi |6 ++ 2 files changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi index aba28dc..8f38d83 100644 ---

[PATCH v4 12/16] media: coda: add byte size slice limit control

2012-09-10 Thread Philipp Zabel
Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda.c | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index 81e3401..863b96a 100644 ---

[PATCH v4 11/16] media: coda: add horizontal / vertical flipping support

2012-09-10 Thread Philipp Zabel
The hardware can also rotate in 90° steps, but there is no corresponding V4L2_CID defined yet. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda.c | 19 ++- drivers/media/platform/coda.h |9 + 2 files changed, 27 insertions(+), 1

[PATCH v4 13/16] ARM i.MX5: Fix CODA7 clock lookup for device tree on i.MX51 and i.MX53

2012-09-10 Thread Philipp Zabel
Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- arch/arm/mach-imx/clk-imx51-imx53.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c index 4bdcaa9..64f9ceb 100644 ---

[PATCH v4 09/16] media: coda: wait for picture run completion in start/stop_streaming

2012-09-10 Thread Philipp Zabel
While the CODA is running a PIC_RUN command, its registers are not to be touched. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v3: - Complete dev-done in coda_timeout. --- drivers/media/platform/coda.c | 42 - 1 file changed, 33

[PATCH v4 06/16] media: coda: keep track of active instances

2012-09-10 Thread Philipp Zabel
Determining the next free instance just by incrementing and decrementing an instance counter does not work: if there are two instances opened, 0 and 1, and instance 0 is released, the next call to coda_open will create a new instance with index 1, but instance 1 is already in use. Instead, scan a

[PATCH v4 07/16] media: coda: stop all queues in case of lockup

2012-09-10 Thread Philipp Zabel
Add a 1 second timeout for each PIC_RUN command to the CODA. In case it locks up, stop all queues and dequeue remaining buffers. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v3: - Add struct coda_dev *dev pointer in coda_start_streaming. - Call cancel_delayed_work in

[PATCH v4 08/16] media: coda: enable user pointer support

2012-09-10 Thread Philipp Zabel
USERPTR buffer support is provided by the videobuf2 framework. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index

Re: pac7302-webcams and libv4lconvert interaction

2012-09-10 Thread Frank Schäfer
Hi Hans, Am 09.09.2012 23:20, schrieb Hans de Goede: Hi, On 09/06/2012 05:13 PM, Frank Schäfer wrote: Hi, I'm currently looking into the gspca_pac7302-driver and how it interacts with libv4lconvert. This is how it currently works - driver announces v4l2_pix_format 640x480 (width x

Re: [PATCH 2/3] libv4lconvert: pac7302-devices: remove unneeded flag V4LCONTROL_WANTS_WB

2012-09-10 Thread Frank Schäfer
Am 09.09.2012 23:24, schrieb Hans de Goede: Hi, On 09/09/2012 08:36 PM, Frank Schäfer wrote: The gspca_pac7302 driver already provides this control. Signed-off-by: Frank Schäfer fschaefer@googlemail.com --- lib/libv4lconvert/control/libv4lcontrol.c | 12 1 files

Re: [PATCH] Support for Asus MyCinema U3100Mini Plus

2012-09-10 Thread Oliver Schinagl
On 09/10/12 16:29, Oliver Schinagl wrote: On 10-09-12 13:46, Antti Palosaari wrote: On 09/10/2012 12:58 PM, Oliver Schinagl wrote: Changed the address as recommended, which after reading 7bit and 8bit addressing makes perfect sense (drop the r/w bit and get the actual address). static struct

Re: pac7302-webcams and libv4lconvert interaction

2012-09-10 Thread Hans de Goede
Hi, On 09/10/2012 05:36 PM, Frank Schäfer wrote: Hi Hans, Am 09.09.2012 23:20, schrieb Hans de Goede: Hi, On 09/06/2012 05:13 PM, Frank Schäfer wrote: Hi, I'm currently looking into the gspca_pac7302-driver and how it interacts with libv4lconvert. This is how it currently works - driver

Re: [PATCH 2/3] libv4lconvert: pac7302-devices: remove unneeded flag V4LCONTROL_WANTS_WB

2012-09-10 Thread Hans de Goede
Hi, On 09/10/2012 05:41 PM, Frank Schäfer wrote: Am 09.09.2012 23:24, schrieb Hans de Goede: Hi, On 09/09/2012 08:36 PM, Frank Schäfer wrote: The gspca_pac7302 driver already provides this control. Signed-off-by: Frank Schäfer fschaefer@googlemail.com ---

Re: [RFC PATCH v6] media: add v4l2 subdev driver for S5K4ECGX sensor

2012-09-10 Thread Francesco Lavra
Hi, On 09/10/2012 05:04 PM, Sylwester Nawrocki wrote: Hi, On 09/09/2012 06:01 PM, Francesco Lavra wrote: +static int s5k4ecgx_load_firmware(struct v4l2_subdev *sd) +{ + const struct firmware *fw; + int err, i, regs_num; + struct i2c_client *client = v4l2_get_subdevdata(sd); +

cron job: media_tree daily build: WARNINGS

2012-09-10 Thread Hans Verkuil
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 Sep 10 19:00:18 CEST 2012 git hash:79e8c7bebb467bbc3f2514d75bba669a3f354324 gcc version: i686-linux-gcc

Re: pac7302-webcams and libv4lconvert interaction

2012-09-10 Thread Frank Schäfer
Hi, Am 10.09.2012 20:31, schrieb Hans de Goede: Hi, On 09/10/2012 05:36 PM, Frank Schäfer wrote: Hi Hans, Am 09.09.2012 23:20, schrieb Hans de Goede: Hi, On 09/06/2012 05:13 PM, Frank Schäfer wrote: Hi, I'm currently looking into the gspca_pac7302-driver and how it interacts with

Re: [RFC PATCH v6] media: add v4l2 subdev driver for S5K4ECGX sensor

2012-09-10 Thread Sylwester Nawrocki
On 09/10/2012 08:52 PM, Francesco Lavra wrote: On 09/10/2012 05:04 PM, Sylwester Nawrocki wrote: On 09/09/2012 06:01 PM, Francesco Lavra wrote: +static int s5k4ecgx_load_firmware(struct v4l2_subdev *sd) +{ + const struct firmware *fw; + int err, i, regs_num; + struct i2c_client *client =

Re: [PATCH 2/3] libv4lconvert: pac7302-devices: remove unneeded flag V4LCONTROL_WANTS_WB

2012-09-10 Thread Frank Schäfer
Am 10.09.2012 20:33, schrieb Hans de Goede: Hi, On 09/10/2012 05:41 PM, Frank Schäfer wrote: Am 09.09.2012 23:24, schrieb Hans de Goede: Hi, On 09/09/2012 08:36 PM, Frank Schäfer wrote: The gspca_pac7302 driver already provides this control. Signed-off-by: Frank Schäfer

Re: [RFC API] Capture Overlay API ambiguities

2012-09-10 Thread Andy Walls
On Mon, 2012-09-10 at 17:14 +0200, Hans Verkuil wrote: Hi all, While working on making bttv compliant with the V4L2 API I managed to resolve all v4l2-compliance errors except for two: fail: v4l2-test-formats.cpp(339): !fmt.width || !fmt.height test VIDIOC_G_FBUF:

Re: go7007 question

2012-09-10 Thread Adam Rosi-Kessel
On 9/10/2012 6:37 AM, vol...@telros.ru wrote: I`m not seeing wis-sa7115 module, (it`s main module for video device) OK, I recompiled with that built in. I had left it out because I had trouble compiling before, but now it went through fine. Seems to help. After recompiling, I have some

[PATCH 1/2] dvb_usb_v2: rename module dvb_usbv2 = dvb_usb_v2

2012-09-10 Thread Antti Palosaari
I think it is better name. At that phase renaming is quite painless as module is not yet merged to mainline Kernel. Signed-off-by: Antti Palosaari cr...@iki.fi --- drivers/media/usb/dvb-usb-v2/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 2/2] dvb_usb_v2: call streaming_ctrl() before kill urbs

2012-09-10 Thread Antti Palosaari
Logically it is better ask hardware to stop streaming before killing urbs carrying stream. Earlier it was just opposite. Now code runs: * submit urbs * start streaming ** streaming ongoing ** * stop streaming * kill urbs Signed-off-by: Antti Palosaari cr...@iki.fi ---

ISPsupport

2012-09-10 Thread John Tobias
Hi all, I tried devel-ISPSUPPORT-IPIPE and devel-ISPSUPPORT, the kernel detected my image sensor (ov5650). But, when I execute the yavta /dev/video0 -c4 -n1 -s2592x1944 -fSGRBG10 -Fov5650-2592x1944-#.bin I was getting Unable to start streaming: Invalid argument (22).. I would like to know if