fix faderport's use of action name
[ardour.git] / libs / surfaces / faderport / faderport.cc
index 19dbfe83ce92d387546be857eda4105b0ac95744..7d8bb46bf13ac6afdc474e1eb7339730c1612317 100644 (file)
@@ -26,7 +26,6 @@
 #include <glibmm/fileutils.h>
 #include <glibmm/miscutils.h>
 
-#include "pbd/controllable_descriptor.h"
 #include "pbd/error.h"
 #include "pbd/failed_constructor.h"
 #include "pbd/file_utils.h"
@@ -39,6 +38,8 @@
 #include "ardour/async_midi_port.h"
 #include "ardour/audioengine.h"
 #include "ardour/amp.h"
+#include "ardour/bundle.h"
+#include "ardour/controllable_descriptor.h"
 #include "ardour/debug.h"
 #include "ardour/filesystem_paths.h"
 #include "ardour/midi_port.h"
 #include "ardour/monitor_processor.h"
 #include "ardour/profile.h"
 #include "ardour/rc_configuration.h"
-#include "ardour/route.h"
+#include "ardour/record_enable_control.h"
+#include "ardour/stripable.h"
 #include "ardour/session.h"
+#include "ardour/session_configuration.h"
 #include "ardour/track.h"
 
 #include "faderport.h"
@@ -63,8 +66,8 @@ using namespace std;
 #include "pbd/abstract_ui.cc" // instantiate template
 
 FaderPort::FaderPort (Session& s)
-       : ControlProtocol (s, _("Faderport"))
-       , AbstractUI<FaderPortRequest> ("faderport")
+       : ControlProtocol (s, _("PreSonus FaderPort"))
+       , AbstractUI<FaderPortRequest> (name())
        , gui (0)
        , connection_state (ConnectionState (0))
        , _device_active (false)
@@ -73,6 +76,7 @@ FaderPort::FaderPort (Session& s)
        , fader_is_touched (false)
        , button_state (ButtonState (0))
        , blink_state (false)
+       , rec_enable_state (false)
 {
        last_encoder_time = 0;
 
@@ -89,7 +93,23 @@ FaderPort::FaderPort (Session& s)
                throw failed_constructor();
        }
 
-       TrackSelectionChanged.connect (selection_connection, MISSING_INVALIDATOR, boost::bind (&FaderPort::gui_track_selection_changed, this, _1), this);
+       _input_bundle.reset (new ARDOUR::Bundle (_("Faderport Support (Receive)"), true));
+       _output_bundle.reset (new ARDOUR::Bundle (_("Faderport Support (Send) "), false));
+
+       _input_bundle->add_channel (
+               inp->name(),
+               ARDOUR::DataType::MIDI,
+               session->engine().make_port_name_non_relative (inp->name())
+               );
+
+       _output_bundle->add_channel (
+               outp->name(),
+               ARDOUR::DataType::MIDI,
+               session->engine().make_port_name_non_relative (outp->name())
+               );
+
+
+       StripableSelectionChanged.connect (selection_connection, MISSING_INVALIDATOR, boost::bind (&FaderPort::gui_track_selection_changed, this, _1), this);
 
        /* Catch port connections and disconnections */
        ARDOUR::AudioEngine::instance()->PortConnectedOrDisconnected.connect (port_connection, MISSING_INVALIDATOR, boost::bind (&FaderPort::connection_handler, this, _1, _2, _3, _4, _5), this);
@@ -118,8 +138,15 @@ FaderPort::FaderPort (Session& s)
        buttons.insert (std::make_pair (Stop, Button (*this, _("Stop"), Stop, 2)));
        buttons.insert (std::make_pair (Play, Button (*this, _("Play"), Play, 1)));
        buttons.insert (std::make_pair (RecEnable, Button (*this, _("RecEnable"), RecEnable, 0)));
+       buttons.insert (std::make_pair (Footswitch, Button (*this, _("Footswitch"), Footswitch, -1)));
        buttons.insert (std::make_pair (FaderTouch, Button (*this, _("Fader (touch)"), FaderTouch, -1)));
 
+       get_button (Shift).set_flash (true);
+       get_button (Mix).set_flash (true);
+       get_button (Proj).set_flash (true);
+       get_button (Trns).set_flash (true);
+       get_button (User).set_flash (true);
+
        get_button (Left).set_action ( boost::bind (&FaderPort::left, this), true);
        get_button (Right).set_action ( boost::bind (&FaderPort::right, this), true);
 
@@ -128,8 +155,11 @@ FaderPort::FaderPort (Session& s)
        get_button (Undo).set_flash (true);
 
        get_button (FP_Read).set_action (boost::bind (&FaderPort::read, this), true);
+       get_button (FP_Read).set_action (boost::bind (&FaderPort::off, this), false, LongPress);
        get_button (FP_Write).set_action (boost::bind (&FaderPort::write, this), true);
