diff --git a/gtk2_ardour/canvas_patch_change.cc b/gtk2_ardour/canvas_patch_change.cc
index 29e30e332d..9e754c953e 100644
--- a/gtk2_ardour/canvas_patch_change.cc
+++ b/gtk2_ardour/canvas_patch_change.cc
@@ -187,7 +187,7 @@ CanvasPatchChange::on_event (GdkEvent* ev)
 		case GDK_Up:
 		case GDK_KP_Up:
 		case GDK_uparrow:
-			if (Keyboard::modifier_state_contains (ev->scroll.state, Keyboard::PrimaryModifier)) {
+			if (Keyboard::modifier_state_contains (ev->key.state, Keyboard::PrimaryModifier)) {
 				_region.previous_bank (*this);
 			} else {
 				_region.previous_patch (*this);
@@ -196,7 +196,7 @@ CanvasPatchChange::on_event (GdkEvent* ev)
 		case GDK_Down:
 		case GDK_KP_Down:
 		case GDK_downarrow:
-			if (Keyboard::modifier_state_contains (ev->scroll.state, Keyboard::PrimaryModifier)) {
+			if (Keyboard::modifier_state_contains (ev->key.state, Keyboard::PrimaryModifier)) {
 				_region.next_bank (*this);
 			} else {
 				_region.next_patch (*this);
diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc
index 4212a3131c..0ede957a45 100644
--- a/gtk2_ardour/midi_region_view.cc
+++ b/gtk2_ardour/midi_region_view.cc
@@ -1972,7 +1972,7 @@ MidiRegionView::next_patch (CanvasPatchChange& patch)
 }
 
 void
-MidiRegionView::previous_bank (CanvasPatchChange& patch)
+MidiRegionView::next_bank (CanvasPatchChange& patch)
 {
 	if (patch.patch()->program() < 127) {
 		MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key (patch.patch());
@@ -1984,7 +1984,7 @@ MidiRegionView::previous_bank (CanvasPatchChange& patch)
 }
 
 void
-MidiRegionView::next_bank (CanvasPatchChange& patch)
+MidiRegionView::previous_bank (CanvasPatchChange& patch)
 {
 	if (patch.patch()->program() > 0) {
 		MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key (patch.patch());