PM, Kees Cook wrote:
> > In preparation for unconditionally passing the struct timer_list pointer to
> > all timer callbacks, switch to using the new timer_setup() and from_timer()
> > to pass the timer pointer explicitly.
> >
> > Cc: Mike Isely
> > Cc: Mauro Ca
Acked-By: Mike Isely
On Tue, 24 Oct 2017, Kees Cook wrote:
> In preparation for unconditionally passing the struct timer_list pointer to
> all timer callbacks, switch to using the new timer_setup() and from_timer()
> to pass the timer pointer explicitly.
>
> Cc: Mike Ise
Acked-By: Mike Isely
On Mon, 9 Oct 2017, Mauro Carvalho Chehab wrote:
> Using enums makes easier to document, as it can use kernel-doc
> markups. It also allows cross-referencing, with increases the
> kAPI readability.
>
> Signed-off-by: Mauro Carvalho Chehab
> ---
>
118
> pvr2_context_thread_func+0x361/0x8c0
> drivers/media/usb/pvrusb2/pvrusb2-context.c:167
> kthread+0x3a1/0x470 kernel/kthread.c:231
> ret_from_fork+0x2a/0x40 arch/x86/entry/entry_64.S:431
> Code: 48 8b 85 30 ff ff ff 48 8d b8 98 00 00 00 e8 ee 82 89 fe 45 89
> e8 44 89 f1 4c
rol infrastructure.
> Add support for the new field which on structure
> v4l2_ext_controls
>
> Acked-by: Mike Isely
> Signed-off-by: Ricardo Ribalda Delgado
> ---
> drivers/media/usb/pvrusb2/pvrusb2-v4l2.c | 16 ++--
> 1 file changed, 14 insertions(+), 2 deletions(-)
>
down
to choosing between pvr2_ctrl_get_value() vs pvr2_ctrl_get_def().
It's not a correctness comment; what you have should work fine. So I'm
ack'ing this in any case:
Acked-By: Mike Isely
But you can do the above pretty easily & safely, and simplify it a bit
further.
GS,
> >>> @@ -269,13 +263,6 @@ static int pvr2_encoder_cmd(void *ctxt,
> >>> ret = -EBUSY;
> >>> break;
> >>> }
> >>> - if (retry_flag) {
> >>> - if (try_count < 20)
Acked-by: Mike Isely
-Mike
On Fri, 4 Oct 2013, Hans Verkuil wrote:
> From: Hans Verkuil
>
> drivers/media/usb/pvrusb2/pvrusb2-hdw.c:2871:13: warning: symbol
> 'pvr2_hdw_get_detected_std' was not declared. Should it be static?
>
> Signed-off-by: Ha
Acked-By: Mike Isely
-Mike
On Wed, 12 Jun 2013, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Signed-off-by: Hans Verkuil
> ---
> drivers/media/usb/pvrusb2/pvrusb2-hdw.c |4
> drivers/media/usb/pvrusb2/pvrusb2-hdw.h |4
> drivers/media/usb/p
Acked-By: Mike Isely
On Sat, 27 Oct 2012, Mauro Carvalho Chehab wrote:
> drivers/media/usb/pvrusb2/pvrusb2-v4l2.c:199:5: warning: no previous
> prototype for 'pvr2_s_std' [-Wmissing-prototypes]
> drivers/media/usb/pvrusb2/pvrusb2-v4l2.c:368:5: warning: no previo
Completely agree! Thanks for spotting that one.
Signed-off-by: Mike Isely
-Mike
On Thu, 25 Oct 2012, Alan Cox wrote:
> From: Alan Cox
>
> This should have break statements in it.
>
> Signed-off-by: Alan Cox
> ---
>
> drivers/media/usb/pvrusb2/pvrusb2-hdw.c
he business of processing the MPEG data in order to adhere
to this proposal, then that will be a very big deal for this driver.
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linu
Acked-By: Mike Isely
-Mike
On Thu, 26 Jul 2012, Tim Gardner wrote:
> Cc: Mike Isely
> Cc: Mauro Carvalho Chehab
> Cc: linux-media@vger.kernel.org
> Signed-off-by: Tim Gardner
> ---
> drivers/media/video/pvrusb2/pvrusb2-devattr.c | 17 -
> 1 file c
Mauro:
Please pull - this includes a long-awaited change courtesy of Hans
Verkuil which finally transitions the driver to video_ioctl2.
-Mike Isely
The following changes since commit a1ac5dc28d2b4ca78e183229f7c595ffd725241c:
[media] gspca - sn9c20x: Change the exposure setting of
On Wed, 5 Oct 2011, Mauro Carvalho Chehab wrote:
> Em 05-10-2011 11:00, Mike Isely escreveu:
> >
> > Mauro:
> >
> > With the line you've just added, then the " = arg" assignment in the
> > immediate prior line is effectivel
static long pvr2_v4l2_do_ioctl(struct file *file,
> unsigned int cmd, void *arg)
> case VIDIOC_QUERYSTD:
> {
> v4l2_std_id *std = arg;
> + *std = V4L2_STD_ALL;
> ret = pvr2_hdw_get_detected_std(hdw, std);
> break;
&
Acked-By: Mike Isely
-Mike
On Mon, 3 Oct 2011, Mauro Carvalho Chehab wrote:
> Signed-off-by: Mauro Carvalho Chehab
> ---
> drivers/media/video/pvrusb2/pvrusb2-hdw.c |7 +++
> drivers/media/video/pvrusb2/pvrusb2-hdw.h |3 +++
> drivers/media/video/pvrusb2
For the pvrusb2 portion of this patch:
Acked-By: Mike Isely
-Mike
On Wed, 6 Jul 2011, Mauro Carvalho Chehab wrote:
> Those drivers are not relying at the V4L2 core to handle the ioctl's.
> So, we need to manually patch them every time a change goes to the
> core.
>
>
I understand that this patch would not have been need had the pvrusb2
driver been using videodev_ioctl2. This is a situation that I'm going
to (finally) remedy ASAP. In the mean time...
Acked-By: Mike Isely
-Mike
On Tue, 14 Jun 2011, Hans Verkuil wrote:
> From: Hans Verkuil
I'll look at the surrounding code and see what makes sense there. Having
an error leg for allocation failures is a useful thing.
-Mike
Dan Carpenter wrote:
On Fri, Mar 25, 2011 at 11:33:36PM -0500, Mike Isely wrote:
Acked-By: Mike Isely
I'd need to reformat this one to
That's an opinion which I as the driver author disagree with. Strongly.
How hard is it to read "not false"?
Nacked-By: Mike Isely
On Sat, 26 Mar 2011, Dan Carpenter wrote:
> Using !0 is less readable than just saying 1.
>
> Signed-off-by: Dan Carpenter
>
&
Are you actually serious about this? Well it's a small change...
Acked-By: Mike Isely
On Sat, 26 Mar 2011, Dan Carpenter wrote:
> The generic_standards_cnt define is only used in one place and it's
> more readable to just call ARRAY_SIZE(generic_standards) directly.
>
&
It not worth this scale of source code disruption to the source code
just to rename a bunch of variables. I'm sorry, but...
Nacked-By: Mike Isely
On Sat, 26 Mar 2011, Dan Carpenter wrote:
> This patch renames some variables to bring them more in line with
> kernel CodingStyle.
Acked-By: Mike Isely
On Sat, 26 Mar 2011, Dan Carpenter wrote:
> This function returns NULL on failure so lets do that if kzalloc()
> fails. There is a separate problem that the caller for this function
> doesn't check for errors...
>
> Signed-off-by: Dan Carpenter
>
I am OK with the #include change, but NOT the if-statement change. But
since it's bundled into one patch...
Nacked-By: Mike Isely
On Sat, 26 Mar 2011, Dan Carpenter wrote:
> * Include instead of .
> * Remove unneeded curly braces.
>
> Signed-off-by: Dan Carpenter
I vehemently object to this scale of disruption to the pvrusb2 driver
source code purely to move around a bunch of braces and whitespace.
ESPECIALLY the massive ridiculous changes having to do with if-statement
syntax!
Nacked-By: Mike Isely
On Sat, 26 Mar 2011, Dan Carpenter wrote
at:
git://git.linuxtv.org/mcisely/pvrusb2-dev.git pvrusb2-merge-2
Mike Isely (2):
pvrusb2: Implement support for Terratec Grabster AV400
pvrusb2: Remove dead code
Xiaochen Wang (1):
pvrusb2: check kmalloc return value
drivers/media/video/pvrusb2/pvrusb2-cx2584x-v4
On Fri, 11 Mar 2011, Mike Isely wrote:
> On Fri, 11 Mar 2011, Mauro Carvalho Chehab wrote:
>
> > /home/mchehab/new_build/v4l/pvrusb2-v4l2.c: In function
> > 'pvr2_v4l2_do_ioctl':
> > /home/mchehab/new_build/v4l/pvrusb2-v4l2.c:798:23: warning: variable 'c
ook into these. I'm a little puzzled right now since silly
stuff like this usually doesn't get by me. Unfortunately I can't look
at it right this minute. Expect to hear from me on Sunday.
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2
-Mike Isely]
The following changes since commit 5ed4bbdae09d207d141759e013a0f3c24ae76ecc:
Mauro Carvalho Chehab (1):
[media] tuner-core: Don't touch at standby during tuner_lookup
are available in the git repository at:
git://git.linuxtv.org/mcisely/pvrusb2-dev.git pvrusb2-mer
On Fri, 21 Jan 2011, Jarod Wilson wrote:
> On Fri, Jan 21, 2011 at 10:31:42AM -0600, Mike Isely wrote:
> >
> > The pvrusb2 change is obviously trivial so I have no issue with it.
> >
> > Acked-By: Mike Isely
> >
> > Note the spelling of my last n
On Fri, 21 Jan 2011, Mike Isely wrote:
>
> Note the spelling of my last name "Isely" not "Isley". A good way to
> remember is to think of the normal word "wisely" and just drop the
> leading "w". (And yes, is...@isely.net and is...@pobox.c
The pvrusb2 change is obviously trivial so I have no issue with it.
Acked-By: Mike Isely
Note the spelling of my last name "Isely" not "Isley". A good way to
remember is to think of the normal word "wisely" and just drop the
leading "w". (And yes,
On Wed, 19 Jan 2011, Jarod Wilson wrote:
> On Jan 19, 2011, at 8:20 AM, Mike Isely wrote:
>
> > This probing behavior does not happen for HVR-1950 (or HVR-1900) since
> > there's only one possible IR configuration there.
>
> Just to be 100% clear, the device I
ter".
Another screwy as aspect of english, and I say this as a native english
speaker. The term "xcvr" is usually just considered to be shorthand for
"transceiver".
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F
ke the
original 29xxx device.)
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
used in two places: (1) If a bus scan is
performed during initialization (normally it isn't), and (2) it is
called once ONLY for a 24xxx device (targeting 0x71) in order to
determine if it is dealing with the MCE variant.
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 7
VR-1950 (or HVR-1900) since
there's only one possible IR configuration there.
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sun, 16 Jan 2011, Andy Walls wrote:
> On Sun, 2011-01-16 at 20:27 -0600, Mike Isely wrote:
[,,,]
>
> Right now, yes. In the near future, I need to use to to pass 3
> non-const items though:
>
> 1. A "struct mutex *transceiver_lock" so that the bridge
only in a single spot.
Just thinking ahead about making the setup table-driven and not
requiring data segment storage.
-Mike
Acked-By: Mike Isely
On Sun, 16 Jan 2011, Andy Walls wrote:
>
> When registering an IR Rx device with the I2C subsystem, provide more detailed
> infor
m_testdev (Pawel Osciak or Marek Szyprowski)
> > cx23885 (Steve Toth)
> > cx18-alsa (Andy Walls)
> > omap24xxcam (Sakari Ailus or David Cohen)
> > au0828 (Janne Grunau)
> > cpia2 (Andy Walls or Hans Verkuil)
> > cx231xx (Mauro Carvalho Chehab)
> > davinci (
For just the pvrusb2 part of the patch series below
Acked-By: Mike Isely
On Fri, 24 Sep 2010, Laurent Pinchart wrote:
> The argument isn't used anymore by the functions, remote it.
>
> Signed-off-by: Laurent Pinchart
> ---
> drivers/media/radio/radio-si4713
Acked-By: Mike Isely
On Fri, 24 Sep 2010, Laurent Pinchart wrote:
> With the v4l2_i2c_new_subdev* functions now supporting loading modules
> based on modaliases, replace the hardcoded module name passed to those
> functions by NULL.
>
> All corresponding I2C modules have been c
Based on the surrounding code (the unconditional dereference), I agree
that this particular bit of coding paranoia is not doing much good.
Acked-by: Mike Isely
On Thu, 19 Aug 2010, Dan Carpenter wrote:
> We dereference "maskptr" unconditionally at the start of the function
>
Andy:
Acked-By: Mike Isely
-Mike
On Tue, 27 Jul 2010, Andy Shevchenko wrote:
> Signed-off-by: Andy Shevchenko
> Cc: Mike Isely
> ---
> drivers/media/video/pvrusb2/pvrusb2-debugifc.c | 14 ++
> 1 files changed, 2 insertions(+), 12 deletions(-)
>
>
l).
>
> P.S.: This email is c/c to the developers where some action is expected.
> If you were copied, please review the patches, acking/nacking or
> submitting an update.
>
[...]
> == Waiting for Mike Isely review ==
>
> Apr,25 2010: Proble
On Wed, 7 Jul 2010, Sven Barth wrote:
> Hi!
>
> Am 06.07.2010 15:06, schrieb Mauro Carvalho Chehab:
> > == Waiting for Mike Isely review ==
> >
> > Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400
> http://patchwork.kernel.org/patch
On Sat, 3 Jul 2010, Douglas Schilling Landgraf wrote:
> Hello Mike,
>
> Mike Isely wrote:
> > Mauro:
> >
> > FYI, I posted an "Acked-By: Mike Isely " weeks ago, back on
> > 27-May, immediately after the patch was posted. It's a great catch, an
Mauro:
FYI, I posted an "Acked-By: Mike Isely " weeks ago,
back on 27-May, immediately after the patch was posted. It's a great
catch, and the bug has been there since basically the beginning of the
driver. Was I ever supposed to see any kind of reaction to that ack
(e.g. ha
I looked through my revision history and that bug has been there in the
driver source since at least May 2005, long before it was ever merged
into the kernel. Wow, what a great catch. Thanks!
Acked-By: Mike Isely
-Mike
On Wed, 26 May 2010, Julia Lawall wrote:
> From: Julia Law
gt; select the composite input: no picture. It also says:
>
> v4l-conf had some trouble, trying to continue anyway
> Warning: Cannot convert string "-*-ledfixed-medium-r-*--39-*-*-*-c-*-*-*" to
> type FontStruct
> v4l2: read: rc=262144/size=442368
>
> So, what els
On Fri, 21 May 2010, Mauro Carvalho Chehab wrote:
> Mike Isely wrote:
[snip]
> > The point when the kernel started complaining about the use of a stack
> > based USB I/O buffers is the relevant point, which was not back in
> > 2.6.12. I learned of this behavio
On Fri, 21 May 2010, Mauro Carvalho Chehab wrote:
> Mike Isely wrote:
> > On Fri, 21 May 2010, Mauro Carvalho Chehab wrote:
> >
> >> Mike Isely wrote:
> >>> Mauro:
> >>>
> >>> You are reading too much into that comment.
> >>
On Fri, 21 May 2010, Mauro Carvalho Chehab wrote:
> Mike Isely wrote:
> > Mauro:
> >
> > You are reading too much into that comment.
> >
> > I never said it was valid to do what had been done, only that for the
> > longest time this is what the d
doing in the past, that it's definitely causing
a problem now and thus that is why this patch exists.
I'd really rather you not mess with my comment. Probably too late
however.
-Mike
On Fri, 21 May 2010, Mauro Carvalho Chehab wrote:
> Mike Isely wrote:
> > Please from ht
insertions(+), 9 deletions(-)
These are primarily a collection of stability fixes.
Thanks,
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a
On Sat, 24 Apr 2010, Sven Barth wrote:
> Hi!
>
> On 24.04.2010 22:24, Mike Isely wrote:
> > On Sat, 24 Apr 2010, Sven Barth wrote:
> > >
> > > Hi!
> > >
> > > Although you never really completed that support for the AV400 it runs
> > &
On Sat, 24 Apr 2010, Sven Barth wrote:
> On 24.04.2010 19:13, Mike Isely wrote:
> >
> > Actually the support in the pvrusb2 driver was never really completed.
> > But since I don't have a sample of the hardware here I went on ahead and
> > merged what was there s
r often with ArchLinux ^^).
>
> Thus I ask how this situation might be solved so that I can use the AV400
> without patching around in the source of v4l.
>
> Attached:
> * dmesg output with unpatched cx25840 module
> * my "quick & dirty" patch for cx25840-core.c
>
Acked-By: Mike Isely
-Mike
On Mon, 5 Apr 2010, Joe Perches wrote:
> There is a macro called dev_info that prints struct device specific
> information. Having variables with the same name can be confusing and
> prevents conversion of the macro to a function.
>
> Rena
Acked-By: Mike Isely
(in the context of the pvrusb2 driver related changes)
-Mike
On Mon, 22 Mar 2010, Wolfram Sang wrote:
> Made necessary by 6992f5334995af474c2b58d010d08bc597f0f2fe.
>
> Found by this semantic patch:
>
> @ init @
> type T;
> identifier A
On Wed, 7 Apr 2010, Mike Isely wrote:
>
> Backwards compatibility is very important and thus any kind of new
> interface deserves a lot of forethought to ensure that choices are made
> in the present that people will regret in the future. Making an
"in the present tha
y all V4L drivers is
obviously a concept I would not have any problem with.
However that does not necessarily mean that I would agree with eventual
removal of the pvrusb2 driver's existing sysfs interface. That would
depend on whether or not doing such a thing loses functionality and wh
On Thu, 8 Apr 2010, hermann pitton wrote:
> Hi,
>
> Am Mittwoch, den 07.04.2010, 20:50 +0200 schrieb Lars Hanisch:
> > Am 06.04.2010 16:33, schrieb Mike Isely:
> [snip]
> > >>
> > >> Mike, do you know of anyone actively using that additional informa
s simply continues to do its job, automatically
reflecting internal changes without any extra work in that module's
code. I don't know if that same strategy could be done in the V4L core.
If it could, then this would probably alleviate a lot of concerns about
testing / mainten
/ group of those nodes. It is
possible to change the owner / group from userspace, and I *think* it's
possible to create a udev rule to do this, but honestly I have not
investigated this possibility so I don't fully know.
This is one serious potential drawback to using sysfs as a driver API
sysfs API, the response is in fact very positive.
Most users of the driver had no concept that such a thing was even
possible until they were exposed to it. Now that's not to say that we
should all be screaming for this - but if people didn't really
understand what was possible,
> sure whether it is worth it though.
Just remember that the more complex / subtle you make the node contents,
then the more parsing will be required for any program that tries to use
it. I also think it's probably a bad idea for example to define a
format where the whitespace convey
should include some introspection capabilities, which will make
it easier (or even possible) to evolve the interface over time while
minimizing backwards compatibility issues.
If such a generic interface were made available, I could see an argument
to remove the sysfs interface from the pvrusb2 d
the feature.)
So you need to not use "initusbreset=0". The advice in the wiki is over
a year out of date.
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media&q
saa7115 video digitizer.
Thanks,
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo inf
On Mon, 18 Jan 2010, Andy Walls wrote:
> On Mon, 2010-01-18 at 14:18 -0600, Mike Isely wrote:
> > On Mon, 18 Jan 2010, Andy Walls wrote:>
> > > Any definitive confirmation anyone can give on any of these drivers
> > > would be helpful and would save me some time.
&g
require locks would be from multiple userspace clients calling into the
driver, but even in those cases, the requested actions are handled
through a work queue so all calls out of the driver into other things
(like cx25840) should all source from a single common thread context per
hardware instance
On Tue, 1 Dec 2009, Mauro Carvalho Chehab wrote:
> Mike Isely wrote:
> > On Mon, 30 Nov 2009, Mauro Carvalho Chehab wrote:
> >
> >> Em Sat, 28 Nov 2009 14:33:30 -0600 (CST)
> >> Mike Isely escreveu:
> >>
> >>> Mauro:
> >>>
> &
On Mon, 30 Nov 2009, Mauro Carvalho Chehab wrote:
> Em Sat, 28 Nov 2009 14:33:30 -0600 (CST)
> Mike Isely escreveu:
>
> >
> > Mauro:
> >
> > I had also posted up two high priority pvrusb2 patches that should
> > really be cherry-picked for 2.6.32. Y
ese within the v4l-dvb Mercurial repository here:
Changeset 13495:87c3853fe2b3
Subject: pvrusb2: Support 16KB FX2 firmware
http://linuxtv.org/hg/v4l-dvb/rev/87c3853fe2b3
Changeset 13500:d4c418d4b25c
Subject: pvrusb2: Fix lingering 16KB FX2 Firmware issues
http://linuxtv.org/hg/v4l-dvb/rev/d4c418d
marked high priority.
Thanks,
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo inf
On Thu, 29 Oct 2009, Mauro Carvalho Chehab wrote:
> Em Sun, 11 Oct 2009 22:53:14 -0500 (CDT)
> Mike Isely escreveu:
>
> >
> > Mauro:
> >
> > Please from http://linuxtv.org/hg/~mcisely/pvrusb2-20091011 for a few
> > various pvrusb2 fixes / improvem
-debugifc.c |3 +++
pvrusb2-encoder.c |5 -
pvrusb2-hdw-internal.h |1 +
pvrusb2-hdw.c | 33 -
pvrusb2-v4l2.c | 21 -
5 files changed, 52 insertions(+), 11 deletions(-)
-Mike
--
Mike Isely
isely
ou either.
But you do need to see the perspective of what you're trying to do
otherwise you may just get frustrated.
Things aren't hopeless. The cxusb module in DVB might be something you
should look at. I guess it depends on how deep you wish to dive here.
-Mike
--
Mike Isel
at none of the about is the answer you're looking for. But
perhaps it will lead you in the right direction. It is entirely
possible that there is another bridge driver out there which can handle
this part, but I can't help you there.
-Mike
--
Mike Isely
isely @ isely (dot)
ch have inherited this bug as well?)
-Mike
>
>
>
> Mike Isely wrote:
> > # HG changeset patch
> > # User Mike Isely
> > # Date 1253739604 18000
> > # Node ID 522a74147753ba59c7f45e368439928090a286f2
> > # Parent e349075171ddf939381fad432c23c1269a
# HG changeset patch
# User Mike Isely
# Date 1253739604 18000
# Node ID 522a74147753ba59c7f45e368439928090a286f2
# Parent e349075171ddf939381fad432c23c1269abc4899
s2255drv: Don't conditionalize video buffer completion on waiting processes
From: Mike Isely
The s2255 driver had logic
changeset patch
# User Mike Isely
# Date 1253547742 18000
# Node ID e349075171ddf939381fad432c23c1269abc4899
# Parent 760a8bc4028014493ccbbe85d0c9e8c91873fc23
bttv: Fix reversed polarity error when switching video standard
From: Mike Isely
The bttv driver function which handles switching of the
# HG changeset patch
# User Mike Isely
# Date 1253545748 18000
# Node ID 760a8bc4028014493ccbbe85d0c9e8c91873fc23
# Parent 29e4ba1a09bcf9a03a653b2124929f5359fef772
bttv: Fix potential out-of-order field processing
From: Mike Isely
There is a subtle interaction in the bttv driver which can result
On Mon, 7 Sep 2009, Mauro Carvalho Chehab wrote:
[...]
>
> I remember I asked both Mikes (Michael Krufky and Mike Isely) on March for
> some
> tests with the new firmware. I'm not sure if they had some time for testing
> it.
Yes, I remember this exchange. At the ti
Acked-By: Mike Isely
-Mike
On Fri, 7 Aug 2009, Hans Verkuil wrote:
> Hi Mauro,
>
> Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-misc for the
> following:
>
> - pvrusb2: fix compile warning
> - cx24113: fix mips compiler warning
> - hdpvr: add m
EQUIRE USB 2.0 high-speed. The default
encoder settings configured by the pvrusb2 driver actually work quite
well over USB 1.1, since the resulting video stream requires
significantly less bandwidth than the 12Mbps that USB 1.1 can
theoretically supply. I've actually successfully tes
"ok" but I think even he wasn't sure if it
was doing everything right.
That was a while ago and the cx25840 driver has undergone enough other
changes to make merging those patches a lot more difficult - plus the
v4l-subdevice stuff needs additional changes to support the extra
cropping
--
pvrusb2-video-v4l.c | 37 +-
5 files changed, 98 insertions(+), 78 deletions(-)
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe
+-
5 files changed, 98 insertions(+), 78 deletions(-)
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message
Well now I feel like an idiot. Thanks for pointing that out in my own
code :-)
Still digging through this.
-Mike
On Fri, 12 Jun 2009, Andy Walls wrote:
> On Fri, 2009-06-12 at 15:33 -0500, Mike Isely wrote:
> > I am unable to reproduce the s5h1411 error here.
> >
> >
ns it will be very difficult for me to track
the problem down here since I don't have that device variant.
-Mike
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media&qu
On Thu, 11 Jun 2009, Steven Toth wrote:
> Mike Isely wrote:
> > On Sun, 7 Jun 2009, Roger wrote:
> >
> > > >From looking at "linux/drivers/media/dvb/frontends/s5h1411.c", The
> > > s5h1411_readreg wants to see "2" but
dn't like and the chip
responded by going deaf on the I2C bus. More than a few I2C-driven
parts can behave this way. It's also possible that the part might have
been busy and unable to respond - but usually in that case the driver
for such a part will be written with this in min
I see no issues here with the pvrusb2 part of it...
Acked-by: Mike Isely
On Wed, 20 May 2009, Steven Toth wrote:
> Mauro,
>
> Please pull from http://kernellabs.com/hg/~stoth/tda10048-merge/
>
>- TDA10048: Ensure the I/F changes during DVB-T 6/7/8 bandwidth changes.
> v4l-dvb tree (compiled yesterday for completeness).
>
> Oldrich, thanks a lot for testing and reporting, this is very
> appreciated.
>
Jean:
I tried the all-in-one patch here on a PVR-USB2 24xxx model (slightly
older v4l-dvb repo and 2.6.27.13 vanilla kernel) and it worked
;t use it, or for debugging purpose.
>
> Signed-off-by: Jean Delvare
> Cc: Mike Isely
Acked-by: Mike Isely
> ---
> linux/drivers/media/video/pvrusb2/pvrusb2-i2c-core.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- v4l-dvb.orig/linux/drivers/media/video/pv
On Mon, 11 May 2009, Mauro Carvalho Chehab wrote:
> Em Mon, 11 May 2009 22:09:26 -0300
> Mauro Carvalho Chehab escreveu:
>
> > Em Sat, 9 May 2009 16:49:31 -0500 (CDT)
> > Mike Isely escreveu:
> >
> > >
> > > Mauro:
> > >
> > >
| 78 -
pvrusb2-i2c-core.c | 53 +++--
pvrusb2-sysfs.c| 22 ++---
pvrusb2-v4l2.c |2 -
7 files changed, 106 insertions(+), 76 deletions(-)
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 7
1 - 100 of 154 matches
Mail list logo
