All of lore.kernel.org
 help / color / mirror / Atom feed
From: Clemens Ladisch <clemens@ladisch.de>
To: Ricard Wanderlof <ricard.wanderlof@axis.com>
Cc: Takashi Iwai <tiwai@suse.de>,
	alsa-devel@alsa-project.org,
	Martin Tarenskeen <m.tarenskeen@zonnet.nl>
Subject: Re: [PATCH 1/2] amidi: ignore not only Active Sensing	but all System Real-Time messages
Date: Mon, 11 Apr 2016 15:29:21 +0200	[thread overview]
Message-ID: <570BA6B1.8060101@ladisch.de> (raw)
In-Reply-To: <alpine.DEB.2.02.1604110858100.9999@lnxricardw1.se.axis.com>

Ricard Wanderlof wrote:
> On Sun, 10 Apr 2016, Clemens Ladisch wrote:
>> By default, amidi ignores Active Sensing messages because they are sent
>> by many devices in the background and would only interfere with the
>> actual messages that amidi is supposed to capture.  However, there are
>> also devices that send Clock messages with the same problem, so it is
>> a better idea to filter out all System Real-Time messages.
>
> I would argue that it would be better to have both options, somehow. I.e.
> active sensing is mostly a nuisance, but I can imagine occasions
> (analyzing the output from a sequencer perhaps) where one would want to
> keep the rest of the real time messages.

I can imagine this too, and even more complex filters.

But the amidi tool is designed to be simple, and works on the lowest
level, with raw MIDI bytes.  This makes it appropriate to handle SysEx
stuff and to debug low-level hardware problems, but when you care about
the semantics of the messages, you should use a higher-level tool, such
as aseqdump.

Filtering out clock messages serves an actual need.  But I am not
willing to add complexity for a problem that is, at the moment, nothing
but a figment of our imaginations.


Regards,
Clemens

  parent reply	other threads:[~2016-04-11 13:29 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-10 19:29 [PATCH 1/2] amidi: ignore not only Active Sensing but all System Real-Time messages Clemens Ladisch
2016-04-10 19:30 ` [PATCH 2/2] amidi: fix timeout handling Clemens Ladisch
2016-04-11  7:00 ` [PATCH 1/2] amidi: ignore not only Active Sensing but all System Real-Time messages Ricard Wanderlof
2016-04-11 11:17   ` Martin Tarenskeen
2016-04-11 11:47     ` Martin Tarenskeen
2016-04-11 13:29   ` Clemens Ladisch [this message]
2016-04-11 13:41     ` Ricard Wanderlof
2016-04-11 14:13       ` Clemens Ladisch
2016-04-11 14:18         ` Ricard Wanderlof
2016-04-17 13:21   ` [PATCH v2 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
2016-04-17 13:21     ` [PATCH v2 2/2] amidi: fix timeout handling Clemens Ladisch

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=570BA6B1.8060101@ladisch.de \
    --to=clemens@ladisch.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=m.tarenskeen@zonnet.nl \
    --cc=ricard.wanderlof@axis.com \
    --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.