* [PATCH 1/3] tea575x: remove freq_fixup from struct
@ 2011-05-12 20:17 Ondrej Zary
2011-05-14 9:47 ` [alsa-devel] " Takashi Iwai
0 siblings, 1 reply; 2+ messages in thread
From: Ondrej Zary @ 2011-05-12 20:17 UTC (permalink / raw)
To: alsa-devel; +Cc: linux-media, Kernel development list
freq_fixup is a constant, no need to hold it in struct snd_tea575x and set in
each driver.
Signed-off-by: Ondrej Zary <linux@rainbow-software.org>
--- linux-2.6.39-rc2-/include/sound/tea575x-tuner.h 2011-05-10 22:31:40.000000000 +0200
+++ linux-2.6.39-rc2/include/sound/tea575x-tuner.h 2011-05-12 21:00:50.000000000 +0200
@@ -26,6 +26,8 @@
#include <media/v4l2-dev.h>
#include <media/v4l2-ioctl.h>
+#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.000000000 +0200
+++ linux-2.6.39-rc2/sound/i2c/other/tea575x-tuner.c 2011-05-12 21:00:37.000000000 +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.000000000 +0200
+++ linux-2.6.39-rc2/sound/pci/es1968.c 2011-05-10 23:47:32.000000000 +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.000000000 +0200
+++ linux-2.6.39-rc2/sound/pci/fm801.c 2011-05-10 23:47:43.000000000 +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
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [alsa-devel] [PATCH 1/3] tea575x: remove freq_fixup from struct
2011-05-12 20:17 [PATCH 1/3] tea575x: remove freq_fixup from struct Ondrej Zary
@ 2011-05-14 9:47 ` Takashi Iwai
0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2011-05-14 9:47 UTC (permalink / raw)
To: Ondrej Zary; +Cc: alsa-devel, Kernel development list, linux-media
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 <linux@rainbow-software.org>
Thanks, applied all three patches.
Takashi
> --- linux-2.6.39-rc2-/include/sound/tea575x-tuner.h 2011-05-10 22:31:40.000000000 +0200
> +++ linux-2.6.39-rc2/include/sound/tea575x-tuner.h 2011-05-12 21:00:50.000000000 +0200
> @@ -26,6 +26,8 @@
> #include <media/v4l2-dev.h>
> #include <media/v4l2-ioctl.h>
>
> +#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.000000000 +0200
> +++ linux-2.6.39-rc2/sound/i2c/other/tea575x-tuner.c 2011-05-12 21:00:37.000000000 +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.000000000 +0200
> +++ linux-2.6.39-rc2/sound/pci/es1968.c 2011-05-10 23:47:32.000000000 +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.000000000 +0200
> +++ linux-2.6.39-rc2/sound/pci/fm801.c 2011-05-10 23:47:43.000000000 +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-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-05-14 9:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-12 20:17 [PATCH 1/3] tea575x: remove freq_fixup from struct Ondrej Zary
2011-05-14 9:47 ` [alsa-devel] " Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox