Re: Some tvaudio fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun June 10 2012 03:43:49 Daniel Glöckner wrote:
> This patchset is made up of changes I did to the tvaudio driver
> back in 2009. IIRC I started these to get automatic mono/stereo
> swiching working again in mplayer. These changes have been tested
> with a TDA9873H only and most of the time there was stereo. The
> last patch is just a few hours old and has received no testing at
> all.
> 
>   Daniel
>       
>  [PATCH 1/9] tvaudio: fix TDA9873 constants
>  [PATCH 2/9] tvaudio: fix tda8425_setmode
>  [PATCH 3/9] tvaudio: use V4L2_TUNER_MODE_SAP for TDA985x SAP
>  [PATCH 4/9] tvaudio: remove watch_stereo
>  [PATCH 5/9] tvaudio: don't use thread for TA8874Z
>  [PATCH 6/9] tvaudio: use V4L2_TUNER_SUB_* for bitfields
>  [PATCH 7/9] tvaudio: obey V4L2 tuner audio matrix
>  [PATCH 8/9] tvaudio: support V4L2_TUNER_MODE_LANG1_LANG2
>  [PATCH 9/9] tvaudio: don't report mono when stereo is received
> 
>  drivers/media/video/tvaudio.c |  189 +++++++++++++++++++++++------------------
>  1 files changed, 107 insertions(+), 82 deletions(-)
> 

Looks good, but I'd like to see one final change: rename setmode to setaudmode
and getmode to getrxsubchans. That would make the code so much more understandable :-)

It's OK to do that as a final 10th patch.

Good work!

Regards,

	Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux