On 26.09.09 21:33, Devin Heitmueller wrote: > On Sat, Sep 26, 2009 at 8:23 PM, Uros Vampl <mobile.leecher@xxxxxxxxx> wrote: > > On 26.09.09 16:59, Devin Heitmueller wrote: > >> On Fri, Sep 25, 2009 at 6:10 PM, Uros Vampl <mobile.leecher@xxxxxxxxx> wrote: > >> > Alright, success!!! > >> > > >> > Since it seems everything for this tuner is set up the same as for the > >> > Hauppauge WinTV HVR 900, I figured let's set things up *exactly* the > >> > same. So, like it's there for the Hauppauge, I added .mts_firmware = 1 > >> > to the definition of the hybrid XS em2882. And well, working TV audio!! > >> > > >> > > >> > dmesg output this time: > >> > > >> > xc2028 4-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000. > >> > MTS (4), id 00000000000000ff: > >> > xc2028 4-0061: Loading firmware for type=MTS (4), id 0000000100000007. > >> > > >> > > >> > So now with the attached patch, everything (analog, digital, remote) > >> > works! > >> > > >> > Regards, > >> > Uroš > >> > > >> > >> Hello Uros, > >> > >> Please test out the following tree, which has all the relevant fixes > >> (enabling dvb, your audio fix, proper gpio setting, etc). > >> > >> http://kernellabs.com/hg/~dheitmueller/misc-fixes2/ > >> > >> If you have any trouble, please let me know. Otherwise I would like > >> to issue a PULL request for this tree. > > > > > > Hi, > > > > Your tree does not work, no audio. I quickly found the problem though: > > gpio is set to default_analog, but it needs to be set to > > hauppauge_wintv_hvr_900_analog. So I guess treating the EM2880 and > > EM2882 as the same will not work, because they require different gpio > > settings. > > > > Regards, > > Uroš > > Hmm.. Interesting. That does make me wonder whether the GPIOs are > setup for audio properly on the em2880 version of the profile, or > whether the user in question just never tested it. I'll have to go > back and check the USB trace. > > Nonetheless, I'll just check in your version of the patch, and scrap > my version entirely for now. Could you please add your SOB to the > patch? > > Thanks, > > Devin Ok, here we go... Make analog audio, dvb and the remote work on a Terratec Cinergy Hybrid XS (em2882). Signed-off-by: Uroš Vampl <mobile.leecher@xxxxxxxxx> diff -r 29e4ba1a09bc linux/drivers/media/video/em28xx/em28xx-cards.c --- a/linux/drivers/media/video/em28xx/em28xx-cards.c Sat Sep 19 09:45:22 2009 -0300 +++ b/linux/drivers/media/video/em28xx/em28xx-cards.c Sat Sep 26 00:06:37 2009 +0200 @@ -1441,11 +1441,12 @@ .valid = EM28XX_BOARD_NOT_VALIDATED, .tuner_type = TUNER_XC2028, .tuner_gpio = default_tuner_gpio, + .mts_firmware = 1, .decoder = EM28XX_TVP5150, -#if 0 /* FIXME: add an entry at em28xx-dvb */ .has_dvb = 1, .dvb_gpio = hauppauge_wintv_hvr_900_digital, -#endif + .ir_codes = &ir_codes_terratec_cinergy_xs_table, + .xclk = EM28XX_XCLK_FREQUENCY_12MHZ, .input = { { .type = EM28XX_VMUX_TELEVISION, .vmux = TVP5150_COMPOSITE0, @@ -2119,6 +2120,7 @@ switch (dev->model) { case EM2880_BOARD_EMPIRE_DUAL_TV: case EM2880_BOARD_HAUPPAUGE_WINTV_HVR_900: + case EM2882_BOARD_TERRATEC_HYBRID_XS: ctl->demod = XC3028_FE_ZARLINK456; break; case EM2880_BOARD_TERRATEC_HYBRID_XS: diff -r 29e4ba1a09bc linux/drivers/media/video/em28xx/em28xx-dvb.c --- a/linux/drivers/media/video/em28xx/em28xx-dvb.c Sat Sep 19 09:45:22 2009 -0300 +++ b/linux/drivers/media/video/em28xx/em28xx-dvb.c Sat Sep 26 00:06:37 2009 +0200 @@ -494,6 +494,7 @@ } break; case EM2880_BOARD_HAUPPAUGE_WINTV_HVR_900: + case EM2882_BOARD_TERRATEC_HYBRID_XS: case EM2880_BOARD_EMPIRE_DUAL_TV: dvb->frontend = dvb_attach(zl10353_attach, &em28xx_zl10353_xc3028_no_i2c_gate, -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html