[alsa-devel] [PATCH 1/6] Remove/replace some deprecated items. Add new compander api. Add a version string define.
Takashi Iwai
tiwai at suse.de
Mon Jul 5 11:25:28 CEST 2010
Eliot,
could you shorten the summary line and put more texts into the commit
log, and resubmit? Two sentences are unusual as a summary line.
Ditto for some other patches.
Otherwise, the changes look good to me.
thanks,
Takashi
At Fri, 02 Jul 2010 11:23:30 +1200,
linux at audioscience.com wrote:
>
> From: Eliot Blennerhassett <eblennerhassett at audioscience.com>
>
> Signed-off-by: Eliot Blennerhassett <eblennerhassett at audioscience.com>
> ---
> pci/asihpi/asihpi.c | 10 +++---
> pci/asihpi/hpi.h | 68 ++++++++++++++++++++++++++++++++++--------------
> pci/asihpi/hpidebug.h | 4 +-
> 3 files changed, 55 insertions(+), 27 deletions(-)
>
> diff --git a/pci/asihpi/asihpi.c b/pci/asihpi/asihpi.c
> index 1db586a..91218f7 100644
> --- a/pci/asihpi/asihpi.c
> +++ b/pci/asihpi/asihpi.c
> @@ -1383,7 +1383,7 @@ static char *asihpi_src_names[] =
>
> compile_time_assert(
> (ARRAY_SIZE(asihpi_src_names) ==
> - (HPI_SOURCENODE_LAST_INDEX-HPI_SOURCENODE_BASE+1)),
> + (HPI_SOURCENODE_LAST_INDEX-HPI_SOURCENODE_NONE+1)),
> assert_src_names_size);
>
> #if ASI_STYLE_NAMES
> @@ -1414,7 +1414,7 @@ static char *asihpi_dst_names[] =
>
> compile_time_assert(
> (ARRAY_SIZE(asihpi_dst_names) ==
> - (HPI_DESTNODE_LAST_INDEX-HPI_DESTNODE_BASE+1)),
> + (HPI_DESTNODE_LAST_INDEX-HPI_DESTNODE_NONE+1)),
> assert_dst_names_size);
>
> static inline int ctl_add(struct snd_card *card, struct snd_kcontrol_new *ctl,
> @@ -2171,7 +2171,7 @@ static int snd_asihpi_mux_info(struct snd_kcontrol *kcontrol,
> &src_node_type, &src_node_index);
>
> sprintf(uinfo->value.enumerated.name, "%s %d",
> - asihpi_src_names[src_node_type - HPI_SOURCENODE_BASE],
> + asihpi_src_names[src_node_type - HPI_SOURCENODE_NONE],
> src_node_index);
> return 0;
> }
> @@ -2603,8 +2603,8 @@ static int __devinit snd_card_asihpi_mixer_new(struct snd_card_asihpi *asihpi)
>
> }
>
> - hpi_ctl.src_node_type -= HPI_SOURCENODE_BASE;
> - hpi_ctl.dst_node_type -= HPI_DESTNODE_BASE;
> + hpi_ctl.src_node_type -= HPI_SOURCENODE_NONE;
> + hpi_ctl.dst_node_type -= HPI_DESTNODE_NONE;
>
> /* ASI50xx in SSX mode has multiple meters on the same node.
> Use subindex to create distinct ALSA controls
> diff --git a/pci/asihpi/hpi.h b/pci/asihpi/hpi.h
> index 0173bbe..cee4df4 100644
> --- a/pci/asihpi/hpi.h
> +++ b/pci/asihpi/hpi.h
> @@ -50,7 +50,8 @@ i.e 3.05.02 is a development version
> #define HPI_VER_RELEASE(v) ((int)(v & 0xFF))
>
> /* Use single digits for versions less that 10 to avoid octal. */
> -#define HPI_VER HPI_VERSION_CONSTRUCTOR(4L, 3, 25)
> +#define HPI_VER HPI_VERSION_CONSTRUCTOR(4L, 3, 36)
> +#define HPI_VER_STRING "4.03.36"
>
> /* Library version as documented in hpi-api-versions.txt */
> #define HPI_LIB_VER HPI_VERSION_CONSTRUCTOR(9, 0, 0)
> @@ -203,8 +204,6 @@ enum HPI_SOURCENODES {
> exists on a destination node can be searched for using a source
> node value of either 0, or HPI_SOURCENODE_NONE */
> HPI_SOURCENODE_NONE = 100,
> - /** \deprecated Use HPI_SOURCENODE_NONE instead. */
> - HPI_SOURCENODE_BASE = 100,
> /** Out Stream (Play) node. */
> HPI_SOURCENODE_OSTREAM = 101,
> /** Line in node - could be analog, AES/EBU or network. */
> @@ -235,8 +234,6 @@ enum HPI_DESTNODES {
> exists on a source node can be searched for using a destination
> node value of either 0, or HPI_DESTNODE_NONE */
> HPI_DESTNODE_NONE = 200,
> - /** \deprecated Use HPI_DESTNODE_NONE instead. */
> - HPI_DESTNODE_BASE = 200,
> /** In Stream (Record) node. */
> HPI_DESTNODE_ISTREAM = 201,
> HPI_DESTNODE_LINEOUT = 202, /**< line out node. */
> @@ -432,7 +429,18 @@ Property 2 - adapter can do stream grouping (supports SSX2)
> Property 1 - adapter can do samplerate conversion (MRX)
> Property 2 - adapter can do timestretch (TSX)
> */
> - HPI_ADAPTER_PROPERTY_CAPS2 = 269
> + HPI_ADAPTER_PROPERTY_CAPS2 = 269,
> +
> +/** Readonly adapter sync header connection count.
> +*/
> + HPI_ADAPTER_PROPERTY_SYNC_HEADER_CONNECTIONS = 270,
> +/** Readonly supports SSX2 property.
> +Indicates the adapter supports SSX2 in some mode setting. The
> +return value is true (1) or false (0). If the current adapter
> +mode is MONO SSX2 is disabled, even though this property will
> +return true.
> +*/
> + HPI_ADAPTER_PROPERTY_SUPPORTS_SSX2 = 271
> };
>
> /** Adapter mode commands
> @@ -813,8 +821,6 @@ enum HPI_SAMPLECLOCK_SOURCES {
> /** The sampleclock output is derived from its local samplerate generator.
> The local samplerate may be set using HPI_SampleClock_SetLocalRate(). */
> HPI_SAMPLECLOCK_SOURCE_LOCAL = 1,
> -/** \deprecated Use HPI_SAMPLECLOCK_SOURCE_LOCAL instead */
> - HPI_SAMPLECLOCK_SOURCE_ADAPTER = 1,
> /** The adapter is clocked from a dedicated AES/EBU SampleClock input.*/
> HPI_SAMPLECLOCK_SOURCE_AESEBU_SYNC = 2,
> /** From external wordclock connector */
> @@ -825,10 +831,6 @@ enum HPI_SAMPLECLOCK_SOURCES {
> HPI_SAMPLECLOCK_SOURCE_SMPTE = 5,
> /** One of the aesebu inputs */
> HPI_SAMPLECLOCK_SOURCE_AESEBU_INPUT = 6,
> -/** \deprecated The first aesebu input with a valid signal
> -Superseded by separate Auto enable flag
> -*/
> - HPI_SAMPLECLOCK_SOURCE_AESEBU_AUTO = 7,
> /** From a network interface e.g. Cobranet or Livewire at either 48 or 96kHz */
> HPI_SAMPLECLOCK_SOURCE_NETWORK = 8,
> /** From previous adjacent module (ASI2416 only)*/
> @@ -1015,8 +1017,6 @@ enum HPI_ERROR_CODES {
> HPI_ERROR_CONTROL_DISABLED = 404,
> /** I2C transaction failed due to a missing ACK. */
> HPI_ERROR_CONTROL_I2C_MISSING_ACK = 405,
> - /** Control attribute is valid, but not supported by this hardware. */
> - HPI_ERROR_UNSUPPORTED_CONTROL_ATTRIBUTE = 406,
> /** Control is busy, or coming out of
> reset and cannot be accessed at this time. */
> HPI_ERROR_CONTROL_NOT_READY = 407,
> @@ -1827,13 +1827,41 @@ u16 hpi_parametricEQ__get_coeffs(const struct hpi_hsubsys *ph_subsys,
> Compressor Expander control
> *******************************/
>
> -u16 hpi_compander_set(const struct hpi_hsubsys *ph_subsys, u32 h_control,
> - u16 attack, u16 decay, short ratio100, short threshold0_01dB,
> - short makeup_gain0_01dB);
> +u16 hpi_compander_set_enable(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 on);
> +
> +u16 hpi_compander_get_enable(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 *pon);
> +
> +u16 hpi_compander_set_makeup_gain(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, short makeup_gain0_01dB);
> +
> +u16 hpi_compander_get_makeup_gain(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, short *pn_makeup_gain0_01dB);
> +
> +u16 hpi_compander_set_attack_time_constant(const struct hpi_hsubsys
> + *ph_subsys, u32 h_control, u32 index, u32 attack);
> +
> +u16 hpi_compander_get_attack_time_constant(const struct hpi_hsubsys
> + *ph_subsys, u32 h_control, u32 index, u32 *pw_attack);
> +
> +u16 hpi_compander_set_decay_time_constant(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 index, u32 decay);
> +
> +u16 hpi_compander_get_decay_time_constant(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 index, u32 *pw_decay);
> +
> +u16 hpi_compander_set_threshold(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 index, short threshold0_01dB);
> +
> +u16 hpi_compander_get_threshold(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 index, short *pn_threshold0_01dB);
> +
> +u16 hpi_compander_set_ratio(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 index, u32 ratio100);
>
> -u16 hpi_compander_get(const struct hpi_hsubsys *ph_subsys, u32 h_control,
> - u16 *pw_attack, u16 *pw_decay, short *pw_ratio100,
> - short *pn_threshold0_01dB, short *pn_makeup_gain0_01dB);
> +u16 hpi_compander_get_ratio(const struct hpi_hsubsys *ph_subsys,
> + u32 h_control, u32 index, u32 *pw_ratio100);
>
> /*******************************
> Cobranet HMI control
> diff --git a/pci/asihpi/hpidebug.h b/pci/asihpi/hpidebug.h
> index 44dccad..a2f0952 100644
> --- a/pci/asihpi/hpidebug.h
> +++ b/pci/asihpi/hpidebug.h
> @@ -356,7 +356,7 @@ compile_time_assert((HPI_CONTROL_LAST_INDEX + 1 == 27),
> "HPI_SOURCENODE_ADAPTER" \
> }
>
> -compile_time_assert((HPI_SOURCENODE_LAST_INDEX - HPI_SOURCENODE_BASE + 1) ==
> +compile_time_assert((HPI_SOURCENODE_LAST_INDEX - HPI_SOURCENODE_NONE + 1) ==
> (12), sourcenode_strings_match_defs);
>
> #define HPI_DESTNODE_STRINGS \
> @@ -370,7 +370,7 @@ compile_time_assert((HPI_SOURCENODE_LAST_INDEX - HPI_SOURCENODE_BASE + 1) ==
> "HPI_DESTNODE_COBRANET", \
> "HPI_DESTNODE_ANALOG" \
> }
> -compile_time_assert((HPI_DESTNODE_LAST_INDEX - HPI_DESTNODE_BASE + 1) == (8),
> +compile_time_assert((HPI_DESTNODE_LAST_INDEX - HPI_DESTNODE_NONE + 1) == (8),
> destnode_strings_match_defs);
>
> #define HPI_CONTROL_CHANNEL_MODE_STRINGS \
> --
> 1.7.0.4
>
More information about the Alsa-devel
mailing list