diff mbox series

[-next] HID: hid-mf: Use list_for_each_entry() helper

Message ID 20230830090108.528257-1-ruanjinjie@huawei.com (mailing list archive)
State New
Delegated to: Jiri Kosina
Headers show
Series [-next] HID: hid-mf: Use list_for_each_entry() helper | expand

Commit Message

Jinjie Ruan Aug. 30, 2023, 9:01 a.m. UTC 
Convert list_for_each() to list_for_each_entry() so that the report_ptr
list_head pointer and list_entry() call are no longer needed, which
can reduce a few lines of code. No functional changed.

Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com>
---
 drivers/hid/hid-mf.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/drivers/hid/hid-mf.c b/drivers/hid/hid-mf.c
index 92d7ecd41a78..2d9a500cf364 100644
--- a/drivers/hid/hid-mf.c
+++ b/drivers/hid/hid-mf.c
@@ -61,7 +61,6 @@  static int mf_init(struct hid_device *hid)
 	struct list_head *report_list =
 			&hid->report_enum[HID_OUTPUT_REPORT].report_list;
 
-	struct list_head *report_ptr;
 	struct hid_report *report;
 
 	struct list_head *input_ptr = &hid->inputs;
@@ -72,9 +71,7 @@  static int mf_init(struct hid_device *hid)
 	int error;
 
 	/* Setup each of the four inputs */
-	list_for_each(report_ptr, report_list) {
-		report = list_entry(report_ptr, struct hid_report, list);
-
+	list_for_each_entry(report, report_list, list) {
 		if (report->maxfield < 1 || report->field[0]->report_count < 2) {
 			hid_err(hid, "Invalid report, this should never happen!\n");
 			return -ENODEV;