[pvrusb2] xawtv fix

[Andreas Beck]

Mike Isely <isely at isely.net> wrote:
> On Wed, 12 Apr 2006, Barry Jett wrote:
> > Mike,
> > That did it !   Xawtv is working now on my 24xxx hardware !

I can confirm this as well. The change in the xawtv sources makes xawtv
work for me.

> And I see now that your message confirms that xawtv wasn't working on new 
> hardware either with that (required) cx23416 command change.  

Right.

> explain why xawtv isn't liking it.  Second, I'm going to try passing '0' 
> as that magic second argument to that previously mentioned cx23416 command 
> and see if that works.  

It does for me. I changed 

ret |= pvr2_write_encoder_vcmd(hdw, CX2341X_ENC_SET_OUTPUT_PORT, 2,
								0x01, 0x02);

to

ret |= pvr2_write_encoder_vcmd(hdw, CX2341X_ENC_SET_OUTPUT_PORT, 2,
								0x01, 0x00);

around line 296 of driver/pvrusb2-encoder.c

and it still works.

I don't know, if this is the change, but that is what I figured from
your post.

I have not checked, however, if it would now work with an unpatched
xawtv.


Moreover, I can confirm audio working for me now, when using the TV
tuner.

However, it still does not work, if I use the composite inputs.

Can you give me a pointer on where to look for stuff like switching 
inputs on the audio encoder chip?

I assume there is a change in the wiring or similar for the new
hardware. When I switch to the composite inputs, audio gets muted
(i.e. there is an audio stream in the output, but it is all silent).


> Using zero there would make the behavior compatible with what is the 
> ivtv-sourced cx23416 document.  

Works for me on my 24xxx based HW.


CU, Andy

-- 
= Andreas Beck                    |  Email :  <becka-sig at bedatec.de>         =