sfc: Fix sign of efx_mcdi_poll_reboot() error in efx_mcdi_poll()
authorBen Hutchings <bhutchings@solarflare.com>
Fri, 19 Feb 2010 13:29:27 +0000 (13:29 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 19 Feb 2010 23:50:35 +0000 (15:50 -0800)
efx_mcdi_poll() uses positive error numbers, matching the MCDI
protocol.  It must negate the result of efx_mcdi_poll_reboot() which
returns the usual negative error numbers.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/sfc/mcdi.c

index 9f035b9f03503245ca38c44d058cf350958d3884..f66b3da6ddffbecc236a7d2e6a83506bb0b71a34 100644 (file)
@@ -127,7 +127,7 @@ static int efx_mcdi_poll(struct efx_nic *efx)
        efx_dword_t reg;
 
        /* Check for a reboot atomically with respect to efx_mcdi_copyout() */
-       rc = efx_mcdi_poll_reboot(efx);
+       rc = -efx_mcdi_poll_reboot(efx);
        if (rc)
                goto out;