[69/82] usb: dwc3: cleanup with list_first_entry_or_null()

Message ID 20161031104914.1990-70-felipe.balbi@linux.intel.com
State Accepted
Commit 785c91f8ae4d65efbe4d3366e30bb9bbe84b5f43
Headers show

Commit Message

Felipe Balbi Oct. 31, 2016, 10:49 a.m.
From: Masahiro Yamada <yamada.masahiro@socionext.com>


The combo of list_empty() check and return list_first_entry()
can be replaced with list_first_entry_or_null().

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>

---
 drivers/usb/dwc3/gadget.h | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

-- 
2.10.1

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/usb/dwc3/gadget.h b/drivers/usb/dwc3/gadget.h
index e4a1d974a5ae..3129bcf74d7d 100644
--- a/drivers/usb/dwc3/gadget.h
+++ b/drivers/usb/dwc3/gadget.h
@@ -62,10 +62,7 @@  struct dwc3;
 
 static inline struct dwc3_request *next_request(struct list_head *list)
 {
-	if (list_empty(list))
-		return NULL;
-
-	return list_first_entry(list, struct dwc3_request, list);
+	return list_first_entry_or_null(list, struct dwc3_request, list);
 }
 
 static inline void dwc3_gadget_move_started_request(struct dwc3_request *req)