All of lore.kernel.org
 help / color / mirror / Atom feed
From: Phil Willoughby <willerz@gmail.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: linux-sound@vger.kernel.org, Jaroslav Kysela <perex@perex.cz>
Subject: Re: [PATCH v4] ALSA: usb-audio: Add quirks for Arturia AF16Rig
Date: Sat, 28 Mar 2026 15:49:28 +0000	[thread overview]
Message-ID: <2787415.vuYhMxLoTh@linux-workstation> (raw)
In-Reply-To: <87wlywum8u.wl-tiwai@suse.de>

On Saturday, 28 March 2026 13:17:21 Greenwich Mean Time you wrote:
> On Sat, 28 Mar 2026 12:08:41 +0100,
> Phil Willoughby wrote:
> > 2. Add the external clock quirks and locally test that it doesn't break
> > anything for people who don't have the external clocks
> > 3. Wrap the external clock quirks in #if 0 and a comment that they are
> > disabled because they haven't been tested.
> > 4. This will be marked signed-off but not tested-by me.
> 
> Hm, a better approach would be rather a creation of a mixer element to
> switch the clock source, I suppose.  But it'll be more complex, of
> course.

The standard mixer code path seems to be creating that element already:

$ amixer -c AF16Rig contents
numid=2,iface=CARD,name='ARTURIA ADAT Clock Validity'
  ; type=BOOLEAN,access=r-------,values=1
  : values=on
numid=1,iface=CARD,name='ARTURIA Internal Clock Validity'
  ; type=BOOLEAN,access=r-------,values=1
  : values=on
numid=3,iface=CARD,name='ARTURIA WORD Clock Validity'
  ; type=BOOLEAN,access=r-------,values=1
  : values=on
numid=4,iface=MIXER,name='ARTURIA Clock Selector Clock Source'
  ; type=ENUMERATED,access=rw------,values=1,items=3
  ; Item #0 'ARTURIA Internal Clock'
  ; Item #1 'ARTURIA ADAT Clock'
  ; Item #2 'ARTURIA WORD Clock'
  : values=0

I also see it in the alsamixer UI. I tried switching it, and it does
communicate that change to the AF16Rig. Of course then it doesn't
work because I have nothing attached as an external clock.



      reply	other threads:[~2026-03-28 15:49 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-28 11:08 [PATCH v4] ALSA: usb-audio: Add quirks for Arturia AF16Rig Phil Willoughby
2026-03-28 13:17 ` Takashi Iwai
2026-03-28 15:49   ` Phil Willoughby [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2787415.vuYhMxLoTh@linux-workstation \
    --to=willerz@gmail.com \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.