All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Oliphant <oliphant@nostatic.org>
To: Lucas <jaffa225man@gmail.com>
Cc: alsa-devel@alsa-project.org,
	Geraldo Nascimento <geraldogabriel@gmail.com>
Subject: Re: Implicit feedback on BOSS GT-1, the saga continues...
Date: Thu, 8 Apr 2021 08:56:45 -0700	[thread overview]
Message-ID: <CAHXb3bc0XSc4g8p-Fd-c80p=o+ZNeP=QAf5tKF=3VvXS1q5+Ew@mail.gmail.com> (raw)
In-Reply-To: <CAOsVg8raD8enyWELM4w17dZCe=1Vy2s+7RLszVko4aRLmj7jDA@mail.gmail.com>

Lucas - great to hear that the endpoint patch fixed your crackles!

I'm curious if you get any different results enabling feedback on the
capture endpoint. I'm also curious about the output from "lsusb -v" for
your device and whether the implicit feedback setup looks like the GT-1.

I'm not exactly sure what it even means to have the capture endpoint set up
with implicit feedback as most of the implicit feedback code seems to deal
only with playback. The bit of code in endpoint.c that I patched out, for
example, only runs on playback endpoints.

I *think* the existing code may effectively just be completely turning off
implicit feedback for these devices. This results in audio playback/capture
that "works", but is susceptible to pops/crackles due to small timing
mismatches.

Mike

On Wed, Apr 7, 2021 at 10:36 PM Lucas <jaffa225man@gmail.com> wrote:

> Well, I just got back from testing Mike's suggestion of setting the Roland
> Boutique D-05 as a playback quirk, disabling its capture quirk, and setting
> that endpoint test section to always be skipped.  Sure, enough, the D-05's
> playback is now perfectly crystal clear without the crackles associated
> with an LP record.
>
> This is just a guess until I recompile again, but my capture from the D-05
> reports a read error, seemingly just as the vanilla mainline kernel does,
> and I think it's because I disabled its capture quirk line.  So, it seems
> to me that the D-05 needs both quirk table entries, whether or not the GT-1
> does.
>

  parent reply	other threads:[~2021-04-08 15:57 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-07 19:54 Implicit feedback on BOSS GT-1, the saga continues Mike Oliphant
2021-04-07 20:08 ` Geraldo Nascimento
2021-04-07 20:16   ` Mike Oliphant
2021-04-07 20:35     ` Geraldo Nascimento
2021-04-08  0:38       ` Mike Oliphant
2021-04-08  2:26         ` Geraldo Nascimento
2021-04-08  2:46           ` Mike Oliphant
2021-04-08  3:08             ` Lucas
2021-04-08  3:25               ` Lucas
     [not found]             ` <CAEsQvcsCJUi8eP_t8MGUKf0zSf7Zmqiof8b0fmk-XhZtUScbmg@mail.gmail.com>
2021-04-08  5:05               ` Geraldo Nascimento
2021-04-08  5:36                 ` Lucas
2021-04-08  6:14                   ` Geraldo Nascimento
2021-04-08  8:40                     ` Lucas
2021-04-08 15:56                   ` Mike Oliphant [this message]
2021-04-08 19:17                 ` Geraldo Nascimento
2021-04-08 19:42                   ` Mike Oliphant
2021-04-08 20:14                     ` Geraldo Nascimento
2021-04-08 20:31                       ` Lucas
2021-04-08 21:05                         ` Geraldo Nascimento
2021-04-08 21:21                           ` Lucas
2021-04-08 20:54                       ` Mike Oliphant
2021-04-08 21:24                         ` Geraldo Nascimento
2021-04-08 22:02                           ` Mike Oliphant
2021-04-09  1:04                             ` Geraldo Nascimento
2021-04-09  1:54                               ` Lucas
2021-04-09  2:40                                 ` Geraldo Nascimento
2021-04-09  3:56                                   ` Lucas
2021-04-09  4:27                                     ` Geraldo Nascimento
2021-04-09  4:55                                       ` Lucas
2021-04-09  6:02                                         ` Geraldo Nascimento
2021-04-09 16:47                                         ` Mike Oliphant
2021-04-09 19:08                                           ` Geraldo Nascimento
2021-04-09 19:09                                           ` Lucas
2021-04-11  7:25                                             ` Lucas
2021-04-11  7:46                                               ` Lucas
2021-04-11 19:46                                               ` Mike Oliphant
2021-04-11 21:11                                                 ` Lucas
2021-04-11 21:33                                               ` Geraldo Nascimento
2021-04-12  5:57                                                 ` Lucas
2021-04-12 23:24                                                   ` Geraldo Nascimento
2021-04-16  1:25                                                     ` Lucas
2021-04-16  1:28                                                       ` Lucas
2021-04-16  7:48                                                         ` Takashi Iwai
2021-04-16 16:19                                                           ` Lucas
2021-04-07 20:18   ` Lucas
2021-04-07 20:23     ` Lucas
2021-04-07 20:24     ` Mike Oliphant
2021-04-07 20:26       ` Mike Oliphant

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='CAHXb3bc0XSc4g8p-Fd-c80p=o+ZNeP=QAf5tKF=3VvXS1q5+Ew@mail.gmail.com' \
    --to=oliphant@nostatic.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=geraldogabriel@gmail.com \
    --cc=jaffa225man@gmail.com \
    /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.