[alsa-devel] [PATCH] Add mute support to FM-only card with FM801 PCI to tuner bridge
Takashi Iwai
tiwai at suse.de
Thu Jan 24 16:45:39 CET 2008
At Thu, 24 Jan 2008 17:13:42 +0200,
Andy Shevchenko wrote:
>
> Hi!
>
> This is improvement of the early support of the FM-only cards where the
> fm801 chip represents the PCI to tuner bridge.
>
> The tuner initialization isn't included the mute on as well as mute support
> via V4L request. Proposed patch should fix this at least for 64-PCR model.
Instead of adding noop callbacks, it's better to add a NULL check of
tea->ops->mute. That is, something like:
if (tea->ops->mute)
tea->ops->mute(tea, (v.flags & VIDEO_AUDIO_MUTE) != 0);
We don't need to define TEA575X_[UN]MUTE. They are just bool, so use
zero and non-zero.
Last but not least, please fix the coding style issues reported via
$LINUX/scripts/checkpatch.pl.
Thanks,
Takashi
More information about the Alsa-devel
mailing list