[alsa-devel] cmi8330: revert comments about AD1848 back

Takashi Iwai tiwai at suse.de
Mon Jun 29 08:16:13 CEST 2009


At Sat, 27 Jun 2009 16:26:35 +0200,
Ondrej Zary wrote:
> 
> In ALSA 1.0.20, the comments were changed to say CMI8330 instead of AD1848.
> The CMI8330 chip includes two codecs - AD1848 and SB16, so the comments were
> correct and are misleading now. Revert them back.
> 
> Signed-off-by: Ondrej Zary <linux at rainbow-software.org>

Applied this one, too.


Takashi


> --- linux-2.6.30-orig/sound/isa/cmi8330.c	2009-06-10 05:05:27.000000000 +0200
> +++ linux-test/sound/isa/cmi8330.c	2009-06-27 16:20:51.000000000 +0200
> @@ -35,7 +35,7 @@
>   *
>   *  This card has two mixers and two PCM devices.  I've cheesed it such
>   *  that recording and playback can be done through the same device.
> - *  The driver "magically" routes the capturing to the CMI8330 codec,
> + *  The driver "magically" routes the capturing to the AD1848 codec,
>   *  and playback to the SB16 codec.  This allows for full-duplex mode
>   *  to some extent.
>   *  The utilities in alsa-utils are aware of both devices, so passing
> @@ -345,7 +345,7 @@
>  
>  	err = pnp_activate_dev(pdev);
>  	if (err < 0) {
> -		snd_printk(KERN_ERR "CMI8330/C3D PnP configure failure\n");
> +		snd_printk(KERN_ERR "CMI8330/C3D (AD1848) PnP configure failure\n");
>  		return -EBUSY;
>  	}
>  	wssport[dev] = pnp_port_start(pdev, 0);
> @@ -527,11 +527,11 @@
>  			     wssdma[dev], -1,
>  			     WSS_HW_DETECT, 0, &acard->wss);
>  	if (err < 0) {
> -		snd_printk(KERN_ERR PFX "(CMI8330) device busy??\n");
> +		snd_printk(KERN_ERR PFX "(AD1848) device busy??\n");
>  		return err;
>  	}
>  	if (acard->wss->hardware != WSS_HW_CMI8330) {
> -		snd_printk(KERN_ERR PFX "(CMI8330) not found during probe\n");
> +		snd_printk(KERN_ERR PFX "(AD1848) not found during probe\n");
>  		return -ENODEV;
>  	}
>  
> 
> -- 
> Ondrej Zary
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 


More information about the Alsa-devel mailing list