[PATCH 1/1] s2255drv: update firmware load.

2017-11-03 Thread Dean Anderson
fixes intermittent soft reboot issue with firmware load increases wait time of reset, as required by HW Signed-off-by: Dean Anderson <d...@sensoray.com> --- drivers/media/usb/s2255/s2255drv.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers

[PATCH] media: usb: s2255drv: update to firmware loading

2017-09-29 Thread dean
fixes intermittent soft reboot issue with firmware load increases wait time of reset, as required by HW Signed-off-by: Dean Anderson <d...@sensoray.com> --- drivers/media/usb/s2255/s2255drv.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/d

[PATCH] [media] s2255drv: fix payload size for JPG, MJPEG

2014-11-17 Thread Dean Anderson
Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b/drivers/media/usb/s2255/s2255drv.c index a56a05b..82014bb 100644 --- a/drivers/media/usb/s2255

[PATCH] [media] s2255drv: fix spinlock issue

2014-11-04 Thread Dean Anderson
qlock spinlock controls access to buf_list and sequence. qlock spinlock should not be locked during a copy to video buffers, an operation that may sleep. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 23 +++ 1 file changed, 11

Re: [PATCH] [media] s2255drv: fix spinlock issue

2014-11-04 Thread Dean Anderson
. On 2014-11-04 14:34, Dean Anderson wrote: qlock spinlock controls access to buf_list and sequence. qlock spinlock should not be locked during a copy to video buffers, an operation that may sleep. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 23

[PATCH] s2255drv: urgent memory leak fix.

2014-02-28 Thread Dean Anderson
Critical bug fix. Fixes memory leak introduced by commit 47d8c881c304642a68d398b87d9e8846e643c81a. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c |5 - 1 file changed, 5 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b

Re: [PATCH] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
, Failed: 3, Warnings: 9 On 2014-02-11 17:07, Hans Verkuil wrote: Hi Dean, On 02/12/2014 12:05 AM, Dean Anderson wrote: The output of v4l2-compliance looks fine. The warnings can be ignored for this patch. Can you update to the latest v4l2-compliance that I just pushed this morning? I

[PATCH v2] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
Upgrade to videobuf2 libraries. No errors reported with v4l2-compliance -s. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 512 +++- 1 file changed, 152 insertions(+), 360 deletions(-) diff --git a/drivers/media

Re: [PATCH v2] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK test read/write: OK test MMAP: OK test USERPTR: OK Total: 39, Succeeded: 39, Failed: 0, Warnings: 9 On 2014-02-12 14:25, Dean Anderson wrote: Upgrade to videobuf2 libraries. No errors reported with v4l2-compliance -s. Signed-off-by: Dean

Re: [PATCH] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
:01, Dean Anderson wrote: ./utils/v4l2-compliance/v4l2-compliance -s Driver Info: Driver name : s2255 Card type : s2255 Bus info : usb-:00:1a.7-3.6 Driver version: 3.13.0 Capabilities : 0x8401 Video Capture Streaming Device

Re: [PATCH] s2255drv: upgrade to videobuf2

2014-02-11 Thread Dean Anderson
not supported test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK Total: 36, Succeeded: 36, Failed: 0, Warnings: 9 On 2014-02-11 16:56, Dean Anderson wrote: Upgrade to videobuf2 libraries. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 507

[PATCH] s2255drv: upgrade to videobuf2

2014-02-11 Thread Dean Anderson
Upgrade to videobuf2 libraries. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 507 +++- 1 file changed, 148 insertions(+), 359 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b/drivers/media/usb/s2255

[PATCH] s2255drv: remove redundant parameter

2014-02-05 Thread Dean Anderson
Removing duplicate fmt from buffer structure. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b/drivers/media/usb/s2255/s2255drv.c index

[PATCH] s2255drv: buffer setup fix

2014-02-05 Thread Dean Anderson
Buffer setup should check if minimum number of buffers is used. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b/drivers/media

[PATCH] s2255drv: fix for return code not checked

2014-02-05 Thread Dean Anderson
Start acquisition return code was not being checked. Return error if start acquisition fails. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/media/usb/s2255

[PATCH] s2255drv: dynamic memory allocation efficiency fix

2014-02-05 Thread Dean Anderson
Driver was allocating a kernel buffer each time it was sending a command. It is better to allocate this buffer once at startup. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 67 1 file changed, 37 insertions

[PATCH v2] s2255drv: fix for return code not checked

