fix missing negation from previous MCP commit
[ardour.git] / libs / surfaces / mackie / surface.cc
index 4374ae2d302d0a4dd2e55bdd03b3894db13b470c..0cc0f02a77fb8ab18dba52181d5ba46056e7b9fc 100644 (file)
@@ -7,18 +7,21 @@
 #include "midi++/port.h"
 #include "midi++/manager.h"
 
+#include "ardour/automation_control.h"
 #include "ardour/debug.h"
 #include "ardour/route.h"
 #include "ardour/panner.h"
 #include "ardour/panner_shell.h"
 #include "ardour/rc_configuration.h"
+#include "ardour/session.h"
+#include "ardour/utils.h"
 
 #include "control_group.h"
 #include "surface_port.h"
 #include "surface.h"
 #include "strip.h"
 #include "mackie_control_protocol.h"
-#include "mackie_jog_wheel.h"
+#include "jog_wheel.h"
 
 #include "strip.h"
 #include "button.h"
@@ -36,7 +39,9 @@ using namespace Mackie;
 using ARDOUR::Route;
 using ARDOUR::Panner;
 using ARDOUR::Pannable;
-using ARDOUR::PannerShell;
+using ARDOUR::AutomationControl;
+
+#define ui_context() MackieControlProtocol::instance() /* a UICallback-derived object that specifies the event loop for signal handling */
 
 // The MCU sysex header.4th byte Will be overwritten
 // when we get an incoming sysex that identifies
@@ -50,87 +55,12 @@ static MidiByteArray mackie_sysex_hdr_xt  (5, MIDI::sysex, 0x0, 0x0, 0x66, 0x15)
 
 static MidiByteArray empty_midi_byte_array;
 
-static GlobalControlDefinition mackie_global_controls[] = {
-       { "jog", 0x3c, Jog::factory, "none" },
-       { "external", 0x2e, Pot::factory, "none" },
-       { "io", 0x28, Button::factory, "assignment" },
-       { "sends", 0x29, Button::factory, "assignment" },
-       { "pan", 0x2a, Button::factory, "assignment" },
-       { "plugin", 0x2b, Button::factory, "assignment" },
-       { "eq", 0x2c, Button::factory, "assignment" },
-       { "dyn", 0x2d, Button::factory, "assignment" },
-       { "left", 0x2e, Button::factory, "bank" },
-       { "right", 0x2f, Button::factory, "bank" },
-       { "channel_left", 0x30, Button::factory, "bank" },
-       { "channel_right", 0x31, Button::factory, "bank" },
-       { "flip", 0x32, Button::factory, "none" },
-       { "edit", 0x33, Button::factory, "none" },
-       { "name_value", 0x34, Button::factory, "display" },
-       { "timecode_beats", 0x35, Button::factory, "display" },
-       { "F1", Button::F1, Button::factory, "none" },
-       { "F2", Button::F2, Button::factory, "none" },
-       { "F3", Button::F3, Button::factory, "none" },
-       { "F4", Button::F4, Button::factory, "none" },
-       { "F5", Button::F5, Button::factory, "none" },
-       { "F6", Button::F6, Button::factory, "none" },
-       { "F7", Button::F7, Button::factory, "none" },
-       { "F8", Button::F8, Button::factory, "none" },
-       { "F9", Button::F9, Button::factory, "none" },
-       { "F10", Button::F10, Button::factory, "none" },
-       { "F11", Button::F11, Button::factory, "none" },
-       { "F12", Button::F12, Button::factory, "none" },
-       { "F13", Button::F13, Button::factory, "none" },
-       { "F14", Button::F14, Button::factory, "none" },
-       { "F15", Button::F15, Button::factory, "none" },
-       { "F16", Button::F16, Button::factory, "none" },
-       { "shift", 0x46, Button::factory, "modifiers" },
-       { "option", 0x47, Button::factory, "modifiers" },
-       { "control", 0x48, Button::factory, "modifiers" },
-       { "cmd_alt", 0x49, Button::factory, "modifiers" },
-       { "on", 0x4a, Button::factory, "automation" },
-       { "rec_ready", 0x4b, Button::factory, "automation" },
-       { "undo", 0x4c, Button::factory, "functions" },
-       { "save", Button::Save, Button::factory, "automation" },
-       { "touch", Button::Touch, Button::factory, "automation" },
-       { "redo", Button::Redo, Button::factory, "functions" },
-       { "marker", Button::Marker, Button::factory, "functions" },
-       { "enter", Button::Enter, Button::factory, "functions" },
-       { "cancel", Button::Cancel, Button::factory, "functions" },
-       { "mixer", Button::Mixer, Button::factory, "functions" },
-       { "frm_left", 0x54, Button::factory, "transport" },
-       { "frm_right", 0x55, Button::factory, "transport" },
-       { "loop", Button::Loop, Button::factory, "transport" },
-       { "punch_in", 0x57, Button::factory, "transport" },
-       { "punch_out", 0x58, Button::factory, "transport" },
-       { "home", Button::Home, Button::factory, "transport" },
-       { "end", Button::End, Button::factory, "transport" },
-       { "rewind", Button::Rewind, Button::factory, "transport" },
-       { "ffwd", Button::Ffwd, Button::factory, "transport" },
-       { "stop", Button::Stop, Button::factory, "transport" },
-       { "play", Button::Play, Button::factory, "transport" },
-       { "record", Button::Record, Button::factory, "transport" },
-       { "cursor_up", Button::CursorUp, Button::factory, "cursor" },
-       { "cursor_down", Button::CursorDown, Button::factory, "cursor" },
-       { "cursor_left", Button::CursorLeft, Button::factory, "cursor" },
-       { "cursor_right", Button::CursorRight, Button::factory, "cursor" },
-       { "zoom", Button::Zoom, Button::factory, "none" },
-       { "scrub", Button::Scrub, Button::factory, "none" },
-       { "user_a", Button::UserA, Button::factory, "user" },
-       { "user_b", Button::UserB, Button::factory, "user" },
-       { "fader_touch", 0x70, Led::factory, "master" },
-       { "timecode", 0x71, Led::factory, "none" },
-       { "beats", 0x72, Led::factory, "none" },
-       { "solo", 0x73, Led::factory, "none" },
-       { "relay_click", 0x73, Led::factory, "none" },
-       { "", 0, Button::factory, "" }
-};
-       
 Surface::Surface (MackieControlProtocol& mcp, const std::string& device_name, uint32_t number, surface_type_t stype)
        : _mcp (mcp)
        , _stype (stype)
        , _number (number)
        , _name (device_name)