+       get_button (FP_Write).set_action (boost::bind (&FaderPort::off, this), false, LongPress);
        get_button (FP_Touch).set_action (boost::bind (&FaderPort::touch, this), true);
+       get_button (FP_Touch).set_action (boost::bind (&FaderPort::off, this), false, LongPress);
        get_button (FP_Off).set_action (boost::bind (&FaderPort::off, this), true);
 
        get_button (Play).set_action (boost::bind (&BasicUI::transport_play, this, true), true);
@@ -149,11 +179,13 @@ FaderPort::FaderPort (Session& s)
        get_button (Ffwd).set_action (boost::bind (&BasicUI::ffwd, this), true);
        get_button (Ffwd).set_action (boost::bind (&BasicUI::goto_end, this), true, ShiftDown);
 
+       get_button (Punch).set_action (boost::bind (&FaderPort::punch, this), true);
+
        get_button (Loop).set_action (boost::bind (&BasicUI::loop_toggle, this), true);
        get_button (Loop).set_action (boost::bind (&BasicUI::add_marker, this, string()), true, ShiftDown);
 
        get_button (Punch).set_action (boost::bind (&BasicUI::prev_marker, this), true, ShiftDown);
-       get_button (User).set_action (boost::bind (&BasicUI::next_marker, this), true, ShiftDown);
+       get_button (User).set_action (boost::bind (&BasicUI::next_marker, this), true, ButtonState(ShiftDown|UserDown));
 
        get_button (Mute).set_action (boost::bind (&FaderPort::mute, this), true);
        get_button (Solo).set_action (boost::bind (&FaderPort::solo, this), true);
@@ -165,6 +197,8 @@ FaderPort::FaderPort (Session& s)
 
 FaderPort::~FaderPort ()
 {
+       all_lights_out ();
+
        if (_input_port) {
                DEBUG_TRACE (DEBUG::FaderPort, string_compose ("unregistering input port %1\n", boost::shared_ptr<ARDOUR::Port>(_input_port)->name()));
                AudioEngine::instance()->unregister_port (_input_port);
@@ -172,13 +206,28 @@ FaderPort::~FaderPort ()
        }
 
        if (_output_port) {
-//             _output_port->drain (10000);  //ToDo:  is this necessary?  It hangs the shutdown, for me
+               _output_port->drain (10000,  250000); /* check every 10 msecs, wait up to 1/4 second for the port to drain */
                DEBUG_TRACE (DEBUG::FaderPort, string_compose ("unregistering output port %1\n", boost::shared_ptr<ARDOUR::Port>(_output_port)->name()));
                AudioEngine::instance()->unregister_port (_output_port);
                _output_port.reset ();
        }
 
        tear_down_gui ();
+
+       /* stop event loop */
+       DEBUG_TRACE (DEBUG::FaderPort, "BaseUI::quit ()\n");
+       BaseUI::quit ();
+}
+
+void*
+FaderPort::request_factory (uint32_t num_requests)
+{
+       /* AbstractUI<T>::request_buffer_factory() is a template method only
+          instantiated in this source module. To provide something visible for
+          use in the interface/descriptor, we have this static method that is
+          template-free.
+       */
+       return request_buffer_factory (num_requests);
 }
 
 void
@@ -238,10 +287,10 @@ FaderPort::thread_init ()
 {
        struct sched_param rtparam;
 
-       pthread_set_name (X_("FaderPort"));
+       pthread_set_name (event_loop_name().c_str());
 
-       PBD::notify_gui_about_thread_creation (X_("gui"), pthread_self(), X_("FaderPort"), 2048);
-       ARDOUR::SessionEvent::create_per_thread_pool (X_("FaderPort"), 128);
+       PBD::notify_event_loops_about_thread_creation (pthread_self(), event_loop_name(), 2048);
+       ARDOUR::SessionEvent::create_per_thread_pool (event_loop_name(), 128);
 
        memset (&rtparam, 0, sizeof (rtparam));
        rtparam.sched_priority = 9; /* XXX should be relative to audio (JACK) thread */
@@ -255,7 +304,7 @@ void
 FaderPort::all_lights_out ()
 {
        for (ButtonMap::iterator b = buttons.begin(); b != buttons.end(); ++b) {
-               b->second.set_led_state (_output_port, false, true);
+               b->second.set_led_state (_output_port, false);
        }
 }
 
@@ -267,31 +316,65 @@ FaderPort::get_button (ButtonID id) const
        return const_cast<Button&>(b->second);
 }
 