2014-02-05 Thread Dean Anderson
Start acquisition return code was not being checked. Return error if start acquisition fails. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/drivers/media/usb

[PATCH] s2255drv: cleanup of s2255_fh

2014-02-05 Thread Dean Anderson
Removal of unnecessary parameters from s2255_fh. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 94 +--- 1 file changed, 44 insertions(+), 50 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b/drivers

Re: [PATCH] s2255drv: port to videobuf2

2014-02-05 Thread Dean Anderson
On 2014-02-04 04:04, Hans Verkuil wrote: Hi Dean, On 02/03/14 18:06, Dean Anderson wrote: On 2014-02-03 03:51, Hans Verkuil wrote: Hi Dean, Some specific comments below, but first two general comments: It is easier to review if at least the removal of the old s2255_fh struct was done

Re: [PATCH] s2255drv: port to videobuf2

2014-02-04 Thread Dean Anderson
On 2014-02-04 04:04, Hans Verkuil wrote: Hi Dean, On 02/03/14 18:06, Dean Anderson wrote: On 2014-02-03 03:51, Hans Verkuil wrote: Hi Dean, Some specific comments below, but first two general comments: It is easier to review if at least the removal of the old s2255_fh struct was done

[PATCH] s2255drv: removal of s2255_dmaqueue structure

2014-02-04 Thread Dean Anderson
Removal of unused and unnecessary s2255dma_queue structure. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 29 ++--- 1 file changed, 10 insertions(+), 19 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b

[PATCH] s2255drv: refactoring s2255_channel to s2255_vc

2014-02-04 Thread Dean Anderson
Renaming s2255_channel and all instances of channel to vc (video channel). Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 469 ++-- 1 file changed, 235 insertions(+), 234 deletions(-) diff --git a/drivers/media/usb

Re: [PATCH] s2255drv: refactoring s2255_channel to s2255_vc

2014-02-04 Thread Dean Anderson
There are some missing changes. It compiles, but please review v2 instead. On 2014-02-04 15:04, Dean Anderson wrote: Renaming s2255_channel and all instances of channel to vc (video channel). Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 469

[PATCH v2] s2255drv: refactoring s2255_channel to s2255_vc

2014-02-04 Thread Dean Anderson
Renaming s2255_channel and all instances of channel to vc (video channel). Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 517 ++-- 1 file changed, 259 insertions(+), 258 deletions(-) diff --git a/drivers/media/usb

[PATCH] s2255drv: file handle cleanup

2014-02-04 Thread Dean Anderson
videobuf_queue_is_busy. videobuf_queue_is busy may be unsafe as noted by the following comment in videobuf-core.c: /* Locking: Only usage in bttv unsafe find way to remove */ Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 224

Re: [PATCH] s2255drv: file handle cleanup

2014-02-04 Thread Dean Anderson
, Dean Anderson wrote: Removes most parameters from s2255_fh. These elements belong in s2255_ch. In the future, s2255_fh will be removed when videobuf2 is used. videobuf2 has convenient and safe functions for locking streaming resources. The removal of s2255_fh (and s2255_fh-resources) was not done

Re: [PATCH]: s2255drv: checkpatch fix: coding style fix

2014-01-29 Thread dean
with dprintk? dev wasn't available in this function without a back-reference from channel, which may be a good idea. The other issue you mention below is also important. I'll update this patch. Thanks, On 2014-01-29 01:19, Hans Verkuil wrote: Hi Dean, A few small nitpicks below... On 01/29/2014

[PATCH v2]: s2255drv: checkpatch fix: coding style fix

2014-01-29 Thread Dean Anderson
Fixes all style warnings from scripts/checkpatch -f Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 333 1 file changed, 151 insertions(+), 182 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b

[PATCH]: s2255drv: checkpatch fix: coding style fix

2014-01-28 Thread dean
Fixes all style warnings from scripts/checkpatch -f Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c | 328 1 file changed, 147 insertions(+), 181 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b

[PATCH] S2255: Removal of unnecessary videobuf_queue_is_busy

2013-07-23 Thread Dean Anderson
Removes unnecessary query of buffer state. The code already checks if stream is active or not. Signed-off-by: Dean Anderson linux-...@sensoray.com --- drivers/media/usb/s2255/s2255drv.c |9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/media/usb/s2255

Re: [patch 2/2] [media] s2255drv: fix some endian bugs