-       , _active (true)
+       , _active (false)
        , _connected (false)
        , _jog_wheel (0)
 {
@@ -138,12 +68,16 @@ Surface::Surface (MackieControlProtocol& mcp, const std::string& device_name, ui
        
        _port = new SurfacePort (*this);
 
-       if (_mcp.device_info().has_global_controls()) {
-               init_controls ();
-       }
+       /* only the first Surface object has global controls */
 
-       if (_mcp.device_info().has_jog_wheel()) {
-               _jog_wheel = new Mackie::JogWheel (_mcp);
+       if (_number == 0) {
+               if (_mcp.device_info().has_global_controls()) {
+                       init_controls ();
+               }
+
+               if (_mcp.device_info().has_master_fader()) {
+                       setup_master ();
+               }
        }
 
        uint32_t n = _mcp.device_info().strip_cnt();
@@ -151,20 +85,9 @@ Surface::Surface (MackieControlProtocol& mcp, const std::string& device_name, ui
        if (n) {
                init_strips (n);
        }
-
+       
        connect_to_signals ();
 
-       /* wakey wakey */
-
-       MidiByteArray wakeup (7, MIDI::sysex, 0x00, 0x00, 0x66, 0x14, 0x00, MIDI::eox);
-       _port->write (wakeup);
-       wakeup[4] = 0x15; /* wakup Mackie XT */
-       _port->write (wakeup);
-       wakeup[4] = 0x10; /* wakupe Logic Control */
-       _port->write (wakeup);
-       wakeup[4] = 0x11; /* wakeup Logic Control XT */
-       _port->write (wakeup);
-
        DEBUG_TRACE (DEBUG::MackieControl, "Surface::init finish\n");
 }
 
@@ -172,12 +95,7 @@ Surface::~Surface ()
 {
        DEBUG_TRACE (DEBUG::MackieControl, "Surface: destructor\n");
 
-       // faders to minimum
-       write_sysex (0x61);
-       // All LEDs off
-       write_sysex (0x62);
-       // Reset (reboot into offline mode)
-       // _write_sysex (0x63);
+       zero_all ();
 
        // delete groups
        for (Groups::iterator it = groups.begin(); it != groups.end(); ++it) {
@@ -204,6 +122,16 @@ Surface::sysex_hdr() const
        return mackie_sysex_hdr;
 }
 
+static GlobalControlDefinition mackie_global_controls[] = {
+       { "external", Pot::External, Pot::factory, "none" },
+       { "fader_touch", Led::FaderTouch, Led::factory, "master" },
+       { "timecode", Led::Timecode, Led::factory, "none" },
+       { "beats", Led::Beats, Led::factory, "none" },
+       { "solo", Led::RudeSolo, Led::factory, "none" },
+       { "relay_click", Led::RelayClick, Led::factory, "none" },
+       { "", 0, Led::factory, "" }
+};
+
 void 
 Surface::init_controls()
 {
@@ -220,49 +148,80 @@ Surface::init_controls()
        groups["transport"] = new Group  ("transport");
        groups["user"] = new Group  ("user");
        groups["master"] = new Group ("master");
+       groups["view"] = new Group ("view");
+               
+       if (_mcp.device_info().has_jog_wheel()) {
+               _jog_wheel = new Mackie::JogWheel (_mcp);
+       }
 
        for (uint32_t n = 0; mackie_global_controls[n].name[0]; ++n) {
                group = groups[mackie_global_controls[n].group_name];
                Control* control = mackie_global_controls[n].factory (*this, mackie_global_controls[n].id, mackie_global_controls[n].name, *group);
-               controls_by_name[mackie_global_controls[n].name] = control;
-               group->add (*control);
-       }
-}
-
-static StripControlDefinition mackie_strip_controls[] = {
-       { "gain", 0, Fader::factory, },
-       { "vpot", Pot::base_id, Pot::factory, },
-       { "recenable", Button::recenable_base_id, Button::factory, },
-       { "solo", Button::solo_base_id, Button::factory, },
-       { "mute", Button::mute_base_id, Button::factory, },
-       { "select", Button::select_base_id, Button::factory, },
-       { "vselect", Button::vselect_base_id, Button::factory, },
-       { "fader_touch", Button::fader_touch_base_id, Button::factory, },
-       { "meter", 0, Meter::factory, },
-       { "", 0, Button::factory, }
-};
+               controls_by_device_independent_id[mackie_global_controls[n].id] = control;
+       }
+
+       /* add global buttons */
+
+       const map<Button::ID,GlobalButtonInfo>& global_buttons (_mcp.device_info().global_buttons());
+
+       for (map<Button::ID,GlobalButtonInfo>::const_iterator b = global_buttons.begin(); b != global_buttons.end(); ++b){
+               group = groups[b->second.group];
+               controls_by_device_independent_id[b->first] = Button::factory (*this, b->first, b->second.id, b->second.label, *group);
+       }
+}
 
 void 
 Surface::init_strips (uint32_t n)
 {
+       const map<Button::ID,StripButtonInfo>& strip_buttons (_mcp.device_info().strip_buttons());
+
        for (uint32_t i = 0; i < n; ++i) {
 
                char name[32];
                
                snprintf (name, sizeof (name), "strip_%d", (8* _number) + i);
 
-               Strip* strip = new Strip (*this, name, i, mackie_strip_controls);
+               Strip* strip = new Strip (*this, name, i, strip_buttons);
                
                groups[name] = strip;
                strips.push_back (strip);
        }
 }
 
+void
+Surface::setup_master ()
+{
+       _master_fader = dynamic_cast<Fader*> (Fader::factory (*this, 8, "master", *groups["master"]));
+       
+       boost::shared_ptr<Route> m;
+       
+       if ((m = _mcp.get_session().monitor_out()) == 0) {
+               m = _mcp.get_session().master_out();
+       } 
+       
+       if (!m) {
+               return;
+       }
+       
+       _master_fader->set_control (m->gain_control());
+       m->gain_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&Surface::master_gain_changed, this), ui_context());
+}
+
+void
+Surface::master_gain_changed ()
+{
+       boost::shared_ptr<AutomationControl> ac = _master_fader->control();
+       float pos = ac->internal_to_interface (ac->get_value());
+       _port->write (_master_fader->set_position (pos));
+}
+
 float 
 Surface::scaled_delta (float delta, float current_speed)
 {
        /* XXX needs work before use */
-       return (std::pow (float(delta + 1), 2) + current_speed) / 100.0;
+       const float sign = delta < 0.0 ? -1.0 : 1.0;
+
+       return ((sign * std::pow (delta + 1.0, 2.0)) + current_speed) / 100.0;
 }
 
 void 