+bool
+FaderPort::button_long_press_timeout (ButtonID id)
+{
+       if (buttons_down.find (id) != buttons_down.end()) {
+               get_button (id).invoke (ButtonState (LongPress|button_state), false);
+       } else {
+               /* release happened and somehow we were not cancelled */
+       }
+
+       /* whichever button this was, we've used it ... don't invoke the
+          release action.
+       */
+       consumed.insert (id);
+
+       return false; /* don't get called again */
+}
+
+void
+FaderPort::start_press_timeout (Button& button, ButtonID id)
+{
+       Glib::RefPtr<Glib::TimeoutSource> timeout = Glib::TimeoutSource::create (500); // milliseconds
+       button.timeout_connection = timeout->connect (sigc::bind (sigc::mem_fun (*this, &FaderPort::button_long_press_timeout), id));
+       timeout->attach (main_loop()->get_context());
+}
+
 void
 FaderPort::button_handler (MIDI::Parser &, MIDI::EventTwoBytes* tb)
 {
        ButtonID id (ButtonID (tb->controller_number));
        Button& button (get_button (id));
 
-       button.do_timing (tb->value ? true : false);
+       DEBUG_TRACE (DEBUG::FaderPort, string_compose ("button event for ID %1 press ? %2\n", (int) tb->controller_number, (tb->value ? "yes" : "no")));
+
+       if (tb->value) {
+               buttons_down.insert (id);
+       } else {
+               buttons_down.erase (id);
+               button.timeout_connection.disconnect ();
+       }
+
+       ButtonState bs (ButtonState (0));
 
        switch (id) {
        case Shift:
-               button_state = (tb->value ? ButtonState (button_state|ShiftDown) : ButtonState (button_state&~ShiftDown));
+               bs = ShiftDown;
                break;
        case Stop:
-               button_state = (tb->value ? ButtonState (button_state|StopDown) : ButtonState (button_state&~StopDown));
+               bs = StopDown;
                break;
        case Rewind:
-               button_state = (tb->value ? ButtonState (button_state|RewindDown) : ButtonState (button_state&~RewindDown));
+               bs = RewindDown;
                break;
        case User:
-               button_state = (tb->value ? ButtonState (button_state|UserDown) : ButtonState (button_state&~UserDown));
+               bs = UserDown;
                break;
        case FaderTouch:
                fader_is_touched = tb->value;
-               if (_current_route) {
-                       boost::shared_ptr<AutomationControl> gain = _current_route->gain_control ();
+               if (_current_stripable) {
+                       boost::shared_ptr<AutomationControl> gain = _current_stripable->gain_control ();
                        if (gain) {
                                framepos_t now = session->engine().sample_time();
                                if (tb->value) {
@@ -303,14 +386,29 @@ FaderPort::button_handler (MIDI::Parser &, MIDI::EventTwoBytes* tb)
                }
                break;
        default:
+               if (tb->value) {
+                       start_press_timeout (button, id);
+               }
                break;
        }
 
+       if (bs) {
+               button_state = (tb->value ? ButtonState (button_state|bs) : ButtonState (button_state&~bs));
+               DEBUG_TRACE (DEBUG::FaderPort, string_compose ("reset button state to %1 using %2\n", button_state, (int) bs));
+       }
+
        if (button.uses_flash()) {
                button.set_led_state (_output_port, (int)tb->value);
        }
 
-       button.invoke (button_state, tb->value ? true : false);
+       set<ButtonID>::iterator c = consumed.find (id);
+
+       if (c == consumed.end()) {
+               button.invoke (button_state, tb->value ? true : false);
+       } else {
+               DEBUG_TRACE (DEBUG::FaderPort, "button was consumed, ignored\n");
+               consumed.erase (c);
+       }
 }
 
 void
@@ -346,7 +444,7 @@ FaderPort::encoder_handler (MIDI::Parser &, MIDI::pitchbend_t pb)
                last_good_encoder_delta = delta;
        }
 
-       if (_current_route) {
+       if (_current_stripable) {
 
                ButtonState trim_modifier;
                ButtonState width_modifier;
@@ -360,11 +458,11 @@ FaderPort::encoder_handler (MIDI::Parser &, MIDI::pitchbend_t pb)
                }
 
                if ((button_state & trim_modifier) == trim_modifier ) {    // mod+encoder = input trim
-                       boost::shared_ptr<AutomationControl> gain = _current_route->trim()->gain_control ();
-                       if (gain) {
-                               float val = gain->get_user();  //for gain elements, the "user" value is in dB
+                       boost::shared_ptr<AutomationControl> trim = _current_stripable->trim_control ();
+                       if (trim) {
+                               float val = trim->get_user();  //for gain elements, the "user" value is in dB
                                val += delta;
-                               gain->set_user(val);
+                               trim->set_user(val);
                        }
                } else if (width_modifier && ((button_state & width_modifier) == width_modifier)) {
                        ardour_pan_width (delta);
@@ -377,6 +475,14 @@ FaderPort::encoder_handler (MIDI::Parser &, MIDI::pitchbend_t pb)
                        }
                }
        }
+
+       /* if the user button was pressed, mark it as consumed so that its
+        * release action has no effect.
+        */
+
+       if (!Profile->get_mixbus() && (button_state & UserDown)) {
+               consumed.insert (User);
+       }
 }
 
 void
@@ -393,12 +499,16 @@ FaderPort::fader_handler (MIDI::Parser &, MIDI::EventTwoBytes* tb)
        }
 
        if (was_fader) {
-               if (_current_route) {
-                       boost::shared_ptr<AutomationControl> gain = _current_route->gain_control ();
+               if (_current_stripable) {
+                       boost::shared_ptr<AutomationControl> gain = _current_stripable->gain_control ();
                        if (gain) {
                                int ival = (fader_msb << 7) | fader_lsb;
                                float val = gain->interface_to_internal (ival/16384.0);
-                               _current_route->set_gain (val, this);
+                               /* even though the faderport only controls a
+                                  single stripable at a time, allow the fader to
+                                  modify the group, if appropriate.
+                               */
+                               _current_stripable->gain_control()->set_value (val, Controllable::UseGroup);
                        }
                }
        }
@@ -407,6 +517,8 @@ FaderPort::fader_handler (MIDI::Parser &, MIDI::EventTwoBytes* tb)
 void
 FaderPort::sysex_handler (MIDI::Parser &p, MIDI::byte *buf, size_t sz)
 {
+        DEBUG_TRACE (DEBUG::FaderPort, string_compose ("sysex message received, size = %1\n", sz));
+
        if (sz < 17) {
                return;
        }
@@ -441,14 +553,17 @@ FaderPort::sysex_handler (MIDI::Parser &p, MIDI::byte *buf, size_t sz)
 
        /* catch up on state */
 
-       notify_transport_state_changed ();
-       notify_record_state_changed ();
+       /* make sure that rec_enable_state is consistent with current device state */
+       get_button (RecEnable).set_led_state (_output_port, rec_enable_state);
+
+       map_transport_state ();
+       map_recenable_state ();
 }
 
 int
 FaderPort::set_active (bool yn)
 {
-       DEBUG_TRACE (DEBUG::FaderPort, string_compose("MackieControlProtocol::set_active init with yn: '%1'\n", yn));
+       DEBUG_TRACE (DEBUG::FaderPort, string_compose("Faderport::set_active init with yn: '%1'\n", yn));
 
        if (yn == active()) {
                return 0;
@@ -466,6 +581,10 @@ FaderPort::set_active (bool yn)
                blink_connection = blink_timeout->connect (sigc::mem_fun (*this, &FaderPort::blink));
                blink_timeout->attach (main_loop()->get_context());
 
+               Glib::RefPtr<Glib::TimeoutSource> periodic_timeout = Glib::TimeoutSource::create (100); // milliseconds
+               periodic_connection = periodic_timeout->connect (sigc::mem_fun (*this, &FaderPort::periodic));
+               periodic_timeout->attach (main_loop()->get_context());
+
        } else {
 
                BaseUI::quit ();
@@ -475,11 +594,41 @@ FaderPort::set_active (bool yn)
 
        ControlProtocol::set_active (yn);
 
-       DEBUG_TRACE (DEBUG::FaderPort, string_compose("MackieControlProtocol::set_active done with yn: '%1'\n", yn));
+       DEBUG_TRACE (DEBUG::FaderPort, string_compose("Faderport::set_active done with yn: '%1'\n", yn));
 
        return 0;
 }
 
+bool
+FaderPort::periodic ()
+{
+       if (!_current_stripable) {
+               return true;
+       }
+
+       ARDOUR::AutoState gain_state = _current_stripable->gain_control()->automation_state();
+
+       if (gain_state == ARDOUR::Touch || gain_state == ARDOUR::Play) {
+               map_gain ();
+       }
+
+       return true;
+}
+
+void
+FaderPort::stop_blinking (ButtonID id)
+{
+       blinkers.remove (id);
+       get_button (id).set_led_state (_output_port, false);
+}
+
+void
+FaderPort::start_blinking (ButtonID id)
+{
+       blinkers.push_back (id);
+       get_button (id).set_led_state (_output_port, true);
+}
+
 bool
 FaderPort::blink ()
 {
@@ -489,6 +638,8 @@ FaderPort::blink ()
                get_button(*b).set_led_state (_output_port, blink_state);
        }
 
+       map_recenable_state ();
+
        return true;
 }
 
@@ -502,47 +653,99 @@ FaderPort::close ()
        port_connection.disconnect ();
        blink_connection.disconnect ();
        selection_connection.disconnect ();
-       route_connections.drop_connections ();
+       stripable_connections.drop_connections ();
 
 #if 0
-       route_connections.drop_connections ();
+       stripable_connections.drop_connections ();
 #endif
 }
 
 void
-FaderPort::notify_record_state_changed ()
+FaderPort::map_recenable_state ()
 {
+       /* special case for RecEnable because its status can change as a
+        * confluence of unrelated parameters: (a) session rec-enable state (b)
+        * rec-enabled tracks. So we don't add the button to the blinkers list,
+        * we just call this:
+        *
+        *  * from the blink callback
+        *  * when the session tells us about a status change
+        *
+        * We do the last one so that the button changes state promptly rather
+        * than waiting for the next blink callback. The change in "blinking"
+        * based on having record-enabled tracks isn't urgent, and that happens
+        * during the blink callback.
+        */
+
+       bool onoff;
+
        switch (session->record_status()) {
        case Session::Disabled:
-               get_button (RecEnable).set_led_state (_output_port, false);
-               blinkers.remove (RecEnable);
+               onoff = false;
                break;
        case Session::Enabled:
-               get_button (RecEnable).set_led_state (_output_port, true);
-               blinkers.push_back (RecEnable);
+               onoff = blink_state;
                break;
        case Session::Recording:
-               get_button (RecEnable).set_led_state (_output_port, true);
-               blinkers.remove (RecEnable);
+               if (session->have_rec_enabled_track ()) {
+                       onoff = true;
+               } else {
+                       onoff = blink_state;
+               }
                break;
        }
+
+       if (onoff != rec_enable_state) {
+               get_button(RecEnable).set_led_state (_output_port, onoff);
+               rec_enable_state = onoff;
+       }
 }
 
 void
