From: Takashi Iwai <tiwai@suse.de>
To: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Cc: alsa-devel@alsa-project.org
Subject: Re: [alsa-lib][PATCH v2 0/2] ctl: deprecate APIs of dimension information
Date: Mon, 13 Nov 2017 16:58:47 +0100 [thread overview]
Message-ID: <s5h1sl2b26g.wl-tiwai@suse.de> (raw)
In-Reply-To: <20171113001411.7430-1-o-takashi@sakamocchi.jp>
On Mon, 13 Nov 2017 01:14:09 +0100,
Takashi Sakamoto wrote:
>
> Hi,
>
> This patchset is to update my previous one:
> [alsa-devel] [alsa-lib][PATCH 0/2] ctl: deprecate APIs of dimension information
> http://mailman.alsa-project.org/pipermail/alsa-devel/2017-November/126972.html
>
> In ALSA control interface of asound.h, 'struct snd_ctl_elem_info' has
> 'dimen' member to deliver information for multi-dimensional array, however
> there's no common way to handle the member. As a result, drivers can
> force userspace applications to handle the information by inconsistent
> ways. A series of echoaudio drivers is an actual example of the
> inconsistent usage.
>
> This issue was addressed in a commit 51db452df07b ('Revert "ALSA: echoaudio:
> purge contradictions between dimension matrix members and total number of
> members"') to Linux kernel[1]. Fortunately, at present, this feature of
> interface is just used by the drivers, and there's a way to obsolete usage
> of the feature. We can obsolete it without large impacts to userland.
>
> As a result of discussion at Linux miniconference 2017, usage of 'dimen'
> member of 'struct snd_ctl_elem_info' is going to be deprecated for future
> removal. This patchset is for proposed tasks at a development period of
> Linux kernel v4.15[3]. Some APIs of dimension information are deprecated.
>
> In a short discussion in my previous patchset, it's clear that actual
> removal of kernel interface is uncertain depending on several conditions.
> Thus the timing to remove the library APIs is not decided yet. This updated
> version reflects the issue in API documentation (but simply removes
> statement of the schedule...).
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/sound/pci/echoaudio?id=51db452df07bb4c5754b73789253ba21681d9dc2
>
> [2] [alsa-devel] [ANNOUNCE] Audio Mini Summit 2017 at Prague, Oct 27
> http://mailman.alsa-project.org/pipermail/alsa-devel/2017-July/123110.html
>
> [3] https://github.com/takaswie/presentations/blob/master/20171027/contents.md
>
> Takashi Sakamoto (2):
> ctl: deprecate APIs of dimensional information
> test: obsolete usage of APIs of dimensional information
Applied both patches now (with your sign-off).
Thanks!
Takashi
prev parent reply other threads:[~2017-11-13 15:58 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-13 0:14 [alsa-lib][PATCH v2 0/2] ctl: deprecate APIs of dimension information Takashi Sakamoto
2017-11-13 0:14 ` [alsa-lib][PATCH v2 1/2] ctl: deprecate APIs of dimensional information Takashi Sakamoto
2017-11-13 0:14 ` [alsa-lib][PATCH v2 2/2] test: obsolete usage of " Takashi Sakamoto
2017-11-13 0:18 ` [alsa-lib][PATCH v2 0/2] ctl: deprecate APIs of dimension information Takashi Sakamoto
2017-11-13 15:58 ` Takashi Iwai [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=s5h1sl2b26g.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=o-takashi@sakamocchi.jp \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox