[PATCH] [media] dib0700: Fix memory leak during initialization

2012-02-12 Thread Jean Delvare
Reported by kmemleak. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@infradead.org Cc: Devin Heitmueller dheitmuel...@kernellabs.com --- I am not familiar with the usb API, are we also supposed to call usb_kill_urb() in the error case maybe? drivers/media/dvb

[PATCH] cx22702: Fix signal strength

2012-02-12 Thread Jean Delvare
. In practice I get 67% with my antenna fully amplified and 51% with no amplification. This is close enough to what I get on my other DVB-T adapter with the same antenna. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Steven Toth st...@kernellabs.com --- This was written without a datasheet so

Re: [PATCH] [media] dib0700: Fix memory leak during initialization

2012-03-12 Thread Jean Delvare
Hi Mauro, Thanks for your reply. On Thu, 08 Mar 2012 08:21:20 -0300, Mauro Carvalho Chehab wrote: Em 12-02-2012 08:19, Jean Delvare escreveu: Reported by kmemleak. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@infradead.org Cc: Devin Heitmueller

Re: [PATCH] [media] dib0700: Fix memory leak during initialization

2012-03-13 Thread Jean Delvare
On Mon, 12 Mar 2012 07:28:02 -0300, Mauro Carvalho Chehab wrote: Em 12-03-2012 07:04, Jean Delvare escreveu: !d can't actually happen, so it doesn't matter. d is passed by dib0700_rc_setup() when calling usb_fill_bulk_urb(), and dib0700_rc_setup() starts with dereferencing d, if it was NULL

[PATCH 1/2] [media] dib0700: Drop useless check when remote key is pressed

2012-03-13 Thread Jean Delvare
would crash right away. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@infradead.org Cc: Devin Heitmueller dheitmuel...@kernellabs.com --- Devin, am I missing something? drivers/media/dvb/dvb-usb/dib0700_core.c |3 --- 1 file changed, 3 deletions(-) --- linux

[PATCH 2/2 v2] [media] dib0700: Fix memory leak during initialization

2012-03-13 Thread Jean Delvare
Reported by kmemleak. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@infradead.org Cc: Devin Heitmueller dheitmuel...@kernellabs.com --- Changes since v1: * Don't free the URB when it is still in use. * Fix a second leak (transfer_buffer). drivers/media/dvb/dvb

Re: [PATCH 1/2] [media] dib0700: Drop useless check when remote key is pressed

2012-03-20 Thread Jean Delvare
Hi Mauro, On Mon, 19 Mar 2012 19:26:11 -0300, Mauro Carvalho Chehab wrote: On Tue, Mar 13, 2012 at 1:50 PM, Jean Delvare kh...@linux-fr.org wrote: --- linux-3.3-rc7.orig/drivers/media/dvb/dvb-usb/dib0700_core.c 2012-03-13 11:09:13.0 +0100 +++ linux-3.3-rc7/drivers/media/dvb

Re: [PATCH 1/2] [media] dib0700: Drop useless check when remote key is pressed

2012-03-21 Thread Jean Delvare
Hi Mauro, On Tue, 20 Mar 2012 09:17:54 -0300, Mauro Carvalho Chehab wrote: Em 20-03-2012 04:20, Jean Delvare escreveu: On Mon, 19 Mar 2012 19:26:11 -0300, Mauro Carvalho Chehab wrote: Yet, I'd be more happy if Jean's patch could check first if the status is below 0, in order to prevent

[media] m920x: Fix uninitialized variable warning

2013-03-31 Thread Jean Delvare
. OTOH the initialization of ret in m920x_init is needless, the function returns with an error as soon as an error happens, so the last return can only be a success and we can hard-code 0 there. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: Antonio

[PATCH v2] [media] m920x: Fix uninitialized variable warning

2013-03-31 Thread Jean Delvare
. OTOH the initialization of ret in m920x_init is needless, the function returns with an error as soon as an error happens, so the last return can only be a success and we can hard-code 0 there. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: Antonio

[PATCH] [media] sony-btf-mpx: Drop needless newline in param description

2013-05-15 Thread Jean Delvare
Module parameter descriptions need not be terminated with a newline. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Hans Verkuil hans.verk...@cisco.com Cc: Mauro Carvalho Chehab mche...@redhat.com --- drivers/media/i2c/sony-btf-mpx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 0/3] dvb-apps: Improve femon

2013-06-03 Thread Jean Delvare
Improvements to dvb-apps/femon: * femon: Share common code * femon: Display SNR in dB * femon: Handle -EOPNOTSUPP -- 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

[PATCH 2/3] femon: Display SNR in dB

2013-06-03 Thread Jean Delvare
); } else { printf (signal %04x | snr %04x | ber %08x | unc %08x | , fe_info.signal_strength, -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org

[PATCH 3/3] femon: Handle -EOPNOTSUPP

2013-06-03 Thread Jean Delvare
| ); } if (fe_info.lock) -- 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

Re: [PATCH] [media] drxk: change it to use request_firmware_nowait()

2012-06-29 Thread Jean Delvare
i2c_transfer(), not at its boundaries. I'm looking into this now. -- 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

[PATCH] i2c: Export an unlocked flavor of i2c_transfer

2012-06-29 Thread Jean Delvare
messages are logged. We don't want to log messages while holding a rt_mutex. * No check is done on the existence of adap-algo-master_xfer. It is thus the caller's responsibility to ensure that the function is OK to call. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche

Re: cannot ret error from probe - switch tuner to I2C driver model

2013-10-15 Thread Jean Delvare
Hi Antti, The driver's .probe() after i2c_new_device() is simply not supposed to fail. You should only use i2c_new_device() if you are 100% sure that there is a device of the given type at the given address. Checking i2c_get_clientdata() is an ugly trick that you should no longer need to use.

Re: [PATCH REVIEW] e4000: convert DVB tuner to I2C driver model

2013-10-16 Thread Jean Delvare
probe? Oh, i2c_new_probed_device() probes the device, what a surprise! :D Try, I don't know, i2c_new_device() maybe if you don't want the probe? ;) -- 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

Re: [PATCH REVIEW] e4000: convert DVB tuner to I2C driver model

2013-10-16 Thread Jean Delvare
to i2c_new_probed_device(). That probe function can read whatever chip ID register exists to decide if the probe should be successful or not. -- 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

Re: [PATCH 2/3] femon: Display SNR in dB

2013-11-25 Thread Jean Delvare
Hi Manu, On Sun, 24 Nov 2013 22:51:33 +0530, Manu Abraham wrote: Sorry, that I came upon this patch quite late. No problem, better late than never! :) On Mon, Jun 3, 2013 at 8:51 PM, Jean Delvare kh...@linux-fr.org wrote: SNR is supposed to be reported by the frontend drivers in dB, so

Re: [PATCH 13/13] drivers/media/tuners/e4000.c: use macros for i2c_msg initialization

2012-12-18 Thread Jean Delvare
subsystem maintainer, so I will not handle this. If you think this is important, you'll have to resubmit and Wolfram will decide what he wants to do about it. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

[PATCH] [media] usbvision: Drop broken 10-bit I2C address support

2011-11-07 Thread Jean Delvare
or the like. So let's simply get rid of the broken and useless code. I'm also adding I2C_FUNC_I2C, as the driver and hardware support plain I2C messaging. [1] http://marc.info/?l=linux-i2cm=117499415208244w=2 Signed-off-by: Jean Delvare kh...@linux-fr.org --- drivers/media/video/usbvision

Re: [RFC 1/2] dvb-core: add generic helper function for I2C register

2011-11-09 Thread 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

Re: [RFC 1/2] dvb-core: add generic helper function for I2C register

2011-11-09 Thread Jean Delvare
messages in the first place. -- 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

[PATCH] [media] video: Drop undue references to i2c-algo-bit

2011-11-09 Thread Jean Delvare
drivers, to avoid any confusion. This is the best way to ensure that the comments won't go out of sync again. Anyone interested in the implementation details would rather look at the code itself. Signed-off-by: Jean Delvare kh...@linux-fr.org --- drivers/media/video/au0828/au0828-i2c.c |2

Re: [RFC 2/2] tda18218: use generic dvb_wr_regs()

2011-11-10 Thread Jean Delvare
on me to drive it, I am afraid it will take months, given my current workload and various tasks with a much higher priority than this. I will however be happy to help with code review. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message

Re: [RFC 1/2] dvb-core: add generic helper function for I2C register

