All of lore.kernel.org
 help / color / mirror / Atom feed
* [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
@ 2020-01-25 13:22 Nicola Lunghi
  2020-01-25 14:08 ` Takashi Iwai
  2020-01-25 14:12 ` Jens Verwiebe
  0 siblings, 2 replies; 8+ messages in thread
From: Nicola Lunghi @ 2020-01-25 13:22 UTC (permalink / raw)
  To: alsa-devel; +Cc: Takashi Iwai, Carlo Calica, Nicola Lunghi

Line6 Helix fw 2.82 still needs a quirk to set the audio rate to 48khz but
in respect to older firmware has changed the USB ID of the device.

Add it to the quirk list for line6 helix family of devices.

Signed-off-by: Nicola Lunghi <nick83ola@gmail.com>
---
 sound/usb/format.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sound/usb/format.c b/sound/usb/format.c
index d79db71305f6..e2b98f181d05 100644
--- a/sound/usb/format.c
+++ b/sound/usb/format.c
@@ -296,6 +296,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, 0x4248): /* Line6 Helix >= fw 2.82 */
 		/* supported rates: 48Khz */
 		kfree(fp->rate_table);
 		fp->rate_table = kmalloc(sizeof(int), GFP_KERNEL);
-- 
2.20.1

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 13:22 [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82 Nicola Lunghi
@ 2020-01-25 14:08 ` Takashi Iwai
  2020-01-25 14:12 ` Jens Verwiebe
  1 sibling, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2020-01-25 14:08 UTC (permalink / raw)
  To: Nicola Lunghi; +Cc: alsa-devel, Carlo Calica

On Sat, 25 Jan 2020 14:22:27 +0100,
Nicola Lunghi wrote:
> 
> Line6 Helix fw 2.82 still needs a quirk to set the audio rate to 48khz but
> in respect to older firmware has changed the USB ID of the device.
> 
> Add it to the quirk list for line6 helix family of devices.
> 
> Signed-off-by: Nicola Lunghi <nick83ola@gmail.com>

Applied now.  Thanks.


Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 13:22 [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82 Nicola Lunghi
  2020-01-25 14:08 ` Takashi Iwai
@ 2020-01-25 14:12 ` Jens Verwiebe
  2020-01-25 14:16   ` Takashi Iwai
  1 sibling, 1 reply; 8+ messages in thread
From: Jens Verwiebe @ 2020-01-25 14:12 UTC (permalink / raw)
  To: alsa-devel

Hi

Why this incomplete patch ???

I longer informed you that with fw >= 2.8 lot of helix kind devices got 
a new ID:

So this patch should include at least the well known/collected data:

case USB_ID(0x0E41, 0x424a): /* Line6 Helix LT - firmware >= 2.8 */


Jens

-- 

Jens Verwiebe
Allerskehre 44 - 22309 Hamburg

Tel.: +49 40 68 78 50
mailto: info@jensverwiebe.de
web: https://www.jensverwiebe.de

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 14:12 ` Jens Verwiebe
@ 2020-01-25 14:16   ` Takashi Iwai
  2020-01-25 14:34     ` nick83ola
                       ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Takashi Iwai @ 2020-01-25 14:16 UTC (permalink / raw)
  To: Jens Verwiebe; +Cc: alsa-devel, Carlo Calica, Nicola Lunghi

On Sat, 25 Jan 2020 15:12:48 +0100,
Jens Verwiebe wrote:
> 
> Hi
> 
> Why this incomplete patch ???
> 
> I longer informed you that with fw >= 2.8 lot of helix kind devices
> got a new ID:
> 
> So this patch should include at least the well known/collected data:
> 
> case USB_ID(0x0E41, 0x424a): /* Line6 Helix LT - firmware >= 2.8 */

Oh, so the patch doesn't work as is?
Then I drop it again.

If anyone can resubmit the comprehensive one and give the tested-by
tag, I'll happily apply it.


thanks,

Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 14:16   ` Takashi Iwai
@ 2020-01-25 14:34     ` nick83ola
  2020-01-25 14:46       ` Jens Verwiebe
  2020-01-25 14:35     ` Alexander Tsoy
  2020-01-25 14:38     ` Jens Verwiebe
  2 siblings, 1 reply; 8+ messages in thread
From: nick83ola @ 2020-01-25 14:34 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: alsa-devel, Carlo Calica, Jens Verwiebe

Hi Jens

On Sat, 25 Jan 2020 at 14:16, Takashi Iwai <tiwai@suse.de> wrote:
>
> On Sat, 25 Jan 2020 15:12:48 +0100,
> Jens Verwiebe wrote:
> >
> > Hi
> >
> > Why this incomplete patch ???
> >
> > I longer informed you that with fw >= 2.8 lot of helix kind devices
> > got a new ID:

Yes you told me but you didn't give me a list of the ID nor confirm
that you can test something.
The reason of this patch is that I created a post in the Line6 forum
when you told me that and the only user that
confirmed that the patch work with newer firmware with Helix is Carlo Calica.

I sadly have only access to an hx stomp and I don't want to put
patches that I cannot verify.

If you have access to other hardware instead of sending email to me
you should submit something yourself.

Cheers
Nicola Lunghi

> >
> > So this patch should include at least the well known/collected data:
> >
> > case USB_ID(0x0E41, 0x424a): /* Line6 Helix LT - firmware >= 2.8 */
>
> Oh, so the patch doesn't work as is?
> Then I drop it again.
>
> If anyone can resubmit the comprehensive one and give the tested-by
> tag, I'll happily apply it.
>
>
> thanks,
>
> Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 14:16   ` Takashi Iwai
  2020-01-25 14:34     ` nick83ola
@ 2020-01-25 14:35     ` Alexander Tsoy
  2020-01-25 14:38     ` Jens Verwiebe
  2 siblings, 0 replies; 8+ messages in thread
From: Alexander Tsoy @ 2020-01-25 14:35 UTC (permalink / raw)
  To: Takashi Iwai, Jens Verwiebe; +Cc: alsa-devel, Carlo Calica, Nicola Lunghi

В Сб, 25/01/2020 в 15:16 +0100, Takashi Iwai пишет:
> On Sat, 25 Jan 2020 15:12:48 +0100,
> Jens Verwiebe wrote:
> > Hi
> > 
> > Why this incomplete patch ???
> > 
> > I longer informed you that with fw >= 2.8 lot of helix kind devices
> > got a new ID:
> > 
> > So this patch should include at least the well known/collected
> > data:
> > 
> > case USB_ID(0x0E41, 0x424a): /* Line6 Helix LT - firmware >= 2.8 */
> 
> Oh, so the patch doesn't work as is?
> Then I drop it again.
> 

No, Jens complained that the quirk should also be applied to other
devices from the Helix line that recieved new firmware. Not sure what's
the problem with submitting them in a follow-up patch.

> If anyone can resubmit the comprehensive one and give the tested-by
> tag, I'll happily apply it.


_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 14:16   ` Takashi Iwai
  2020-01-25 14:34     ` nick83ola
  2020-01-25 14:35     ` Alexander Tsoy
@ 2020-01-25 14:38     ` Jens Verwiebe
  2 siblings, 0 replies; 8+ messages in thread
From: Jens Verwiebe @ 2020-01-25 14:38 UTC (permalink / raw)
  To: alsa-devel

Kinda

Nicola made a request in the line6 forums and got at least helix lt 
confirmed.

As he took over my original patch + a fallback i want leave this to him to

maintain all new ( core ) ID's now.

Also i'd like to mention again that this "hardclocking" only makes helix 
devices

setup initially but does not prevent the sync shifting away after some time.

I was hoping the solution Mike Oliphant found for the Boss device would 
also fix the Helix sync,

but there is a bit more todo i fear. ( implicit feedback not working )

Still on that ....


Jens

-- 

Jens Verwiebe
Allerskehre 44 - 22309 Hamburg

Tel.: +49 40 68 78 50
mailto: info@jensverwiebe.de
web: https://www.jensverwiebe.de

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82
  2020-01-25 14:34     ` nick83ola
@ 2020-01-25 14:46       ` Jens Verwiebe
  0 siblings, 0 replies; 8+ messages in thread
From: Jens Verwiebe @ 2020-01-25 14:46 UTC (permalink / raw)
  To: alsa-devel

Shall i show you the line6 forum entry where ppl confirmed ?

https://line6.com/support/topic/52539-need-linux-users-with-helix-helix-lt-helix-rack-helix-something/

Also my much older thread has info i gave you before:

https://line6.com/support/topic/4426-line-6-with-linux/#comments

So please don't miss that. The response from users is poor enough.


Jens

-- 

Jens Verwiebe
Allerskehre 44 - 22309 Hamburg

Tel.: +49 40 68 78 50
mailto: info@jensverwiebe.de
web: https://www.jensverwiebe.de

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-01-25 14:47 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-25 13:22 [alsa-devel] [PATCH] ALSA: usb-audio: add quirks for Line6 Helix fw 2.82 Nicola Lunghi
2020-01-25 14:08 ` Takashi Iwai
2020-01-25 14:12 ` Jens Verwiebe
2020-01-25 14:16   ` Takashi Iwai
2020-01-25 14:34     ` nick83ola
2020-01-25 14:46       ` Jens Verwiebe
2020-01-25 14:35     ` Alexander Tsoy
2020-01-25 14:38     ` Jens Verwiebe

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.