From abae00825066c49a8eda262ee8500be341b52a0d Mon Sep 17 00:00:00 2001
From: Thomas Watson <twatson52@icloud.com>
Date: Fri, 2 Aug 2024 22:07:42 -0500
Subject: [PATCH] GCS_MAVLink: remove vestigial SERIAL_CONTROL_DEV_SHELL
 support

The HAL always returns nullptr so the function always returns, which is
handled already by the default.
---
 libraries/GCS_MAVLink/GCS_serial_control.cpp | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/libraries/GCS_MAVLink/GCS_serial_control.cpp b/libraries/GCS_MAVLink/GCS_serial_control.cpp
index 0c60dcb4e2..e0a2b3fa09 100644
--- a/libraries/GCS_MAVLink/GCS_serial_control.cpp
+++ b/libraries/GCS_MAVLink/GCS_serial_control.cpp
@@ -74,12 +74,6 @@ void GCS_MAVLINK::handle_serial_control(const mavlink_message_t &msg)
         AP::gps().lock_port(1, exclusive);
         break;
 #endif  // AP_GPS_ENABLED
-    case SERIAL_CONTROL_DEV_SHELL:
-        stream = hal.util->get_shell_stream();
-        if (stream == nullptr) {
-            return;
-        }
-        break;
     case SERIAL_CONTROL_SERIAL0 ... SERIAL_CONTROL_SERIAL9: {
         // direct access to a SERIALn port
         stream = port = AP::serialmanager().get_serial_by_id(packet.device - SERIAL_CONTROL_SERIAL0);