2012-02-22 Thread dean anderson
Hi Dan, The original code has an issue on big endian hardware. The patch looks ok. Signed-off-by: Dean Anderson linux-...@sensoray.com On 2/16/2012 10:44 PM, Dan Carpenter wrote: I don't have this hardware and I don't know the subsystem very well. So please review this patch carefully

Re: [patch 1/2] [media] s2255drv: cleanup vidioc_enum_fmt_cap()

2012-02-17 Thread dean anderson
Thanks Dan, I can sign off on this. The check isn't required. Signed-off-by: Dean Anderson linux-...@sensoray.com On 2/16/2012 10:43 PM, Dan Carpenter wrote: f wasn't checked consistently, so static checkers complain. This function is always called with a valid f pointer, so I have removed

[PATCH] s2255drv: video_device_alloc call not checked fix

2010-03-31 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1270045856 25200 # Node ID 2ab296deae938864b06b29cc224eb4b670ae3aa9 # Parent 18586e4ac3ed5972dac2015600f8c21e26c0fc16 s2255drv: video_device allocation fix From: Dean Anderson d...@sensoray.com call to video_device_alloc

[PATCH] s2255drv: removal of big kernel lock

2010-03-31 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1270046291 25200 # Node ID c72bdc8732abc0cf7bc376babfd06b2d999bdcf4 # Parent 2ab296deae938864b06b29cc224eb4b670ae3aa9 s2255drv: removal of BKL From: Dean Anderson d...@sensoray.com big kernel lock removed from open function

[PATCH] s2255drv: cleanup of debug messages

2010-03-31 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1270056198 25200 # Node ID db81e2f83909ee79dcca8496ceeda2653775e60a # Parent 0690e4e1d81e785af1a5f06a13573dcf2cc5cb0c s2255drv: cleanup of debug messages From: Dean Anderson d...@sensoray.com Priority: normal Signed-off

Re: [PATCH] s2255drv: v4l2 device added

