The default switch case is unreachable as the MAX_CHANNEL_NUM == 4.

Fixes: a5e73bb ("ethtool:QSFP Plus/QSFP28 Diagnostics Information Support")
Cc: Vidya Sagar Ravipati <vi...@cumulusnetworks.com>
Signed-off-by: Ivan Vecera <c...@cera.cz>
---
 qsfp.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/qsfp.c b/qsfp.c
index aecd5bb..32e195d 100644
--- a/qsfp.c
+++ b/qsfp.c
@@ -661,9 +661,6 @@ static void sff8636_dom_parse(const __u8 *id, struct 
sff_diags *sd)
                        tx_power_offset = SFF8636_TX_PWR_4_OFFSET;
                        tx_bias_offset = SFF8636_TX_BIAS_4_OFFSET;
                        break;
-               default:
-                       printf(" Invalid channel: %d\n", i);
-                       break;
                }
                sd->scd[i].bias_cur = OFFSET_TO_U16(tx_bias_offset);
                sd->scd[i].rx_power = OFFSET_TO_U16(rx_power_offset);
-- 
2.16.4

Reply via email to