[pvrusb2] AV400 continued

[Sven Barth]

Hello Mike!

With Christmas holidays approaching I decided to play around with the 
Grabster AV400, again. I'm currently using your latest snapshot from 
November, a v4l-dvb from around November and a 2.6.31 kernel.

The device itself works well, but the audio is still an issue. The 
attached patch for v4l's cx25840 solves this for me, but of course this 
can't be the correct solution.

The chip on the AV400 is recognized as an cx25837. Now the question is 
whether all such chips don't support a specific audio route or it's only 
"mine".

If this can't be answered we should ask the v4l guys to add a settable 
workaround flag like they did for the PVR170 (search for "workaround" in 
cx25840-core.{h,c}) and modify the two IF statements I commented in my 
patch accordingly.

Regards,
Sven

-------------- next part --------------
--- ../v4l-src/linux/drivers/media/video/cx25840/cx25840-core.c	2009-12-13 16:50:37.119374534 +0100
+++ linux/drivers/media/video/cx25840/cx25840-core.c	2009-12-20 20:33:28.873264595 +0100
@@ -851,10 +851,10 @@
 
 	state->vid_input = vid_input;
 	state->aud_input = aud_input;
-	if (!is_cx2583x(state)) {
+/*	if (!is_cx2583x(state)) {*/
 		cx25840_audio_set_path(client);
 		input_change(client);
-	}
+//	}
 
 	if (is_cx2388x(state)) {
 		/* Audio channel 1 src : Parallel 1 */
@@ -1477,8 +1477,8 @@
 	struct cx25840_state *state = to_state(sd);
 	struct i2c_client *client = v4l2_get_subdevdata(sd);
 
-	if (is_cx2583x(state))
-		return -EINVAL;
+/*	if (is_cx2583x(state))
+		return -EINVAL;*/
 	return set_input(client, state->vid_input, input);
 }