From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Wood Subject: [PatchV3 3/3] usb: plusb: Add debug to reset function Date: Thu, 7 Apr 2011 00:40:15 -0700 Message-ID: <1302162015-22504-3-git-send-email-simon@mungewell.org> References: <1301456667-1648-1-git-send-email-simon@mungewell.org> <1302162015-22504-1-git-send-email-simon@mungewell.org> Cc: Sergei Shtylyov , davem@davemloft.net, linux-usb@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Simon Wood To: Greg Kroah-Hartman Return-path: In-Reply-To: <1302162015-22504-1-git-send-email-simon@mungewell.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: simon This patch adds some debug to the reset function to print out the reason why it fails. Signed-off-by: Simon Wood --- drivers/net/usb/plusb.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/net/usb/plusb.c b/drivers/net/usb/plusb.c index f46aa07..217aec8 100644 --- a/drivers/net/usb/plusb.c +++ b/drivers/net/usb/plusb.c @@ -94,11 +94,15 @@ pl_set_QuickLink_features(struct usbnet *dev, int val) static int pl_reset(struct usbnet *dev) { + int status; + /* some units seem to need this reset, others reject it utterly. * FIXME be more like "naplink" or windows drivers. */ - (void) pl_set_QuickLink_features(dev, + status = pl_set_QuickLink_features(dev, PL_S_EN|PL_RESET_OUT|PL_RESET_IN|PL_PEER_E); + if (status != 0 && netif_msg_probe(dev)) + netif_dbg(dev, link, dev->net, "pl_reset --> %d\n", status); return 0; } -- 1.7.4.1