* [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
@ 2018-11-25 23:12 Colin King
2018-11-26 8:27 ` Takashi Iwai
0 siblings, 1 reply; 8+ messages in thread
From: Colin King @ 2018-11-25 23:12 UTC (permalink / raw)
To: Jaroslav Kysela, Takashi Iwai, alsa-devel; +Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There are spelling mistakes in equalizer name fields, fix them.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
sound/usb/mixer_us16x08.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/sound/usb/mixer_us16x08.c b/sound/usb/mixer_us16x08.c
index 26ed23b18b77..4f594f9552a8 100644
--- a/sound/usb/mixer_us16x08.c
+++ b/sound/usb/mixer_us16x08.c
@@ -1119,7 +1119,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
.control_id = SND_US16X08_ID_EQLOWFREQ,
.type = USB_MIXER_U8,
.num_channels = 16,
- .name = "EQ Low Frequence",
+ .name = "EQ Low Frequency",
},
{ /* EQ mid low gain */
.kcontrol_new = &snd_us16x08_eq_gain_ctl,
@@ -1133,7 +1133,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
.control_id = SND_US16X08_ID_EQLOWMIDFREQ,
.type = USB_MIXER_U8,
.num_channels = 16,
- .name = "EQ MidLow Frequence",
+ .name = "EQ MidLow Frequency",
},
{ /* EQ mid low Q */
.kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
@@ -1154,7 +1154,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
.control_id = SND_US16X08_ID_EQHIGHMIDFREQ,
.type = USB_MIXER_U8,
.num_channels = 16,
- .name = "EQ MidHigh Frequence",
+ .name = "EQ MidHigh Frequency",
},
{ /* EQ mid high Q */
.kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
@@ -1175,7 +1175,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
.control_id = SND_US16X08_ID_EQHIGHFREQ,
.type = USB_MIXER_U8,
.num_channels = 16,
- .name = "EQ High Frequence",
+ .name = "EQ High Frequency",
},
};
--
2.19.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
2018-11-25 23:12 [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency" Colin King
2018-11-26 8:27 ` Takashi Iwai
@ 2018-11-26 8:27 ` Takashi Iwai
0 siblings, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2018-11-26 8:27 UTC (permalink / raw)
To: Colin King; +Cc: alsa-devel, Jaroslav Kysela, kernel-janitors, linux-kernel
On Mon, 26 Nov 2018 00:12:08 +0100,
Colin King wrote:
>
> From: Colin Ian King <colin.king@canonical.com>
>
> There are spelling mistakes in equalizer name fields, fix them.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
In general, it's not good to rename the control names even if they are
fixes. But these are rather device-specific ones, and they are EQ's,
which aren't too fatal to change, so I took it as-is now.
thanks,
Takashi
> ---
> sound/usb/mixer_us16x08.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/sound/usb/mixer_us16x08.c b/sound/usb/mixer_us16x08.c
> index 26ed23b18b77..4f594f9552a8 100644
> --- a/sound/usb/mixer_us16x08.c
> +++ b/sound/usb/mixer_us16x08.c
> @@ -1119,7 +1119,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQLOWFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ Low Frequence",
> + .name = "EQ Low Frequency",
> },
> { /* EQ mid low gain */
> .kcontrol_new = &snd_us16x08_eq_gain_ctl,
> @@ -1133,7 +1133,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQLOWMIDFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ MidLow Frequence",
> + .name = "EQ MidLow Frequency",
> },
> { /* EQ mid low Q */
> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
> @@ -1154,7 +1154,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQHIGHMIDFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ MidHigh Frequence",
> + .name = "EQ MidHigh Frequency",
> },
> { /* EQ mid high Q */
> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
> @@ -1175,7 +1175,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQHIGHFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ High Frequence",
> + .name = "EQ High Frequency",
> },
> };
>
> --
> 2.19.1
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
@ 2018-11-26 8:27 ` Takashi Iwai
0 siblings, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2018-11-26 8:27 UTC (permalink / raw)
To: Colin King; +Cc: alsa-devel, Jaroslav Kysela, kernel-janitors, linux-kernel
On Mon, 26 Nov 2018 00:12:08 +0100,
Colin King wrote:
>
> From: Colin Ian King <colin.king@canonical.com>
>
> There are spelling mistakes in equalizer name fields, fix them.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
In general, it's not good to rename the control names even if they are
fixes. But these are rather device-specific ones, and they are EQ's,
which aren't too fatal to change, so I took it as-is now.
thanks,
Takashi
> ---
> sound/usb/mixer_us16x08.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/sound/usb/mixer_us16x08.c b/sound/usb/mixer_us16x08.c
> index 26ed23b18b77..4f594f9552a8 100644
> --- a/sound/usb/mixer_us16x08.c
> +++ b/sound/usb/mixer_us16x08.c
> @@ -1119,7 +1119,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQLOWFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ Low Frequence",
> + .name = "EQ Low Frequency",
> },
> { /* EQ mid low gain */
> .kcontrol_new = &snd_us16x08_eq_gain_ctl,
> @@ -1133,7 +1133,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQLOWMIDFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ MidLow Frequence",
> + .name = "EQ MidLow Frequency",
> },
> { /* EQ mid low Q */
> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
> @@ -1154,7 +1154,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQHIGHMIDFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ MidHigh Frequence",
> + .name = "EQ MidHigh Frequency",
> },
> { /* EQ mid high Q */
> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
> @@ -1175,7 +1175,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQHIGHFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ High Frequence",
> + .name = "EQ High Frequency",
> },
> };
>
> --
> 2.19.1
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
@ 2018-11-26 8:27 ` Takashi Iwai
0 siblings, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2018-11-26 8:27 UTC (permalink / raw)
To: Colin King; +Cc: alsa-devel, Jaroslav Kysela, kernel-janitors, linux-kernel
On Mon, 26 Nov 2018 00:12:08 +0100,
Colin King wrote:
>
> From: Colin Ian King <colin.king@canonical.com>
>
> There are spelling mistakes in equalizer name fields, fix them.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
In general, it's not good to rename the control names even if they are
fixes. But these are rather device-specific ones, and they are EQ's,
which aren't too fatal to change, so I took it as-is now.
thanks,
Takashi
> ---
> sound/usb/mixer_us16x08.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/sound/usb/mixer_us16x08.c b/sound/usb/mixer_us16x08.c
> index 26ed23b18b77..4f594f9552a8 100644
> --- a/sound/usb/mixer_us16x08.c
> +++ b/sound/usb/mixer_us16x08.c
> @@ -1119,7 +1119,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQLOWFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ Low Frequence",
> + .name = "EQ Low Frequency",
> },
> { /* EQ mid low gain */
> .kcontrol_new = &snd_us16x08_eq_gain_ctl,
> @@ -1133,7 +1133,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQLOWMIDFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ MidLow Frequence",
> + .name = "EQ MidLow Frequency",
> },
> { /* EQ mid low Q */
> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
> @@ -1154,7 +1154,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQHIGHMIDFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ MidHigh Frequence",
> + .name = "EQ MidHigh Frequency",
> },
> { /* EQ mid high Q */
> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
> @@ -1175,7 +1175,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
> .control_id = SND_US16X08_ID_EQHIGHFREQ,
> .type = USB_MIXER_U8,
> .num_channels = 16,
> - .name = "EQ High Frequence",
> + .name = "EQ High Frequency",
> },
> };
>
> --
> 2.19.1
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
2018-11-26 8:27 ` Takashi Iwai
@ 2020-10-02 9:18 ` Jaroslav Kysela
-1 siblings, 0 replies; 8+ messages in thread
From: Jaroslav Kysela @ 2020-10-02 9:18 UTC (permalink / raw)
To: Takashi Iwai, Colin King; +Cc: alsa-devel, kernel-janitors, linux-kernel
Dne 26. 11. 18 v 9:27 Takashi Iwai napsal(a):
> On Mon, 26 Nov 2018 00:12:08 +0100,
> Colin King wrote:
>>
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> There are spelling mistakes in equalizer name fields, fix them.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> In general, it's not good to rename the control names even if they are
> fixes. But these are rather device-specific ones, and they are EQ's,
> which aren't too fatal to change, so I took it as-is now.
It seems that this patch has not found a way to kernel. It is present
in #master at git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git,
but it is not present in linux-next.
Thanks,
Jaroslav
>
>
> thanks,
>
> Takashi
>
>
>> ---
>> sound/usb/mixer_us16x08.c | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/sound/usb/mixer_us16x08.c b/sound/usb/mixer_us16x08.c
>> index 26ed23b18b77..4f594f9552a8 100644
>> --- a/sound/usb/mixer_us16x08.c
>> +++ b/sound/usb/mixer_us16x08.c
>> @@ -1119,7 +1119,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQLOWFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ Low Frequence",
>> + .name = "EQ Low Frequency",
>> },
>> { /* EQ mid low gain */
>> .kcontrol_new = &snd_us16x08_eq_gain_ctl,
>> @@ -1133,7 +1133,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQLOWMIDFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ MidLow Frequence",
>> + .name = "EQ MidLow Frequency",
>> },
>> { /* EQ mid low Q */
>> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
>> @@ -1154,7 +1154,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQHIGHMIDFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ MidHigh Frequence",
>> + .name = "EQ MidHigh Frequency",
>> },
>> { /* EQ mid high Q */
>> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
>> @@ -1175,7 +1175,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQHIGHFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ High Frequence",
>> + .name = "EQ High Frequency",
>> },
>> };
>>
>> --
>> 2.19.1
>>
>>
--
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
@ 2020-10-02 9:18 ` Jaroslav Kysela
0 siblings, 0 replies; 8+ messages in thread
From: Jaroslav Kysela @ 2020-10-02 9:18 UTC (permalink / raw)
To: Takashi Iwai, Colin King; +Cc: alsa-devel, kernel-janitors, linux-kernel
Dne 26. 11. 18 v 9:27 Takashi Iwai napsal(a):
> On Mon, 26 Nov 2018 00:12:08 +0100,
> Colin King wrote:
>>
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> There are spelling mistakes in equalizer name fields, fix them.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> In general, it's not good to rename the control names even if they are
> fixes. But these are rather device-specific ones, and they are EQ's,
> which aren't too fatal to change, so I took it as-is now.
It seems that this patch has not found a way to kernel. It is present
in #master at git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git,
but it is not present in linux-next.
Thanks,
Jaroslav
>
>
> thanks,
>
> Takashi
>
>
>> ---
>> sound/usb/mixer_us16x08.c | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/sound/usb/mixer_us16x08.c b/sound/usb/mixer_us16x08.c
>> index 26ed23b18b77..4f594f9552a8 100644
>> --- a/sound/usb/mixer_us16x08.c
>> +++ b/sound/usb/mixer_us16x08.c
>> @@ -1119,7 +1119,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQLOWFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ Low Frequence",
>> + .name = "EQ Low Frequency",
>> },
>> { /* EQ mid low gain */
>> .kcontrol_new = &snd_us16x08_eq_gain_ctl,
>> @@ -1133,7 +1133,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQLOWMIDFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ MidLow Frequence",
>> + .name = "EQ MidLow Frequency",
>> },
>> { /* EQ mid low Q */
>> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
>> @@ -1154,7 +1154,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQHIGHMIDFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ MidHigh Frequence",
>> + .name = "EQ MidHigh Frequency",
>> },
>> { /* EQ mid high Q */
>> .kcontrol_new = &snd_us16x08_eq_mid_width_ctl,
>> @@ -1175,7 +1175,7 @@ static const struct snd_us16x08_control_params eq_controls[] = {
>> .control_id = SND_US16X08_ID_EQHIGHFREQ,
>> .type = USB_MIXER_U8,
>> .num_channels = 16,
>> - .name = "EQ High Frequence",
>> + .name = "EQ High Frequency",
>> },
>> };
>>
>> --
>> 2.19.1
>>
>>
--
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
2020-10-02 9:18 ` Jaroslav Kysela
@ 2020-10-02 9:26 ` Takashi Iwai
-1 siblings, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2020-10-02 9:26 UTC (permalink / raw)
To: Jaroslav Kysela; +Cc: Colin King, alsa-devel, kernel-janitors, linux-kernel
On Fri, 02 Oct 2020 11:18:46 +0200,
Jaroslav Kysela wrote:
>
> Dne 26. 11. 18 v 9:27 Takashi Iwai napsal(a):
> > On Mon, 26 Nov 2018 00:12:08 +0100,
> > Colin King wrote:
> >>
> >> From: Colin Ian King <colin.king@canonical.com>
> >>
> >> There are spelling mistakes in equalizer name fields, fix them.
> >>
> >> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> >
> > In general, it's not good to rename the control names even if they are
> > fixes. But these are rather device-specific ones, and they are EQ's,
> > which aren't too fatal to change, so I took it as-is now.
>
> It seems that this patch has not found a way to kernel. It is present
> in #master at git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git,
> but it is not present in linux-next.
Indeed, it was applied to a wrong branch. I cherry-picked now.
Thanks!
Takashi
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency"
@ 2020-10-02 9:26 ` Takashi Iwai
0 siblings, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2020-10-02 9:26 UTC (permalink / raw)
To: Jaroslav Kysela; +Cc: Colin King, alsa-devel, kernel-janitors, linux-kernel
On Fri, 02 Oct 2020 11:18:46 +0200,
Jaroslav Kysela wrote:
>
> Dne 26. 11. 18 v 9:27 Takashi Iwai napsal(a):
> > On Mon, 26 Nov 2018 00:12:08 +0100,
> > Colin King wrote:
> >>
> >> From: Colin Ian King <colin.king@canonical.com>
> >>
> >> There are spelling mistakes in equalizer name fields, fix them.
> >>
> >> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> >
> > In general, it's not good to rename the control names even if they are
> > fixes. But these are rather device-specific ones, and they are EQ's,
> > which aren't too fatal to change, so I took it as-is now.
>
> It seems that this patch has not found a way to kernel. It is present
> in #master at git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git,
> but it is not present in linux-next.
Indeed, it was applied to a wrong branch. I cherry-picked now.
Thanks!
Takashi
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-10-02 9:26 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-25 23:12 [PATCH] ALSA: usb-audio: fix spelling mistake "Frequence" -> "Frequency" Colin King
2018-11-26 8:27 ` Takashi Iwai
2018-11-26 8:27 ` Takashi Iwai
2018-11-26 8:27 ` Takashi Iwai
2020-10-02 9:18 ` Jaroslav Kysela
2020-10-02 9:18 ` Jaroslav Kysela
2020-10-02 9:26 ` Takashi Iwai
2020-10-02 9:26 ` Takashi Iwai
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.