zarfld / LinuxCnc_PokeysLibComp

Pokeys comp for LinuxCnc using https://bitbucket.org/mbosnak/pokeyslib.git
MIT License
5 stars 0 forks source link

Issue: Create Branch for Version Without Custom Homing Support #19

Closed zarfld closed 22 hours ago

zarfld commented 1 month ago

Issue: Create Branch 'no-custom-homing' for Version Without Custom Homing Support

Description

This issue outlines the requirement to create a separate branch of the pokeys_py repository, targeting versions of LinuxCNC that do not yet support custom homing via the homecomp component.

Objectives

Key Modifications

  1. Remove or Disable Custom Homing Support:

    • The custom homing logic defined in PEv2 for AxisState handling, including states like HOMINGSEARCH, HOMINGBACK, etc., will be excluded or conditionally disabled.
    • Existing pins for homing switches (home-sw-in, neg-lim-sw-in, pos-lim-sw-in) will be used purely for signaling, as defined in canonical LinuxCNC behavior.
  2. Modify HAL Pin Definitions:

    • Adjust HAL pin mappings to reflect the lack of custom homing by LinuxCNC, while retaining canonical LinuxCNC joint.* pin structures for non-custom homing.
    • Ensure that homing states (AxisState) from the PoKeys device are ignored in this branch.
  3. Versioning and Compatibility:

    • Ensure compatibility with LinuxCNC versions prior to the introduction of the homecomp component.
    • Tag this branch as no-custom-homing to differentiate from other branches.

Additional Information

Tasks

  1. Create the branch no-custom-homing.
  2. Remove custom homing logic (homecomp) from the current homing implementation in PEv2.
  3. Modify HAL configuration to utilize standard homing mechanisms from LinuxCNC.
  4. Ensure proper version tagging and release notes indicating custom homing is not available in this branch.
  5. Update the README to reflect the limitations of this branch compared to the master branch.

Reference Links:

Labels:


Please assign this issue to the responsible developers and link to the branch once created.