You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/09/04 16:21:16 UTC
[GitHub] [incubator-nuttx] davids5 opened a new pull request #1714: stm32h7:i2c driver fixed interrupt storm
davids5 opened a new pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714
## Summary
Driver was getting into a state that it would keep generating interrupts and not service them.
The I2C Killer is the lower trace.
## Impact
Critical. for all H7 application: Noise on I2C will hang the CPU.
## Testing
PX4 FMUV5X with the I2C Killer.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] acassis commented on pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714#issuecomment-687278980
@davids5 what is this "I2C Killer"? I searched for it and only found an old post pointing to his I2C Breaker: https://gist.github.com/bartslinger/d68aff3b867483c20c57678e60fb10e5
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] davids5 commented on pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
davids5 commented on pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714#issuecomment-687281214
@acassis It is a derived from that - but with randomness
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] acassis commented on pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714#issuecomment-687329306
Thank you @davids5 ! Yes, that was the thread where I found the gist.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] davids5 commented on pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
davids5 commented on pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714#issuecomment-687323126
@acassis How about reviewing and merging it?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] davids5 commented on pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
davids5 commented on pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714#issuecomment-687322874
> @davids5 is there a git repository with it?
Hi @acassis Sorry no. I will someday do in on a RT1010 with a proper cli.
All the information is here https://github.com/PX4/Firmware/issues/7968
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] acassis commented on pull request #1714: stm32h7:i2c driver fixed interrupt storm
Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #1714:
URL: https://github.com/apache/incubator-nuttx/pull/1714#issuecomment-687289042
@davids5 is there a git repository with it?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org