From: James Courtier-Dutton <James@superbug.co.uk>
To: Takashi Iwai <tiwai@suse.de>
Cc: Jaroslav Kysela <perex@suse.cz>,
Clemens Ladisch <clemens@ladisch.de>,
alsa-devel <alsa-devel@lists.sourceforge.net>
Subject: Re: [PATCH] Implement support for display of dB gain in alsamixer.
Date: Sat, 26 Nov 2005 12:02:17 +0000 [thread overview]
Message-ID: <43884EC9.3090609@superbug.co.uk> (raw)
In-Reply-To: <s5h64sabx5i.wl%tiwai@suse.de>
Takashi Iwai wrote:
>>I think that a simple TLV(type, length, value) approach might be more
>>efficient for kernel -> alsa-lib info passing.
>>The message can easily be made hyrachical. One starts with one TLV as a
>>wrapper around all information passed. Inside that TLV will be other
>>TLVs, and inside those others. This allows the alsa-lib to simply skip
>>values it does not yet know about. We can even use a TLV in the request
>>from alsa-lib to the kernel.
>>
>>The T(Type) value is a simple 32bit integer. The receiving function
>>simply uses a lookup to find the parameter name, and also the type of
>>the value, e.g. Integer, string etc.
>>
>>For an example of a protocol already using this approach, look at the
>>RFCs for RADIUS.
>
>
> I think it's OK to use such a protocol, as long as the kernel
> implemention is simple. Maybe less allergy than XML texts :)
> One-to-one mapping would be the easiest for the driver side.
> And, above all, it's driver specific how complex it will be.
> Not many drivers require the dynamic topology information.
>
>
> My concern about the new ioctl is: Avoid architecture dependency.
> It implies:
> - Don't use union.
> - Don't use long.
> - Don't use long long.
> - Don't use bit fields.
> - Avoid pointer as much as possible.
> - Consider the alignment.
>
> This is one of the reasons why I don't want to merge the dB attribute
> into snd_ctl_elem_info/value.
>
>
> Takashi
>
>
Ok, so I am going to implement a new IOCTL that will use TLVs.
The first implementation will be used to support the dB gain information
passing, but it will be flexible enough to request any information needed.
James
-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems? Stop! Download the new AJAX search engine that makes
searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
next prev parent reply other threads:[~2005-11-26 12:02 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-09-18 13:31 [PATCH] Implement support for display of dB gain in alsamixer James Courtier-Dutton
2005-09-19 13:36 ` Jaroslav Kysela
2005-09-19 15:20 ` Takashi Iwai
2005-09-19 15:36 ` Jaroslav Kysela
2005-09-19 15:58 ` Takashi Iwai
2005-09-20 12:36 ` Jaroslav Kysela
2005-09-20 14:21 ` Clemens Ladisch
2005-09-21 8:02 ` Jaroslav Kysela
2005-10-06 12:54 ` James Courtier-Dutton
2005-10-06 13:19 ` Takashi Iwai
2005-11-26 12:02 ` James Courtier-Dutton [this message]
2005-09-19 19:12 ` James Courtier-Dutton
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=43884EC9.3090609@superbug.co.uk \
--to=james@superbug.co.uk \
--cc=alsa-devel@lists.sourceforge.net \
--cc=clemens@ladisch.de \
--cc=perex@suse.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.