Commit 730a5865 authored by Jaroslav Kysela's avatar Jaroslav Kysela Committed by Takashi Iwai

ALSA: hdspm - remove unused arrays, reduce stack usage in hwdep_ioctl

Signed-off-by: default avatarJaroslav Kysela <perex@perex.cz>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 55a57606
...@@ -575,12 +575,6 @@ static char *texts_freq[] = { ...@@ -575,12 +575,6 @@ static char *texts_freq[] = {
"192 kHz" "192 kHz"
}; };
static char *texts_sync_status[] = {
"no lock",
"lock",
"sync"
};
static char *texts_ports_madi[] = { static char *texts_ports_madi[] = {
"MADI.1", "MADI.2", "MADI.3", "MADI.4", "MADI.5", "MADI.6", "MADI.1", "MADI.2", "MADI.3", "MADI.4", "MADI.5", "MADI.6",
"MADI.7", "MADI.8", "MADI.9", "MADI.10", "MADI.11", "MADI.12", "MADI.7", "MADI.8", "MADI.9", "MADI.10", "MADI.11", "MADI.12",
...@@ -692,28 +686,6 @@ static char channel_map_unity_ss[HDSPM_MAX_CHANNELS] = { ...@@ -692,28 +686,6 @@ static char channel_map_unity_ss[HDSPM_MAX_CHANNELS] = {
56, 57, 58, 59, 60, 61, 62, 63 56, 57, 58, 59, 60, 61, 62, 63
}; };
static char channel_map_unity_ds[HDSPM_MAX_CHANNELS] = {
0, 2, 4, 6, 8, 10, 12, 14,
16, 18, 20, 22, 24, 26, 28, 30,
32, 34, 36, 38, 40, 42, 44, 46,
48, 50, 52, 54, 56, 58, 60, 62,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
};
static char channel_map_unity_qs[HDSPM_MAX_CHANNELS] = {
0, 4, 8, 12, 16, 20, 24, 28,
32, 36, 40, 44, 48, 52, 56, 60,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
-1, -1, -1, -1, -1, -1, -1, -1,
};
static char channel_map_raydat_ss[HDSPM_MAX_CHANNELS] = { static char channel_map_raydat_ss[HDSPM_MAX_CHANNELS] = {
4, 5, 6, 7, 8, 9, 10, 11, /* ADAT 1 */ 4, 5, 6, 7, 8, 9, 10, 11, /* ADAT 1 */
12, 13, 14, 15, 16, 17, 18, 19, /* ADAT 2 */ 12, 13, 14, 15, 16, 17, 18, 19, /* ADAT 2 */
...@@ -952,6 +924,8 @@ struct hdspm { ...@@ -952,6 +924,8 @@ struct hdspm {
int texts_autosync_items; int texts_autosync_items;
cycles_t last_interrupt; cycles_t last_interrupt;
struct hdspm_peak_rms peak_rms;
}; };
...@@ -5952,7 +5926,7 @@ static int snd_hdspm_hwdep_ioctl(struct snd_hwdep *hw, struct file *file, ...@@ -5952,7 +5926,7 @@ static int snd_hdspm_hwdep_ioctl(struct snd_hwdep *hw, struct file *file,
struct hdspm_config info; struct hdspm_config info;
struct hdspm_status status; struct hdspm_status status;
struct hdspm_version hdspm_version; struct hdspm_version hdspm_version;
struct hdspm_peak_rms levels; struct hdspm_peak_rms *levels;
struct hdspm_ltc ltc; struct hdspm_ltc ltc;
unsigned int statusregister; unsigned int statusregister;
long unsigned int s; long unsigned int s;
...@@ -5961,28 +5935,29 @@ static int snd_hdspm_hwdep_ioctl(struct snd_hwdep *hw, struct file *file, ...@@ -5961,28 +5935,29 @@ static int snd_hdspm_hwdep_ioctl(struct snd_hwdep *hw, struct file *file,
switch (cmd) { switch (cmd) {
case SNDRV_HDSPM_IOCTL_GET_PEAK_RMS: case SNDRV_HDSPM_IOCTL_GET_PEAK_RMS:
levels = &hdspm->peak_rms;
for (i = 0; i < HDSPM_MAX_CHANNELS; i++) { for (i = 0; i < HDSPM_MAX_CHANNELS; i++) {
levels.input_peaks[i] = levels->input_peaks[i] =
readl(hdspm->iobase + readl(hdspm->iobase +
HDSPM_MADI_INPUT_PEAK + i*4); HDSPM_MADI_INPUT_PEAK + i*4);
levels.playback_peaks[i] = levels->playback_peaks[i] =
readl(hdspm->iobase + readl(hdspm->iobase +
HDSPM_MADI_PLAYBACK_PEAK + i*4); HDSPM_MADI_PLAYBACK_PEAK + i*4);
levels.output_peaks[i] = levels->output_peaks[i] =
readl(hdspm->iobase + readl(hdspm->iobase +
HDSPM_MADI_OUTPUT_PEAK + i*4); HDSPM_MADI_OUTPUT_PEAK + i*4);
levels.input_rms[i] = levels->input_rms[i] =
((uint64_t) readl(hdspm->iobase + ((uint64_t) readl(hdspm->iobase +
HDSPM_MADI_INPUT_RMS_H + i*4) << 32) | HDSPM_MADI_INPUT_RMS_H + i*4) << 32) |
(uint64_t) readl(hdspm->iobase + (uint64_t) readl(hdspm->iobase +
HDSPM_MADI_INPUT_RMS_L + i*4); HDSPM_MADI_INPUT_RMS_L + i*4);
levels.playback_rms[i] = levels->playback_rms[i] =
((uint64_t)readl(hdspm->iobase + ((uint64_t)readl(hdspm->iobase +
HDSPM_MADI_PLAYBACK_RMS_H+i*4) << 32) | HDSPM_MADI_PLAYBACK_RMS_H+i*4) << 32) |
(uint64_t)readl(hdspm->iobase + (uint64_t)readl(hdspm->iobase +
HDSPM_MADI_PLAYBACK_RMS_L + i*4); HDSPM_MADI_PLAYBACK_RMS_L + i*4);
levels.output_rms[i] = levels->output_rms[i] =
((uint64_t)readl(hdspm->iobase + ((uint64_t)readl(hdspm->iobase +
HDSPM_MADI_OUTPUT_RMS_H + i*4) << 32) | HDSPM_MADI_OUTPUT_RMS_H + i*4) << 32) |
(uint64_t)readl(hdspm->iobase + (uint64_t)readl(hdspm->iobase +
...@@ -5990,15 +5965,15 @@ static int snd_hdspm_hwdep_ioctl(struct snd_hwdep *hw, struct file *file, ...@@ -5990,15 +5965,15 @@ static int snd_hdspm_hwdep_ioctl(struct snd_hwdep *hw, struct file *file,
} }
if (hdspm->system_sample_rate > 96000) { if (hdspm->system_sample_rate > 96000) {
levels.speed = qs; levels->speed = qs;
} else if (hdspm->system_sample_rate > 48000) { } else if (hdspm->system_sample_rate > 48000) {
levels.speed = ds; levels->speed = ds;
} else { } else {
levels.speed = ss; levels->speed = ss;
} }
levels.status2 = hdspm_read(hdspm, HDSPM_statusRegister2); levels->status2 = hdspm_read(hdspm, HDSPM_statusRegister2);
s = copy_to_user(argp, &levels, sizeof(struct hdspm_peak_rms)); s = copy_to_user(argp, levels, sizeof(struct hdspm_peak_rms));
if (0 != s) { if (0 != s) {
/* snd_printk(KERN_ERR "copy_to_user(.., .., %lu): %lu /* snd_printk(KERN_ERR "copy_to_user(.., .., %lu): %lu
[Levels]\n", sizeof(struct hdspm_peak_rms), s); [Levels]\n", sizeof(struct hdspm_peak_rms), s);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment