From: Michael Krufky <mkrufky@linuxtv.org>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: video4linux-list@redhat.com, linux-dvb@linuxtv.org, fragabr@gmail.com
Subject: Re: [linux-dvb] [EXPERIMENTAL] cx88+xc3028 - tests are required - was: Re: Wh en xc3028/xc2028 will be supported?
Date: Fri, 22 Feb 2008 09:13:44 -0500 [thread overview]
Message-ID: <47BED898.4070902@linuxtv.org> (raw)
In-Reply-To: <47BED1E4.1010806@linuxtv.org>
Michael Krufky wrote:
> Mauro Carvalho Chehab wrote:
>
>> On Thu, 21 Feb 2008 20:38:09 -0500
>> Michael Krufky <mkrufky@linuxtv.org> wrote:
>>
>>
>>
>>> Mauro Carvalho Chehab wrote:
>>>
>>>
>>>>>> It is not that simple. Steven patch works for DTV on PCI Nano; Christopher
>>>>>> patches for some other DiVCO boards (DTV also); my port of Markus patch
>>>>>>
>>>>>>
>>>>>>
>>>>> for
>>>>>
>>>>>
>>>>>
>>>>>> other boards (tested by Dâniel Fraga - Analog TV).
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>> What does one board have to do with another? Just because these boards
>>>>> all use xceive tuners does not mean that they should be grouped together.
>>>>>
>>>>>
>>>>>
>>>> No, but we have patches for all of them.
>>>>
>>>>
>>>>
>>>>
>>>>> Because the user has the ability to load cx8800 without cx88-dvb, and
>>>>> likewise, the user has the ability to load cx88-dvb without cx8800, the
>>>>> attach call must be fully qualified such that the other side of the
>>>>> driver is not required to be loaded in order for the tuner to work.
>>>>>
>>>>>
>>>>>
>>>> If you take a look at the code, you'll see that all code is at cx88xx. This
>>>> module is loaded by cx8800, if you're using analog, or by cx8802, if you're
>>>> using cx88-dvb or cx88-blackbird.
>>>>
>>>> The code initializes xc3028 before tuner attach.
>>>>
>>>>
>>>>
>>>>
>>>>> In the case of FusionHDTV5 pci nano, there will never be an attach call
>>>>> from the analog side of the driver, since the tuner is hidden behind the
>>>>> s5h1409's i2c gate, and analog mode is not supported with the current
>>>>> driver. If you set i2c_scan=1 on the PCI nano, the xceive tuner will
>>>>> not even show up in the scan.
>>>>>
>>>>>
>>>>>
>>>> The proper fix is to open the i2c gate before loading tuner. This will allow
>>>> i2c_scan to work and both analog and digital modes will work. Btw, this
>>>> somewhat similar to what dvico_fusionhdtv_hybrid_init() does on cx88-cards.
>>>>
>>>> I've added a patch that should open the bridge for s5h1409. Please test.
>>>>
>>>>
>>> Mauro,
>>>
>>> It does not work. See my prior email in this thread for the explanation.
>>>
>>> [ 2912.355967] Linux video capture interface: v2.00
>>> [ 2912.373470] cx88/0: cx2388x v4l2 driver version 0.0.6 loaded
>>> [ 2912.373536] ACPI: PCI Interrupt 0000:02:07.0[A] -> GSI 19 (level,
>>> low) -> IRQ 17
>>> [ 2912.373601] cx88[0]: subsystem: 18ac:d530, board: DVICO HDTV5 PCI
>>> Nano [card=59,autodetected]
>>> [ 2912.373607] cx88[0]: TV tuner type 71, Radio tuner type -1
>>> [ 2912.555088] cx88[0]: Asking xc2028/3028 to load firmware xc3028-v27.fw
>>>
>>>
>> ^^^^
>> The above message were generated by cx88-cards. So, as I said before, this were
>> called _before_ running dvb_register, defined at cx88-dvb.
>>
>> The issue is simple: the i2c gate code were wrong. So, xc3028 is not visible on
>> that point of the code. With this, analog support will never work. The proper
>> fix is to enable xc3028 before enabling i2c, meaning to correct the code at
>> cx88_pci_nano_init().
>>
>> After re-visiting s5h1409 code, I noticed that I've used the wrong sequence for
>> the gate. The state should be i2c closed, for xc3028 to work. The following
>> patch should fix:
>>
>> http://linuxtv.org/hg/~mchehab/cx88-xc2028/rev/871db4e0451c
>>
>> Please test it, with i2c_scan=1, and post the results.
>>
>>
>
> Doesn't work.
>
> [ 795.056020] Linux video capture interface: v2.00
> [ 798.235564] cx88/0: cx2388x v4l2 driver version 0.0.6 loaded
> [ 798.235643] ACPI: PCI Interrupt 0000:02:07.0[A] -> GSI 19 (level,
> low) -> IRQ 18
> [ 798.235718] cx88[0]: subsystem: 18ac:d530, board: DVICO HDTV5 PCI
> Nano [card=59,autodetected]
> [ 798.235722] cx88[0]: TV tuner type 71, Radio tuner type -1
> [ 798.371224] cx88[0]: i2c scan: found device @ 0x32 [???]
> [ 798.371391] cx88[0]: i2c scan: found device @ 0x34 [???]
> [ 798.399887] cx88[0]: i2c scan: found device @ 0xa0 [eeprom]
> [ 798.400319] cx88[0]: i2c scan: found device @ 0xa2 [???]
> [ 798.400482] cx88[0]: i2c scan: found device @ 0xa4 [???]
> [ 798.400642] cx88[0]: i2c scan: found device @ 0xa6 [???]
> [ 798.400800] cx88[0]: i2c scan: found device @ 0xa8 [???]
> [ 798.400957] cx88[0]: i2c scan: found device @ 0xaa [???]
> [ 798.401116] cx88[0]: i2c scan: found device @ 0xac [???]
> [ 798.401272] cx88[0]: i2c scan: found device @ 0xae [???]
> [ 798.411996] cx88[0]: i2c scan: found device @ 0xd6 [???]
> [ 798.488821] Closing s5h1409 i2c gate to allow xc3028 detection
> [ 798.489283] cx88[0]: Asking xc2028/3028 to load firmware xc3028-v27.fw
> [ 798.489301] cx88[0]/0: found at 0000:02:07.0, rev: 5, irq: 18,
> latency: 64, mmio: 0xe4000000
> [ 798.489376] cx88[0]/0: registered device video0 [v4l2]
> [ 798.489408] cx88[0]/0: registered device vbi0
> [ 818.236239] cx88/2: cx2388x MPEG-TS Driver Manager version 0.0.6 loaded
> [ 818.236443] cx88[0]/2: cx2388x 8802 Driver Manager
> [ 818.236465] ACPI: PCI Interrupt 0000:02:07.2[A] -> GSI 19 (level,
> low) -> IRQ 18
> [ 818.236480] cx88[0]/2: found at 0000:02:07.2, rev: 5, irq: 18,
> latency: 64, mmio: 0xe6000000
> [ 818.254906] cx88/2: cx2388x dvb driver version 0.0.6 loaded
> [ 818.254913] cx88/2: registering cx8802 driver, type: dvb access: shared
> [ 818.254918] cx88[0]/2: subsystem: 18ac:d530, board: DVICO HDTV5 PCI
> Nano [card=59]
> [ 818.254922] cx88[0]/2: cx2388x based DVB/ATSC card
> [ 818.309763] xc2028 1-0061: type set to XCeive xc2028/xc3028 tuner
> [ 818.309768] cx88[0]/2: xc3028 attached
> [ 818.310968] DVB: registering new adapter (cx88[0])
> [ 818.311080] DVB: registering frontend 0 (Samsung S5H1409 QAM/8VSB
> Frontend)...
> [ 826.542292] xc2028 1-0061: xc2028/3028 firmware name not set!
> [ 827.643657] xc2028 1-0061: xc2028/3028 firmware name not set!
> [ 828.745093] xc2028 1-0061: xc2028/3028 firmware name not set!
> [ 829.846543] xc2028 1-0061: xc2028/3028 firmware name not set!
> [ 830.844337] xc2028 1-0061: Error on line 1068: -121
>
> -Mike
>
>
BTW, there was a build error -- I had to change your,
warn_printk("Closing s5h1409 i2c gate to allow xc3028 detection\n");
to:
warn_printk("Closing s5h1409 i2c gate to allow xc3028 detection\n");
-Mike
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
next prev parent reply other threads:[~2008-02-22 14:14 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20080127173132.551401d9@tux.abusar.org.br>
[not found] ` <20080128165403.1f7137e0@gaivota>
[not found] ` <20080128182634.345bd4e8@tux.abusar.org.br>
[not found] ` <20080128184534.7af7a41b@gaivota>
[not found] ` <20080128192230.59921445@tux.abusar.org.br>
[not found] ` <20080129004104.17e20224@gaivota>
[not found] ` <20080129021904.1d3047d1@tux.abusar.org.br>
[not found] ` <20080129025020.60fa33de@gaivota>
[not found] ` <20080129050103.2fae9d61@tux.abusar.org.br>
[not found] ` <20080129122547.63214371@gaivota>
2008-02-16 19:30 ` [EXPERIMENTAL] cx88+xc3028 - tests are required - was: Re: When xc3028/xc2028 will be supported? Daniel Gimpelevich
2008-02-17 15:35 ` Markus Rechberger
2008-02-19 4:44 ` Michael Krufky
2008-02-19 9:51 ` Mauro Carvalho Chehab
2008-02-19 16:38 ` Mauro Carvalho Chehab
2008-02-19 17:24 ` Daniel Gimpelevich
2008-02-19 21:20 ` Mauro Carvalho Chehab
2008-02-19 22:27 ` Dâniel Fraga
2008-02-19 20:43 ` [EXPERIMENTAL] cx88+xc3028 - tests are required - was: Re: Wh en " mkrufky
2008-02-19 21:20 ` Mauro Carvalho Chehab
2008-02-22 1:38 ` Michael Krufky
2008-02-22 12:13 ` Mauro Carvalho Chehab
2008-02-22 13:45 ` [linux-dvb] " Michael Krufky
2008-02-22 14:13 ` Michael Krufky [this message]
2008-02-22 14:47 ` Vanessa Ezekowitz
2008-02-21 13:48 ` [EXPERIMENTAL] cx88+xc3028 - tests are required - was: Re: When " Adam Pribyl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=47BED898.4070902@linuxtv.org \
--to=mkrufky@linuxtv.org \
--cc=fragabr@gmail.com \
--cc=linux-dvb@linuxtv.org \
--cc=mchehab@infradead.org \
--cc=video4linux-list@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.