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

   ## Summary
   
   This PR comes from InSpace.
   
   For some reason, the CRC check in its current state fails for the MS5611 
sensors that InSpace has. We've tested on four different sensors to confirm 
that this is not due to the sensors being faulty. The official CRC4 code from 
the manufacturer is slightly different than the NuttX implementation and it 
passes when used on the MS5611s.
   
   ## Impact
   
   MS56xx driver no longer fails to register due to faulty CRC.
   
   ## Testing
   
   Tested the previous version of the driver on 4 different MS5611s and viewed 
failing CRC in the debug logs. Then tested with this version from the 
manufacturer and had successful device registration.


-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to