first changeset towards allowing parallel HOGP connections

Author: benedekkupper

device/.vscode/c_cpp_properties.json

@@ -10,7 +10,10 @@
             "cppStandard": "gnu++20",
             "configurationProvider": "nordic-semiconductor.nrf-connect",
             "browse": {
-                "limitSymbolsToIncludedHeaders": true
+                "limitSymbolsToIncludedHeaders": true,
+                "path": [
+                    "${workspaceFolder}/src/**"
+                ]
             }
         }
     ],

device/prj.conf.overlays/ble_hid.conf

@@ -6,7 +6,7 @@ CONFIG_BT_KEYS_SAVE_AGING_COUNTER_ON_PAIRING=y
 CONFIG_BT_MAX_PAIRED=20
 
 # use BLE HID over GATT from c2usb
-CONFIG_C2USB_BLUETOOTH=y
+CONFIG_C2USB_HOGP=y
 CONFIG_C2USB_HOGP_LOG_LEVEL_DBG=y
 # battery status to generic client
 CONFIG_BT_BAS=y

device/prj.conf.overlays/c2usb.conf

@@ -10,6 +10,7 @@ CONFIG_C2USB_UDC_MAC_LOG_LEVEL_DBG=y
 
 CONFIG_WARN_EXPERIMENTAL=n
 
+CONFIG_C2USB_HID_BOOT_PROTOCOL=y
 CONFIG_C2USB_UDC_MAC=y
 
 # Apply config requires deep stack, and since Agent expects response code,

device/src/bt_manager.c

@@ -65,14 +65,6 @@ void BtManager_StartBt() {
         return;
     }
 
-    if (DEVICE_IS_UHK80_RIGHT) {
-        if (HOGP_Enable()) {
-            LOG_INF("HOGP service enabled successfully");
-        } else {
-            LOG_ERR("Failed to enable HOGP service");
-        }
-    }
-
     BtManager_StartScanningAndAdvertising();
 }
 
@@ -103,12 +95,6 @@ void BtManager_StopBt() {
 
     k_sleep(K_MSEC(10));
 
-
-    if (DEVICE_IS_UHK80_RIGHT) {
-        LOG_WRN("Disabling HOGP service.");
-        HOGP_Disable();
-    }
-
     BtAdvertise_DisableAdvertisingIcon();
 
     LOG_INF("OOB: Bluetooth stopped");

device/src/main.c

@@ -250,7 +250,6 @@ void mainRuntime(void) {
 
     // 6.6mA
 
-    HID_SetGamepadActive(false);
     USB_Enable(); // +2.2mA, 8.8mA; has to be after USB_SetSerialNumber
 
     // 8.8mA

nrfsdk.code-workspace

@@ -9,8 +9,13 @@
 			"path": "."
 		},
 		{
-			"name": "workspace",
-			"path": ".."
+			"path": "../zephyr"
+		},
+		{
+			"path": "../c2usb/c2usb"
+		},
+		{
+			"path": "../modules/lib/hid-rp/hid-rp"
 		}
 	],
 	"settings": {
@@ -33,6 +38,9 @@
 			"*.hpp": "cpp",
 			"*.cpp": "cpp"
 		},
+		"C_Cpp.files.exclude": {
+			"${workspaceFolder:c2usb}/c2usb/**/*.test.cpp": true
+		},
 		"C_Cpp.default.configurationProvider": "ms-vscode.cmake-tools",
 		"C_Cpp.workspaceParsingPriority": "high",
 		"C_Cpp.autoAddFileAssociations": false,

…n-t-flush-top-level-compiler-flags.patch …n-t-flush-top-level-compiler-flags.patchpatches/core/0006-cmake-don-t-flush-top-level-compiler-flags.patch renamed to patches/core/0001-cmake-don-t-flush-top-level-compiler-flags.patch patches/core/0006-cmake-don-t-flush-top-level-compiler-flags.patch renamed to patches/core/0001-cmake-don-t-flush-top-level-compiler-flags.patch

@@ -1,24 +1,23 @@
-From 2c8942648651f955bba2990e92354b04a46e07b7 Mon Sep 17 00:00:00 2001
+From c1aa6de8d459d59d6903943f9fcd540ca535604e Mon Sep 17 00:00:00 2001
 From: Benedek Kupper <kupper.benedek@gmail.com>
-Date: Mon, 23 Feb 2026 20:03:41 +0100
+Date: Fri, 10 Jul 2026 16:19:13 +0200
 Subject: [PATCH] cmake: don't flush top-level compiler flags
 
 This was preventing CMAKE_C_FLAGS_RELEASE and similar from being set in the presets.
 
 ---
- cmake/extension/mcux.cmake | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
+ cmake/extension/mcux.cmake | 1 -
+ 1 file changed, 1 deletion(-)
 
 diff --git a/cmake/extension/mcux.cmake b/cmake/extension/mcux.cmake
-index db62e4e..8b2b203 100644
+index 57c92ef5..5607bace 100644
 --- a/cmake/extension/mcux.cmake
 +++ b/cmake/extension/mcux.cmake
-@@ -71,7 +71,7 @@ macro(project project_name)
-   # valiate compiler version
-   # The reason to validate compiler version here is that cmake engine code will get CMAKE_C_COMPILER_VERSION and CMAKE_CXX_COMPILER_VERSION just in "project" macro invocation.
-   _validate_compiler_version()
+@@ -133,7 +133,6 @@ macro(project project_name)
+   if(NOT (DEFINED MCUX_SKIP_COMPILER_CHECKS AND MCUX_SKIP_COMPILER_CHECKS))
+     _validate_compiler_version()
+   endif()
 -  clear_default_added_compiler_flags()
-+
    # parse arguments
    set(options NO_DEFAULT_CONFIG)
    set(single_value PROJECT_BOARD_PORT_PATH PROJECT_DEVICE_PORT_PATH PROJECT_TYPE CUSTOM_LINKER)

patches/core/0005-cmake-set-_IS_TOP_LEVEL-project-variables.patch

@@ -18,6 +18,7 @@ CONFIG_USB_DEVICE_CONFIG_LOW_POWER_MODE=y
 CONFIG_USB_DEVICE_CONFIG_REMOTE_WAKEUP=y
 CONFIG_C2USB_MCUX_MAC=y
 CONFIG_C2USB_MCUX_USB_COEXISTENCE=y
+CONFIG_C2USB_HID_BOOT_PROTOCOL=y
 
 # disable board (and device) specific source files, as the project has its own sources
 # examples/_common/project_segments/common/Kconfig.prjseg