Commit 071f1344 authored by Julia Lawall's avatar Julia Lawall Committed by Takashi Iwai

ALSA: pci: constify local structures

For structure types defined in the same file or local header files, find
top-level static structure declarations that have the following
properties:
1. Never reassigned.
2. Address never taken
3. Not passed to a top-level macro call
4. No pointer or array-typed field passed to a function or stored in a
variable.
Declare structures having all of these properties as const.

Done using Coccinelle.
Based on a suggestion by Joe Perches <joe@perches.com>.
Signed-off-by: default avatarJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 3d2f4d0c
...@@ -1623,7 +1623,7 @@ static int atc_resume(struct ct_atc *atc) ...@@ -1623,7 +1623,7 @@ static int atc_resume(struct ct_atc *atc)
} }
#endif #endif
static struct ct_atc atc_preset = { static const struct ct_atc atc_preset = {
.map_audio_buffer = ct_map_audio_buffer, .map_audio_buffer = ct_map_audio_buffer,
.unmap_audio_buffer = ct_unmap_audio_buffer, .unmap_audio_buffer = ct_unmap_audio_buffer,
.pcm_playback_prepare = atc_pcm_playback_prepare, .pcm_playback_prepare = atc_pcm_playback_prepare,
......
...@@ -4018,7 +4018,7 @@ static int ca0132_build_controls(struct hda_codec *codec) ...@@ -4018,7 +4018,7 @@ static int ca0132_build_controls(struct hda_codec *codec)
/* /*
* PCM * PCM
*/ */
static struct hda_pcm_stream ca0132_pcm_analog_playback = { static const struct hda_pcm_stream ca0132_pcm_analog_playback = {
.substreams = 1, .substreams = 1,
.channels_min = 2, .channels_min = 2,
.channels_max = 6, .channels_max = 6,
...@@ -4029,7 +4029,7 @@ static struct hda_pcm_stream ca0132_pcm_analog_playback = { ...@@ -4029,7 +4029,7 @@ static struct hda_pcm_stream ca0132_pcm_analog_playback = {
}, },
}; };
static struct hda_pcm_stream ca0132_pcm_analog_capture = { static const struct hda_pcm_stream ca0132_pcm_analog_capture = {
.substreams = 1, .substreams = 1,
.channels_min = 2, .channels_min = 2,
.channels_max = 2, .channels_max = 2,
...@@ -4040,7 +4040,7 @@ static struct hda_pcm_stream ca0132_pcm_analog_capture = { ...@@ -4040,7 +4040,7 @@ static struct hda_pcm_stream ca0132_pcm_analog_capture = {
}, },
}; };
static struct hda_pcm_stream ca0132_pcm_digital_playback = { static const struct hda_pcm_stream ca0132_pcm_digital_playback = {
.substreams = 1, .substreams = 1,
.channels_min = 2, .channels_min = 2,
.channels_max = 2, .channels_max = 2,
...@@ -4052,7 +4052,7 @@ static struct hda_pcm_stream ca0132_pcm_digital_playback = { ...@@ -4052,7 +4052,7 @@ static struct hda_pcm_stream ca0132_pcm_digital_playback = {
}, },
}; };
static struct hda_pcm_stream ca0132_pcm_digital_capture = { static const struct hda_pcm_stream ca0132_pcm_digital_capture = {
.substreams = 1, .substreams = 1,
.channels_min = 2, .channels_min = 2,
.channels_max = 2, .channels_max = 2,
...@@ -4614,7 +4614,7 @@ static void ca0132_free(struct hda_codec *codec) ...@@ -4614,7 +4614,7 @@ static void ca0132_free(struct hda_codec *codec)
kfree(codec->spec); kfree(codec->spec);
} }
static struct hda_codec_ops ca0132_patch_ops = { static const struct hda_codec_ops ca0132_patch_ops = {
.build_controls = ca0132_build_controls, .build_controls = ca0132_build_controls,
.build_pcms = ca0132_build_pcms, .build_pcms = ca0132_build_pcms,
.init = ca0132_init, .init = ca0132_init,
......
...@@ -644,7 +644,7 @@ static struct lbuspath lbus_play_paths[] = { ...@@ -644,7 +644,7 @@ static struct lbuspath lbus_play_paths[] = {
.mono = lbus_play_mono3, .mono = lbus_play_mono3,
}, },
}; };
static struct lbuspath lbus_rec_path = { static const struct lbuspath lbus_rec_path = {
.noconv = lbus_rec_noconv1, .noconv = lbus_rec_noconv1,
.stereo = lbus_rec_stereo1, .stereo = lbus_rec_stereo1,
.mono = lbus_rec_mono1, .mono = lbus_rec_mono1,
......
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