From: Takashi Iwai <tiwai@suse.de>
To: Jaroslav Kysela <perex@perex.cz>
Cc: Takashi Iwai <tiwai@suse.de>, linux-sound@vger.kernel.org
Subject: Re: [PATCH 1/8] ALSA: rawmidi: Expose the tied device number in info ioctl
Date: Tue, 14 Jan 2025 11:37:52 +0100 [thread overview]
Message-ID: <87seplznnj.wl-tiwai@suse.de> (raw)
In-Reply-To: <c44312ea-8995-4027-9616-846e73959eee@perex.cz>
On Tue, 14 Jan 2025 11:25:05 +0100,
Jaroslav Kysela wrote:
>
> On 10. 01. 25 16:59, Takashi Iwai wrote:
> > The UMP legacy rawmidi is derived from the UMP rawmidi, but currently
> > there is no way to know which device is involved in other side.
> >
> > This patch extends the rawmidi info ioctl to show the tied device
> > number. As default it stores -1, indicating that no tied device.
>
> Seeing the alsa-lib changes, it may be probably better to keep zero as
> unknown and define this field as (device number + 1) to avoid bad
> interpretation for older kernels without protocol version checks.
It might be safer, yes, although it'd need an extra interpretation.
But maybe the safety should win.
OK, I'll submit the patch to update it before 6.14.
thanks,
Takashi
next prev parent reply other threads:[~2025-01-14 10:37 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-10 15:59 [PATCH 0/8] ALSA: Small extensions for UMP Takashi Iwai
2025-01-10 15:59 ` [PATCH 1/8] ALSA: rawmidi: Expose the tied device number in info ioctl Takashi Iwai
2025-01-14 10:25 ` Jaroslav Kysela
2025-01-14 10:37 ` Takashi Iwai [this message]
2025-01-10 15:59 ` [PATCH 2/8] ALSA: rawmidi: Show substream activity " Takashi Iwai
2025-01-10 15:59 ` [PATCH 3/8] ALSA: rawmidi: Bump protocol version to 2.0.5 Takashi Iwai
2025-01-10 15:59 ` [PATCH 4/8] ALSA: ump: Copy FB name string more safely Takashi Iwai
2025-01-10 15:59 ` [PATCH 5/8] ALSA: ump: Copy safe string name to rawmidi Takashi Iwai
2025-01-10 15:59 ` [PATCH 6/8] ALSA: ump: Update rawmidi name per EP name update Takashi Iwai
2025-01-10 15:59 ` [PATCH 7/8] ALSA: seq: Allow system notification in atomic Takashi Iwai
2025-01-10 15:59 ` [PATCH 8/8] ALSA: seq: Notify UMP EP and FB changes Takashi Iwai
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=87seplznnj.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=linux-sound@vger.kernel.org \
--cc=perex@perex.cz \
/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