-FaderPort::notify_transport_state_changed ()
+FaderPort::map_transport_state ()
 {
        get_button (Loop).set_led_state (_output_port, session->get_play_loop());
-       get_button (Play).set_led_state (_output_port, session->transport_speed() == 1.0);
+
+       float ts = session->transport_speed();
+
+       if (ts == 0) {
+               stop_blinking (Play);
+       } else if (fabs (ts) == 1.0) {
+               stop_blinking (Play);
+               get_button (Play).set_led_state (_output_port, true);
+       } else {
+               start_blinking (Play);
+       }
+
        get_button (Stop).set_led_state (_output_port, session->transport_stopped ());
        get_button (Rewind).set_led_state (_output_port, session->transport_speed() < 0.0);
        get_button (Ffwd).set_led_state (_output_port, session->transport_speed() > 1.0);
 }
 
+void
+FaderPort::parameter_changed (string what)
+{
+       if (what == "punch-in" || what == "punch-out") {
+               bool in = session->config.get_punch_in ();
+               bool out = session->config.get_punch_out ();
+               if (in && out) {
+                       get_button (Punch).set_led_state (_output_port, true);
+                       blinkers.remove (Punch);
+               } else if (in || out) {
+                       start_blinking (Punch);
+               } else {
+                       stop_blinking (Punch);
+               }
+       }
+}
+
 void
 FaderPort::connect_session_signals()
 {
-       session->RecordStateChanged.connect(session_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::notify_record_state_changed, this), this);
-       session->TransportStateChange.connect(session_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::notify_transport_state_changed, this), this);
+       session->RecordStateChanged.connect(session_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_recenable_state, this), this);
+       session->TransportStateChange.connect(session_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_transport_state, this), this);
+       /* not session, but treat it similarly */
+       session->config.ParameterChanged.connect (session_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::parameter_changed, this, _1), this);
 }
 
 bool
