Re: ATI TV Wonder support

On Nov 22, 2004, at 22:56, Anish Mistry wrote:

> On Monday 22 November 2004 09:29 pm, you wrote:
>> On Nov 22, 2004, at 21:22, Anish Mistry wrote:
>>> On Sunday 21 November 2004 10:00 pm, Justin Hibbits wrote:
>>>> This patch gives more or less full ATI TV Wonder support to the bktr
>>>> driver.
>>>
>>> Awesome.
>>>
>>>> The sound doesn't mute at close, but that might be xawtv's fault, 
>>>> but
>>>> I
>>>> don't know which to accuse or look at.  But, other than that, it
>>>> seems pretty good.
>>>
>>> I've having some trouble using it though, the sound doesn't seem to
>>> work
>>> just as before.  Is there something extra I need to add to my kernel
>>> besides bktr?
>>> On boot:
>>> bktr0: <BrookTree 878> mem 0xe3101000-0xe3101fff irq 11 at device 8.0
>>> on
>>> pci0
>>> bktr0: [GIANT-LOCKED]
>>> bktr0: Detected a MSP3445G-B8 at 0x80
>>> bktr0: ATI TV Wonder, Philips NTSC tuner, msp3400c stereo.
>>> pci0: <multimedia> at device 8.1 (no driver attached)
>>> And when I start it:
>>> bktr0: Detected a MSP3445G-B8 at 0x80
>>>
>>> You should also send-pr this if you haven't already.
>>>
>>> Thanks,
>>>
>>> --
>>> Anish Mistry
>>
>> Oh, you need to build it with the msp3400c driver, so add the line:
>>
>> options BKTR_NEW_MSP34XX_DRIVER
>>
>> to your config file.  The card doesn't have any mux, only that chip, 
>> so
>> you need the driver.  I've found a hackish solution to the sound
>> problem, by resetting the card on exit.  It's horrible, so I won't
>> include it in the patch, but it works for me.  If you want to use this
>> too, add:
>>
>>  if (bktr->card.msp3400c )
>>   msp_dpl_reset( bktr, bktr->msp_addr );
>>
>> to bktr_core.c, at line 1171.
>>
>>
>> I'll resend my previous message as a send-pr later, when I have a
>> little more time to test it.
>>
> Thanks, but I'm getting a weird panic with 5-STABLE, which I'm 
> assuming you
> aren't seeing.  Any ideas?
>
> panic: sleeping without a mutex
> KDB: enter: panic
> [thread 100012]
> Stopped at kbd_enter+0x2c: leave
> db> tr
> kbd_enter(c0635ca1,100,c1b6c54c,c1b41320,0) at kbd_enter+0x2c
> panic(c06363e5,0,0,0,c1b37c00) at panic+0x10a
> msleep(c1b6c54c,0,4c,c064007d,0) at msleep+0x2bf
> msp3410d_thread(c1bd8000,d0181d48,c1db8000,c845e1d4,0) at
> msp3410d_thread+0x5b
> fork_exit(c045e1d4,c1db8000,d0181d48) at fork_exit+0x7e
> fork_trampoline() at fork_trampoline+0x8
> --- trap 0x1, eip=0, esp=0xd0181d7c, ebd=0 ---
>
> dmesg:
> http://am-productions.biz/docs/bigguy.txt.gz
> kernel config:
> http://am-productions.biz/docs/BIGGUY.gz
> -- 
> Anish Mistry

Oh yes, you need to compile it without witness support, or invariants 
(DDB is fine, though).  It, of course, cuts out a ton of useful 
debugging info for other parts, but it's the only way, unfortunately.

- Justin
--
"And now, if you'll excuse me, I'm in the middle of 15 things, all 
annoying" -- Lt. Cmdr Susan Ivanova, Babylon 5