From c1887d77e0eca39e158d2fc143d5eed086e01d05 Mon Sep 17 00:00:00 2001 From: helloradiosky <157123396+helloradiosky@users.noreply.github.com> Date: Fri, 30 May 2025 18:48:45 +0800 Subject: [PATCH] fix(v16): trim T5/T6 swapped (#6315) Co-authored-by: Peter Feerick --- radio/src/targets/horus/hal.h | 43 ++++++++++++++++++++++++++--------- 1 file changed, 32 insertions(+), 11 deletions(-) diff --git a/radio/src/targets/horus/hal.h b/radio/src/targets/horus/hal.h index 5f89c5ecd..8ea02e510 100644 --- a/radio/src/targets/horus/hal.h +++ b/radio/src/targets/horus/hal.h @@ -268,9 +268,9 @@ // 6POS SW #if defined(RADIO_V16) #define SIXPOS_SWITCH_INDEX 5 - #define SIXPOS_LED_RED 255 - #define SIXPOS_LED_GREEN 255 - #define SIXPOS_LED_BLUE 255 + #define SIXPOS_LED_RED 200 + #define SIXPOS_LED_GREEN 0 + #define SIXPOS_LED_BLUE 0 #endif // Trims @@ -317,6 +317,7 @@ #define TRIMS_GPIO_REG_LSU GPIOB #define TRIMS_GPIO_PIN_LSU LL_GPIO_PIN_13 // PB.13 #elif defined(PCBX10) + // Left/Right Horizontal trims (LHL/LHR) #if defined(RADIO_TX16S) || defined(RADIO_F16) || defined(RADIO_V16) #define TRIMS_GPIO_REG_LHL GPIOA #define TRIMS_GPIO_PIN_LHL LL_GPIO_PIN_6 // PA.06 @@ -328,33 +329,53 @@ #define TRIMS_GPIO_REG_LHR GPIOB #define TRIMS_GPIO_PIN_LHR LL_GPIO_PIN_9 // PB.09 #endif + + // Vertical trims - common across all PCBX10 variants #define TRIMS_GPIO_REG_LVD GPIOG #define TRIMS_GPIO_PIN_LVD LL_GPIO_PIN_12 // PG.12 #define TRIMS_GPIO_REG_LVU GPIOJ #define TRIMS_GPIO_PIN_LVU LL_GPIO_PIN_14 // PJ.14 #define TRIMS_GPIO_REG_RVD GPIOJ #define TRIMS_GPIO_PIN_RVD LL_GPIO_PIN_13 // PJ.13 - #define TRIMS_GPIO_REG_RHL GPIOD - #define TRIMS_GPIO_PIN_RHL LL_GPIO_PIN_3 // PD.03 #define TRIMS_GPIO_REG_RVU GPIOJ #define TRIMS_GPIO_PIN_RVU LL_GPIO_PIN_12 // PJ.12 + + // Right Horizontal trims - common across all PCBX10 variants + #define TRIMS_GPIO_REG_RHL GPIOD + #define TRIMS_GPIO_PIN_RHL LL_GPIO_PIN_3 // PD.03 #define TRIMS_GPIO_REG_RHR GPIOD #define TRIMS_GPIO_PIN_RHR LL_GPIO_PIN_7 // PD.07 - #if defined(RADIO_FAMILY_T16) + + // T5/T6 trim assignments + #if defined(RADIO_V16) + // T5/T6 trims are in exchanged positions to other handsets + #define TRIMS_GPIO_REG_LSU GPIOB + #define TRIMS_GPIO_PIN_LSU LL_GPIO_PIN_14 // PB.14 + #define TRIMS_GPIO_REG_LSD GPIOB + #define TRIMS_GPIO_PIN_LSD LL_GPIO_PIN_13 // PB.13 + #define TRIMS_GPIO_REG_RSU GPIOD + #define TRIMS_GPIO_PIN_RSU LL_GPIO_PIN_13 // PD.13 + #define TRIMS_GPIO_REG_RSD GPIOJ + #define TRIMS_GPIO_PIN_RSD LL_GPIO_PIN_8 // PJ.08 + #elif defined(RADIO_FAMILY_T16) #define TRIMS_GPIO_REG_LSU GPIOD #define TRIMS_GPIO_PIN_LSU LL_GPIO_PIN_13 // PD.13 #define TRIMS_GPIO_REG_LSD GPIOJ #define TRIMS_GPIO_PIN_LSD LL_GPIO_PIN_8 // PJ.08 - #else + #define TRIMS_GPIO_REG_RSU GPIOB + #define TRIMS_GPIO_PIN_RSU LL_GPIO_PIN_14 // PB.14 + #define TRIMS_GPIO_REG_RSD GPIOB + #define TRIMS_GPIO_PIN_RSD LL_GPIO_PIN_13 // PB.13 + #else // Other PCBX10 variants #define TRIMS_GPIO_REG_LSU GPIOJ #define TRIMS_GPIO_PIN_LSU LL_GPIO_PIN_8 // PJ.08 #define TRIMS_GPIO_REG_LSD GPIOD #define TRIMS_GPIO_PIN_LSD LL_GPIO_PIN_13 // PD.13 + #define TRIMS_GPIO_REG_RSU GPIOB + #define TRIMS_GPIO_PIN_RSU LL_GPIO_PIN_14 // PB.14 + #define TRIMS_GPIO_REG_RSD GPIOB + #define TRIMS_GPIO_PIN_RSD LL_GPIO_PIN_13 // PB.13 #endif - #define TRIMS_GPIO_REG_RSU GPIOB - #define TRIMS_GPIO_PIN_RSU LL_GPIO_PIN_14 // PB.14 - #define TRIMS_GPIO_REG_RSD GPIOB - #define TRIMS_GPIO_PIN_RSD LL_GPIO_PIN_13 // PB.13 #endif