subsys: mgmt: mcumgr: serial transport rework

cuts down some unused Kconfigs

Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>

Author: michalek-no

subsys/mgmt/mcumgr/transport/include/serial.h

@@ -48,14 +48,6 @@ struct mcumgr_serial_rx_ctxt {
 	/** Length of full packet, as read from header. */
 	uint16_t pkt_len;
 
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE) && \
-	defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
-	/**
-	 * Set to true by transports which use raw MCUmgr over UART (not SMP over console) to know
-	 * not to deal with the extra base64 etc. processing.
-	 */
-	const bool raw_transport;
-#endif
 };
 
 /** @typedef mcumgr_serial_tx_cb

subsys/mgmt/mcumgr/transport/src/serial_util.c

@@ -15,11 +15,7 @@
 #include <zephyr/sys/base64.h>
 #include <zephyr/mgmt/mcumgr/mgmt/mgmt.h>
 #include <zephyr/mgmt/mcumgr/smp/smp.h>
-#include <zephyr/mgmt/mcumgr/transport/serial.h>
-
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
-#include <mgmt/mcumgr/transport/smp_internal.h>
-#endif
+#include <serial.h>
 
 static void mcumgr_serial_free_rx_ctxt(struct mcumgr_serial_rx_ctxt *rx_ctxt)
 {
@@ -29,7 +25,6 @@ static void mcumgr_serial_free_rx_ctxt(struct mcumgr_serial_rx_ctxt *rx_ctxt)
 	}
 }
 
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE)
 static uint16_t mcumgr_serial_calc_crc(const uint8_t *data, int len)
 {
 	return crc16_itu_t(0x0000, data, len);
@@ -62,49 +57,7 @@ static int mcumgr_serial_decode_frag(struct mcumgr_serial_rx_ctxt *rx_ctxt,
 
 	return 0;
 }
-#endif
-
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
-static inline bool mcumgr_serial_process_frag_raw(struct mcumgr_serial_rx_ctxt *rx_ctxt,
-							     const uint8_t *frag, int frag_len,
-							     struct smp_hdr *rec_hdr)
-{
-	uint16_t total_size;
 
-	net_buf_add_mem(rx_ctxt->nb, frag, frag_len);
-
-	if (rx_ctxt->nb->len < sizeof(struct smp_hdr)) {
-		/* Missing header */
-		return false;
-	}
-
-	/*
-	 * Perform some basic cursory checks to ensure some fields of the packet are
-	 * actually valid.
-	 */
-	rec_hdr = (struct smp_hdr *)rx_ctxt->nb->data;
-	total_size = sys_be16_to_cpu(rec_hdr->nh_len) + sizeof(struct smp_hdr);
-
-	if (total_size > CONFIG_MCUMGR_TRANSPORT_NETBUF_SIZE) {
-		/* Payload is longer than the maximum supported MTU. */
-		mcumgr_serial_free_rx_ctxt(rx_ctxt);
-		return false;
-	} else if (rec_hdr->nh_op >= MGMT_OP_COUNT) {
-		/* Unknown op-code, likely not a valid MCUmgr message. */
-		mcumgr_serial_free_rx_ctxt(rx_ctxt);
-		return false;
-	} else if (rx_ctxt->nb->len < total_size) {
-		/* More fragments expected. */
-		return false;
-	} else if (rx_ctxt->nb->len > total_size) {
-		/* Payload longer than indicated in header. */
-		mcumgr_serial_free_rx_ctxt(rx_ctxt);
-		return false;
-	}
-
-	return true;
-}
-#endif
 
 /**
  * Processes a received mcumgr frame.
@@ -118,14 +71,9 @@ struct net_buf *mcumgr_serial_process_frag(struct mcumgr_serial_rx_ctxt *rx_ctxt
 {
 	struct net_buf *nb;
 
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE)
 	int rc;
 	uint16_t crc;
 	uint16_t op;
-#endif
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
-	struct smp_hdr *rec_hdr;
-#endif
 
 	if (rx_ctxt->nb == NULL) {
 		rx_ctxt->nb = smp_packet_alloc();
@@ -135,20 +83,8 @@ struct net_buf *mcumgr_serial_process_frag(struct mcumgr_serial_rx_ctxt *rx_ctxt
 		}
 	}
 
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE) && \
-	defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
 	if (rx_ctxt->raw_transport == true) {
-#endif
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
-		if (mcumgr_serial_process_frag_raw(rx_ctxt, frag, frag_len, rec_hdr) == false) {
-			return NULL;
-		}
-#endif
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE) && \
-	defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
 	} else {
-#endif
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE)
 		if (frag_len < sizeof(op)) {
 			return NULL;
 		}
@@ -203,18 +139,13 @@ struct net_buf *mcumgr_serial_process_frag(struct mcumgr_serial_rx_ctxt *rx_ctxt
 
 		/* Packet is complete; strip the CRC. */
 		rx_ctxt->nb->len -= 2U;
-#endif
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE) && \
-	defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_RAW_BINARY_NON_SMP_OVER_CONSOLE)
 	}
-#endif
 
 	nb = rx_ctxt->nb;
 	rx_ctxt->nb = NULL;
 	return nb;
 }
 
-#if defined(CONFIG_MCUMGR_TRANSPORT_SERIAL_HAS_SMP_OVER_CONSOLE)
 /**
  * Base64-encodes a small chunk of data and transmits it. The data must be no larger than three
  * bytes.
@@ -386,4 +317,3 @@ int mcumgr_serial_tx_pkt(const uint8_t *data, int len, mcumgr_serial_tx_cb cb)
 
 	return 0;
 }
-#endif