From de0893cc8d190fd7d7bc2283873964194e839850 Mon Sep 17 00:00:00 2001
From: Todd Naugle <toddn@harrisonconsoles.com>
Date: Fri, 30 Apr 2021 15:28:34 -0500
Subject: [PATCH] backport to Ardour: Don't set the MCU Click led state from
 the button

Allow the update process to get the correct state and update the LED.
---
 libs/surfaces/mackie/mcp_buttons.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libs/surfaces/mackie/mcp_buttons.cc b/libs/surfaces/mackie/mcp_buttons.cc
index 6cca07f1f7..da4a7e523b 100644
--- a/libs/surfaces/mackie/mcp_buttons.cc
+++ b/libs/surfaces/mackie/mcp_buttons.cc
@@ -1209,11 +1209,11 @@ MackieControlProtocol::click_press (Mackie::Button&)
 {
 	if (main_modifier_state() & MODIFIER_SHIFT) {
 		access_action ("Editor/set-punch-from-edit-range");
-		return off;
+		return none;
 	} else {
 		bool state = !Config->get_clicking();
 		Config->set_clicking (state);
-		return state;
+		return none;
 	}
 }
 Mackie::LedState