Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
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 kernel and apply > > the patch: > > > > $ cd ~/src/linux-2.6.32 > > $ patch -p2< > > ~/download/saa7134-Fix-IR-support-of-some-ASUS-TV-FM-7135-variants.patch > > > > Adjust the path in each command to match your own setup. Then just > > build and install the kernel: > > > > $ make > > $ sudo make modules_install > > $ sudo make install > > > > Or whatever method you use to install your self-compiled kernels. Then > > reboot to kernel 2.6.32.2 and test that the remote control works even > > when _not_ passing any card 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, > > > > Hi Jean! > > It works! > dmesg output is attached. Thanks for reporting! Mauro, you can pick my (second) patch now and apply it. > However I will have to go back to "generic" kernel as under my > "self-built" kernels fglrx driver stops working and I did not manage to > solve it :( > Or maybe you could give me a hint what to do with it? I can't help you with that, sorry, I don't use binary drivers. -- 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
W dniu 14.03.2010 09:26, Jean Delvare pisze: Hi Daro, On Sun, 14 Mar 2010 03:38:11 +0100, Daro wrote: Hi Jean, I am back and ready to go :) As I am not much experienced Linux user I would apprieciate some more details: I have few linux kernels installed; which one should I test or it does not matter? 2.6.31-14-generic 2.6.31-16-generic 2.6.31-17-generic 2.6.31-19-generic 2.6.31-20-generic and one I compiled myself 2.6.32.2 I assume that to proceed with a test I should patch the certain version of kernel and compile it or could it be done other way? 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 kernel and apply the patch: $ cd ~/src/linux-2.6.32 $ patch -p2< ~/download/saa7134-Fix-IR-support-of-some-ASUS-TV-FM-7135-variants.patch Adjust the path in each command to match your own setup. Then just build and install the kernel: $ make $ sudo make modules_install $ sudo make install Or whatever method you use to install your self-compiled kernels. Then reboot to kernel 2.6.32.2 and test that the remote control works even when _not_ passing any card 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, Hi Jean! It works! dmesg output is attached. However I will have to go back to "generic" kernel as under my "self-built" kernels fglrx driver stops working and I did not manage to solve it :( Or maybe you could give me a hint what to do with it? Best regards Daro [0.00] Initializing cgroup subsys cpuset [0.00] Initializing cgroup subsys cpu [0.00] Linux version 2.6.32.2 (r...@domowy) (gcc version 4.4.1 (Ubuntu 4.4.1-4ubuntu9) ) #2 SMP Sun Mar 14 14:11:25 CET 2010 [0.00] Command line: root=/dev/mapper/isw_bgccbabfgf_SYSTEM2 ro [0.00] KERNEL supported cpus: [0.00] Intel GenuineIntel [0.00] AMD AuthenticAMD [0.00] Centaur CentaurHauls [0.00] BIOS-provided physical RAM map: [0.00] BIOS-e820: - 00096c00 (usable) [0.00] BIOS-e820: 00096c00 - 000a (reserved) [0.00] BIOS-e820: 000e4000 - 0010 (reserved) [0.00] BIOS-e820: 0010 - 7ff8 (usable) [0.00] BIOS-e820: 7ff8 - 7ff8e000 (ACPI data) [0.00] BIOS-e820: 7ff8e000 - 7ffd (ACPI NVS) [0.00] BIOS-e820: 7ffd - 8000 (reserved) [0.00] BIOS-e820: fee0 - fee01000 (reserved) [0.00] BIOS-e820: fff0 - 0001 (reserved) [0.00] DMI present. [0.00] AMI BIOS detected: BIOS may corrupt low RAM, working around it. [0.00] e820 update range: - 0001 (usable) ==> (reserved) [0.00] last_pfn = 0x7ff80 max_arch_pfn = 0x4 [0.00] MTRR default type: uncachable [0.00] MTRR fixed ranges enabled: [0.00] 0-9 write-back [0.00] A-B uncachable [0.00] C-D write-protect [0.00] E-E write-through [0.00] F-F write-protect [0.00] MTRR variable ranges enabled: [0.00] 0 base 0 mask F8000 write-back [0.00] 1 disabled [0.00] 2 disabled [0.00] 3 disabled [0.00] 4 disabled [0.00] 5 disabled [0.00] 6 disabled [0.00] 7 disabled [0.00] x86 PAT enabled: cpu 0, old 0x7040600070406, new 0x7010600070106 [0.00] Scanning 0 areas for low memory corruption [0.00] modified physical RAM map: [0.00] modified: - 0001 (reserved) [0.00] modified: 0001 - 00096c00 (usable) [0.00] modified: 00096c00 - 000a (reserved) [0.00] modified: 000e4000 - 0010 (reserved) [0.00] modified: 0010 - 7ff8 (usable) [0.00] modified: 7ff8 - 7ff8e000 (ACPI data) [0.00] modified: 7ff8e000 - 7ffd (ACPI NVS) [0.00] modified: 7ffd - 8000 (reserved) [0.00] modified: fee0 - fee01000 (reserved) [0.00] modified: fff0 - 0001 (reserved) [0.00] initial memory mapped : 0 - 2000 [0.00] init_memory_mapping: -7ff8 [0.00] 00 - 007fe0 page 2M [0.00] 007fe0 - 007ff8 page 4k [0.00] kernel direct mapping tables up to 7ff8 @ 1-14000 [
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Hi Daro, On Sun, 14 Mar 2010 03:38:11 +0100, Daro wrote: > Hi Jean, > > I am back and ready to go :) > As I am not much experienced Linux user I would apprieciate some more > details: > > I have few linux kernels installed; which one should I test or it does > not matter? > 2.6.31-14-generic > 2.6.31-16-generic > 2.6.31-17-generic > 2.6.31-19-generic > 2.6.31-20-generic > > and one I compiled myself > 2.6.32.2 > > I assume that to proceed with a test I should patch the certain version > of kernel and compile it or could it be done other way? 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 kernel and apply the patch: $ cd ~/src/linux-2.6.32 $ patch -p2 < ~/download/saa7134-Fix-IR-support-of-some-ASUS-TV-FM-7135-variants.patch Adjust the path in each command to match your own setup. Then just build and install the kernel: $ make $ sudo make modules_install $ sudo make install Or whatever method you use to install your self-compiled kernels. Then reboot to kernel 2.6.32.2 and test that the remote control works even when _not_ passing any card 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 -- 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
Am Sonntag, den 14.03.2010, 03:38 +0100 schrieb Daro: > Hi Jean, > > I am back and ready to go :) > As I am not much experienced Linux user I would apprieciate some more > details: > > I have few linux kernels installed; which one should I test or it does > not matter? > 2.6.31-14-generic > 2.6.31-16-generic > 2.6.31-17-generic > 2.6.31-19-generic > 2.6.31-20-generic > > and one I compiled myself > 2.6.32.2 > > I assume that to proceed with a test I should patch the certain version > of kernel and compile it or could it be done other way? > > Best regards > Daro > > > > W dniu 12.03.2010 10:38, Jean Delvare pisze: > > Hi Daro, > > > > On Fri, 26 Feb 2010 17:19:38 +0100, Daro wrote: > > > >> I did not forget I had offered to test the patch however I am now on > >> vacation skiing so I will get back to you as soon I am back home. > >> > > Are you back home by now? We are still waiting for your test results. > > We can't push the patch upstream without it, and if it takes too long, > > I'll probably just discard the patch and move to other tasks. > > > > Hi Daro, thanks for being back on it. Damned jokes aside, and not made. Sorry, let know if you have any problems to get Jeans patch stuff applied. Jean's review is good and valuable and we might to have to come back to it, if stuff with the same PCI subsystem and different remotes is validated to be out. (The LNA hell is even much more burning ;) It very much looks like that is already the case. It is a little confusing these days, "patch" p0, p1, hg or git patches. I let it to Jean, how to test best for now, but will help to prepare you with stuff you can test easily under any conditions. Cheers, Hermann -- 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
Hi Jean, I am back and ready to go :) As I am not much experienced Linux user I would apprieciate some more details: I have few linux kernels installed; which one should I test or it does not matter? 2.6.31-14-generic 2.6.31-16-generic 2.6.31-17-generic 2.6.31-19-generic 2.6.31-20-generic and one I compiled myself 2.6.32.2 I assume that to proceed with a test I should patch the certain version of kernel and compile it or could it be done other way? Best regards Daro W dniu 12.03.2010 10:38, Jean Delvare pisze: Hi Daro, On Fri, 26 Feb 2010 17:19:38 +0100, Daro wrote: I did not forget I had offered to test the patch however I am now on vacation skiing so I will get back to you as soon I am back home. Are you back home by now? We are still waiting for your test results. We can't push the patch upstream without it, and if it takes too long, I'll probably just discard the patch and move to other tasks. -- 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
Hi Daro, On Fri, 26 Feb 2010 17:19:38 +0100, Daro wrote: > I did not forget I had offered to test the patch however I am now on vacation > skiing so I will get back to you as soon I am back home. Are you back home by now? We are still waiting for your test results. We can't push the patch 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/majordomo-info.html
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Hi Jean, Am Donnerstag, den 25.02.2010, 14:12 +0100 schrieb Jean Delvare: > On Sat, 20 Feb 2010 04:07:05 +0100, hermann pitton wrote: > > Are you still waiting for Daro's report? > > Yes, I am still waiting. Unfortunately neither Daro nor Roman reported > any test result so far. Now, if we go for my second patch, I guess we > might as well apply it right now. It only affects this one card (Asus > P7131 analog), and it was broken so far, so I don't think my patch can > do any bad. yes, we can go for your second patch without any risk. It even will work, but I'm wondering if I would have to buy such card, to get it through. Thanks for your time to review that. > > As said, I would prefer to see all OEMs _not_ following Philips/NXP > > eeprom rules running into their own trash on GNU/Linux too. > > > > Then we have facts. > > > > That is much better than to provide a golden cloud for them. At least I > > won't help to debug such later ... > > > > If you did not manage to decipher all OEM eeprom content already, > > just let's go with the per card solution for now. > > > > Are you aware, that my intention is _not_ to spread the use of random > > and potentially invalid eeprom content for some sort of such auto > > detection? > > > > The other solution is not lost and in mind, if we should need to come > > back to it and are in details. Preferably the OEMs should take the > > responsibility for such. > > > > We can see, that even those always doing best on it, can't provide the > > missing informations 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. Never mind. I'll keep you posted when it comes to further progress on such and input_init2 is the needed better solution. We have a whole chain of previously latest different Asus cards supported by PCI subsystem identification, only this single one fails on it and needs eeprom bogus detection. On the Pinnacle stuff we have a complete mess, concerning LNAs, likely even different LNAs, no LNA at all, different remotes etc., all with the same PCI subsystem. You can't even detect the LNA type, if you would know the meaning of the complete eeprom content, since some to us unknown check-sums are in use for that. That I try to tell. You are welcome, Hermann -- 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
On Sat, 20 Feb 2010 04:07:05 +0100, hermann pitton wrote: > Are you still waiting for Daro's report? Yes, I am still waiting. Unfortunately neither Daro nor Roman reported any test result so far. Now, if we go for my second patch, I guess we might as well apply it right now. It only affects this one card (Asus P7131 analog), and it was broken so far, so I don't think my patch can do any bad. > As said, I would prefer to see all OEMs _not_ following Philips/NXP > eeprom rules running into their own trash on GNU/Linux too. > > Then we have facts. > > That is much better than to provide a golden cloud for them. At least I > won't help to debug such later ... > > If you did not manage to decipher all OEM eeprom content already, > just let's go with the per card solution for now. > > Are you aware, that my intention is _not_ to spread the use of random > and potentially invalid eeprom content for some sort of such auto > detection? > > The other solution is not lost and in mind, if we should need to come > back to it and are in details. Preferably the OEMs should take the > responsibility for such. > > We can see, that even those always doing best on it, can't provide the > missing informations 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-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
Am Montag, den 15.02.2010, 06:31 +0100 schrieb hermann pitton: > Am Donnerstag, den 11.02.2010, 01:58 +0100 schrieb hermann pitton: > > Hi, > > > > Am Mittwoch, den 10.02.2010, 20:36 +0100 schrieb 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 > > > > > on non-IR code. > > > > > > > > Now, i suspect that you're messing things again: are you referring to > > > > saa7134_hwinit1() or > > > > to saa7134_input_init1()? > > > > > > > > I suspect that you're talking about moving saa7134_input_init1(), since > > > > saa7134_hwinit1() > > > > has the muted and spinlock inits. It also has the setups for video, vbi > > > > and mpeg. > > > > So, moving it require more care. > > > > > > Err, you're right, I meant saa7134_input_init1() and not > > > saa7134_hwinit1(), copy-and-paste error. Sorry for adding more > > > confusion where it really wasn't needed... > > > > > > > both attempts of Jean will work. > > > > If we are only talking about moving input_init, only that Jean did > > suggest initially, it should work, since only some GPIOs for enabling > > remote chips are affected. > > > > I can give the crappy tester, but don't have such a remote, but should > > not be a problem to trigger the GPIOs later. > > > > Cheers, > > Hermann > > > > Hi Jean, > > I did test your patch, only following Roman's initial patch already > known, on eight different cards for now, also with three slightly > different remotes and it does not have any negative impact. > > Please consider, that it is only about that single card for now and a > per card solution is enough. > > I strongly remind, that we should not rely on unknown eeprom bytes, as > told previously and should not expand such into any direction. > > If we make progress there, we should change it for all cards, but again, > what had happened on the m$ drivers previously is not encouraging to do > it without any need. > > To do it per card in need for now seems enough "service" to me. > > If more such should come, unlikely on that driver, I would at first deny > auto detection support, since they are breaking rules. > > The problem likely will time out very soon. > > Cheers, > Hermann Jean, a slight ping. Are you still waiting for Daro's report? As said, I would prefer to see all OEMs _not_ following Philips/NXP eeprom rules running into their own trash on GNU/Linux too. Then we have facts. That is much better than to provide a golden cloud for them. At least I won't help to debug such later ... If you did not manage to decipher all OEM eeprom content already, just let's go with the per card solution for now. Are you aware, that my intention is _not_ to spread the use of random and potentially invalid eeprom content for some sort of such auto detection? The other solution is not lost and in mind, if we should need to come back to it and are in details. Preferably the OEMs should take the responsibility for such. We can see, that even those always doing best on it, can't provide the missing informations for different LNA stuff after the Hauppauge/Pinnacle merge until now. If you claim to know it better, please share with us. Cheers, Hermann -- 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
Am Donnerstag, den 11.02.2010, 01:58 +0100 schrieb hermann pitton: > Hi, > > Am Mittwoch, den 10.02.2010, 20:36 +0100 schrieb 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 > > > > on non-IR code. > > > > > > Now, i suspect that you're messing things again: are you referring to > > > saa7134_hwinit1() or > > > to saa7134_input_init1()? > > > > > > I suspect that you're talking about moving saa7134_input_init1(), since > > > saa7134_hwinit1() > > > has the muted and spinlock inits. It also has the setups for video, vbi > > > and mpeg. > > > So, moving it require more care. > > > > Err, you're right, I meant saa7134_input_init1() and not > > saa7134_hwinit1(), copy-and-paste error. Sorry for adding more > > confusion where it really wasn't needed... > > > > both attempts of Jean will work. > > If we are only talking about moving input_init, only that Jean did > suggest initially, it should work, since only some GPIOs for enabling > remote chips are affected. > > I can give the crappy tester, but don't have such a remote, but should > not be a problem to trigger the GPIOs later. > > Cheers, > Hermann > Hi Jean, I did test your patch, only following Roman's initial patch already known, on eight different cards for now, also with three slightly different remotes and it does not have any negative impact. Please consider, that it is only about that single card for now and a per card solution is enough. I strongly remind, that we should not rely on unknown eeprom bytes, as told previously and should not expand such into any direction. If we make progress there, we should change it for all cards, but again, what had happened on the m$ drivers previously is not encouraging to do it without any need. To do it per card in need for now seems enough "service" to me. If more such should come, unlikely on that driver, I would at first deny auto detection support, since they are breaking rules. The problem likely will time out very soon. Cheers, Hermann -- 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
Hi, Am Mittwoch, den 10.02.2010, 20:36 +0100 schrieb 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 > > > on non-IR code. > > > > Now, i suspect that you're messing things again: are you referring to > > saa7134_hwinit1() or > > to saa7134_input_init1()? > > > > I suspect that you're talking about moving saa7134_input_init1(), since > > saa7134_hwinit1() > > has the muted and spinlock inits. It also has the setups for video, vbi and > > mpeg. > > So, moving it require more care. > > Err, you're right, I meant saa7134_input_init1() and not > saa7134_hwinit1(), copy-and-paste error. Sorry for adding more > confusion where it really wasn't needed... > both attempts of Jean will work. If we are only talking about moving input_init, only that Jean did suggest initially, it should work, since only some GPIOs for enabling remote chips are affected. I can give the crappy tester, but don't have such a remote, but should not be a problem to trigger the GPIOs later. Cheers, Hermann -- 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
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 > > on non-IR code. > > Now, i suspect that you're messing things again: are you referring to > saa7134_hwinit1() or > to saa7134_input_init1()? > > I suspect that you're talking about moving saa7134_input_init1(), since > saa7134_hwinit1() > has the muted and spinlock inits. It also has the setups for video, vbi and > mpeg. > So, moving it require more care. Err, you're right, I meant saa7134_input_init1() and not saa7134_hwinit1(), copy-and-paste error. Sorry for adding more confusion where it really wasn't 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 info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Jean Delvare wrote: > Hi Mauro, > > Sorry for the late answer. I'm tracking so many things in parallel... No problem. >> What happens is that the saa7134_board_init1(dev) code has lots of gpio >> codes, >> and most of those code is needed in order to enable i2c bridges or to turn >> on/reset >> some chips that would otherwise be on OFF or undefined state. Without the >> gpio code, >> done by init1, you may not be able to read eeprom or to detect the devices >> as needed >> by saa7134_board_init2(). > > I don't think I ever said otherwise. I never said that init1 as a whole > only cared about GPIO IR. That's what I said of function > saa7134_input_init1() and only this function! Ah, ok. I suspect I miss-understood when you talked about "init1", since 99% of the patches about init1 are related to board init, and not to input init. > My first proposed patch didn't move all of init1 to init2. It was only > moving saa7134_input_init1 (which doesn't yet have a counterpart in > init2), and it is moving it from the end of init1 to the beginning of > init2, so the movement isn't as big as it may sound. The steps > saa7134_input_init1() is moving over are, in order: > * saa7134_hw_enable1 > * request_irq > * saa7134_i2c_register > > So my point was that none of these 3 functions seemed to be dependent > on saa7134_input_init1() having been called beforehand. Which is why I > strongly question the fact that moving saa7134_input_init1() _after_ > these 3 other initialization steps can have any negative effect. I > wouldn't have claimed that moving saa7134_input_init1() _earlier_ in > the init sequence would be safe, because there's nothing obvious about > that. But moving it forward where I had put it did not seem terrific. I agree. In thesis, after board-specific init1 and init2, the device should be in good health and all the needs for IR should already be available. The only point that requires a double check is at the suspend function (as someone may try to suspend the machine in the middle of the board setup). > I really would like a few users of this driver to just give it a try > and report, because it seems a much more elegant fix to the bug at > hand, than the workaround you'd accept instead. Seems fair to me. >> That's why I'm saying that, if in the specific case of the board you're >> dealing with, >> you're sure that an unknown GPIO state won't affect the code at >> saa7134_hwinit1() and >> at saa7134_i2c_register(), then you can move the GPIO code for this board to >> init2. >> >> Moving things between init1 and init2 are very tricky and requires testing >> on all the >> affected boards. So a change like what your patch proposed would require a >> test of all >> the 107 boards that are on init1. I bet you'll break several of them with >> this change. > > 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 > on non-IR code. Now, i suspect that you're messing things again: are you referring to saa7134_hwinit1() or to saa7134_input_init1()? I suspect that you're talking about moving saa7134_input_init1(), since saa7134_hwinit1() has the muted and spinlock inits. It also has the setups for video, vbi and mpeg. So, moving it require more care. (btw, IMO all those *init1/*init2 names were a terrible choice) > Now, please don't get me wrong. I don't have any of these devices. I've > posted that patch because a user incidentally pointed me to a problem > and I had an idea how it could be fixed. I prefer my initial proposal > because it looks better in the long run. 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. Please, don't excuse yourself. You're right proposing a patch to help the user. Thanks for that. >From my side, I just want to make sure that we won't add any regressions with >other boards. Experience shows that touching at those init sequences (especially at the board init1/init2) cause lots of problem. For example, before the i2c redesign, there were several GPIO's inside init2. Due to that, if the i2c module got loaded before init2, several boards would fail. On those boards, people were required to load the modules on an specific order with some boards (and sometimes unload/reload an i2c driver), to be sure that the i2c devices would get properly initialized. Due to that, several boards didn't work if the drivers were compiled builtin. Also, there are some cases where it were impossible to have all saa7134 devices working on a machine with multiple saa7134 devices. During the time the i2c changes got merged, those GPIO sequences were moved to init1, but this broke several devices, and the init sequences needed to be manually adj
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Hi Mauro, Sorry for the late answer. I'm tracking so many things in parallel... On Tue, 02 Feb 2010 09:50:11 -0200, Mauro Carvalho Chehab wrote: > The init1 code has 107 boards listed: > > SAA7134_BOARD_10MOONSTVMASTER3 > SAA7134_BOARD_ADS_DUO_CARDBUS_PTV331 > SAA7134_BOARD_ASUST > SAA7134_BOARD_AVACSSMARTTV > SAA7134_BOARD_AVERMEDIA_305 > SAA7134_BOARD_AVERMEDIA_307 > SAA7134_BOARD_AVERMEDIA_777 > SAA7134_BOARD_AVERMEDIA_A169_B > SAA7134_BOARD_AVERMEDIA_A16AR > SAA7134_BOARD_AVERMEDIA_A16D > SAA7134_BOARD_AVERMEDIA_A700_HYBRID > SAA7134_BOARD_AVERMEDIA_A700_PRO > SAA7134_BOARD_AVERMEDIA_CARDBUS > SAA7134_BOARD_AVERMEDIA_CARDBUS_501 > SAA7134_BOARD_AVERMEDIA_CARDBUS_506 > SAA7134_BOARD_AVERMEDIA_GO_007_FM > SAA7134_BOARD_AVERMEDIA_GO_007_FM_PLUS > SAA7134_BOARD_AVERMEDIA_M102 > SAA7134_BOARD_AVERMEDIA_M103 > SAA7134_BOARD_AVERMEDIA_M115 > SAA7134_BOARD_AVERMEDIA_M135A > SAA7134_BOARD_AVERMEDIA_STUDIO_305 > SAA7134_BOARD_AVERMEDIA_STUDIO_307 > SAA7134_BOARD_AVERMEDIA_STUDIO_505 > SAA7134_BOARD_AVERMEDIA_STUDIO_507 > SAA7134_BOARD_BEHOLD_401 > SAA7134_BOARD_BEHOLD_403 > SAA7134_BOARD_BEHOLD_403FM > SAA7134_BOARD_BEHOLD_405 > SAA7134_BOARD_BEHOLD_405FM > SAA7134_BOARD_BEHOLD_407 > SAA7134_BOARD_BEHOLD_407FM > SAA7134_BOARD_BEHOLD_409 > SAA7134_BOARD_BEHOLD_409FM > SAA7134_BOARD_BEHOLD_505FM > SAA7134_BOARD_BEHOLD_505RDS_MK3 > SAA7134_BOARD_BEHOLD_505RDS_MK5 > SAA7134_BOARD_BEHOLD_507_9FM > SAA7134_BOARD_BEHOLD_507RDS_MK3 > SAA7134_BOARD_BEHOLD_507RDS_MK5 > SAA7134_BOARD_BEHOLD_607FM_MK3 > SAA7134_BOARD_BEHOLD_607FM_MK5 > SAA7134_BOARD_BEHOLD_607RDS_MK3 > SAA7134_BOARD_BEHOLD_607RDS_MK5 > SAA7134_BOARD_BEHOLD_609FM_MK3 > SAA7134_BOARD_BEHOLD_609FM_MK5 > SAA7134_BOARD_BEHOLD_609RDS_MK3 > SAA7134_BOARD_BEHOLD_609RDS_MK5 > SAA7134_BOARD_BEHOLD_COLUMBUS_TVFM > SAA7134_BOARD_BEHOLD_H6 > SAA7134_BOARD_BEHOLD_M6 > SAA7134_BOARD_BEHOLD_M63 > SAA7134_BOARD_BEHOLD_M6_EXTRA > SAA7134_BOARD_BEHOLD_X7 > SAA7134_BOARD_CINERGY400 > SAA7134_BOARD_CINERGY400_CARDBUS > SAA7134_BOARD_CINERGY600 > SAA7134_BOARD_CINERGY600_MK3 > SAA7134_BOARD_ECS_TVP3XP > SAA7134_BOARD_ECS_TVP3XP_4CB5 > SAA7134_BOARD_ECS_TVP3XP_4CB6 > SAA7134_BOARD_ENCORE_ENLTV > SAA7134_BOARD_ENCORE_ENLTV_FM > SAA7134_BOARD_ENCORE_ENLTV_FM53 > SAA7134_BOARD_FLYDVBS_LR300 > SAA7134_BOARD_FLYDVBTDUO > SAA7134_BOARD_FLYDVBT_DUO_CARDBUS > SAA7134_BOARD_FLYDVBT_HYBRID_CARDBUS > SAA7134_BOARD_FLYDVBT_LR301 > SAA7134_BOARD_FLYTVPLATINUM_FM > SAA7134_BOARD_FLYTVPLATINUM_MINI2 > SAA7134_BOARD_FLYVIDEO2000 > SAA7134_BOARD_FLYVIDEO3000 > SAA7134_BOARD_FLYVIDEO3000_NTSC > SAA7134_BOARD_GENIUS_TVGO_A11MCE > SAA7134_BOARD_GOTVIEW_7135 > SAA7134_BOARD_HAUPPAUGE_HVR1110 > SAA7134_BOARD_HAUPPAUGE_HVR1120 > SAA7134_BOARD_HAUPPAUGE_HVR1150 > SAA7134_BOARD_KWORLD_PLUS_TV_ANALOG > SAA7134_BOARD_KWORLD_TERMINATOR > SAA7134_BOARD_KWORLD_VSTREAM_XPERT > SAA7134_BOARD_KWORLD_XPERT > SAA7134_BOARD_LEADTEK_WINFAST_DTV1000S > SAA7134_BOARD_MANLI_MTV001 > SAA7134_BOARD_MANLI_MTV002 > SAA7134_BOARD_MD2819 > SAA7134_BOARD_MD5044 > SAA7134_BOARD_MONSTERTV_MOBILE > SAA7134_BOARD_MSI_TVATANYWHERE_PLUS > SAA7134_BOARD_PINNACLE_300I_DVBT_PAL > SAA7134_BOARD_PINNACLE_PCTV_110 > SAA7134_BOARD_PINNACLE_PCTV_310 > SAA7134_BOARD_PROTEUS_2309 > SAA7134_BOARD_REAL_ANGEL_220 > SAA7134_BOARD_ROVERMEDIA_LINK_PRO_FM > SAA7134_BOARD_RTD_VFG7350 > SAA7134_BOARD_SABRENT_SBTTVFM > SAA7134_BOARD_SEDNA_PC_TV_CARDBUS > SAA7134_BOARD_UPMOST_PURPLE_TV > SAA7134_BOARD_VIDEOMATE_DVBT_200 > SAA7134_BOARD_VIDEOMATE_DVBT_200A > SAA7134_BOARD_VIDEOMATE_DVBT_300 > SAA7134_BOARD_VIDEOMATE_GOLD_PLUS > SAA7134_BOARD_VIDEOMATE_S350 > SAA7134_BOARD_VIDEOMATE_TV_GOLD_PLUSII > SAA7134_BOARD_VIDEOMATE_TV_PVR > > The init2 code has 32 boards listed: > > SAA7134_BOARD_ADS_DUO_CARDBUS_PTV331 > SAA7134_BOARD_ADS_INSTANT_HDTV_PCI > SAA7134_BOARD_ASUS_EUROPA2_HYBRID > SAA7134_BOARD_ASUS_EUROPA_HYBRID > SAA7134_BOARD_ASUST > SAA7134_BOARD_AVERMEDIA_CARDBUS_501 > SAA7134_BOARD_AVERMEDIA_SUPER_007 > SAA7134_BOARD_BEHOLD_COLUMBUS_TVFM > SAA7134_BOARD_BMK_MPEX_NOTUNER > SAA7134_BOARD_BMK_MPEX_TUNER > SAA7134_BOARD_CINERGY_HT_PCI > SAA7134_BOARD_CINERGY_HT_PCMCIA > SAA7134_BOARD_CREATIX_CTX953 > SAA7134_BOARD_FLYDVBT_HYBRID_CARDBUS > SAA7134_BOARD_FLYDVB_TRIO > SAA7134_BOARD_HAUPPAUGE_HVR1110 > SAA7134_BOARD_HAUPPAUGE_HVR1120 > SAA7134_BOARD_HAUPPAUGE_HVR1150 > SAA7134_BOARD_KWORLD_ATSC110 > SAA7134_BOARD_KWORLD_DVBT_210 > SAA7134_BOARD_MD7134 > SAA7134_BOARD_MEDION_MD8800_QUADRO > SAA7134_BOARD_PHILIPS_EUROPA > SAA7134_BOARD_PHILIPS_SNAKE > SAA7134_BOARD_PHILIPS_TIGER > SAA7134_BOARD_PHILIPS_TIGER_S > SAA7134_BOARD_PINNACLE_PCTV_310 > SAA7134_BOARD_TEVION_DVBT_220RF > SAA7134_BOARD_TWINHAN_DTV_DVB_3056 > SAA7134_BOARD_VIDEOMATE_DVBT_200 > SAA7134_BOARD_VIDEOMATE_DVBT_200A > SAA7134_BOARD_VIDEOMATE_DVBT_300 > > From all those entries, there are 15 boards that are listed on both init1 and > init2: > > SAA7134_BOARD_ADS_DUO_CARDBUS_PTV331 > SAA7134_BOARD_ASUST > SAA7134_BOARD_AVERMEDIA_CARDBUS_501 > SAA7134_BOARD_BEHOLD_COLUMBUS_TVFM > SA
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Hi Mauro, On Tue, 02 Feb 2010 17:09:05 -0200, Mauro Carvalho Chehab wrote: > > From: Jean Delvare > > 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, by the time we find out, some > > card-specific initialization is missed. In particular, the fact that > > the IR is GPIO-based. Set it when we change the card type, and run > > saa7134_input_init1(). > > > > Signed-off-by: Jean Delvare > > Cc: Daro > > Cc: Roman Kellner > > --- > > linux/drivers/media/video/saa7134/saa7134-cards.c |5 + > > 1 file changed, 5 insertions(+) > > > > --- v4l-dvb.orig/linux/drivers/media/video/saa7134/saa7134-cards.c > > 2010-01-30 10:56:50.0 +0100 > > +++ v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c > > 2010-01-30 11:52:18.0 +0100 > > @@ -7299,6 +7299,11 @@ int saa7134_board_init2(struct saa7134_d > >printk(KERN_INFO "%s: P7131 analog only, using " > >"entry of %s\n", > >dev->name, saa7134_boards[dev->board].name); > > + > > + /* IR init has already happened for other cards, so > > +* we have to catch up. */ > > + dev->has_remote = SAA7134_REMOTE_GPIO; > > + saa7134_input_init1(dev); > >} > >break; > > case SAA7134_BOARD_HAUPPAUGE_HVR1150: > > This version of your patch makes sense to me. > > This logic will only apply for board SAA7134_BOARD_ASUSTeK_P7131_ANALOG, > so, provided that someone with this board test it, I'm OK with it. > > Had Roman or Daro already test it? Not yet, but Daro just volunteered to do so... let's give him/her some time to proceed. -- 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
Hi Daro, On Wed, 10 Feb 2010 17:38:18 +0100, Daro wrote: > If some tests on my machine could be helpfull just let me know. Definitely. If you could please test both patches I sent (first one on 2010-01-27, second one on 2010-01-30, both should be in your mailbox) and confirm that they both work for you (so you no longer need to pass card=146 to the driver), that would be great. Mauro doesn't seem to be willing to apply the first patch, but for future reference I would still be interested to know if it would work at least in your case. The second patch is what Mauro is likely to apply, so 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
W dniu 03.02.2010 00:32, hermann pitton pisze: Hi Jean, Mauro and all, Am Dienstag, den 02.02.2010, 08:54 +0100 schrieb 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 with recent, you can see that gpio 0x4 doesn't go high, but your patch should enable the remote on that P7131 analog only. I'm not sure why you had to fake anything? What I'd like to know is simply if my first patch had any negative effect on other cards. because I simply don't have that Asus My Cinema analog only in question. To recap, you previously announced a patch, tested by Daro, claiming to get the remote up under auto detection for that device and I told you having some doubts on it. My first patch was not actually tested by Daro. What he tested was loading the driver with card=146. At first I thought it was equivalent, but since then I have realized it wasn't. That's the reason why the "Tested-by:" was turned into a mere "Cc:" on my second and third patches. Mauro prefers to have a fix for that single card in need for now. Since nobody else cares, "For now", see above, I can confirm that your last patch for that single device should work to get IR up with auto detection in delay after we change the card such late with eeprom detection. The meaning of that byte in use here is unknown to me, we should avoid such as much we can! It can turn out to be only some pseudo service. If your call for testers on your previous attempt, really reaches some for some reason, I'm with you, but for now I have to keep the car operable within all such snow. That I understand. What I don't understand is: if you have a SAA7134-based card, why don't you test my second patch (the one moving the call to saa7134_input_init1 to saa7134_hwinit2) on it, without faking anything? This would be a first, useful data point. sorry, the snow fall did not stop and we will need trucks next day to get it out of town. No place left. I did not reread any single line of code until now, but told you that Roman has tested a equivalent patch on his P7131_ANALOG already and I can confirm that it also had no side effects on a FlyVideo3000 card=2. For now, I would at least need some time to see, if input_init can be decoupled from all other hardware init, what you seem to suggest, and looking closer to Mauro's concerns. Thought you are asking for some test with a i2c remote next to confirm your analysis there. No such card in any machine currently, but can be done. Cheers, Hermann Hi All, If some tests on my machine could be helpfull just let me know. Best regards Darek -- 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
Hi Jean, Mauro and all, Am Dienstag, den 02.02.2010, 08:54 +0100 schrieb 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 > > > > with recent, you can see that gpio 0x4 doesn't go high, but your > > > > patch should enable the remote on that P7131 analog only. > > > > > > I'm not sure why you had to fake anything? What I'd like to know is > > > simply if my first patch had any negative effect on other cards. > > > > because I simply don't have that Asus My Cinema analog only in question. > > > > To recap, you previously announced a patch, tested by Daro, claiming to > > get the remote up under auto detection for that device and I told you > > having some doubts on it. > > My first patch was not actually tested by Daro. What he tested was > loading the driver with card=146. At first I thought it was equivalent, > but since then I have realized it wasn't. That's the reason why the > "Tested-by:" was turned into a mere "Cc:" on my second and third > patches. > > > Mauro prefers to have a fix for that single card in need for now. > > > > Since nobody else cares, "For now", see above, I can confirm that your > > last patch for that single device should work to get IR up with auto > > detection in delay after we change the card such late with eeprom > > detection. > > > > The meaning of that byte in use here is unknown to me, we should avoid > > such as much we can! It can turn out to be only some pseudo service. > > > > If your call for testers on your previous attempt, really reaches some > > for some reason, I'm with you, but for now I have to keep the car > > operable within all such snow. > > That I understand. What I don't understand is: if you have a > SAA7134-based card, why don't you test my second patch (the one moving > the call to saa7134_input_init1 to saa7134_hwinit2) on it, without > faking anything? This would be a first, useful data point. > sorry, the snow fall did not stop and we will need trucks next day to get it out of town. No place left. I did not reread any single line of code until now, but told you that Roman has tested a equivalent patch on his P7131_ANALOG already and I can confirm that it also had no side effects on a FlyVideo3000 card=2. For now, I would at least need some time to see, if input_init can be decoupled from all other hardware init, what you seem to suggest, and looking closer to Mauro's concerns. Thought you are asking for some test with a i2c remote next to confirm your analysis there. No such card in any machine currently, but can be done. Cheers, Hermann -- 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
Hi Jean, > From: Jean Delvare > 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, by the time we find out, some > card-specific initialization is missed. In particular, the fact that > the IR is GPIO-based. Set it when we change the card type, and run > saa7134_input_init1(). > > Signed-off-by: Jean Delvare > Cc: Daro > Cc: Roman Kellner > --- > linux/drivers/media/video/saa7134/saa7134-cards.c |5 + > 1 file changed, 5 insertions(+) > > --- v4l-dvb.orig/linux/drivers/media/video/saa7134/saa7134-cards.c > 2010-01-30 10:56:50.0 +0100 > +++ v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c 2010-01-30 > 11:52:18.0 +0100 > @@ -7299,6 +7299,11 @@ int saa7134_board_init2(struct saa7134_d > printk(KERN_INFO "%s: P7131 analog only, using " > "entry of %s\n", > dev->name, saa7134_boards[dev->board].name); > + > + /* IR init has already happened for other cards, so > + * we have to catch up. */ > + dev->has_remote = SAA7134_REMOTE_GPIO; > + saa7134_input_init1(dev); > } > break; > case SAA7134_BOARD_HAUPPAUGE_HVR1150: This version of your patch makes sense to me. This logic will only apply for board SAA7134_BOARD_ASUSTeK_P7131_ANALOG, so, provided that someone with this board test it, I'm OK with it. Had Roman or Daro already test it? -- Cheers, Mauro -- 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
Jean Delvare wrote: > 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 with recent, you can see that gpio 0x4 doesn't go high, but your patch should enable the remote on that P7131 analog only. >>> I'm not sure why you had to fake anything? What I'd like to know is >>> simply if my first patch had any negative effect on other cards. >> because I simply don't have that Asus My Cinema analog only in question. >> >> To recap, you previously announced a patch, tested by Daro, claiming to >> get the remote up under auto detection for that device and I told you >> having some doubts on it. > > My first patch was not actually tested by Daro. What he tested was > loading the driver with card=146. At first I thought it was equivalent, > but since then I have realized it wasn't. That's the reason why the > "Tested-by:" was turned into a mere "Cc:" on my second and third > patches. > >> Mauro prefers to have a fix for that single card in need for now. >> >> Since nobody else cares, "For now", see above, I can confirm that your >> last patch for that single device should work to get IR up with auto >> detection in delay after we change the card such late with eeprom >> detection. >> >> The meaning of that byte in use here is unknown to me, we should avoid >> such as much we can! It can turn out to be only some pseudo service. >> >> If your call for testers on your previous attempt, really reaches some >> for some reason, I'm with you, but for now I have to keep the car >> operable within all such snow. > > That I understand. What I don't understand is: if you have a > SAA7134-based card, why don't you test my second patch (the one moving > the call to saa7134_input_init1 to saa7134_hwinit2) on it, without > faking anything? This would be a first, useful data point. The init1 code has 107 boards listed: SAA7134_BOARD_10MOONSTVMASTER3 SAA7134_BOARD_ADS_DUO_CARDBUS_PTV331 SAA7134_BOARD_ASUST SAA7134_BOARD_AVACSSMARTTV SAA7134_BOARD_AVERMEDIA_305 SAA7134_BOARD_AVERMEDIA_307 SAA7134_BOARD_AVERMEDIA_777 SAA7134_BOARD_AVERMEDIA_A169_B SAA7134_BOARD_AVERMEDIA_A16AR SAA7134_BOARD_AVERMEDIA_A16D SAA7134_BOARD_AVERMEDIA_A700_HYBRID SAA7134_BOARD_AVERMEDIA_A700_PRO SAA7134_BOARD_AVERMEDIA_CARDBUS SAA7134_BOARD_AVERMEDIA_CARDBUS_501 SAA7134_BOARD_AVERMEDIA_CARDBUS_506 SAA7134_BOARD_AVERMEDIA_GO_007_FM SAA7134_BOARD_AVERMEDIA_GO_007_FM_PLUS SAA7134_BOARD_AVERMEDIA_M102 SAA7134_BOARD_AVERMEDIA_M103 SAA7134_BOARD_AVERMEDIA_M115 SAA7134_BOARD_AVERMEDIA_M135A SAA7134_BOARD_AVERMEDIA_STUDIO_305 SAA7134_BOARD_AVERMEDIA_STUDIO_307 SAA7134_BOARD_AVERMEDIA_STUDIO_505 SAA7134_BOARD_AVERMEDIA_STUDIO_507 SAA7134_BOARD_BEHOLD_401 SAA7134_BOARD_BEHOLD_403 SAA7134_BOARD_BEHOLD_403FM SAA7134_BOARD_BEHOLD_405 SAA7134_BOARD_BEHOLD_405FM SAA7134_BOARD_BEHOLD_407 SAA7134_BOARD_BEHOLD_407FM SAA7134_BOARD_BEHOLD_409 SAA7134_BOARD_BEHOLD_409FM SAA7134_BOARD_BEHOLD_505FM SAA7134_BOARD_BEHOLD_505RDS_MK3 SAA7134_BOARD_BEHOLD_505RDS_MK5 SAA7134_BOARD_BEHOLD_507_9FM SAA7134_BOARD_BEHOLD_507RDS_MK3 SAA7134_BOARD_BEHOLD_507RDS_MK5 SAA7134_BOARD_BEHOLD_607FM_MK3 SAA7134_BOARD_BEHOLD_607FM_MK5 SAA7134_BOARD_BEHOLD_607RDS_MK3 SAA7134_BOARD_BEHOLD_607RDS_MK5 SAA7134_BOARD_BEHOLD_609FM_MK3 SAA7134_BOARD_BEHOLD_609FM_MK5 SAA7134_BOARD_BEHOLD_609RDS_MK3 SAA7134_BOARD_BEHOLD_609RDS_MK5 SAA7134_BOARD_BEHOLD_COLUMBUS_TVFM SAA7134_BOARD_BEHOLD_H6 SAA7134_BOARD_BEHOLD_M6 SAA7134_BOARD_BEHOLD_M63 SAA7134_BOARD_BEHOLD_M6_EXTRA SAA7134_BOARD_BEHOLD_X7 SAA7134_BOARD_CINERGY400 SAA7134_BOARD_CINERGY400_CARDBUS SAA7134_BOARD_CINERGY600 SAA7134_BOARD_CINERGY600_MK3 SAA7134_BOARD_ECS_TVP3XP SAA7134_BOARD_ECS_TVP3XP_4CB5 SAA7134_BOARD_ECS_TVP3XP_4CB6 SAA7134_BOARD_ENCORE_ENLTV SAA7134_BOARD_ENCORE_ENLTV_FM SAA7134_BOARD_ENCORE_ENLTV_FM53 SAA7134_BOARD_FLYDVBS_LR300 SAA7134_BOARD_FLYDVBTDUO SAA7134_BOARD_FLYDVBT_DUO_CARDBUS SAA7134_BOARD_FLYDVBT_HYBRID_CARDBUS SAA7134_BOARD_FLYDVBT_LR301 SAA7134_BOARD_FLYTVPLATINUM_FM SAA7134_BOARD_FLYTVPLATINUM_MINI2 SAA7134_BOARD_FLYVIDEO2000 SAA7134_BOARD_FLYVIDEO3000 SAA7134_BOARD_FLYVIDEO3000_NTSC SAA7134_BOARD_GENIUS_TVGO_A11MCE SAA7134_BOARD_GOTVIEW_7135 SAA7134_BOARD_HAUPPAUGE_HVR1110 SAA7134_BOARD_HAUPPAUGE_HVR1120 SAA7134_BOARD_HAUPPAUGE_HVR1150 SAA7134_BOARD_KWORLD_PLUS_TV_ANALOG SAA7134_BOARD_KWORLD_TERMINATOR SAA7134_BOARD_KWORLD_VSTREAM_XPERT SAA7134_BOARD_KWORLD_XPERT SAA7134_BOARD_LEADTEK_WINFAST_DTV1000S SAA7134_BOARD_MANLI_MTV001 SAA7134_BOARD_MANLI_MTV002 SAA7134_BOARD_MD2819 SAA7134_BOARD_MD5044 SAA7134_BOARD_MONSTERTV_MOBILE SAA7134_BOARD_MSI_TVATANYWHERE_PLUS SAA7134_BOARD_PINNACLE_300I_DVBT_PAL SAA7134_BOARD_PINNACLE_PCTV_110 SAA7134_BOARD_PINNACLE_PCTV_310 SAA7134_BOARD_PROTEUS_2309 SAA7134_BOARD_REAL_ANGEL_220 SAA7134_BOARD_ROVERMEDIA_LINK_PRO_FM
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
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 > > > with recent, you can see that gpio 0x4 doesn't go high, but your > > > patch should enable the remote on that P7131 analog only. > > > > I'm not sure why you had to fake anything? What I'd like to know is > > simply if my first patch had any negative effect on other cards. > > because I simply don't have that Asus My Cinema analog only in question. > > To recap, you previously announced a patch, tested by Daro, claiming to > get the remote up under auto detection for that device and I told you > having some doubts on it. My first patch was not actually tested by Daro. What he tested was loading the driver with card=146. At first I thought it was equivalent, but since then I have realized it wasn't. That's the reason why the "Tested-by:" was turned into a mere "Cc:" on my second and third patches. > Mauro prefers to have a fix for that single card in need for now. > > Since nobody else cares, "For now", see above, I can confirm that your > last patch for that single device should work to get IR up with auto > detection in delay after we change the card such late with eeprom > detection. > > The meaning of that byte in use here is unknown to me, we should avoid > such as much we can! It can turn out to be only some pseudo service. > > If your call for testers on your previous attempt, really reaches some > for some reason, I'm with you, but for now I have to keep the car > operable within all such snow. That I understand. What I don't understand is: if you have a SAA7134-based card, why don't you test my second patch (the one moving the call to saa7134_input_init1 to saa7134_hwinit2) on it, without faking anything? This would be a first, useful data point. 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
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 > > with recent, you can see that gpio 0x4 doesn't go high, but your > > patch should enable the remote on that P7131 analog only. > > I'm not sure why you had to fake anything? What I'd like to know is > simply if my first patch had any negative effect on other cards. because I simply don't have that Asus My Cinema analog only in question. To recap, you previously announced a patch, tested by Daro, claiming to get the remote up under auto detection for that device and I told you having some doubts on it. Mauro prefers to have a fix for that single card in need for now. Since nobody else cares, "For now", see above, I can confirm that your last patch for that single device should work to get IR up with auto detection in delay after we change the card such late with eeprom detection. The meaning of that byte in use here is unknown to me, we should avoid such as much we can! It can turn out to be only some pseudo service. If your call for testers on your previous attempt, really reaches some for some reason, I'm with you, but for now I have to keep the car operable within all such snow. Cheers, Hermann -- 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
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 > with recent, you can see that gpio 0x4 doesn't go high, but your > patch should enable the remote on that P7131 analog only. I'm not sure why you 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/majordomo-info.html
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Hi all interested, Am Samstag, den 30.01.2010, 11:56 +0100 schrieb 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 > > > > 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, by the time we find out, some > > > > card-specific initialization is missed. In particular, the fact that > > > > the IR is GPIO-based. Set it when we change the card type. > > > > > > > > We also have to move the initialization of IR until after the card > > > > number has been changed. I hope that this won't cause any problem. > > > > > > Hi Jean, > > > > > > Moving the initialization will likely cause regressions. The reason why > > > there > > > are two init codes there were due to the way the old i2c code used to > > > work. > > > This got fixed after the i2c rework, but it caused regressions on that > > > time. > > I don't think there is any problem with having two init sequences. You > need the EEPROM to identify some devices, you need I2C support to > access the EEPROM, and you need some initialization to get I2C > operational. > > This doesn't mean that some adjustments to the exact sequence aren't > possible. In particular, I don't see what else can depend on IR being > initialized, so I can't really see what harm can be done in moving IR > init code _later_ in the sequence. Looking at saa7134_input_init1(), I > see that it is essentially setting up software parameters in the > saa7134_dev structure, there is almost no hardware access. Only for a > few cards, we change a couple bits in registers GPIO_GPMODE* and > GPIO_GPSTATUS*. I honestly can't see how doing this _later_ in the init > sequence could be a problem. > > > > The proper way would be to just muve the IR initialization on this board > > > from init1 to init2, instead of changing it for all other devices. > > Hmm, OK. I think it's wrong, but I'm not the one to decide. Patch below. > > > Mauro, I do agree with you that it is likely better to go a way with > > minimum chances for regressions, also given the current testing base and > > that only this single card is involved.. > > I admit I am very surprised that we apparently can't get anyone to test > changes to a driver that supports 176 different models of TV cards :( > > > Do we end up with something card specific in core code here? > > After all, we know this is a no go. > > > > Hartmut and me thought back and forth on how to deal with it for quite > > some while, unfortunately Hartmut is not present currently on the list, > > but he voted for to have a separate entry for that card finally too. > > > > What we seem to have now is: > > > > 1. We don't know, if Jean's patch really would cause regressions, > >but it is likely hard to get all the testing done. No problems with a > >FlyVideo3000 gpio remote at the time Roman suggested it, but I had > >not any i2c remote that time ... > > I doubt it matters, given that saa7134_input_init1() only cares about > GPIO-based IR: > > int saa7134_input_init1(struct saa7134_dev *dev) > { > (...) > if (dev->has_remote != SAA7134_REMOTE_GPIO) > return -ENODEV; > > So the moving the call to this function should have no effect on boards > with I2C-based IR. > > > (...) > > Given what is also in the cruft for bttv, I would not care too much for > > that single card on that now also ancient driver, just print what the > > user can do to escape and any google would find it quickly too. For Asus > > it is a unique problem on that driver so far. > > This isn't how we're going to make Linux popular. > > > I should have some time on Sunday afternoon for testing, if we should go > > that way. > > Any testing you can provide is very welcome, thanks. > > * * * * * > > From: Jean Delvare > 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, by the time we find out, some > card-specific initialization is missed. In particular, the fact that > the IR is GPIO-based. Set it when we change the card type, and run > saa7134_input_init1(). > > Signed-off-by: Jean Delvare > Cc: Daro > Cc: Roman Kellner > --- > linux/drivers/media/video/saa7134/saa7134-cards.c |5 + > 1 file changed, 5 insertions(+) > > --- v4l-dvb.orig/linux/drivers/media/video/saa7134/saa7134-cards.c > 2010-01-30 10:56:50.0 +0100 > +++ v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c 2010-01-30 > 11:52:18.0 +0100 > @@ -7299,6 +7299,11 @@ int saa7134_board_init2(struct saa7134_d > printk(KERN_INFO "%s: P7131 analog only, using " >
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
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 > > > 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, by the time we find out, some > > > card-specific initialization is missed. In particular, the fact that > > > the IR is GPIO-based. Set it when we change the card type. > > > > > > We also have to move the initialization of IR until after the card > > > number has been changed. I hope that this won't cause any problem. > > > > Hi Jean, > > > > Moving the initialization will likely cause regressions. The reason why > > there > > are two init codes there were due to the way the old i2c code used to work. > > This got fixed after the i2c rework, but it caused regressions on that time. I don't think there is any problem with having two init sequences. You need the EEPROM to identify some devices, you need I2C support to access the EEPROM, and you need some initialization to get I2C operational. This doesn't mean that some adjustments to the exact sequence aren't possible. In particular, I don't see what else can depend on IR being initialized, so I can't really see what harm can be done in moving IR init code _later_ in the sequence. Looking at saa7134_input_init1(), I see that it is essentially setting up software parameters in the saa7134_dev structure, there is almost no hardware access. Only for a few cards, we change a couple bits in registers GPIO_GPMODE* and GPIO_GPSTATUS*. I honestly can't see how doing this _later_ in the init sequence could be a problem. > > The proper way would be to just muve the IR initialization on this board > > from init1 to init2, instead of changing it for all other devices. Hmm, OK. I think it's wrong, but I'm not the one to decide. Patch below. > Mauro, I do agree with you that it is likely better to go a way with > minimum chances for regressions, also given the current testing base and > that only this single card is involved.. I admit I am very surprised that we apparently can't get anyone to test changes to a driver that supports 176 different models of TV cards :( > Do we end up with something card specific in core code here? > After all, we know this is a no go. > > Hartmut and me thought back and forth on how to deal with it for quite > some while, unfortunately Hartmut is not present currently on the list, > but he voted for to have a separate entry for that card finally too. > > What we seem to have now is: > > 1. We don't know, if Jean's patch really would cause regressions, >but it is likely hard to get all the testing done. No problems with a >FlyVideo3000 gpio remote at the time Roman suggested it, but I had >not any i2c remote that time ... I doubt it matters, given that saa7134_input_init1() only cares about GPIO-based IR: int saa7134_input_init1(struct saa7134_dev *dev) { (...) if (dev->has_remote != SAA7134_REMOTE_GPIO) return -ENODEV; So the moving the call to this function should have no effect on boards with I2C-based IR. > (...) > Given what is also in the cruft for bttv, I would not care too much for > that single card on that now also ancient driver, just print what the > user can do to escape and any google would find it quickly too. For Asus > it is a unique problem on that driver so far. This isn't how we're going to make Linux popular. > I should have some time on Sunday afternoon for testing, if we should go > that way. Any testing you can provide is very welcome, thanks. * * * * * From: Jean Delvare 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, by the time we find out, some card-specific initialization is missed. In particular, the fact that the IR is GPIO-based. Set it when we change the card type, and run saa7134_input_init1(). Signed-off-by: Jean Delvare Cc: Daro Cc: Roman Kellner --- linux/drivers/media/video/saa7134/saa7134-cards.c |5 + 1 file changed, 5 insertions(+) --- v4l-dvb.orig/linux/drivers/media/video/saa7134/saa7134-cards.c 2010-01-30 10:56:50.0 +0100 +++ v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c 2010-01-30 11:52:18.0 +0100 @@ -7299,6 +7299,11 @@ int saa7134_board_init2(struct saa7134_d printk(KERN_INFO "%s: P7131 analog only, using " "entry of %s\n", dev->name, saa7134_boards[dev->board].name); + + /* IR init has already happened for other cards, so +* we have to catch up. */ + dev->has_remote = SAA7134_REMOTE_GPIO; + sa
Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants
Hi, Am Freitag, den 29.01.2010, 13:40 -0200 schrieb Mauro Carvalho Chehab: > Jean Delvare wrote: > > From: Jean Delvare > > 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, by the time we find out, some > > card-specific initialization is missed. In particular, the fact that > > the IR is GPIO-based. Set it when we change the card type. > > > > We also have to move the initialization of IR until after the card > > number has been changed. I hope that this won't cause any problem. > > Hi Jean, > > Moving the initialization will likely cause regressions. The reason why there > are two init codes there were due to the way the old i2c code used to work. > This got fixed after the i2c rework, but it caused regressions on that time. > > The proper way would be to just muve the IR initialization on this board > from init1 to init2, instead of changing it for all other devices. > > cheers, > Mauro Mauro, I do agree with you that it is likely better to go a way with minimum chances for regressions, also given the current testing base and that only this single card is involved.. Do we end up with something card specific in core code here? After all, we know this is a no go. Hartmut and me thought back and forth on how to deal with it for quite some while, unfortunately Hartmut is not present currently on the list, but he voted for to have a separate entry for that card finally too. What we seem to have now is: 1. We don't know, if Jean's patch really would cause regressions, but it is likely hard to get all the testing done. No problems with a FlyVideo3000 gpio remote at the time Roman suggested it, but I had not any i2c remote that time ... 2. The previous situation was, that this analog only card did use the entry of the Ausus P7131 Dual forcing card=78 without problems, but getting fake support for DVB-T announced and printing results of some fall through. 3. If we agree, that unique PCI subsystems should have highest priority for auto detection, in such a case, we likely could also set the PC-39 remote for the older card with USB remote. IIRC, that should survive the later change of the card caused by the work around eeprom detection later, and disable IR based on eeprom for the older then. To be honest, as pointed to already in the other thread around this, we should not try to become better than all others on m$ previously for some very small gain. We are already much, much better than drivers there, excluding each others, don't follow Philips/NXP eeprom rules and so on. We could just print something like use card=number to get the remote up too, if people, not reading the lists ;), hope to rely on auto detection in vain ... About all the LNA and IR mess we have for other manufactures, nobody talks about anymore ... Given what is also in the cruft for bttv, I would not care too much for that single card on that now also ancient driver, just print what the user can do to escape and any google would find it quickly too. For Asus it is a unique problem on that driver so far. I should have some time on Sunday afternoon for testing, if we should go that way. Cheers, Hermann -- 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
Jean Delvare wrote: > From: Jean Delvare > 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, by the time we find out, some > card-specific initialization is missed. In particular, the fact that > the IR is GPIO-based. Set it when we change the card type. > > We also have to move the initialization of IR until after the card > number has been changed. I hope that this won't cause any problem. Hi Jean, Moving the initialization will likely cause regressions. The reason why there are two init codes there were due to the way the old i2c code used to work. This got fixed after the i2c rework, but it caused regressions on that time. The proper way would be to just muve the IR initialization on this board from init1 to init2, instead of changing it for all other devices. cheers, Mauro -- 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