jpaali opened a new pull request, #15450:
URL: https://github.com/apache/nuttx/pull/15450

   ## Summary
   
   This PR includes two minor networking related error handling improvements.
   
   drivers/net/ksz9477.c:
   Return values of register writes are not checked in ksz9477_init() 
   which may lead to misbehaving ksz9477 ethernet switch in case of 
configuration failure.
   Check return values of register writes and abort init in case of error.
   
   arch/risc-v/src/mpfs/mpfs_ethernet.c:
   Error is not returned in mpfs_phyinit() in case of mpfs_i2cbus_initialize() 
failure.
   Return error in case of failure.
   
   ## Impact
   
   Gives possibility to react to initialization failures in ksz9477 / 
mpfs_ethernet.
   Check return values of register writes in drivers/net/ksz9477.c
   Return error value in case of i2c bus initialization failure in 
arch/risc-v/src/mpfs/mpfs_ethernet.c
   
   ## Testing
   
   Tested with custom mpfs board with ksz9477 ethernet switch.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to