Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers device instances #8813

Closed
dagar opened this issue Feb 3, 2018 · 3 comments
Closed

drivers device instances #8813

dagar opened this issue Feb 3, 2018 · 3 comments

Comments

@dagar
Copy link
Member

dagar commented Feb 3, 2018

We should keep lists (probably per bus) of all the Device (I2C/SPI/CAN) instances in the system.

  • this would make deconfliction easier and explicit
  • allow us to drop unnecessary CDev usage (/dev/mpu6000_accel, /dev/mpu6000_accel1, /dev/mpu6000_accel_ext, /dev/mpu6000_accel_ext1, etc https://github.com/PX4/Firmware/blob/master/src/drivers/mpu6000/mpu6000.h#L52)
  • improve the debug tooling/reporting
    • print a simple summary of all drivers by bus, address, configuration, driver type
  • it might make sense for serial drivers (eg most lidar, px4io) to become Devices as well

@davids5

@dagar dagar added this to the Release v1.8.0 milestone Feb 3, 2018
@dagar dagar self-assigned this Feb 3, 2018
@dagar dagar removed this from the Release v1.8.0 milestone May 4, 2018
@stale stale bot closed this as completed Feb 11, 2019
@dagar dagar reopened this Feb 11, 2019
@stale stale bot removed the Admin: Wont fix label Feb 11, 2019
@PX4 PX4 deleted a comment from stale bot Feb 11, 2019
@PX4 PX4 deleted a comment from stale bot Feb 11, 2019
@PX4 PX4 deleted a comment from PX4BuildBot Feb 11, 2019
@dagar dagar added this to the Release v1.10.0 milestone Feb 11, 2019
@PX4 PX4 deleted a comment from stale bot Jun 25, 2019
@stale stale bot removed the Admin: Wont fix label Jun 25, 2019
@stale
Copy link

stale bot commented Sep 23, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@TSC21
Copy link
Member

TSC21 commented Sep 23, 2019

Still relevant

@dagar
Copy link
Member Author

dagar commented Apr 8, 2020

Done!

@dagar dagar closed this as completed Apr 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants