aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIdo Schimmel <idosch@nvidia.com>2021-10-01 18:06:26 +0300
committerMichal Kubecek <mkubecek@suse.cz>2021-10-11 11:27:10 +0200
commit128e97c5f6e5392241a87902745d415fa53e3619 (patch)
treea85269b007338f5665fc4ca3874d6bc12357dddb
parent7ff603badbbb6a74bde75c93bc371d4e4893ceac (diff)
downloadethtool-128e97c5f6e5392241a87902745d415fa53e3619.tar.gz
sff-8636: Convert if statement to switch-case
The indentation is wrong and the statement can be more clearly represented using a switch-case statement. Convert it. Signed-off-by: Ido Schimmel <idosch@nvidia.com>
-rw-r--r--qsfp.c8
1 files changed, 5 insertions, 3 deletions
diff --git a/qsfp.c b/qsfp.c
index 3401db8..d1464cb 100644
--- a/qsfp.c
+++ b/qsfp.c
@@ -863,11 +863,13 @@ void sff8636_show_all(const __u8 *id, __u32 eeprom_len)
}
sff8636_show_identifier(id);
- if ((id[SFF8636_ID_OFFSET] == SFF8024_ID_QSFP) ||
- (id[SFF8636_ID_OFFSET] == SFF8024_ID_QSFP_PLUS) ||
- (id[SFF8636_ID_OFFSET] == SFF8024_ID_QSFP28)) {
+ switch (id[SFF8636_ID_OFFSET]) {
+ case SFF8024_ID_QSFP:
+ case SFF8024_ID_QSFP_PLUS:
+ case SFF8024_ID_QSFP28:
sff8636_show_page_zero(id);
sff8636_show_dom(id, id + 3 * 0x80, eeprom_len);
+ break;
}
}