Skip to content

Commit

Permalink
pulley: HomeBack state is never used
Browse files Browse the repository at this point in the history
Also remove unused FinishHomingAndPlanMoveToParkPos.
  • Loading branch information
gudnimg committed Jan 5, 2025
1 parent dc68b70 commit 14ad674
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
9 changes: 0 additions & 9 deletions src/modules/pulley.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,6 @@ namespace pulley {

Pulley pulley;

bool __attribute__((noinline)) Pulley::FinishHomingAndPlanMoveToParkPos() {
mm::motion.SetPosition(mm::Pulley, 0);
return true;
}

bool Pulley::Step() {
if (IsOnHold()) {
return true; // just wait, do nothing!
Expand All @@ -28,10 +23,6 @@ bool Pulley::Step() {
case Moving:
PerformMove();
return false;
case HomeBack:
homingValid = true;
FinishHomingAndPlanMoveToParkPos();
return true;
case Ready:
return true;
case TMCFailed:
Expand Down
2 changes: 1 addition & 1 deletion src/modules/pulley.h
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ class Pulley : public motion::MovableBase {
virtual void PrepareMoveToPlannedSlot() override {}
virtual void PlanHomingMoveForward() override {}
virtual void PlanHomingMoveBack() override {}
virtual bool FinishHomingAndPlanMoveToParkPos() override;
virtual bool FinishHomingAndPlanMoveToParkPos() override { return true; }
virtual void FinishMove() override {}
};

Expand Down

0 comments on commit 14ad674

Please sign in to comment.