2011-11-10 Thread Jean Delvare
On Wed, 09 Nov 2011 10:02:08 -0200, Mauro Carvalho Chehab wrote: Em 09-11-2011 08:37, Jean Delvare escreveu: Speaking of struct i2c_client, I seem to remember that the dvb subsystem doesn't use it much at the moment. This might be an issue if you intend to get the generic code into i2c-core

Re: [RFC 1/2] dvb-core: add generic helper function for I2C register

2011-11-14 Thread Jean Delvare
Hi Antti, As an additional note, it just occurred to me that what you are working on is somewhat related to Mark Brown's regmap. Look in drivers/base/regmap and see if maybe you can reuse and/or extend Mark's approach. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 10/10] media/dvb: add __init/__exit macros to drivers/media/dvb/bt8xx/bt878.c

2010-01-05 Thread Jean Delvare
it through your tree, or please ack' it so Jiri can pull it through the trivial tree. Patch against linux-next-tree, 22. Dez 08:38:18 CET 2009 but also present in linus tree. Signed-off-by: Peter Huewe peterhu...@gmx.de Acked-by: Jean Delvare kh...@linux-fr.org --- drivers/media/dvb/bt8xx/bt878

Re: IR device at I2C address 0x7a

2010-01-06 Thread Jean Delvare
with no known IR setup. It doesn't imply your board has any I2C device at address 0x7a. So chances are that the message is harmless and you can simply ignore it. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

Re: IR device at I2C address 0x7a:

2010-01-06 Thread Jean Delvare
it. -- 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

Re: IR device at I2C address 0x7a

2010-01-06 Thread Jean Delvare
On Wed, 06 Jan 2010 20:10:30 +0100, Daro wrote: W dniu 06.01.2010 19:40, Jean Delvare pisze: On Wed, 06 Jan 2010 18:58:58 +0100, Daro wrote: It is not the error message itself that bothers me but the fact that IR remote control device is not detected and I cannot use it (I checked

Re: IR device at I2C address 0x7a

2010-01-09 Thread Jean Delvare
On Sat, 09 Jan 2010 13:08:36 +0100, Daro wrote: W dniu 06.01.2010 21:21, Jean Delvare pisze: On Wed, 06 Jan 2010 18:58:58 +0100, Daro wrote: It is not the error message itself that bothers me but the fact that IR remote control device is not detected and I cannot use it (I checked

Re: IR device at I2C address 0x7a

2010-01-10 Thread Jean Delvare
On Sun, 10 Jan 2010 00:18:46 +0100, hermann pitton wrote: Hi, Am Samstag, den 09.01.2010, 17:14 +0100 schrieb Jean Delvare: On Sat, 09 Jan 2010 13:08:36 +0100, Daro wrote: W dniu 06.01.2010 21:21, Jean Delvare pisze: On Wed, 06 Jan 2010 18:58:58 +0100, Daro wrote

Re: I2C transaction questions: irq context and client locking

2010-01-17 Thread Jean Delvare
of i2c_transfer() won't please you. Either it sleeps, or it returns -EAGAIN, which you'll have to handle. If you retry forever without sleeping, it might take long before you can proceed, and performance will suffer. Hope that helps, -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-01-30 Thread Jean Delvare
Hi Mauro, Hermann, On Sat, 30 Jan 2010 01:47:41 +0100, hermann pitton wrote: Am Freitag, den 29.01.2010, 13:40 -0200 schrieb Mauro Carvalho Chehab: Jean Delvare wrote: From: Jean Delvare kh...@linux-fr.org Subject: saa7134: Fix IR support of some ASUS TV-FM 7135 variants Some

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-01 Thread Jean Delvare
had to fake anything? What I'd like to know is simply if my first patch had any negative effect on other cards. -- 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

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-01 Thread Jean Delvare
Hi Hermann, On Tue, 02 Feb 2010 02:47:53 +0100, hermann pitton wrote: Hi Jean, Am Montag, den 01.02.2010, 10:56 +0100 schrieb Jean Delvare: Hi Hermann, On Mon, 01 Feb 2010 02:16:35 +0100, hermann pitton wrote: For now, I only faked a P7131 Dual with a broken IR receiver on a 2.6.29

[PATCH] adv7180 builds since kernel 2.6.26

2010-02-05 Thread Jean Delvare
VIDEO_ADV7180 is listed twice in v4l/versions.txt: once in [2.6.31] and once in [2.6.26]. As I have tested that it builds fine in 2.6.26, drop the former entry. Signed-off-by: Jean Delvare kh...@linux-fr.org --- v4l/versions.txt |1 - 1 file changed, 1 deletion(-) --- v4l-dvb.orig/v4l

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-10 Thread Jean Delvare
it would be good to make sure it fixes your problem before actually applying it. 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

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-10 Thread Jean Delvare
Hi Mauro, On Tue, 02 Feb 2010 17:09:05 -0200, Mauro Carvalho Chehab wrote: From: Jean Delvare kh...@linux-fr.org Subject: saa7134: Fix IR support of some ASUS TV-FM 7135 variants Some variants of the ASUS TV-FM 7135 are handled as the ASUSTeK P7131 Analog (card=146). However

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-10 Thread Jean Delvare
. If you don't like it and prefer the second proposal even though I think it's more of a workaround than a proper fix, it's really up to you. You're maintaining the subsystem and I am not, so you're the one deciding. Thanks, -- Jean Delvare -- To unsubscribe from this list: send the line

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-10 Thread Jean Delvare
On Wed, 10 Feb 2010 16:40:03 -0200, Mauro Carvalho Chehab wrote: Jean Delvare wrote: Under the assumption that saa7134_hwinit1() only touches GPIOs connected to IR receivers (and it certainly looks like this to me) I fail to see how these pins not being initialized could have any effect

[PATCH 1/2] bttv: Move I2C IR initialization

2010-02-16 Thread Jean Delvare
. This fixes bug #15184: http://bugzilla.kernel.org/show_bug.cgi?id=15184 Signed-off-by: Jean Delvare kh...@linux-fr.org --- As this fixes a regression, I suggest pushing to Linus quickly. This is a candidate for 2.6.32-stable too. linux/drivers/media/video/bt8xx/bttv-driver.c |1 + linux

Re: Status of the patches under review (29 patches)

2010-02-25 Thread Jean Delvare
On Wed, 24 Feb 2010 10:10:16 -0300, Mauro Carvalho Chehab wrote: Hi Jean, Jean Delvare wrote: I have 3 patches pending which aren't in your list. I can see them in patchwork: http://patchwork.kernel.org/patch/79755/ http://patchwork.kernel.org/patch/79754/ http

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-25 Thread Jean Delvare
for different LNA stuff after the Hauppauge/Pinnacle merge until now. If you claim to know it better, please share with us. I'm not claiming anything, and to be honest, I have no idea what you're talking about. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux

Re: [PATCH] bttv: fix compiler warning before kernel 2.6.30

2010-03-06 Thread Jean Delvare
function Signed-off-by: Márton Németh nm...@freemail.hu Good catch. Acked-by: Jean Delvare kh...@linux-fr.org Douglas, please apply quickly. --- diff -r 41c5482f2dac linux/drivers/media/video/bt8xx/bttv-i2c.c --- a/linux/drivers/media/video/bt8xx/bttv-i2c.c Thu Mar 04 02:49:46 2010 -0300

Re: [IR RC, REGRESSION] Didn't work IR RC

2010-03-09 Thread Jean Delvare
) on it. The getkey function will need to return the full code as well. Sorry for the late reply. Is the problem solved by now, or is my help still needed? -- 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

Re: [IR RC, REGRESSION] Didn't work IR RC

2010-03-10 Thread Jean Delvare
, so I admit I don't quite understand how I my i2c changes could be responsible for the regression. Anyway, glad that you managed to fix it. -- 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

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-03-12 Thread Jean Delvare
upstream without it, and if it takes too long, I'll probably just discard the patch and move to other tasks. -- 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

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-03-14 Thread Jean Delvare
parameter to the driver. If you ever need to remove the patch, use: $ cd ~/src/linux-2.6.32 $ patch -p2 -R ~/download/saa7134-Fix-IR-support-of-some-ASUS-TV-FM-7135-variants.patch I hope my instructions are clear enough, if you have any problem, just ask. Thanks, -- Jean Delvare

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-03-14 Thread Jean Delvare
On Sun, 14 Mar 2010 20:34:34 +0100, Daro wrote: W dniu 14.03.2010 09:26, Jean Delvare pisze: It will be easier with the kernel you compiled yourself. First of all, download the patch from: http://patchwork.kernel.org/patch/75883/raw/ Then, move to the source directory of your 2.6.32.2

[PATCH] FusionHDTV: Use quick reads for I2C IR device probing

2010-03-19 Thread Jean Delvare
On Tue, 16 Mar 2010 12:05:02 +0100, Jean Delvare wrote: Executive summary (as I understand it): the card that no longer works is a DViCO FusionHDTV7 Dual Express (CX23885_BOARD_DVICO_FUSIONHDTV_7_DUAL_EXP), bridge driver cx23885. It has 2 xc5000 chips at I2C address 0x64 (on 2 different I2C

Re: [PATCH 12/24] media/video: fix dangling pointers

2010-03-21 Thread Jean Delvare
but not in saa7115.c, which has exactly the same construct? For that matter, I think almost all v4l i2c drivers do this. That I can't say, sorry. -- 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

Re: [PATCH 12/24] media/video: fix dangling pointers

2010-03-22 Thread Jean Delvare
Replying to myself... On Sun, 21 Mar 2010 14:46:55 +0100, Jean Delvare wrote: I get the feeling that this would be a job for managed resources as some drivers already do for I/O ports and IRQs. Managed resources don't care about symmetry of allocation and freeing, by design (so it can violate

Re: i2c interface bugs in dvb drivers

2010-03-24 Thread Jean Delvare
. This is sufficient in most cases in my experience. If not, these tools can certainly get extended, at least to cover all of SMBus. -- 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

Re: [PATCH] FusionHDTV: Use quick reads for I2C IR device probing

2010-03-29 Thread Jean Delvare
Can the fix below please be picked quickly? This is a regression, the fix should go upstream ASAP. Thanks. On Fri, 19 Mar 2010 14:42:50 +0100, Jean Delvare wrote: IR support on FusionHDTV cards is broken since kernel 2.6.31. One side effect of the switch to the standard binding model for IR I2C

[PATCH] cx88: High resolution timer for Remote Controls

2009-07-02 Thread Jean Delvare
From: Andrzej Hajda andrzej.ha...@wp.pl Patch solves problem of missed keystrokes on some remote controls, as reported on http://bugzilla.kernel.org/show_bug.cgi?id=9637 . Signed-off-by: Andrzej Hajda andrzej.ha...@wp.pl Signed-off-by: Jean Delvare kh...@linux-fr.org --- Resending because last

Re: [PATCH] cx88: High resolution timer for Remote Controls

2009-07-03 Thread Jean Delvare
On Thu, 2 Jul 2009 16:50:35 +0200, Jean Delvare wrote: From: Andrzej Hajda andrzej.ha...@wp.pl Patch solves problem of missed keystrokes on some remote controls, as reported on http://bugzilla.kernel.org/show_bug.cgi?id=9637 . Signed-off-by: Andrzej Hajda andrzej.ha...@wp.pl Signed-off

[PATCH 2/2] cx88: High resolution timer for Remote Controls

2009-07-03 Thread Jean Delvare
Patch solves problem of missed keystrokes on some remote controls, as reported on http://bugzilla.kernel.org/show_bug.cgi?id=9637 . Signed-off-by: Andrzej Hajda andrzej.ha...@wp.pl Signed-off-by: Jean Delvare kh...@linux-fr.org --- Changes: * Driver no longer builds on kernels 2.6.22, so add

Re: [PATCH 1/2] Compatibility layer for hrtimer API

2009-07-05 Thread Jean Delvare
Hi Trent, On Sun, 5 Jul 2009 01:13:14 -0700 (PDT), Trent Piepho wrote: On Fri, 3 Jul 2009, Jean Delvare wrote: Kernels 2.6.22 to 2.6.24 (inclusive) need some compatibility quirks for the hrtimer API. For older kernels, some required functions were not exported so there's nothing we can do

Re: [RFC] Anticipating lirc breakage

2009-07-09 Thread Jean Delvare
On Thu, 9 Jul 2009 11:44:46 -0400, Jarod Wilson wrote: On Tuesday 07 April 2009 08:36:17 Jean Delvare wrote: So, let's just forget the workarounds and go straight to the point: focus on merging lirc-i2c drivers. Will this happen next week? I fear not. Which is why I can't wait

Re: [PATCH 1/3] ir-kbd-i2c: Allow use of ir-kdb-i2c internal get_key funcs and set ir_type

2009-07-19 Thread Jean Delvare
regression. This is bad. If there a bugzilla entry? If not, where can I read more details / get in touch with an affected user? -- 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

Re: [PATCH 1/3] ir-kbd-i2c: Allow use of ir-kdb-i2c internal get_key funcs and set ir_type

2009-07-19 Thread Jean Delvare
)(struct IR_i2c*, u32*, u32*); + enum ir_kbd_get_key_fn internal_get_key_func; }; #endif -- 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

