At Thu, 12 May 2011 22:17:56 +0200,
Ondrej Zary wrote:
>
> freq_fixup is a constant, no need to hold it in struct snd_tea575x and set in
> each driver.
>
> Signed-off-by: Ondrej Zary
Thanks, applied all three patches.
Takashi
> --- linux-2.6.39-rc2-/include/sound/tea575x-tuner.h 2011-05-10
> 22:31:40.0 +0200
> +++ linux-2.6.39-rc2/include/sound/tea575x-tuner.h2011-05-12
> 21:00:50.0 +0200
> @@ -26,6 +26,8 @@
> #include
> #include
>
> +#define TEA575X_FMIF 10700
> +
> #define TEA575X_DATA (1 << 0)
> #define TEA575X_CLK (1 << 1)
> #define TEA575X_WREN (1 << 2)
> @@ -46,7 +48,6 @@ struct snd_tea575x {
> bool mute; /* Device is muted? */
> bool stereo;/* receiving stereo */
> bool tuned; /* tuned to a station */
> - unsigned int freq_fixup;/* crystal onboard */
> unsigned int val; /* hw value */
> unsigned long freq; /* frequency */
> unsigned long in_use; /* set if the device is in use */
> --- linux-2.6.39-rc2-/sound/i2c/other/tea575x-tuner.c 2011-05-10
> 22:31:40.0 +0200
> +++ linux-2.6.39-rc2/sound/i2c/other/tea575x-tuner.c 2011-05-12
> 21:00:37.0 +0200
> @@ -141,9 +141,9 @@ static void snd_tea575x_get_freq(struct
> freq /= 10;
> /* crystal fixup */
> if (tea->tea5759)
> - freq += tea->freq_fixup;
> + freq += TEA575X_FMIF;
> else
> - freq -= tea->freq_fixup;
> + freq -= TEA575X_FMIF;
>
> tea->freq = freq * 16; /* from kHz */
> }
> @@ -156,9 +156,9 @@ static void snd_tea575x_set_freq(struct
> freq /= 16; /* to kHz */
> /* crystal fixup */
> if (tea->tea5759)
> - freq -= tea->freq_fixup;
> + freq -= TEA575X_FMIF;
> else
> - freq += tea->freq_fixup;
> + freq += TEA575X_FMIF;
> /* freq /= 12.5 */
> freq *= 10;
> freq /= 125;
> --- linux-2.6.39-rc2-/sound/pci/es1968.c 2011-05-10 22:31:43.0
> +0200
> +++ linux-2.6.39-rc2/sound/pci/es1968.c 2011-05-10 23:47:32.0
> +0200
> @@ -2794,7 +2794,6 @@ static int __devinit snd_es1968_create(s
>
> #ifdef CONFIG_SND_ES1968_RADIO
> chip->tea.card = card;
> - chip->tea.freq_fixup = 10700;
> chip->tea.private_data = chip;
> chip->tea.ops = &snd_es1968_tea_ops;
> if (!snd_tea575x_init(&chip->tea))
> --- linux-2.6.39-rc2-/sound/pci/fm801.c 2011-05-10 23:24:39.0
> +0200
> +++ linux-2.6.39-rc2/sound/pci/fm801.c2011-05-10 23:47:43.0
> +0200
> @@ -1231,7 +1231,6 @@ static int __devinit snd_fm801_create(st
>
> #ifdef TEA575X_RADIO
> chip->tea.card = card;
> - chip->tea.freq_fixup = 10700;
> chip->tea.private_data = chip;
> chip->tea.ops = &snd_fm801_tea_ops;
> if ((tea575x_tuner & TUNER_TYPE_MASK) > 0 &&
>
>
> --
> Ondrej Zary
> ___
> Alsa-devel mailing list
> alsa-de...@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
