Skip to content

Commit

Permalink
[media] au0828: make sure video standard is setup in tuner-core
Browse files Browse the repository at this point in the history
If the user performs a tuning attempt without explicitly calling the
s_std ioctl(), a value of zero is sent from tuner-core to xc5000.  This
causes the xc5000 driver to leave the standard unchanged.  The problem
was masked by the fact that the xc5000 driver defaulted to NTSC, but if
you happened to perform an ATSC/ClearQAM tuning attempt and then do an
analog tune, the net effect is an analog tune with the standard still
set to DTV6.

Keep track of whether the standard has ever been sent to tuner-core.  We
don't make an s_std subdev call explicitly during probe because that
will cause a firmware load (which is very time consuming on the 950q).
With the logic in this patch, the s_std call will occur automatically on
the s_freq call if it hasn't already been set.

Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Devin Heitmueller authored and Mauro Carvalho Chehab committed Aug 9, 2012
1 parent 549ee4d commit c98bc03
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
10 changes: 10 additions & 0 deletions drivers/media/video/au0828/au0828-video.c
Original file line number Diff line number Diff line change
Expand Up @@ -1330,6 +1330,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id * norm)
buffer, which is currently hardcoded at 720x480 */

v4l2_device_call_all(&dev->v4l2_dev, 0, core, s_std, *norm);
dev->std_set_in_tuner_core = 1;
return 0;
}

Expand Down Expand Up @@ -1540,6 +1541,15 @@ static int vidioc_s_frequency(struct file *file, void *priv,

dev->ctrl_freq = freq->frequency;

if (dev->std_set_in_tuner_core == 0) {
/* If we've never sent the standard in tuner core, do so now. We
don't do this at device probe because we don't want to incur
the cost of a firmware load */
v4l2_device_call_all(&dev->v4l2_dev, 0, core, s_std,
dev->vdev->tvnorms);
dev->std_set_in_tuner_core = 1;
}

v4l2_device_call_all(&dev->v4l2_dev, 0, tuner, s_frequency, freq);

au0828_analog_stream_reset(dev);
Expand Down
1 change: 1 addition & 0 deletions drivers/media/video/au0828/au0828.h
Original file line number Diff line number Diff line change
Expand Up @@ -225,6 +225,7 @@ struct au0828_dev {
unsigned int frame_count;
int ctrl_freq;
int input_type;
int std_set_in_tuner_core;
unsigned int ctrl_input;
enum au0828_dev_state dev_state;
enum au0828_stream_state stream_state;
Expand Down

0 comments on commit c98bc03

Please sign in to comment.