All of lore.kernel.org
 help / color / mirror / Atom feed
From: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org, Jaroslav Kysela <perex@perex.cz>
Subject: Re: [PATCH v2 0/8] ALSA: emu10k1: add support for high-bitrate modes of E-MU cards
Date: Mon, 17 Jul 2023 12:19:49 +0200	[thread overview]
Message-ID: <ZLUVxVUHtUJvcuZQ@ugly> (raw)
In-Reply-To: <87mt02wvsa.wl-tiwai@suse.de>

On Tue, Jul 11, 2023 at 01:08:05PM +0200, Takashi Iwai wrote:
>On Tue, 11 Jul 2023 12:15:31 +0200,
>Oswald Buddenhagen wrote:
>> 
>> On Mon, Jul 10, 2023 at 05:06:36PM +0200, Takashi Iwai wrote:
>> > I still can't agree with the basic design using the dynamic kctl
>> > addition / deletion in kcontrol's put action.
>> > 
>> you are not being constructive. please provide specific, actionable
>> responses to _all_ challenges/questions.
>
>The fundamental idea to add / delete the kctls from the put callback
>is unacceptable; as repeated many times, this is known to break
>existing applications.  As long as you are sticking with this idea, it
>can go [no] further.  Please avoid it and use the (more or less) static
>configuration instead.
>
to put the implications of this in clear words:
you want me to spend additional time
on a driver barely anyone still cares about
to actively degrade the (my!) user experience
to avoid hypothetical / likely obsolete crashes
that would happen upon a rare user-controlled event
in unspecified buggy (mixer? (!)) applications,
while a known-good fallback exists (alsamixer).

i fail to see how that is even _remotely_ a reasonable request, and 
therefore have no intention whatsoever to follow it.

regards

  reply	other threads:[~2023-07-17 10:21 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-30 14:45 [PATCH v2 0/8] ALSA: emu10k1: add support for high-bitrate modes of E-MU cards Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 1/8] ALSA: emu10k1: introduce alternative E-MU D.A.S. mode Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 2/8] ALSA: emu10k1: improve mixer control naming in " Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 3/8] ALSA: emu10k1: set the "no filtering" bits on PCM voices Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 4/8] ALSA: emu10k1: make playback in E-MU D.A.S. mode 32-bit Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 5/8] ALSA: add snd_ctl_add_locked() Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 6/8] ALSA: emu10k1: add support for 2x/4x word clocks in E-MU D.A.S. mode Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 7/8] ALSA: emu10k1: add high-rate capture " Oswald Buddenhagen
2023-06-30 14:45 ` [PATCH v2 8/8] ALSA: emu10k1: add high-rate playback " Oswald Buddenhagen
2023-07-10 15:06 ` [PATCH v2 0/8] ALSA: emu10k1: add support for high-bitrate modes of E-MU cards Takashi Iwai
2023-07-11 10:15   ` Oswald Buddenhagen
2023-07-11 11:08     ` Takashi Iwai
2023-07-17 10:19       ` Oswald Buddenhagen [this message]
2023-07-17 12:53         ` Takashi Iwai
2023-07-17 15:32           ` Oswald Buddenhagen

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=ZLUVxVUHtUJvcuZQ@ugly \
    --to=oswald.buddenhagen@gmx.de \
    --cc=alsa-devel@alsa-project.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.