search

ut-robotics / picr22-team-teletupsud

0 stars 0 forks source link

Mainboard documentation #60

Closed ReikoR closed 1 year ago

ReikoR commented 1 year ago

  1. Documentation should be in electronics/README.asciidoc, not in README.asciidoc.

  2. Under functionality describe how many and what type of motors and at which voltages and currents the board can control. Example functionality description can be seen here: https://github.com/ReikoR/MC33886_driver_board/blob/master/README.asciidoc

  3. Programmer connector pinout is incorrect. \ On the schematic: image \ In README: image

  4. There is no need to describe USB connector pinout as it's standardised.

  5. J10 is thrower connector not motor 1 encoder connector. image \ Unused pins (3 and 4) can be documented as NC (not connected).

ReikoR commented 1 year ago

  1. This is not really correct: "These motors need 12V and 5.5A to work properly. However, DRV8243 is capable with 40V and output 12 amps." \ 12V is nominal voltage. 5.5A is stall current at 12V. These motors work properly at lower power and at different voltage. Usually the max power needs to be limited to avoid damaging the motor. Recommended max power is only 12W, but we run them at an higher power to get more performance at the cost of damaging them. https://www.pololu.com/category/116/37d-metal-gearmotors \ There is not really need to describe specific motors. Mainboard capabilities should just describe output voltage and current. \ Voltage for the motor drivers is limited by the voltage rating (25V) of the motor driver VM capacitors. It safe to say that maximum input voltage can be 16.8V (4S battery) to have some margin below 25V. Current is limited by the motor driver configuration resistors. This current limit should be added to the README. Also the motor drivers are pre-production variants, which have maximum current limit at 9A, but it's not important to specify that if current limit is configured to be lower. \ From datasheet: image

  2. Programmer connector pinout is still incorrect. Pin 9 needs to be GND and pin 10 is missing. image

ReikoR commented 1 year ago

  1. Pin 9 is GND, not NC. image

  2. Format "Connector pinouts:" as a title. image