diff mbox series

[03/11] net: dc2114x: Clean up dc21x4x_recv()

Message ID 20200419020957.495654-3-marek.vasut+renesas@gmail.com
State New
Headers show
Series [01/11] net: dc2114x: Clean up init code | expand

Commit Message

Marek Vasut April 19, 2020, 2:09 a.m. UTC
Clean up the driver recv code to bring it up to standards with
U-Boot coding style. No functional change.

Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
Cc: Joe Hershberger <joe.hershberger at ni.com>
---
 drivers/net/dc2114x.c | 44 +++++++++++++++++++------------------------
 1 file changed, 19 insertions(+), 25 deletions(-)
diff mbox series

Patch

diff --git a/drivers/net/dc2114x.c b/drivers/net/dc2114x.c
index 11ebea9c1a..4d2e11672e 100644
--- a/drivers/net/dc2114x.c
+++ b/drivers/net/dc2114x.c
@@ -346,46 +346,40 @@  done:
 	return status;
 }
 
-static int dc21x4x_recv(struct eth_device* dev)
+static int dc21x4x_recv(struct eth_device *dev)
 {
-	s32		status;
-	int		length    = 0;
+	int length = 0;
+	u32 status;
 
-	for ( ; ; ) {
-		status = (s32)le32_to_cpu(rx_ring[rx_new].status);
+	while (true) {
+		status = le32_to_cpu(rx_ring[rx_new].status);
 
-		if (status & R_OWN) {
+		if (status & R_OWN)
 			break;
-		}
 
 		if (status & RD_LS) {
-			/* Valid frame status.
-			 */
+			/* Valid frame status. */
 			if (status & RD_ES) {
-
-				/* There was an error.
-				 */
+				/* There was an error. */
 				printf("RX error status = 0x%08X\n", status);
 			} else {
-				/* A valid frame received.
-				 */
-				length = (le32_to_cpu(rx_ring[rx_new].status) >> 16);
-
-				/* Pass the packet up to the protocol
-				 * layers.
-				 */
-				net_process_received_packet(
-					net_rx_packets[rx_new], length - 4);
+				/* A valid frame received. */
+				length = (le32_to_cpu(rx_ring[rx_new].status)
+					  >> 16);
+
+				/* Pass the packet up to the protocol layers */
+				net_process_received_packet
+					(net_rx_packets[rx_new], length - 4);
 			}
 
-			/* Change buffer ownership for this frame, back
-			 * to the adapter.
+			/*
+			 * Change buffer ownership for this frame,
+			 * back to the adapter.
 			 */
 			rx_ring[rx_new].status = cpu_to_le32(R_OWN);
 		}
 
-		/* Update entry information.
-		 */
+		/* Update entry information. */
 		rx_new = (rx_new + 1) % rxRingSize;
 	}