Commit 616e4aa1 authored by Adam Wujek's avatar Adam Wujek 💬

shell/sfp: improve handling errors returned from storage_get_sfp

Signed-off-by: Adam Wujek's avatarAdam Wujek <adam.wujek@cern.ch>
parent 6a53a978
......@@ -62,6 +62,9 @@ static int cmd_sfp(const char *args[])
} else if (temp == EE_RET_I2CERR) {
pp_printf("I2C error\n");
return -EIO;
} else if (temp < 0) {
pp_printf("SFP database error (%d)\n", temp);
return -EFAULT;
}
pp_printf("%d SFPs in DB\n", temp);
return 0;
......@@ -69,10 +72,11 @@ static int cmd_sfp(const char *args[])
for (i = 0; i < sfpcount; ++i) {
sfpcount = storage_get_sfp(&sfp, 0, i);
if (sfpcount == 0) {
pp_printf("SFP database empty...\n");
pp_printf("SFP database empty\n");
return 0;
} else if (sfpcount == -1) {
pp_printf("SFP database corrupted...\n");
} else if (sfpcount < 0) {
pp_printf("SFP database error (%d)\n",
sfpcount);
return -EFAULT;
}
pp_printf("%d: PN:", i + 1);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment