diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2010-04-28 05:28:36 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-04-28 15:44:36 -0400 |
commit | b548a988a9d08f2b06f6810b1a4cde6f632bd307 (patch) | |
tree | 6533b6eaa30cc549ed131f69b4830594b0dcbf6d /drivers/net/sfc/mcdi.c | |
parent | 41b7e4c3268d9d2056b9c94cceb386649f7b185b (diff) |
sfc: Log specific message for failure of NVRAM self-test
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/sfc/mcdi.c')
-rw-r--r-- | drivers/net/sfc/mcdi.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/net/sfc/mcdi.c b/drivers/net/sfc/mcdi.c index 1344afac4673..93cc3c1b9450 100644 --- a/drivers/net/sfc/mcdi.c +++ b/drivers/net/sfc/mcdi.c | |||
@@ -932,20 +932,26 @@ int efx_mcdi_nvram_test_all(struct efx_nic *efx) | |||
932 | 932 | ||
933 | rc = efx_mcdi_nvram_types(efx, &nvram_types); | 933 | rc = efx_mcdi_nvram_types(efx, &nvram_types); |
934 | if (rc) | 934 | if (rc) |
935 | return rc; | 935 | goto fail1; |
936 | 936 | ||
937 | type = 0; | 937 | type = 0; |
938 | while (nvram_types != 0) { | 938 | while (nvram_types != 0) { |
939 | if (nvram_types & 1) { | 939 | if (nvram_types & 1) { |
940 | rc = efx_mcdi_nvram_test(efx, type); | 940 | rc = efx_mcdi_nvram_test(efx, type); |
941 | if (rc) | 941 | if (rc) |
942 | return rc; | 942 | goto fail2; |
943 | } | 943 | } |
944 | type++; | 944 | type++; |
945 | nvram_types >>= 1; | 945 | nvram_types >>= 1; |
946 | } | 946 | } |
947 | 947 | ||
948 | return 0; | 948 | return 0; |
949 | |||
950 | fail2: | ||
951 | EFX_ERR(efx, "%s: failed type=%u\n", __func__, type); | ||
952 | fail1: | ||
953 | EFX_ERR(efx, "%s: failed rc=%d\n", __func__, rc); | ||
954 | return rc; | ||
949 | } | 955 | } |
950 | 956 | ||
951 | static int efx_mcdi_read_assertion(struct efx_nic *efx) | 957 | static int efx_mcdi_read_assertion(struct efx_nic *efx) |