Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a9c2dfc8 authored by Takashi Iwai's avatar Takashi Iwai
Browse files

ALSA: hda - Use a macro for snd_array iteration loops 



Introduce a new helper macro, snd_array_for_each(), to iterate for
each snd_array element.  It slightly improves the readability than
lengthy open codes at each place.

Along with it, add const prefix to some obvious places.

There should be no functional changes by this.

Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent f656891c

include/sound/hdaudio.h

+5 −0
Original line number Original line Diff line number Diff line
@@ -571,4 +571,9 @@ static inline unsigned int snd_array_index(struct snd_array *array, void *ptr) 
	return (unsigned long)(ptr - array->list) / array->elem_size;

	return (unsigned long)(ptr - array->list) / array->elem_size;

}

}





/* a helper macro to iterate for each snd_array element */

#define snd_array_for_each(array, idx, ptr) \

	for ((idx) = 0, (ptr) = (array)->list; (idx) < (array)->used; \

	     (ptr) = snd_array_elem(array, ++(idx)))



#endif /* __SOUND_HDAUDIO_H */
 
#endif /* __SOUND_HDAUDIO_H */

sound/hda/hdac_regmap.c

+2 −2
Original line number Original line Diff line number Diff line
@@ -65,10 +65,10 @@ static bool hda_writeable_reg(struct device *dev, unsigned int reg) 
{

{

	struct hdac_device *codec = dev_to_hdac_dev(dev);

	struct hdac_device *codec = dev_to_hdac_dev(dev);

	unsigned int verb = get_verb(reg);

	unsigned int verb = get_verb(reg);

	const unsigned int *v;

	int i;

	int i;





	for (i = 0; i < codec->vendor_verbs.used; i++) {

	snd_array_for_each(&codec->vendor_verbs, i, v) {

		unsigned int *v = snd_array_elem(&codec->vendor_verbs, i);

		if (verb == *v)

		if (verb == *v)

			return true;

			return true;

	}

	}

sound/pci/hda/hda_auto_parser.c

+5 −5
Original line number Original line Diff line number Diff line
@@ -793,12 +793,12 @@ EXPORT_SYMBOL_GPL(snd_hda_add_verbs); 
 */

 */

void snd_hda_apply_verbs(struct hda_codec *codec)

void snd_hda_apply_verbs(struct hda_codec *codec)

{

{

	const struct hda_verb **v;

	int i;

	int i;

	for (i = 0; i < codec->verbs.used; i++) {



		struct hda_verb **v = snd_array_elem(&codec->verbs, i);

	snd_array_for_each(&codec->verbs, i, v)

		snd_hda_sequence_write(codec, *v);

		snd_hda_sequence_write(codec, *v);

}

}

}

EXPORT_SYMBOL_GPL(snd_hda_apply_verbs);

EXPORT_SYMBOL_GPL(snd_hda_apply_verbs);





/**

/**
@@ -890,10 +890,10 @@ EXPORT_SYMBOL_GPL(snd_hda_apply_fixup); 
static bool pin_config_match(struct hda_codec *codec,

static bool pin_config_match(struct hda_codec *codec,

			     const struct hda_pintbl *pins)

			     const struct hda_pintbl *pins)

{

{

	const struct hda_pincfg *pin;

	int i;

	int i;





	for (i = 0; i < codec->init_pins.used; i++) {

	snd_array_for_each(&codec->init_pins, i, pin) {

		struct hda_pincfg *pin = snd_array_elem(&codec->init_pins, i);

		hda_nid_t nid = pin->nid;

		hda_nid_t nid = pin->nid;

		u32 cfg = pin->cfg;

		u32 cfg = pin->cfg;

		const struct hda_pintbl *t_pins;

		const struct hda_pintbl *t_pins;

sound/pci/hda/hda_codec.c

+18 −18
Original line number Original line Diff line number Diff line
@@ -481,9 +481,10 @@ static struct hda_pincfg *look_up_pincfg(struct hda_codec *codec, 
					 struct snd_array *array,

					 struct snd_array *array,

					 hda_nid_t nid)

					 hda_nid_t nid)

{

{

	struct hda_pincfg *pin;

	int i;

	int i;

	for (i = 0; i < array->used; i++) {



		struct hda_pincfg *pin = snd_array_elem(array, i);

	snd_array_for_each(array, i, pin) {

		if (pin->nid == nid)

		if (pin->nid == nid)

			return pin;

			return pin;

	}

	}
@@ -618,14 +619,15 @@ EXPORT_SYMBOL_GPL(snd_hda_codec_get_pin_target); 
 */

 */

void snd_hda_shutup_pins(struct hda_codec *codec)

void snd_hda_shutup_pins(struct hda_codec *codec)

{

{

	const struct hda_pincfg *pin;

	int i;

	int i;



	/* don't shut up pins when unloading the driver; otherwise it breaks

	/* don't shut up pins when unloading the driver; otherwise it breaks

	 * the default pin setup at the next load of the driver

	 * the default pin setup at the next load of the driver

	 */

	 */

	if (codec->bus->shutdown)

	if (codec->bus->shutdown)

		return;

		return;

	for (i = 0; i < codec->init_pins.used; i++) {

	snd_array_for_each(&codec->init_pins, i, pin) {

		struct hda_pincfg *pin = snd_array_elem(&codec->init_pins, i);

		/* use read here for syncing after issuing each verb */

		/* use read here for syncing after issuing each verb */

		snd_hda_codec_read(codec, pin->nid, 0,

		snd_hda_codec_read(codec, pin->nid, 0,

				   AC_VERB_SET_PIN_WIDGET_CONTROL, 0);

				   AC_VERB_SET_PIN_WIDGET_CONTROL, 0);
@@ -638,13 +640,14 @@ EXPORT_SYMBOL_GPL(snd_hda_shutup_pins); 
/* Restore the pin controls cleared previously via snd_hda_shutup_pins() */

/* Restore the pin controls cleared previously via snd_hda_shutup_pins() */

static void restore_shutup_pins(struct hda_codec *codec)

static void restore_shutup_pins(struct hda_codec *codec)

{

{

	const struct hda_pincfg *pin;

	int i;

	int i;



	if (!codec->pins_shutup)

	if (!codec->pins_shutup)

		return;

		return;

	if (codec->bus->shutdown)

	if (codec->bus->shutdown)

		return;

		return;

	for (i = 0; i < codec->init_pins.used; i++) {

	snd_array_for_each(&codec->init_pins, i, pin) {

		struct hda_pincfg *pin = snd_array_elem(&codec->init_pins, i);

		snd_hda_codec_write(codec, pin->nid, 0,

		snd_hda_codec_write(codec, pin->nid, 0,

				    AC_VERB_SET_PIN_WIDGET_CONTROL,

				    AC_VERB_SET_PIN_WIDGET_CONTROL,

				    pin->ctrl);

				    pin->ctrl);
@@ -697,8 +700,7 @@ get_hda_cvt_setup(struct hda_codec *codec, hda_nid_t nid) 
	struct hda_cvt_setup *p;

	struct hda_cvt_setup *p;

	int i;

	int i;





	for (i = 0; i < codec->cvt_setups.used; i++) {

	snd_array_for_each(&codec->cvt_setups, i, p) {

		p = snd_array_elem(&codec->cvt_setups, i);

		if (p->nid == nid)

		if (p->nid == nid)

			return p;

			return p;

	}

	}
@@ -1076,8 +1078,7 @@ void snd_hda_codec_setup_stream(struct hda_codec *codec, hda_nid_t nid, 
	/* make other inactive cvts with the same stream-tag dirty */

	/* make other inactive cvts with the same stream-tag dirty */

	type = get_wcaps_type(get_wcaps(codec, nid));

	type = get_wcaps_type(get_wcaps(codec, nid));

	list_for_each_codec(c, codec->bus) {

	list_for_each_codec(c, codec->bus) {

		for (i = 0; i < c->cvt_setups.used; i++) {

		snd_array_for_each(&c->cvt_setups, i, p) {

			p = snd_array_elem(&c->cvt_setups, i);

			if (!p->active && p->stream_tag == stream_tag &&

			if (!p->active && p->stream_tag == stream_tag &&

			    get_wcaps_type(get_wcaps(c, p->nid)) == type)

			    get_wcaps_type(get_wcaps(c, p->nid)) == type)

				p->dirty = 1;

				p->dirty = 1;
@@ -1140,12 +1141,11 @@ static void really_cleanup_stream(struct hda_codec *codec, 
static void purify_inactive_streams(struct hda_codec *codec)

static void purify_inactive_streams(struct hda_codec *codec)

{

{

	struct hda_codec *c;

	struct hda_codec *c;

	struct hda_cvt_setup *p;

	int i;

	int i;





	list_for_each_codec(c, codec->bus) {

	list_for_each_codec(c, codec->bus) {

		for (i = 0; i < c->cvt_setups.used; i++) {

		snd_array_for_each(&c->cvt_setups, i, p) {

			struct hda_cvt_setup *p;

			p = snd_array_elem(&c->cvt_setups, i);

			if (p->dirty)

			if (p->dirty)

				really_cleanup_stream(c, p);

				really_cleanup_stream(c, p);

		}

		}
@@ -1156,10 +1156,10 @@ static void purify_inactive_streams(struct hda_codec *codec) 
/* clean up all streams; called from suspend */

/* clean up all streams; called from suspend */

static void hda_cleanup_all_streams(struct hda_codec *codec)

static void hda_cleanup_all_streams(struct hda_codec *codec)

{

{

	struct hda_cvt_setup *p;

	int i;

	int i;





	for (i = 0; i < codec->cvt_setups.used; i++) {

	snd_array_for_each(&codec->cvt_setups, i, p) {

		struct hda_cvt_setup *p = snd_array_elem(&codec->cvt_setups, i);

		if (p->stream_tag)

		if (p->stream_tag)

			really_cleanup_stream(codec, p);

			really_cleanup_stream(codec, p);

	}

	}
@@ -2461,10 +2461,10 @@ EXPORT_SYMBOL_GPL(snd_hda_create_dig_out_ctls); 
struct hda_spdif_out *snd_hda_spdif_out_of_nid(struct hda_codec *codec,

struct hda_spdif_out *snd_hda_spdif_out_of_nid(struct hda_codec *codec,

					       hda_nid_t nid)

					       hda_nid_t nid)

{

{

	struct hda_spdif_out *spdif;

	int i;

	int i;

	for (i = 0; i < codec->spdif_out.used; i++) {



		struct hda_spdif_out *spdif =

	snd_array_for_each(&codec->spdif_out, i, spdif) {

				snd_array_elem(&codec->spdif_out, i);

		if (spdif->nid == nid)

		if (spdif->nid == nid)

			return spdif;

			return spdif;

	}

	}

sound/pci/hda/hda_generic.c

+13 −14
Original line number Original line Diff line number Diff line
@@ -264,10 +264,10 @@ static struct nid_path *get_nid_path(struct hda_codec *codec, 
				     int anchor_nid)

				     int anchor_nid)

{

{

	struct hda_gen_spec *spec = codec->spec;

	struct hda_gen_spec *spec = codec->spec;

	struct nid_path *path;

	int i;

	int i;





	for (i = 0; i < spec->paths.used; i++) {

	snd_array_for_each(&spec->paths, i, path) {

		struct nid_path *path = snd_array_elem(&spec->paths, i);

		if (path->depth <= 0)

		if (path->depth <= 0)

			continue;

			continue;

		if ((!from_nid || path->path[0] == from_nid) &&

		if ((!from_nid || path->path[0] == from_nid) &&
@@ -325,10 +325,10 @@ EXPORT_SYMBOL_GPL(snd_hda_get_path_from_idx); 
static bool is_dac_already_used(struct hda_codec *codec, hda_nid_t nid)

static bool is_dac_already_used(struct hda_codec *codec, hda_nid_t nid)

{

{

	struct hda_gen_spec *spec = codec->spec;

	struct hda_gen_spec *spec = codec->spec;

	const struct nid_path *path;

	int i;

	int i;





	for (i = 0; i < spec->paths.used; i++) {

	snd_array_for_each(&spec->paths, i, path) {

		struct nid_path *path = snd_array_elem(&spec->paths, i);

		if (path->path[0] == nid)

		if (path->path[0] == nid)

			return true;

			return true;

	}

	}
@@ -351,11 +351,11 @@ static bool is_reachable_path(struct hda_codec *codec, 
static bool is_ctl_used(struct hda_codec *codec, unsigned int val, int type)

static bool is_ctl_used(struct hda_codec *codec, unsigned int val, int type)

{

{

	struct hda_gen_spec *spec = codec->spec;

	struct hda_gen_spec *spec = codec->spec;

	const struct nid_path *path;

	int i;

	int i;





	val &= AMP_VAL_COMPARE_MASK;

	val &= AMP_VAL_COMPARE_MASK;

	for (i = 0; i < spec->paths.used; i++) {

	snd_array_for_each(&spec->paths, i, path) {

		struct nid_path *path = snd_array_elem(&spec->paths, i);

		if ((path->ctls[type] & AMP_VAL_COMPARE_MASK) == val)

		if ((path->ctls[type] & AMP_VAL_COMPARE_MASK) == val)

			return true;

			return true;

	}

	}
@@ -638,13 +638,13 @@ static bool is_active_nid(struct hda_codec *codec, hda_nid_t nid, 
{

{

	struct hda_gen_spec *spec = codec->spec;

	struct hda_gen_spec *spec = codec->spec;

	int type = get_wcaps_type(get_wcaps(codec, nid));

	int type = get_wcaps_type(get_wcaps(codec, nid));

	const struct nid_path *path;

	int i, n;

	int i, n;





	if (nid == codec->core.afg)

	if (nid == codec->core.afg)

		return true;

		return true;





	for (n = 0; n < spec->paths.used; n++) {

	snd_array_for_each(&spec->paths, n, path) {

		struct nid_path *path = snd_array_elem(&spec->paths, n);

		if (!path->active)

		if (!path->active)

			continue;

			continue;

		if (codec->power_save_node) {

		if (codec->power_save_node) {
@@ -2696,10 +2696,10 @@ static const struct snd_kcontrol_new out_jack_mode_enum = { 
static bool find_kctl_name(struct hda_codec *codec, const char *name, int idx)

static bool find_kctl_name(struct hda_codec *codec, const char *name, int idx)

{

{

	struct hda_gen_spec *spec = codec->spec;

	struct hda_gen_spec *spec = codec->spec;

	const struct snd_kcontrol_new *kctl;

	int i;

	int i;





	for (i = 0; i < spec->kctls.used; i++) {

	snd_array_for_each(&spec->kctls, i, kctl) {

		struct snd_kcontrol_new *kctl = snd_array_elem(&spec->kctls, i);

		if (!strcmp(kctl->name, name) && kctl->index == idx)

		if (!strcmp(kctl->name, name) && kctl->index == idx)

			return true;

			return true;

	}

	}
@@ -4021,8 +4021,7 @@ static hda_nid_t set_path_power(struct hda_codec *codec, hda_nid_t nid, 
	struct nid_path *path;

	struct nid_path *path;

	int n;

	int n;





	for (n = 0; n < spec->paths.used; n++) {

	snd_array_for_each(&spec->paths, n, path) {

		path = snd_array_elem(&spec->paths, n);

		if (!path->depth)

		if (!path->depth)

			continue;

			continue;

		if (path->path[0] == nid ||

		if (path->path[0] == nid ||
@@ -5831,10 +5830,10 @@ static void init_digital(struct hda_codec *codec) 
 */

 */

static void clear_unsol_on_unused_pins(struct hda_codec *codec)

static void clear_unsol_on_unused_pins(struct hda_codec *codec)

{

{

	const struct hda_pincfg *pin;

	int i;

	int i;





	for (i = 0; i < codec->init_pins.used; i++) {

	snd_array_for_each(&codec->init_pins, i, pin) {

		struct hda_pincfg *pin = snd_array_elem(&codec->init_pins, i);

		hda_nid_t nid = pin->nid;

		hda_nid_t nid = pin->nid;

		if (is_jack_detectable(codec, nid) &&

		if (is_jack_detectable(codec, nid) &&

		    !snd_hda_jack_tbl_get(codec, nid))

		    !snd_hda_jack_tbl_get(codec, nid))