@@ -556,10 +759,7 @@ FaderPort::midi_input_handler (Glib::IOCondition ioc, boost::shared_ptr<ARDOUR::
 
        if (ioc & IO_IN) {
 
-               if (port) {
-                       port->clear ();
-               }
-
+               port->clear ();
                DEBUG_TRACE (DEBUG::FaderPort, string_compose ("data available on %1\n", boost::shared_ptr<MIDI::Port>(port)->name()));
                framepos_t now = session->engine().sample_time();
                port->parse (now);
@@ -595,6 +795,7 @@ FaderPort::get_state ()
        node.add_child_nocopy (get_button (Proj).get_state());
        node.add_child_nocopy (get_button (Trns).get_state());
        node.add_child_nocopy (get_button (User).get_state());
+       node.add_child_nocopy (get_button (Footswitch).get_state());
 
        return node;
 }
@@ -645,6 +846,7 @@ FaderPort::set_state (const XMLNode& node, int version)
 bool
 FaderPort::connection_handler (boost::weak_ptr<ARDOUR::Port>, std::string name1, boost::weak_ptr<ARDOUR::Port>, std::string name2, bool yn)
 {
+       DEBUG_TRACE (DEBUG::FaderPort, "FaderPort::connection_handler  start\n");
        if (!_input_port || !_output_port) {
                return false;
        }
@@ -665,6 +867,7 @@ FaderPort::connection_handler (boost::weak_ptr<ARDOUR::Port>, std::string name1,
                        connection_state &= ~OutputConnected;
                }
        } else {
+               DEBUG_TRACE (DEBUG::FaderPort, string_compose ("Connections between %1 and %2 changed, but I ignored it\n", name1, name2));
                /* not our ports */
                return false;
        }
@@ -677,6 +880,7 @@ FaderPort::connection_handler (boost::weak_ptr<ARDOUR::Port>, std::string name1,
                */
 
                g_usleep (100000);
+                DEBUG_TRACE (DEBUG::FaderPort, "device now connected for both input and output\n");
                connected ();
 
        } else {
@@ -686,13 +890,15 @@ FaderPort::connection_handler (boost::weak_ptr<ARDOUR::Port>, std::string name1,
 
        ConnectionChange (); /* emit signal for our GUI */
 
+       DEBUG_TRACE (DEBUG::FaderPort, "FaderPort::connection_handler  end\n");
+
        return true; /* connection status changed */
 }
 
 void
 FaderPort::connected ()
 {
-       DEBUG_TRACE (DEBUG::FaderPort, "connection status changed\n");
+       DEBUG_TRACE (DEBUG::FaderPort, "sending device inquiry message...\n");
 
        start_midi_handling ();
 
@@ -713,24 +919,18 @@ FaderPort::connected ()
 void
 FaderPort::Button::invoke (FaderPort::ButtonState bs, bool press)
 {
-       if (!press) {
-               if (long_press == 1) {
-                       bs = FaderPort::ButtonState (bs | LongishPress);
-               } else if (long_press == 2) {
-                       bs = FaderPort::ButtonState (bs | LongPress);
-               }
-       }
+       DEBUG_TRACE (DEBUG::FaderPort, string_compose ("invoke button %1 for %2 state %3%4%5\n", id, (press ? "press":"release"), hex, bs, dec));
 
        ToDoMap::iterator x;
 
        if (press) {
                if ((x = on_press.find (bs)) == on_press.end()) {
-                       DEBUG_TRACE (DEBUG::FaderPort, string_compose ("no press action for button %1 state %2%3\%4\n", id, hex, bs, dec));
+                       DEBUG_TRACE (DEBUG::FaderPort, string_compose ("no press action for button %1 state %2 @ %3 in %4\n", id, bs, this, &on_press));
                        return;
                }
        } else {
                if ((x = on_release.find (bs)) == on_release.end()) {
-                       DEBUG_TRACE (DEBUG::FaderPort, string_compose ("no release action for button %1 state %2%3\%4\n", id, hex, bs, dec));
+                       DEBUG_TRACE (DEBUG::FaderPort, string_compose ("no release action for button %1 state %2 @%3 in %4\n", id, bs, this, &on_release));
                        return;
                }
        }
@@ -748,27 +948,6 @@ FaderPort::Button::invoke (FaderPort::ButtonState bs, bool press)
        }
 }
 
-void
-FaderPort::Button::do_timing (bool press)
-{
-       if (press) {
-               pressed_at = get_microseconds ();
-               long_press = 0;
-       } else {
-               if (pressed_at > 0) {
-                       const ARDOUR::microseconds_t delta = ARDOUR::get_microseconds () - pressed_at;
-                       if (delta < 500000) {
-                               long_press = 0;
-                       } else if (delta < 1000000) {
-                               long_press = 1;
-                       } else {
-                               long_press = 2;
-                       }
-                       pressed_at = 0;
-               }
-       }
-}
-
 void
 FaderPort::Button::set_action (string const& name, bool when_pressed, FaderPort::ButtonState bs)
 {
@@ -826,22 +1005,19 @@ FaderPort::Button::set_action (boost::function<void()> f, bool when_pressed, Fad
        todo.type = InternalFunction;
 
        if (when_pressed) {
+               DEBUG_TRACE (DEBUG::FaderPort, string_compose ("set button %1 (%2) @ %5 to some functor on press + %3 in %4\n", id, name, bs, &on_press, this));
                todo.function = f;
                on_press[bs] = todo;
        } else {
+               DEBUG_TRACE (DEBUG::FaderPort, string_compose ("set button %1 (%2) @ %5 to some functor on release + %3\n", id, name, bs, this));
                todo.function = f;
                on_release[bs] = todo;
        }
 }
 
 void
-FaderPort::Button::set_led_state (boost::shared_ptr<MIDI::Port> port, int onoff, bool force)
+FaderPort::Button::set_led_state (boost::shared_ptr<MIDI::Port> port, bool onoff)
 {
-       if (!force && (led_on == (bool) onoff)) {
-               /* nothing to do */
-               return;
-       }
-
        if (out < 0) {
                /* fader button ID - no LED */
                return;
@@ -852,7 +1028,6 @@ FaderPort::Button::set_led_state (boost::shared_ptr<MIDI::Port> port, int onoff,
        buf[1] = out;
        buf[2] = onoff ? 1 : 0;
        port->write (buf, 3, 0);
-       led_on = (onoff ? true : false);
 }
 
 int
@@ -873,12 +1048,8 @@ FaderPort::Button::set_state (XMLNode const& node)
 
        state_pairs.push_back (make_pair (string ("plain"), ButtonState (0)));
        state_pairs.push_back (make_pair (string ("shift"), ShiftDown));
-       state_pairs.push_back (make_pair (string ("longish"), LongishPress));
        state_pairs.push_back (make_pair (string ("long"), LongPress));
 
-       on_press.clear ();
-       on_release.clear ();
-
        for (vector<state_pair_t>::const_iterator sp = state_pairs.begin(); sp != state_pairs.end(); ++sp) {
                string propname;
 
@@ -914,7 +1085,6 @@ FaderPort::Button::get_state () const
 
        state_pairs.push_back (make_pair (string ("plain"), ButtonState (0)));
        state_pairs.push_back (make_pair (string ("shift"), ShiftDown));
-       state_pairs.push_back (make_pair (string ("longish"), LongishPress));
        state_pairs.push_back (make_pair (string ("long"), LongPress));
 
        for (vector<state_pair_t>::const_iterator sp = state_pairs.begin(); sp != state_pairs.end(); ++sp) {
@@ -935,111 +1105,155 @@ FaderPort::Button::get_state () const
 }
 
 void
-FaderPort::gui_track_selection_changed (RouteNotificationListPtr routes)
+FaderPort::gui_track_selection_changed (StripableNotificationListPtr stripables)
 {
-       boost::shared_ptr<Route> r;
+       boost::shared_ptr<Stripable> r;
 
-       if (!routes->empty()) {
-               r = routes->front().lock();
+       if (!stripables->empty()) {
+               r = stripables->front().lock();
        }
 
-       set_current_route (r);
+       set_current_stripable (r);
 }
 
 void
-FaderPort::drop_current_route ()
+FaderPort::drop_current_stripable ()
 {
-       if (_current_route) {
-               if (_current_route == session->monitor_out()) {
-                       set_current_route (session->master_out());
+       if (_current_stripable) {
+               if (_current_stripable == session->monitor_out()) {
+                       set_current_stripable (session->master_out());
                } else {
-                       set_current_route (boost::shared_ptr<Route>());
+                       set_current_stripable (boost::shared_ptr<Stripable>());
                }
        }
 }
 
 void
-FaderPort::set_current_route (boost::shared_ptr<Route> r)
+FaderPort::set_current_stripable (boost::shared_ptr<Stripable> r)
 {
-       route_connections.drop_connections ();
+       stripable_connections.drop_connections ();
 
-       _current_route = r;
+       _current_stripable = r;
 
        /* turn this off. It will be turned on back on in use_master() or
           use_monitor() as appropriate.
        */
        get_button(Output).set_led_state (_output_port, false);
 
-       if (_current_route) {
-               _current_route->DropReferences.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::drop_current_route, this), this);
+       if (_current_stripable) {
+               _current_stripable->DropReferences.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::drop_current_stripable, this), this);
 
-               _current_route->mute_changed.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_mute, this, _1), this);
-               _current_route->solo_changed.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_solo, this, _1, _2, _3), this);
-               _current_route->listen_changed.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_listen, this, _1, _2), this);
+               _current_stripable->mute_control()->Changed.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_mute, this), this);
+               _current_stripable->solo_control()->Changed.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_solo, this), this);
 