@@ -270,32 +229,26 @@ Surface::display_bank_start (uint32_t current_bank)
 {
        if  (current_bank == 0) {
                // send Ar. to 2-char display on the master
-               _port->write (two_char_display ("Ar", ".."));
+               show_two_char_display ("Ar", "..");
        } else {
                // write the current first remote_id to the 2-char display
-               _port->write (two_char_display (current_bank));
+               show_two_char_display (current_bank);
        }
 }
 
 void 
 Surface::blank_jog_ring ()
 {
-       Control* control = controls_by_name["jog"];
+       Control* control = controls_by_device_independent_id[Jog::ID];
 
        if (control) {
                Pot* pot = dynamic_cast<Pot*> (control);
                if (pot) {
-                       _port->write (pot->set_onoff (false));
+                       _port->write (pot->set (0.0, false, Pot::spread));
                }
        }
 }
 
-bool 
-Surface::has_timecode_display () const
-{
-       return false;
-}
-
 float
 Surface::scrub_scaling_factor () const
 {
@@ -319,6 +272,8 @@ Surface::connect_to_signals ()
                p->controller.connect_same_thread (*this, boost::bind (&Surface::handle_midi_controller_message, this, _1, _2));
                /* Button messages are NoteOn */
                p->note_on.connect_same_thread (*this, boost::bind (&Surface::handle_midi_note_on_message, this, _1, _2));
+               /* Button messages are NoteOn. libmidi++ sends note-on w/velocity = 0 as note-off so catch them too */
+               p->note_off.connect_same_thread (*this, boost::bind (&Surface::handle_midi_note_on_message, this, _1, _2));
                /* Fader messages are Pitchbend */
                p->channel_pitchbend[0].connect_same_thread (*this, boost::bind (&Surface::handle_midi_pitchbend_message, this, _1, _2, 0U));
                p->channel_pitchbend[1].connect_same_thread (*this, boost::bind (&Surface::handle_midi_pitchbend_message, this, _1, _2, 1U));
@@ -344,18 +299,25 @@ Surface::handle_midi_pitchbend_message (MIDI::Parser&, MIDI::pitchbend_t pb, uin
         * when we connected to the per-channel pitchbend events.
         */
 
+
        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("handle_midi pitchbend on port %3, fader = %1 value = %2\n", 
                                                           fader_id, pb, _number));
        
+       if (_mcp.device_info().no_handshake()) {
+               turn_it_on ();
+       }
+
        Fader* fader = faders[fader_id];
 
        if (fader) {
                Strip* strip = dynamic_cast<Strip*> (&fader->group());
+               float pos = (pb >> 4)/1023.0; // only the top 10 bytes are used
                if (strip) {
-                       float midi_pos = pb >> 4; // only the top 10 bytes are used
-                       strip->handle_fader (*fader, midi_pos/1023.0);
+                       strip->handle_fader (*fader, pos);
                } else {
                        /* master fader */
+                       fader->set_value (pos); // alter master gain
+                       _port->write (fader->set_position (pos)); // write back value (required for servo)
                }
        } else {
                DEBUG_TRACE (DEBUG::MackieControl, "fader not found\n");
@@ -367,6 +329,10 @@ Surface::handle_midi_note_on_message (MIDI::Parser &, MIDI::EventTwoBytes* ev)
 {
        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("SurfacePort::handle_note_on %1 = %2\n", (int) ev->note_number, (int) ev->velocity));
        
+       if (_mcp.device_info().no_handshake()) {
+               turn_it_on ();
+       }
+
        Button* button = buttons[ev->note_number];
 
        if (button) {
@@ -374,12 +340,12 @@ Surface::handle_midi_note_on_message (MIDI::Parser &, MIDI::EventTwoBytes* ev)
 
                if (strip) {
                        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("strip %1 button %2 pressed ? %3\n",
-                                                                          strip->index(), button->name(), (ev->velocity == 0x7f)));
-                       strip->handle_button (*button, ev->velocity == 0x7f ? press : release);
+                                                                          strip->index(), button->name(), (ev->velocity > 64)));
+                       strip->handle_button (*button, ev->velocity > 64 ? press : release);
                } else {
                        /* global button */
                        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("global button %1\n", button->id()));
