Thread (10 messages) 10 messages, 3 authors, 2017-06-21

Re: [PATCH net-next 4/4] s390/qeth: use diag26c to get MAC address on L2

flat view

From: David Miller <davem@davemloft.net>
Date: 2017-06-19 14:48:25
Also in: linux-s390

From: Julian Wiedmann <redacted>
Date: Mon, 19 Jun 2017 13:22:25 +0200
quoted hunk ↗ jump to hunk
+{
+	struct diag26c_mac_req *request;
+	struct diag26c_mac_resp *response;
+	struct ccw_dev_id id;
+	int rc;
+
Please order local variable declarations from longest to shortest
line (reverse christmas tree).
+	if ((request->resp_buf_len < sizeof(*response)) ||
+	    (response->version != request->resp_version)) {
Excessive parenthesis here.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help