Re: [PATCH 2/3] 2/3: cx18: Add i2c initialization for Z8F0811/Hauppage IR transceivers

2009-07-19 Thread Jean Delvare
*/ sd = v4l2_i2c_new_subdev(cx-v4l2_dev, adap, mod, type, hw_addrs[idx]); if (sd != NULL) sd-grp_id = hw; The rest looks OK. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

Re: [PATCH 1/3] ir-kbd-i2c: Allow use of ir-kdb-i2c internal get_key funcs and set ir_type

2009-07-19 Thread Jean Delvare
On Sun, 19 Jul 2009 09:17:30 -0400, Mark Lord wrote: Mark Lord wrote: Jean Delvare wrote: Hi Mark, On Sun, 19 Jul 2009 08:52:09 -0400, Mark Lord wrote: While you folks are looking into ir-kbd-i2c, perhaps one of you will fix the regressions introduced in 2.6.31-* ? The drive

Re: [PATCH 1/3] ir-kbd-i2c: Allow use of ir-kdb-i2c internal get_key funcs and set ir_type

2009-07-19 Thread Jean Delvare
they first try 2.6.31 after release ? Where does lsinput get the string from? What exactly was it before, and what is it exactly in 2.6.31? -- 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

Re: [PATCH 3/3] ir-kbd-i2c: Add support for Z8F0811/Hauppage IR transceivers

2009-07-19 Thread Jean Delvare
entry for any IR receiver */ { ir_video, 0 }, - /* IR device specific entries could be added here */ + /* IR device specific entries should be added here */ + { ir_rx_z8f0811_haup, 0 }, { } }; Yes, looks good. -- Jean Delvare -- To unsubscribe from this list: send

Re: Regression 2.6.31: ioctl(EVIOCGNAME) no longer returns device name

2009-07-19 Thread Jean Delvare
On Sun, 19 Jul 2009 15:20:50 -0400, Mark Lord wrote: Mark Lord wrote: (resending.. somebody trimmed linux-kernel from the CC: earlier) FWIW I don't think it was there in the first place. Jean Delvare wrote: On Sun, 19 Jul 2009 10:38:37 -0400, Mark Lord wrote: I'm debugging various other

Re: ir-kbd-i2c: Drop irrelevant inline keywords

2009-07-21 Thread Jean Delvare
On Mon, 20 Jul 2009 20:09:44 -0400, Andy Walls wrote: On Sun, 2009-07-19 at 14:59 +0200, Jean Delvare wrote: Functions which are referenced by their address can't be inlined by definition. Signed-off-by: Jean Delvare kh...@linux-fr.org Jean, Looks godd to me, but you forgot to add

Re: [PATCH 1/3] ir-kbd-i2c: Allow use of ir-kdb-i2c internal get_key funcs and set ir_type

2009-07-21 Thread Jean Delvare
Hi Andy, On Mon, 20 Jul 2009 20:07:50 -0400, Andy Walls wrote: On Sun, 2009-07-19 at 14:47 +0200, Jean Delvare wrote: Hi Andy, On Fri, 17 Jul 2009 16:35:37 -0400, Andy Walls wrote: This patch augments the init data passed by bridge drivers to ir-kbd-i2c so that the ir_type can

Re: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb

2009-09-05 Thread Jean Delvare
kernel for me. Thanks go to Brian Rogers for pointing out the solution in the context of submitting a patch for a few other drivers. Good catch. Acked-by: Jean Delvare kh...@linux-fr.org As far as I can see, the em28xx and saa7134 drivers have the exact same problem. Is there anyone working

Re: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb

2009-09-06 Thread Jean Delvare
On Sun, 06 Sep 2009 12:31:24 -0400, Andy Walls wrote: On Sat, 2009-09-05 at 20:46 +0200, Jean Delvare wrote: Hi Andy, On Sat, 05 Sep 2009 10:13:41 -0400, Andy Walls wrote: Mauro, Please pull from http://linuxtv.org/hg/~awalls/v4l-dvb for the following changeset: 01

Re: [2.6.31] ir-kbd-i2c oops.

