Skip to content

Commit

Permalink
net: qed: Move static keyword to the front of declaration
Browse files Browse the repository at this point in the history
Move the static keyword to the front of declaration of iwarp_state_names,
and resolve the following compiler warning that can be seen when building
with warnings enabled (W=1):

drivers/net/ethernet/qlogic/qed/qed_iwarp.c:385:1: warning:
  ‘static’ is not at beginning of declaration [-Wold-style-declaration]

Also, resolve checkpatch.pl script warning:

WARNING: static const char * array should probably be
  static const char * const

Signed-off-by: Krzysztof Wilczynski <kw@linux.com>
Acked-by: Michal Kalderon <michal.kalderon@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Krzysztof Wilczynski authored and David S. Miller committed Sep 5, 2019
1 parent a8a213c commit ee4c3de
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/net/ethernet/qlogic/qed/qed_iwarp.c
Original file line number Diff line number Diff line change
Expand Up @@ -382,7 +382,7 @@ qed_iwarp2roce_state(enum qed_iwarp_qp_state state)
}
}

const static char *iwarp_state_names[] = {
static const char * const iwarp_state_names[] = {
"IDLE",
"RTS",
"TERMINATE",
Expand Down

0 comments on commit ee4c3de

Please sign in to comment.