From patchwork Mon Nov 14 08:28:06 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [v3,net-next,09/12] bnx2x: add fan failure event handling From: Dmitry Kravkov X-Patchwork-Id: 125500 Message-Id: <1321259286.4651.9.camel@lb-tlvb-dmitry> To: "Joe Perches" , davem@davemloft.net, netdev@vger.kernel.org Cc: "Ariel Elior" , "Eilon Greenstein" Date: Mon, 14 Nov 2011 10:28:06 +0200 > -----Original Message----- > From: Joe Perches [mailto:joe@perches.com] > Sent: Sunday, November 13, 2011 11:12 PM > To: Dmitry Kravkov > Cc: davem@davemloft.net; netdev@vger.kernel.org; Ariel Elior; Eilon Greenstein > Subject: Re: [PATCH v3 net-next 09/12] bnx2x: add fan failure event handling > > On Sun, 2011-11-13 at 16:34 +0200, Dmitry Kravkov wrote: > > From: Ariel Elior > > Shut down the device in case of fan failure to prevent HW damage. > trivia. > > diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c > [] > > @@ -8503,6 +8514,17 @@ sp_rtnl_not_reset: > [] > > + if (test_and_clear_bit(BNX2X_SP_RTNL_FAN_FAILURE, &bp->sp_rtnl_state)) { > > + DP(BNX2X_MSG_SP, "fan failure detected. Unloading driver"); > > Missing a trailing "\n". > Reported-by: Joe Perches Signed-off-by: Dmitry Kravkov --- drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c index 33ff60d..c09e59a 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c @@ -8520,7 +8520,7 @@ sp_rtnl_not_reset: * damage */ if (test_and_clear_bit(BNX2X_SP_RTNL_FAN_FAILURE, &bp->sp_rtnl_state)) { - DP(BNX2X_MSG_SP, "fan failure detected. Unloading driver"); + DP(BNX2X_MSG_SP, "fan failure detected. Unloading driver\n"); netif_device_detach(bp->dev); bnx2x_close(bp->dev); }