[alsa-devel] [PATCH] hda: STAC927x DMIC Cleanup

Takashi Iwai tiwai at suse.de
Mon Dec 17 10:23:47 CET 2007


At Fri, 14 Dec 2007 15:02:40 -0500,
Matthew Ranostay wrote:
> 
> @@ -1408,22 +1409,24 @@ static unsigned int dell_3st_pin_configs
>  static unsigned int dell_3st_pin_configs[14] = {
>  	0x02211230, 0x02a11220, 0x01a19040, 0x01114210,
>  	0x01111212, 0x01116211, 0x01813050, 0x01112214,
> -	0x403003fa, 0x40000100, 0x40000100, 0x404003fb,
> +	0x403003fa, 0x90a60040, 0x90a60040, 0x404003fb,
>  	0x40c003fc, 0x40000100
>  };

Just to be sure - is it safe to change this?

> @@ -3293,8 +3304,11 @@ static int patch_stac927x(struct hda_cod
>  							stac927x_models,
>  							stac927x_cfg_tbl);
>   again:
> -	if (spec->board_config < 0) {
> -                snd_printdd(KERN_INFO "hda_codec: Unknown model for STAC927x, using BIOS defaults\n");
> +	if (spec->board_config < 0 ||
> +	    stac927x_brd_tbl[spec->board_config] != NULL) {

Should be stac927x_brd_tbl[spec->board_config] == NULL.
Also, the following else-if should be fixed (the check is nonsense).

I applied the fixed patch to HG tree.  Could you check whether it's
OK?


Thanks,

Takashi


More information about the Alsa-devel mailing list