Update of /cvsroot/alsa/alsa-kernel/pci/rme9652
In directory sc8-pr-cvs1:/tmp/cvs-serv20235

Modified Files:
        hdsp.c 
Log Message:
Thomas Charbonnel <[EMAIL PROTECTED]>:

The attached patch fixes matrix mixer and metering problems spotted by 
Pentti Ala-Vannesluoma for H9632 cards and gcc 2.9x compile errors 
reported by Martin Langer.


Index: hdsp.c
===================================================================
RCS file: /cvsroot/alsa/alsa-kernel/pci/rme9652/hdsp.c,v
retrieving revision 1.46
retrieving revision 1.47
diff -u -r1.46 -r1.47
--- hdsp.c      3 Nov 2003 18:24:53 -0000       1.46
+++ hdsp.c      7 Nov 2003 18:25:54 -0000       1.47
@@ -454,6 +454,7 @@
        unsigned short        state;                 /* stores state bits */
        u32                   firmware_cache[24413]; /* this helps recover from 
accidental iobox power failure */
        size_t                period_bytes;          /* guess what this is */
+       unsigned char         max_channels;
        unsigned char         qs_in_channels;        /* quad speed mode for H9632 */
        unsigned char         ds_in_channels;
        unsigned char         ss_in_channels;       /* different for 
multiface/digiface */
@@ -1169,11 +1170,11 @@
                /* set thru for all channels */
 
                if (enable) {
-                       for (i = 0; i < HDSP_MAX_CHANNELS; i++) {
+                       for (i = 0; i < hdsp->max_channels; i++) {
                                hdsp_write_gain (hdsp, 
hdsp_input_to_output_key(hdsp,i,i), UNITY_GAIN);
                        }
                } else {
-                       for (i = 0; i < HDSP_MAX_CHANNELS; i++) {
+                       for (i = 0; i < hdsp->max_channels; i++) {
                                hdsp_write_gain (hdsp, 
hdsp_input_to_output_key(hdsp,i,i), MINUS_INFINITY_GAIN);
                        }
                }
@@ -1181,7 +1182,7 @@
        } else {
                int mapped_channel;
 
-               snd_assert(channel < HDSP_MAX_CHANNELS, return);
+               snd_assert(channel < hdsp->max_channels, return);
 
                mapped_channel = hdsp->channel_map[channel];
 
@@ -2902,9 +2903,9 @@
 
        source = ucontrol->value.integer.value[0];
        destination = ucontrol->value.integer.value[1];
-
-       if (source > 25) {
-               addr = hdsp_playback_to_output_key(hdsp,source-26,destination);
+       
+       if (source >= hdsp->max_channels) {
+               addr = 
hdsp_playback_to_output_key(hdsp,source-hdsp->max_channels,destination);
        } else {
                addr = hdsp_input_to_output_key(hdsp,source, destination);
        }
@@ -2931,8 +2932,8 @@
        source = ucontrol->value.integer.value[0];
        destination = ucontrol->value.integer.value[1];
 
-       if (source > 25) {
-               addr = hdsp_playback_to_output_key(hdsp,source-26, destination);
+       if (source >= hdsp->max_channels) {
+               addr = hdsp_playback_to_output_key(hdsp,source-hdsp->max_channels, 
destination);
        } else {
                addr = hdsp_input_to_output_key(hdsp,source, destination);
        }
@@ -3177,7 +3178,7 @@
 
 int snd_hdsp_create_controls(snd_card_t *card, hdsp_t *hdsp)
 {
-       unsigned int idx, limit;
+       unsigned int idx;
        int err;
        snd_kcontrol_t *kctl;
 
@@ -3676,12 +3677,13 @@
                   odd numbered channels to right, even to left.
                */
                if (hdsp->io_type == H9632) {
-                       lineouts_base = 14;
+                       /* this is the phones/analog output */
+                       lineouts_base = 10;
                } else {
                        lineouts_base = 26;
                }
                
-               for (i = 0; i < HDSP_MAX_CHANNELS; i++) {
+               for (i = 0; i < hdsp->max_channels; i++) {
                        if (i & 1) { 
                                if (hdsp_write_gain (hdsp, hdsp_input_to_output_key 
(hdsp, i, lineouts_base), UNITY_GAIN) ||
                                    hdsp_write_gain (hdsp, hdsp_playback_to_output_key 
(hdsp, i, lineouts_base), UNITY_GAIN)) {
@@ -3793,7 +3795,7 @@
 {
        int mapped_channel;
 
-        snd_assert(channel >= 0 || channel < HDSP_MAX_CHANNELS, return NULL);
+        snd_assert(channel >= 0 || channel < hdsp->max_channels, return NULL);
         
        if ((mapped_channel = hdsp->channel_map[channel]) < 0) {
                return NULL;
@@ -3963,7 +3965,7 @@
        hdsp_t *hdsp = _snd_pcm_substream_chip(substream);
        int mapped_channel;
 
-       snd_assert(info->channel < HDSP_MAX_CHANNELS, return -EINVAL);
+       snd_assert(info->channel < hdsp->max_channels, return -EINVAL);
 
        if ((mapped_channel = hdsp->channel_map[info->channel]) < 0) {
                return -EINVAL;
@@ -4487,8 +4489,8 @@
 
 static int snd_hdsp_hwdep_dummy_op(snd_hwdep_t *hw, struct file *file)
 {
-    /* we have nothing to initialize but the call is required */
-    return 0;
+       /* we have nothing to initialize but the call is required */
+       return 0;
 }
 
 
@@ -4545,10 +4547,11 @@
                }
                if (hdsp->io_type == H9632) {
                        int j;
+                       hdsp_9632_meters_t *m;
                        int doublespeed = 0;
                        if (hdsp_read (hdsp, HDSP_statusRegister) & 
HDSP_DoubleSpeedStatus)
                                doublespeed = 1;
-                       hdsp_9632_meters_t *m = (hdsp_9632_meters_t 
*)hdsp->iobase+HDSP_9632_metersBase;
+                       m = (hdsp_9632_meters_t *)(hdsp->iobase+HDSP_9632_metersBase);
                        peak_rms = (hdsp_peak_rms_t *)arg;
                        for (i = 0, j = 0; i < 16; ++i, ++j) {
                                if (copy_to_user((void *)peak_rms->input_peaks+i*4, 
&(m->input_peak[j]), 4) != 0)
@@ -4814,7 +4817,7 @@
                return -EIO;
        }
        
-       for (i = 0; i < HDSP_MAX_CHANNELS; ++i) {
+       for (i = 0; i < hdsp->max_channels; ++i) {
                hdsp_write (hdsp, HDSP_inputEnable + (4 * i), 1);
                hdsp_write (hdsp, HDSP_outputEnable + (4 * i), 1);
        }
@@ -4841,8 +4844,9 @@
        
        case H9632:
                status = hdsp_read(hdsp, HDSP_statusRegister);
-               aebi_channels = (status & HDSP_AEBI) ? 4 : 0;
-               aebo_channels = (status & HDSP_AEBO) ? 4 : 0;
+               /* HDSP_AEBx bits are low when AEB are connected */
+               aebi_channels = (status & HDSP_AEBI) ? 0 : 4;
+               aebo_channels = (status & HDSP_AEBO) ? 0 : 4;
                hdsp->card_name = "RME Hammerfall HDSP 9632";
                hdsp->ss_in_channels = H9632_SS_CHANNELS+aebi_channels;
                hdsp->ds_in_channels = H9632_DS_CHANNELS+aebi_channels;
@@ -4929,7 +4933,6 @@
 {
        struct pci_dev *pci = hdsp->pci;
        int err;
-       int i;
        int is_9652 = 0;
        int is_9632 = 0;
 
@@ -4948,6 +4951,7 @@
        hdsp->control_register = 0;
        hdsp->control2_register = 0;
        hdsp->io_type = Undefined;
+       hdsp->max_channels = 26;
 
        hdsp->card = card;
        
@@ -4974,6 +4978,7 @@
                break;
        case 0x96:
                hdsp->card_name = "RME HDSP 9632";
+               hdsp->max_channels = 16;
                is_9632 = 1;
                break;
        default:



-------------------------------------------------------
This SF.Net email sponsored by: ApacheCon 2003,
16-19 November in Las Vegas. Learn firsthand the latest
developments in Apache, PHP, Perl, XML, Java, MySQL,
WebDAV, and more! http://www.apachecon.com/
_______________________________________________
Alsa-cvslog mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/alsa-cvslog

Reply via email to