diff options
author | Ido Schimmel <idosch@nvidia.com> | 2021-10-01 18:06:26 +0300 |
---|---|---|
committer | Michal Kubecek <mkubecek@suse.cz> | 2021-10-11 11:27:10 +0200 |
commit | 128e97c5f6e5392241a87902745d415fa53e3619 (patch) | |
tree | a85269b007338f5665fc4ca3874d6bc12357dddb | |
parent | 7ff603badbbb6a74bde75c93bc371d4e4893ceac (diff) | |
download | ethtool-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.c | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -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; } } |