-               boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track> (_current_route);
+               boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track> (_current_stripable);
                if (t) {
-                       t->RecordEnableChanged.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_recenable, this), this);
+                       t->rec_enable_control()->Changed.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_recenable, this), this);
                }
 
-               boost::shared_ptr<AutomationControl> control = _current_route->gain_control ();
+               boost::shared_ptr<AutomationControl> control = _current_stripable->gain_control ();
                if (control) {
-                       control->Changed.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_gain, this), this);
+                       control->Changed.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_gain, this), this);
+                       control->alist()->automation_state_changed.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_auto, this), this);
                }
 
-               boost::shared_ptr<MonitorProcessor> mp = _current_route->monitor_control();
+               boost::shared_ptr<MonitorProcessor> mp = _current_stripable->monitor_control();
                if (mp) {
-                       mp->cut_control()->Changed.connect (route_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_cut, this), this);
+                       mp->cut_control()->Changed.connect (stripable_connections, MISSING_INVALIDATOR, boost::bind (&FaderPort::map_cut, this), this);
                }
        }
 
        //ToDo: subscribe to the fader automation modes so we can light the LEDs
 
-       map_route_state ();
+       map_stripable_state ();
 }
 
+void
+FaderPort::map_auto ()
+{
+       /* Under no circumstances send a message to "enable" the LED state of
+        * the Off button, because this will disable the fader.
+        */
+
+       boost::shared_ptr<AutomationControl> control = _current_stripable->gain_control ();
+       const AutoState as = control->automation_state ();
+
+       switch (as) {
+               case ARDOUR::Play:
+                       get_button (FP_Read).set_led_state (_output_port, true);
+                       get_button (FP_Write).set_led_state (_output_port, false);
+                       get_button (FP_Touch).set_led_state (_output_port, false);
+               break;
+               case ARDOUR::Write:
+                       get_button (FP_Read).set_led_state (_output_port, false);
+                       get_button (FP_Write).set_led_state (_output_port, true);
+                       get_button (FP_Touch).set_led_state (_output_port, false);
+               break;
+               case ARDOUR::Touch:
+                       get_button (FP_Read).set_led_state (_output_port, false);
+                       get_button (FP_Write).set_led_state (_output_port, false);
+                       get_button (FP_Touch).set_led_state (_output_port, true);
+               break;
+               case ARDOUR::Off:
+                       get_button (FP_Read).set_led_state (_output_port, false);
+                       get_button (FP_Write).set_led_state (_output_port, false);
+                       get_button (FP_Touch).set_led_state (_output_port, false);
+               break;
+       }
+
+}
+
+
 void
 FaderPort::map_cut ()
 {
-       boost::shared_ptr<MonitorProcessor> mp = _current_route->monitor_control();
+       boost::shared_ptr<MonitorProcessor> mp = _current_stripable->monitor_control();
 
        if (mp) {
                bool yn = mp->cut_all ();
-               get_button (Mute).set_led_state (_output_port, yn);
                if (yn) {
-                       blinkers.push_back (Mute);
+                       start_blinking (Mute);
                } else {
-                       blinkers.remove (Mute);
+                       stop_blinking (Mute);
                }
        } else {
-               blinkers.remove (Mute);
+               stop_blinking (Mute);
        }
 }
 
 void
