From 14ad67429173d91bfa77ce3a96ebe808da19f7bf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gu=C3=B0ni=20M=C3=A1r=20Gilbert?= <gudni.m.g@gmail.com>
Date: Sun, 5 Jan 2025 17:44:18 +0000
Subject: [PATCH] pulley: HomeBack state is never used

Also remove unused FinishHomingAndPlanMoveToParkPos.
---
 src/modules/pulley.cpp | 9 ---------
 src/modules/pulley.h   | 2 +-
 2 files changed, 1 insertion(+), 10 deletions(-)

diff --git a/src/modules/pulley.cpp b/src/modules/pulley.cpp
index 81d43a20..aa74a8e0 100644
--- a/src/modules/pulley.cpp
+++ b/src/modules/pulley.cpp
@@ -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!
@@ -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:
diff --git a/src/modules/pulley.h b/src/modules/pulley.h
index b2c2d2fd..54262b70 100644
--- a/src/modules/pulley.h
+++ b/src/modules/pulley.h
@@ -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 {}
 };