-                       _mcp.handle_button_event (*this, *button, ev->velocity == 0x7f ? press : release);
+                       _mcp.handle_button_event (*this, *button, ev->velocity > 64 ? press : release);
                }
        } else {
                DEBUG_TRACE (DEBUG::MackieControl, string_compose ("no button found for %1\n", ev->note_number));
@@ -391,45 +357,40 @@ Surface::handle_midi_controller_message (MIDI::Parser &, MIDI::EventTwoBytes* ev
 {
        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("SurfacePort::handle_midi_controller %1 = %2\n", (int) ev->controller_number, (int) ev->value));
 
+       if (_mcp.device_info().no_handshake()) {
+               turn_it_on ();
+       }
+
        Pot* pot = pots[ev->controller_number];
 
-       if (!pot && ev->controller_number == Jog::base_id) {
-               pot = dynamic_cast<Pot*> (controls_by_name["jog"]);
+       // bit 6 gives the sign
+       float sign = (ev->value & 0x40) == 0 ? 1.0 : -1.0; 
+       // bits 0..5 give the velocity. we interpret this as "ticks
+       // moved before this message was sent"
+       float ticks = (ev->value & 0x3f);
+       if (ticks == 0) {
+               /* euphonix and perhaps other devices send zero
+                  when they mean 1, we think.
+               */
+               ticks = 1;
        }
+       float delta = sign * (ticks / (float) 0x3f);
+       
+       if (!pot) {
+               if (ev->controller_number == Jog::ID && _jog_wheel) {
 
-       if (pot) {
-               ControlState state;
-               
-               // bit 6 gives the sign
-               float sign = (ev->value & 0x40) == 0 ? 1.0 : -1.0; 
-               // bits 0..5 give the velocity. we interpret this as "ticks
-               // moved before this message was sent"
-               float ticks = (ev->value & 0x3f);
-               if (ticks == 0) {
-                       /* euphonix and perhaps other devices send zero
-                          when they mean 1, we think.
-                       */
-                       ticks = 1;
+                       DEBUG_TRACE (DEBUG::MackieControl, string_compose ("Jog wheel moved %1\n", ticks));
+                       _jog_wheel->jog_event (delta);
+                       return;
                }
-               float delta = sign * (ticks / (float) 0x3f);
-
-               Strip* strip = dynamic_cast<Strip*> (&pot->group());
 
-               if (strip) {
-                       strip->handle_pot (*pot, delta);
-               } else {
-                       JogWheel* wheel = dynamic_cast<JogWheel*> (pot);
-                       if (wheel) {
-                               DEBUG_TRACE (DEBUG::MackieControl, string_compose ("Jog wheel moved %1\n", state.ticks));
-                               wheel->jog_event (*_port, *pot, delta);
-                       } else {
-                               DEBUG_TRACE (DEBUG::MackieControl, string_compose ("External controller moved %1\n", state.ticks));
-                               cout << "external controller" << delta << endl;
-                       }
-               }
-       } else {
-               DEBUG_TRACE (DEBUG::MackieControl, "pot not found\n");
+               return;
        }
+
+       Strip* strip = dynamic_cast<Strip*> (&pot->group());
+       if (strip) {
+               strip->handle_pot (*pot, delta);
+       } 
 }
 
 void 
@@ -437,9 +398,12 @@ Surface::handle_midi_sysex (MIDI::Parser &, MIDI::byte * raw_bytes, size_t count
 {
        MidiByteArray bytes (count, raw_bytes);
 
-
        DEBUG_TRACE (DEBUG::MackieControl, string_compose ("handle_midi_sysex: %1\n", bytes));
 
+       if (_mcp.device_info().no_handshake()) {
+               turn_it_on ();
+       }
+
        /* always save the device type ID so that our outgoing sysex messages
         * are correct 
         */
@@ -458,7 +422,9 @@ Surface::handle_midi_sysex (MIDI::Parser &, MIDI::byte * raw_bytes, size_t count
                if (bytes[4] == 0x10 || bytes[4] == 0x11) {
                        write_sysex (host_connection_query (bytes));
                } else {
-                       _active = true;
+                       if (!_active) {
+                               turn_it_on ();
+                       }
                }
                break;
 
@@ -539,8 +505,21 @@ Surface::host_connection_confirmation (const MidiByteArray & bytes)
        return MidiByteArray (2, 0x13, 0x00);
 }
 
+void
+Surface::turn_it_on ()
+{
+       if (!_active) {
+               _active = true;
+               zero_controls ();
+               for (Strips::iterator s = strips.begin(); s != strips.end(); ++s) {
+                       (*s)->notify_all ();
+               }
+               update_view_mode_display ();
+       }
+}
+
 void 
-Surface::handle_port_inactive (SurfacePort * port)
+Surface::handle_port_inactive (SurfacePort*)
 {
        _active = false;
 }
@@ -565,18 +544,21 @@ Surface::write_sysex (MIDI::byte msg)
        _port->write (buf);
 }
 
-void
-Surface::drop_routes ()
-{
-       for (Strips::iterator s = strips.begin(); s != strips.end(); ++s) {
-               (*s)->set_route (boost::shared_ptr<Route>());
-       }
-}
-
 uint32_t
-Surface::n_strips () const
+Surface::n_strips (bool with_locked_strips) const
 {
-       return strips.size();
+       if (with_locked_strips) {
+               return strips.size();
+       } 
+
+       uint32_t n = 0;
+
+       for (Strips::const_iterator it = strips.begin(); it != strips.end(); ++it) {
+               if (!(*it)->locked()) {
+                       ++n;
+               }
+       }
+       return n;
 }
 
 Strip*
@@ -595,7 +577,17 @@ Surface::zero_all ()
 
        // zero all strips
        for (Strips::iterator it = strips.begin(); it != strips.end(); ++it) {
-               _port->write ((*it)->zero());
+               (*it)->zero();
+       }
+
+       zero_controls ();
+}
+
+void
+Surface::zero_controls ()
+{
+       if (_stype != mcu || !_mcp.device_info().has_global_controls()) {
+               return;
        }
 
        // turn off global buttons and leds
@@ -609,9 +601,11 @@ Surface::zero_all ()
                }
        }
 
-       // any hardware-specific stuff
-       // clear 2-char display
-       _port->write (two_char_display ("  "));
+       if (_number == 0 && _mcp.device_info().has_two_character_display()) {
+               // any hardware-specific stuff
+               // clear 2-char display
+               show_two_char_display (" ");
+       }
 
        // and the led ring for the master strip
        blank_jog_ring ();
@@ -623,37 +617,13 @@ Surface::periodic (uint64_t now_usecs)
        for (Strips::iterator s = strips.begin(); s != strips.end(); ++s) {
                (*s)->periodic (now_usecs);
        }
-
 }
 
 void
 Surface::write (const MidiByteArray& data) 
 {
-       _port->write (data);
-}
-
-void 
-Surface::jog_wheel_state_display (JogWheel::State state)
-{
-       switch (state) {
-       case JogWheel::zoom:
-                       _port->write (two_char_display ("Zm"));
-                       break;
-               case JogWheel::scroll:
-                       _port->write (two_char_display ("Sc"));
-                       break;
-               case JogWheel::scrub:
-                       _port->write (two_char_display ("Sb"));
-                       break;
-               case JogWheel::shuttle:
-                       _port->write (two_char_display ("Sh"));
-                       break;
-               case JogWheel::speed:
-                       _port->write (two_char_display ("Sp"));
-                       break;
-               case JogWheel::select:
-                       _port->write (two_char_display ("Se"));
-                       break;
+       if (_active) {
+               _port->write (data);
        }
 }
 
@@ -661,69 +631,75 @@ void
 Surface::map_routes (const vector<boost::shared_ptr<Route> >& routes)
 {
        vector<boost::shared_ptr<Route> >::const_iterator r;
-       Strips::iterator s;
+       Strips::iterator s = strips.begin();
 
-       for (s = strips.begin(); s != strips.end(); ++s) {
-               (*s)->set_route (boost::shared_ptr<Route>());
+       for (r = routes.begin(); r != routes.end() && s != strips.end(); ++s) {
+
+               /* don't try to assign routes to a locked strip. it won't
+                  use it anyway, but if we do, then we get out of sync
+                  with the proposed mapping.
+               */
+
+               if (!(*s)->locked()) {
+                       (*s)->set_route (*r);
+                       ++r;
+               }
        }
 
-       for (r = routes.begin(), s = strips.begin(); r != routes.end() && s != strips.end(); ++r, ++s) {
-               (*s)->set_route (*r);
+       for (; s != strips.end(); ++s) {
+               (*s)->set_route (boost::shared_ptr<Route>());
        }
+
+
 }
 
-static char translate_seven_segment (char achar)
+static char 
+translate_seven_segment (char achar)
 {
        achar = toupper (achar);
-       if  (achar >= 0x40 && achar <= 0x60)
+
+       if  (achar >= 0x40 && achar <= 0x60) {
                return achar - 0x40;
-       else if  (achar >= 0x21 && achar <= 0x3f)
-      return achar;
-       else
-      return 0x00;
+       } else if  (achar >= 0x21 && achar <= 0x3f) {
+               return achar;
+       } else {
+               return 0x00;
+       }
 }
 
-MidiByteArray 
-Surface::two_char_display (const std::string & msg, const std::string & dots)
+void
+Surface::show_two_char_display (const std::string & msg, const std::string & dots)
 {
-       if (_stype != mcu) {
-               return MidiByteArray();
+       if (_stype != mcu || !_mcp.device_info().has_two_character_display() || msg.length() != 2 || dots.length() != 2) {
+               return;
        }
-
-       if  (msg.length() != 2) throw MackieControlException ("MackieMidiBuilder::two_char_display: msg must be exactly 2 characters");
-       if  (dots.length() != 2) throw MackieControlException ("MackieMidiBuilder::two_char_display: dots must be exactly 2 characters");
        
-       MidiByteArray bytes (6, 0xb0, 0x4a, 0x00, 0xb0, 0x4b, 0x00);
+       MidiByteArray right (3, 0xb0, 0x4b, 0x00);
+       MidiByteArray left (3, 0xb0, 0x4a, 0x00);
        
-       // chars are understood by the surface in right-to-left order
-       // could also exchange the 0x4a and 0x4b, above
-       bytes[5] = translate_seven_segment (msg[0]) +  (dots[0] == '.' ? 0x40 : 0x00);
-       bytes[2] = translate_seven_segment (msg[1]) +  (dots[1] == '.' ? 0x40 : 0x00);
+       right[2] = translate_seven_segment (msg[0]) +  (dots[0] == '.' ? 0x40 : 0x00);
+       left[2] = translate_seven_segment (msg[1]) +  (dots[1] == '.' ? 0x40 : 0x00);
        
-       return bytes;
+       _port->write (right);
+       _port->write (left);
 }
 
-MidiByteArray 
-Surface::two_char_display (unsigned int value, const std::string & /*dots*/)
+void
+Surface::show_two_char_display (unsigned int value, const std::string & /*dots*/)
 {
        ostringstream os;
        os << setfill('0') << setw(2) << value % 100;
-       return two_char_display (os.str());
+       show_two_char_display (os.str());
 }
 
-void 
-Surface::display_timecode (const std::string & timecode, const std::string & timecode_last)
+void
+Surface::display_timecode (const std::string & timecode, const std::string & last_timecode)
 {
-       if (has_timecode_display()) {
-               _port->write (timecode_display (timecode, timecode_last));
+       if (!_active || !_mcp.device_info().has_timecode_display()) {
+               return;
        }
-}
-
-MidiByteArray 
-Surface::timecode_display (const std::string & timecode, const std::string & last_timecode)
-{
        // if there's no change, send nothing, not even sysex header
-       if  (timecode == last_timecode) return MidiByteArray();
+       if  (timecode == last_timecode) return;
        
        // length sanity checking
        string local_timecode = timecode;
@@ -741,25 +717,16 @@ Surface::timecode_display (const std::string & timecode, const std::string & las
        // find the suffix of local_timecode that differs from last_timecode
        std::pair<string::const_iterator,string::iterator> pp = mismatch (last_timecode.begin(), last_timecode.end(), local_timecode.begin());
        
-       MidiByteArray retval;
-       
-       // sysex header
-       retval << sysex_hdr();
-       
-       // code for timecode display
-       retval << 0x10;
-       
+       int position = 0x40;
+
        // translate characters. These are sent in reverse order of display
        // hence the reverse iterators
        string::reverse_iterator rend = reverse_iterator<string::iterator> (pp.second);
        for  (string::reverse_iterator it = local_timecode.rbegin(); it != rend; ++it) {
+               MidiByteArray retval (2, 0xb0, position++);
                retval << translate_seven_segment (*it);
+               _port->write (retval);
        }
-       
-       // sysex trailer
-       retval << MIDI::eox;
-       
-       return retval;
 }
 
 void
@@ -774,46 +741,59 @@ void
 Surface::update_view_mode_display ()
 {
        string text;
-       Button* button = 0;
+       int id = -1;
+
+       if (!_active) {
+               return;
+       }
 
        switch (_mcp.view_mode()) {
        case MackieControlProtocol::Mixer:
-               _port->write (two_char_display ("Mx"));
-               button = buttons[Button::Pan];
+               show_two_char_display ("Mx");
+               id = Button::Pan;
                break;
        case MackieControlProtocol::Dynamics:
-               _port->write (two_char_display ("Dy"));
-               button = buttons[Button::Dyn];
+               show_two_char_display ("Dy");
+               id = Button::Dyn;
                break;
        case MackieControlProtocol::EQ:
-               _port->write (two_char_display ("EQ"));
-               button = buttons[Button::Eq];
+               show_two_char_display ("EQ");
+               id = Button::Eq;
                break;
        case MackieControlProtocol::Loop:
-               _port->write (two_char_display ("LP"));
-               button = buttons[Button::Loop];
+               show_two_char_display ("LP");
+               id = Button::Loop;
                break;
        case MackieControlProtocol::AudioTracks:
-               _port->write (two_char_display ("AT"));
+               show_two_char_display ("AT");
                break;
        case MackieControlProtocol::MidiTracks:
-               _port->write (two_char_display ("MT"));
-               break;
-       case MackieControlProtocol::Busses:
-               _port->write (two_char_display ("Bs"));
+               show_two_char_display ("MT");
                break;
        case MackieControlProtocol::Sends:
-               _port->write (two_char_display ("Sn"));
-               button = buttons[Button::Sends];
+               show_two_char_display ("Sn");
+               id = Button::Sends;
                break;
        case MackieControlProtocol::Plugins:
-               _port->write (two_char_display ("Pl"));
-               button = buttons[Button::Plugin];
+               show_two_char_display ("Pl");
+               id = Button::Plugin;
+               break;
+       default:
                break;
        }
 
-       if (button) {
-               _port->write (button->set_state (on));
+       if (id >= 0) {
+               
+               /* we are attempting to turn a global button/LED on */
+
+               map<int,Control*>::iterator x = controls_by_device_independent_id.find (id);
+
+               if (x != controls_by_device_independent_id.end()) {
+                       Button* button = dynamic_cast<Button*> (x->second);
+                       if (button) {
+                               _port->write (button->set_state (on));
+                       }
+               }
        }
 
        if (!text.empty()) {
@@ -824,10 +804,44 @@ Surface::update_view_mode_display ()
 }
 
 void
-Surface::gui_selection_changed (ARDOUR::RouteNotificationListPtr routes)
+Surface::gui_selection_changed (const ARDOUR::StrongRouteNotificationList& routes)
 {
        for (Strips::iterator s = strips.begin(); s != strips.end(); ++s) {
-               _port->write ((*s)->gui_selection_changed (routes));
+               (*s)->gui_selection_changed (routes);
        }
 }
 
+void
+Surface::say_hello ()
+{
+       /* wakeup for Mackie Control */
+       MidiByteArray wakeup (7, MIDI::sysex, 0x00, 0x00, 0x66, 0x14, 0x00, MIDI::eox);
+       _port->write (wakeup);
+       wakeup[4] = 0x15; /* wakup Mackie XT */
+       _port->write (wakeup);
+       wakeup[4] = 0x10; /* wakupe Logic Control */
+       _port->write (wakeup);
+       wakeup[4] = 0x11; /* wakeup Logic Control XT */
+       _port->write (wakeup);
+}
+
+void
+Surface::next_jog_mode ()
+{
+}
+
+void
+Surface::set_jog_mode (JogWheel::Mode)
+{
+}      
+
+bool
+Surface::route_is_locked_to_strip (boost::shared_ptr<Route> r) const
+{
+       for (Strips::const_iterator s = strips.begin(); s != strips.end(); ++s) {
+               if ((*s)->route() == r && (*s)->locked()) {
+                       return true;
+               }
+       }
+       return false;
+}