2009-09-16 Thread Jean Delvare
2.6.31 would be affected, 2.6.30 wouldn't be. -- 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

[PATCH] Fix adv7180 build failures with old kernels

2009-09-26 Thread Jean Delvare
The adv7180 driver is a new-style i2c driver, unconditionally using struct i2c_device_id. As such, it can't be built on kernels older than 2.6.26. Signed-off-by: Jean Delvare kh...@linux-fr.org --- v4l/versions.txt |1 + 1 file changed, 1 insertion(+) --- v4l-dvb.orig/v4l/versions.txt

Re: [2.6.31] ir-kbd-i2c oops.

2009-09-30 Thread Jean Delvare
Hi Pawel, I am removing the linux-i2c list from Cc, because it seems clear that your problem is related to specific media drivers and not the i2c subsystem. On Wed, 30 Sep 2009 10:16:15 +0200, Paweł Sikora wrote: On Tuesday 29 September 2009 16:16:29 Jean Delvare wrote: On Wed, 16 Sep 2009 10

Re: [2.6.31] ir-kbd-i2c oops.

2009-09-30 Thread Jean Delvare
On Wed, 30 Sep 2009 13:52:27 +0200, Paweł Sikora wrote: On Wednesday 30 September 2009 12:57:37 Jean Delvare wrote: Are you running distribution kernels or self-compiled ones? Any local patches applied? Would you be able to apply debug patches and rebuild your kernel? yes, i'm using

Re: [2.6.31] ir-kbd-i2c oops.

2009-10-01 Thread Jean Delvare
Hi Andy, On Wed, 30 Sep 2009 19:42:46 -0400, Andy Walls wrote: On Wed, 2009-09-30 at 12:57 +0200, Jean Delvare wrote: Not sure why you look at address 0x83e? The stack trace says +0x64. As function ir_input_init() starts at 0x800, the oops address would be 0x864, which is: 864

Re: [2.6.31] ir-kbd-i2c oops.

2009-10-01 Thread Jean Delvare
On Thu, 01 Oct 2009 12:17:20 +0200, Paweł Sikora wrote: Dnia 01-10-2009 o 12:06:09 Jean Delvare kh...@linux-fr.org napisał(a): I'm not sure if it is the problem here, but it may be prudent to check that there's no mismatch between the module and the structure definitions being pulled

Re: [2.6.31] ir-kbd-i2c oops.

2009-10-01 Thread Jean Delvare
ts; struct saa7134_dmaqueuets_q; -- 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

[PATCH] i2c_board_info can be local

