Re: [alsa-devel] [PATCH] Add support for Cyrix/NatSemi Geode CS5530 (VSA1)
I'm sorry. I forgot to mention that these patches replace the two previous patches :)
Ash
----- Original Message ----- From: "Ash Willis" ashwillis@programmer.net To: "Takashi Iwai" tiwai@suse.de Subject: [alsa-devel] [PATCH] Add support for Cyrix/NatSemi Geode CS5530 (VSA1) Date: Thu, 24 May 2007 09:09:29 -0500
Takashi, here are my modified patches. One small one for alsa-driver. I took your advice and merged the rest into alsa-kernel. Everything should be in order this time.
Add support for Cyrix/NatSemi Geode SC5530 (VSA1)
Signed-off-by Ash Willis ashwillis@programmer.net
diff -r fb391138d29c -r 994136cf989e pci/cs5530.c --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/pci/cs5530.c Thu May 24 20:45:25 2007 +0100 @@ -0,0 +1,2 @@ +#include "../alsa-kernel/pci/cs5530.c" +EXPORT_NO_SYMBOLS;
diff -r 45179b325c8e -r 0b0c03c6c6ca Documentation/ALSA-Configuration.txt --- a/Documentation/ALSA-Configuration.txt Wed May 23 16:27:32 2007 +0200 +++ b/Documentation/ALSA-Configuration.txt Thu May 24 20:58:27 2007 +0100 @@ -467,7 +467,12 @@ Prior to version 0.9.0rc4 options had a above explicitly.
The power-management is supported.
- Module snd-cs5530
- Module for Cyrix/NatSemi Geode 5530 chip.
Module snd-cs5535audio
diff -r 45179b325c8e -r 0b0c03c6c6ca include/sb.h --- a/include/sb.h Wed May 23 16:27:32 2007 +0200 +++ b/include/sb.h Thu May 24 20:58:27 2007 +0100 @@ -38,6 +38,7 @@ enum sb_hw_type { SB_HW_ALS100, /* Avance Logic ALS100 chip */ SB_HW_ALS4000, /* Avance Logic ALS4000 chip */ SB_HW_DT019X, /* Diamond Tech. DT-019X / Avance Logic ALS-007 */
SB_HW_CS5530, /* Cyrix/NatSemi 5530 VSA1 */ };
#define SB_OPEN_PCM 0x01
diff -r 45179b325c8e -r 0b0c03c6c6ca isa/sb/sb16_main.c --- a/isa/sb/sb16_main.c Wed May 23 16:27:32 2007 +0200 +++ b/isa/sb/sb16_main.c Thu May 24 20:58:27 2007 +0100 @@ -563,6 +563,11 @@ static int snd_sb16_playback_open(struct __open_ok: if (chip->hardware == SB_HW_ALS100) runtime->hw.rate_max = 48000;
- if (chip->hardware == SB_HW_CS5530) {
runtime->hw.buffer_bytes_max = 32 * 1024;
runtime->hw.periods_min = 2;
runtime->hw.rate_min = 44100;
- } if (chip->mode & SB_RATE_LOCK) runtime->hw.rate_min = runtime->hw.rate_max = chip->locked_rate; chip->playback_substream = substream;
@@ -633,6 +638,11 @@ static int snd_sb16_capture_open(struct __open_ok: if (chip->hardware == SB_HW_ALS100) runtime->hw.rate_max = 48000;
- if (chip->hardware == SB_HW_CS5530) {
runtime->hw.buffer_bytes_max = 32 * 1024;
runtime->hw.periods_min = 2;
runtime->hw.rate_min = 44100;
- } if (chip->mode & SB_RATE_LOCK) runtime->hw.rate_min = runtime->hw.rate_max = chip->locked_rate; chip->capture_substream = substream;
diff -r 45179b325c8e -r 0b0c03c6c6ca isa/sb/sb_common.c --- a/isa/sb/sb_common.c Wed May 23 16:27:32 2007 +0200 +++ b/isa/sb/sb_common.c Thu May 24 20:58:27 2007 +0100 @@ -128,7 +128,7 @@ static int snd_sbdsp_probe(struct snd_sb minor = version & 0xff; snd_printdd("SB [0x%lx]: DSP chip found, version = %i.%i\n", chip->port, major, minor);
- switch (chip->hardware) { case SB_HW_AUTO: switch (major) {
@@ -167,6 +167,9 @@ static int snd_sbdsp_probe(struct snd_sb break; case SB_HW_DT019X: str = "(DT019X/ALS007)";
break;
- case SB_HW_CS5530:
break; default: return -ENODEV;str = "16 (CS5530)";
diff -r 45179b325c8e -r 0b0c03c6c6ca isa/sb/sb_mixer.c --- a/isa/sb/sb_mixer.c Wed May 23 16:27:32 2007 +0200 +++ b/isa/sb/sb_mixer.c Thu May 24 20:58:27 2007 +0100 @@ -821,6 +821,7 @@ int snd_sbmixer_new(struct snd_sb *chip) break; case SB_HW_16: case SB_HW_ALS100:
- case SB_HW_CS5530: if ((err = snd_sbmixer_init(chip, snd_sb16_controls, ARRAY_SIZE(snd_sb16_controls),
@@ -950,6 +951,7 @@ void snd_sbmixer_suspend(struct snd_sb * break; case SB_HW_16: case SB_HW_ALS100:
- case SB_HW_CS5530: save_mixer(chip, sb16_saved_regs, ARRAY_SIZE(sb16_saved_regs)); break; case SB_HW_ALS4000:
@@ -975,6 +977,7 @@ void snd_sbmixer_resume(struct snd_sb *c break; case SB_HW_16: case SB_HW_ALS100:
- case SB_HW_CS5530: restore_mixer(chip, sb16_saved_regs, ARRAY_SIZE(sb16_saved_regs)); break; case SB_HW_ALS4000:
diff -r 45179b325c8e -r 0b0c03c6c6ca pci/Kconfig --- a/pci/Kconfig Wed May 23 16:27:32 2007 +0200 +++ b/pci/Kconfig Thu May 24 20:58:27 2007 +0100 @@ -214,6 +214,16 @@ config SND_CS46XX_NEW_DSP Say Y here to use a new DSP image for SPDIF and dual codecs.
This works better than the old code, so say Y.
+config SND_CS5530
tristate "CS5530 Audio"
depends on SND
select SND_SB16
help
Say Y here to include support for audio on Cyrix/NatSemi CS5530 chips.
To compile this driver as a module, choose M here: the module
will be called snd-cs5530.
config SND_CS5535AUDIO tristate "CS5535/CS5536 Audio"
diff -r 45179b325c8e -r 0b0c03c6c6ca pci/Makefile --- a/pci/Makefile Wed May 23 16:27:32 2007 +0200 +++ b/pci/Makefile Thu May 24 20:58:27 2007 +0100 @@ -12,6 +12,7 @@ snd-bt87x-objs := bt87x.o snd-bt87x-objs := bt87x.o snd-cmipci-objs := cmipci.o snd-cs4281-objs := cs4281.o +snd-cs5530-objs := cs5530.o snd-ens1370-objs := ens1370.o snd-ens1371-objs := ens1371.o snd-es1938-objs := es1938.o @@ -36,6 +37,7 @@ obj-$(CONFIG_SND_BT87X) += snd-bt87x.o obj-$(CONFIG_SND_BT87X) += snd-bt87x.o obj-$(CONFIG_SND_CMIPCI) += snd-cmipci.o obj-$(CONFIG_SND_CS4281) += snd-cs4281.o +obj-$(CONFIG_SND_CS5530) += snd-cs5530.o obj-$(CONFIG_SND_ENS1370) += snd-ens1370.o obj-$(CONFIG_SND_ENS1371) += snd-ens1371.o obj-$(CONFIG_SND_ES1938) += snd-es1938.o diff -r 45179b325c8e -r 0b0c03c6c6ca pci/cs5530.c --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/pci/cs5530.c Thu May 24 20:58:27 2007 +0100 @@ -0,0 +1,306 @@ +/*
- cs5530.c - Initialisation code for Cyrix/NatSemi VSA1 softaudio
- (C) Copyright 2007 Ash Willis ashwillis@programmer.net
- (C) Copyright 2003 Red Hat Inc alan@redhat.com
- This driver was ported (shamelessly ripped ;) from oss/kahlua.c but I did
- mess with it a bit. The chip seems to have to have trouble with
full duplex
- mode. If we're recording in 8bit 8000kHz, say, and we then attempt to
- simultaneously play back audio at 16bit 44100kHz, the device
actually plays
- back in the same format in which it is capturing. By forcing the chip to
- always play/capture in 16/44100, we can let alsa-lib convert
the samples and
- that way we can hack up some full duplex audio.
- XpressAudio(tm) is used on the Cyrix MediaGX (now NatSemi Geode) systems.
- The older version (VSA1) provides fairly good soundblaster emulation
- although there are a couple of bugs: large DMA buffers break record,
- and the MPU event handling seems suspect. VSA2 allows the native driver
- to control the AC97 audio engine directly and requires a different driver.
- Thanks to National Semiconductor for providing the needed information
- on the XpressAudio(tm) internals.
- This program is free software; you can redistribute it and/or modify it
- under the terms of the GNU General Public License as published by the
- Free Software Foundation; either version 2, or (at your option) any
- later version.
- This program is distributed in the hope that it will be useful, but
- WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- General Public License for more details.
- TO DO:
- Investigate whether we can portably support Cognac (5520) in the
- same manner.
- */
+#include <sound/driver.h> +#include <linux/delay.h> +#include <linux/moduleparam.h> +#include <linux/pci.h> +#include <sound/core.h> +#include <sound/sb.h> +#include <sound/initval.h>
+MODULE_AUTHOR("Ash Willis"); +MODULE_DESCRIPTION("CS5530 Audio"); +MODULE_LICENSE("GPL");
+static int index[SNDRV_CARDS] = SNDRV_DEFAULT_IDX; +static char *id[SNDRV_CARDS] = SNDRV_DEFAULT_STR; +static int enable[SNDRV_CARDS] = SNDRV_DEFAULT_ENABLE_PNP;
+struct snd_cs5530 {
- struct snd_card *card;
- struct pci_dev *pci;
- struct snd_sb *sb;
- unsigned long pci_base;
+};
+static struct pci_device_id snd_cs5530_ids[] = {
- {PCI_VENDOR_ID_CYRIX, PCI_DEVICE_ID_CYRIX_5530_AUDIO, PCI_ANY_ID,
PCI_ANY_ID, 0, 0},
- {0,}
+};
+MODULE_DEVICE_TABLE(pci, snd_cs5530_ids);
+static int snd_cs5530_free(struct snd_cs5530 *chip) +{
- pci_release_regions(chip->pci);
- pci_disable_device(chip->pci);
- kfree(chip);
- return 0;
+}
+static int snd_cs5530_dev_free(struct snd_device *device) +{
- struct snd_cs5530 *chip = device->device_data;
- return snd_cs5530_free(chip);
+}
+static void __devexit snd_cs5530_remove(struct pci_dev *pci) +{
- snd_card_free(pci_get_drvdata(pci));
- pci_set_drvdata(pci, NULL);
+}
+static u8 __devinit snd_cs5530_mixer_read(unsigned long io, u8 reg) +{
- outb(reg, io + 4);
- udelay(20);
- reg = inb(io + 5);
- udelay(20);
- return reg;
+}
+static int __devinit snd_cs5530_create(struct snd_card *card,
struct pci_dev *pci,
struct snd_cs5530 **rchip)
+{
- struct snd_cs5530 *chip;
- unsigned long sb_base;
- u8 irq, dma8, dma16 = 0;
- u16 map;
- void __iomem *mem;
- int err;
- static struct snd_device_ops ops = {
.dev_free = snd_cs5530_dev_free,
- };
- *rchip = NULL;
- err = pci_enable_device(pci);
- if (err < 0)
return err;
- chip = kzalloc(sizeof(*chip), GFP_KERNEL);
- if (chip == NULL) {
pci_disable_device(pci);
return -ENOMEM;
- }
- chip->card = card;
- chip->pci = pci;
- err = pci_request_regions(pci, "CS5530");
- if (err < 0) {
kfree(chip);
pci_disable_device(pci);
return err;
- }
- chip->pci_base = pci_resource_start(pci, 0);
- mem = ioremap_nocache(chip->pci_base, pci_resource_len(pci, 0));
- if (mem == NULL) {
kfree(chip);
pci_disable_device(pci);
return -EBUSY;
- }
- map = readw(mem + 0x18);
- iounmap(mem);
- /* Map bits
0:1 * 0x20 + 0x200 = sb base
2 sb enable
3 adlib enable
5 MPU enable 0x330
6 MPU enable 0x300
The other bits may be used internally so must be masked */
- sb_base = 0x220 + 0x20 * (map & 3);
- if (map & (1<<2))
printk(KERN_INFO "CS5530: XpressAudio at 0x%lx\n", sb_base);
- else {
printk(KERN_ERR "Could not find XpressAudio!\n");
snd_cs5530_free(chip);
return -ENODEV;
- }
- if (map & (1<<5))
printk(KERN_INFO "CS5530: MPU at 0x300\n");
- else if (map & (1<<6))
printk(KERN_INFO "CS5530: MPU at 0x330\n");
- irq = snd_cs5530_mixer_read(sb_base, 0x80) & 0x0F;
- dma8 = snd_cs5530_mixer_read(sb_base, 0x81);
- if (dma8 & 0x20)
dma16 = 5;
- else if (dma8 & 0x40)
dma16 = 6;
- else if (dma8 & 0x80)
dma16 = 7;
- else {
printk(KERN_ERR "CS5530: No 16bit DMA enabled\n");
snd_cs5530_free(chip);
return -ENODEV;
- }
- if (dma8 & 0x01)
dma8 = 0;
- else if (dma8 & 02)
dma8 = 1;
- else if (dma8 & 0x08)
dma8 = 3;
- else {
printk(KERN_ERR "CS5530: No 8bit DMA enabled\n");
snd_cs5530_free(chip);
return -ENODEV;
- }
- if (irq & 1)
irq = 9;
- else if (irq & 2)
irq = 5;
- else if (irq & 4)
irq = 7;
- else if (irq & 8)
irq = 10;
- else {
printk(KERN_ERR "CS5530: SoundBlaster IRQ not set\n");
snd_cs5530_free(chip);
return -ENODEV;
- }
- printk(KERN_INFO "CS5530: IRQ: %d DMA8: %d DMA16: %d\n", irq, dma8,
dma16);
- err = snd_sbdsp_create(card, sb_base, irq, snd_sb16dsp_interrupt, dma8,
dma16, SB_HW_CS5530, &chip->sb);
- if (err < 0) {
printk(KERN_ERR "CS5530: Could not create SoundBlaster\n");
snd_cs5530_free(chip);
return err;
- }
- err = snd_sb16dsp_pcm(chip->sb, 0, &chip->sb->pcm);
- if (err < 0) {
printk(KERN_ERR "CS5530: Could not create PCM\n");
snd_cs5530_free(chip);
return err;
- }
- err = snd_sbmixer_new(chip->sb);
- if (err < 0) {
printk(KERN_ERR "CS5530: Could not create Mixer\n");
snd_cs5530_free(chip);
return err;
- }
- err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops);
- if (err < 0) {
snd_cs5530_free(chip);
return err;
- }
- snd_card_set_dev(card, &pci->dev);
- *rchip = chip;
- return 0;
+}
+static int __devinit snd_cs5530_probe(struct pci_dev *pci,
const struct pci_device_id *pci_id)
+{
- static int dev;
- struct snd_card *card;
- struct snd_cs5530 *chip = NULL;
- int err;
- if (dev >= SNDRV_CARDS)
return -ENODEV;
- if (!enable[dev]) {
dev++;
return -ENOENT;
- }
- card = snd_card_new(index[dev], id[dev], THIS_MODULE, 0);
- if (card == NULL)
return -ENOMEM;
- err = snd_cs5530_create(card, pci, &chip);
- if (err < 0) {
snd_card_free(card);
return err;
- }
- strcpy(card->driver, "CS5530");
- strcpy(card->shortname, "CS5530 Audio");
- sprintf(card->longname, "%s at 0x%lx", card->shortname, chip->pci_base);
- err = snd_card_register(card);
- if (err < 0) {
snd_card_free(card);
return err;
- }
- pci_set_drvdata(pci, card);
- dev++;
- return 0;
+}
+static struct pci_driver driver = {
- .name = "CS5530_Audio",
- .id_table = snd_cs5530_ids,
- .probe = snd_cs5530_probe,
- .remove = __devexit_p(snd_cs5530_remove),
+};
+static int __init alsa_card_cs5530_init(void) +{
- return pci_register_driver(&driver);
+}
+static void __exit alsa_card_cs5530_exit(void) +{
- pci_unregister_driver(&driver);
+}
+module_init(alsa_card_cs5530_init) +module_exit(alsa_card_cs5530_exit)
=
Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
=
participants (1)
-
Ash Willis