Commit d93f27b7 authored by Bhumika Goyal's avatar Bhumika Goyal Committed by Greg Kroah-Hartman
Browse files

Staging: most: Use list_first_entry instead of list_entry 



This patch replaces list_entry with list_first_entry as it makes the
code more clear.
Done using coccinelle:

@@
expression e;
@@
(
- list_entry(e->next,
+ list_first_entry(e,
  ...)
|
- list_entry(e->prev,
+ list_last_entry(e,
  ...)
)

Signed-off-by: default avatarBhumika Goyal <bhumirks@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7b50908a

drivers/staging/most/hdm-dim2/dim2_hdm.c

+3 −3
Original line number Diff line number Diff line
@@ -251,7 +251,7 @@ static int try_start_dim_transfer(struct hdm_channel *hdm_ch) 
		return -EAGAIN;

	}



	mbo = list_entry(head->next, struct mbo, list);

	mbo = list_first_entry(head, struct mbo, list);

	buf_size = mbo->buffer_length;



	BUG_ON(mbo->bus_address == 0);
@@ -362,7 +362,7 @@ static void service_done_flag(struct dim2_hdm *dev, int ch_idx) 
			break;

		}



		mbo = list_entry(head->next, struct mbo, list);

		mbo = list_first_entry(head, struct mbo, list);

		list_del(head->next);

		spin_unlock_irqrestore(&dim_lock, flags);
@@ -495,7 +495,7 @@ static void complete_all_mbos(struct list_head *head) 
			break;

		}



		mbo = list_entry(head->next, struct mbo, list);

		mbo = list_first_entry(head, struct mbo, list);

		list_del(head->next);

		spin_unlock_irqrestore(&dim_lock, flags);

CRA Git | Maintained and supported by SUSTech CRA and CCSE