2009-10-02 Thread Jean Delvare
saa7134_dev { /* I2C keyboard data */ #if LINUX_VERSION_CODE = KERNEL_VERSION(2, 6, 30) - struct i2c_board_info info; struct IR_i2c_init_datainit_data; #endif -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body

[PATCH] Fix wrong sizeof

2009-10-02 Thread Jean Delvare
Which is why I have always preferred sizeof(struct foo) over sizeof(var). Signed-off-by: Jean Delvare kh...@linux-fr.org --- linux/drivers/media/dvb/dvb-usb/ce6230.c|2 +- linux/drivers/media/video/saa7164/saa7164-cmd.c |2 +- 2 files changed, 2 insertions(+), 2 deletions

IR device at I2C address 0x7a

2009-10-02 Thread Jean Delvare
should be limited to the hardware in question. If we only have to care about the Upmost Purple TV, then the following patch should solve the problem: * * * * * From: Jean Delvare kh...@linux-fr.org Subject: saa7134: Fix IR support for Purple TV The i2c core prevents us from probing I2C address

Re: [2.6.31] ir-kbd-i2c oops.

2009-10-03 Thread Jean Delvare
Hi Pawel, On Sat, 3 Oct 2009 12:08:36 +0200, Paweł Sikora wrote: On Thursday 01 October 2009 13:43:43 Jean Delvare wrote: Pawel, please give a try to the following patch. Please keep the debug patches apply too, in case we need additional info. the second patch helps. here's a dmesg log

Re: [2.6.31] ir-kbd-i2c oops.

2009-10-03 Thread Jean Delvare
these messages? -- Jean Delvare http://khali.linux-fr.org/wishlist.html -- 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

Re: [REVIEW] ivtv, ir-kbd-i2c: Explicit IR support for the AVerTV M116 for newer kernels

2009-10-04 Thread Jean Delvare
this in the details reply I'm writing right now. -- 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

Re: [REVIEW] ivtv, ir-kbd-i2c: Explicit IR support for the AVerTV M116 for newer kernels

2009-10-04 Thread Jean Delvare
adapters then it makes some sense to move the definitions into ir-kbd-i2c. But if devices are heavily adapter-dependent, and moving the definitions into ir-kbd-i2c doesn't allow for any code refactoring, then I don't quite see the point. -- Jean Delvare -- To unsubscribe from this list: send the line

TerraTec Cinergy T PCIe Dual not working

2013-03-06 Thread Jean Delvare
results. How would I debug this further? 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

Re: TerraTec Cinergy T PCIe Dual not working

2013-03-06 Thread Jean Delvare
, what antenna are you using? The Terratec-provided one, or another one? -- 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

drxk driver statistics

2013-03-06 Thread Jean Delvare
. I would like to see these statistics improved. I am willing to help, however the drxk driver is rather complex (at least to my eyes) and I do not have a datasheet so I wouldn't know where to start. Is there anyone who can work on this and/or provide some guidance? Thanks, -- Jean Delvare

Re: TerraTec Cinergy T PCIe Dual not working

2013-03-06 Thread Jean Delvare
Hi Oliver, On Wed, 06 Mar 2013 19:07:01 +0100, Oliver Schinagl wrote: On 03/06/13 16:03, Jean Delvare wrote: It turns out that my problem is the antenna. I was using the antenna I have been using with my previous card, which is an internal DVB-T antenna with amplification (external power

[media] drxk_hard: Drop unused parameter

2013-03-20 Thread Jean Delvare
Last parameter of function GetLockStatus() isn't used so drop it. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Mauro Carvalho Chehab mche...@redhat.com --- drivers/media/dvb-frontends/drxk_hard.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) --- linux-3.8.orig/drivers

[PATCH] [media] mceusb: Optimize DIV_ROUND_CLOSEST call

2012-09-01 Thread Jean Delvare
DIV_ROUND_CLOSEST is faster if the compiler knows it will only be dealing with unsigned dividends. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Andrew Morton a...@linux-foundation.org Cc: Guenter Roeck li...@roeck-us.net Cc: Mauro Carvalho Chehab mche...@infradead.org --- drivers/media/rc

[PATCH] [media] cx23885: Select drivers for Terratec Cinergy T PCIe Dual

2012-09-17 Thread Jean Delvare
The Terratec Cinergy T PCIe Dual is based on the CX23885, and uses MT2063, DRX-3913k and DRX-3916k chips, so select the relevant drivers. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Stefan Ringel linu...@stefanringel.de Cc: Mauro Carvalho Chehab mche...@infradead.org --- drivers/media

Re: [PATCH] [media] mceusb: Optimize DIV_ROUND_CLOSEST call

2012-09-18 Thread Jean Delvare
Hi Mauro, On Tue, 18 Sep 2012 12:49:53 -0300, Mauro Carvalho Chehab wrote: Em 01-09-2012 15:53, Jean Delvare escreveu: DIV_ROUND_CLOSEST is faster if the compiler knows it will only be dealing with unsigned dividends. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: Andrew Morton

Re: [PATCH 9/13] drivers/media/tuners/fc0011.c: use macros for i2c_msg initialization

2012-10-09 Thread Jean Delvare
that. -- 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

Re: [PATCH 3/13] drivers/media/tuners/qt1010.c: use macros for i2c_msg initialization

2012-10-09 Thread Jean Delvare
can't, not in the general case at least. sizeof(*val) will return the size of the _first_ element of the destination buffer, which has nothing to do with the length of that buffer (which in turn might be rightfully longer than the read length for this specific message.) -- Jean Delvare

Re: [PATCH 2/13] drivers/media/tuners/mxl5007t.c: use macros for i2c_msg initialization

2012-10-09 Thread Jean Delvare
not opposed to the idea of i2c_msg initialization helper macros, but please don't mix that with actual code changes which could have bad side effects. Thanks, -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

Re: [PATCH 0/11] introduce macros for i2c_msg initialization

2012-10-09 Thread Jean Delvare
priority list. And I see the extra work for the pre-processor, so we need a good reason for doing that. -- 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

cx23885_wakeup: 3 buffers handled (should be 1)

2013-08-24 Thread Jean Delvare
of debugging or experimentation. 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

<    1   2   3   4   >