You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "raiden00pl (via GitHub)" <gi...@apache.org> on 2024/03/14 18:19:17 UTC

[PR] sensors/bmi270: fix accel and gyro data position [nuttx]

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

   ## Summary
   
   - sensors/bmi270: fix accel and gyro data position
   accel registers are DATA_8 to DATA_13
   gyro registers are DATA_14 to DATA_19
   
   ## Impact
   
   ## Testing
   CI
   


-- 
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-unsubscribe@nuttx.apache.org

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

Re: [PR] sensors/bmi270: fix accel and gyro data position [nuttx]

Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis commented on PR #11923:
URL: https://github.com/apache/nuttx/pull/11923#issuecomment-1999710445

   > @acassis I'm porting the sensors present on Thingy boards to the new sensor framework. Data returned by bmi270 after conversion to g-force didn't make sense and it turned out that regs were in the wrong order.
   > 
   > I don't see any obstacles to using the new sensor framework with sensor_fusion.
   
   Thank you @raiden00pl 


-- 
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-unsubscribe@nuttx.apache.org

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

Re: [PR] sensors/bmi270: fix accel and gyro data position [nuttx]

Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis commented on PR #11923:
URL: https://github.com/apache/nuttx/pull/11923#issuecomment-1998324736

   @raiden00pl how did you spot this issue? Are you testing this device?
   
   I just noticed that the sensor_fusion example is only for MPU6050, it should be IMU agnostic. Maybe using the new sensor orb it could be possible


-- 
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-unsubscribe@nuttx.apache.org

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

Re: [PR] sensors/bmi270: fix accel and gyro data position [nuttx]

Posted by "raiden00pl (via GitHub)" <gi...@apache.org>.
raiden00pl commented on PR #11923:
URL: https://github.com/apache/nuttx/pull/11923#issuecomment-1999042210

   @acassis I'm porting the sensors present on Thingy boards to the new sensor framework. Data returned by bmi270 after conversion to g-force didn't make sense and it turned out that regs were in the wrong order.
   
   I don't see any obstacles to using the new sensor framework with sensor_fusion.


-- 
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-unsubscribe@nuttx.apache.org

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

Re: [PR] sensors/bmi270: fix accel and gyro data position [nuttx]

Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis merged PR #11923:
URL: https://github.com/apache/nuttx/pull/11923


-- 
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-unsubscribe@nuttx.apache.org

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