* Re: A: VIA Technologies Inc. VT1720/24 [not found] <45AFE785.6020307@informatik.uni-oldenburg.de> @ 2007-01-24 12:10 ` Philipp Matthias Hahn 2007-01-24 15:12 ` [Alsa-devel] " Takashi Iwai 0 siblings, 1 reply; 15+ messages in thread From: Philipp Matthias Hahn @ 2007-01-24 12:10 UTC (permalink / raw) To: ALSA developers; +Cc: ALSA users [-- Attachment #1: Type: text/plain, Size: 3780 bytes --] Philipp Matthias Hahn wrote: > I'm not subscribed, so please cc: me on replies. > > I have several Fujitsu-Siemens Scenic W600 with an Onboard AD1885 AC97 > Audio Codec. (See > http://www.fujitsu-siemens.co.uk/rl/servicesupport/techsupport/boards/Motherboards/FSC/D1321/D1321.htm Wrong link, the right one is http://www.fujitsu-siemens.co.uk/rl/servicesupport/techsupport/Boards/Motherboards/FSC/D1527/D1527.htm > for the board description). This one is working fine with the > snd-intel8x0 module from the Linux-2.6.18.6 kernel. > > In one of those computers I plugged in an additional sound card: > > AudioTrak Prodigy 7.1 HiFi > > XXXXX:/dev# lspci -s 2:0d.0 > 02:0d.0 Multimedia audio controller: VIA Technologies Inc. VT1720/24 > [Envy24PT/HT] PCI Multi-Channel Audio Controller (rev 01) > > XXXXX:/dev# lspci -s 2:0d.0 -nvxxx > 02:0d.0 0401: 1412:1724 (rev 01) > Subsystem: 3138:4154 > Flags: medium devsel, IRQ 21 > I/O ports at 3c00 [disabled] [size=32] > I/O ports at 3800 [disabled] [size=128] > Capabilities: [80] Power Management version 1 > 00: 12 14 24 17 00 00 10 02 01 00 01 04 00 40 00 00 > 10: 01 3c 00 00 01 38 00 00 00 00 00 00 00 00 00 00 > 20: 00 00 00 00 00 00 00 00 00 00 00 00 38 31 54 41 > 30: 00 00 00 00 80 00 00 00 00 00 00 00 05 01 00 00 > 40: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 60: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 80: 01 00 01 04 00 00 00 00 00 00 00 00 00 00 00 00 > 90: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > d0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > > Now I have two problems: > > 1. The onboard intel8x0-sound is now missing, that is, the following > Device is no longer listed using "lspci": > 0000:00:1f.5 Multimedia audio controller: Intel Corp. 82801DB/DBL/DBM > (ICH4/ICH4-L/ICH4-M) AC'97 Audio Controller (rev 01) > > Is there a way to re-enable it, so I can use both cards? > (On a side node not relevant for ALSA: the board also has an onboard > integrated graphics controller, which is also disables by plugging in an > AGP card. Would also be nice to re-enable it.) Solved: I had to switch the BIOS from "Auto" to "Always enable". > 2. Loading the "snd-ice1724" module reports this failure: > > ACPI: PCI Interrupt 0000:02:0d.0[A] -> GSI 21 (level, low) -> IRQ 21 > ice1724: No matching model found for ID 0x38315441 > ice1724: Invalid EEPROM (size = 120) > ACPI: PCI interrupt for device 0000:02:0d.0 disabled > ICE1724: probe of 0000:02:0d.0 failed with error -5 > > Can somebody help me to get this working? > > Seemd I'm not alone, but I can't understand the language: > http://www.google.com/search?q=cache:ZaBwG4Ge5b4J:forum.ubuntu.pl/viewtopic.php%3Ft%3D16017%26view%3Dprevious%26sid%3D0a6db6f9be5123927dfb6c3a582e3eef+ice1724+0x38315441&hl=de&gl=de&ct=clnk&cd=1&client=firefox I did some reading and hacking myself, since it just seems to be the same as the Prodigy 7.1 with just a different model ID. See the attached patch for a first version. It seems to work thus far without any problems. Sincerly Philipp Hahn -- Dipl.-Inform. Philipp.Hahn@informatik.uni-oldenburg.de Abteilung Systemsoftware und verteilte Systeme, Fk. II Carl von Ossietzky Universitaet Oldenburg, 26111 Oldenburg, Germany http://www.svs.informatik.uni-oldenburg.de/contact/pmhahn/ Telefon: +49 441 798-2866 Telefax: +49 441 798-2756 [-- Attachment #2: sound_pci_ice1712.diff --] [-- Type: text/x-patch, Size: 2901 bytes --] From: Philipp Matthias Hahn <pmhahn@pmhahn.de> Fix small typo in comment of Prodigy 7.1 Light Add support for AudioTrak Prodigt 7.1 HiFi Signed-of-by: Philipp Matthias Hahn <pmhahn@pmhahn.de> --- --- sound/pci/ice1712/aureon.c 2006-12-17 01:21:00.000000000 +0100 +++ sound/pci/ice1712/aureon.c 2007-01-19 10:50:19.000000000 +0100 @@ -2085,7 +2085,7 @@ static unsigned char prodigy71_eeprom[] }; static unsigned char prodigy71lt_eeprom[] __devinitdata = { - 0x4b, /* SYSCINF: clock 512, spdif-in/ADC, 4DACs */ + 0x4b, /* SYSCONF: clock 384, spdif-in/ADC, 4DACs */ 0x80, /* ACLINK: I2S */ 0xfc, /* I2S: vol, 96k, 24bit, 192k */ 0xc3, /* SPDIF: out-en, out-int, spdif-in */ @@ -2100,6 +2100,22 @@ static unsigned char prodigy71lt_eeprom[ 0x00, /* GPIO_STATE2 */ }; +static unsigned char prodigy71hifi_eeprom[] __devinitdata = { + [ICE_EEP2_SYSCONF] = 0x0b, /* clock 512, spdif-in/ADC, 4DACs */ + [ICE_EEP2_ACLINK] = 0x80, /* I2S */ + [ICE_EEP2_I2S] = 0xfc, /* vol, 96k, 24bit, 192k */ + [ICE_EEP2_SPDIF] = 0xc3, /* out-en, out-int, spdif-in */ + [ICE_EEP2_GPIO_DIR] = 0xff, + [ICE_EEP2_GPIO_DIR1] = 0xff, + [ICE_EEP2_GPIO_DIR2] = 0x5f, + [ICE_EEP2_GPIO_MASK] = 0x00, + [ICE_EEP2_GPIO_MASK1] = 0x00, + [ICE_EEP2_GPIO_MASK2] = 0x00, + [ICE_EEP2_GPIO_STATE] = 0x00, + [ICE_EEP2_GPIO_STATE1] = 0x00, + [ICE_EEP2_GPIO_STATE2] = 0x00, +}; + /* entry point */ struct snd_ice1712_card_info snd_vt1724_aureon_cards[] __devinitdata = { @@ -2153,5 +2169,15 @@ struct snd_ice1712_card_info snd_vt1724_ .eeprom_data = prodigy71lt_eeprom, .driver = "Prodigy71LT", }, + { + .subvendor = VT1724_SUBDEVICE_PRODIGY71HIFI, + .name = "Audiotrak Prodigy 7.1 HiFi", + .model = "prodigy71hifi", + .chip_init = aureon_init, + .build_controls = aureon_add_controls, + .eeprom_size = sizeof(prodigy71hifi_eeprom), + .eeprom_data = prodigy71hifi_eeprom, + .driver = "Prodigy71HiFi", /* should be identical with Prodigy71 */ + }, { } /* terminator */ }; --- sound/pci/ice1712/aureon.h 2006-12-17 01:21:00.000000000 +0100 +++ sound/pci/ice1712/aureon.h 2007-01-19 10:50:19.000000000 +0100 @@ -28,13 +28,15 @@ "{Terratec,Aureon 7.1 Space},"\ "{Terratec,Aureon 7.1 Universe}," \ "{AudioTrak,Prodigy 7.1}," \ - "{AudioTrak,Prodigy 7.1 LT}," + "{AudioTrak,Prodigy 7.1 LT},"\ + "{AudioTrak,Prodigy 7.1 HiFi}," #define VT1724_SUBDEVICE_AUREON51_SKY 0x3b154711 /* Aureon 5.1 Sky */ #define VT1724_SUBDEVICE_AUREON71_SPACE 0x3b154511 /* Aureon 7.1 Space */ #define VT1724_SUBDEVICE_AUREON71_UNIVERSE 0x3b155311 /* Aureon 7.1 Universe */ #define VT1724_SUBDEVICE_PRODIGY71 0x33495345 /* PRODIGY 7.1 */ #define VT1724_SUBDEVICE_PRODIGY71LT 0x32315441 /* PRODIGY 7.1 LT */ +#define VT1724_SUBDEVICE_PRODIGY71HIFI 0x38315441 /* PRODIGY 7.1 HiFi */ extern struct snd_ice1712_card_info snd_vt1724_aureon_cards[]; [-- Attachment #3: Type: text/plain, Size: 347 bytes --] ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV [-- Attachment #4: Type: text/plain, Size: 161 bytes --] _______________________________________________ Alsa-devel mailing list Alsa-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/alsa-devel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Alsa-devel] A: VIA Technologies Inc. VT1720/24 2007-01-24 12:10 ` A: VIA Technologies Inc. VT1720/24 Philipp Matthias Hahn @ 2007-01-24 15:12 ` Takashi Iwai 2007-01-25 10:13 ` Philipp Matthias Hahn 0 siblings, 1 reply; 15+ messages in thread From: Takashi Iwai @ 2007-01-24 15:12 UTC (permalink / raw) To: Philipp Matthias Hahn; +Cc: ALSA users, ALSA developers At Wed, 24 Jan 2007 13:10:40 +0100, Philipp Matthias Hahn wrote: > > From: Philipp Matthias Hahn <pmhahn@pmhahn.de> > > Fix small typo in comment of Prodigy 7.1 Light > Add support for AudioTrak Prodigt 7.1 HiFi > > Signed-of-by: Philipp Matthias Hahn <pmhahn@pmhahn.de> (snip) > /* entry point */ > struct snd_ice1712_card_info snd_vt1724_aureon_cards[] __devinitdata = { > @@ -2153,5 +2169,15 @@ struct snd_ice1712_card_info snd_vt1724_ > .eeprom_data = prodigy71lt_eeprom, > .driver = "Prodigy71LT", > }, > + { > + .subvendor = VT1724_SUBDEVICE_PRODIGY71HIFI, > + .name = "Audiotrak Prodigy 7.1 HiFi", > + .model = "prodigy71hifi", > + .chip_init = aureon_init, > + .build_controls = aureon_add_controls, > + .eeprom_size = sizeof(prodigy71hifi_eeprom), > + .eeprom_data = prodigy71hifi_eeprom, > + .driver = "Prodigy71HiFi", /* should be identical with Prodigy71 */ I recommend you to use the same string for this driver name here. This string is referred by alsa-lib to pick up the corresponding config file. As Prodigy 7.1 Hifi is almost identical with Prodigy 7.1 LT from the configuration POV, it should use "Prodigy71LT", too, so that you don't have to change alsa-lib at all. Otherwise changes look OK to me, but your patch conflicts with the latest ALSA HG tree because of the patch applied prior to this. Could you regenerate the patch to ALSA HG tree (with the fix above)? thanks, Takashi ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-24 15:12 ` [Alsa-devel] " Takashi Iwai @ 2007-01-25 10:13 ` Philipp Matthias Hahn 2007-01-25 11:19 ` Takashi Iwai ` (2 more replies) 0 siblings, 3 replies; 15+ messages in thread From: Philipp Matthias Hahn @ 2007-01-25 10:13 UTC (permalink / raw) To: Takashi Iwai; +Cc: ALSA developers Hello Takashi! On Wed, Jan 24, 2007 at 04:12:02PM +0100, Takashi Iwai wrote: > I recommend you to use the same string for this driver name here. Yes. > This string is referred by alsa-lib to pick up the corresponding > config file. As Prodigy 7.1 Hifi is almost identical with Prodigy 7.1 > LT from the configuration POV, it should use "Prodigy71LT", too, so > that you don't have to change alsa-lib at all. No, the HiFi seems to be more like the Prodigy71 (without 'LT' or 'XT') But I might err. Is there something I can do to test is it has 3 or 4 DACs, which seems to be the main difference? > Otherwise changes look OK to me, but your patch conflicts with the > latest ALSA HG tree because of the patch applied prior to this. > Could you regenerate the patch to ALSA HG tree (with the fix above)? Yes, see below. Three more questions: 1. Since the "Aureon7.1", "Prodigy7.1" and my "Prodigy7.1HiFi"; as well as the "Prodigy7.1LT" and the "Prodigy7.1XT" respectively seem so share the same EEPROM-Date, wouldn't it be better to re-use/share those eeprom-data instead of declaring them 3 respectively 2 times? Not much space saving, but gcc seems to be too stupid to recognize this; Even const'ifying the structures doesn't help. 2. Most arrays and structures seem to be read only, what about adding 'const' to them? 3. Do you prefer array initialization in the form of {1, 2, 3, } /* IDX0, IDX1, IDX2 */ or as { [IDX0] = 1, [IDX1] = 2, [IDX2] = 3, } > thanks, You're doing the main job, I have to thank you. > Takashi Philipp From: Philipp Matthias Hahn <pmhahn@pmhahn.de> Fix small typo in comment of Prodigy 7.1 Light/XT Add support for AudioTrak Prodigy 7.1 HiFi Signed-of-by: Philipp Matthias Hahn <pmhahn@pmhahn.de> --- --- pci/ice1712/aureon.c~ 2007-01-25 10:07:50.000000000 +0100 +++ pci/ice1712/aureon.c 2007-01-25 10:46:56.000000000 +0100 @@ -2141,6 +2141,9 @@ static unsigned char aureon71_eeprom[] _ 0x00, /* GPIO_STATE2 */ }; +#if 1 /* Prodigy7.1 has same EEPROM as Aureon7.1 */ +#define prodigy71_eeprom aureon71_eeprom +#else static unsigned char prodigy71_eeprom[] __devinitdata = { 0x0b, /* SYSCONF: clock 512, spdif-in/ADC, 4DACs */ 0x80, /* ACLINK: I2S */ @@ -2156,9 +2159,10 @@ static unsigned char prodigy71_eeprom[] 0x00, /* GPIO_STATE1 */ 0x00, /* GPIO_STATE2 */ }; +#endif static unsigned char prodigy71lt_eeprom[] __devinitdata = { - 0x4b, /* SYSCINF: clock 512, spdif-in/ADC, 4DACs */ + 0x4b, /* SYSCONF: clock 384, spdif-in/ADC, 4DACs */ 0x80, /* ACLINK: I2S */ 0xfc, /* I2S: vol, 96k, 24bit, 192k */ 0xc3, /* SPDIF: out-en, out-int, spdif-in */ @@ -2173,8 +2177,11 @@ static unsigned char prodigy71lt_eeprom[ 0x00, /* GPIO_STATE2 */ }; +#if 1 /* Prodigy7.1XT has same EEPROM as Prodigy7.1LT */ +#define prodigy71xt_eeprom prodigy71lt_eeprom +#else static unsigned char prodigy71xt_eeprom[] __devinitdata = { - 0x4b, /* SYSCINF: clock 512, spdif-in/ADC, 4DACs */ + 0x4b, /* SYSCONF: clock 384, spdif-in/ADC, 4DACs */ 0x80, /* ACLINK: I2S */ 0xfc, /* I2S: vol, 96k, 24bit, 192k */ 0xc3, /* SPDIF: out-en, out-int, spdif-in */ @@ -2188,6 +2195,27 @@ static unsigned char prodigy71xt_eeprom[ 0x00, /* GPIO_STATE1 */ 0x00, /* GPIO_STATE2 */ }; +#endif + +#if 1 /* Prodigy7.1HiFi has same EEPROM as Aureon7.1 */ +#define prodigy71hifi_eeprom aureon71_eeprom +#else +static unsigned char prodigy71hifi_eeprom[] __devinitdata = { + [ICE_EEP2_SYSCONF] = 0x0b, /* clock 512, spdif-in/ADC, 4DACs */ + [ICE_EEP2_ACLINK] = 0x80, /* I2S */ + [ICE_EEP2_I2S] = 0xfc, /* vol, 96k, 24bit, 192k */ + [ICE_EEP2_SPDIF] = 0xc3, /* out-en, out-int, spdif-in */ + [ICE_EEP2_GPIO_DIR] = 0xff, + [ICE_EEP2_GPIO_DIR1] = 0xff, + [ICE_EEP2_GPIO_DIR2] = 0x5f, + [ICE_EEP2_GPIO_MASK] = 0x00, + [ICE_EEP2_GPIO_MASK1] = 0x00, + [ICE_EEP2_GPIO_MASK2] = 0x00, + [ICE_EEP2_GPIO_STATE] = 0x00, + [ICE_EEP2_GPIO_STATE1] = 0x00, + [ICE_EEP2_GPIO_STATE2] = 0x00, +}; +#endif /* entry point */ struct snd_ice1712_card_info snd_vt1724_aureon_cards[] __devinitdata = { @@ -2251,5 +2279,15 @@ struct snd_ice1712_card_info snd_vt1724_ .eeprom_data = prodigy71xt_eeprom, .driver = "Prodigy71LT", }, + { + .subvendor = VT1724_SUBDEVICE_PRODIGY71HIFI, + .name = "Audiotrak Prodigy 7.1 HiFi", + .model = "prodigy71hifi", + .chip_init = aureon_init, + .build_controls = aureon_add_controls, + .eeprom_size = sizeof(prodigy71hifi_eeprom), + .eeprom_data = prodigy71hifi_eeprom, + .driver = "Prodigy71", /* should be identical with Aureon71 */ + }, { } /* terminator */ }; --- pci/ice1712/aureon.h~ 2007-01-25 10:07:50.000000000 +0100 +++ pci/ice1712/aureon.h 2007-01-25 10:20:04.000000000 +0100 @@ -29,7 +29,8 @@ "{Terratec,Aureon 7.1 Universe}," \ "{AudioTrak,Prodigy 7.1}," \ "{AudioTrak,Prodigy 7.1 LT},"\ - "{AudioTrak,Prodigy 7.1 XT}," + "{AudioTrak,Prodigy 7.1 XT},"\ + "{AudioTrak,Prodigy 7.1 HiFi}," #define VT1724_SUBDEVICE_AUREON51_SKY 0x3b154711 /* Aureon 5.1 Sky */ #define VT1724_SUBDEVICE_AUREON71_SPACE 0x3b154511 /* Aureon 7.1 Space */ @@ -37,6 +38,7 @@ #define VT1724_SUBDEVICE_PRODIGY71 0x33495345 /* PRODIGY 7.1 */ #define VT1724_SUBDEVICE_PRODIGY71LT 0x32315441 /* PRODIGY 7.1 LT */ #define VT1724_SUBDEVICE_PRODIGY71XT 0x36315441 /* PRODIGY 7.1 XT*/ +#define VT1724_SUBDEVICE_PRODIGY71HIFI 0x38315441 /* PRODIGY 7.1 HiFi */ extern struct snd_ice1712_card_info snd_vt1724_aureon_cards[]; -- / / (_)__ __ ____ __ Philipp Hahn / /__/ / _ \/ // /\ \/ / /____/_/_//_/\_,_/ /_/\_\ pmhahn@titan.lahn.de ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-25 10:13 ` Philipp Matthias Hahn @ 2007-01-25 11:19 ` Takashi Iwai [not found] ` <20070126202950.GA5721@titan.lahn.de> 2007-01-25 12:36 ` Konstantin Kletschke 2007-01-25 19:58 ` Konstantin Kletschke 2 siblings, 1 reply; 15+ messages in thread From: Takashi Iwai @ 2007-01-25 11:19 UTC (permalink / raw) To: Philipp Matthias Hahn; +Cc: ALSA developers At Thu, 25 Jan 2007 11:13:19 +0100, Philipp Matthias Hahn wrote: > > Hello Takashi! > > On Wed, Jan 24, 2007 at 04:12:02PM +0100, Takashi Iwai wrote: > > I recommend you to use the same string for this driver name here. > > Yes. > > > This string is referred by alsa-lib to pick up the corresponding > > config file. As Prodigy 7.1 Hifi is almost identical with Prodigy 7.1 > > LT from the configuration POV, it should use "Prodigy71LT", too, so > > that you don't have to change alsa-lib at all. > > No, the HiFi seems to be more like the Prodigy71 (without 'LT' or 'XT') > But I might err. Is there something I can do to test is it has 3 or 4 > DACs, which seems to be the main difference? Well, with our without LT doesn't matter because both are aliased to Aureon71 in alsa-lib/src/conf/cards/alias.conf. > > Otherwise changes look OK to me, but your patch conflicts with the > > latest ALSA HG tree because of the patch applied prior to this. > > Could you regenerate the patch to ALSA HG tree (with the fix above)? > > Yes, see below. Three more questions: > 1. Since the "Aureon7.1", "Prodigy7.1" and my "Prodigy7.1HiFi"; as well > as the "Prodigy7.1LT" and the "Prodigy7.1XT" respectively seem so share > the same EEPROM-Date, wouldn't it be better to re-use/share those > eeprom-data instead of declaring them 3 respectively 2 times? > Not much space saving, but gcc seems to be too stupid to recognize this; > Even const'ifying the structures doesn't help. Good point. It's fine to merge them, of course. > 2. Most arrays and structures seem to be read only, what about adding > 'const' to them? Yes, it's better. > 3. Do you prefer array initialization in the form of > {1, 2, 3, } /* IDX0, IDX1, IDX2 */ > or as > { [IDX0] = 1, [IDX1] = 2, [IDX2] = 3, } I don't care about this so much. The former style has advantage that you can copy the byte stream as it is while the latter is a better coding style in general. Though, if we change these things at once, I'd take the latter style now. IMO, it'd be better to split the changes to two patches, one for cleaning up the above (merge to a single struct, add const and change the struct definition), and another to add 7.1 Hifi definition. Could you work on these two patches? thanks, Takashi ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
[parent not found: <20070126202950.GA5721@titan.lahn.de>]
* Re: A: VIA Technologies Inc. VT1720/24 [not found] ` <20070126202950.GA5721@titan.lahn.de> @ 2007-01-29 14:41 ` Takashi Iwai 2007-01-29 15:30 ` Konstantin Kletschke 2007-02-14 15:30 ` Takashi Iwai 0 siblings, 2 replies; 15+ messages in thread From: Takashi Iwai @ 2007-01-29 14:41 UTC (permalink / raw) To: Philipp Matthias Hahn; +Cc: ALSA developers At Fri, 26 Jan 2007 21:29:50 +0100, Philipp Matthias Hahn wrote: > > Hello Takashi! > > On Thu, Jan 25, 2007 at 12:19:51PM +0100, Takashi Iwai wrote: > > IMO, it'd be better to split the changes to two patches, one for > > cleaning up the above (merge to a single struct, add const and change > > the struct definition), and another to add 7.1 Hifi definition. > > > > Could you work on these two patches? > > Attached are 5 patches: > 1. Reorganize existing eeprom data > 2. Add Prodigy 7.1 Hifi > 3. Add "const" to files in pci/ice1712/ > 4. Add some more "const", but needs changes in i2c/other/ak4* > 5. Add even more "const" to everything related to TLV. > > The patches apply in that order. Each step in between does compile > without warnings, but I only did test-run the version with all 5 patches > applied. > The first 2 are the important ones, the other three are more or less > cosmetic. Thanks for the patches. Now I applied 4 clean up patches (and fixed some warnings), committed to HG tree. But the patch for Prodigy 7.1 Hifi is still pending since another deveopment by Konstantin is ongoing, too. Could you both of you cooperate to make more solid support for that hardware? Of course, I can take one of your patches as a prototype, and make it merged to the standard tree as a start up. Anyway, I'd like hear the opinions from both of you at first. thanks, Takashi ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-29 14:41 ` Takashi Iwai @ 2007-01-29 15:30 ` Konstantin Kletschke 2007-01-29 15:33 ` Konstantin Kletschke 2007-01-29 15:43 ` A: VIA Technologies Inc. VT1720/24 Takashi Iwai 2007-02-14 15:30 ` Takashi Iwai 1 sibling, 2 replies; 15+ messages in thread From: Konstantin Kletschke @ 2007-01-29 15:30 UTC (permalink / raw) To: ALSA developers I am here *wink* :-) Am 2007-01-29 15:41 +0100 schrieb Takashi Iwai: > But the patch for Prodigy 7.1 Hifi is still pending since another > deveopment by Konstantin is ongoing, too. Could you both of you > cooperate to make more solid support for that hardware? Well, I strongly recommend. As I asked before, I am still wondering how the initial patch works on Philipps Computer. While the wm8776 is implementend as a SPI device in aureon.c (and used this way in the Prodigy 7.1 Hifi driver in aureon.c here) I use it in i2c mode with success. What still missing is spi read/write stuff for the wm8766 I am currently working on. Also there is a cs8415a in i2c mode connected, whereas aureon.c expects it in SPI mode currently. Due to that essential differences I started a (quite good working!) skeleton based on a Hercules fortissmo4 approach I discovered in bugtrack.alsa-project.org. There I went into a prodigy hifi thread https://bugtrack.alsa-project.org/alsa-bug/view.php?id=2384 and put in my prodigy_hifi.c (which envolved meanwhile) for discussion. > Of course, I can take one of your patches as a prototype, and make > it I really wonder how/if Philipps stuff works. It gives a working card, but (as I expected) the main volume mixers are not working. All essential chips known to be connected in SPI mode need to be initialized to speak i2c on this board. > merged to the standard tree as a start up. Anyway, I'd like hear the > opinions from both of you at first. May be Philps patch needs only slight work, my approach of a completely additional file prodigy_hifi.c might be most probably work of greenhorn in hacking alsa device drivers, so I am here for discussion. At the moment I am trying to get hands onto the wm8766 chip, which needs proper SPI functions. I can nowhere adopt existing ones as far as I can see. Kind Regards, Konstantin Kletschke PS.: Takashi, the mail you are replying to, has this gone to this ml or to you directly? -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-29 15:30 ` Konstantin Kletschke @ 2007-01-29 15:33 ` Konstantin Kletschke 2007-01-29 15:55 ` Subject: Does anyone know if this chipsset VT82C686 will output 24 bit, 96 kHz to USB 1.1 or pcmcia card...using a Linux 2.4 platform marion rundell 2007-01-29 15:43 ` A: VIA Technologies Inc. VT1720/24 Takashi Iwai 1 sibling, 1 reply; 15+ messages in thread From: Konstantin Kletschke @ 2007-01-29 15:33 UTC (permalink / raw) To: ALSA developers Am 2007-01-29 16:30 +0100 schrieb Konstantin Kletschke: > May be Philps patch needs only slight work, my approach of a What I forgot to point out: It gives basic functionality but main volume sliders are not working. Sound is constant at moderate level. This is expected by me, since aureon.c seems to speak in spi mode to wm8776, it needs i2c functions. Konsti -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Subject: Does anyone know if this chipsset VT82C686 will output 24 bit, 96 kHz to USB 1.1 or pcmcia card...using a Linux 2.4 platform 2007-01-29 15:33 ` Konstantin Kletschke @ 2007-01-29 15:55 ` marion rundell 0 siblings, 0 replies; 15+ messages in thread From: marion rundell @ 2007-01-29 15:55 UTC (permalink / raw) To: Konstantin Kletschke, ALSA developers [-- Attachment #1.1: Type: text/plain, Size: 1549 bytes --] I am trying to determine if I can get spdif out of a system with the VT82C686 chipset. Does anyone know if this chipset is capable of outputting, on a Linux 2.4 platform...24bit, 96kHz bandwidth to a USB 1.1 or pcmcia card. I have only been able to get unreliable sound out so far. Marion mrundell@gmail.com ----- Original Message ----- From: Konstantin Kletschke To: ALSA developers Sent: Monday, January 29, 2007 9:33 AM Subject: Re: [Alsa-devel] A: VIA Technologies Inc. VT1720/24 Am 2007-01-29 16:30 +0100 schrieb Konstantin Kletschke: > May be Philps patch needs only slight work, my approach of a What I forgot to point out: It gives basic functionality but main volume sliders are not working. Sound is constant at moderate level. This is expected by me, since aureon.c seems to speak in spi mode to wm8776, it needs i2c functions. Konsti -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Alsa-devel mailing list Alsa-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/alsa-devel [-- Attachment #1.2: Type: text/html, Size: 3211 bytes --] [-- Attachment #2: Type: text/plain, Size: 347 bytes --] ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV [-- Attachment #3: Type: text/plain, Size: 161 bytes --] _______________________________________________ Alsa-devel mailing list Alsa-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/alsa-devel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-29 15:30 ` Konstantin Kletschke 2007-01-29 15:33 ` Konstantin Kletschke @ 2007-01-29 15:43 ` Takashi Iwai 2007-01-29 16:45 ` Konstantin Kletschke 1 sibling, 1 reply; 15+ messages in thread From: Takashi Iwai @ 2007-01-29 15:43 UTC (permalink / raw) To: Konstantin Kletschke; +Cc: ALSA developers At Mon, 29 Jan 2007 16:30:12 +0100, Konstantin Kletschke wrote: > > PS.: Takashi, the mail you are replying to, has this gone to this ml > or to you directly? It's directly to me, and looks like no post appearing on ML. Maybe it's filtered out due to too many attachments. FWIW, most of patches in that post have been already committed to ALSA HG tree, and only the addition of the new entry for 7.1 Hifi was missing. It's almost identical with what Philipp posted. Takashi ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-29 15:43 ` A: VIA Technologies Inc. VT1720/24 Takashi Iwai @ 2007-01-29 16:45 ` Konstantin Kletschke 0 siblings, 0 replies; 15+ messages in thread From: Konstantin Kletschke @ 2007-01-29 16:45 UTC (permalink / raw) To: ALSA developers Am 2007-01-29 16:43 +0100 schrieb Takashi Iwai: > It's directly to me, and looks like no post appearing on ML. Ok, so my setup os all right. I just wondered... > missing. It's almost identical with what Philipp posted. Okay, so lets see what he says about my questions :-) Konsti -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-29 14:41 ` Takashi Iwai 2007-01-29 15:30 ` Konstantin Kletschke @ 2007-02-14 15:30 ` Takashi Iwai 2007-02-14 15:49 ` Konstantin Kletschke 1 sibling, 1 reply; 15+ messages in thread From: Takashi Iwai @ 2007-02-14 15:30 UTC (permalink / raw) To: Philipp Matthias Hahn; +Cc: ALSA developers At Mon, 29 Jan 2007 15:41:53 +0100, I wrote: > > At Fri, 26 Jan 2007 21:29:50 +0100, > Philipp Matthias Hahn wrote: > > > > Hello Takashi! > > > > On Thu, Jan 25, 2007 at 12:19:51PM +0100, Takashi Iwai wrote: > > > IMO, it'd be better to split the changes to two patches, one for > > > cleaning up the above (merge to a single struct, add const and change > > > the struct definition), and another to add 7.1 Hifi definition. > > > > > > Could you work on these two patches? > > > > Attached are 5 patches: > > 1. Reorganize existing eeprom data > > 2. Add Prodigy 7.1 Hifi > > 3. Add "const" to files in pci/ice1712/ > > 4. Add some more "const", but needs changes in i2c/other/ak4* > > 5. Add even more "const" to everything related to TLV. > > > > The patches apply in that order. Each step in between does compile > > without warnings, but I only did test-run the version with all 5 patches > > applied. > > The first 2 are the important ones, the other three are more or less > > cosmetic. > > Thanks for the patches. Now I applied 4 clean up patches (and fixed > some warnings), committed to HG tree. > > But the patch for Prodigy 7.1 Hifi is still pending since another > deveopment by Konstantin is ongoing, too. Could you both of you > cooperate to make more solid support for that hardware? > > Of course, I can take one of your patches as a prototype, and make it > merged to the standard tree as a start up. Anyway, I'd like hear the > opinions from both of you at first. Ping. Just wondering if you have any progress wrt this hardware... Takashi ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-02-14 15:30 ` Takashi Iwai @ 2007-02-14 15:49 ` Konstantin Kletschke 0 siblings, 0 replies; 15+ messages in thread From: Konstantin Kletschke @ 2007-02-14 15:49 UTC (permalink / raw) To: ALSA developers; +Cc: pmhahn Am 2007-02-14 16:30 +0100 schrieb Takashi Iwai: > Ping. Pong :-) > Just wondering if you have any progress wrt this hardware... Well, I am still at it also did philip mail me, he is alive. I had so god damn much work recently I could not spend work into the prodigy hifi support (spi for wm8766 and i2c for cs8415a are TODO). Konsti -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-25 10:13 ` Philipp Matthias Hahn 2007-01-25 11:19 ` Takashi Iwai @ 2007-01-25 12:36 ` Konstantin Kletschke 2007-01-25 14:22 ` Konstantin Kletschke 2007-01-25 19:58 ` Konstantin Kletschke 2 siblings, 1 reply; 15+ messages in thread From: Konstantin Kletschke @ 2007-01-25 12:36 UTC (permalink / raw) To: alsa-devel Well actually I am fiddling around with the same card also. Since nobody responded here in this mailing list anymore I went into https://bugtrack.alsa-project.org/alsa-bug/view.php?id=2384 and evolved a separate prodigy_hifi.c which is quite redundant due to the simialirities but I have to get know to als coding and wanted to clean up later. This here looks clean and seems to work though. I went back here Raymond pointing me to the alsa-devel archive. This is the first mail which arrives at my mail account so I answer now here. I applied your stuff onto alsa-driver-1.0.14rc2. Am 2007-01-25 11:13 +0100 schrieb Philipp Matthias Hahn: > Yes, see below. Three more questions: > 1. Since the "Aureon7.1", "Prodigy7.1" and my "Prodigy7.1HiFi"; as well > as the "Prodigy7.1LT" and the "Prodigy7.1XT" respectively seem so share > the same EEPROM-Date, wouldn't it be better to re-use/share those > eeprom-data instead of declaring them 3 respectively 2 times? In my opinion Prodigy7.1HiFi eeprom_ data has to be slightly different, I need to put [ICE_EEP2_SYSCONF] = 0x4b instead of [ICE_EEP2_SYSCONF] = 0x0b Otherwise I got speeded up sound playback. See note (0014172) in this thread by me. What I am currently fiddling about is the CS8415A Chip: ALSA /XXX/alsa-kernel/pci/ice1712/aureon.c:1921: No CS8415 chip. Skipping CS8415 controls. In the note (0014176) I am trying to search it at the i2c address it should appear at. Still I did not found it :-) Regards, Konsti -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-25 12:36 ` Konstantin Kletschke @ 2007-01-25 14:22 ` Konstantin Kletschke 0 siblings, 0 replies; 15+ messages in thread From: Konstantin Kletschke @ 2007-01-25 14:22 UTC (permalink / raw) To: alsa-devel Am 2007-01-25 13:36 +0100 schrieb Konstantin Kletschke: > ALSA /XXX/alsa-kernel/pci/ice1712/aureon.c:1921: No CS8415 > chip. Skipping CS8415 controls. > > In the note (0014176) I am trying to search it at the i2c address it > should appear at. Still I did not found it :-) Okay, I got the CS8451 at the i2c bus, I had to left shift my bitmask to get the idea of how i2c addressing works: printk(KERN_DEBUG "addr_cs: %02x value 0x7F: %02x\n", addr_cs, snd_vt1724_read_i2c(ice, addr_cs, CS8415_ID)); printk(KERN_DEBUG "addr_cs: %02x value 0x7F: %02x\n", addr_cs, snd_vt1724_read_i2c(ice, addr_cs+1, CS8415_ID)); gives: i2c_read: [0x2e,0x7f] = 0x41 addr_cs: 2e value 0x7F: 41 i2c_read: [0x2f,0x7f] = 0x41 addr_cs: 2e value 0x7F: 41 Hmmm... -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: A: VIA Technologies Inc. VT1720/24 2007-01-25 10:13 ` Philipp Matthias Hahn 2007-01-25 11:19 ` Takashi Iwai 2007-01-25 12:36 ` Konstantin Kletschke @ 2007-01-25 19:58 ` Konstantin Kletschke 2 siblings, 0 replies; 15+ messages in thread From: Konstantin Kletschke @ 2007-01-25 19:58 UTC (permalink / raw) To: alsa-devel Now I inspected your approach more in detail. As I said things are working well, but other as my approach the volume sliders (Master and PCM) have no effect. They are moving but the sound keeps being at a moderate level (constant). I suppose the wm8776 is not addressed correctly, where does aureon.c expect it? I am still fiddling this out. Am 2007-01-25 11:13 +0100 schrieb Philipp Matthias Hahn: > > config file. As Prodigy 7.1 Hifi is almost identical with Prodigy > > 7.1 Are you both really shure? From code side of view the HiFi seems to be the first combination of vt1724 and wm8776 and 8766 and cs8415a. Philipp, is your prodigy 7.1 hifi really functional with your patch mailed in this thread? Mine still needs the other initialisation register value and cs8415a is not initialized here. As said, now I realize the main mixer don't have effect on output volume. Kind Regards, Konsti -- GPG KeyID EF62FCEF Fingerprint: 13C9 B16B 9844 EC15 CC2E A080 1E69 3FDA EF62 FCEF ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2007-02-14 15:49 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <45AFE785.6020307@informatik.uni-oldenburg.de> 2007-01-24 12:10 ` A: VIA Technologies Inc. VT1720/24 Philipp Matthias Hahn 2007-01-24 15:12 ` [Alsa-devel] " Takashi Iwai 2007-01-25 10:13 ` Philipp Matthias Hahn 2007-01-25 11:19 ` Takashi Iwai [not found] ` <20070126202950.GA5721@titan.lahn.de> 2007-01-29 14:41 ` Takashi Iwai 2007-01-29 15:30 ` Konstantin Kletschke 2007-01-29 15:33 ` Konstantin Kletschke 2007-01-29 15:55 ` Subject: Does anyone know if this chipsset VT82C686 will output 24 bit, 96 kHz to USB 1.1 or pcmcia card...using a Linux 2.4 platform marion rundell 2007-01-29 15:43 ` A: VIA Technologies Inc. VT1720/24 Takashi Iwai 2007-01-29 16:45 ` Konstantin Kletschke 2007-02-14 15:30 ` Takashi Iwai 2007-02-14 15:49 ` Konstantin Kletschke 2007-01-25 12:36 ` Konstantin Kletschke 2007-01-25 14:22 ` Konstantin Kletschke 2007-01-25 19:58 ` Konstantin Kletschke
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.