lkml.org 
[lkml]   [2004]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [Alsa-devel] [2.6 patch] ALSA core: misc cleanups
On Fri, Nov 26, 2004 at 11:38:07AM +0100, Takashi Iwai wrote:
> At Fri, 26 Nov 2004 01:24:48 +0100,
> Adrian Bunk wrote:
> >
> > The patch below does the following cleanups under sound/core/ :
> > - make needlessly global code static
> > - remove the following stale prototypes from pcm.h
> > (the functions are not or no longer present):
> > - snd_pcm_capture_ready_jiffies
> > - snd_pcm_playback_ready_jiffies
> > - remove the following unused global funxtions:
> > - oss/pcm_plugin.c: snd_pcm_plug_capture_channels_mask
> > - pcm_lib.c: snd_pcm_playback_ready
> > - pcm_lib.c: snd_pcm_capture_ready
> > - pcm_lib.c: snd_pcm_capture_empty
> > - pcm_misc.c: snd_pcm_format_cpu_endian
> > - pcm_misc.c: snd_pcm_format_size
> > - seq/seq_instr.c: snd_seq_cluster_new
> > - seq/seq_instr.c: snd_seq_cluster_free
> > - seq/seq_midi_event.c: snd_midi_event_init
> > - seq/seq_midi_event.c: snd_midi_event_resize_buffer
> > - seq/seq_virmidi.c: snd_virmidi_receive
> > - remove the following unused EXPORT_SYMBOL's:
> > - snd_create_proc_entry
> > - snd_interval_ratden
> > - snd_midi_channel_init
> > - snd_midi_channel_init_set
> > - snd_midi_event_init
> > - snd_midi_event_resize_buffer
> > - snd_pcm_capture_empty
> > - snd_pcm_capture_poll
> > - snd_pcm_capture_ready
> > - snd_pcm_format_size
> > - snd_pcm_lib_preallocate_free
> > - snd_pcm_open
> > - snd_pcm_playback_poll
> > - snd_pcm_playback_ready
> > - snd_pcm_release
> > - snd_pcm_subformat_name
> > - snd_pcm_suspend
> > - snd_rawmidi_drain_input
> > - snd_rawmidi_drop_output
> > - snd_rawmidi_info
> > - snd_remove_proc_entry
> > - snd_timer_continue
> > - snd_timer_system_resolution
> > - snd_virmidi_receive
>
> I disagree to remove all of these functions from the middle layer.
> Some of them could be reduced, but others were once used, and might be
> used in future, too.

The intention is not to blindly remove code you might need in the near
future.

Can you comment on which of them are actually candidates for being
removed? I'll then send you an updated patch.

> Takashi

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
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.041 / U:0.748 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site