[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