Duet3D Logo Duet3D
    • Tags
    • Documentation
    • Order
    • Register
    • Login

    Documentation issue regarding M917

    Scheduled Pinned Locked Moved
    General Discussion
    2
    2
    339
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • ampex_fhmundefined
      ampex_fhm
      last edited by ampex_fhm

      In the docs, the section about M917 states

      Some motor drivers (e.g. TMC2660)

      but the TMC2660 does not support the standstill motor current reduction as evident here:

      https://github.com/dc42/RepRapFirmware/blob/v2-dev/src/Movement/StepperDrivers/TMC2660.cpp#L896

      This should be changed to avoid confusion.

      edit: Also the docstring at the top of the TMC22xx files read TMC2660.cpp and ExternalDrivers.h respectively

      Voron CoreXY V022 300x300x230mm single extruder CoreXY machine

      1 Reply Last reply Reply Quote 0
      • dc42undefined
        dc42 administrators
        last edited by

        My intention was to implement standstill current reduction in firmware, but I haven't done that yet.

        I will correct the docstrings in the source files at the next commit.

        Duet WiFi hardware designer and firmware engineer
        Please do not ask me for Duet support via PM or email, use the forum
        http://www.escher3d.com, https://miscsolutions.wordpress.com

        1 Reply Last reply Reply Quote 0
        • First post
          Last post
        Unless otherwise noted, all forum content is licensed under CC-BY-SA