Skip to content

Document Z-axis limit switch symbol locations (Z_Limit_sw_1 / Z_Limit_sw_2)#12

Draft
Copilot wants to merge 1 commit intomainfrom
copilot/find-z-limit-usage
Draft

Document Z-axis limit switch symbol locations (Z_Limit_sw_1 / Z_Limit_sw_2)#12
Copilot wants to merge 1 commit intomainfrom
copilot/find-z-limit-usage

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 24, 2026

The issue asked where Z_limit_sw_1 and Z_limit_sw_2 are referenced. In this codebase, those symbols appear with a capital L (Z_Limit_sw_1, Z_Limit_sw_2), so the update is a focused location map rather than a behavioral code change.

  • Symbol naming clarification

    • Confirmed no occurrences of lowercase Z_limit_sw_1/2.
    • Identified canonical symbols: Z_Limit_sw_1, Z_Limit_sw_2.
  • Primary reference points

    • GPIO/pin definitions: RobotArmController/Core/Inc/main.h (Z_Limit_sw_1_Pin, Z_Limit_sw_2_Pin)
    • Axis semantic mapping: RobotArmController/Core/Inc/translation.h (lsw_z_max_Pin -> Z_Limit_sw_1_Pin, lsw_z_null_Pin -> Z_Limit_sw_2_Pin)
    • Interrupt handling: RobotArmController/Core/Src/stm32f7xx_it.c (HAL_GPIO_EXTI_IRQHandler(Z_Limit_sw_1_Pin/2_Pin))
    • CubeMX labels: RobotArmController/RobotArmController.ioc (PD3/PD4 GPIO_Label entries)
  • Code snippet (mapping used by runtime logic)

    #define lsw_z_null_Pin (Z_Limit_sw_2_Pin)
    #define lsw_z_max_Pin  (Z_Limit_sw_1_Pin)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Investigate usage of Z_limit_sw_1 and Z_limit_sw_2 Document Z-axis limit switch symbol locations (Z_Limit_sw_1 / Z_Limit_sw_2) Mar 24, 2026
Copilot AI requested a review from smarcel858bme March 24, 2026 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants