Use list_for_each_entry() helper instead of list_for_each() and list_entry() to simplify code a bit. No functional change.
Signed-off-by: Yang Yingliang yangyingliang@huawei.com --- sound/drivers/mtpav.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/sound/drivers/mtpav.c b/sound/drivers/mtpav.c index f212f233ea61..717577e2a609 100644 --- a/sound/drivers/mtpav.c +++ b/sound/drivers/mtpav.c @@ -623,7 +623,6 @@ static int snd_mtpav_get_RAWMIDI(struct mtpav *mcard) int rval; struct snd_rawmidi *rawmidi; struct snd_rawmidi_substream *substream; - struct list_head *list;
if (hwports < 1) hwports = 1; @@ -640,13 +639,15 @@ static int snd_mtpav_get_RAWMIDI(struct mtpav *mcard) rawmidi = mcard->rmidi; rawmidi->private_data = mcard;
- list_for_each(list, &rawmidi->streams[SNDRV_RAWMIDI_STREAM_INPUT].substreams) { - substream = list_entry(list, struct snd_rawmidi_substream, list); + list_for_each_entry(substream, + &rawmidi->streams[SNDRV_RAWMIDI_STREAM_INPUT].substreams, + list) { snd_mtpav_set_name(mcard, substream); substream->ops = &snd_mtpav_input; } - list_for_each(list, &rawmidi->streams[SNDRV_RAWMIDI_STREAM_OUTPUT].substreams) { - substream = list_entry(list, struct snd_rawmidi_substream, list); + list_for_each_entry(substream, + &rawmidi->streams[SNDRV_RAWMIDI_STREAM_OUTPUT].substreams, + list) { snd_mtpav_set_name(mcard, substream); substream->ops = &snd_mtpav_output; mcard->ports[substream->number].hwport = translate_subdevice_to_hwport(mcard, substream->number);