2021-10-29 18:36:30

by Jason Ormes

Subject: [PATCH] HX-Stomp XL USB_ID.

Signed-off-by: Jason Ormes <[email protected]>
---
sound/usb/format.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/sound/usb/format.c b/sound/usb/format.c
index 50efccbffb8a..e8fc2c350a9f 100644
--- a/sound/usb/format.c
+++ b/sound/usb/format.c
@@ -414,6 +414,7 @@ static int line6_parse_audio_format_rates_quirk(struct snd_usb_audio *chip,
case USB_ID(0x0e41, 0x4242): /* Line6 Helix Rack */
case USB_ID(0x0e41, 0x4244): /* Line6 Helix LT */
case USB_ID(0x0e41, 0x4246): /* Line6 HX-Stomp */
+ case USB_ID(0x0e41, 0x4253): /* Line6 HX-Stomp XL*/
case USB_ID(0x0e41, 0x4247): /* Line6 Pod Go */
case USB_ID(0x0e41, 0x4248): /* Line6 Helix >= fw 2.82 */
case USB_ID(0x0e41, 0x4249): /* Line6 Helix Rack >= fw 2.82 */
--
2.25.1

2021-10-30 09:23:39

by Takashi Iwai

Subject: Re: [PATCH] HX-Stomp XL USB_ID.

On Fri, 29 Oct 2021 20:31:05 +0200,
Jason Ormes wrote:
>
> Signed-off-by: Jason Ormes <[email protected]>

Could you give a bit more description?
e.g. does the device work fully compatibly?


thanks,

Takashi

> ---
> sound/usb/format.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/usb/format.c b/sound/usb/format.c
> index 50efccbffb8a..e8fc2c350a9f 100644
> --- a/sound/usb/format.c
> +++ b/sound/usb/format.c
> @@ -414,6 +414,7 @@ static int line6_parse_audio_format_rates_quirk(struct snd_usb_audio *chip,
> case USB_ID(0x0e41, 0x4242): /* Line6 Helix Rack */
> case USB_ID(0x0e41, 0x4244): /* Line6 Helix LT */
> case USB_ID(0x0e41, 0x4246): /* Line6 HX-Stomp */
> + case USB_ID(0x0e41, 0x4253): /* Line6 HX-Stomp XL*/
> case USB_ID(0x0e41, 0x4247): /* Line6 Pod Go */
> case USB_ID(0x0e41, 0x4248): /* Line6 Helix >= fw 2.82 */
> case USB_ID(0x0e41, 0x4249): /* Line6 Helix Rack >= fw 2.82 */
> --
> 2.25.1
>

2021-10-30 15:05:20

by Takashi Iwai

Subject: Re: [PATCH] HX-Stomp XL USB_ID.

On Sat, 30 Oct 2021 16:32:36 +0200,
Jason Ormes wrote:
>
> It appears to, it's basically just the HX Stomp that they have added a few
> more buttons to.  The USB ID has changed so it doesn't get the fixed rate that
> it needs.  After I added it, I've spent several hours recording through it
> without any issues.  Even the midi functionality works.  

So give such useful information in the patch description and
resubmit. At best, align the Subject line to the other patches,
i.e. with the prefix "ALSA: usb-audio: .....", and put the maintainer
(me) to Cc in addition to alsa-devel ML.


thanks,

Takashi

>
> Jason Ormes
>
> On Sat, Oct 30, 2021, 4:22 AM Takashi Iwai <[email protected]> wrote:
>
> On Fri, 29 Oct 2021 20:31:05 +0200,
> Jason Ormes wrote:
> >
> > Signed-off-by: Jason Ormes <[email protected]>
>
> Could you give a bit more description?
> e.g. does the device work fully compatibly?
>
> thanks,
>
> Takashi
>
> > ---
> >  sound/usb/format.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/sound/usb/format.c b/sound/usb/format.c
> > index 50efccbffb8a..e8fc2c350a9f 100644
> > --- a/sound/usb/format.c
> > +++ b/sound/usb/format.c
> > @@ -414,6 +414,7 @@ static int line6_parse_audio_format_rates_quirk
> (struct snd_usb_audio *chip,
> >       case USB_ID(0x0e41, 0x4242): /* Line6 Helix Rack */
> >       case USB_ID(0x0e41, 0x4244): /* Line6 Helix LT */
> >       case USB_ID(0x0e41, 0x4246): /* Line6 HX-Stomp */
> > +        case USB_ID(0x0e41, 0x4253): /* Line6 HX-Stomp XL*/
> >       case USB_ID(0x0e41, 0x4247): /* Line6 Pod Go */
> >       case USB_ID(0x0e41, 0x4248): /* Line6 Helix >= fw 2.82 */
> >       case USB_ID(0x0e41, 0x4249): /* Line6 Helix Rack >= fw 2.82 */
> > --
> > 2.25.1
> >
>
>