2010-03-31 Thread dean
You are correct. Please withdraw this patch(https://patchwork.kernel.org/patch/89985/). Regards, Hans Verkuil wrote: On Wednesday 31 March 2010 18:34:39 Dean A. wrote: # HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1270053044 25200 # Node ID

Re: [PATCH] s2255drv: cleanup of driver disconnect code

2010-03-30 Thread dean
to solution #2. The call definitely needs checked, but I'd like some more feedback on this. Thanks and best regards, Dean David Ellingsworth wrote: This patch looks good, but there was one other thing that caught my eye. In s2255_probe_v4l, video_device_alloc is called for each video device

[PATCH] s2255drv: cleanup of driver disconnect code

2010-03-29 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1269899865 25200 # Node ID c437bd6f3659885afbe20ad12857347f0850156b # Parent a539e5b689454b8feb6b5acf5a67516b142c2823 s2255drv: cleanup of driver disconnect code From: Dean Anderson d...@sensoray.com simplifies use of kref

[PATCH] s2255drv: code cleanup

2010-03-29 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1269900678 25200 # Node ID 18586e4ac3ed5972dac2015600f8c21e26c0fc16 # Parent c437bd6f3659885afbe20ad12857347f0850156b s2255drv: code cleanup From: Dean Anderson d...@sensoray.com removal of unused pipe array (of size one

[PATCH] s2255drv: support for frame skipping

2010-03-08 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1268089334 28800 # Node ID 3ac8a7637cf3db984665e5aa43e03c22ee0c16f2 # Parent 08932f075cb147c47e2acc4b1f464c736080bc96 s2255drv: frame decimation support From: Dean Anderson d...@sensoray.com adds hardware frame skipping using

[PATCH] s2255drv: cleanup of V4L2 controls

2010-03-05 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1267809968 28800 # Node ID bdfee5ee52c800a24e1231cd52eeeb91636c013c # Parent ff343fa70b4def92d35f9ef7ee56a953b012d169 s2255drv cleanup of V4L2 video controls From: Dean Anderson d...@sensoray.com s2255drv cleanup of V4L2 video

[PATCH] s2255drv: support for 2257 device

2010-03-05 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1267829723 28800 # Node ID 54b44c52d908b363f69488fed84f1e8b9ccaf038 # Parent bdfee5ee52c800a24e1231cd52eeeb91636c013c s2255drv: support for 2257 device From: Dean Anderson d...@sensoray.com 2257 is 2255 with 2 svideo inputs

[PATCH] s2255drv: fixes for big endian arch

2010-03-04 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1267746208 28800 # Node ID ff343fa70b4def92d35f9ef7ee56a953b012d169 # Parent c9c1bcda21557cea425589f281b6b3100e2c15a0 s2255drv: fixes for big endian arch From: Dean Anderson d...@sensoray.com s2255drv fixes for big endian

[PATCH] s2255drv : adding video input status capability

2010-03-03 Thread Dean A.
# HG changeset patch # User Dean Anderson d...@sensoray.com # Date 1267655333 28800 # Node ID 3bf09a2ab1100269d6424c487ca98a1b1e4ead54 # Parent 5e0d6be7f020d6c0741795b0bd039ace7d54d310 Adds video status capability to 2255 driver From: Dean Anderson d...@sensoray.com Video status capability

OnAir USB HDTV Creator

2010-02-26 Thread Dean
I am trying to use an 'OnAir USB HDTV Creator' (from autumnwave.com). According to http://www.linuxtv.org/wiki/index.php/OnAir_USB_HDTV_Creator This device is supported, however it's not working for me. Following the instructions at above link, I tried this: modprobe pvrusb2 initusbreset=0

Re: eb1a:2860 eMPIA em28xx device to usb1 ??? usb hub problem?

2010-02-23 Thread Dean
Devin Heitmueller wrote: On Tue, Feb 23, 2010 at 1:37 AM, Dean r...@linuxstation.net wrote: Hi, I have the KWorld DVB-T 305U, an em28xx device. Only the video works for me under Linux, no audio. In case anyone wants to see it, I have attached the full dmesg text, solely from this device

Re: eb1a:2860 eMPIA em28xx device to usb1 ??? usb hub problem?

2010-02-22 Thread Dean
Hi, I have the KWorld DVB-T 305U, an em28xx device. Only the video works for me under Linux, no audio. In case anyone wants to see it, I have attached the full dmesg text, solely from this device. Cheers, Dean Devin Heitmueller wrote: Also, please provide the *full* dmesg output, so

KWorld DVB-T 305U now working

2010-02-14 Thread Dean
My dmesg gave the following (excerpt) when I connected my KWorld 305U. em28xx #0: Identified as KWorld DVB-T 305U (card=47) em28xx #0: em28xx #0: The support for this board weren't valid yet. em28xx #0: Please send a report of having this working em28xx #0: not to V4L mailing list (and/or to

Re: [PATCH] s2255drv: Don't conditionalize video buffer completion on waiting processes

2009-09-23 Thread dean
This seems ok. This portion of code was based on vivi.c, so that might be checked also. Mike Isely wrote: # HG changeset patch # User Mike Isely is...@pobox.com # Date 1253739604 18000 # Node ID 522a74147753ba59c7f45e368439928090a286f2 # Parent e349075171ddf939381fad432c23c1269abc4899

patch: s2255drv: adding V4L2_MODE_HIGHQUALITY

2009-05-15 Thread Dean A.
From: Dean Anderson d...@sensoray.com Adding V4L2_MODE_HIGHQUALITY feature. Signed-off-by: Dean Anderson d...@sensoray.com --- v4l-dvb-0018ed9bbca3/linux/drivers/media/video/s2255drv.c.orig 2009-05-15 10:15:36.0 -0700 +++ v4l-dvb-0018ed9bbca3/linux/drivers/media/video/s2255drv.c

patch: s2255drv: urb completion routine fixes

2009-04-30 Thread Dean A.
From: Dean Anderson d...@sensoray.com Error count in read pipe completion corrected. URB not resubmitted if shutting down. URB not freed in completion routine if new urb_submit_fails. (URB is freed on shutdown). Signed-off-by: Dean Anderson d...@sensoray.com --- v4l-dvb-83712d149893/linux

patch: s2255drv: code cleanup

2009-04-23 Thread Dean A.
From: Dean Anderson d...@sensoray.com Removal of unused structure items. Response values defined. Driver revision printk. Signed-off-by: Dean Anderson d...@sensoray.com --- v4l-dvb-ebb7b82f2b48/linux/drivers/media/video/s2255drv.c.orig 2009-04-23 11:37:28.0 -0700 +++ v4l-dvb

patch: s2255drv high quality mode and video status querying

2009-04-07 Thread Dean A.
From: Dean Anderson d...@sensoray.com This patch adds V4L2 video status capability and V4L2_MODE_HIGHQUALITY operation. Signed-off-by: Dean Anderson d...@sensoray.com --- v4l-dvb-1e670024659d/linux/drivers/media/video/s2255drv.c.orig 2009-04-07 10:38:42.0 -0700 +++ v4l-dvb