sergiotarxz/nuttx
sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

drivers/net/enc28j60.c: Adjust debug trace levels. Correct typos in some comments.

Browse Source
This commit is contained in:
Harri Luhtala 2018-09-25 07:07:04 -06:00 committed by Gregory Nutt
parent 2d0f1b85e3
commit 2d750d461b
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
drivers/net/enc28j60.c
View File

@ -1486,7 +1486,8 @@ static void enc_rxdispatch(FAR struct enc_driver_s *priv)
else else
#endif #endif
{ {
nerr("ERROR: Unsupported packet type dropped (%02x)\n", htons(BUF->type)); nwarn("WARNING: Unsupported packet type dropped (%02x)\n",
htons(BUF->type));
NETDEV_RXDROPPED(&priv->dev); NETDEV_RXDROPPED(&priv->dev);
} }
} }
@ -1768,7 +1769,7 @@ static void enc_irqworker(FAR void *arg)
uint8_t pktcnt = enc_rdbreg(priv, ENC_EPKTCNT); uint8_t pktcnt = enc_rdbreg(priv, ENC_EPKTCNT);
if (pktcnt > 0) if (pktcnt > 0)
{ {
nerr("EPKTCNT: %02x\n", pktcnt); ninfo("EPKTCNT: %02x\n", pktcnt);
/* Handle packet receipt */ /* Handle packet receipt */
@ -1796,7 +1797,7 @@ static void enc_irqworker(FAR void *arg)
* clear the EIR.RXERIF bit. * clear the EIR.RXERIF bit.
*/ */
if ((eir & EIR_RXERIF) != 0) /* Receive Errror Interrupts */ if ((eir & EIR_RXERIF) != 0) /* Receive Error Interrupts */
{ {
enc_rxerif(priv); /* Handle the RX error */ enc_rxerif(priv); /* Handle the RX error */
enc_bfcgreg(priv, ENC_EIR, EIR_RXERIF); /* Clear the RXERIF interrupt */ enc_bfcgreg(priv, ENC_EIR, EIR_RXERIF); /* Clear the RXERIF interrupt */
@ -1934,7 +1935,7 @@ static void enc_txtimeout(int argc, uint32_t arg, ...)
FAR struct enc_driver_s *priv = (FAR struct enc_driver_s *)arg; FAR struct enc_driver_s *priv = (FAR struct enc_driver_s *)arg;
int ret; int ret;
/* In complex environments, we cannot do SPI transfers from the timout /* In complex environments, we cannot do SPI transfers from the timeout
* handler because semaphores are probably used to lock the SPI bus. In * handler because semaphores are probably used to lock the SPI bus. In
* this case, we will defer processing to the worker thread. This is also * this case, we will defer processing to the worker thread. This is also
* much kinder in the use of system resources and is, therefore, probably * much kinder in the use of system resources and is, therefore, probably
@ -2029,7 +2030,7 @@ static void enc_polltimer(int argc, uint32_t arg, ...)
FAR struct enc_driver_s *priv = (FAR struct enc_driver_s *)arg; FAR struct enc_driver_s *priv = (FAR struct enc_driver_s *)arg;
int ret; int ret;
/* In complex environments, we cannot do SPI transfers from the timout /* In complex environments, we cannot do SPI transfers from the timeout
* handler because semaphores are probably used to lock the SPI bus. In * handler because semaphores are probably used to lock the SPI bus. In
* this case, we will defer processing to the worker thread. This is also * this case, we will defer processing to the worker thread. This is also
* much kinder in the use of system resources and is, therefore, probably * much kinder in the use of system resources and is, therefore, probably
@ -2643,7 +2644,7 @@ int enc_initialize(FAR struct spi_dev_s *spi,
priv->lower = lower; /* Save the low-level MCU interface */ priv->lower = lower; /* Save the low-level MCU interface */
/* The interface should be in the down state. However, this function is called /* The interface should be in the down state. However, this function is called
* too early in initalization to perform the ENC28J60 reset in enc_ifdown. We * too early in initialization to perform the ENC28J60 reset in enc_ifdown. We
* are depending upon the fact that the application level logic will call enc_ifdown * are depending upon the fact that the application level logic will call enc_ifdown
* later to reset the ENC28J60. NOTE: The MAC address will not be set up until * later to reset the ENC28J60. NOTE: The MAC address will not be set up until
* enc_ifup() is called. That gives the app time to set the MAC address before * enc_ifup() is called. That gives the app time to set the MAC address before