ion-ALL-IN-ONE.patch
But for review the individual patches are much better.
Thanks,
--
Jean Delvare
--
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
o finished. I'm waiting for a last test by Jean
> Delvare before I post that one as well.
Testing complete, it seems to work just fine. Thanks!
> - Added support for saa6588 to saa7134: needed to drop the legacy i2c API
> from saa6588. This is in my pull request for my v4l-dvb t
Hi Trent,
On Mon, 16 Mar 2009 15:47:17 -0700 (PDT), Trent Piepho wrote:
> On Mon, 16 Mar 2009, Jean Delvare wrote:
> > You are unfair. The pull request came with a short log of all the
> > changes.
>
> "short" log. His entire series was decribed with fewer words
On Mon, 16 Mar 2009 12:43:33 -0700 (PDT), Trent Piepho wrote:
> On Mon, 16 Mar 2009, Jean Delvare wrote:
> > Come on, just look at ir-kbd-i2c and tvaudio again, see how great are
> > these drivers which have been "designed" on top of the legacy i2c
> > binding model.
Hi Mauro,
On Mon, 16 Mar 2009 09:52:37 -0300, Mauro Carvalho Chehab wrote:
> On Mon, 16 Mar 2009 12:18:01 +0100
> Jean Delvare wrote:
> > I really don't see any problem there. There are many drivers (i2c or
> > not) in the kernel which do exactly this and this works just f
Hi Andy,
On Sun, 15 Mar 2009 20:52:33 -0400, Andy Walls wrote:
> On Sun, 2009-03-15 at 23:26 +0100, Jean Delvare wrote:
> > No. Once again: Linux doesn't support binding more than one driver to a
> > device. This has _nothing_ to do with any choice done in i2c-core.
>
Hi Mauro,
On Mon, 16 Mar 2009 06:34:02 -0300, Mauro Carvalho Chehab wrote:
> On Sun, 15 Mar 2009 18:53:13 +0100
> Jean Delvare wrote:
> > On Sun, 15 Mar 2009 10:42:41 -0700 (PDT), Trent Piepho wrote:
> > > You can also split the "device" into multiple devices. Mo
So, all in all, great care should always be taken when more than one
driver accesses the same I2C device. And even if only one driver
accesses the I2C device, all accesses that can happen in parallel (for
example triggered by sysfs attribute reads or writes) may have to be
serialized by the driv
y we
> remain bug-compatible with the current situation. Or even slightly better
> since we can now ensure that audio is working at least.
Agreed.
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...
Hi Andy,
On Sun, 15 Mar 2009 15:34:33 -0400, Andy Walls wrote:
> On Sun, 2009-03-15 at 18:12 +0100, Jean Delvare wrote:
> > Hi Hans,
> >
> > On Sun, 15 Mar 2009 13:44:01 +0100, Hans Verkuil wrote:
> > > Hi Mauro, Jean,
> > >
> > > When conve
On Sun, 15 Mar 2009 10:42:41 -0700 (PDT), Trent Piepho wrote:
> On Sun, 15 Mar 2009, Jean Delvare wrote:
> > On Sun, 15 Mar 2009 13:44:01 +0100, Hans Verkuil wrote:
> > This is the typical multifunction device problem. It isn't specifically
> > related to I2C, the exac
exported by the
ir-kbd-i2c and tvaudio modules.
But this is all said without having much knowledge of the bttv, tvaudio
and ir-kbd-i2c drivers, so I might as well be completely off track.
--
Jean Delvare
--
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
I see the following warning when building the zoran driver:
v4l/zoran_card.c: In function 'zoran_probe':
v4l/zoran_card.c:1243: warning: assignment from incompatible pointer type
Fixing the notify callback prototype solves it.
Signed-off-by: Jean Delvare
Cc: Hans Verkuil
Cc: Tr
Hi Hans,
On Sat, 14 Mar 2009 22:39:35 +0100, Hans Verkuil wrote:
> On Saturday 14 March 2009 22:25:14 Jean Delvare wrote:
> > The video4linux-l...@redhat.com list is deprecated, point the users to
> > the new linux-media list instead.
> >
> > Signed-off-by: Jean
The video4linux-l...@redhat.com list is deprecated, point the users to
the new linux-media list instead.
Signed-off-by: Jean Delvare
---
linux/Documentation/video4linux/bttv/README |4 ++--
linux/drivers/media/radio/radio-si470x.c |4 ++--
linux/drivers/media/video/bt8xx/bttv
Mailing list video4linux-l...@redhat.com is deprecated, so drop
references to it in MAINTAINERS.
Signed-off-by: Jean Delvare
Cc: Mauro Carvalho Chehab
---
MAINTAINERS |2 --
1 file changed, 2 deletions(-)
--- linux-2.6.29-rc8.orig/MAINTAINERS 2009-03-13 08:39:30.0 +0100
On Wed, 11 Mar 2009 04:55:19 +0100, Patch from Jean Delvare wrote:
> The patch number 10935 was added via Mauro Carvalho Chehab
>
> to http://linuxtv.org/hg/v4l-dvb master development tree.
>
> Kernel patches in this development tree may be modified to be backward
> com
The removal of the timer which polls the infrared input is racy.
Replacing the timer with a delayed work solves the problem.
Signed-off-by: Jean Delvare
---
linux/drivers/media/video/saa6588.c | 26 +++---
1 file changed, 7 insertions(+), 19 deletions(-)
--- v4l-dvb.orig
The removal of the timer which polls the infrared input is racy.
Replacing the timer with a delayed work solves the problem.
Signed-off-by: Jean Delvare
---
linux/drivers/media/video/ir-kbd-i2c.c | 22 ++
linux/include/media/ir-kbd-i2c.h |3 +--
2 files changed
The removal of the timer which polls the infrared input is racy.
Replacing the timer with a delayed work solves the problem.
Signed-off-by: Jean Delvare
Cc: Devin Heitmueller
---
linux/drivers/media/video/em28xx/em28xx-input.c | 24 ++-
1 file changed, 7 insertions(+), 17
When unloading the cx8800 driver I sometimes get a general protection
fault. Analysis revealed a race in cx88_ir_stop(). It can be solved by
using a delayed work instead of a timer for infrared input polling.
Signed-off-by: Jean Delvare
---
Thanks to Trent's compatibility patches, we c
kmalloc() can fail for large video buffers. By default the kernel
complains loudly about allocation failures, but we don't want to
frighten the user, so ask kmalloc() to keep quiet on such failures.
Signed-off-by: Jean Delvare
Cc: Trent Piepho
Cc: Hans Verkuil
---
linux/drivers/media/
The lock_norm module parameter doesn't look terribly useful. If you
don't want to change the norm, just don't change it. As a matter of
fact, no other v4l driver has such a parameter.
Signed-off-by: Jean Delvare
Cc: Trent Piepho
Cc: Hans Verkuil
---
linux/Documentation/vi
Hi Devin,
On Thu, 5 Mar 2009 13:03:00 -0500, Devin Heitmueller wrote:
> On Thu, Mar 5, 2009 at 4:39 AM, Jean Delvare wrote:
> I would suggest that this patch be broken into three separate patches,
> and then they can go in as the individual maintainers have the
> opportunity to t
On Thu, 5 Mar 2009 01:43:55 -0800 (PST), Trent Piepho wrote:
> On Thu, 5 Mar 2009, Jean Delvare wrote:
> > +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,20)
> > struct work_struct work;
> > struct timer_list timer;
> > +#else
> > + struct delayed_work w
From: Jean Delvare
Subject: Fix race in infrared polling on rmmod
The race on rmmod I just fixed in cx88-input affects 3 other drivers.
Fix these the same way.
Signed-off-by: Jean Delvare
---
Note that I do not have any of these devices, so this patch is
untested. Testers welcome!
linux
From: Jean Delvare
Subject: cx88: Prevent general protection fault on rmmod
When unloading the cx8800 driver I sometimes get a general protection
fault. Analysis revealed a race in cx88_ir_stop(). It can be solved by
using a delayed work instead of a timer for infrared input polling.
This fixes
On Tue, 3 Mar 2009 12:14:06 -0800 (PST), Trent Piepho wrote:
> On Tue, 3 Mar 2009, Jean Delvare wrote:
> > On Tue, 3 Mar 2009 01:40:00 -0800 (PST), Trent Piepho wrote:
> > > On Mon, 2 Mar 2009, Jean Delvare wrote:
> > > In 2.6.20 delayed_work was split from work_stru
I2C_M_IGNORE_NAK is defined since kernel 2.5.54 so we don't need to
declare it in the compatibility header.
Signed-off-by: Jean Delvare
---
v4l/compat.h |4
1 file changed, 4 deletions(-)
--- v4l-dvb.orig/v4l/compat.h 2009-03-01 16:09:10.0 +0100
+++ v4l-dvb/v4l/com
On Tue, 3 Mar 2009 01:40:00 -0800 (PST), Trent Piepho wrote:
> On Mon, 2 Mar 2009, Jean Delvare wrote:
> > > Makes the most sense to me. I was just about to make a patch to do the
> > > same thing when I got your email. Though I was going to patch the v4l-dvb
> > >
Hi Trent,
On Mon, 2 Mar 2009 13:12:24 -0800 (PST), Trent Piepho wrote:
> On Mon, 2 Mar 2009, Jean Delvare wrote:
> > On Mon, 2 Mar 2009 17:03:49 +0100, Jean Delvare wrote:
> > > As far as I can see the key difference between bttv-input and
> > > cx88-input is that bt
On Mon, 2 Mar 2009 17:03:49 +0100, Jean Delvare wrote:
> As far as I can see the key difference between bttv-input and
> cx88-input is that bttv-input only uses a simple self-rearming timer,
> while cx88-input uses a timer and a separate workqueue. The timer runs
> the workqueue, whic
Hi Andy,
On Mon, 02 Mar 2009 09:16:05 -0500, Andy Walls wrote:
> On Mon, 2009-03-02 at 13:39 +0100, Jean Delvare wrote:
> > On Sun, 15 Feb 2009 21:41:08 +0100, Jean Delvare wrote:
> > > Hi all,
> > >
> > > Today I have hit the following general protection fau
On Sun, 15 Feb 2009 21:41:08 +0100, Jean Delvare wrote:
> Hi all,
>
> Today I have hit the following general protection fault when removing
> module cx8800:
This has just happened to me again today, with kernel 2.6.28.7. I have
opened a bug in bugzilla:
http://bugzilla.kernel.org/
But my
understanding was that the ZR36067's requirements were making this very
difficult if not impossible.
--
Jean Delvare
--
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
192/0x343
This can be "fixed" with the following patch:
Subject: zoran: Don't frighten users with failed buffer allocation
kmalloc() can fail for large video buffers. By default the kernel
complains loudly about allocation failures, but we don't want to
frighten the user, so a
re
possible.
Signed-off-by: Jean Delvare
---
Note: totally untested!
linux/drivers/media/video/Kconfig |1
linux/drivers/media/video/vino.c | 169 ++---
2 files changed, 156 insertions(+), 14 deletions(-)
--- v4l-dvb-vino2.orig/linux/drivers/media/video/Kconfig
to do
it. If we can find a tester for your V4L2 conversion then maybe the
same tester will be able to test my own changes.
But i2c-algo-sgi isn't a big problem per se, it doesn't block further
evolutions or anything like that, so if we can't find a tester and it
has to st
Hi John,
(re-adding linux-media on Cc as I doubt you dropped it on purpose...)
On Tue, 24 Feb 2009 11:46:04 +, John Pilkington wrote:
> Jean Delvare wrote:
>
> > * Enterprise-class distributions (RHEL, SLED) are not the right target
> > for the v4l-dvb repository, so w
;s just drop support for kernels < 2.6.22 and focus on better
supporting upstream and recent kernels.
Thanks,
--
Jean Delvare
--
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
y.
>
> What is not working is playback of video. However, it turns out that that
> has been broken for some time now. The hope is that now that the driver has
> been cleaned up someone can actually go in and try to discover what broke.
> Note that video pass-through is working fin
007 ICs found on some cards)
The KS0127 is a video decoder, and it's made by Samsung.
--
Jean Delvare
--
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
Hi Trent,
On Sat, 21 Feb 2009 05:58:10 -0800 (PST), Trent Piepho wrote:
> On Sat, 21 Feb 2009, Jean Delvare wrote:
> > Well, that's basically what Hans has been doing with
> > v4l2-i2c-drv-legacy.h for months now, isn't it? This is the easy part
> > (even though e
unless they started to shout at me, at which point I would start to
> ignore them.
>
> Probably all of the above would please nobody, and it is a good that I am
> not in charge of anything.
Actually, it would totally please me :)
--
Jean Delvare
--
To unsubscribe from this list: s
value back
from the chip. You can't do that on all I2C buses (writing to
arbitrary registers is bad) but you can do it on your specific
adapter. Depending on the result you then call i2c_new_device(A) or
i2c_new_device(B).
The 3rd method is very similar to the legacy i2c binding model.
--
On Sat, 21 Feb 2009 14:28:31 +0100, Hans Verkuil wrote:
> On Saturday 21 February 2009 14:11:30 Jean Delvare wrote:
> > Well, that's basically what Hans has been doing with
> > v4l2-i2c-drv-legacy.h for months now, isn't it? This is the easy part
> > (even though e
> As I've said before, the v4l2-i2c headers are lot more complicated than
> they need to be. I have a tree that's shrunk them greatly. I don't think
> it's fair to give the current headers as an example of how complicated i2c
> backward compat _must_ be.
Well, why don
Hi Mauro,
Only answering points Hans didn't already answered (and I agree with
him):
On Fri, 20 Feb 2009 21:23:27 -0300, Mauro Carvalho Chehab wrote:
> On Wed, 18 Feb 2009 14:01:05 +0100
> Jean Delvare wrote:
> > Not necessarily something to be proud about. This only shows h
The lock_norm module parameter doesn't look terribly useful. If you
don't want to change the norm, just don't change it. As a matter of
fact, no other v4l driver has such a parameter.
Signed-off-by: Jean Delvare
Cc: Trent Piepho
Cc: Hans Verkuil
---
Hans, this goes on top of you
s we support officially. In the past this was more
> based on hope and a prayer :-)
That's better than before, but just because it builds doesn't mean it
works...
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a m
hat kernels you want to support. I think we have to be
pragmatic here. We want to support kernels which users really care
about (and these are the ones in maintained popular non-enterprise
distributions) and which don't cost too much to support from a
technical point of view.
Now, if you thi
The v4l-dvb repository supports kernel versions 2.6.16 and later only,
so no need to test for kernel version 2.6.15.
Signed-off-by: Jean Delvare
---
linux/drivers/media/dvb/dvb-usb/af9015.c |3 ---
1 file changed, 3 deletions(-)
--- v4l-dvb-zoran.orig/linux/drivers/media/dvb/dvb-usb/af9015
to be too high, so there's a balance to be found between users and
developers. At the moment the balance isn't right IMHO.
Thanks,
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
Mor
(SysRq or remote logging,
killing X and reboot.)
It doesn't happen on each rmmod cx8800, but as I managed to reproduce
it once, I guess I should be able to reproduce it again if needed.
Please let me know if you need more information or want me to test
something.
--
Jean Delvare
--
To unsub
On Sun, 1 Feb 2009 11:42:57 -0200, Mauro Carvalho Chehab wrote:
> On Sun, 1 Feb 2009 14:29:27 +0100
> Jean Delvare wrote:
>
> > Hi Mauro,
> >
> > On Thu, 29 Jan 2009 19:24:31 -0200, Mauro Carvalho Chehab wrote:
> > > On Thu, 29 Jan 2009 22:1
On Sun, 1 Feb 2009 11:42:57 -0200, Mauro Carvalho Chehab wrote:
> On Sun, 1 Feb 2009 14:29:27 +0100
> Jean Delvare wrote:
>
> > On Thu, 29 Jan 2009 19:24:31 -0200, Mauro Carvalho Chehab wrote:
> > > I have already a pull request almost ready that will fix this issue. I
Hi Mauro,
On Thu, 29 Jan 2009 19:24:31 -0200, Mauro Carvalho Chehab wrote:
> On Thu, 29 Jan 2009 22:10:12 +0100
> Jean Delvare wrote:
>
> > Hi folks,
> >
> > I have a CX88-based DVB-T adapter which worked fine up to 2.6.28 but is
> > broken since 2.6.29-rc1
pretty wrong, however the linux-i2c list isn't the right
place to discuss this.
--
Jean Delvare
--
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
add+0x1f/0x98
RSP
CR2:
---[ end trace c6ad9ed793b52081 ]---
This is on x86-64. Any additional information available on request.
Thanks,
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.
tv.org/hg/v4l-dvb
> +S: Odd Fixes
>
> ZS DECSTATION Z85C30 SERIAL DRIVER
> P: Maciej W. Rozycki
Acked-by: Jean Delvare
--
Jean Delvare
--
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
Hi Trent,
On Sat, 17 Jan 2009 11:45:57 -0800 (PST), Trent Piepho wrote:
> On Fri, 16 Jan 2009, Jean Delvare wrote:
> > On Fri, 16 Jan 2009 05:34:59 -0800 (PST), Trent Piepho wrote:
> > > How will this work for drivers like bttv, where the i2c address of the
> > > tuner
Hi Trent,
On Fri, 16 Jan 2009 05:34:59 -0800 (PST), Trent Piepho wrote:
> On Fri, 16 Jan 2009, Jean Delvare wrote:
> > Hi Mauro, Trent,
> > On Fri, 16 Jan 2009 00:02:52 -0200, Mauro Carvalho Chehab wrote:
> > > For now, we should finish the Hans approach, since it also he
e of transparency as the bus segment one:
access through i2c-dev for example is done as if the gate did not
exist, and the gate can't accidentally be left open. I think this
approach has my preference, because it is cheap and can be implemented
right now.
* At the chip driver level, using a cus
On Fri, 9 Jan 2009 21:28:31 +0100, Jean Delvare wrote:
> On Fri, 9 Jan 2009 12:12:09 -0800 (PST), Trent Piepho wrote:
> > I have some more patches at http://linuxtv.org/hg/~tap/zoran
>
> I'll take a look, thanks.
As discussed on IRC: all patches tested, they worked fine for
> repository which isn't quite the same as what's in the kernel.
> >
> > I have some more patches at http://linuxtv.org/hg/~tap/zoran
>
> Forgot the patch
Patch tested successfully. Nice cleanup! I wanted to do it for quite
some time but could never find the time.
On Fri, 9 Jan 2009 12:12:09 -0800 (PST), Trent Piepho wrote:
> On Fri, 9 Jan 2009, Jean Delvare wrote:
> > On Fri, 9 Jan 2009 09:20:18 -0200, Mauro Carvalho Chehab wrote:
> > > Unfortunately, I don't have any Zoran card here to test.
> > >
> > > Je
(zoran_num < BUZ_MAX &&
> > >(dev = pci_get_device(PCI_VENDOR_ID_ZORAN,
> > > PCI_DEVICE_ID_ZORAN_36057, dev)) != NULL) {
> > > ...
> > > }
> >
> > Yuck, why don't we fix this instead?
>
> This will be mu
> ...
> > }
>
> Yuck, why don't we fix this instead?
>
> Here's an initial test. I haven't yet found my dc10+ to test it with.
I'm all for it, but what tree was your patch built against? It doesn't
seem to apply to anything I have.
--
J
68c46f
>
> Changeset 60b4bde48b36c0315ef41fd38c339b9c7e68c46f removed an unused struct on
> zoran driver, when compiled with "Y". However, as pointed by Jean Delvare
> ,
> this is neeeded when the driver is compiled as a module, since udev relies on
> it to
> auto
301 - 369 of 369 matches
Mail list logo