Le 29/08/2018 à 15:13, David Gounaris a écrit :
Added tx timerout handler. This helps
when troubleshooting.

Signed-off-by: David Gounaris <david.gouna...@infinera.com>
---
  drivers/net/wan/fsl_ucc_hdlc.c | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/drivers/net/wan/fsl_ucc_hdlc.c b/drivers/net/wan/fsl_ucc_hdlc.c
index 248f1f5bcd04..629ef5049d27 100644
--- a/drivers/net/wan/fsl_ucc_hdlc.c
+++ b/drivers/net/wan/fsl_ucc_hdlc.c
@@ -1002,11 +1002,15 @@ static const struct dev_pm_ops uhdlc_pm_ops = {
  #define HDLC_PM_OPS NULL
#endif
+static void uhdlc_tx_timeout(struct net_device *ndev) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#22: FILE: drivers/net/wan/fsl_ucc_hdlc.c:1004:
+static void uhdlc_tx_timeout(struct net_device *ndev) {

+       netdev_err(ndev, "%s\n", __FUNCTION__);

WARNING:USE_FUNC: __func__ should be used instead of gcc specific __FUNCTION__
#23: FILE: drivers/net/wan/fsl_ucc_hdlc.c:1005:
+       netdev_err(ndev, "%s\n", __FUNCTION__);

+}
  static const struct net_device_ops uhdlc_ops = {
        .ndo_open       = uhdlc_open,
        .ndo_stop       = uhdlc_close,
        .ndo_start_xmit = hdlc_start_xmit,
        .ndo_do_ioctl   = uhdlc_ioctl,
+       .ndo_tx_timeout = uhdlc_tx_timeout,
  };
static int ucc_hdlc_probe(struct platform_device *pdev)
@@ -1125,6 +1129,7 @@ static int ucc_hdlc_probe(struct platform_device *pdev)
        hdlc = dev_to_hdlc(dev);
        dev->tx_queue_len = 16;
        dev->netdev_ops = &uhdlc_ops;
+       dev->watchdog_timeo = 2*HZ;

CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#38: FILE: drivers/net/wan/fsl_ucc_hdlc.c:1131:
+       dev->watchdog_timeo = 2*HZ;

        hdlc->attach = ucc_hdlc_attach;
        hdlc->xmit = ucc_hdlc_tx;
        netif_napi_add(dev, &uhdlc_priv->napi, ucc_hdlc_poll, 32);


---
L'absence de virus dans ce courrier électronique a été vérifiée par le logiciel 
antivirus Avast.
https://www.avast.com/antivirus

Reply via email to