lkml.org 
[lkml]   [2004]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [Alsa-devel] [2.6 patch] ALSA PCI drivers: misc cleanups
At Mon, 22 Nov 2004 00:58:55 +0100,
Adrian Bunk wrote:
>
> The patch below does the following cleanups under sound/pci/ :
> - cs46xx/dsp_spos_scb_lib.c: cs46xx_dsp_create_filter_scb
> - cs46xx/dsp_spos_scb_lib.c: cs46xx_dsp_create_output_snoop_scb

I'd rather like to keep them with #if 0.
These functions will be needed when someone extends the DSP code.

> --- linux-2.6.10-rc2-mm2-full/sound/pci/ac97/ac97_local.h.old 2004-11-22 00:00:07.000000000 +0100
> +++ linux-2.6.10-rc2-mm2-full/sound/pci/ac97/ac97_local.h 2004-11-22 00:00:27.000000000 +0100
> @@ -45,9 +45,6 @@
> int snd_ac97_page_get_single(snd_kcontrol_t * kcontrol, snd_ctl_elem_value_t * ucontrol);
> int snd_ac97_page_put_single(snd_kcontrol_t * kcontrol, snd_ctl_elem_value_t * ucontrol);
> int snd_ac97_try_bit(ac97_t * ac97, int reg, int bit);
> -int snd_ac97_remove_ctl(ac97_t *ac97, const char *name, const char *suffix);
> -int snd_ac97_rename_ctl(ac97_t *ac97, const char *src, const char *dst, const char *suffix);
> -int snd_ac97_swap_ctl(ac97_t *ac97, const char *s1, const char *s2, const char *suffix);

They are intentionally non-static to allow ac97_patch.c functions to
access them (in future). Please keep them so.


Otherwise the patch looks fine.
Thanks!


Takashi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:08    [W:0.050 / U:1.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site