From 32bcf606b5444766f36c708e8469af3da1310f7f Mon Sep 17 00:00:00 2001 From: mikeller Date: Fri, 22 Mar 2019 09:36:46 +1300 Subject: [PATCH] Fixed bug in SUMD V3 channel number handling. --- src/main/rx/sumd.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/main/rx/sumd.c b/src/main/rx/sumd.c index 717d126ba..717a4ba41 100644 --- a/src/main/rx/sumd.c +++ b/src/main/rx/sumd.c @@ -112,8 +112,6 @@ static uint8_t sumdFrameStatus(rxRuntimeConfig_t *rxRuntimeConfig) { UNUSED(rxRuntimeConfig); - uint8_t channelIndex; - uint8_t frameStatus = RX_FRAME_PENDING; if (!sumdFrameDone) { @@ -139,10 +137,9 @@ static uint8_t sumdFrameStatus(rxRuntimeConfig_t *rxRuntimeConfig) return frameStatus; } - if (sumdChannelCount > MAX_SUPPORTED_RC_CHANNEL_COUNT) - sumdChannelCount = MAX_SUPPORTED_RC_CHANNEL_COUNT; + unsigned channelsToProcess = MIN(sumdChannelCount, MAX_SUPPORTED_RC_CHANNEL_COUNT); - for (channelIndex = 0; channelIndex < sumdChannelCount; channelIndex++) { + for (unsigned channelIndex = 0; channelIndex < channelsToProcess; channelIndex++) { sumdChannels[channelIndex] = ( (sumd[SUMD_BYTES_PER_CHANNEL * channelIndex + SUMD_OFFSET_CHANNEL_1_HIGH] << 8) | sumd[SUMD_BYTES_PER_CHANNEL * channelIndex + SUMD_OFFSET_CHANNEL_1_LOW] -- 2.11.4.GIT