-FaderPort::map_mute (void*)
-{
-       get_button (Mute).set_led_state (_output_port, _current_route->muted());
-}
-
-void
-FaderPort::map_solo (bool, void*, bool)
+FaderPort::map_mute ()
 {
-       get_button (Solo).set_led_state (_output_port, _current_route->soloed() || _current_route->listening_via_monitor());
+       if (_current_stripable) {
+               if (_current_stripable->mute_control()->muted()) {
+                       stop_blinking (Mute);
+                       get_button (Mute).set_led_state (_output_port, true);
+               } else if (_current_stripable->mute_control()->muted_by_others_soloing () || _current_stripable->mute_control()->muted_by_masters()) {
+                       start_blinking (Mute);
+               } else {
+                       stop_blinking (Mute);
+               }
+       } else {
+               stop_blinking (Mute);
+       }
 }
 
 void
-FaderPort::map_listen (void*, bool)
+FaderPort::map_solo ()
 {
-       get_button (Solo).set_led_state (_output_port, _current_route->listening_via_monitor());
+       if (_current_stripable) {
+               get_button (Solo).set_led_state (_output_port, _current_stripable->solo_control()->soloed());
+       } else {
+               get_button (Solo).set_led_state (_output_port, false);
+       }
 }
 
 void
 FaderPort::map_recenable ()
 {
-       boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track> (_current_route);
+       boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track> (_current_stripable);
        if (t) {
-               get_button (Rec).set_led_state (_output_port, t->record_enabled());
+               get_button (Rec).set_led_state (_output_port, t->rec_enable_control()->get_value());
        } else {
                get_button (Rec).set_led_state (_output_port, false);
        }
@@ -1053,11 +1267,11 @@ FaderPort::map_gain ()
                return;
        }
 
-       if (!_current_route) {
+       if (!_current_stripable) {
                return;
        }
 
-       boost::shared_ptr<AutomationControl> control = _current_route->gain_control ();
+       boost::shared_ptr<AutomationControl> control = _current_stripable->gain_control ();
        double val;
 
        if (!control) {
@@ -1096,22 +1310,37 @@ FaderPort::map_gain ()
 }
 
 void
-FaderPort::map_route_state ()
+FaderPort::map_stripable_state ()
 {
-       if (!_current_route) {
-               get_button (Mute).set_led_state (_output_port, false);
-               get_button (Solo).set_led_state (_output_port, false);
+       if (!_current_stripable) {
+               stop_blinking (Mute);
+               stop_blinking (Solo);
                get_button (Rec).set_led_state (_output_port, false);
-               blinkers.remove (Mute);
-               blinkers.remove (Solo);
        } else {
-               /* arguments to these map_*() methods are all ignored */
-               map_mute (0);
-               map_solo (false, 0, false);
+               map_solo ();
                map_recenable ();
                map_gain ();
-               map_cut ();
+               map_auto ();
+
+               if (_current_stripable == session->monitor_out()) {
+                       map_cut ();
+               } else {
+                       map_mute ();
+               }
+       }
+}
+
+list<boost::shared_ptr<ARDOUR::Bundle> >
+FaderPort::bundles ()
+{
+       list<boost::shared_ptr<ARDOUR::Bundle> > b;
+
+       if (_input_bundle) {
+               b.push_back (_input_bundle);
+               b.push_back (_output_bundle);
        }
+
+       return b;
 }
 
 boost::shared_ptr<Port>