Re: [PATCH 4/4] ALSA: isight: use position offset macro of TLV data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Takashi Sakamoto wrote:
> A series of SNDRV_CTL_TLVO_XXX macro was introduced for position offset
> of TLV data. This commit applies a code optimization.
>
> Signed-off-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>

Acked-by: Clemens Ladisch <clemens@xxxxxxxxxx>

> ---
>  sound/firewire/isight.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/sound/firewire/isight.c b/sound/firewire/isight.c
> index 46092fa3ff9b..3919e186a30b 100644
> --- a/sound/firewire/isight.c
> +++ b/sound/firewire/isight.c
> @@ -569,18 +569,20 @@ static int isight_create_mixer(struct isight *isight)
>  		return err;
>  	isight->gain_max = be32_to_cpu(value);
>
> -	isight->gain_tlv[0] = SNDRV_CTL_TLVT_DB_MINMAX;
> -	isight->gain_tlv[1] = 2 * sizeof(unsigned int);
> +	isight->gain_tlv[SNDRV_CTL_TLVO_TYPE] = SNDRV_CTL_TLVT_DB_MINMAX;
> +	isight->gain_tlv[SNDRV_CTL_TLVO_LEN] = 2 * sizeof(unsigned int);
>
>  	err = reg_read(isight, REG_GAIN_DB_START, &value);
>  	if (err < 0)
>  		return err;
> -	isight->gain_tlv[2] = (s32)be32_to_cpu(value) * 100;
> +	isight->gain_tlv[SNDRV_CTL_TLVO_DB_MINMAX_MIN] =
> +						(s32)be32_to_cpu(value) * 100;
>
>  	err = reg_read(isight, REG_GAIN_DB_END, &value);
>  	if (err < 0)
>  		return err;
> -	isight->gain_tlv[3] = (s32)be32_to_cpu(value) * 100;
> +	isight->gain_tlv[SNDRV_CTL_TLVO_DB_MINMAX_MAX] =
> +						(s32)be32_to_cpu(value) * 100;
>
>  	ctl = snd_ctl_new1(&gain_control, isight);
>  	if (ctl)
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux