docs: 添加Blinky键盘2.4G端详细设计文档

- 定义了键盘外设固件内实现2.4G端的设计方案
- 描述了基于专用local identity的dongle peer设计方案
- 详细说明了身份规划和模块职责分工
- 包含了行为模型、状态事件设计和UI设计
- 提供了分阶段实施建议和风险评估

CMakeLists.txt

@@ -22,6 +22,7 @@ zephyr_nanopb_sources(app
 target_sources(app PRIVATE
   src/main.c
   src/battery_module.c
+  src/ble_bond_multi_module.c
   src/ble_adv_uuid16.c
   src/ble_bas_module.c
   src/ble_hid_module.c
@@ -35,13 +36,21 @@ target_sources(app PRIVATE
   src/led_strip_module.c
   src/mode_policy_module.c
   src/time_sync_module.c
+  src/settings_module.c
+  src/swift_pair_module.c
   src/ui/ui_main.c
+  src/ui/ui_settings.c
+  src/ui/ui_settings_controller.c
+  src/ui/ui_settings_root.c
+  src/ui/ui_settings_ble.c
+  src/ui/ui_settings_theme.c
   src/protocol_module.c
   src/usb_cdc_module.c
   src/usb_device_module.c
   src/usb_hid_consumer_module.c
   src/usb_hid_keyboard_module.c
   src/events/bat_state_event.c
+  src/events/ble_bond_multi_event.c
   src/events/datetime_event.c
   src/events/encoder_event.c
   src/events/function_bitmap_state_event.c
@@ -58,8 +67,11 @@ target_sources(app PRIVATE
   src/events/proto_transport_state_event.c
   src/events/proto_tx_event.c
   src/events/set_protocol_event.c
+  src/events/settings_mode_event.c
+  src/events/settings_view_event.c
   src/events/theme_rgb_update_event.c
   src/events/time_sync_event.c
+  src/events/transport_policy_event.c
   src/events/usb_control_event.c
   src/events/usb_state_event.c
 )

Kconfig

@@ -2,6 +2,19 @@ mainmenu "blinky"
 
 source "Kconfig.zephyr"
 
+menu "Application"
+
+config BLINKY_BLE_BOND_MULTI_INTERNAL
+	bool
+	depends on BT_BONDABLE
+	depends on BT_SETTINGS
+	depends on CAF_SETTINGS_LOADER
+	depends on CAF_BLE_COMMON_EVENTS
+	select CAF_BLE_BOND_SUPPORTED
+	default y
+
+endmenu
+
 menu "Application Drivers"
 rsource "drivers/Kconfig"
 endmenu

boards/atguigu/mini_keyboard/mini_keyboard.dts

@@ -16,6 +16,8 @@
 		zephyr,code-partition = &slot0_partition;
 		zephyr,display = &screen_lcd;
 		zephyr,led-strip = &led_strip;
+		zephyr,console = &cdc_acm_uart1;
+		zephyr,log-uart = &log_uarts;
 	};
 
 	aliases {
@@ -25,6 +27,11 @@
 		backlight = &backlight;
     };
 
+	log_uarts: log_uarts {
+		compatible = "zephyr,log-uart";
+		uarts = <&cdc_acm_uart1>;
+	};
+
 	hid_kbd: hid_kbd {
 		compatible = "zephyr,hid-device";
 		label = "HID_KBD";
@@ -260,4 +267,9 @@
 		compatible = "zephyr,cdc-acm-uart";
 		label = "CDC_ACM_0";
 	};
+
+	cdc_acm_uart1: cdc_acm_uart1 {
+		compatible = "zephyr,cdc-acm-uart";
+		label = "CDC_ACM_1";
+	};
 };

docs/ble_multi_slot_design.md

@@ -0,0 +1,582 @@
+# Blinky 多槽蓝牙设计方案
+
+## 1. 目标与约束
+
+为 `C:\projects\blinky` 增加多槽蓝牙能力，当前阶段只实现 BLE，多 dongle 方案先预留结构，不实现完整业务流程。
+
+本方案已经按当前讨论结论收敛为以下硬约束：
+
+- 支持 3 个 BLE 槽位 `Slot 1~3`
+- 每个槽位固定对应一个 Bluetooth local identity
+- 槽位切换本质上是切换 `bt_identity_id`
+- 不引入 `app_slot_id`
+- 不使用临时 identity
+- 不做 erase advertising
+- 不做擦除确认
+- 不做擦除回滚
+- 擦除当前槽位时直接删除该 identity 上的 bond
+- settings 只持久化 `current_slot` 和各槽位 `meta`
+- `slot_meta` 中必须保留 `display_name`，用于 UI 显示
+- `24G / dongle` 采用方案 A，但本阶段只预留专用槽位
+
+## 2. 本地参考基线
+
+本方案主要参考本地 NCS 3.2.3 中以下实现和文档：
+
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\doc\ble_bond.rst`
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\src\modules\ble_bond.c`
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\src\modules\Kconfig.ble_bond`
+- `c:\ncs\v3.2.3\nrf\include\caf\events\ble_common_event.h`
+- `c:\ncs\v3.2.3\nrf\subsys\caf\modules\ble_adv.c`
+- `c:\ncs\v3.2.3\nrf\subsys\caf\modules\Kconfig.ble_state`
+- `c:\ncs\v3.2.3\nrf\doc\nrf\libraries\caf\ble_state.rst`
+
+参考方式如下：
+
+- 继承 `nrf_desktop` 的总体方向：应用自定义 bond 管理模块，而不是 CAF 默认 `ble_bond`
+- 继续通过 `ble_peer_operation_event` 驱动 CAF `ble_adv`
+- 保留 dongle 专用 identity 的规划方式
+
+明确不采用 `nrf_desktop` 的部分：
+
+- 不使用 `app_slot_id <-> bt_identity_id` 的二级映射
+- 不使用临时 identity
+- 不采用 `ERASE_ADV`
+- 不采用“新配对成功后再替换旧槽位”的回滚保护机制
+
+## 3. 对现有项目的判断
+
+`blinky` 当前已经具备多槽蓝牙需要的大部分基础能力：
+
+- 已启用 `CONFIG_CAF_BLE_STATE`
+- 已启用 `CONFIG_CAF_BLE_ADV`
+- 已启用 `CONFIG_BT_SETTINGS`
+- HID 和 NUS 都已基于 `ble_peer_event` 跟踪当前连接
+- UI 已经预留 3 个蓝牙槽位和擦除当前槽位入口
+- 模式策略层已经区分：
+  - `MODE_SWITCH_BLE -> BLE_PROFILE_POLICY_GENERAL`
+  - `MODE_SWITCH_24G -> BLE_PROFILE_POLICY_DONGLE`
+
+相关文件：
+
+- `C:\projects\blinky\src\ble_hid_module.c`
+- `C:\projects\blinky\src\ble_nus_module.c`
+- `C:\projects\blinky\src\ui\ui_settings_ble.c`
+- `C:\projects\blinky\src\ui\ui_settings_controller.c`
+- `C:\projects\blinky\src\mode_policy_module.c`
+- `C:\projects\blinky\prj.conf`
+
+当前限制也很明确：
+
+- `CONFIG_BT_MAX_PAIRED=1`，全设备只能保存 1 个 bond
+- 当前是单槽逻辑
+- 还在使用 CAF 默认 `CONFIG_CAF_BLE_BOND=y`
+- UI 的槽位状态仍是内存假数据
+
+## 4. 配置语义结论
+
+基于本地 Kconfig 和 CAF 文档，关键配置的含义如下：
+
+- `CONFIG_BT_MAX_PAIRED`
+  说明：整个设备允许保存的 bond 总数
+- `CONFIG_CAF_BLE_STATE_MAX_LOCAL_ID_BONDS`
+  说明：每个 Bluetooth local identity 允许的 bond 数上限
+- `CONFIG_BT_ID_MAX`
+  说明：设备允许使用的 local identity 总数
+
+如果目标是：
+
+- 3 个 BLE 槽位
+- 每槽 1 个 bond
+- 预留 1 个 dongle 槽位
+
+那么推荐配置应为：
+
+```conf
+CONFIG_BT_MAX_CONN=1
+CONFIG_BT_MAX_PAIRED=4
+CONFIG_CAF_BLE_STATE_MAX_LOCAL_ID_BONDS=1
+CONFIG_BT_ID_MAX=5
+```
+
+语义如下：
+
+- `BT_MAX_CONN=1`
+  说明：同时只保持 1 条 BLE 连接
+- `BT_MAX_PAIRED=4`
+  说明：全设备最多保存 4 个 bond
+- `CAF_BLE_STATE_MAX_LOCAL_ID_BONDS=1`
+  说明：每个 identity 只允许绑定 1 个主机
+- `BT_ID_MAX=5`
+  说明：可用 identity 为 `0..4`
+
+配合本方案的 identity 规划：
+
+- `0`：默认 identity，不参与槽位
+- `1`：Slot 1
+- `2`：Slot 2
+- `3`：Slot 3
+- `4`：Dongle Slot 预留
+
+## 5. 总体架构
+
+## 5.1 设计结论
+
+建议采用：
+
+- 保留 `CAF_BLE_STATE`
+- 保留 `CAF_BLE_ADV`
+- 关闭 CAF 默认 `BLE_BOND`
+- 新增应用自定义模块 `ble_bond_multi_module`
+
+即：
+
+- `CONFIG_CAF_BLE_BOND=n`
+- 新增 `CONFIG_BLINKY_BLE_BOND_MULTI`
+- 自定义模块 `select CAF_BLE_BOND_SUPPORTED`
+- 由自定义模块向 CAF 提交 `ble_peer_operation_event`
+
+原因：
+
+- CAF 默认 `ble_bond` 不适合固定多 identity 槽位管理
+- 你的需求已经明确不需要 `nrf_desktop` 那种复杂映射和回滚流程
+- 自定义模块更适合和现有 UI、模式策略、settings 同步
+
+## 5.2 核心模型
+
+本方案直接把“槽位”和“identity”绑定：
+
+- `Slot 1` = `bt_identity_id 1`
+- `Slot 2` = `bt_identity_id 2`
+- `Slot 3` = `bt_identity_id 3`
+- `Dongle Slot` = `bt_identity_id 4`
+
+所以：
+
+- `current_slot` 实际上就是当前使用的 `bt_identity_id`
+- UI 显示“Slot 1/2/3”时，后端直接映射到固定 identity
+- 不再存在可变的 `id_lut`
+
+这个设计的优点：
+
+- 逻辑简单
+- settings 简单
+- 调试简单
+- 后续接入 dongle 不需要迁移已有 BLE 槽位
+
+## 6. identity 规划
+
+建议固定如下：
+
+| identity | 角色 | 当前阶段 |
+| --- | --- | --- |
+| `0` | 默认 identity | 不使用 |
+| `1` | BLE Slot 1 | 使用 |
+| `2` | BLE Slot 2 | 使用 |
+| `3` | BLE Slot 3 | 使用 |
+| `4` | Dongle Slot | 预留 |
+
+设计原则：
+
+- 不使用 `BT_ID_DEFAULT`
+  原因：默认 identity 的 reset/unpair 行为不适合作为固定槽位基线
+- 3 个 BLE 槽位 identity 固定不变
+- `identity 4` 只给 `BLE_PROFILE_POLICY_DONGLE`
+
+## 7. 模块设计
+
+## 7.1 新增模块和文件
+
+建议新增：
+
+- `src/ble_bond_multi_module.c`
+- `src/events/ble_bond_multi_event.c`
+- `inc/events/ble_bond_multi_event.h`
+
+不建议新增：
+
+- `inc/ble_bond_multi.h`
+
+原因：
+
+- 当前设计是事件驱动模块
+- 没有必要向外暴露一组直接调用 API
+- 共享内容更适合放在事件头文件里
+
+只有在后续出现明确的跨模块直接调用需求时，才需要再补 `inc/ble_bond_multi.h`
+
+## 7.2 模块职责边界
+
+### `ble_bond_multi_module`
+
+负责：
+
+- 管理当前活动 `bt_identity_id`
+- 启动时恢复 `current_slot`
+- 维护各 identity 槽位的 `slot_meta`
+- 处理“切换当前槽位”
+- 处理“直接擦除当前槽位”
+- 监听连接、加密、断链事件，更新 `slot_meta`
+- 向 CAF 提交 `ble_peer_operation_event`
+- 向 UI / 显示层广播槽位状态
+
+不负责：
+
+- 发 HID/NUS 数据
+- 改 GATT 服务
+- 绘制 UI
+
+### `ble_hid_module` / `ble_nus_module`
+
+保持现有思路，不理解“多槽逻辑”，只跟随当前活动连接：
+
+- 连接上谁，就服务谁
+- 切槽或擦除造成断链时，内部复位连接状态
+
+### `mode_policy_module`
+
+负责决定当前是：
+
+- `BLE_PROFILE_POLICY_GENERAL`
+- `BLE_PROFILE_POLICY_DONGLE`
+
+但不负责槽位 bond 细节。
+
+## 8. 状态机设计
+
+第一阶段建议极简化，只保留：
+
+- `DISABLED`
+- `IDLE`
+- `SWITCHING`
+- `ERASING`
+
+明确不引入：
+
+- `SELECT_SLOT`
+- `ERASE_PENDING`
+- `ERASE_ADV`
+- `TEMP_IDENTITY`
+- `ROLLBACK`
+
+## 8.1 切槽流程
+
+1. 用户在 UI 选择 `Slot N`
+2. 模块判断：
+   - 如果 `N == current_slot`，直接返回
+   - 否则进入 `SWITCHING`
+3. 提交 `PEER_OPERATION_SELECTED`
+4. CAF `ble_adv` 切换当前 advertising identity
+5. 当前连接断开
+6. 设备以新 identity 重新广播
+7. 目标主机重新连接
+8. 保存新的 `current_slot`
+9. 返回 `IDLE`
+
+这里采用“直接切换”，不做预选确认。
+
+## 8.2 擦除当前槽位流程
+
+1. 用户选择“Erase Bond”
+2. 模块进入 `ERASING`
+3. 对当前 `bt_identity_id` 执行：
+   - `bt_unpair(identity, BT_ADDR_LE_ANY)`
+4. 清除该槽位 `slot_meta`
+5. 重新以当前 identity 进入可配对广播
+6. 返回 `IDLE`
+
+这里采用产品级简化语义：
+
+- 不确认
+- 不保护旧配对
+- 不回滚
+- 擦除后旧主机立即失效
+
+## 9. Settings 设计
+
+## 9.1 持久化范围
+
+只持久化：
+
+- `current_slot`
+- 各槽位 `slot_meta`
+
+不持久化：
+
+- `id_lut`
+- 临时 identity 状态
+- 擦除中间态
+
+## 9.2 settings namespace
+
+建议采用：
+
+- `ble_multi/current_slot`
+- `ble_multi/meta/1`
+- `ble_multi/meta/2`
+- `ble_multi/meta/3`
+- `ble_multi/meta/4`
+
+其中：
+
+- `1/2/3` 是 BLE 槽位
+- `4` 是 dongle 预留槽位
+
+## 9.3 slot_meta 结构
+
+`slot_meta` 建议固定包含以下字段：
+
+- `occupied`
+- `last_peer_addr`
+- `display_name`
+
+字段说明：
+
+- `occupied`
+  说明：该槽位当前是否有有效 bond
+- `last_peer_addr`
+  说明：最近一次绑定主机的 BLE 地址，用于识别和调试
+- `display_name`
+  说明：UI 显示名，不做简化，作为正式字段保留
+
+这里 `display_name` 是明确保留项，不是可选优化项。  
+设计目的就是让 UI 能直接显示：
+
+- `MacBook Pro`
+- `iPad Mini`
+- `Windows-PC`
+
+而不是只能显示 `Bonded` / `Empty`。
+
+## 9.4 display_name 来源
+
+需要注意一点：
+
+- Zephyr bond 本身不会自动提供“友好主机名”
+
+因此 `display_name` 的来源需要在实现阶段明确，一般有两个方向：
+
+1. 配对后由应用层通过协议或自定义方式写入
+2. 若当前阶段无法拿到真实主机名，则先允许 UI/上位机为该槽位写入名称
+
+也就是说：
+
+- `display_name` 必须作为正式持久化字段存在
+- 但其填充机制可以分阶段实现
+
+第一阶段如果没有自动名称来源，建议默认值如下：
+
+- 有 bond 但无名称：`"Bonded Device"`
+- 无 bond：空串或 `"Empty"`
+
+## 10. UI 设计
+
+现有 UI 结构已经适合复用，不建议重做。
+
+现有文件：
+
+- `C:\projects\blinky\src\ui\ui_settings_ble.c`
+- `C:\projects\blinky\src\ui\ui_settings_controller.c`
+
+建议交互规则如下：
+
+- 进入 BLE 设置页后显示 `Slot 1~3`
+- 每个槽位 value 直接显示 `display_name`
+- 若该槽位无 bond，则显示 `Empty`
+- 当前槽位显示为 `Slot N`
+- 擦除项显示为 `Erase Slot N`
+
+操作方式：
+
+- 选中某个槽位后直接切换
+- 选中擦除项后直接擦除
+- 不加确认页
+
+UI 需要的数据来源：
+
+- `current_slot`
+- `slot_meta[1]`
+- `slot_meta[2]`
+- `slot_meta[3]`
+
+所以建议 UI 不再维护本地假数据，而改为订阅 `ble_bond_multi_event` 或读取统一状态缓存。
+
+## 11. 与模式切换的兼容设计
+
+## 11.1 BLE 档
+
+`MODE_SWITCH_BLE` 下：
+
+- 只允许使用 `identity 1/2/3`
+- UI 允许切槽和擦除
+- `current_slot` 必须属于 `1..3`
+
+## 11.2 24G / dongle 档
+
+采用方案 A：
+
+- `MODE_SWITCH_24G` 绑定专用 `identity 4`
+- `identity 4` 不与普通 BLE 槽位混用
+- 第一阶段只预留，不实现完整 dongle bond 生命周期
+
+这样做的好处：
+
+- 保持 `mode_policy_module.c` 现有设计语义
+- 普通 BLE 槽位和 dongle 配对状态不会互相污染
+- 后续接入 dongle 时无需改动 BLE 三槽数据模型
+
+## 12. 事件设计
+
+建议新增应用事件：
+
+- `ble_bond_multi_event`
+
+放在：
+
+- `inc/events/ble_bond_multi_event.h`
+- `src/events/ble_bond_multi_event.c`
+
+字段建议：
+
+- `current_slot`
+- `active_identity_id`
+- `slot_occupied_bitmap`
+- `slot_meta_summary`
+- `op`
+
+至少要能支持：
+
+- UI 刷新槽位名称
+- 显示层提示当前槽位
+- 设置层同步当前 active slot
+
+继续复用 CAF 事件：
+
+- `ble_peer_event`
+- `ble_peer_operation_event`
+
+第一阶段不需要依赖：
+
+- `ble_peer_search_event`
+
+## 13. 对现有代码的影响评估
+
+## 13.1 必改
+
+- `prj.conf`
+- `CMakeLists.txt`
+- 新增 `src/ble_bond_multi_module.c`
+- 新增 `inc/events/ble_bond_multi_event.h`
+- 新增 `src/events/ble_bond_multi_event.c`
+- `ui_settings_controller.c`
+
+## 13.2 小改
+
+- `mode_policy_module.c`
+  说明：把 `BLE_PROFILE_POLICY_GENERAL` 约束到 `identity 1/2/3`，把 `BLE_PROFILE_POLICY_DONGLE` 预留到 `identity 4`
+- `display_module.c`
+  说明：如果要显示当前槽位或名称，需要订阅新事件
+
+## 13.3 原则上不应大改
+
+- `ble_hid_module.c`
+- `ble_nus_module.c`
+- `keyboard_core_module.c`
+
+这些模块只应该感知当前连接状态，不承担槽位管理职责。
+
+## 14. 实施分阶段建议
+
+## 阶段 1：固定 identity 多槽骨架
+
+- 替换 CAF 默认 `ble_bond`
+- 新增 `ble_bond_multi_module`
+- 固定 `identity 1/2/3` 为 BLE 槽位
+- 预留 `identity 4` 为 dongle 槽位
+- 恢复和保存 `current_slot`
+- 打通直接切槽
+
+交付标准：
+
+- `Slot 1~3` 可以切换
+- 每个槽位 bond 独立
+- 重启后当前槽位可恢复
+
+## 阶段 2：slot_meta 与 UI 打通
+
+- 维护 `occupied`
+- 维护 `last_peer_addr`
+- 正式持久化 `display_name`
+- UI 改为显示真实槽位数据
+
+交付标准：
+
+- 屏幕上显示真实槽位名
+- `display_name` 可稳定持久化
+
+## 阶段 3：直接擦除与 dongle 预留对齐
+
+- 实现当前槽位直接 `unpair`
+- 擦除后刷新 `slot_meta`
+- 擦除后重新进入可配对状态
+- 明确 `identity 4` 的模式入口
+
+交付标准：
+
+- 擦除后旧主机不能再连接
+- 当前槽位可重新配对
+- `24G` 路径已有干净预留点
+
+## 15. 风险与注意事项
+
+## 15.1 直接擦除的用户体验
+
+本方案放弃回滚能力。  
+一旦擦除：
+
+- 旧 bond 立即失效
+- 用户必须重新配对
+
+这是当前明确接受的产品行为。
+
+## 15.2 display_name 获取机制
+
+`display_name` 虽然是正式字段，但自动来源未必现成。  
+实现阶段必须明确：
+
+- 名称是自动采集
+- 还是通过 UI / 上位机写入
+
+这不是字段是否保留的问题，而是字段填充路径的问题。
+
+## 15.3 `BT_ID_MAX=5` 可用性
+
+必须在目标板上验证：
+
+- identity 创建正常
+- 切换 advertising identity 正常
+- settings 恢复正常
+
+## 15.4 默认 identity 不参与槽位
+
+本方案建议 `BT_ID_DEFAULT` 不参与槽位管理，避免后续行为不一致。
+
+## 16. 当前版本结论
+
+按目前所有已确认信息，最终设计结论如下：
+
+1. 只做 BLE 三槽，dongle 只预留专用槽位
+2. 三个 BLE 槽位直接固定到 `bt_identity_id 1/2/3`
+3. `bt_identity_id 4` 预留给 dongle
+4. 不使用 `app_slot_id`
+5. 不使用临时 identity
+6. 不做 erase advertising
+7. 不做擦除确认和回滚
+8. `slot_meta` 固定包含：
+   - `occupied`
+   - `last_peer_addr`
+   - `display_name`
+9. `display_name` 作为正式持久化字段保留，用于 UI 显示
+10. 不新增 `inc/ble_bond_multi.h`，共享接口放到事件头文件
+
+如果你认可这个版本，下一步就可以按这份方案进入实现阶段。

docs/device_communication_protocol_v1.md

@@ -0,0 +1,302 @@
+# 下位机通信协议 v1 修订版
+
+本版基于原 `device_communication_protocol_v1.md` 修订，变更点如下：
+
+- `CDC` 去掉外层协议，直接传业务消息
+- `BLE GATT` 改为使用 `NUS (Nordic UART Service)`
+- `Bitmap` 统一为 29 字节键盘位图格式
+- `FunctionKeyEvent` 不再上报单键按下/释放，改为上报全键盘位图，格式与 `Bitmap` 一致
+- 顶层消息增加 `msg_id` / `reply_to`
+- `Ack` 与 `Error` 合并为统一 `Response`
+- `CDC` 与 `NUS` 均增加统一外层帧：`magic(2) + len(1) + protobuf`
+
+## 1. 总体原则
+
+- 标准键盘输入继续走 `HID / BLE HID`，不变。
+- 私有通信继续保留两条通道：
+  - `USB`：走 `CDC`
+  - `BLE`：走 `NUS`
+- 两条通道统一承载同一套 `protobuf` 业务消息。
+- `CDC` 与 `NUS` 均使用完全相同的外层帧格式，帧内承载 `protobuf` 业务消息。
+- 当前版本不做应用层分片。
+
+说明：
+
+- 顶层 `protobuf` 消息名改为 `DeviceMessage`，表示独立于具体传输通道的统一业务信封。
+- 当前版本使用统一外层帧，`CDC` 与 `NUS` 都按完整帧进行发送与解析。
+
+## 2. 共享业务消息
+
+统一的 `protobuf` 业务消息为 `DeviceMessage`，内部 `oneof body` 可取以下类型：
+
+| 消息 | 方向 | 说明 |
+| --- | --- | --- |
+| `HelloReq` | Host -> Device | 握手请求 |
+| `HelloRsp` | Device -> Host | 握手响应 |
+| `Bitmap` | Host -> Device | 下发功能键位图配置 |
+| `FunctionKeyEvent` | Device -> Host | 上报当前全键盘状态位图 |
+| `LedState` | Device -> Host | 上报键盘 LED 状态 |
+| `TimeSync` | Host -> Device | 下发时间同步 |
+| `ThemeRgb` | Host -> Device | 下发主题颜色 |
+| `Response` | Device -> Host | 控制命令统一响应 |
+
+顶层公共字段：
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `msg_id` | `uint32` | 当前消息 ID；`0` 表示未使用 |
+| `reply_to` | `uint32` | 当前消息响应的请求 ID；`0` 表示不是响应 |
+
+规则：
+
+- Host 发起的请求消息应填写唯一 `msg_id`。
+- `HelloRsp` 与 `Response` 应填写 `reply_to = 原请求.msg_id`。
+- `FunctionKeyEvent`、`LedState` 等异步上报消息应设置 `reply_to = 0`。
+- 当前版本建议每条链路同一时刻仅保留 `1` 条 in-flight 控制请求。
+
+## 3. CDC 协议
+
+CDC 在线路上的真实字节格式如下：
+
+```text
+magic(2B) + len(1B) + protobuf(DeviceMessage)
+```
+
+约束：
+
+- `magic` 固定为 `0xAA55`，按小端发送，即线路字节序为 `0x55 0xAA`。
+- `len` 为后续 `protobuf(DeviceMessage)` 的字节长度，范围 `0..64`。
+- Host -> Device 与 Device -> Host 都发送完整帧。
+- 当前版本要求单条业务消息一次完整发送，不做应用层分片与重组。
+
+## 4. NUS 协议
+
+BLE 私有通信改为使用 `NUS (Nordic UART Service)`。
+
+NUS 线路上的真实字节格式如下：
+
+```text
+magic(2B) + len(1B) + protobuf(DeviceMessage)
+```
+
+方向定义：
+
+- Host -> Device：向 `RX` 特征写入完整帧
+- Device -> Host：通过 `TX Notify` 上报完整帧
+
+## 5. NUS 服务定义
+
+Service UUID：
+
+```text
+6E400001-B5A3-F393-E0A9-E50E24DCCA9E
+```
+
+Characteristic 定义：
+
+| 名称 | UUID | 属性 | 用途 |
+| --- | --- | --- | --- |
+| `RX` | `6E400002-B5A3-F393-E0A9-E50E24DCCA9E` | `Write`, `Write Without Response` | Host -> Device |
+| `TX` | `6E400003-B5A3-F393-E0A9-E50E24DCCA9E` | `Notify` | Device -> Host |
+
+约束：
+
+- 连接后主机必须先订阅 `TX`。
+- 当前版本不做应用层分片。
+- 因为增加了 `magic + len` 外层帧，建议协商 `ATT_MTU >= 64`，保证完整帧可一次发送。
+
+## 6. 外层帧定义
+
+外层帧字段如下：
+
+| 字段 | 大小 | 说明 |
+| --- | --- | --- |
+| `magic` | `2` 字节 | 固定 `0xAA55`，线路字节序 `55 AA` |
+| `len` | `1` 字节 | `protobuf(DeviceMessage)` 长度，最大 `64` |
+| `payload` | `len` 字节 | `protobuf(DeviceMessage)` |
+
+## 7. protobuf 字段定义
+
+以下是业务字段语义。
+
+### HelloReq
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `protocol_version` | `uint32` | 当前固定为 `1` |
+
+### HelloRsp
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `protocol_version` | `uint32` | 当前固定为 `1` |
+| `vendor_id` | `uint32` | 当前建议 `0x1209` |
+| `product_id` | `uint32` | 当前建议 `0x0001` |
+| `firmware_major` | `uint32` | 固件主版本 |
+| `firmware_minor` | `uint32` | 固件次版本 |
+| `capability_flags` | `uint32` | 能力位 |
+
+### Bitmap
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `usage_bitmap` | `bytes` | 固定 29 字节，表示功能键配置位图 |
+
+语义：
+
+- `Bitmap.usage_bitmap` 长度固定为 `29` 字节。
+- 第 `0` 字节表示 `0xE0` 到 `0xE7` 这 8 个控制键。
+- 第 `1` 到第 `28` 字节表示 `0x00` 到 `0xDF`。
+- 位值为 `1` 表示该 usage 被配置为功能键。
+- 位值为 `0` 表示该 usage 不是功能键，继续按普通键处理。
+
+位映射规则：
+
+- 第 `0` 字节：
+  - `bit0 -> 0xE0`
+  - `bit1 -> 0xE1`
+  - ...
+  - `bit7 -> 0xE7`
+- 第 `1` 字节：
+  - `bit0 -> 0x00`
+  - `bit1 -> 0x01`
+  - ...
+  - `bit7 -> 0x07`
+- 第 `2` 字节：
+  - `bit0 -> 0x08`
+  - ...
+  - `bit7 -> 0x0F`
+- 依此类推。
+- 第 `28` 字节：
+  - `bit0 -> 0xD8`
+  - ...
+  - `bit7 -> 0xDF`
+
+### FunctionKeyEvent
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `usage_bitmap` | `bytes` | 固定 29 字节，表示当前全键盘状态位图 |
+
+语义：
+
+- `FunctionKeyEvent` 不再使用 `usage + action` 的单键事件格式。
+- `FunctionKeyEvent.usage_bitmap` 长度固定为 `29` 字节。
+- 位图编码方式与 `Bitmap.usage_bitmap` 完全一致。
+- 位值为 `1` 表示该 usage 当前处于按下状态。
+- 位值为 `0` 表示该 usage 当前处于释放状态。
+- 该消息表示一次完整键盘状态快照，而不是单个按键变化。
+
+### LedState
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `led_mask` | `uint32` | NumLock/CapsLock 等位掩码 |
+
+### TimeSync
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `version` | `uint32` | 当前固定 `1` |
+| `flags` | `uint32` | 标志位 |
+| `timezone_min` | `sint32` | 时区偏移，单位分钟 |
+| `utc_ms` | `fixed64` | UTC 毫秒时间戳 |
+| `accuracy_ms` | `fixed32` | 精度，毫秒 |
+
+### ThemeRgb
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `red` | `uint32` | `0..255` |
+| `green` | `uint32` | `0..255` |
+| `blue` | `uint32` | `0..255` |
+
+### Response
+
+| 字段 | 类型 | 说明 |
+| --- | --- | --- |
+| `error_code` | `enum ResponseCode` | 响应结果；`OK` 表示成功，其余表示失败原因 |
+
+## 8. 枚举定义
+
+### ResponseCode
+
+| 值 | 含义 |
+| ---: | --- |
+| `0` | `OK` |
+| `1` | `UNKNOWN_TYPE` |
+| `2` | `INVALID_LENGTH` |
+| `3` | `INVALID_PARAM` |
+| `4` | `NOT_READY` |
+
+说明：
+
+- 原 `KeyAction` 枚举已不再使用，因为 `FunctionKeyEvent` 改为完整位图上报。
+
+## 9. CapabilityFlags 定义
+
+`HelloRsp.capability_flags` 先按以下位定义：
+
+| Bit | 含义 |
+| ---: | --- |
+| `0` | 支持功能位图配置 |
+| `1` | 支持时间同步 |
+| `2` | 支持主题切换 |
+| `3` | 支持 LED 状态上报 |
+| `4` | 支持全键盘位图事件上报 |
+
+当前建议最小返回值：
+
+- 若仅先做握手：可先返回 `0`
+- 若 `Bitmap` 可用：打开 `bit0`
+- 若 `FunctionKeyEvent` 全键盘位图上报可用：打开 `bit4`
+
+## 10. 握手流程
+
+### CDC 握手
+
+1. Host 打开串口
+2. Host 发送 `magic + len + protobuf(DeviceMessage{hello_req})`
+3. Device 解析外层帧与 `protobuf(DeviceMessage)`
+4. Device 取出 `hello_req`
+5. Device 返回 `magic + len + protobuf(DeviceMessage{hello_rsp})`
+
+### NUS 握手
+
+1. Host 连接 BLE
+2. Host 发现 `NUS Service`
+3. Host 订阅 `TX Notify`
+4. Host 向 `RX` 写入完整帧
+5. Device 解析外层帧与 `protobuf(DeviceMessage)`
+6. Device 通过 `TX Notify` 发回完整帧
+
+说明：
+
+- 文中 `protobuf(HelloReq)` / `protobuf(HelloRsp)` 指业务上发送对应的 `DeviceMessage` 消息，其 `oneof body` 分别为 `hello_req` / `hello_rsp`。
+
+## 11. 设备行为规则
+
+- 上电默认所有按键都是普通键。
+- 普通键的标准输入行为继续走 `HID / BLE HID`，不变。
+- 收到 `Bitmap` 后，位图中标记为功能键的按键按功能键逻辑处理。
+- `FunctionKeyEvent` 上报的是当前全键盘状态快照，编码格式与 `Bitmap` 一致。
+- `LedState` 在 LED 状态变化时上报。
+- `TimeSync` 收到后更新时间管理模块。
+- `ThemeRgb` 收到后更新主题模块。
+- 控制类请求处理完成后统一返回 `Response`。
+- 不支持的消息类型也应返回 `Response { error_code = UNKNOWN_TYPE }`，不得静默丢弃。
+
+## 12. Response 规则
+
+- `HelloReq` 不返回 `Response`，而是直接返回 `HelloRsp`，且 `reply_to = HelloReq.msg_id`。
+- `Bitmap`、`TimeSync`、`ThemeRgb` 处理完成后应返回 `Response`。
+- `Response.error_code = OK` 表示成功。
+- 不支持的消息类型返回 `Response { error_code = UNKNOWN_TYPE }`。
+- 长度错误、参数错误、模块未就绪时分别返回 `INVALID_LENGTH`、`INVALID_PARAM`、`NOT_READY`。
+
+示例：
+
+- 收到 `Bitmap` 成功处理  
+  返回 `Response { error_code = OK }`
+- 收到 `ThemeRgb` 参数非法  
+  返回 `Response { error_code = INVALID_PARAM }`

docs/dongle-peripheral-design.md

@@ -0,0 +1,476 @@
+# Blinky 键盘 2.4G 端详细设计稿
+
+## 1. 背景
+
+当前 `C:\projects\blinky` 已经完成了外设侧多槽 BLE 基础能力：
+
+- `Slot 1~3` 对应普通 BLE 主机
+- `identity 4` 预留为 `Dongle Slot`
+- `mode_policy_module` 已经区分：
+  - `MODE_SWITCH_BLE -> BLE_PROFILE_POLICY_GENERAL`
+  - `MODE_SWITCH_24G -> BLE_PROFILE_POLICY_DONGLE`
+
+但目前 `identity 4` 还只是“预留位”，没有真正落成完整的 dongle peer 行为。  
+本设计稿的目标，是定义如何在**键盘外设固件内**实现“2.4G 端”，参考 `nrf_desktop` 的外围设备侧 dongle peer 设计。
+
+这里的“2.4G 端”在第一阶段的实现语义不是 ESB 或私有无线协议，而是：
+
+- 键盘继续作为 BLE Peripheral
+- 通过专用 local identity 与自家 dongle 建立专用连接
+- 在产品层把该模式呈现为“2.4G”
+
+这和 `nrf_desktop` 的 `dongle peer` 设计方向一致。
+
+## 2. 设计目标
+
+本方案的目标如下：
+
+- 在当前键盘固件内实现专用 `dongle peer`
+- `MODE_SWITCH_24G` 时固定使用专用 `identity 4`
+- `MODE_SWITCH_BLE` 时继续使用普通 BLE 槽位 `identity 1/2/3`
+- `dongle peer` 与普通 BLE 槽位完全隔离
+- `dongle peer` 有独立 bond 和独立状态显示
+- Swift Pair 对普通 BLE 开启，对 `dongle peer` 默认关闭
+- 后续为自家 dongle 识别准备一个最小 `dev_descr` 风格自定义 GATT 服务
+
+本设计稿不包含“dongle central 固件”的实现，只定义键盘外设侧。
+
+## 3. 参考基线
+
+本设计主要参考本地 `nrf_desktop` 的外围设备侧设计：
+
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\doc\ble_bond.rst`
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\doc\dev_descr.rst`
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\doc\qos.rst`
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\src\modules\dev_descr.c`
+- `c:\ncs\v3.2.3\nrf\applications\nrf_desktop\src\modules\qos.c`
+
+参考结论：
+
+- `dongle peer` 的核心不是新的无线协议，而是**专用 Bluetooth local identity**
+- 外设与 dongle 的专用配对要和普通 BLE peer 隔离
+- `dev_descr` 是未来 dongle 识别外设的关键入口
+- `QoS` 是增强项，不是第一阶段前置条件
+
+## 4. 当前工程现状
+
+当前工程已经具备实现 `dongle peer` 的关键基础：
+
+- `ble_bond_multi_module` 已支持固定 identity 多槽
+- `identity 1/2/3` 已用于普通 BLE 槽位
+- `identity 4` 已预留为 `Dongle Slot`
+- `mode_policy_module` 已输出 `BLE_PROFILE_POLICY_GENERAL / DONGLE`
+- Swift Pair 已默认开启，并且在 `identity 4` 上默认关闭
+
+相关文件：
+
+- `C:\projects\blinky\src\ble_bond_multi_module.c`
+- `C:\projects\blinky\src\mode_policy_module.c`
+- `C:\projects\blinky\inc\events\transport_policy_event.h`
+- `C:\projects\blinky\src\swift_pair_module.c`
+
+当前缺失点：
+
+- `MODE_SWITCH_24G` 没有真正强制切到 `identity 4`
+- `dongle peer` 没有独立 UI 状态
+- 没有 `dev_descr` 风格自定义服务
+- 没有独立的 dongle 管理接口
+
+## 5. 总体设计
+
+## 5.1 核心设计结论
+
+采用 `nrf_desktop` 风格的“专用 dongle peer”设计：
+
+- 保持当前键盘为 BLE Peripheral
+- 使用 `identity 4` 作为专用 dongle identity
+- 以 `MODE_SWITCH_24G` 作为选择 `dongle peer` 的产品模式
+- 普通 BLE 槽位 `1/2/3` 与 `dongle peer` 独立管理
+
+也就是说：
+
+- `2.4G` 是产品语义
+- 第一阶段实现仍然是 BLE Peripheral + 专用 identity
+
+## 5.2 identity 规划
+
+identity 规划继续固定如下：
+
+| Identity | 角色 | 用途 |
+| --- | --- | --- |
+| `0` | 默认 identity | 不使用 |
+| `1` | BLE Slot 1 | 普通 BLE |
+| `2` | BLE Slot 2 | 普通 BLE |
+| `3` | BLE Slot 3 | 普通 BLE |
+| `4` | Dongle Slot | 专用 2.4G / dongle peer |
+
+设计原则：
+
+- `identity 4` 不参与普通槽位轮换
+- `identity 4` 不参与普通 BLE 设置页
+- `identity 4` 有独立 bond 和独立状态
+
+## 6. 模块职责分工
+
+## 6.1 `mode_policy_module`
+
+职责保持不变：
+
+- 根据拨杆位置输出 transport policy
+
+当前定义：
+
+- `MODE_SWITCH_BLE -> BLE_PROFILE_POLICY_GENERAL`
+- `MODE_SWITCH_24G -> BLE_PROFILE_POLICY_DONGLE`
+
+它不直接管理 bond，只输出“当前应该走哪种 BLE profile”。
+
+## 6.2 `ble_bond_multi_module`
+
+这是本方案第一阶段的主修改模块。
+
+需要扩展职责：
+
+- 支持根据 `BLE_PROFILE_POLICY_DONGLE` 固定选择 `identity 4`
+- 普通 BLE 与 `dongle peer` 分开管理
+- 提供 `dongle peer` 独立状态输出
+- 提供 `dongle peer` 独立擦除操作
+
+仍然保留已有职责：
+
+- 普通 BLE 三槽切换
+- 普通 BLE 三槽擦除
+- slot meta 持久化
+
+## 6.3 `swift_pair_module`
+
+职责保持：
+
+- 普通 BLE 槽位启用 Swift Pair
+- `dongle peer` 默认关闭 Swift Pair payload
+
+这和 `nrf_desktop` 的思路一致。
+
+## 6.4 新增 `dev_descr_module`
+
+建议新增一个最小版 `dev_descr` 风格服务。
+
+目标：
+
+- 给未来的 dongle 提供外设识别信息
+- 不依赖设备名做唯一识别
+
+第一阶段最小字段：
+
+- `caps`
+- `hwid`
+
+后续可扩展：
+
+- `role`
+- `product type`
+- `firmware revision`
+
+## 6.5 `qos_module`
+
+第一阶段不做。
+
+理由：
+
+- `nrf_desktop` 已明确说明自家 dongle 不依赖外围设备 QoS
+- 当前先做 `dongle peer` identity 和识别就足够
+
+## 7. 行为模型
+
+## 7.1 普通 BLE 模式
+
+条件：
+
+- `MODE_SWITCH_BLE`
+- `BLE_PROFILE_POLICY_GENERAL`
+
+行为：
+
+- 当前 active slot 只能为 `1/2/3`
+- UI 显示普通 BLE 三槽
+- 允许普通 BLE 切槽和擦除
+- Swift Pair 默认开启
+
+## 7.2 2.4G / Dongle 模式
+
+条件：
+
+- `MODE_SWITCH_24G`
+- `BLE_PROFILE_POLICY_DONGLE`
+
+行为：
+
+- 当前 active identity 固定为 `4`
+- 当前 bond 只属于 `identity 4`
+- 不允许普通 BLE 三槽逻辑影响 `identity 4`
+- Swift Pair 默认关闭
+- UI 应显示：
+  - `Dongle Searching`
+  - `Dongle Paired`
+  - `Dongle Connected`
+  - `Dongle Empty`
+
+## 7.3 模式切换语义
+
+### BLE -> 24G
+
+1. `mode_policy_module` 发布 `BLE_PROFILE_POLICY_DONGLE`
+2. `ble_bond_multi_module` 切 active identity 到 `4`
+3. 提交 `PEER_OPERATION_SELECTED`
+4. CAF `ble_adv` 切到 `identity 4`
+5. 当前连接断开
+6. 使用 `identity 4` 广播
+7. dongle 重连
+
+### 24G -> BLE
+
+1. `mode_policy_module` 发布 `BLE_PROFILE_POLICY_GENERAL`
+2. `ble_bond_multi_module` 恢复普通 BLE 当前槽位 `1/2/3`
+3. 提交 `PEER_OPERATION_SELECTED`
+4. CAF `ble_adv` 切回对应 identity
+5. 当前连接断开
+6. 使用普通 BLE 槽位广播
+
+## 8. 状态与事件设计
+
+## 8.1 当前已有事件
+
+可继续复用：
+
+- `transport_policy_event`
+- `ble_peer_event`
+- `ble_peer_operation_event`
+- `ble_bond_multi_event`
+
+## 8.2 建议扩展 `ble_bond_multi_event`
+
+建议增加字段或语义：
+
+- `active_profile`
+  - `GENERAL`
+  - `DONGLE`
+- `dongle_slot_meta`
+  - `occupied`
+  - `last_peer_addr`
+  - `display_name`
+- `current_identity`
+
+如果不想改现有事件结构，也可以新增一个独立事件：
+
+- `ble_dongle_state_event`
+
+推荐字段：
+
+- `state`
+  - `EMPTY`
+  - `SEARCHING`
+  - `PAIRED`
+  - `CONNECTED`
+- `identity_id`
+- `display_name`
+
+## 8.3 UI 状态来源
+
+主界面与设置页可以通过以下信息判断：
+
+- 当前 `mode`
+- 当前 `active_identity_id`
+- `identity 4` 是否有 bond
+- 当前是否有连接且该连接属于 `identity 4`
+
+## 9. `dev_descr` 设计
+
+## 9.1 目标
+
+最小化模仿 `nrf_desktop dev_descr`：
+
+- 为 dongle 提供一个稳定可读的识别服务
+- 避免仅靠设备名判断
+
+## 9.2 服务内容
+
+建议服务包含两个 characteristic：
+
+### Characteristic 1: `caps`
+
+建议字节布局：
+
+- bit0: `supports_llpm`
+- bit1: `supports_dongle_peer`
+- bit2: `is_keyboard`
+
+第一阶段只要能读出：
+
+- 这个设备支持 dongle peer
+- 这是 keyboard 类型
+
+### Characteristic 2: `hwid`
+
+固定长度硬件 ID：
+
+- 用于以后让 dongle 唯一识别这把键盘
+- 也可用于配置通道映射
+
+## 9.3 权限建议
+
+建议与 `nrf_desktop` 保持一致：
+
+- `BT_GATT_PERM_READ_ENCRYPT`
+
+原因：
+
+- 避免未加密链路上暴露识别信息
+
+## 9.4 依赖
+
+需要：
+
+- `CONFIG_HWINFO`
+- 固件内已有或新增 `hwid_get()` 辅助逻辑
+
+## 10. Settings 设计
+
+继续沿用现有 namespace：
+
+- `ble_multi/current_slot`
+- `ble_multi/meta/1`
+- `ble_multi/meta/2`
+- `ble_multi/meta/3`
+- `ble_multi/meta/4`
+
+其中 `meta/4` 现在正式用于 dongle peer。
+
+建议 `meta/4` 字段仍保持：
+
+- `occupied`
+- `last_peer_addr`
+- `display_name`
+
+显示策略：
+
+- 有真实名字则显示名字
+- 否则显示 MAC 地址
+- 无 bond 显示 `Empty`
+
+## 11. UI 设计
+
+## 11.1 主界面
+
+当前主界面可新增或扩展显示：
+
+- 当前模式 `USB / BLE / 24G`
+- 如果 `24G` 模式：
+  - 右上角额外状态位显示：
+    - searching
+    - connected
+
+现有 BLE LINK 状态位可以复用，但语义建议细化：
+
+- BLE 模式下：表示普通 BLE 当前槽状态
+- 24G 模式下：表示 dongle peer 状态
+
+## 11.2 设置页
+
+建议新增二级入口：
+
+- `Bluetooth`
+  - Slot 1
+  - Slot 2
+  - Slot 3
+  - Erase Current
+- `Dongle`
+  - Status
+  - Erase Dongle Bond
+
+第一阶段如果不想扩菜单，至少要做到：
+
+- 主界面显示 dongle peer 状态
+- 设置页不把 `identity 4` 混入 Slot 1~3
+
+## 12. 分阶段实施建议
+
+## 阶段 1：Dongle Slot 真正接线
+
+内容：
+
+- `ble_bond_multi_module` 支持 `BLE_PROFILE_POLICY_DONGLE`
+- `MODE_SWITCH_24G` 时固定切到 `identity 4`
+- `identity 4` 独立 bond 和独立状态
+- Swift Pair 在 `identity 4` 默认关闭
+
+交付标准：
+
+- 24G 模式和 BLE 模式切换时 identity 正确切换
+- `identity 4` 与 `1/2/3` 完全隔离
+
+## 阶段 2：UI / 状态管理
+
+内容：
+
+- 增加 dongle 状态显示
+- 增加独立擦除 dongle bond 入口
+- 不把 dongle 混入普通 BLE 三槽 UI
+
+交付标准：
+
+- 用户可以明确区分：
+  - 普通 BLE 槽位
+  - 2.4G / dongle peer
+
+## 阶段 3：最小 `dev_descr`
+
+内容：
+
+- 实现最小自定义 GATT service
+- 提供 `caps + hwid`
+
+交付标准：
+
+- 未来 dongle 可稳定识别这把键盘
+
+## 阶段 4：后续增强（非当前范围）
+
+可选项：
+
+- QoS
+- 更强的 dongle 配对约束
+- 生产预绑定
+- 更明确的 dongle selector/恢复流程
+
+## 13. 风险与边界
+
+## 13.1 “2.4G” 实际仍然是 BLE
+
+第一阶段产品语义是 2.4G，链路实现仍是 BLE。  
+这没有问题，但必须在代码和文档中明确，避免后续和真正私有 2.4G 链路混淆。
+
+## 13.2 `dev_descr` 与未来 dongle 要对齐
+
+如果未来 dongle 端也参考 `nrf_desktop central`，那这里的 UUID 和字段应该尽量稳定，不要后面再频繁改。
+
+## 13.3 `identity 4` 与普通 UI 必须隔离
+
+一旦 `identity 4` 被误混入普通三槽，就会造成用户理解混乱：
+
+- 不知道哪个是普通蓝牙
+- 哪个是 2.4G
+
+这是 UI 设计上最需要避免的问题。
+
+## 14. 结论
+
+本方案建议把当前键盘的“2.4G 端”实现为：
+
+- `BLE Peripheral + 专用 Dongle identity`
+- `MODE_SWITCH_24G` 时固定走 `identity 4`
+- 与普通 BLE 三槽完全隔离
+- 第一阶段先打通 identity / bond / UI 状态
+- 第二阶段补最小 `dev_descr`
+- `QoS` 暂缓
+
+这条路径与 `nrf_desktop` 外设侧的 dongle peer 设计一致，也与当前工程已有的 `transport_policy_event` 语义完全匹配。

inc/events/ble_bond_multi_event.h

@@ -0,0 +1,46 @@
+#ifndef BLINKY_BLE_BOND_MULTI_EVENT_H_
+#define BLINKY_BLE_BOND_MULTI_EVENT_H_
+
+#include <stdint.h>
+
+#include <app_event_manager.h>
+#include <app_event_manager_profiler_tracer.h>
+#include <zephyr/bluetooth/addr.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+#define BLE_BOND_MULTI_DISPLAY_NAME_MAX_LEN 32U
+#define BLE_BOND_MULTI_BLE_SLOT_COUNT 3U
+#define BLE_BOND_MULTI_DONGLE_SLOT_ID 4U
+
+enum ble_bond_multi_op {
+	BLE_BOND_MULTI_OP_NONE = 0,
+	BLE_BOND_MULTI_OP_REFRESH,
+	BLE_BOND_MULTI_OP_SWITCH,
+	BLE_BOND_MULTI_OP_ERASE,
+};
+
+struct ble_bond_multi_slot_meta {
+	bool occupied;
+	bt_addr_le_t last_peer_addr;
+	char display_name[BLE_BOND_MULTI_DISPLAY_NAME_MAX_LEN];
+};
+
+struct ble_bond_multi_event {
+	struct app_event_header header;
+	uint8_t current_slot;
+	uint8_t active_identity_id;
+	enum ble_bond_multi_op op;
+	uint8_t slot_occupied_bitmap;
+	struct ble_bond_multi_slot_meta slots[BLE_BOND_MULTI_BLE_SLOT_COUNT];
+};
+
+APP_EVENT_TYPE_DECLARE(ble_bond_multi_event);
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_BLE_BOND_MULTI_EVENT_H_ */

inc/events/proto_common.h

@@ -16,6 +16,11 @@ enum proto_transport_link_state {
 	PROTO_TRANSPORT_LINK_READY,
 };
 
+#define PROTO_FRAME_MAGIC 0xAA55U
+#define PROTO_FRAME_HEADER_SIZE 3U
+#define PROTO_MAX_PAYLOAD_LEN 64U
+#define PROTO_MAX_FRAME_LEN (PROTO_FRAME_HEADER_SIZE + PROTO_MAX_PAYLOAD_LEN)
+
 #ifdef __cplusplus
 }
 #endif

inc/events/proto_rx_event.h

@@ -28,7 +28,8 @@ static inline int submit_proto_rx_event(enum proto_transport transport,
 	struct proto_rx_event *event;
 
 	if ((transport >= PROTO_TRANSPORT_COUNT) ||
-	    ((data == NULL) && (len > 0U))) {
+	    ((data == NULL) && (len > 0U)) ||
+	    (len > PROTO_MAX_FRAME_LEN)) {
 		return -EINVAL;
 	}
 

inc/events/proto_tx_event.h

@@ -28,7 +28,8 @@ static inline int submit_proto_tx_event(enum proto_transport transport,
 	struct proto_tx_event *event;
 
 	if ((transport >= PROTO_TRANSPORT_COUNT) ||
-	    ((data == NULL) && (len > 0U))) {
+	    ((data == NULL) && (len > 0U)) ||
+	    (len > PROTO_MAX_FRAME_LEN)) {
 		return -EINVAL;
 	}
 

inc/events/settings_mode_event.h

@@ -0,0 +1,30 @@
+#ifndef BLINKY_SETTINGS_MODE_EVENT_H_
+#define BLINKY_SETTINGS_MODE_EVENT_H_
+
+#include <app_event_manager.h>
+#include <app_event_manager_profiler_tracer.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+struct settings_mode_event {
+	struct app_event_header header;
+	bool active;
+};
+
+APP_EVENT_TYPE_DECLARE(settings_mode_event);
+
+static inline void submit_settings_mode_event(bool active)
+{
+	struct settings_mode_event *event = new_settings_mode_event();
+
+	event->active = active;
+	APP_EVENT_SUBMIT(event);
+}
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_SETTINGS_MODE_EVENT_H_ */

inc/events/settings_view_event.h

@@ -0,0 +1,35 @@
+#ifndef BLINKY_SETTINGS_VIEW_EVENT_H_
+#define BLINKY_SETTINGS_VIEW_EVENT_H_
+
+#include <app_event_manager.h>
+#include <app_event_manager_profiler_tracer.h>
+
+#include "ui/ui_settings_page.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+struct settings_view_event {
+	struct app_event_header header;
+	struct ui_settings_page *page;
+	bool animate;
+};
+
+APP_EVENT_TYPE_DECLARE(settings_view_event);
+
+static inline void submit_settings_view_event(struct ui_settings_page *page,
+					      bool animate)
+{
+	struct settings_view_event *event = new_settings_view_event();
+
+	event->page = page;
+	event->animate = animate;
+	APP_EVENT_SUBMIT(event);
+}
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_SETTINGS_VIEW_EVENT_H_ */

inc/events/transport_policy_event.h

@@ -0,0 +1,42 @@
+#ifndef BLINKY_TRANSPORT_POLICY_EVENT_H_
+#define BLINKY_TRANSPORT_POLICY_EVENT_H_
+
+#include <app_event_manager.h>
+
+#include "mode_switch_event.h"
+
+enum hid_transport_policy {
+	HID_TRANSPORT_POLICY_NONE = 0,
+	HID_TRANSPORT_POLICY_USB,
+	HID_TRANSPORT_POLICY_BLE,
+};
+
+enum ble_profile_policy {
+	BLE_PROFILE_POLICY_NONE = 0,
+	BLE_PROFILE_POLICY_GENERAL,
+	BLE_PROFILE_POLICY_DONGLE,
+};
+
+struct transport_policy_event {
+	struct app_event_header header;
+	enum mode_switch_mode source_mode;
+	enum hid_transport_policy hid_transport;
+	enum ble_profile_policy ble_profile;
+};
+
+APP_EVENT_TYPE_DECLARE(transport_policy_event);
+
+static inline void submit_transport_policy_event(
+	enum mode_switch_mode source_mode,
+	enum hid_transport_policy hid_transport,
+	enum ble_profile_policy ble_profile)
+{
+	struct transport_policy_event *event = new_transport_policy_event();
+
+	event->source_mode = source_mode;
+	event->hid_transport = hid_transport;
+	event->ble_profile = ble_profile;
+	APP_EVENT_SUBMIT(event);
+}
+
+#endif /* BLINKY_TRANSPORT_POLICY_EVENT_H_ */

inc/module_lifecycle.h

@@ -8,9 +8,8 @@
 
 enum module_lifecycle {
 	LC_UNINIT,
-	LC_RUNNING,
 	LC_STOPPED,
-	LC_SUSPENDED,
+	LC_RUNNING,
 	LC_ERROR,
 };
 
@@ -49,10 +48,43 @@ static inline bool module_lifecycle_is_initialized(
 	return (ctx->state != LC_UNINIT) && (ctx->state != LC_ERROR);
 }
 
-static inline bool module_lifecycle_reports_stopped_state(
+static inline const char *module_lifecycle_name(enum module_lifecycle state)
+{
+	switch (state) {
+	case LC_UNINIT:
+		return "UNINIT";
+	case LC_STOPPED:
+		return "STOPPED";
+	case LC_RUNNING:
+		return "RUNNING";
+	case LC_ERROR:
+		return "ERROR";
+	default:
+		return "?";
+	}
+}
+
+static inline int module_lifecycle_validate_ops(
 	const struct module_lifecycle_ctx *ctx)
 {
-	return ctx->cfg->mode != ML_MODE_NONE;
+	if ((ctx == NULL) || (ctx->cfg == NULL) || (ctx->ops == NULL) ||
+	    (ctx->ops->do_init == NULL)) {
+		return -EINVAL;
+	}
+
+	switch (ctx->cfg->mode) {
+	case ML_MODE_NONE:
+		return 0;
+
+	case ML_MODE_POWER:
+	case ML_MODE_SUSPEND:
+		return (ctx->ops->do_start != NULL) && (ctx->ops->do_stop != NULL) ?
+			       0 :
+			       -EINVAL;
+
+	default:
+		return -EINVAL;
+	}
 }
 
 static inline int module_lifecycle_report_state(
@@ -65,17 +97,22 @@ static inline int module_lifecycle_report_state(
 		return 0;
 
 	case LC_STOPPED:
-		if (!module_lifecycle_reports_stopped_state(ctx)) {
+		switch (ctx->cfg->mode) {
+		case ML_MODE_POWER:
+			module_set_state(ctx->cfg->stopped_state);
 			return 0;
+
+		case ML_MODE_SUSPEND:
+			module_set_state(MODULE_STATE_SUSPENDED);
+			return 0;
+
+		case ML_MODE_NONE:
+			return 0;
+
+		default:
+			return -EINVAL;
 		}
 
-		module_set_state(ctx->cfg->stopped_state);
-		return 0;
-
-	case LC_SUSPENDED:
-		module_set_state(MODULE_STATE_SUSPENDED);
-		return 0;
-
 	case LC_ERROR:
 		module_set_state(MODULE_STATE_ERROR);
 		return 0;
@@ -94,50 +131,14 @@ static inline int module_lifecycle_fail(struct module_lifecycle_ctx *ctx, int er
 	return err ? err : -EIO;
 }
 
-static inline int module_lifecycle_do_init(struct module_lifecycle_ctx *ctx)
-{
-	int err = ctx->ops->do_init();
-
-	if (err) {
-		return module_lifecycle_fail(ctx, err);
-	}
-
-	return 0;
-}
-
-static inline int module_lifecycle_do_start(struct module_lifecycle_ctx *ctx,
-					    enum module_lifecycle target)
-{
-	int err = ctx->ops->do_start();
-
-	if (err) {
-		return module_lifecycle_fail(ctx, err);
-	}
-
-	ctx->state = target;
-	return module_lifecycle_report_state(ctx, target);
-}
-
-static inline int module_lifecycle_do_stop(struct module_lifecycle_ctx *ctx,
-					   enum module_lifecycle target)
-{
-	int err = ctx->ops->do_stop();
-
-	if (err) {
-		return module_lifecycle_fail(ctx, err);
-	}
-
-	ctx->state = target;
-	return module_lifecycle_report_state(ctx, target);
-}
-
 static inline int module_set_lifecycle(struct module_lifecycle_ctx *ctx,
 				       enum module_lifecycle target)
 {
-	if ((ctx == NULL) || (ctx->cfg == NULL) || (ctx->ops == NULL) ||
-	    (ctx->ops->do_init == NULL) || (ctx->ops->do_start == NULL) ||
-	    (ctx->ops->do_stop == NULL)) {
-		return -EINVAL;
+	int err;
+
+	err = module_lifecycle_validate_ops(ctx);
+	if (err) {
+		return err;
 	}
 
 	if (ctx->state == LC_ERROR) {
@@ -152,53 +153,55 @@ static inline int module_set_lifecycle(struct module_lifecycle_ctx *ctx,
 		return module_lifecycle_fail(ctx, -EIO);
 	}
 
+	if ((target != LC_STOPPED) && (target != LC_RUNNING)) {
+		return -EPERM;
+	}
+
 	switch (ctx->state) {
 	case LC_UNINIT:
-		if (target == LC_RUNNING) {
-			int err = module_lifecycle_do_init(ctx);
-
-			if (err) {
-				return err;
-			}
-
-			ctx->state = LC_STOPPED;
-			return module_lifecycle_do_start(ctx, LC_RUNNING);
+		err = ctx->ops->do_init();
+		if (err) {
+			return module_lifecycle_fail(ctx, err);
 		}
 
-		if (target == LC_STOPPED) {
-			int err = module_lifecycle_do_init(ctx);
-
-			if (err) {
-				return err;
-			}
-
-			ctx->state = LC_STOPPED;
-			return module_lifecycle_report_state(ctx, LC_STOPPED);
+		ctx->state = LC_STOPPED;
+		err = module_lifecycle_report_state(ctx, LC_STOPPED);
+		if (err) {
+			return module_lifecycle_fail(ctx, err);
 		}
 
-		break;
-
-	case LC_RUNNING:
-		if ((target == LC_STOPPED) || (target == LC_SUSPENDED)) {
-			return module_lifecycle_do_stop(ctx, target);
-		}
-
-		break;
+		return module_set_lifecycle(ctx, target);
 
 	case LC_STOPPED:
-	case LC_SUSPENDED:
-		if (target == LC_RUNNING) {
-			return module_lifecycle_do_start(ctx, LC_RUNNING);
+		if (target != LC_RUNNING) {
+			return -EPERM;
 		}
 
-		break;
+		err = ctx->ops->do_start ? ctx->ops->do_start() : 0;
+		if (err) {
+			return module_lifecycle_fail(ctx, err);
+		}
+
+		ctx->state = LC_RUNNING;
+		return module_lifecycle_report_state(ctx, LC_RUNNING);
+
+	case LC_RUNNING:
+		if (target != LC_STOPPED) {
+			return -EPERM;
+		}
+
+		err = ctx->ops->do_stop ? ctx->ops->do_stop() : 0;
+		if (err) {
+			return module_lifecycle_fail(ctx, err);
+		}
+
+		ctx->state = LC_STOPPED;
+		return module_lifecycle_report_state(ctx, LC_STOPPED);
 
 	case LC_ERROR:
 	default:
-		break;
+		return -EPERM;
 	}
-
-	return -EPERM;
 }
 
 #endif /* MODULE_LIFECYCLE_H_ */

inc/ui/ui_page.h

@@ -0,0 +1,66 @@
+#ifndef BLINKY_UI_PAGE_H_
+#define BLINKY_UI_PAGE_H_
+
+#include <stdbool.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+struct ui_page;
+
+struct ui_page_ops {
+	void (*init)(struct ui_page *page);
+	void (*deinit)(struct ui_page *page);
+	void (*refresh)(struct ui_page *page);
+};
+
+struct ui_page {
+	const struct ui_page_ops *ops;
+	struct ui_page *parent;
+	bool initialized;
+};
+
+static inline void ui_page_init(struct ui_page *page)
+{
+	if ((page == NULL) || (page->ops == NULL)) {
+		return;
+	}
+
+	if (page->initialized) {
+		return;
+	}
+
+	if (page->ops->init != NULL) {
+		page->ops->init(page);
+	}
+	page->initialized = true;
+}
+
+static inline void ui_page_deinit(struct ui_page *page)
+{
+	if ((page == NULL) || !page->initialized || (page->ops == NULL)) {
+		return;
+	}
+
+	if (page->ops->deinit != NULL) {
+		page->ops->deinit(page);
+	}
+	page->initialized = false;
+}
+
+static inline void ui_page_refresh(struct ui_page *page)
+{
+	if ((page == NULL) || !page->initialized ||
+	    (page->ops == NULL) || (page->ops->refresh == NULL)) {
+		return;
+	}
+
+	page->ops->refresh(page);
+}
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_UI_PAGE_H_ */

inc/ui/ui_settings_controller.h

@@ -0,0 +1,43 @@
+#ifndef BLINKY_UI_SETTINGS_CONTROLLER_H_
+#define BLINKY_UI_SETTINGS_CONTROLLER_H_
+
+#include <stdbool.h>
+#include <stdint.h>
+
+#include "theme_color.h"
+#include "ui_page.h"
+#include "ui_settings_page.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+struct ble_bond_multi_slot_meta;
+
+void ui_settings_controller_open(void);
+void ui_settings_controller_close(void);
+bool ui_settings_controller_back(void);
+void ui_settings_controller_select(void);
+void ui_settings_controller_move(int8_t delta);
+void ui_settings_controller_refresh(bool animate);
+bool ui_settings_controller_is_active(void);
+
+void ui_settings_controller_switch_to(struct ui_settings_page *page,
+				      struct ui_page *parent);
+
+const char *ui_settings_ble_current_label(void);
+const char *ui_settings_ble_slot_label(uint8_t slot);
+void ui_settings_ble_select_slot(uint8_t slot);
+void ui_settings_ble_erase_current(void);
+void ui_settings_ble_set_current_slot(uint8_t slot);
+void ui_settings_ble_set_slot_meta(uint8_t slot,
+				   const struct ble_bond_multi_slot_meta *meta);
+void ui_settings_theme_set_current(struct theme_rgb theme);
+const char *ui_settings_theme_current_name(void);
+uint8_t ui_settings_theme_current_index(void);
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_UI_SETTINGS_CONTROLLER_H_ */

inc/ui/ui_settings_page.h

@@ -0,0 +1,56 @@
+#ifndef BLINKY_UI_SETTINGS_PAGE_H_
+#define BLINKY_UI_SETTINGS_PAGE_H_
+
+#include <stdint.h>
+
+#include <lvgl.h>
+
+#include "ui_page.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+struct ui_settings_page;
+struct ui_settings_item;
+
+typedef void (*ui_settings_item_draw_fn)(const struct ui_settings_item *item,
+					 lv_obj_t *container);
+
+struct ui_settings_item {
+	const char *icon;
+	const char *title;
+	const char *value;
+	ui_settings_item_draw_fn draw;
+	void *user_data;
+};
+
+struct ui_settings_page_ops {
+	struct ui_page_ops base;
+
+	uint8_t (*get_count)(struct ui_settings_page *page);
+	void (*get_item)(struct ui_settings_page *page, uint8_t index,
+			 struct ui_settings_item *item);
+	void (*on_enter)(struct ui_settings_page *page);
+	void (*on_select)(struct ui_settings_page *page, uint8_t index);
+	void (*on_back)(struct ui_settings_page *page);
+};
+
+struct ui_settings_page {
+	struct ui_page base;
+	const struct ui_settings_page_ops *ops;
+	const char *title;
+	const char *hint;
+	uint8_t selected;
+};
+
+static inline struct ui_settings_page *ui_page_to_settings(struct ui_page *page)
+{
+	return (struct ui_settings_page *)page;
+}
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_UI_SETTINGS_PAGE_H_ */

prj.conf

@@ -3,6 +3,7 @@ CONFIG_CAF_BUTTONS=y
 CONFIG_CAF_BUTTONS_DEF_PATH="buttons_def.h"
 CONFIG_CAF_CLICK_DETECTOR=y
 CONFIG_CAF_CLICK_DETECTOR_DEF_PATH="click_detector_def.h"
+CONFIG_CAF_CLICK_DETECTOR_LONG_CLICK_MSEC=1500
 CONFIG_GPIO=y
 CONFIG_I2C=y
 CONFIG_LED=y
@@ -27,12 +28,18 @@ CONFIG_NVS=y
 CONFIG_HEAP_MEM_POOL_SIZE=4096
 CONFIG_LOG=y
 CONFIG_LOG_BUFFER_SIZE=16384
+CONFIG_LOG_BACKEND_UART=y
+CONFIG_LOG_BACKEND_SHOW_COLOR=n
 CONFIG_LOG_BACKEND_RTT_OUTPUT_BUFFER_SIZE=16384
 CONFIG_ASSERT=y
 CONFIG_SEGGER_RTT_BUFFER_SIZE_UP=16384
 CONFIG_APP_EVENT_MANAGER_MAX_EVENT_CNT=64
 CONFIG_LED_STRIP=y
 CONFIG_WS2812_STRIP_SPI=y
+CONFIG_CONSOLE=y
+CONFIG_UART_CONSOLE=y
+CONFIG_STDOUT_CONSOLE=y
+CONFIG_PRINTK=y
 
 # USB HID next stack
 CONFIG_USB_DEVICE_STACK_NEXT=y
@@ -53,7 +60,8 @@ CONFIG_BT_SMP=y
 CONFIG_BT_BONDABLE=y
 CONFIG_BT_SETTINGS=y
 CONFIG_BT_MAX_CONN=1
-CONFIG_BT_MAX_PAIRED=1
+CONFIG_BT_MAX_PAIRED=4
+CONFIG_BT_ID_MAX=5
 CONFIG_BT_ATT_TX_COUNT=5
 CONFIG_BT_L2CAP_TX_MTU=65
 CONFIG_BT_BUF_ACL_RX_SIZE=69
@@ -103,16 +111,13 @@ CONFIG_CAF_BLE_ADV_SUSPEND_ON_READY=y
 CONFIG_CAF_BLE_ADV_FAST_ADV=y
 CONFIG_CAF_BLE_ADV_FILTER_ACCEPT_LIST=y
 CONFIG_CAF_BLE_ADV_MODULE_SUSPEND_EVENTS=y
-CONFIG_CAF_BLE_BOND=y
-CONFIG_CAF_BLE_BOND_PEER_ERASE_CLICK=y
-CONFIG_CAF_BLE_BOND_PEER_ERASE_CLICK_KEY_ID=0x180
-CONFIG_CAF_BLE_BOND_PEER_ERASE_CLICK_LONG=y
-CONFIG_CAF_BLE_BOND_PEER_ERASE_CLICK_TIMEOUT=-1
+CONFIG_CAF_BLE_BOND=n
 CONFIG_CAF_MODULE_SUSPEND_EVENTS=y
 CONFIG_BT_ADV_PROV_FLAGS=y
 CONFIG_BT_ADV_PROV_GAP_APPEARANCE=y
 CONFIG_BT_ADV_PROV_DEVICE_NAME=y
 CONFIG_BT_ADV_PROV_DEVICE_NAME_SD=y
+CONFIG_BT_ADV_PROV_SWIFT_PAIR=y
 
 # LVGL
 CONFIG_LVGL=y
@@ -125,8 +130,11 @@ CONFIG_LV_COLOR_16_SWAP=y
 CONFIG_LV_Z_BITS_PER_PIXEL=16
 CONFIG_LV_Z_VDB_SIZE=25
 CONFIG_LV_Z_DOUBLE_VDB=y
-CONFIG_LV_Z_MEM_POOL_SIZE=16384
+CONFIG_LV_Z_MEM_POOL_SIZE=32768
 CONFIG_LV_USE_LABEL=y
 CONFIG_LV_FONT_MONTSERRAT_14=y
 CONFIG_LV_FONT_MONTSERRAT_32=y
 CONFIG_MAIN_STACK_SIZE=4096
+
+CONFIG_USE_SEGGER_RTT=y
+CONFIG_SPEED_OPTIMIZATIONS=y

proto/device_comm.proto

@@ -1,5 +1,13 @@
 syntax = "proto3";
 
+enum ResponseCode {
+  RESPONSE_CODE_OK = 0;
+  RESPONSE_CODE_UNKNOWN_TYPE = 1;
+  RESPONSE_CODE_INVALID_LENGTH = 2;
+  RESPONSE_CODE_INVALID_PARAM = 3;
+  RESPONSE_CODE_NOT_READY = 4;
+}
+
 message HelloReq {
   uint32 protocol_version = 1;
 }
@@ -39,24 +47,14 @@ message ThemeRgb {
   uint32 blue = 3;
 }
 
-message Ack {
-  uint32 acked_type = 1;
+message Response {
+  ResponseCode error_code = 1;
 }
 
-enum ErrorCode {
-  ERROR_CODE_NONE = 0;
-  ERROR_CODE_UNKNOWN_TYPE = 1;
-  ERROR_CODE_INVALID_LENGTH = 2;
-  ERROR_CODE_INVALID_PARAM = 3;
-  ERROR_CODE_NOT_READY = 4;
-}
+message DeviceMessage {
+  uint32 msg_id = 10;
+  uint32 reply_to = 11;
 
-message Error {
-  uint32 error_type = 1;
-  ErrorCode error_code = 2;
-}
-
-message CdcPacketBody {
   oneof body {
     HelloReq hello_req = 1;
     HelloRsp hello_rsp = 2;
@@ -65,7 +63,6 @@ message CdcPacketBody {
     LedState led_state = 5;
     TimeSync time_sync = 6;
     ThemeRgb theme_rgb = 7;
-    Ack ack = 8;
-    Error error = 9;
+    Response response = 8;
   }
 }

src/ble_bas_module.c

@@ -105,9 +105,7 @@ static bool app_event_handler(const struct app_event_header *aeh)
 	if (is_module_state_event(aeh)) {
 		const struct module_state_event *event = cast_module_state_event(aeh);
 		if (check_state(event, MODULE_ID(main), MODULE_STATE_READY)) {
-			(void)module_set_lifecycle(&ctx.lc,
-						   ctx.ble_ready ? LC_RUNNING :
-								   LC_STOPPED);
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
 
 			return false;
 		}
@@ -115,9 +113,7 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		if (check_state(event, MODULE_ID(ble_state), MODULE_STATE_READY)) {
 			ctx.ble_ready = true;
 
-			if (ctx.lc.state == LC_STOPPED) {
-				(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
-			} else if (module_lifecycle_is_running(&ctx.lc)) {
+			if (module_lifecycle_is_running(&ctx.lc)) {
 				(void)bas_update_level();
 			}
 			return false;

src/ble_bond_multi_module.c

@@ -0,0 +1,555 @@
+#include <stdlib.h>
+#include <errno.h>
+#include <stdbool.h>
+#include <stddef.h>
+#include <string.h>
+
+#include <app_event_manager.h>
+
+#define MODULE ble_bond
+#include <caf/events/module_state_event.h>
+
+#include <caf/events/ble_common_event.h>
+#include <zephyr/bluetooth/bluetooth.h>
+#include <zephyr/bluetooth/conn.h>
+#include <zephyr/bluetooth/addr.h>
+#include <zephyr/settings/settings.h>
+#include <zephyr/sys/printk.h>
+#include <zephyr/sys/util.h>
+#include <zephyr/logging/log.h>
+
+#include "ble_bond_multi_event.h"
+#include "module_lifecycle.h"
+
+LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
+
+#define BLE_SLOT_MIN 1U
+#define BLE_SLOT_MAX BLE_BOND_MULTI_BLE_SLOT_COUNT
+#define IDENTITY_DONGLE BLE_BOND_MULTI_DONGLE_SLOT_ID
+#define SETTINGS_KEY_CURRENT_SLOT "current_slot"
+#define SETTINGS_KEY_META_PREFIX "meta/"
+#define DEFAULT_DISPLAY_NAME_EMPTY "Empty"
+
+struct ble_bond_multi_slot_meta_storage {
+	uint8_t occupied;
+	bt_addr_le_t last_peer_addr;
+	char display_name[BLE_BOND_MULTI_DISPLAY_NAME_MAX_LEN];
+};
+
+struct ble_bond_multi_ctx {
+	struct module_lifecycle_ctx lc;
+	uint8_t current_slot;
+	uint8_t pending_slot;
+	bool current_slot_valid;
+	bool identities_ready;
+	struct ble_bond_multi_slot_meta slot_meta[CONFIG_BT_ID_MAX];
+	struct bt_conn *active_conn;
+};
+
+static int do_init(void);
+static int do_start(void);
+static int do_stop(void);
+static int identity_ensure_exists(uint8_t identity);
+static void slot_bond_cnt_cb(const struct bt_bond_info *info, void *user_data);
+int ble_bond_multi_select_slot(uint8_t slot);
+int ble_bond_multi_erase_current_slot(void);
+const struct ble_bond_multi_slot_meta *ble_bond_multi_get_slot_meta(uint8_t slot);
+uint8_t ble_bond_multi_get_current_slot(void);
+
+static const struct module_lifecycle_cfg lifecycle_cfg = {
+	.mode = ML_MODE_NONE,
+	.stopped_state = MODULE_STATE_OFF,
+};
+
+static const struct module_lifecycle_ops lifecycle_ops = {
+	.do_init = do_init,
+	.do_start = do_start,
+	.do_stop = do_stop,
+};
+
+static struct ble_bond_multi_ctx ctx = {
+	.lc = {
+		.state = LC_UNINIT,
+		.cfg = &lifecycle_cfg,
+		.ops = &lifecycle_ops,
+	},
+	.current_slot = BLE_SLOT_MIN,
+	.pending_slot = BLE_SLOT_MIN,
+};
+
+BUILD_ASSERT(CONFIG_BT_ID_MAX > IDENTITY_DONGLE,
+	     "BT_ID_MAX must include BLE slots and dongle slot");
+BUILD_ASSERT(CONFIG_BT_MAX_PAIRED >= 4,
+	     "BT_MAX_PAIRED must allow three BLE slots and dongle slot");
+
+static bool is_ble_slot(uint8_t slot)
+{
+	return (slot >= BLE_SLOT_MIN) && (slot <= BLE_SLOT_MAX);
+}
+
+static void display_name_set_addr(uint8_t slot, const bt_addr_le_t *addr)
+{
+	struct ble_bond_multi_slot_meta *meta = &ctx.slot_meta[slot];
+
+	if ((addr == NULL) || !bt_addr_le_cmp(addr, BT_ADDR_LE_ANY)) {
+		strncpy(meta->display_name, DEFAULT_DISPLAY_NAME_EMPTY,
+			sizeof(meta->display_name));
+		meta->display_name[sizeof(meta->display_name) - 1U] = '\0';
+		return;
+	}
+
+	bt_addr_le_to_str(addr, meta->display_name, sizeof(meta->display_name));
+	meta->display_name[sizeof(meta->display_name) - 1U] = '\0';
+}
+
+static void display_name_set_default(uint8_t slot)
+{
+	struct ble_bond_multi_slot_meta *meta = &ctx.slot_meta[slot];
+	const char *name = DEFAULT_DISPLAY_NAME_EMPTY;
+
+	if (meta->occupied &&
+	    bt_addr_le_cmp(&meta->last_peer_addr, BT_ADDR_LE_ANY)) {
+		display_name_set_addr(slot, &meta->last_peer_addr);
+		return;
+	}
+
+	strncpy(meta->display_name, name, sizeof(meta->display_name));
+	meta->display_name[sizeof(meta->display_name) - 1U] = '\0';
+}
+
+static void slot_meta_clear(uint8_t slot)
+{
+	struct ble_bond_multi_slot_meta *meta = &ctx.slot_meta[slot];
+
+	memset(meta, 0, sizeof(*meta));
+	bt_addr_le_copy(&meta->last_peer_addr, BT_ADDR_LE_ANY);
+	display_name_set_default(slot);
+}
+
+static void slot_meta_ensure_name(uint8_t slot)
+{
+	if (ctx.slot_meta[slot].display_name[0] == '\0') {
+		display_name_set_default(slot);
+	}
+}
+
+static void slot_meta_from_storage(uint8_t slot,
+				   const struct ble_bond_multi_slot_meta_storage *storage)
+{
+	struct ble_bond_multi_slot_meta *meta = &ctx.slot_meta[slot];
+
+	meta->occupied = (storage->occupied != 0U);
+	bt_addr_le_copy(&meta->last_peer_addr, &storage->last_peer_addr);
+	memcpy(meta->display_name, storage->display_name, sizeof(meta->display_name));
+	meta->display_name[sizeof(meta->display_name) - 1U] = '\0';
+	slot_meta_ensure_name(slot);
+}
+
+static void slot_meta_to_storage(uint8_t slot,
+				 struct ble_bond_multi_slot_meta_storage *storage)
+{
+	const struct ble_bond_multi_slot_meta *meta = &ctx.slot_meta[slot];
+
+	memset(storage, 0, sizeof(*storage));
+	storage->occupied = meta->occupied ? 1U : 0U;
+	bt_addr_le_copy(&storage->last_peer_addr, &meta->last_peer_addr);
+	memcpy(storage->display_name, meta->display_name,
+	       sizeof(storage->display_name));
+}
+
+static int settings_set(const char *key, size_t len_rd,
+			settings_read_cb read_cb, void *cb_arg)
+{
+	ssize_t rc;
+
+	if (!strcmp(key, SETTINGS_KEY_CURRENT_SLOT)) {
+		uint8_t stored_slot;
+
+		if (len_rd != sizeof(stored_slot)) {
+			return 0;
+		}
+
+		rc = read_cb(cb_arg, &stored_slot, sizeof(stored_slot));
+		if (rc == sizeof(stored_slot) && is_ble_slot(stored_slot)) {
+			ctx.current_slot = stored_slot;
+			ctx.pending_slot = stored_slot;
+			ctx.current_slot_valid = true;
+		}
+
+		return 0;
+	}
+
+	if (!strncmp(key, SETTINGS_KEY_META_PREFIX,
+		     sizeof(SETTINGS_KEY_META_PREFIX) - 1)) {
+		const char *slot_str = key + (sizeof(SETTINGS_KEY_META_PREFIX) - 1);
+		long slot = strtol(slot_str, NULL, 10);
+		struct ble_bond_multi_slot_meta_storage storage;
+
+		if ((slot < BLE_SLOT_MIN) || (slot > IDENTITY_DONGLE) ||
+		    (len_rd != sizeof(storage))) {
+			return 0;
+		}
+
+		rc = read_cb(cb_arg, &storage, sizeof(storage));
+		if (rc == sizeof(storage)) {
+			slot_meta_from_storage((uint8_t)slot, &storage);
+		}
+
+		return 0;
+	}
+
+	return 0;
+}
+
+SETTINGS_STATIC_HANDLER_DEFINE(ble_bond_multi, "ble_multi", NULL, settings_set,
+			       NULL, NULL);
+
+static int settings_save_current_slot(void)
+{
+	int err;
+
+	err = settings_save_one("ble_multi/" SETTINGS_KEY_CURRENT_SLOT,
+				&ctx.current_slot,
+				sizeof(ctx.current_slot));
+	if (err) {
+		LOG_ERR("Save current slot failed (%d)", err);
+	}
+
+	return err;
+}
+
+static int settings_save_slot_meta(uint8_t slot)
+{
+	int err;
+	char key[24];
+	struct ble_bond_multi_slot_meta_storage storage;
+
+	slot_meta_to_storage(slot, &storage);
+	snprintk(key, sizeof(key), "ble_multi/" SETTINGS_KEY_META_PREFIX "%u", slot);
+	err = settings_save_one(key, &storage, sizeof(storage));
+	if (err) {
+		LOG_ERR("Save slot %u meta failed (%d)", slot, err);
+	}
+
+	return err;
+}
+
+static void active_conn_clear(void)
+{
+	ctx.active_conn = NULL;
+}
+
+static void bond_addr_get_cb(const struct bt_bond_info *info, void *user_data)
+{
+	bt_addr_le_t *addr = user_data;
+
+	if (!bt_addr_le_cmp(addr, BT_ADDR_LE_ANY)) {
+		bt_addr_le_copy(addr, &info->addr);
+	}
+}
+
+static void publish_state(enum ble_bond_multi_op op)
+{
+	struct ble_bond_multi_event *event = new_ble_bond_multi_event();
+	uint8_t occ = 0U;
+
+	event->current_slot = ctx.current_slot;
+	event->active_identity_id = ctx.current_slot;
+	event->op = op;
+
+	for (uint8_t slot = BLE_SLOT_MIN; slot <= BLE_SLOT_MAX; slot++) {
+		if (ctx.slot_meta[slot].occupied) {
+			occ |= BIT(slot - BLE_SLOT_MIN);
+		}
+
+		event->slots[slot - BLE_SLOT_MIN] = ctx.slot_meta[slot];
+	}
+
+	event->slot_occupied_bitmap = occ;
+	APP_EVENT_SUBMIT(event);
+}
+
+static int identity_ensure_exists(uint8_t identity)
+{
+	size_t count = CONFIG_BT_ID_MAX;
+	bt_addr_le_t addrs[CONFIG_BT_ID_MAX];
+
+	(void)bt_id_get(addrs, &count);
+
+	while (count <= identity) {
+		int id = bt_id_create(NULL, NULL);
+
+		if (id < 0) {
+			LOG_ERR("bt_id_create failed (%d)", id);
+			return id;
+		}
+
+		count++;
+	}
+
+	return 0;
+}
+
+static bool slot_has_bond(uint8_t slot)
+{
+	size_t cnt = 0;
+
+	bt_foreach_bond(slot, slot_bond_cnt_cb, &cnt);
+
+	return cnt > 0U;
+}
+
+static void slot_bond_cnt_cb(const struct bt_bond_info *info, void *user_data)
+{
+	size_t *count = user_data;
+
+	ARG_UNUSED(info);
+	(*count)++;
+}
+
+static void slot_update_from_bonds(uint8_t slot)
+{
+	struct ble_bond_multi_slot_meta *meta = &ctx.slot_meta[slot];
+
+	meta->occupied = slot_has_bond(slot);
+	if (!meta->occupied) {
+		bt_addr_le_copy(&meta->last_peer_addr, BT_ADDR_LE_ANY);
+	} else if (!bt_addr_le_cmp(&meta->last_peer_addr, BT_ADDR_LE_ANY)) {
+		bt_foreach_bond(slot, bond_addr_get_cb, &meta->last_peer_addr);
+	}
+
+	if (meta->occupied &&
+	    ((meta->display_name[0] == '\0') ||
+	     !strcmp(meta->display_name, DEFAULT_DISPLAY_NAME_EMPTY))) {
+		display_name_set_addr(slot, &meta->last_peer_addr);
+	}
+
+	slot_meta_ensure_name(slot);
+}
+
+static void all_slots_refresh_from_bonds(void)
+{
+	for (uint8_t slot = BLE_SLOT_MIN; slot <= IDENTITY_DONGLE; slot++) {
+		slot_update_from_bonds(slot);
+	}
+}
+
+static void submit_selected_event(uint8_t identity)
+{
+	struct ble_peer_operation_event *event = new_ble_peer_operation_event();
+
+	event->op = PEER_OPERATION_SELECTED;
+	event->bt_app_id = identity;
+	event->bt_stack_id = identity;
+	APP_EVENT_SUBMIT(event);
+}
+
+static void submit_erased_event(uint8_t identity)
+{
+	struct ble_peer_operation_event *event = new_ble_peer_operation_event();
+
+	event->op = PEER_OPERATION_ERASED;
+	event->bt_app_id = identity;
+	event->bt_stack_id = identity;
+	APP_EVENT_SUBMIT(event);
+}
+
+static int switch_slot(uint8_t slot)
+{
+	if (!is_ble_slot(slot) || (slot == ctx.current_slot)) {
+		return 0;
+	}
+
+	ctx.pending_slot = slot;
+	ctx.current_slot = slot;
+	submit_selected_event(slot);
+	return settings_save_current_slot();
+}
+
+static int erase_slot(uint8_t slot)
+{
+	int err;
+
+	err = bt_unpair(slot, BT_ADDR_LE_ANY);
+	if (err) {
+		LOG_ERR("bt_unpair slot %u failed (%d)", slot, err);
+		return err;
+	}
+
+	slot_meta_clear(slot);
+	err = settings_save_slot_meta(slot);
+	if (err) {
+		return err;
+	}
+
+	submit_erased_event(slot);
+	submit_selected_event(slot);
+	return 0;
+}
+
+static int do_init(void)
+{
+	for (uint8_t slot = BLE_SLOT_MIN; slot <= IDENTITY_DONGLE; slot++) {
+		slot_meta_clear(slot);
+	}
+
+	if (!ctx.current_slot_valid) {
+		ctx.current_slot = BLE_SLOT_MIN;
+		ctx.pending_slot = BLE_SLOT_MIN;
+	}
+
+	active_conn_clear();
+
+	return 0;
+}
+
+static int do_start(void)
+{
+	int err;
+
+	for (uint8_t identity = BLE_SLOT_MIN; identity <= IDENTITY_DONGLE;
+	     identity++) {
+		err = identity_ensure_exists(identity);
+		if (err) {
+			return err;
+		}
+	}
+	ctx.identities_ready = true;
+
+	all_slots_refresh_from_bonds();
+	for (uint8_t slot = BLE_SLOT_MIN; slot <= IDENTITY_DONGLE; slot++) {
+		(void)settings_save_slot_meta(slot);
+	}
+
+	submit_selected_event(ctx.current_slot);
+	publish_state(BLE_BOND_MULTI_OP_REFRESH);
+
+	return 0;
+}
+
+static int do_stop(void)
+{
+	active_conn_clear();
+	return 0;
+}
+
+static bool handle_ble_peer_event(const struct ble_peer_event *event)
+{
+	switch (event->state) {
+	case PEER_STATE_CONNECTED:
+		ctx.active_conn = event->id;
+		return false;
+
+	case PEER_STATE_SECURED:
+		if (ctx.active_conn != event->id) {
+			return false;
+		}
+
+		ctx.slot_meta[ctx.current_slot].occupied = true;
+		bt_addr_le_copy(&ctx.slot_meta[ctx.current_slot].last_peer_addr,
+				bt_conn_get_dst(ctx.active_conn));
+		if ((ctx.slot_meta[ctx.current_slot].display_name[0] == '\0') ||
+		    !strcmp(ctx.slot_meta[ctx.current_slot].display_name,
+			    DEFAULT_DISPLAY_NAME_EMPTY)) {
+			display_name_set_addr(ctx.current_slot,
+					      &ctx.slot_meta[ctx.current_slot]
+						       .last_peer_addr);
+		}
+
+		(void)settings_save_slot_meta(ctx.current_slot);
+		publish_state(BLE_BOND_MULTI_OP_REFRESH);
+		return false;
+
+	case PEER_STATE_DISCONNECTED:
+	case PEER_STATE_CONN_FAILED:
+		if (ctx.active_conn == event->id) {
+			active_conn_clear();
+		}
+		return false;
+
+	default:
+		return false;
+	}
+}
+
+static bool app_event_handler(const struct app_event_header *aeh)
+{
+	if (is_module_state_event(aeh)) {
+		const struct module_state_event *event = cast_module_state_event(aeh);
+
+		if (check_state(event, MODULE_ID(settings_loader), MODULE_STATE_READY)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
+			return false;
+		}
+
+		return false;
+	}
+
+	if (is_ble_peer_event(aeh)) {
+		return handle_ble_peer_event(cast_ble_peer_event(aeh));
+	}
+
+	if (is_ble_bond_multi_event(aeh)) {
+		return false;
+	}
+
+	return false;
+}
+
+int ble_bond_multi_select_slot(uint8_t slot)
+{
+	int err;
+
+	if (!module_lifecycle_is_running(&ctx.lc)) {
+		return -EAGAIN;
+	}
+
+	if (!ctx.identities_ready) {
+		return -EAGAIN;
+	}
+
+	err = switch_slot(slot);
+	if (!err) {
+		publish_state(BLE_BOND_MULTI_OP_SWITCH);
+	}
+
+	return err;
+}
+
+int ble_bond_multi_erase_current_slot(void)
+{
+	int err;
+
+	if (!module_lifecycle_is_running(&ctx.lc)) {
+		return -EAGAIN;
+	}
+
+	if (!ctx.identities_ready) {
+		return -EAGAIN;
+	}
+
+	err = erase_slot(ctx.current_slot);
+	if (!err) {
+		publish_state(BLE_BOND_MULTI_OP_ERASE);
+	}
+
+	return err;
+}
+
+const struct ble_bond_multi_slot_meta *ble_bond_multi_get_slot_meta(uint8_t slot)
+{
+	if (!is_ble_slot(slot)) {
+		return NULL;
+	}
+
+	return &ctx.slot_meta[slot];
+}
+
+uint8_t ble_bond_multi_get_current_slot(void)
+{
+	return ctx.current_slot;
+}
+
+APP_EVENT_LISTENER(MODULE, app_event_handler);
+APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
+APP_EVENT_SUBSCRIBE_EARLY(MODULE, ble_peer_event);

src/ble_hid_module.c

@@ -7,7 +7,6 @@
 
 #define MODULE ble_hid_module
 #include <caf/events/module_state_event.h>
-#include <caf/events/power_event.h>
 
 #include <caf/events/ble_common_event.h>
 #include <bluetooth/services/hids.h>
@@ -57,8 +56,8 @@ static int do_start(void);
 static int do_stop(void);
 
 static const struct module_lifecycle_cfg lifecycle_cfg = {
-	.mode = ML_MODE_POWER,
-	.stopped_state = MODULE_STATE_STANDBY,
+	.mode = ML_MODE_NONE,
+	.stopped_state = MODULE_STATE_OFF,
 };
 
 static const struct module_lifecycle_ops lifecycle_ops = {
@@ -446,22 +445,6 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		return false;
 	}
 
-	if (is_power_down_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)module_set_lifecycle(&ctx.lc, LC_STOPPED);
-		}
-
-		return false;
-	}
-
-	if (is_wake_up_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
-		}
-
-		return false;
-	}
-
 	return false;
 }
 
@@ -469,5 +452,3 @@ APP_EVENT_LISTENER(MODULE, app_event_handler);
 APP_EVENT_SUBSCRIBE_EARLY(MODULE, module_state_event);
 APP_EVENT_SUBSCRIBE(MODULE, hid_tx_report_event);
 APP_EVENT_SUBSCRIBE_EARLY(MODULE, ble_peer_event);
-APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
-APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/ble_nus_module.c

@@ -6,7 +6,6 @@
 
 #define MODULE ble_nus_module
 #include <caf/events/module_state_event.h>
-#include <caf/events/power_event.h>
 
 #include <caf/events/ble_common_event.h>
 #include <zephyr/bluetooth/conn.h>
@@ -38,8 +37,8 @@ static int do_start(void);
 static int do_stop(void);
 
 static const struct module_lifecycle_cfg lifecycle_cfg = {
-	.mode = ML_MODE_POWER,
-	.stopped_state = MODULE_STATE_STANDBY,
+	.mode = ML_MODE_NONE,
+	.stopped_state = MODULE_STATE_OFF,
 };
 
 static const struct module_lifecycle_ops lifecycle_ops = {
@@ -58,24 +57,6 @@ static struct ble_nus_ctx ctx = {
 	.active_conn = NULL,
 };
 
-static const char *lifecycle_name(enum module_lifecycle state)
-{
-	switch (state) {
-	case LC_UNINIT:
-		return "UNINIT";
-	case LC_RUNNING:
-		return "RUNNING";
-	case LC_STOPPED:
-		return "STOPPED";
-	case LC_SUSPENDED:
-		return "SUSPENDED";
-	case LC_ERROR:
-		return "ERROR";
-	default:
-		return "?";
-	}
-}
-
 static const char *business_state_name(enum ble_nus_business_state state)
 {
 	switch (state) {
@@ -186,13 +167,18 @@ static void received(struct bt_conn *conn, const void *data, uint16_t len,
 	if (!lifecycle_is_ready() || (ctx.business == BLE_NUS_STACK_OFFLINE) ||
 	    (conn != ctx.active_conn)) {
 		LOG_WRN("BLE NUS drop RX len:%u lc:%s business:%s active_conn:%p conn:%p link:%s",
-			len, lifecycle_name(ctx.lc.state),
+			len, module_lifecycle_name(ctx.lc.state),
 			business_state_name(ctx.business),
 			(void *)ctx.active_conn, (void *)conn,
 			link_state_name(transport_link_state_get()));
 		return;
 	}
 
+	if ((len < PROTO_FRAME_HEADER_SIZE) || (len > PROTO_MAX_FRAME_LEN)) {
+		LOG_WRN("BLE NUS drop invalid framed RX len:%u", len);
+		return;
+	}
+
 	(void)submit_proto_rx_event(PROTO_TRANSPORT_BLE_NUS, data, len);
 }
 
@@ -246,7 +232,7 @@ static int apply_lifecycle(enum module_lifecycle target)
 		state_reconcile(old_lifecycle, old_business);
 	} else {
 		LOG_WRN("BLE NUS lifecycle change failed target:%s err:%d",
-			lifecycle_name(target), err);
+			module_lifecycle_name(target), err);
 	}
 
 	return err;
@@ -329,22 +315,6 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		return false;
 	}
 
-	if (is_power_down_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)apply_lifecycle(LC_STOPPED);
-		}
-
-		return false;
-	}
-
-	if (is_wake_up_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)apply_lifecycle(LC_RUNNING);
-		}
-
-		return false;
-	}
-
 	return false;
 }
 
@@ -352,5 +322,3 @@ APP_EVENT_LISTENER(MODULE, app_event_handler);
 APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
 APP_EVENT_SUBSCRIBE(MODULE, proto_tx_event);
 APP_EVENT_SUBSCRIBE_EARLY(MODULE, ble_peer_event);
-APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
-APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/display_module.c

@@ -5,6 +5,7 @@
 
 #define MODULE display_module
 #include <caf/events/module_state_event.h>
+#include <caf/events/ble_common_event.h>
 #include <caf/events/power_event.h>
 
 #include <lvgl_zephyr.h>
@@ -14,13 +15,18 @@
 #include <zephyr/logging/log.h>
 
 #include "bat_state_event.h"
+#include "ble_bond_multi_event.h"
 #include "datetime_event.h"
 #include "hid_led_event.h"
 #include "module_lifecycle.h"
 #include "mode_switch_event.h"
+#include "settings_mode_event.h"
+#include "settings_view_event.h"
 #include "theme_rgb_update_event.h"
 #include "theme_color.h"
+#include "ui/ui_page.h"
 #include "ui/ui_main.h"
+#include "ui/ui_settings.h"
 
 LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
 
@@ -34,6 +40,8 @@ struct display_module_ctx {
 	const struct device *backlight_dev;
 	uint32_t backlight_idx;
 	struct ui_main_model ui_model;
+	struct ui_settings_page *settings_page;
+	bool settings_active;
 	bool lvgl_initialized;
 	char date_text[DATETIME_EVENT_DATE_TEXT_LEN];
 	char time_text[DATETIME_EVENT_TIME_TEXT_LEN];
@@ -69,11 +77,17 @@ static struct display_module_ctx ctx = {
 					     BLINKY_THEME_DEFAULT_B),
 		.inactive_border_color = LV_COLOR_MAKE(0x3A, 0x44, 0x52),
 		.mode = MODE_SWITCH_BLE,
+		.ble_link_state = UI_BLE_LINK_HIDDEN,
 	},
 	.date_text = "1970/01/01",
 	.time_text = "00:00:00",
 };
 
+static struct ui_page *main_page(void)
+{
+	return ui_main_page_get(&ctx.ui_model, ctx.date_text, ctx.time_text);
+}
+
 static int backlight_set(bool on)
 {
 	if (on) {
@@ -126,10 +140,16 @@ static int do_start(void)
 		ctx.lvgl_initialized = true;
 
 		lvgl_lock();
-		ui_main_init(&ctx.ui_model, ctx.date_text, ctx.time_text);
+		ui_page_init(main_page());
 		lvgl_unlock();
 	}
 
+	lvgl_lock();
+	if (!ctx.settings_active) {
+		ui_page_init(main_page());
+	}
+	lvgl_unlock();
+
 	err = backlight_set(true);
 	if (err) {
 		LOG_ERR("Backlight enable failed (%d)", err);
@@ -168,7 +188,11 @@ static void refresh_ui(void)
 	}
 
 	lvgl_lock();
-	ui_main_refresh_all(&ctx.ui_model, ctx.date_text, ctx.time_text);
+	if (ctx.settings_active && (ctx.settings_page != NULL)) {
+		ui_settings_show(ctx.settings_page, true);
+	} else {
+		ui_page_refresh(main_page());
+	}
 	lvgl_unlock();
 }
 
@@ -188,6 +212,53 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		const struct mode_switch_event *event = cast_mode_switch_event(aeh);
 
 		ctx.ui_model.mode = event->mode;
+		if (ctx.ui_model.mode != MODE_SWITCH_BLE) {
+			ctx.ui_model.ble_link_state = UI_BLE_LINK_HIDDEN;
+		}
+		refresh_ui();
+		return false;
+	}
+
+	if (is_ble_bond_multi_event(aeh)) {
+		refresh_ui();
+		return false;
+	}
+
+	if (is_ble_peer_search_event(aeh)) {
+		const struct ble_peer_search_event *event =
+			cast_ble_peer_search_event(aeh);
+
+		if (ctx.ui_model.mode == MODE_SWITCH_BLE) {
+			ctx.ui_model.ble_link_state = event->active ?
+				UI_BLE_LINK_SEARCHING : UI_BLE_LINK_HIDDEN;
+			refresh_ui();
+		}
+
+		return false;
+	}
+
+	if (is_ble_peer_event(aeh)) {
+		const struct ble_peer_event *event = cast_ble_peer_event(aeh);
+
+		if (ctx.ui_model.mode != MODE_SWITCH_BLE) {
+			return false;
+		}
+
+		switch (event->state) {
+		case PEER_STATE_CONNECTED:
+		case PEER_STATE_SECURED:
+			ctx.ui_model.ble_link_state = UI_BLE_LINK_CONNECTED;
+			break;
+
+		case PEER_STATE_DISCONNECTED:
+		case PEER_STATE_CONN_FAILED:
+			ctx.ui_model.ble_link_state = UI_BLE_LINK_SEARCHING;
+			break;
+
+		default:
+			return false;
+		}
+
 		refresh_ui();
 		return false;
 	}
@@ -211,6 +282,48 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		return false;
 	}
 
+	if (is_settings_mode_event(aeh)) {
+		const struct settings_mode_event *event =
+			cast_settings_mode_event(aeh);
+
+		ctx.settings_active = event->active;
+		if (!ctx.settings_active) {
+			ctx.settings_page = NULL;
+		}
+		if (!ctx.lvgl_initialized) {
+			return false;
+		}
+
+		lvgl_lock();
+		if (ctx.settings_active) {
+			ui_page_deinit(main_page());
+			ui_settings_init();
+			if (ctx.settings_page != NULL) {
+				ui_settings_show(ctx.settings_page, false);
+			}
+		} else {
+			ui_settings_deinit();
+			ui_page_init(main_page());
+			ui_page_refresh(main_page());
+		}
+		lvgl_unlock();
+		return false;
+	}
+
+	if (is_settings_view_event(aeh)) {
+		const struct settings_view_event *event =
+			cast_settings_view_event(aeh);
+
+		ctx.settings_page = event->page;
+		if (ctx.settings_active && ctx.lvgl_initialized) {
+			lvgl_lock();
+			ui_settings_show(ctx.settings_page, event->animate);
+			lvgl_unlock();
+		}
+
+		return false;
+	}
+
 	if (is_datetime_event(aeh)) {
 		const struct datetime_event *event = cast_datetime_event(aeh);
 
@@ -253,10 +366,15 @@ static bool app_event_handler(const struct app_event_header *aeh)
 
 APP_EVENT_LISTENER(MODULE, app_event_handler);
 APP_EVENT_SUBSCRIBE(MODULE, bat_state_event);
+APP_EVENT_SUBSCRIBE(MODULE, ble_bond_multi_event);
+APP_EVENT_SUBSCRIBE(MODULE, ble_peer_event);
+APP_EVENT_SUBSCRIBE(MODULE, ble_peer_search_event);
 APP_EVENT_SUBSCRIBE(MODULE, datetime_event);
 APP_EVENT_SUBSCRIBE(MODULE, hid_led_event);
 APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
 APP_EVENT_SUBSCRIBE(MODULE, mode_switch_event);
+APP_EVENT_SUBSCRIBE(MODULE, settings_mode_event);
+APP_EVENT_SUBSCRIBE(MODULE, settings_view_event);
 APP_EVENT_SUBSCRIBE(MODULE, theme_rgb_update_event);
 APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
 APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/events/ble_bond_multi_event.c

@@ -0,0 +1,38 @@
+#include "ble_bond_multi_event.h"
+
+static void log_ble_bond_multi_event(const struct app_event_header *aeh)
+{
+	const struct ble_bond_multi_event *event =
+		cast_ble_bond_multi_event(aeh);
+
+	APP_EVENT_MANAGER_LOG(aeh, "slot:%u identity:%u op:%u occ:0x%02x",
+			      event->current_slot, event->active_identity_id,
+			      event->op, event->slot_occupied_bitmap);
+}
+
+static void profile_ble_bond_multi_event(struct log_event_buf *buf,
+					 const struct app_event_header *aeh)
+{
+	const struct ble_bond_multi_event *event =
+		cast_ble_bond_multi_event(aeh);
+
+	nrf_profiler_log_encode_uint8(buf, event->current_slot);
+	nrf_profiler_log_encode_uint8(buf, event->active_identity_id);
+	nrf_profiler_log_encode_uint8(buf, event->op);
+	nrf_profiler_log_encode_uint8(buf, event->slot_occupied_bitmap);
+}
+
+APP_EVENT_INFO_DEFINE(ble_bond_multi_event,
+		      ENCODE(NRF_PROFILER_ARG_U8,
+			     NRF_PROFILER_ARG_U8,
+			     NRF_PROFILER_ARG_U8,
+			     NRF_PROFILER_ARG_U8),
+		      ENCODE("current_slot", "active_identity_id", "op",
+			     "slot_occupied_bitmap"),
+		      profile_ble_bond_multi_event);
+
+APP_EVENT_TYPE_DEFINE(ble_bond_multi_event,
+		      log_ble_bond_multi_event,
+		      &ble_bond_multi_event_info,
+		      APP_EVENT_FLAGS_CREATE(
+			      APP_EVENT_TYPE_FLAGS_INIT_LOG_ENABLE));

src/events/settings_mode_event.c

@@ -0,0 +1,29 @@
+#include "settings_mode_event.h"
+
+static void log_settings_mode_event(const struct app_event_header *aeh)
+{
+	const struct settings_mode_event *event =
+		cast_settings_mode_event(aeh);
+
+	APP_EVENT_MANAGER_LOG(aeh, "active:%u", event->active);
+}
+
+static void profile_settings_mode_event(struct log_event_buf *buf,
+					const struct app_event_header *aeh)
+{
+	const struct settings_mode_event *event =
+		cast_settings_mode_event(aeh);
+
+	nrf_profiler_log_encode_uint8(buf, event->active ? 1U : 0U);
+}
+
+APP_EVENT_INFO_DEFINE(settings_mode_event,
+		      ENCODE(NRF_PROFILER_ARG_U8),
+		      ENCODE("active"),
+		      profile_settings_mode_event);
+
+APP_EVENT_TYPE_DEFINE(settings_mode_event,
+		      log_settings_mode_event,
+		      &settings_mode_event_info,
+		      APP_EVENT_FLAGS_CREATE(
+			      APP_EVENT_TYPE_FLAGS_INIT_LOG_ENABLE));

src/events/settings_view_event.c

@@ -0,0 +1,33 @@
+#include <stdint.h>
+
+#include "settings_view_event.h"
+
+static void log_settings_view_event(const struct app_event_header *aeh)
+{
+	const struct settings_view_event *event =
+		cast_settings_view_event(aeh);
+
+	APP_EVENT_MANAGER_LOG(aeh, "page:%p animate:%u",
+			      event->page, event->animate);
+}
+
+static void profile_settings_view_event(struct log_event_buf *buf,
+					const struct app_event_header *aeh)
+{
+	const struct settings_view_event *event =
+		cast_settings_view_event(aeh);
+
+	nrf_profiler_log_encode_uint32(buf, (uint32_t)(uintptr_t)event->page);
+	nrf_profiler_log_encode_uint8(buf, event->animate ? 1U : 0U);
+}
+
+APP_EVENT_INFO_DEFINE(settings_view_event,
+		      ENCODE(NRF_PROFILER_ARG_U32, NRF_PROFILER_ARG_U8),
+		      ENCODE("page", "animate"),
+		      profile_settings_view_event);
+
+APP_EVENT_TYPE_DEFINE(settings_view_event,
+		      log_settings_view_event,
+		      &settings_view_event_info,
+		      APP_EVENT_FLAGS_CREATE(
+			      APP_EVENT_TYPE_FLAGS_INIT_LOG_ENABLE));

src/events/transport_policy_event.c

@@ -0,0 +1,78 @@
+#include "transport_policy_event.h"
+
+static const char *source_mode_name(enum mode_switch_mode mode)
+{
+	switch (mode) {
+	case MODE_SWITCH_USB:
+		return "USB";
+	case MODE_SWITCH_BLE:
+		return "BLE";
+	case MODE_SWITCH_24G:
+		return "24G";
+	default:
+		return "?";
+	}
+}
+
+static const char *hid_transport_policy_name(enum hid_transport_policy transport)
+{
+	switch (transport) {
+	case HID_TRANSPORT_POLICY_NONE:
+		return "none";
+	case HID_TRANSPORT_POLICY_USB:
+		return "usb";
+	case HID_TRANSPORT_POLICY_BLE:
+		return "ble";
+	default:
+		return "?";
+	}
+}
+
+static const char *ble_profile_policy_name(enum ble_profile_policy profile)
+{
+	switch (profile) {
+	case BLE_PROFILE_POLICY_NONE:
+		return "none";
+	case BLE_PROFILE_POLICY_GENERAL:
+		return "general";
+	case BLE_PROFILE_POLICY_DONGLE:
+		return "dongle";
+	default:
+		return "?";
+	}
+}
+
+static void log_transport_policy_event(const struct app_event_header *aeh)
+{
+	const struct transport_policy_event *event =
+		cast_transport_policy_event(aeh);
+
+	APP_EVENT_MANAGER_LOG(aeh, "source:%s hid:%s ble:%s",
+			      source_mode_name(event->source_mode),
+			      hid_transport_policy_name(event->hid_transport),
+			      ble_profile_policy_name(event->ble_profile));
+}
+
+static void profile_transport_policy_event(struct log_event_buf *buf,
+					   const struct app_event_header *aeh)
+{
+	const struct transport_policy_event *event =
+		cast_transport_policy_event(aeh);
+
+	nrf_profiler_log_encode_uint8(buf, event->source_mode);
+	nrf_profiler_log_encode_uint8(buf, event->hid_transport);
+	nrf_profiler_log_encode_uint8(buf, event->ble_profile);
+}
+
+APP_EVENT_INFO_DEFINE(transport_policy_event,
+		      ENCODE(NRF_PROFILER_ARG_U8, NRF_PROFILER_ARG_U8,
+			     NRF_PROFILER_ARG_U8),
+		      ENCODE("source_mode", "hid_transport", "ble_profile"),
+		      profile_transport_policy_event);
+
+APP_EVENT_TYPE_DEFINE(transport_policy_event,
+		      log_transport_policy_event,
+		      &transport_policy_event_info,
+		      APP_EVENT_FLAGS_CREATE(
+			      IF_ENABLED(CONFIG_BRIDGE_LOG_TRANSPORT_POLICY_EVENT,
+					 (APP_EVENT_TYPE_FLAGS_INIT_LOG_ENABLE))));

src/hid_flowctrl_module.c

@@ -16,7 +16,7 @@
 #include "keyboard_core.h"
 #include "keyboard_hid_report_event.h"
 #include "module_lifecycle.h"
-#include "mode_switch_event.h"
+#include "transport_policy_event.h"
 
 LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
 
@@ -60,7 +60,7 @@ struct hid_flowctrl_module_ctx {
 	uint8_t consumer_fifo_tail;
 	uint8_t consumer_fifo_count;
 	struct in_flight_report in_flight[HID_SEND_CH_COUNT];
-	enum mode_switch_mode current_mode;
+	enum hid_transport_policy current_transport;
 	uint16_t next_sequence;
 };
 
@@ -98,34 +98,21 @@ static struct hid_flowctrl_module_ctx ctx = {
 	},
 };
 
-#define lifecycle ctx.lc
-#define channel_state ctx.channel_state
-#define pending_keys ctx.pending_keys
-#define pending_consumer_latest ctx.pending_consumer_latest
-#define consumer_fifo ctx.consumer_fifo
-#define consumer_fifo_head ctx.consumer_fifo_head
-#define consumer_fifo_tail ctx.consumer_fifo_tail
-#define consumer_fifo_count ctx.consumer_fifo_count
-#define in_flight ctx.in_flight
-#define current_mode ctx.current_mode
-#define next_sequence ctx.next_sequence
-#define running module_lifecycle_is_running(&ctx.lc)
-
-static bool current_mode_to_channel(enum keyboard_report_type report_type,
-				    enum hid_send_channel *channel)
+static bool current_transport_to_channel(enum keyboard_report_type report_type,
+					 enum hid_send_channel *channel)
 {
 	if (channel == NULL) {
 		return false;
 	}
 
-	switch (current_mode) {
-	case MODE_SWITCH_USB:
+	switch (ctx.current_transport) {
+	case HID_TRANSPORT_POLICY_USB:
 		*channel = (report_type == KEYBOARD_REPORT_TYPE_KEYS) ?
 			   HID_SEND_CH_USB_KEYS :
 			   HID_SEND_CH_USB_CONSUMER;
 		return true;
 
-	case MODE_SWITCH_BLE:
+	case HID_TRANSPORT_POLICY_BLE:
 		*channel = HID_SEND_CH_BLE_SHARED;
 		return true;
 
@@ -136,44 +123,47 @@ static bool current_mode_to_channel(enum keyboard_report_type report_type,
 
 static void clear_pending_reports(void)
 {
-	memset(&pending_keys, 0, sizeof(pending_keys));
-	memset(&pending_consumer_latest, 0, sizeof(pending_consumer_latest));
-	consumer_fifo_head = 0U;
-	consumer_fifo_tail = 0U;
-	consumer_fifo_count = 0U;
-	memset(&in_flight, 0, sizeof(in_flight));
+	memset(&ctx.pending_keys, 0, sizeof(ctx.pending_keys));
+	memset(&ctx.pending_consumer_latest, 0, sizeof(ctx.pending_consumer_latest));
+	ctx.consumer_fifo_head = 0U;
+	ctx.consumer_fifo_tail = 0U;
+	ctx.consumer_fifo_count = 0U;
+	memset(&ctx.in_flight, 0, sizeof(ctx.in_flight));
 }
 
 static void consumer_fifo_push(enum keyboard_report_type report_type,
 			       enum keyboard_protocol_mode protocol_mode,
 			       const uint8_t *data, size_t size)
 {
-	if (consumer_fifo_count == HID_FLOWCTRL_FIFO_DEPTH) {
+	if (ctx.consumer_fifo_count == HID_FLOWCTRL_FIFO_DEPTH) {
 		LOG_WRN("Consumer FIFO full, dropping oldest pulse");
-		consumer_fifo_head = (consumer_fifo_head + 1U) % HID_FLOWCTRL_FIFO_DEPTH;
-		consumer_fifo_count--;
+		ctx.consumer_fifo_head =
+			(ctx.consumer_fifo_head + 1U) % HID_FLOWCTRL_FIFO_DEPTH;
+		ctx.consumer_fifo_count--;
 	}
 
-	struct queued_report *entry = &consumer_fifo[consumer_fifo_tail];
+	struct queued_report *entry = &ctx.consumer_fifo[ctx.consumer_fifo_tail];
 
 	entry->report_type = report_type;
 	entry->protocol_mode = protocol_mode;
 	entry->size = size;
 	memcpy(entry->data, data, size);
 
-	consumer_fifo_tail = (consumer_fifo_tail + 1U) % HID_FLOWCTRL_FIFO_DEPTH;
-	consumer_fifo_count++;
+	ctx.consumer_fifo_tail =
+		(ctx.consumer_fifo_tail + 1U) % HID_FLOWCTRL_FIFO_DEPTH;
+	ctx.consumer_fifo_count++;
 }
 
 static bool consumer_fifo_pop(struct queued_report *entry)
 {
-	if (consumer_fifo_count == 0U) {
+	if (ctx.consumer_fifo_count == 0U) {
 		return false;
 	}
 
-	*entry = consumer_fifo[consumer_fifo_head];
-	consumer_fifo_head = (consumer_fifo_head + 1U) % HID_FLOWCTRL_FIFO_DEPTH;
-	consumer_fifo_count--;
+	*entry = ctx.consumer_fifo[ctx.consumer_fifo_head];
+	ctx.consumer_fifo_head =
+		(ctx.consumer_fifo_head + 1U) % HID_FLOWCTRL_FIFO_DEPTH;
+	ctx.consumer_fifo_count--;
 
 	return true;
 }
@@ -182,9 +172,9 @@ static bool channel_can_send_report(enum hid_send_channel channel,
 				    enum keyboard_report_type report_type,
 				    enum keyboard_protocol_mode protocol_mode)
 {
-	const struct hid_channel_state_data *state = &channel_state[channel];
+	const struct hid_channel_state_data *state = &ctx.channel_state[channel];
 
-	if (in_flight[channel].active) {
+	if (ctx.in_flight[channel].active) {
 		return false;
 	}
 
@@ -205,28 +195,28 @@ static void try_send_keys(void)
 {
 	enum hid_send_channel channel;
 
-	if (!pending_keys.valid) {
+	if (!ctx.pending_keys.valid) {
 		return;
 	}
 
-	if (!current_mode_to_channel(KEYBOARD_REPORT_TYPE_KEYS, &channel)) {
+	if (!current_transport_to_channel(KEYBOARD_REPORT_TYPE_KEYS, &channel)) {
 		return;
 	}
 
-	if (!channel_can_send_report(channel, pending_keys.report_type,
-				     pending_keys.protocol_mode)) {
+	if (!channel_can_send_report(channel, ctx.pending_keys.report_type,
+				     ctx.pending_keys.protocol_mode)) {
 		return;
 	}
 
-	in_flight[channel].active = true;
-	in_flight[channel].channel = channel;
-	in_flight[channel].report_type = pending_keys.report_type;
-	in_flight[channel].sequence = next_sequence++;
-	(void)submit_hid_tx_report_event(channel, pending_keys.report_type,
-					 pending_keys.protocol_mode,
-					 in_flight[channel].sequence,
-					 pending_keys.data, pending_keys.size);
-	pending_keys.valid = false;
+	ctx.in_flight[channel].active = true;
+	ctx.in_flight[channel].channel = channel;
+	ctx.in_flight[channel].report_type = ctx.pending_keys.report_type;
+	ctx.in_flight[channel].sequence = ctx.next_sequence++;
+	(void)submit_hid_tx_report_event(channel, ctx.pending_keys.report_type,
+					 ctx.pending_keys.protocol_mode,
+					 ctx.in_flight[channel].sequence,
+					 ctx.pending_keys.data, ctx.pending_keys.size);
+	ctx.pending_keys.valid = false;
 }
 
 static void try_send_consumer_fifo(void)
@@ -234,11 +224,11 @@ static void try_send_consumer_fifo(void)
 	struct queued_report queued;
 	enum hid_send_channel channel;
 
-	if (consumer_fifo_count == 0U) {
+	if (ctx.consumer_fifo_count == 0U) {
 		return;
 	}
 
-	if (!current_mode_to_channel(KEYBOARD_REPORT_TYPE_CONSUMER, &channel)) {
+	if (!current_transport_to_channel(KEYBOARD_REPORT_TYPE_CONSUMER, &channel)) {
 		return;
 	}
 
@@ -248,20 +238,20 @@ static void try_send_consumer_fifo(void)
 
 	if (!channel_can_send_report(channel, queued.report_type,
 				     queued.protocol_mode)) {
-		if (queued.protocol_mode == channel_state[channel].protocol_mode) {
+		if (queued.protocol_mode == ctx.channel_state[channel].protocol_mode) {
 			consumer_fifo_push(queued.report_type, queued.protocol_mode,
 					   queued.data, queued.size);
 		}
 		return;
 	}
 
-	in_flight[channel].active = true;
-	in_flight[channel].channel = channel;
-	in_flight[channel].report_type = queued.report_type;
-	in_flight[channel].sequence = next_sequence++;
+	ctx.in_flight[channel].active = true;
+	ctx.in_flight[channel].channel = channel;
+	ctx.in_flight[channel].report_type = queued.report_type;
+	ctx.in_flight[channel].sequence = ctx.next_sequence++;
 	(void)submit_hid_tx_report_event(channel, queued.report_type,
 					 queued.protocol_mode,
-					 in_flight[channel].sequence,
+					 ctx.in_flight[channel].sequence,
 					 queued.data, queued.size);
 }
 
@@ -269,36 +259,37 @@ static void try_send_consumer_latest(void)
 {
 	enum hid_send_channel channel;
 
-	if (!pending_consumer_latest.valid) {
+	if (!ctx.pending_consumer_latest.valid) {
 		return;
 	}
 
-	if (!current_mode_to_channel(KEYBOARD_REPORT_TYPE_CONSUMER, &channel)) {
+	if (!current_transport_to_channel(KEYBOARD_REPORT_TYPE_CONSUMER, &channel)) {
 		return;
 	}
 
 	if (!channel_can_send_report(channel,
-				     pending_consumer_latest.report_type,
-				     pending_consumer_latest.protocol_mode)) {
+				     ctx.pending_consumer_latest.report_type,
+				     ctx.pending_consumer_latest.protocol_mode)) {
 		return;
 	}
 
-	in_flight[channel].active = true;
-	in_flight[channel].channel = channel;
-	in_flight[channel].report_type = pending_consumer_latest.report_type;
-	in_flight[channel].sequence = next_sequence++;
+	ctx.in_flight[channel].active = true;
+	ctx.in_flight[channel].channel = channel;
+	ctx.in_flight[channel].report_type =
+		ctx.pending_consumer_latest.report_type;
+	ctx.in_flight[channel].sequence = ctx.next_sequence++;
 	(void)submit_hid_tx_report_event(channel,
-					 pending_consumer_latest.report_type,
-					 pending_consumer_latest.protocol_mode,
-					 in_flight[channel].sequence,
-					 pending_consumer_latest.data,
-					 pending_consumer_latest.size);
-	pending_consumer_latest.valid = false;
+					 ctx.pending_consumer_latest.report_type,
+					 ctx.pending_consumer_latest.protocol_mode,
+					 ctx.in_flight[channel].sequence,
+					 ctx.pending_consumer_latest.data,
+					 ctx.pending_consumer_latest.size);
+	ctx.pending_consumer_latest.valid = false;
 }
 
 static void try_send_next(void)
 {
-	if (!running) {
+	if (!module_lifecycle_is_running(&ctx.lc)) {
 		return;
 	}
 
@@ -310,7 +301,7 @@ static void try_send_next(void)
 static bool handle_keyboard_hid_report_event(
 	const struct keyboard_hid_report_event *event)
 {
-	if (!running ||
+	if (!module_lifecycle_is_running(&ctx.lc) ||
 	    ((event->mode != MODE_SWITCH_USB) && (event->mode != MODE_SWITCH_BLE))) {
 		return false;
 	}
@@ -321,17 +312,18 @@ static bool handle_keyboard_hid_report_event(
 				   event->dyndata.data,
 				   event->dyndata.size);
 	} else if (event->report_type == KEYBOARD_REPORT_TYPE_KEYS) {
-		pending_keys.valid = true;
-		pending_keys.report_type = event->report_type;
-		pending_keys.protocol_mode = event->protocol_mode;
-		pending_keys.size = event->dyndata.size;
-		memcpy(pending_keys.data, event->dyndata.data, event->dyndata.size);
+		ctx.pending_keys.valid = true;
+		ctx.pending_keys.report_type = event->report_type;
+		ctx.pending_keys.protocol_mode = event->protocol_mode;
+		ctx.pending_keys.size = event->dyndata.size;
+		memcpy(ctx.pending_keys.data, event->dyndata.data,
+		       event->dyndata.size);
 	} else {
-		pending_consumer_latest.valid = true;
-		pending_consumer_latest.report_type = event->report_type;
-		pending_consumer_latest.protocol_mode = event->protocol_mode;
-		pending_consumer_latest.size = event->dyndata.size;
-		memcpy(pending_consumer_latest.data, event->dyndata.data,
+		ctx.pending_consumer_latest.valid = true;
+		ctx.pending_consumer_latest.report_type = event->report_type;
+		ctx.pending_consumer_latest.protocol_mode = event->protocol_mode;
+		ctx.pending_consumer_latest.size = event->dyndata.size;
+		memcpy(ctx.pending_consumer_latest.data, event->dyndata.data,
 		       event->dyndata.size);
 	}
 
@@ -346,11 +338,12 @@ static bool handle_hid_channel_state_event(
 		return false;
 	}
 
-	channel_state[event->channel].report_ready_bm = event->report_ready_bm;
-	channel_state[event->channel].protocol_mode = event->protocol_mode;
+	ctx.channel_state[event->channel].report_ready_bm =
+		event->report_ready_bm;
+	ctx.channel_state[event->channel].protocol_mode = event->protocol_mode;
 
 	if (event->report_ready_bm == 0U) {
-		in_flight[event->channel].active = false;
+		ctx.in_flight[event->channel].active = false;
 	}
 
 	try_send_next();
@@ -363,17 +356,17 @@ static bool handle_hid_report_sent_event(const struct hid_report_sent_event *eve
 		return false;
 	}
 
-	if (!in_flight[event->channel].active) {
+	if (!ctx.in_flight[event->channel].active) {
 		return false;
 	}
 
-	if (event->sequence != in_flight[event->channel].sequence) {
+	if (event->sequence != ctx.in_flight[event->channel].sequence) {
 		LOG_WRN("Unexpected HID sent sequence %u (expected %u)",
-			event->sequence, in_flight[event->channel].sequence);
+			event->sequence, ctx.in_flight[event->channel].sequence);
 		return false;
 	}
 
-	in_flight[event->channel].active = false;
+	ctx.in_flight[event->channel].active = false;
 
 	if (event->error) {
 		LOG_WRN("HID report send failed for seq %u", event->sequence);
@@ -383,14 +376,16 @@ static bool handle_hid_report_sent_event(const struct hid_report_sent_event *eve
 	return false;
 }
 
-static bool handle_mode_switch_event(const struct mode_switch_event *event)
+static bool handle_transport_policy_event(
+	const struct transport_policy_event *event)
 {
-	bool mode_changed = (current_mode != event->mode);
+	bool transport_changed =
+		(ctx.current_transport != event->hid_transport);
 
-	current_mode = event->mode;
+	ctx.current_transport = event->hid_transport;
 
-	if (mode_changed || ((current_mode != MODE_SWITCH_USB) &&
-			     (current_mode != MODE_SWITCH_BLE))) {
+	if (transport_changed ||
+	    (ctx.current_transport == HID_TRANSPORT_POLICY_NONE)) {
 		clear_pending_reports();
 	}
 
@@ -401,21 +396,21 @@ static bool handle_mode_switch_event(const struct mode_switch_event *event)
 static int do_init(void)
 {
 	clear_pending_reports();
-	current_mode = MODE_SWITCH_USB;
-	memset(channel_state, 0, sizeof(channel_state));
-	channel_state[HID_SEND_CH_USB_KEYS].protocol_mode =
+	ctx.current_transport = HID_TRANSPORT_POLICY_USB;
+	memset(ctx.channel_state, 0, sizeof(ctx.channel_state));
+	ctx.channel_state[HID_SEND_CH_USB_KEYS].protocol_mode =
 		KEYBOARD_PROTOCOL_MODE_REPORT;
-	channel_state[HID_SEND_CH_USB_CONSUMER].protocol_mode =
+	ctx.channel_state[HID_SEND_CH_USB_CONSUMER].protocol_mode =
 		KEYBOARD_PROTOCOL_MODE_REPORT;
-	channel_state[HID_SEND_CH_BLE_SHARED].protocol_mode =
+	ctx.channel_state[HID_SEND_CH_BLE_SHARED].protocol_mode =
 		KEYBOARD_PROTOCOL_MODE_REPORT;
-	next_sequence = 1U;
+	ctx.next_sequence = 1U;
 	return 0;
 }
 
 static int do_start(void)
 {
-	if (running) {
+	if (module_lifecycle_is_running(&ctx.lc)) {
 		return 0;
 	}
 
@@ -425,7 +420,7 @@ static int do_start(void)
 
 static int do_stop(void)
 {
-	if (!running) {
+	if (!module_lifecycle_is_running(&ctx.lc)) {
 		return 0;
 	}
 
@@ -451,8 +446,9 @@ static bool app_event_handler(const struct app_event_header *aeh)
 			cast_hid_report_sent_event(aeh));
 	}
 
-	if (is_mode_switch_event(aeh)) {
-		return handle_mode_switch_event(cast_mode_switch_event(aeh));
+	if (is_transport_policy_event(aeh)) {
+		return handle_transport_policy_event(
+			cast_transport_policy_event(aeh));
 	}
 
 	if (is_module_state_event(aeh)) {
@@ -460,23 +456,23 @@ static bool app_event_handler(const struct app_event_header *aeh)
 			cast_module_state_event(aeh);
 
 		if (check_state(event, MODULE_ID(main), MODULE_STATE_READY)) {
-			(void)module_set_lifecycle(&lifecycle, LC_RUNNING);
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
 		}
 
 		return false;
 	}
 
 	if (is_power_down_event(aeh)) {
-		if (module_lifecycle_is_initialized(&lifecycle)) {
-			(void)module_set_lifecycle(&lifecycle, LC_STOPPED);
+		if (module_lifecycle_is_initialized(&ctx.lc)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_STOPPED);
 		}
 
 		return false;
 	}
 
 	if (is_wake_up_event(aeh)) {
-		if (module_lifecycle_is_initialized(&lifecycle)) {
-			(void)module_set_lifecycle(&lifecycle, LC_RUNNING);
+		if (module_lifecycle_is_initialized(&ctx.lc)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
 		}
 
 		return false;
@@ -490,7 +486,7 @@ APP_EVENT_LISTENER(MODULE, app_event_handler);
 APP_EVENT_SUBSCRIBE(MODULE, keyboard_hid_report_event);
 APP_EVENT_SUBSCRIBE(MODULE, hid_channel_state_event);
 APP_EVENT_SUBSCRIBE(MODULE, hid_report_sent_event);
-APP_EVENT_SUBSCRIBE(MODULE, mode_switch_event);
+APP_EVENT_SUBSCRIBE(MODULE, transport_policy_event);
 APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
 APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
 APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/keyboard_core_module.c

@@ -20,8 +20,9 @@
 #include "keyboard_core.h"
 #include "keyboard_hid_report_event.h"
 #include "module_lifecycle.h"
-#include "mode_switch_event.h"
+#include "settings_mode_event.h"
 #include "set_protocol_event.h"
+#include "transport_policy_event.h"
 
 LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
 
@@ -92,8 +93,8 @@ struct keyboard_core_module_ctx {
 	struct keyboard_reports_cache reports_cache;
 	uint8_t function_usage_mask[KEYBOARD_PROTOCOL_BITMAP_BYTES];
 	enum keyboard_protocol_mode transport_protocol_modes[HID_TRANSPORT_COUNT];
-	enum mode_switch_mode current_mode;
-	bool mode_valid;
+	enum hid_transport_policy current_transport;
+	bool settings_active;
 };
 
 static int do_init(void);
@@ -123,23 +124,15 @@ static struct keyboard_core_module_ctx ctx = {
 	},
 };
 
-#define lifecycle ctx.lc
-#define keyboard_state ctx.keyboard_state
-#define reports_cache ctx.reports_cache
-#define function_usage_mask ctx.function_usage_mask
-#define transport_protocol_modes ctx.transport_protocol_modes
-#define current_mode ctx.current_mode
-#define mode_valid ctx.mode_valid
-#define running module_lifecycle_is_running(&ctx.lc)
-
-static bool mode_to_transport(enum mode_switch_mode mode, enum hid_transport *transport)
+static bool policy_to_transport(enum hid_transport_policy policy,
+				enum hid_transport *transport)
 {
-	switch (mode) {
-	case MODE_SWITCH_USB:
+	switch (policy) {
+	case HID_TRANSPORT_POLICY_USB:
 		*transport = HID_TRANSPORT_USB;
 		return true;
 
-	case MODE_SWITCH_BLE:
+	case HID_TRANSPORT_POLICY_BLE:
 		*transport = HID_TRANSPORT_BLE;
 		return true;
 
@@ -152,13 +145,32 @@ static enum keyboard_protocol_mode active_protocol_mode_get(void)
 {
 	enum hid_transport transport;
 
-	if (mode_valid && mode_to_transport(current_mode, &transport)) {
-		return transport_protocol_modes[transport];
+	if (policy_to_transport(ctx.current_transport, &transport)) {
+		return ctx.transport_protocol_modes[transport];
 	}
 
 	return KEYBOARD_PROTOCOL_MODE_REPORT;
 }
 
+static bool transport_policy_to_mode(enum hid_transport_policy policy,
+				     enum mode_switch_mode *mode)
+{
+	if (mode == NULL) {
+		return false;
+	}
+
+	switch (policy) {
+	case HID_TRANSPORT_POLICY_USB:
+		*mode = MODE_SWITCH_USB;
+		return true;
+	case HID_TRANSPORT_POLICY_BLE:
+		*mode = MODE_SWITCH_BLE;
+		return true;
+	default:
+		return false;
+	}
+}
+
 static const struct keymap_entry *keymap_get(uint16_t key_id)
 {
 	size_t left = 0;
@@ -246,38 +258,39 @@ static bool consumer_key_update(uint16_t consumer_id, bool pressed)
 		return false;
 	}
 
-	bool was_pressed = (keyboard_state.consumer_bits & BIT(consumer_id)) != 0U;
+	bool was_pressed =
+		(ctx.keyboard_state.consumer_bits & BIT(consumer_id)) != 0U;
 
 	if (was_pressed == pressed) {
 		return false;
 	}
 
-	WRITE_BIT(keyboard_state.consumer_bits, consumer_id, pressed);
+	WRITE_BIT(ctx.keyboard_state.consumer_bits, consumer_id, pressed);
 	return true;
 }
 
 static void keyboard_state_clear(void)
 {
-	memset(&keyboard_state, 0, sizeof(keyboard_state));
+	memset(&ctx.keyboard_state, 0, sizeof(ctx.keyboard_state));
 }
 
 static void function_usage_mask_clear(void)
 {
-	memset(function_usage_mask, 0, sizeof(function_usage_mask));
+	memset(ctx.function_usage_mask, 0, sizeof(ctx.function_usage_mask));
 }
 
 static void reports_cache_invalidate(void)
 {
-	reports_cache.boot_valid = false;
-	reports_cache.nkro_valid = false;
-	reports_cache.consumer_valid = false;
+	ctx.reports_cache.boot_valid = false;
+	ctx.reports_cache.nkro_valid = false;
+	ctx.reports_cache.consumer_valid = false;
 }
 
 static void build_effective_hid_bitmap(uint8_t bitmap[KEYBOARD_PROTOCOL_BITMAP_BYTES])
 {
 	for (size_t i = 0; i < KEYBOARD_PROTOCOL_BITMAP_BYTES; i++) {
-		bitmap[i] = keyboard_state.pressed_usage_bitmap[i] &
-			    (uint8_t)~keyboard_state.function_pressed_bitmap[i];
+		bitmap[i] = ctx.keyboard_state.pressed_usage_bitmap[i] &
+			    (uint8_t)~ctx.keyboard_state.function_pressed_bitmap[i];
 	}
 }
 
@@ -324,7 +337,7 @@ static void build_nkro_report(uint8_t report[KEYBOARD_NKRO_REPORT_SIZE])
 static uint16_t active_consumer_usage_get(void)
 {
 	for (uint8_t consumer_id = 0; consumer_id < KEYBOARD_CONSUMER_CTRL_COUNT; consumer_id++) {
-		if ((keyboard_state.consumer_bits & BIT(consumer_id)) != 0U) {
+		if ((ctx.keyboard_state.consumer_bits & BIT(consumer_id)) != 0U) {
 			return consumer_usage_map[consumer_id];
 		}
 	}
@@ -341,15 +354,18 @@ static void submit_consumer_fifo_frame(uint16_t usage_id)
 {
 	uint8_t report_buf[KEYBOARD_CONSUMER_REPORT_SIZE];
 	enum keyboard_protocol_mode protocol_mode = active_protocol_mode_get();
+	enum mode_switch_mode mode;
 
-	if (!running || !mode_valid ||
+	if (!module_lifecycle_is_running(&ctx.lc) ||
+	    ctx.settings_active ||
+	    !transport_policy_to_mode(ctx.current_transport, &mode) ||
 	    (protocol_mode == KEYBOARD_PROTOCOL_MODE_BOOT)) {
 		return;
 	}
 
 	sys_put_le16(usage_id, report_buf);
 	(void)submit_keyboard_hid_report_event(
-		current_mode, KEYBOARD_REPORT_TYPE_CONSUMER, protocol_mode,
+		mode, KEYBOARD_REPORT_TYPE_CONSUMER, protocol_mode,
 		HID_QUEUE_POLICY_FIFO, report_buf, KEYBOARD_CONSUMER_REPORT_SIZE);
 }
 
@@ -386,21 +402,26 @@ static void emit_keys_report(bool force)
 	uint8_t *cache_buf;
 	bool *cache_valid;
 	enum keyboard_protocol_mode protocol_mode = active_protocol_mode_get();
+	enum mode_switch_mode mode;
 
-	if (!mode_valid) {
+	if (!transport_policy_to_mode(ctx.current_transport, &mode)) {
+		return;
+	}
+
+	if (ctx.settings_active) {
 		return;
 	}
 
 	if (protocol_mode == KEYBOARD_PROTOCOL_MODE_BOOT) {
 		build_boot_report(report_buf);
 		report_size = KEYBOARD_BOOT_REPORT_SIZE;
-		cache_buf = reports_cache.boot_report;
-		cache_valid = &reports_cache.boot_valid;
+		cache_buf = ctx.reports_cache.boot_report;
+		cache_valid = &ctx.reports_cache.boot_valid;
 	} else {
 		build_nkro_report(report_buf);
 		report_size = KEYBOARD_NKRO_REPORT_SIZE;
-		cache_buf = reports_cache.nkro_report;
-		cache_valid = &reports_cache.nkro_valid;
+		cache_buf = ctx.reports_cache.nkro_report;
+		cache_valid = &ctx.reports_cache.nkro_valid;
 	}
 
 	if (!force && *cache_valid && (memcmp(cache_buf, report_buf, report_size) == 0)) {
@@ -411,7 +432,7 @@ static void emit_keys_report(bool force)
 	*cache_valid = true;
 
 	(void)submit_keyboard_hid_report_event(
-		current_mode, KEYBOARD_REPORT_TYPE_KEYS, protocol_mode,
+		mode, KEYBOARD_REPORT_TYPE_KEYS, protocol_mode,
 		HID_QUEUE_POLICY_LATEST, report_buf, report_size);
 }
 
@@ -419,23 +440,27 @@ static void emit_consumer_report(bool force)
 {
 	uint8_t report_buf[KEYBOARD_CONSUMER_REPORT_SIZE];
 	enum keyboard_protocol_mode protocol_mode = active_protocol_mode_get();
+	enum mode_switch_mode mode;
 
-	if (!mode_valid || (protocol_mode == KEYBOARD_PROTOCOL_MODE_BOOT)) {
+	if (!transport_policy_to_mode(ctx.current_transport, &mode) ||
+	    ctx.settings_active ||
+	    (protocol_mode == KEYBOARD_PROTOCOL_MODE_BOOT)) {
 		return;
 	}
 
 	build_consumer_report(report_buf);
-	if (!force && reports_cache.consumer_valid &&
-	    (memcmp(reports_cache.consumer_report, report_buf,
+	if (!force && ctx.reports_cache.consumer_valid &&
+	    (memcmp(ctx.reports_cache.consumer_report, report_buf,
 		    KEYBOARD_CONSUMER_REPORT_SIZE) == 0)) {
 		return;
 	}
 
-	memcpy(reports_cache.consumer_report, report_buf, KEYBOARD_CONSUMER_REPORT_SIZE);
-	reports_cache.consumer_valid = true;
+	memcpy(ctx.reports_cache.consumer_report, report_buf,
+	       KEYBOARD_CONSUMER_REPORT_SIZE);
+	ctx.reports_cache.consumer_valid = true;
 
 	(void)submit_keyboard_hid_report_event(
-		current_mode, KEYBOARD_REPORT_TYPE_CONSUMER, protocol_mode,
+		mode, KEYBOARD_REPORT_TYPE_CONSUMER, protocol_mode,
 		HID_QUEUE_POLICY_LATEST, report_buf,
 		KEYBOARD_CONSUMER_REPORT_SIZE);
 }
@@ -451,18 +476,31 @@ static void emit_all_reports(bool force)
 
 static void emit_function_state_event(void)
 {
-	(void)submit_function_bitmap_state_event(keyboard_state.pressed_usage_bitmap);
+	(void)submit_function_bitmap_state_event(
+		ctx.keyboard_state.pressed_usage_bitmap);
 }
 
-static void emit_release_reports(enum mode_switch_mode mode)
+static void emit_release_reports(enum hid_transport_policy transport_policy)
 {
 	uint8_t keys_report[KEYBOARD_NKRO_REPORT_SIZE] = { 0 };
 	uint8_t consumer_report[KEYBOARD_CONSUMER_REPORT_SIZE] = { 0 };
 	enum keyboard_protocol_mode protocol_mode = active_protocol_mode_get();
+	enum mode_switch_mode mode;
 	size_t keys_report_size =
 		(protocol_mode == KEYBOARD_PROTOCOL_MODE_BOOT) ?
 			KEYBOARD_BOOT_REPORT_SIZE : KEYBOARD_NKRO_REPORT_SIZE;
 
+	switch (transport_policy) {
+	case HID_TRANSPORT_POLICY_USB:
+		mode = MODE_SWITCH_USB;
+		break;
+	case HID_TRANSPORT_POLICY_BLE:
+		mode = MODE_SWITCH_BLE;
+		break;
+	default:
+		return;
+	}
+
 	(void)submit_keyboard_hid_report_event(
 		mode, KEYBOARD_REPORT_TYPE_KEYS, protocol_mode,
 		HID_QUEUE_POLICY_LATEST, keys_report, keys_report_size);
@@ -480,10 +518,11 @@ static int do_init(void)
 	keyboard_state_clear();
 	reports_cache_invalidate();
 	function_usage_mask_clear();
-	mode_valid = false;
-	transport_protocol_modes[HID_TRANSPORT_USB] =
+	ctx.settings_active = false;
+	ctx.current_transport = HID_TRANSPORT_POLICY_USB;
+	ctx.transport_protocol_modes[HID_TRANSPORT_USB] =
 		KEYBOARD_PROTOCOL_MODE_REPORT;
-	transport_protocol_modes[HID_TRANSPORT_BLE] =
+	ctx.transport_protocol_modes[HID_TRANSPORT_BLE] =
 		KEYBOARD_PROTOCOL_MODE_REPORT;
 
 	return 0;
@@ -491,7 +530,7 @@ static int do_init(void)
 
 static int do_start(void)
 {
-	if (running) {
+	if (module_lifecycle_is_running(&ctx.lc)) {
 		return 0;
 	}
 
@@ -500,18 +539,17 @@ static int do_start(void)
 
 static int do_stop(void)
 {
-	if (!running) {
+	if (!module_lifecycle_is_running(&ctx.lc)) {
 		return 0;
 	}
 
-	if (mode_valid) {
-		emit_release_reports(current_mode);
+	if (ctx.current_transport != HID_TRANSPORT_POLICY_NONE) {
+		emit_release_reports(ctx.current_transport);
 	}
 	emit_function_state_event();
 
 	keyboard_state_clear();
 	reports_cache_invalidate();
-	mode_valid = false;
 
 	return 0;
 }
@@ -521,7 +559,11 @@ static bool handle_button_event(const struct button_event *event)
 	const struct keymap_entry *entry;
 	bool changed;
 
-	if (!running) {
+	if (!module_lifecycle_is_running(&ctx.lc)) {
+		return false;
+	}
+
+	if (ctx.settings_active) {
 		return false;
 	}
 
@@ -534,7 +576,7 @@ static bool handle_button_event(const struct button_event *event)
 	if (entry->usage_type == KEY_USAGE_TYPE_KEYBOARD) {
 		bool routed_to_function;
 
-		changed = usage_bitmap_write(keyboard_state.pressed_usage_bitmap,
+		changed = usage_bitmap_write(ctx.keyboard_state.pressed_usage_bitmap,
 						entry->usage_id, event->pressed);
 		if (!changed) {
 			return false;
@@ -542,14 +584,14 @@ static bool handle_button_event(const struct button_event *event)
 
 		if (event->pressed) {
 			routed_to_function =
-				usage_bitmap_test(function_usage_mask, entry->usage_id);
-			(void)usage_bitmap_write(keyboard_state.function_pressed_bitmap,
+				usage_bitmap_test(ctx.function_usage_mask, entry->usage_id);
+			(void)usage_bitmap_write(ctx.keyboard_state.function_pressed_bitmap,
 						 entry->usage_id, routed_to_function);
 		} else {
 			routed_to_function =
-				usage_bitmap_test(keyboard_state.function_pressed_bitmap,
+				usage_bitmap_test(ctx.keyboard_state.function_pressed_bitmap,
 						  entry->usage_id);
-			(void)usage_bitmap_write(keyboard_state.function_pressed_bitmap,
+			(void)usage_bitmap_write(ctx.keyboard_state.function_pressed_bitmap,
 						 entry->usage_id, false);
 		}
 
@@ -568,33 +610,42 @@ static bool handle_button_event(const struct button_event *event)
 	return false;
 }
 
-static bool handle_mode_switch_event(const struct mode_switch_event *event)
+static bool handle_transport_policy_event(
+	const struct transport_policy_event *event)
 {
-	bool mode_changed;
+	bool transport_changed;
 
-	if (!running) {
-		current_mode = event->mode;
+	if (!module_lifecycle_is_running(&ctx.lc)) {
+		ctx.current_transport = event->hid_transport;
 		return false;
 	}
 
-	mode_changed = mode_valid && (current_mode != event->mode);
-	if (mode_changed) {
-		emit_release_reports(current_mode);
+	transport_changed = (ctx.current_transport != HID_TRANSPORT_POLICY_NONE) &&
+			    (ctx.current_transport != event->hid_transport);
+	if (transport_changed) {
+		emit_release_reports(ctx.current_transport);
 		emit_function_state_event();
 		keyboard_state_clear();
 		reports_cache_invalidate();
 	}
 
-	current_mode = event->mode;
-	mode_valid = true;
-	emit_all_reports(true);
+	ctx.current_transport = event->hid_transport;
+
+	if (ctx.current_transport != HID_TRANSPORT_POLICY_NONE) {
+		emit_all_reports(true);
+	}
 
 	return false;
 }
 
 static bool handle_encoder_event(const struct encoder_event *event)
 {
-	if (!running || !mode_valid) {
+	if (!module_lifecycle_is_running(&ctx.lc) ||
+	    (ctx.current_transport == HID_TRANSPORT_POLICY_NONE)) {
+		return false;
+	}
+
+	if (ctx.settings_active) {
 		return false;
 	}
 
@@ -612,7 +663,8 @@ static bool handle_encoder_event(const struct encoder_event *event)
 static bool handle_function_bitmap_update_event(
 	const struct function_bitmap_update_event *event)
 {
-	memcpy(function_usage_mask, event->bitmap, sizeof(function_usage_mask));
+	memcpy(ctx.function_usage_mask, event->bitmap,
+	       sizeof(ctx.function_usage_mask));
 	return false;
 }
 
@@ -639,11 +691,13 @@ static bool app_event_handler(const struct app_event_header *aeh)
 			return false;
 		}
 
-		if (transport_protocol_modes[event->transport] != event->protocol_mode) {
-			transport_protocol_modes[event->transport] = event->protocol_mode;
+		if (ctx.transport_protocol_modes[event->transport] !=
+		    event->protocol_mode) {
+			ctx.transport_protocol_modes[event->transport] =
+				event->protocol_mode;
 
-			if (running && mode_valid &&
-			    mode_to_transport(current_mode, &active_transport) &&
+			if (module_lifecycle_is_running(&ctx.lc) &&
+			    policy_to_transport(ctx.current_transport, &active_transport) &&
 			    (active_transport == event->transport)) {
 				reports_cache_invalidate();
 				emit_keys_report(true);
@@ -657,31 +711,53 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		return false;
 	}
 
-	if (is_mode_switch_event(aeh)) {
-		return handle_mode_switch_event(cast_mode_switch_event(aeh));
+	if (is_transport_policy_event(aeh)) {
+		return handle_transport_policy_event(
+			cast_transport_policy_event(aeh));
+	}
+
+	if (is_settings_mode_event(aeh)) {
+		const struct settings_mode_event *event =
+			cast_settings_mode_event(aeh);
+
+		if (ctx.settings_active == event->active) {
+			return false;
+		}
+
+		ctx.settings_active = event->active;
+		if (ctx.settings_active) {
+			if (ctx.current_transport != HID_TRANSPORT_POLICY_NONE) {
+				emit_release_reports(ctx.current_transport);
+			}
+			emit_function_state_event();
+			keyboard_state_clear();
+			reports_cache_invalidate();
+		}
+
+		return false;
 	}
 
 	if (is_module_state_event(aeh)) {
 		const struct module_state_event *event = cast_module_state_event(aeh);
 
 		if (check_state(event, MODULE_ID(main), MODULE_STATE_READY)) {
-			(void)module_set_lifecycle(&lifecycle, LC_RUNNING);
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
 		}
 
 		return false;
 	}
 
 	if (is_power_down_event(aeh)) {
-		if (module_lifecycle_is_initialized(&lifecycle)) {
-			(void)module_set_lifecycle(&lifecycle, LC_STOPPED);
+		if (module_lifecycle_is_initialized(&ctx.lc)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_STOPPED);
 		}
 
 		return false;
 	}
 
 	if (is_wake_up_event(aeh)) {
-		if (module_lifecycle_is_initialized(&lifecycle)) {
-			(void)module_set_lifecycle(&lifecycle, LC_RUNNING);
+		if (module_lifecycle_is_initialized(&ctx.lc)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
 		}
 
 		return false;
@@ -696,7 +772,8 @@ APP_EVENT_SUBSCRIBE(MODULE, button_event);
 APP_EVENT_SUBSCRIBE(MODULE, encoder_event);
 APP_EVENT_SUBSCRIBE(MODULE, function_bitmap_update_event);
 APP_EVENT_SUBSCRIBE(MODULE, set_protocol_event);
-APP_EVENT_SUBSCRIBE(MODULE, mode_switch_event);
+APP_EVENT_SUBSCRIBE(MODULE, settings_mode_event);
+APP_EVENT_SUBSCRIBE(MODULE, transport_policy_event);
 APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
 APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
 APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/mode_policy_module.c

@@ -5,18 +5,24 @@
 #define MODULE mode_policy_module
 #include <caf/events/module_state_event.h>
 #include <caf/events/module_suspend_event.h>
-#include <caf/events/power_event.h>
 
 #include <zephyr/logging/log.h>
 
 #include "module_lifecycle.h"
 #include "mode_switch_event.h"
+#include "transport_policy_event.h"
 
 LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
 
+struct transport_policy_state {
+	enum hid_transport_policy hid_transport;
+	enum ble_profile_policy ble_profile;
+};
+
 struct mode_policy_module_ctx {
 	struct module_lifecycle_ctx lc;
 	enum mode_switch_mode active_mode;
+	struct transport_policy_state policy;
 };
 
 static int do_init(void);
@@ -24,8 +30,8 @@ static int do_start(void);
 static int do_stop(void);
 
 static const struct module_lifecycle_cfg lifecycle_cfg = {
-	.mode = ML_MODE_POWER,
-	.stopped_state = MODULE_STATE_STANDBY,
+	.mode = ML_MODE_NONE,
+	.stopped_state = MODULE_STATE_OFF,
 };
 
 static const struct module_lifecycle_ops lifecycle_ops = {
@@ -41,8 +47,51 @@ static struct mode_policy_module_ctx ctx = {
 		.ops = &lifecycle_ops,
 	},
 	.active_mode = MODE_SWITCH_USB,
+	.policy = {
+		.hid_transport = HID_TRANSPORT_POLICY_USB,
+		.ble_profile = BLE_PROFILE_POLICY_NONE,
+	},
 };
 
+static void build_policy_from_mode(enum mode_switch_mode mode,
+				   struct transport_policy_state *policy)
+{
+	switch (mode) {
+	case MODE_SWITCH_USB:
+		policy->hid_transport = HID_TRANSPORT_POLICY_USB;
+		policy->ble_profile = BLE_PROFILE_POLICY_NONE;
+		break;
+
+	case MODE_SWITCH_BLE:
+		policy->hid_transport = HID_TRANSPORT_POLICY_BLE;
+		policy->ble_profile = BLE_PROFILE_POLICY_GENERAL;
+		break;
+
+	case MODE_SWITCH_24G:
+		policy->hid_transport = HID_TRANSPORT_POLICY_BLE;
+		policy->ble_profile = BLE_PROFILE_POLICY_DONGLE;
+		break;
+
+	default:
+		policy->hid_transport = HID_TRANSPORT_POLICY_NONE;
+		policy->ble_profile = BLE_PROFILE_POLICY_NONE;
+		break;
+	}
+}
+
+static bool policy_equal(const struct transport_policy_state *a,
+			 const struct transport_policy_state *b)
+{
+	return (a->hid_transport == b->hid_transport) &&
+	       (a->ble_profile == b->ble_profile);
+}
+
+static void broadcast_policy(void)
+{
+	submit_transport_policy_event(ctx.active_mode, ctx.policy.hid_transport,
+				      ctx.policy.ble_profile);
+}
+
 static void mode_policy_set_ble(bool enable)
 {
 	if (enable) {
@@ -77,7 +126,7 @@ static void mode_policy_set_usb(bool enable)
 	}
 }
 
-static void apply_active_mode(void)
+static void apply_stack_policy(void)
 {
 	switch (ctx.active_mode) {
 	case MODE_SWITCH_BLE:
@@ -91,7 +140,7 @@ static void apply_active_mode(void)
 		break;
 
 	case MODE_SWITCH_24G:
-		mode_policy_set_ble(false);
+		mode_policy_set_ble(true);
 		mode_policy_set_usb(false);
 		break;
 
@@ -112,7 +161,7 @@ static void apply_ble_mode_policy(void)
 		break;
 
 	case MODE_SWITCH_24G:
-		mode_policy_set_ble(false);
+		mode_policy_set_ble(true);
 		break;
 
 	default:
@@ -140,15 +189,36 @@ static void apply_usb_mode_policy(void)
 	}
 }
 
+static void update_mode_policy(enum mode_switch_mode mode)
+{
+	struct transport_policy_state next_policy;
+	bool mode_changed = (ctx.active_mode != mode);
+	bool policy_changed;
+
+	build_policy_from_mode(mode, &next_policy);
+	policy_changed = !policy_equal(&ctx.policy, &next_policy);
+
+	ctx.active_mode = mode;
+	ctx.policy = next_policy;
+
+	if (policy_changed || mode_changed) {
+		broadcast_policy();
+	}
+
+	apply_stack_policy();
+}
+
 static int do_init(void)
 {
 	ctx.active_mode = MODE_SWITCH_USB;
+	build_policy_from_mode(ctx.active_mode, &ctx.policy);
 	return 0;
 }
 
 static int do_start(void)
 {
-	apply_active_mode();
+	broadcast_policy();
+	apply_stack_policy();
 	return 0;
 }
 
@@ -163,8 +233,7 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		const struct mode_switch_event *event = cast_mode_switch_event(aeh);
 
 		if (module_lifecycle_is_running(&ctx.lc)) {
-			ctx.active_mode = event->mode;
-			apply_active_mode();
+			update_mode_policy(event->mode);
 		}
 
 		return false;
@@ -195,27 +264,9 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		return false;
 	}
 
-	if (is_power_down_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)module_set_lifecycle(&ctx.lc, LC_STOPPED);
-		}
-
-		return false;
-	}
-
-	if (is_wake_up_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
-		}
-
-		return false;
-	}
-
 	return false;
 }
 
 APP_EVENT_LISTENER(MODULE, app_event_handler);
 APP_EVENT_SUBSCRIBE(MODULE, mode_switch_event);
 APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
-APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
-APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/mode_switch_module.c

@@ -201,6 +201,10 @@ static bool app_event_handler(const struct app_event_header *aeh)
 	if (is_wake_up_event(aeh)) {
 		if (module_lifecycle_is_initialized(&ctx.lc)) {
 			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
+
+			if (ctx.last_mode != MODE_SWITCH_INVALID) {
+				submit_mode_switch_event(ctx.last_mode, 0U);
+			}
 		}
 
 		return false;

src/protocol_module.c

@@ -8,7 +8,6 @@
 
 #define MODULE protocol_module
 #include <caf/events/module_state_event.h>
-#include <caf/events/power_event.h>
 
 #include <zephyr/logging/log.h>
 #include <zephyr/sys/util.h>
@@ -37,7 +36,7 @@ LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
 #define PROTOCOL_FIRMWARE_MAJOR 0U
 #define PROTOCOL_FIRMWARE_MINOR 0U
 #define PROTOCOL_CAPABILITY_FLAGS (BIT(0) | BIT(1) | BIT(2) | BIT(3) | BIT(4))
-#define PROTOCOL_MAX_MSG_LEN 128U
+#define PROTOCOL_MAX_MSG_LEN PROTO_MAX_FRAME_LEN
 
 enum proto_session_state {
 	PROTO_SESSION_DOWN = 0,
@@ -55,8 +54,8 @@ static int do_start(void);
 static int do_stop(void);
 
 static const struct module_lifecycle_cfg lifecycle_cfg = {
-	.mode = ML_MODE_POWER,
-	.stopped_state = MODULE_STATE_STANDBY,
+	.mode = ML_MODE_NONE,
+	.stopped_state = MODULE_STATE_OFF,
 };
 
 static const struct module_lifecycle_ops lifecycle_ops = {
@@ -73,26 +72,6 @@ static struct protocol_module_ctx ctx = {
 	},
 };
 
-#define session_state ctx.session_state
-
-static const char *lifecycle_name(enum module_lifecycle state)
-{
-	switch (state) {
-	case LC_UNINIT:
-		return "UNINIT";
-	case LC_RUNNING:
-		return "RUNNING";
-	case LC_STOPPED:
-		return "STOPPED";
-	case LC_SUSPENDED:
-		return "SUSPENDED";
-	case LC_ERROR:
-		return "ERROR";
-	default:
-		return "?";
-	}
-}
-
 static const char *proto_session_name(enum proto_session_state state)
 {
 	switch (state) {
@@ -119,8 +98,32 @@ static const char *proto_transport_name(enum proto_transport transport)
 	}
 }
 
+static void proto_session_set(enum proto_transport transport,
+			      enum proto_session_state new_state,
+			      const char *reason)
+{
+	enum proto_session_state old_state;
+
+	if (transport >= PROTO_TRANSPORT_COUNT) {
+		return;
+	}
+
+	old_state = ctx.session_state[transport];
+	if (old_state == new_state) {
+		LOG_INF("Protocol session keep transport:%s state:%s reason:%s",
+			proto_transport_name(transport),
+			proto_session_name(new_state), reason);
+		return;
+	}
+
+	ctx.session_state[transport] = new_state;
+	LOG_INF("Protocol session transport:%s %s -> %s reason:%s",
+		proto_transport_name(transport), proto_session_name(old_state),
+		proto_session_name(new_state), reason);
+}
+
 static int decode_body(const uint8_t *payload, size_t payload_len,
-		       CdcPacketBody *body)
+		       DeviceMessage *body)
 {
 	pb_istream_t stream;
 
@@ -128,10 +131,10 @@ static int decode_body(const uint8_t *payload, size_t payload_len,
 		return -EINVAL;
 	}
 
-	*body = (CdcPacketBody)CdcPacketBody_init_zero;
+	*body = (DeviceMessage)DeviceMessage_init_zero;
 	stream = pb_istream_from_buffer(payload, payload_len);
 
-	if (!pb_decode(&stream, CdcPacketBody_fields, body)) {
+	if (!pb_decode(&stream, DeviceMessage_fields, body)) {
 		LOG_WRN("pb_decode failed: %s", PB_GET_ERROR(&stream));
 		return -EBADMSG;
 	}
@@ -139,7 +142,37 @@ static int decode_body(const uint8_t *payload, size_t payload_len,
 	return 0;
 }
 
-static int encode_body(const CdcPacketBody *body, uint8_t *payload,
+static int decode_frame(const uint8_t *frame, size_t frame_len,
+			const uint8_t **payload, size_t *payload_len)
+{
+	uint16_t magic;
+	uint8_t len;
+
+	if ((frame == NULL) || (payload == NULL) || (payload_len == NULL)) {
+		return -EINVAL;
+	}
+
+	if (frame_len < PROTO_FRAME_HEADER_SIZE) {
+		return -EBADMSG;
+	}
+
+	magic = (uint16_t)frame[0] | ((uint16_t)frame[1] << 8);
+	if (magic != PROTO_FRAME_MAGIC) {
+		return -EBADMSG;
+	}
+
+	len = frame[2];
+	if ((len > PROTO_MAX_PAYLOAD_LEN) ||
+	    (frame_len != (PROTO_FRAME_HEADER_SIZE + len))) {
+		return -EBADMSG;
+	}
+
+	*payload = &frame[PROTO_FRAME_HEADER_SIZE];
+	*payload_len = len;
+	return 0;
+}
+
+static int encode_body(const DeviceMessage *body, uint8_t *payload,
 		       size_t payload_buf_size, size_t *payload_len)
 {
 	pb_ostream_t stream;
@@ -149,7 +182,7 @@ static int encode_body(const CdcPacketBody *body, uint8_t *payload,
 	}
 
 	stream = pb_ostream_from_buffer(payload, payload_buf_size);
-	if (!pb_encode(&stream, CdcPacketBody_fields, body)) {
+	if (!pb_encode(&stream, DeviceMessage_fields, body)) {
 		LOG_WRN("pb_encode failed: %s", PB_GET_ERROR(&stream));
 		return -EIO;
 	}
@@ -158,12 +191,45 @@ static int encode_body(const CdcPacketBody *body, uint8_t *payload,
 	return 0;
 }
 
-static int encode_hello_rsp(uint8_t *rsp_payload, size_t rsp_payload_buf_size,
+static int encode_frame(const DeviceMessage *body, uint8_t *frame,
+			size_t frame_buf_size, size_t *frame_len)
+{
+	size_t payload_len;
+	int err;
+
+	if ((frame == NULL) || (frame_len == NULL)) {
+		return -EINVAL;
+	}
+
+	if (frame_buf_size < PROTO_FRAME_HEADER_SIZE) {
+		return -ENOSPC;
+	}
+
+	err = encode_body(body, &frame[PROTO_FRAME_HEADER_SIZE],
+			  frame_buf_size - PROTO_FRAME_HEADER_SIZE, &payload_len);
+	if (err) {
+		return err;
+	}
+
+	if (payload_len > PROTO_MAX_PAYLOAD_LEN) {
+		return -EMSGSIZE;
+	}
+
+	frame[0] = (uint8_t)(PROTO_FRAME_MAGIC & 0xFFU);
+	frame[1] = (uint8_t)(PROTO_FRAME_MAGIC >> 8);
+	frame[2] = (uint8_t)payload_len;
+	*frame_len = PROTO_FRAME_HEADER_SIZE + payload_len;
+	return 0;
+}
+
+static int encode_hello_rsp(uint32_t reply_to, uint8_t *rsp_payload,
+			    size_t rsp_payload_buf_size,
 			    size_t *rsp_payload_len)
 {
-	CdcPacketBody body = CdcPacketBody_init_zero;
+	DeviceMessage body = DeviceMessage_init_zero;
 
-	body.which_body = CdcPacketBody_hello_rsp_tag;
+	body.reply_to = reply_to;
+	body.which_body = DeviceMessage_hello_rsp_tag;
 	body.body.hello_rsp.protocol_version = PROTOCOL_VERSION;
 	body.body.hello_rsp.vendor_id = PROTOCOL_VENDOR_ID;
 	body.body.hello_rsp.product_id = PROTOCOL_PRODUCT_ID;
@@ -171,68 +237,59 @@ static int encode_hello_rsp(uint8_t *rsp_payload, size_t rsp_payload_buf_size,
 	body.body.hello_rsp.firmware_minor = PROTOCOL_FIRMWARE_MINOR;
 	body.body.hello_rsp.capability_flags = PROTOCOL_CAPABILITY_FLAGS;
 
-	return encode_body(&body, rsp_payload, rsp_payload_buf_size, rsp_payload_len);
+	return encode_frame(&body, rsp_payload, rsp_payload_buf_size, rsp_payload_len);
 }
 
-static int encode_ack(uint8_t acked_type, uint8_t *rsp_payload,
-		      size_t rsp_payload_buf_size, size_t *rsp_payload_len)
+static int encode_response(uint32_t reply_to, ResponseCode error_code,
+			   uint8_t *rsp_payload, size_t rsp_payload_buf_size,
+			   size_t *rsp_payload_len)
 {
-	CdcPacketBody body = CdcPacketBody_init_zero;
+	DeviceMessage body = DeviceMessage_init_zero;
 
-	body.which_body = CdcPacketBody_ack_tag;
-	body.body.ack.acked_type = acked_type;
+	body.reply_to = reply_to;
+	body.which_body = DeviceMessage_response_tag;
+	body.body.response.error_code = error_code;
 
-	return encode_body(&body, rsp_payload, rsp_payload_buf_size, rsp_payload_len);
-}
-
-static int encode_error(uint8_t error_type, ErrorCode error_code,
-			uint8_t *rsp_payload, size_t rsp_payload_buf_size,
-			size_t *rsp_payload_len)
-{
-	CdcPacketBody body = CdcPacketBody_init_zero;
-
-	body.which_body = CdcPacketBody_error_tag;
-	body.body.error.error_type = error_type;
-	body.body.error.error_code = error_code;
-
-	return encode_body(&body, rsp_payload, rsp_payload_buf_size, rsp_payload_len);
+	return encode_frame(&body, rsp_payload, rsp_payload_buf_size, rsp_payload_len);
 }
 
 static int encode_led_state(uint32_t led_mask, uint8_t *payload,
 			    size_t payload_buf_size, size_t *payload_len)
 {
-	CdcPacketBody body = CdcPacketBody_init_zero;
+	DeviceMessage body = DeviceMessage_init_zero;
 
-	body.which_body = CdcPacketBody_led_state_tag;
+	body.which_body = DeviceMessage_led_state_tag;
 	body.body.led_state.led_mask = led_mask;
 
-	return encode_body(&body, payload, payload_buf_size, payload_len);
+	return encode_frame(&body, payload, payload_buf_size, payload_len);
 }
 
 static int encode_function_bitmap_state(const uint8_t *bitmap, uint8_t *payload,
 					size_t payload_buf_size,
 					size_t *payload_len)
 {
-	CdcPacketBody body = CdcPacketBody_init_zero;
+	DeviceMessage body = DeviceMessage_init_zero;
 
 	if (bitmap == NULL) {
 		return -EINVAL;
 	}
 
-	body.which_body = CdcPacketBody_function_key_event_tag;
+	body.which_body = DeviceMessage_function_key_event_tag;
 	body.body.function_key_event.usage_bitmap.size = KEYBOARD_PROTOCOL_BITMAP_BYTES;
 	memcpy(body.body.function_key_event.usage_bitmap.bytes, bitmap,
 	       KEYBOARD_PROTOCOL_BITMAP_BYTES);
 
-	return encode_body(&body, payload, payload_buf_size, payload_len);
+	return encode_frame(&body, payload, payload_buf_size, payload_len);
 }
 
 static int do_init(void)
 {
-	for (size_t i = 0; i < ARRAY_SIZE(session_state); i++) {
-		session_state[i] = PROTO_SESSION_DOWN;
+	for (size_t i = 0; i < ARRAY_SIZE(ctx.session_state); i++) {
+		ctx.session_state[i] = PROTO_SESSION_DOWN;
 	}
 
+	LOG_INF("Protocol init done");
+
 	return 0;
 }
 
@@ -251,8 +308,9 @@ static int do_stop(void)
 		return 0;
 	}
 
-	for (size_t i = 0; i < ARRAY_SIZE(session_state); i++) {
-		session_state[i] = PROTO_SESSION_DOWN;
+	for (size_t i = 0; i < ARRAY_SIZE(ctx.session_state); i++) {
+		proto_session_set((enum proto_transport)i, PROTO_SESSION_DOWN,
+				  "module_stop");
 	}
 
 	return 0;
@@ -265,7 +323,9 @@ int protocol_module_process_message(enum proto_transport transport,
 				    size_t rsp_payload_buf_size,
 				    size_t *rsp_payload_len)
 {
-	CdcPacketBody body;
+	DeviceMessage body;
+	const uint8_t *payload;
+	size_t payload_len;
 	int err;
 
 	if ((transport >= PROTO_TRANSPORT_COUNT) ||
@@ -276,22 +336,27 @@ int protocol_module_process_message(enum proto_transport transport,
 	if (!module_lifecycle_is_running(&ctx.lc)) {
 		LOG_WRN("Reject proto msg transport:%s len:%u lc:%s",
 			proto_transport_name(transport), (uint32_t)req_payload_len,
-			lifecycle_name(ctx.lc.state));
+			module_lifecycle_name(ctx.lc.state));
 		return -EAGAIN;
 	}
 
-	err = decode_body(req_payload, req_payload_len, &body);
+	err = decode_frame(req_payload, req_payload_len, &payload, &payload_len);
+	if (err) {
+		return err;
+	}
+
+	err = decode_body(payload, payload_len, &body);
 	if (err) {
 		return err;
 	}
 
 	switch (body.which_body) {
-	case CdcPacketBody_hello_req_tag:
-		if (session_state[transport] == PROTO_SESSION_DOWN) {
+	case DeviceMessage_hello_req_tag:
+		if (ctx.session_state[transport] == PROTO_SESSION_DOWN) {
 			LOG_WRN("Reject HelloReq transport:%s session:%s lc:%s",
 				proto_transport_name(transport),
-				proto_session_name(session_state[transport]),
-				lifecycle_name(ctx.lc.state));
+				proto_session_name(ctx.session_state[transport]),
+				module_lifecycle_name(ctx.lc.state));
 			return -EAGAIN;
 		}
 
@@ -303,49 +368,48 @@ int protocol_module_process_message(enum proto_transport transport,
 				body.body.hello_req.protocol_version);
 		}
 
-		session_state[transport] = PROTO_SESSION_ACTIVE;
-		return encode_hello_rsp(rsp_payload, rsp_payload_buf_size, rsp_payload_len);
+		proto_session_set(transport, PROTO_SESSION_ACTIVE, "hello_req");
+		return encode_hello_rsp(body.msg_id, rsp_payload, rsp_payload_buf_size,
+					rsp_payload_len);
 
-	case CdcPacketBody_bitmap_tag:
-		if (session_state[transport] != PROTO_SESSION_ACTIVE) {
-			return encode_error(CdcPacketBody_bitmap_tag,
-					    ErrorCode_ERROR_CODE_NOT_READY,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+	case DeviceMessage_bitmap_tag:
+		if (ctx.session_state[transport] != PROTO_SESSION_ACTIVE) {
+			return encode_response(body.msg_id,
+					       ResponseCode_RESPONSE_CODE_NOT_READY,
+					       rsp_payload, rsp_payload_buf_size,
+					       rsp_payload_len);
 		}
 
 		if (body.body.bitmap.usage_bitmap.size != KEYBOARD_PROTOCOL_BITMAP_BYTES) {
-			return encode_error(CdcPacketBody_bitmap_tag,
-					    ErrorCode_ERROR_CODE_INVALID_LENGTH,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+			return encode_response(
+				body.msg_id, ResponseCode_RESPONSE_CODE_INVALID_LENGTH,
+				rsp_payload, rsp_payload_buf_size, rsp_payload_len);
 		}
 
 		err = submit_function_bitmap_update_event(
 			body.body.bitmap.usage_bitmap.bytes);
 		if (err) {
-			return encode_error(CdcPacketBody_bitmap_tag,
-					    ErrorCode_ERROR_CODE_INVALID_PARAM,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+			return encode_response(
+				body.msg_id, ResponseCode_RESPONSE_CODE_INVALID_PARAM,
+				rsp_payload, rsp_payload_buf_size, rsp_payload_len);
 		}
 
-		return encode_ack(CdcPacketBody_bitmap_tag, rsp_payload,
-				  rsp_payload_buf_size, rsp_payload_len);
+		return encode_response(body.msg_id, ResponseCode_RESPONSE_CODE_OK,
+				       rsp_payload, rsp_payload_buf_size,
+				       rsp_payload_len);
 
-	case CdcPacketBody_time_sync_tag:
-		if (session_state[transport] != PROTO_SESSION_ACTIVE) {
-			return encode_error(CdcPacketBody_time_sync_tag,
-					    ErrorCode_ERROR_CODE_NOT_READY,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+	case DeviceMessage_time_sync_tag:
+		if (ctx.session_state[transport] != PROTO_SESSION_ACTIVE) {
+			return encode_response(body.msg_id,
+					       ResponseCode_RESPONSE_CODE_NOT_READY,
+					       rsp_payload, rsp_payload_buf_size,
+					       rsp_payload_len);
 		}
 
 		if (body.body.time_sync.version != 1U) {
-			return encode_error(CdcPacketBody_time_sync_tag,
-					    ErrorCode_ERROR_CODE_INVALID_PARAM,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+			return encode_response(
+				body.msg_id, ResponseCode_RESPONSE_CODE_INVALID_PARAM,
+				rsp_payload, rsp_payload_buf_size, rsp_payload_len);
 		}
 
 		submit_time_sync_event(body.body.time_sync.version,
@@ -353,24 +417,24 @@ int protocol_module_process_message(enum proto_transport transport,
 				       body.body.time_sync.timezone_min,
 				       body.body.time_sync.utc_ms,
 				       body.body.time_sync.accuracy_ms);
-		return encode_ack(CdcPacketBody_time_sync_tag, rsp_payload,
-				  rsp_payload_buf_size, rsp_payload_len);
+		return encode_response(body.msg_id, ResponseCode_RESPONSE_CODE_OK,
+				       rsp_payload, rsp_payload_buf_size,
+				       rsp_payload_len);
 
-	case CdcPacketBody_theme_rgb_tag:
-		if (session_state[transport] != PROTO_SESSION_ACTIVE) {
-			return encode_error(CdcPacketBody_theme_rgb_tag,
-					    ErrorCode_ERROR_CODE_NOT_READY,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+	case DeviceMessage_theme_rgb_tag:
+		if (ctx.session_state[transport] != PROTO_SESSION_ACTIVE) {
+			return encode_response(body.msg_id,
+					       ResponseCode_RESPONSE_CODE_NOT_READY,
+					       rsp_payload, rsp_payload_buf_size,
+					       rsp_payload_len);
 		}
 
 		if ((body.body.theme_rgb.red > 255U) ||
 		    (body.body.theme_rgb.green > 255U) ||
 		    (body.body.theme_rgb.blue > 255U)) {
-			return encode_error(CdcPacketBody_theme_rgb_tag,
-					    ErrorCode_ERROR_CODE_INVALID_PARAM,
-					    rsp_payload, rsp_payload_buf_size,
-					    rsp_payload_len);
+			return encode_response(
+				body.msg_id, ResponseCode_RESPONSE_CODE_INVALID_PARAM,
+				rsp_payload, rsp_payload_buf_size, rsp_payload_len);
 		}
 
 		submit_theme_rgb_update_event((struct theme_rgb) {
@@ -378,12 +442,16 @@ int protocol_module_process_message(enum proto_transport transport,
 			.g = (uint8_t)body.body.theme_rgb.green,
 			.b = (uint8_t)body.body.theme_rgb.blue,
 		});
-		return encode_ack(CdcPacketBody_theme_rgb_tag, rsp_payload,
-				  rsp_payload_buf_size, rsp_payload_len);
+		return encode_response(body.msg_id, ResponseCode_RESPONSE_CODE_OK,
+				       rsp_payload, rsp_payload_buf_size,
+				       rsp_payload_len);
 
 	default:
 		LOG_WRN("Unsupported protobuf body case %d", body.which_body);
-		return -ENOTSUP;
+		return encode_response(body.msg_id,
+				       ResponseCode_RESPONSE_CODE_UNKNOWN_TYPE,
+				       rsp_payload, rsp_payload_buf_size,
+				       rsp_payload_len);
 	}
 }
 
@@ -407,14 +475,17 @@ static bool handle_proto_rx_event(const struct proto_rx_event *event)
 		if (err != -ENOTSUP) {
 			LOG_WRN("Protocol processing failed (%d) transport:%s session:%s lc:%s len:%u",
 				err, proto_transport_name(event->transport),
-				proto_session_name(session_state[event->transport]),
-				lifecycle_name(ctx.lc.state),
+				proto_session_name(ctx.session_state[event->transport]),
+				module_lifecycle_name(ctx.lc.state),
 				(uint32_t)event->dyndata.size);
 		}
 
 		return false;
 	}
 
+	LOG_INF("Protocol response ready transport:%s rsp_len:%u session:%s",
+		proto_transport_name(event->transport), (unsigned int)rsp_payload_len,
+		proto_session_name(ctx.session_state[event->transport]));
 	err = submit_proto_tx_event(event->transport, rsp_payload, rsp_payload_len);
 	if (err) {
 		LOG_WRN("Proto TX submit failed (%d)", err);
@@ -432,11 +503,13 @@ static bool handle_proto_transport_state_event(
 
 	switch (event->state) {
 	case PROTO_TRANSPORT_LINK_DOWN:
-		session_state[event->transport] = PROTO_SESSION_DOWN;
+		proto_session_set(event->transport, PROTO_SESSION_DOWN,
+				  "transport_link_down");
 		break;
 
 	case PROTO_TRANSPORT_LINK_READY:
-		session_state[event->transport] = PROTO_SESSION_WAIT_HELLO;
+		proto_session_set(event->transport, PROTO_SESSION_WAIT_HELLO,
+				  "transport_link_ready");
 		break;
 
 	default:
@@ -459,7 +532,10 @@ static bool handle_function_bitmap_state_event(
 
 	for (enum proto_transport transport = 0; transport < PROTO_TRANSPORT_COUNT;
 	     transport++) {
-		if (session_state[transport] != PROTO_SESSION_ACTIVE) {
+		if (ctx.session_state[transport] != PROTO_SESSION_ACTIVE) {
+			LOG_INF("FunctionKeyEvent skip transport:%s session:%s",
+				proto_transport_name(transport),
+				proto_session_name(ctx.session_state[transport]));
 			continue;
 		}
 
@@ -473,6 +549,10 @@ static bool handle_function_bitmap_state_event(
 		err = submit_proto_tx_event(transport, payload, payload_len);
 		if (err) {
 			LOG_WRN("FunctionKeyEvent submit failed (%d)", err);
+		} else {
+			LOG_INF("FunctionKeyEvent submit transport:%s len:%u",
+				proto_transport_name(transport),
+				(unsigned int)payload_len);
 		}
 	}
 
@@ -494,7 +574,10 @@ static bool handle_hid_led_event(const struct hid_led_event *event)
 			    PROTO_TRANSPORT_USB_CDC :
 			    PROTO_TRANSPORT_BLE_NUS;
 
-	if (session_state[transport] != PROTO_SESSION_ACTIVE) {
+	if (ctx.session_state[transport] != PROTO_SESSION_ACTIVE) {
+		LOG_INF("LedState skip transport:%s session:%s led_mask:0x%02x",
+			proto_transport_name(transport),
+			proto_session_name(ctx.session_state[transport]), event->led_bm);
 		return false;
 	}
 
@@ -507,6 +590,10 @@ static bool handle_hid_led_event(const struct hid_led_event *event)
 	err = submit_proto_tx_event(transport, payload, payload_len);
 	if (err) {
 		LOG_WRN("LedState submit failed (%d)", err);
+	} else {
+		LOG_INF("LedState submit transport:%s len:%u led_mask:0x%02x",
+			proto_transport_name(transport), (unsigned int)payload_len,
+			event->led_bm);
 	}
 
 	return false;
@@ -544,22 +631,6 @@ static bool app_event_handler(const struct app_event_header *aeh)
 		return false;
 	}
 
-	if (is_power_down_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)module_set_lifecycle(&ctx.lc, LC_STOPPED);
-		}
-
-		return false;
-	}
-
-	if (is_wake_up_event(aeh)) {
-		if (module_lifecycle_is_initialized(&ctx.lc)) {
-			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
-		}
-
-		return false;
-	}
-
 	return false;
 }
 
@@ -569,5 +640,3 @@ APP_EVENT_SUBSCRIBE(MODULE, hid_led_event);
 APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
 APP_EVENT_SUBSCRIBE(MODULE, proto_rx_event);
 APP_EVENT_SUBSCRIBE(MODULE, proto_transport_state_event);
-APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
-APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/settings_module.c

@@ -0,0 +1,234 @@
+#include <stdbool.h>
+#include <stdint.h>
+#include <string.h>
+
+#include <app_event_manager.h>
+
+#define MODULE settings_module
+#include <caf/events/click_event.h>
+#include <caf/events/module_state_event.h>
+#include <caf/events/power_event.h>
+
+#include <zephyr/logging/log.h>
+#include <zephyr/sys/util.h>
+
+#include "ble_bond_multi_event.h"
+#include "encoder_event.h"
+#include "module_lifecycle.h"
+#include "settings_mode_event.h"
+#include "theme_color.h"
+#include "theme_rgb_update_event.h"
+#include "ui/ui_settings_controller.h"
+
+LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
+
+#define SETTINGS_MUTE_KEY_ID 0x180U
+
+struct settings_module_ctx {
+	struct module_lifecycle_ctx lc;
+	bool active;
+	struct theme_rgb current_theme;
+};
+
+static int do_init(void);
+static int do_start(void);
+static int do_stop(void);
+
+static const struct module_lifecycle_cfg lifecycle_cfg = {
+	.mode = ML_MODE_POWER,
+	.stopped_state = MODULE_STATE_STANDBY,
+};
+
+static const struct module_lifecycle_ops lifecycle_ops = {
+	.do_init = do_init,
+	.do_start = do_start,
+	.do_stop = do_stop,
+};
+
+static struct settings_module_ctx ctx = {
+	.lc = {
+		.state = LC_UNINIT,
+		.cfg = &lifecycle_cfg,
+		.ops = &lifecycle_ops,
+	},
+};
+
+static void settings_exit(void)
+{
+	if (!ctx.active) {
+		return;
+	}
+
+	ctx.active = false;
+	ui_settings_controller_close();
+	submit_settings_mode_event(false);
+}
+
+static void settings_enter(void)
+{
+	if (ctx.active) {
+		return;
+	}
+
+	ctx.active = true;
+	submit_settings_mode_event(true);
+	ui_settings_controller_open();
+}
+
+static int do_init(void)
+{
+	ctx.active = false;
+	ctx.current_theme = (struct theme_rgb) {
+		.r = BLINKY_THEME_DEFAULT_R,
+		.g = BLINKY_THEME_DEFAULT_G,
+		.b = BLINKY_THEME_DEFAULT_B,
+	};
+	ui_settings_theme_set_current(ctx.current_theme);
+
+	return 0;
+}
+
+static int do_start(void)
+{
+	return 0;
+}
+
+static int do_stop(void)
+{
+	settings_exit();
+	return 0;
+}
+
+static bool handle_click_event(const struct click_event *event)
+{
+	if (!module_lifecycle_is_running(&ctx.lc) ||
+	    (event->key_id != SETTINGS_MUTE_KEY_ID)) {
+		return false;
+	}
+
+	if (!ctx.active) {
+		if (event->click == CLICK_LONG) {
+			settings_enter();
+		}
+
+		return false;
+	}
+
+	switch (event->click) {
+	case CLICK_SHORT:
+		ui_settings_controller_select();
+		break;
+
+	case CLICK_LONG:
+		{
+			bool active = ui_settings_controller_back();
+
+			if (!active) {
+				settings_exit();
+			}
+		}
+		break;
+
+	default:
+		break;
+	}
+
+	return false;
+}
+
+static bool handle_encoder_event(const struct encoder_event *event)
+{
+	if (!module_lifecycle_is_running(&ctx.lc) || !ctx.active) {
+		return false;
+	}
+
+	ui_settings_controller_move(event->detents);
+	return false;
+}
+
+static bool handle_theme_rgb_update_event(
+	const struct theme_rgb_update_event *event)
+{
+	ctx.current_theme = event->theme;
+	ui_settings_theme_set_current(event->theme);
+
+	if (ctx.active) {
+		ui_settings_controller_refresh(false);
+	}
+
+	return false;
+}
+
+static bool handle_ble_bond_multi_event(
+	const struct ble_bond_multi_event *event)
+{
+	ui_settings_ble_set_current_slot(event->current_slot);
+
+	for (uint8_t i = 0U; i < ARRAY_SIZE(event->slots); i++) {
+		ui_settings_ble_set_slot_meta(i + 1U, &event->slots[i]);
+	}
+
+	if (ctx.active) {
+		ui_settings_controller_refresh(false);
+	}
+
+	return false;
+}
+
+static bool app_event_handler(const struct app_event_header *aeh)
+{
+	if (is_click_event(aeh)) {
+		return handle_click_event(cast_click_event(aeh));
+	}
+
+	if (is_encoder_event(aeh)) {
+		return handle_encoder_event(cast_encoder_event(aeh));
+	}
+
+	if (is_theme_rgb_update_event(aeh)) {
+		return handle_theme_rgb_update_event(
+			cast_theme_rgb_update_event(aeh));
+	}
+
+	if (is_ble_bond_multi_event(aeh)) {
+		return handle_ble_bond_multi_event(
+			cast_ble_bond_multi_event(aeh));
+	}
+
+	if (is_module_state_event(aeh)) {
+		const struct module_state_event *event = cast_module_state_event(aeh);
+
+		if (check_state(event, MODULE_ID(main), MODULE_STATE_READY)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
+		}
+
+		return false;
+	}
+
+	if (is_power_down_event(aeh)) {
+		if (module_lifecycle_is_initialized(&ctx.lc)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_STOPPED);
+		}
+
+		return false;
+	}
+
+	if (is_wake_up_event(aeh)) {
+		if (module_lifecycle_is_initialized(&ctx.lc)) {
+			(void)module_set_lifecycle(&ctx.lc, LC_RUNNING);
+		}
+
+		return false;
+	}
+
+	return false;
+}
+
+APP_EVENT_LISTENER(MODULE, app_event_handler);
+APP_EVENT_SUBSCRIBE(MODULE, click_event);
+APP_EVENT_SUBSCRIBE(MODULE, encoder_event);
+APP_EVENT_SUBSCRIBE(MODULE, module_state_event);
+APP_EVENT_SUBSCRIBE(MODULE, theme_rgb_update_event);
+APP_EVENT_SUBSCRIBE(MODULE, ble_bond_multi_event);
+APP_EVENT_SUBSCRIBE_EARLY(MODULE, power_down_event);
+APP_EVENT_SUBSCRIBE(MODULE, wake_up_event);

src/swift_pair_module.c

@@ -0,0 +1,58 @@
+#include <bluetooth/adv_prov/swift_pair.h>
+
+#define MODULE swift_pair_module
+#include <caf/events/ble_common_event.h>
+
+#include <zephyr/logging/log.h>
+
+#include "ble_bond_multi_event.h"
+
+LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
+
+static void update_swift_pair_payload(uint8_t selected_identity)
+{
+	bool enable = (selected_identity != BLE_BOND_MULTI_DONGLE_SLOT_ID);
+
+	bt_le_adv_prov_swift_pair_enable(enable);
+	LOG_INF("Swift Pair payload %s for identity %u",
+		enable ? "enabled" : "disabled", selected_identity);
+}
+
+static bool handle_ble_bond_multi_event(const struct ble_bond_multi_event *event)
+{
+	update_swift_pair_payload(event->active_identity_id);
+	return false;
+}
+
+static bool handle_ble_peer_operation_event(const struct ble_peer_operation_event *event)
+{
+	switch (event->op) {
+	case PEER_OPERATION_SELECTED:
+	case PEER_OPERATION_ERASED:
+		update_swift_pair_payload(event->bt_stack_id);
+		break;
+
+	default:
+		break;
+	}
+
+	return false;
+}
+
+static bool app_event_handler(const struct app_event_header *aeh)
+{
+	if (is_ble_bond_multi_event(aeh)) {
+		return handle_ble_bond_multi_event(cast_ble_bond_multi_event(aeh));
+	}
+
+	if (is_ble_peer_operation_event(aeh)) {
+		return handle_ble_peer_operation_event(
+			cast_ble_peer_operation_event(aeh));
+	}
+
+	return false;
+}
+
+APP_EVENT_LISTENER(MODULE, app_event_handler);
+APP_EVENT_SUBSCRIBE(MODULE, ble_bond_multi_event);
+APP_EVENT_SUBSCRIBE_EARLY(MODULE, ble_peer_operation_event);

src/ui/ui_main.c

@@ -19,17 +19,24 @@ enum {
 };
 
 struct ui_main_ctx {
+	lv_obj_t *content;
 	lv_obj_t *status_badges[UI_STATUS_COUNT];
 	lv_obj_t *status_labels[UI_STATUS_COUNT];
 	lv_obj_t *battery_icon;
 	lv_obj_t *battery_label;
 	lv_obj_t *battery_state_label;
+	lv_obj_t *ble_link_wrap;
+	lv_obj_t *ble_link_spinner;
+	lv_obj_t *ble_link_icon;
 	lv_obj_t *date_label;
 	lv_obj_t *time_label;
 };
 
 static struct ui_main_ctx g_ui;
 static bool ui_initialized;
+static const struct ui_main_model *page_model;
+static const char *page_date_text;
+static const char *page_time_text;
 
 static const char *const status_texts[UI_STATUS_COUNT] = {
 	LV_SYMBOL_USB,
@@ -38,6 +45,8 @@ static const char *const status_texts[UI_STATUS_COUNT] = {
 	"A",
 };
 
+#define UI_BLE_LINK_BLUE lv_color_hex(0x0082FC)
+
 static lv_color_t ui_main_get_battery_color(uint8_t battery_level)
 {
 	if (battery_level > 70U) {
@@ -116,6 +125,10 @@ static void ui_main_create_status_chip(lv_obj_t *parent, enum ui_status_id id)
 
 void ui_main_refresh_status_bar(const struct ui_main_model *model)
 {
+	if (!ui_initialized) {
+		return;
+	}
+
 	for (uint32_t i = 0; i < UI_STATUS_COUNT; i++) {
 		lv_obj_t *badge = g_ui.status_badges[i];
 		lv_obj_t *label = g_ui.status_labels[i];
@@ -145,7 +158,8 @@ void ui_main_refresh_battery(const struct ui_main_model *model)
 	lv_color_t battery_color;
 	lv_color_t state_color = lv_color_white();
 
-	if ((g_ui.battery_icon == NULL) || (g_ui.battery_label == NULL) ||
+	if (!ui_initialized ||
+	    (g_ui.battery_icon == NULL) || (g_ui.battery_label == NULL) ||
 	    (g_ui.battery_state_label == NULL)) {
 		return;
 	}
@@ -167,11 +181,37 @@ void ui_main_refresh_battery(const struct ui_main_model *model)
 	lv_label_set_text(g_ui.battery_label, battery_text);
 	lv_label_set_text(g_ui.battery_state_label, state_symbol);
 	lv_obj_set_style_text_color(g_ui.battery_state_label, state_color, 0);
+
+	if ((g_ui.ble_link_wrap == NULL) || (g_ui.ble_link_spinner == NULL) ||
+	    (g_ui.ble_link_icon == NULL)) {
+		return;
+	}
+
+	switch (model->ble_link_state) {
+	case UI_BLE_LINK_SEARCHING:
+		lv_obj_clear_flag(g_ui.ble_link_wrap, LV_OBJ_FLAG_HIDDEN);
+		lv_obj_clear_flag(g_ui.ble_link_spinner, LV_OBJ_FLAG_HIDDEN);
+		lv_obj_add_flag(g_ui.ble_link_icon, LV_OBJ_FLAG_HIDDEN);
+		break;
+
+	case UI_BLE_LINK_CONNECTED:
+		lv_obj_clear_flag(g_ui.ble_link_wrap, LV_OBJ_FLAG_HIDDEN);
+		lv_obj_add_flag(g_ui.ble_link_spinner, LV_OBJ_FLAG_HIDDEN);
+		lv_obj_clear_flag(g_ui.ble_link_icon, LV_OBJ_FLAG_HIDDEN);
+		lv_obj_set_style_text_color(g_ui.ble_link_icon, UI_BLE_LINK_BLUE, 0);
+		break;
+
+	case UI_BLE_LINK_HIDDEN:
+	default:
+		lv_obj_add_flag(g_ui.ble_link_wrap, LV_OBJ_FLAG_HIDDEN);
+		break;
+	}
 }
 
 void ui_main_refresh_datetime(const char *date_text, const char *time_text)
 {
-	if ((g_ui.date_label == NULL) || (g_ui.time_label == NULL)) {
+	if (!ui_initialized ||
+	    (g_ui.date_label == NULL) || (g_ui.time_label == NULL)) {
 		return;
 	}
 
@@ -200,6 +240,7 @@ void ui_main_init(const struct ui_main_model *model,
 	lv_obj_t *bottom_row;
 
 	if (ui_initialized) {
+		ui_main_refresh_all(model, date_text, time_text);
 		return;
 	}
 
@@ -213,6 +254,7 @@ void ui_main_init(const struct ui_main_model *model,
 	lv_obj_set_scrollbar_mode(screen, LV_SCROLLBAR_MODE_OFF);
 
 	content = lv_obj_create(screen);
+	g_ui.content = content;
 	lv_obj_remove_style_all(content);
 	lv_obj_set_size(content, LV_PCT(100), LV_PCT(100));
 	lv_obj_set_style_bg_opa(content, LV_OPA_TRANSP, 0);
@@ -258,6 +300,31 @@ void ui_main_init(const struct ui_main_model *model,
 	g_ui.battery_state_label = lv_label_create(battery_wrap);
 	lv_obj_set_style_text_font(g_ui.battery_state_label, &lv_font_montserrat_14, 0);
 
+	g_ui.ble_link_wrap = lv_obj_create(battery_wrap);
+	lv_obj_remove_style_all(g_ui.ble_link_wrap);
+	lv_obj_set_size(g_ui.ble_link_wrap, 18, 18);
+	lv_obj_add_flag(g_ui.ble_link_wrap, LV_OBJ_FLAG_HIDDEN);
+
+	g_ui.ble_link_spinner = lv_spinner_create(g_ui.ble_link_wrap);
+	lv_obj_center(g_ui.ble_link_spinner);
+	lv_obj_set_size(g_ui.ble_link_spinner, 16, 16);
+	lv_spinner_set_anim_params(g_ui.ble_link_spinner, 2000, 200);
+	lv_obj_set_style_arc_width(g_ui.ble_link_spinner, 2, LV_PART_MAIN);
+	lv_obj_set_style_arc_width(g_ui.ble_link_spinner, 2, LV_PART_INDICATOR);
+	lv_obj_set_style_arc_color(g_ui.ble_link_spinner, lv_color_hex(0x2A3442),
+				   LV_PART_MAIN);
+	lv_obj_set_style_arc_color(g_ui.ble_link_spinner, model->theme_color,
+				   LV_PART_INDICATOR);
+	lv_obj_remove_style(g_ui.ble_link_spinner, NULL, LV_PART_KNOB);
+	lv_obj_clear_flag(g_ui.ble_link_spinner, LV_OBJ_FLAG_CLICKABLE);
+
+	g_ui.ble_link_icon = lv_label_create(g_ui.ble_link_wrap);
+	lv_label_set_text(g_ui.ble_link_icon, LV_SYMBOL_BLUETOOTH);
+	lv_obj_center(g_ui.ble_link_icon);
+	lv_obj_set_style_text_font(g_ui.ble_link_icon, &lv_font_montserrat_14, 0);
+	lv_obj_set_style_text_color(g_ui.ble_link_icon, UI_BLE_LINK_BLUE, 0);
+	lv_obj_add_flag(g_ui.ble_link_icon, LV_OBJ_FLAG_HIDDEN);
+
 	middle_row = lv_obj_create(content);
 	lv_obj_remove_style_all(middle_row);
 	lv_obj_set_width(middle_row, LV_PCT(100));
@@ -287,3 +354,56 @@ void ui_main_init(const struct ui_main_model *model,
 	ui_main_refresh_all(model, date_text, time_text);
 	ui_initialized = true;
 }
+
+void ui_main_deinit(void)
+{
+	if (!ui_initialized) {
+		return;
+	}
+
+	if (g_ui.content != NULL) {
+		lv_obj_delete(g_ui.content);
+	}
+
+	memset(&g_ui, 0, sizeof(g_ui));
+	ui_initialized = false;
+}
+
+static void main_page_init(struct ui_page *page)
+{
+	ARG_UNUSED(page);
+	ui_main_init(page_model, page_date_text, page_time_text);
+}
+
+static void main_page_deinit(struct ui_page *page)
+{
+	ARG_UNUSED(page);
+	ui_main_deinit();
+}
+
+static void main_page_refresh(struct ui_page *page)
+{
+	ARG_UNUSED(page);
+	ui_main_refresh_all(page_model, page_date_text, page_time_text);
+}
+
+static const struct ui_page_ops main_page_ops = {
+	.init = main_page_init,
+	.deinit = main_page_deinit,
+	.refresh = main_page_refresh,
+};
+
+static struct ui_page main_page = {
+	.ops = &main_page_ops,
+};
+
+struct ui_page *ui_main_page_get(const struct ui_main_model *model,
+				 const char *date_text,
+				 const char *time_text)
+{
+	page_model = model;
+	page_date_text = date_text;
+	page_time_text = time_text;
+
+	return &main_page;
+}

src/ui/ui_main.h

@@ -7,16 +7,24 @@
 #include <lvgl.h>
 
 #include "mode_switch_event.h"
+#include "ui/ui_page.h"
 
 #ifdef __cplusplus
 extern "C" {
 #endif
 
+enum ui_ble_link_state {
+	UI_BLE_LINK_HIDDEN = 0,
+	UI_BLE_LINK_SEARCHING,
+	UI_BLE_LINK_CONNECTED,
+};
+
 struct ui_main_model {
 	lv_color_t theme_color;
 	lv_color_t inactive_border_color;
 	uint8_t battery_level;
 	enum mode_switch_mode mode;
+	enum ui_ble_link_state ble_link_state;
 	uint8_t led_mask;
 	bool charging;
 	bool full;
@@ -31,6 +39,10 @@ void ui_main_refresh_all(const struct ui_main_model *model,
 void ui_main_refresh_status_bar(const struct ui_main_model *model);
 void ui_main_refresh_battery(const struct ui_main_model *model);
 void ui_main_refresh_datetime(const char *date_text, const char *time_text);
+void ui_main_deinit(void);
+struct ui_page *ui_main_page_get(const struct ui_main_model *model,
+				 const char *date_text,
+				 const char *time_text);
 
 #ifdef __cplusplus
 }

src/ui/ui_settings.c

@@ -0,0 +1,233 @@
+#include <string.h>
+
+#include <lvgl.h>
+
+#include "ui_settings.h"
+
+#define UI_SETTINGS_LIST_H 104
+#define UI_SETTINGS_MAX_ITEMS 8U
+
+struct ui_settings_row {
+	lv_obj_t *row;
+	lv_obj_t *icon;
+	lv_obj_t *title;
+	lv_obj_t *value;
+	lv_obj_t *custom;
+};
+
+struct ui_settings_ctx {
+	lv_obj_t *content;
+	lv_obj_t *title;
+	lv_obj_t *hint;
+	lv_obj_t *scroll;
+	lv_obj_t *value;
+	struct ui_settings_row rows[UI_SETTINGS_MAX_ITEMS];
+	lv_group_t *focus_group;
+	uint8_t row_count;
+	struct ui_settings_page *page;
+	bool initialized;
+};
+
+static struct ui_settings_ctx g_ui;
+
+static void row_clear_custom(struct ui_settings_row *row)
+{
+	lv_obj_clean(row->custom);
+	lv_obj_set_style_bg_opa(row->custom, LV_OPA_TRANSP, 0);
+	lv_obj_set_style_bg_color(row->custom, lv_color_hex(0x0B1017), 0);
+}
+
+static void create_row(struct ui_settings_row *row)
+{
+	row->row = lv_obj_create(g_ui.scroll);
+	lv_obj_remove_style_all(row->row);
+	lv_obj_set_width(row->row, LV_PCT(100));
+	lv_obj_set_height(row->row, 30);
+	lv_obj_set_style_bg_opa(row->row, LV_OPA_TRANSP, 0);
+	lv_obj_set_style_radius(row->row, 9, 0);
+	lv_obj_set_style_border_width(row->row, 2, 0);
+	lv_obj_set_style_border_color(row->row, lv_color_hex(0x0B1017), 0);
+	lv_obj_set_style_pad_hor(row->row, 6, 0);
+	lv_obj_set_layout(row->row, LV_LAYOUT_FLEX);
+	lv_obj_set_flex_flow(row->row, LV_FLEX_FLOW_ROW);
+	lv_obj_set_flex_align(row->row, LV_FLEX_ALIGN_START,
+			      LV_FLEX_ALIGN_CENTER, LV_FLEX_ALIGN_CENTER);
+	lv_obj_set_style_pad_column(row->row, 6, 0);
+	lv_obj_add_flag(row->row, LV_OBJ_FLAG_SCROLL_ON_FOCUS);
+	lv_group_add_obj(g_ui.focus_group, row->row);
+
+	row->icon = lv_label_create(row->row);
+	lv_obj_set_width(row->icon, 20);
+	lv_obj_set_style_text_font(row->icon, &lv_font_montserrat_14, 0);
+	lv_obj_set_style_text_color(row->icon, lv_color_hex(0x8A95A5), 0);
+
+	row->title = lv_label_create(row->row);
+	lv_obj_set_width(row->title, 118);
+	lv_obj_set_style_text_font(row->title, &lv_font_montserrat_14, 0);
+	lv_obj_set_style_text_color(row->title, lv_color_hex(0xD8DEE9), 0);
+	lv_label_set_long_mode(row->title, LV_LABEL_LONG_CLIP);
+
+	row->value = lv_label_create(row->row);
+	lv_obj_set_flex_grow(row->value, 1);
+	lv_obj_set_style_text_font(row->value, &lv_font_montserrat_14, 0);
+	lv_obj_set_style_text_color(row->value, lv_color_hex(0x97A3B5), 0);
+	lv_obj_set_style_text_align(row->value, LV_TEXT_ALIGN_LEFT, 0);
+	lv_label_set_long_mode(row->value, LV_LABEL_LONG_SCROLL);
+
+	row->custom = lv_obj_create(row->row);
+	lv_obj_remove_style_all(row->custom);
+	lv_obj_set_size(row->custom, 16, 16);
+	row_clear_custom(row);
+}
+
+static void rebuild_rows(uint8_t count)
+{
+	if (count > UI_SETTINGS_MAX_ITEMS) {
+		count = UI_SETTINGS_MAX_ITEMS;
+	}
+
+	if (g_ui.row_count == count) {
+		return;
+	}
+
+	lv_obj_clean(g_ui.scroll);
+	memset(g_ui.rows, 0, sizeof(g_ui.rows));
+	g_ui.row_count = count;
+	lv_group_delete(g_ui.focus_group);
+	g_ui.focus_group = lv_group_create();
+
+	for (uint8_t i = 0; i < g_ui.row_count; i++) {
+		create_row(&g_ui.rows[i]);
+	}
+}
+
+static void row_set(struct ui_settings_row *row,
+		    const struct ui_settings_item *item,
+		    bool selected,
+		    lv_color_t accent)
+{
+	lv_label_set_text(row->icon, item->icon ? item->icon : "");
+	lv_label_set_text(row->title, item->title ? item->title : "");
+	lv_label_set_text(row->value, item->value ? item->value : "");
+	lv_obj_set_style_border_color(row->row,
+				      selected ? accent : lv_color_hex(0x0B1017),
+				      0);
+
+	row_clear_custom(row);
+	if (item->draw != NULL) {
+		item->draw(item, row->custom);
+	}
+}
+
+static void focus_row(uint8_t index, bool animate)
+{
+	if (index >= g_ui.row_count) {
+		return;
+	}
+
+	lv_obj_update_layout(g_ui.scroll);
+	lv_group_focus_obj(g_ui.rows[index].row);
+	lv_obj_scroll_to_view(g_ui.rows[index].row,
+			      animate ? LV_ANIM_ON : LV_ANIM_OFF);
+}
+
+void ui_settings_init(void)
+{
+	lv_obj_t *screen = lv_screen_active();
+
+	if (g_ui.initialized) {
+		return;
+	}
+
+	memset(&g_ui, 0, sizeof(g_ui));
+	g_ui.focus_group = lv_group_create();
+
+	g_ui.content = lv_obj_create(screen);
+	lv_obj_remove_style_all(g_ui.content);
+	lv_obj_set_size(g_ui.content, LV_PCT(100), LV_PCT(100));
+	lv_obj_set_style_bg_color(g_ui.content, lv_color_hex(0x0B1017), 0);
+	lv_obj_set_style_bg_opa(g_ui.content, LV_OPA_COVER, 0);
+	lv_obj_set_style_pad_left(g_ui.content, 14, 0);
+	lv_obj_set_style_pad_right(g_ui.content, 14, 0);
+	lv_obj_set_style_pad_top(g_ui.content, 8, 0);
+	lv_obj_set_style_pad_bottom(g_ui.content, 8, 0);
+	lv_obj_set_layout(g_ui.content, LV_LAYOUT_FLEX);
+	lv_obj_set_flex_flow(g_ui.content, LV_FLEX_FLOW_COLUMN);
+	lv_obj_set_flex_align(g_ui.content, LV_FLEX_ALIGN_START,
+			      LV_FLEX_ALIGN_START, LV_FLEX_ALIGN_START);
+	lv_obj_set_style_pad_row(g_ui.content, 6, 0);
+
+	g_ui.title = lv_label_create(g_ui.content);
+	lv_obj_set_style_text_font(g_ui.title, &lv_font_montserrat_32, 0);
+	lv_obj_set_style_text_color(g_ui.title, lv_color_white(), 0);
+
+	g_ui.hint = lv_label_create(g_ui.content);
+	lv_obj_set_style_text_font(g_ui.hint, &lv_font_montserrat_14, 0);
+	lv_obj_set_style_text_color(g_ui.hint, lv_color_hex(0x97A3B5), 0);
+
+	g_ui.scroll = lv_obj_create(g_ui.content);
+	lv_obj_remove_style_all(g_ui.scroll);
+	lv_obj_set_width(g_ui.scroll, LV_PCT(100));
+	lv_obj_set_height(g_ui.scroll, UI_SETTINGS_LIST_H);
+	lv_obj_set_style_bg_opa(g_ui.scroll, LV_OPA_TRANSP, 0);
+	lv_obj_set_layout(g_ui.scroll, LV_LAYOUT_FLEX);
+	lv_obj_set_flex_flow(g_ui.scroll, LV_FLEX_FLOW_COLUMN);
+	lv_obj_set_flex_align(g_ui.scroll, LV_FLEX_ALIGN_START,
+			      LV_FLEX_ALIGN_START, LV_FLEX_ALIGN_START);
+	lv_obj_set_style_pad_row(g_ui.scroll, 6, 0);
+	lv_obj_set_scroll_dir(g_ui.scroll, LV_DIR_VER);
+	lv_obj_set_scrollbar_mode(g_ui.scroll, LV_SCROLLBAR_MODE_OFF);
+
+	g_ui.value = lv_label_create(g_ui.content);
+	lv_obj_set_width(g_ui.value, LV_PCT(100));
+	lv_obj_set_style_text_font(g_ui.value, &lv_font_montserrat_14, 0);
+	lv_obj_set_style_text_color(g_ui.value, lv_color_hex(0x97A3B5), 0);
+
+	g_ui.initialized = true;
+}
+
+void ui_settings_show(struct ui_settings_page *page, bool animate)
+{
+	uint8_t count;
+	lv_color_t accent = lv_color_hex(0x4C9EF5);
+
+	if ((page == NULL) || !g_ui.initialized ||
+	    (page->ops == NULL) || (page->ops->get_count == NULL) ||
+	    (page->ops->get_item == NULL)) {
+		return;
+	}
+
+	count = page->ops->get_count(page);
+	rebuild_rows(count);
+	g_ui.page = page;
+
+	lv_label_set_text(g_ui.title, page->title ? page->title : "");
+	lv_label_set_text(g_ui.hint, page->hint ? page->hint : "");
+	lv_label_set_text(g_ui.value, "");
+
+	for (uint8_t i = 0; i < g_ui.row_count; i++) {
+		struct ui_settings_item item = { 0 };
+
+		page->ops->get_item(page, i, &item);
+		row_set(&g_ui.rows[i], &item, page->selected == i, accent);
+	}
+
+	focus_row(page->selected, animate);
+}
+
+void ui_settings_deinit(void)
+{
+	if (!g_ui.initialized) {
+		return;
+	}
+
+	if (g_ui.focus_group != NULL) {
+		lv_group_delete(g_ui.focus_group);
+	}
+
+	if (g_ui.content != NULL) {
+		lv_obj_delete(g_ui.content);
+	}
+
+	memset(&g_ui, 0, sizeof(g_ui));
+}

src/ui/ui_settings.h

@@ -0,0 +1,18 @@
+#ifndef BLINKY_UI_SETTINGS_H_
+#define BLINKY_UI_SETTINGS_H_
+
+#include "ui/ui_settings_page.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+void ui_settings_init(void);
+void ui_settings_show(struct ui_settings_page *page, bool animate);
+void ui_settings_deinit(void);
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* BLINKY_UI_SETTINGS_H_ */

src/ui/ui_settings_ble.c

@@ -0,0 +1,60 @@
+#include <lvgl.h>
+
+#include <zephyr/sys/util.h>
+
+#include "ui/ui_settings_controller.h"
+
+static uint8_t ble_get_count(struct ui_settings_page *page)
+{
+	ARG_UNUSED(page);
+	return 4U;
+}
+
+static void ble_get_item(struct ui_settings_page *page, uint8_t index,
+			 struct ui_settings_item *item)
+{
+	ARG_UNUSED(page);
+
+	if (index < 3U) {
+		static const char *const titles[] = {
+			"Slot 1",
+			"Slot 2",
+			"Slot 3",
+		};
+
+		item->icon = LV_SYMBOL_BLUETOOTH;
+		item->title = titles[index];
+		item->value = ui_settings_ble_slot_label(index);
+		return;
+	}
+
+	item->icon = LV_SYMBOL_TRASH;
+	item->title = "Erase Bond";
+	item->value = ui_settings_ble_current_label();
+}
+
+static void ble_on_select(struct ui_settings_page *page, uint8_t index)
+{
+	if (index < 3U) {
+		ui_settings_ble_select_slot(index);
+	} else {
+		ui_settings_ble_erase_current();
+	}
+
+	(void)ui_settings_controller_back();
+}
+
+static const struct ui_settings_page_ops ble_ops = {
+	.get_count = ble_get_count,
+	.get_item = ble_get_item,
+	.on_select = ble_on_select,
+};
+
+struct ui_settings_page ui_settings_ble_page = {
+	.base = {
+		.ops = &ble_ops.base,
+	},
+	.ops = &ble_ops,
+	.title = LV_SYMBOL_BLUETOOTH " Bluetooth",
+	.hint = "Short: select and return",
+};

src/ui/ui_settings_controller.c

@@ -0,0 +1,226 @@
+#include <stdbool.h>
+#include <stddef.h>
+#include <stdint.h>
+#include <string.h>
+
+#include <zephyr/sys/util.h>
+
+#include "ble_bond_multi_event.h"
+#include "ui/ui_settings_controller.h"
+#include "settings_view_event.h"
+#include "ui_settings_pages.h"
+
+#define BLE_SLOT_COUNT 3U
+
+int ble_bond_multi_select_slot(uint8_t slot);
+int ble_bond_multi_erase_current_slot(void);
+
+struct controller_ctx {
+	struct ui_settings_page *current;
+	bool active;
+	uint8_t active_ble_slot;
+	char ble_labels[BLE_SLOT_COUNT][BLE_BOND_MULTI_DISPLAY_NAME_MAX_LEN];
+	struct theme_rgb theme;
+};
+
+static struct controller_ctx ctx = {
+	.active_ble_slot = 0U,
+	.ble_labels = {
+		"Empty",
+		"Empty",
+		"Empty",
+	},
+	.theme = {
+		.r = BLINKY_THEME_DEFAULT_R,
+		.g = BLINKY_THEME_DEFAULT_G,
+		.b = BLINKY_THEME_DEFAULT_B,
+	},
+};
+
+static uint8_t wrap_index(uint8_t current, uint8_t count, int8_t delta)
+{
+	int32_t next = current;
+
+	if (count == 0U) {
+		return 0U;
+	}
+
+	next += delta;
+	while (next < 0) {
+		next += count;
+	}
+
+	return (uint8_t)(next % count);
+}
+
+static void publish_view(bool animate)
+{
+	if (ctx.active && (ctx.current != NULL)) {
+		submit_settings_view_event(ctx.current, animate);
+	}
+}
+
+void ui_settings_controller_switch_to(struct ui_settings_page *page,
+				      struct ui_page *parent)
+{
+	if (page == NULL) {
+		return;
+	}
+
+	if (ctx.current != NULL) {
+		ui_page_deinit(&ctx.current->base);
+	}
+
+	page->base.parent = parent;
+	ctx.current = page;
+	ui_page_init(&page->base);
+	if ((page->ops != NULL) && (page->ops->on_enter != NULL)) {
+		page->ops->on_enter(page);
+	}
+}
+
+void ui_settings_controller_open(void)
+{
+	ctx.active = true;
+	ui_settings_controller_switch_to(&ui_settings_root_page, NULL);
+	publish_view(false);
+}
+
+void ui_settings_controller_close(void)
+{
+	if (ctx.current != NULL) {
+		ui_page_deinit(&ctx.current->base);
+		ctx.current = NULL;
+	}
+
+	ctx.active = false;
+}
+
+bool ui_settings_controller_back(void)
+{
+	struct ui_page *parent;
+
+	if (ctx.current == NULL) {
+		return false;
+	}
+
+	if ((ctx.current->ops != NULL) && (ctx.current->ops->on_back != NULL)) {
+		ctx.current->ops->on_back(ctx.current);
+		publish_view(true);
+		return ctx.active;
+	}
+
+	parent = ctx.current->base.parent;
+	if (parent == NULL) {
+		ui_settings_controller_close();
+		return false;
+	}
+
+	ui_settings_controller_switch_to(ui_page_to_settings(parent),
+					 parent->parent);
+	publish_view(true);
+	return true;
+}
+
+void ui_settings_controller_select(void)
+{
+	if ((ctx.current == NULL) || (ctx.current->ops == NULL) ||
+	    (ctx.current->ops->on_select == NULL)) {
+		return;
+	}
+
+	ctx.current->ops->on_select(ctx.current, ctx.current->selected);
+	publish_view(true);
+}
+
+void ui_settings_controller_move(int8_t delta)
+{
+	uint8_t count;
+
+	if ((ctx.current == NULL) || (ctx.current->ops == NULL) ||
+	    (ctx.current->ops->get_count == NULL)) {
+		return;
+	}
+
+	count = ctx.current->ops->get_count(ctx.current);
+	ctx.current->selected = wrap_index(ctx.current->selected, count, delta);
+	publish_view(true);
+}
+
+void ui_settings_controller_refresh(bool animate)
+{
+	if (ctx.current != NULL) {
+		publish_view(animate);
+	}
+}
+
+bool ui_settings_controller_is_active(void)
+{
+	return ctx.active;
+}
+
+const char *ui_settings_ble_current_label(void)
+{
+	return (ctx.active_ble_slot == 0U) ? "Slot 1" :
+	       (ctx.active_ble_slot == 1U) ? "Slot 2" : "Slot 3";
+}
+
+void ui_settings_ble_select_slot(uint8_t slot)
+{
+	if (slot < BLE_SLOT_COUNT) {
+		(void)ble_bond_multi_select_slot(slot + 1U);
+	}
+}
+
+void ui_settings_ble_erase_current(void)
+{
+	(void)ble_bond_multi_erase_current_slot();
+}
+
+const char *ui_settings_ble_slot_label(uint8_t slot)
+{
+	if (slot >= BLE_SLOT_COUNT) {
+		return "";
+	}
+
+	return ctx.ble_labels[slot];
+}
+
+void ui_settings_ble_set_current_slot(uint8_t slot)
+{
+	if ((slot >= 1U) && (slot <= BLE_SLOT_COUNT)) {
+		ctx.active_ble_slot = slot - 1U;
+	}
+}
+
+void ui_settings_ble_set_slot_meta(uint8_t slot,
+				   const struct ble_bond_multi_slot_meta *meta)
+{
+	if ((meta == NULL) || (slot < 1U) || (slot > BLE_SLOT_COUNT)) {
+		return;
+	}
+
+	strncpy(ctx.ble_labels[slot - 1U],
+		meta->display_name[0] ? meta->display_name : "Empty",
+		sizeof(ctx.ble_labels[slot - 1U]));
+	ctx.ble_labels[slot - 1U][sizeof(ctx.ble_labels[slot - 1U]) - 1U] = '\0';
+}
+
+void ui_settings_theme_set_current(struct theme_rgb theme)
+{
+	ctx.theme = theme;
+}
+
+const char *ui_settings_theme_current_name(void)
+{
+	extern const char *ui_settings_theme_name_for_color(struct theme_rgb theme);
+
+	return ui_settings_theme_name_for_color(ctx.theme);
+}
+
+uint8_t ui_settings_theme_current_index(void)
+{
+	extern uint8_t ui_settings_theme_index_for_color(struct theme_rgb theme);
+
+	return ui_settings_theme_index_for_color(ctx.theme);
+}

src/ui/ui_settings_pages.h

@@ -0,0 +1,10 @@
+#ifndef BLINKY_UI_SETTINGS_PAGES_H_
+#define BLINKY_UI_SETTINGS_PAGES_H_
+
+#include "ui/ui_settings_page.h"
+
+extern struct ui_settings_page ui_settings_root_page;
+extern struct ui_settings_page ui_settings_ble_page;
+extern struct ui_settings_page ui_settings_theme_page;
+
+#endif /* BLINKY_UI_SETTINGS_PAGES_H_ */

src/ui/ui_settings_root.c

@@ -0,0 +1,64 @@
+#include <lvgl.h>
+
+#include <zephyr/sys/util.h>
+
+#include "ui/ui_settings_controller.h"
+#include "ui_settings_pages.h"
+
+static uint8_t root_get_count(struct ui_settings_page *page)
+{
+	ARG_UNUSED(page);
+	return 2U;
+}
+
+static void root_get_item(struct ui_settings_page *page, uint8_t index,
+			  struct ui_settings_item *item)
+{
+	ARG_UNUSED(page);
+
+	if (index == 0U) {
+		item->icon = LV_SYMBOL_BLUETOOTH;
+		item->title = "Bluetooth";
+		item->value = ui_settings_ble_current_label();
+		return;
+	}
+
+	item->icon = LV_SYMBOL_TINT;
+	item->title = "Theme";
+	item->value = ui_settings_theme_current_name();
+}
+
+static void root_on_enter(struct ui_settings_page *page)
+{
+	page->selected = 0U;
+}
+
+static void root_on_select(struct ui_settings_page *page, uint8_t index)
+{
+	ui_settings_controller_switch_to(
+		(index == 0U) ? &ui_settings_ble_page : &ui_settings_theme_page,
+		&page->base);
+}
+
+static void root_on_back(struct ui_settings_page *page)
+{
+	ARG_UNUSED(page);
+	ui_settings_controller_close();
+}
+
+static const struct ui_settings_page_ops root_ops = {
+	.get_count = root_get_count,
+	.get_item = root_get_item,
+	.on_enter = root_on_enter,
+	.on_select = root_on_select,
+	.on_back = root_on_back,
+};
+
+struct ui_settings_page ui_settings_root_page = {
+	.base = {
+		.ops = &root_ops.base,
+	},
+	.ops = &root_ops,
+	.title = LV_SYMBOL_SETTINGS " Settings",
+	.hint = "Rotate select  Tap OK  Hold exit",
+};

src/ui/ui_settings_theme.c

@@ -0,0 +1,109 @@
+#include <lvgl.h>
+
+#include <zephyr/sys/util.h>
+
+#include "theme_rgb_update_event.h"
+#include "ui/ui_settings_controller.h"
+
+struct theme_option {
+	const char *name;
+	struct theme_rgb color;
+};
+
+static const struct theme_option themes[] = {
+	{ "Red", { .r = 0xFF, .g = 0x00, .b = 0x00 } },
+	{ "Amber", { .r = 0xFF, .g = 0x95, .b = 0x00 } },
+	{ "Default", { .r = BLINKY_THEME_DEFAULT_R,
+		       .g = BLINKY_THEME_DEFAULT_G,
+		       .b = BLINKY_THEME_DEFAULT_B } },
+	{ "Green", { .r = 0x34, .g = 0xC7, .b = 0x59 } },
+	{ "Purple", { .r = 0xBF, .g = 0x5A, .b = 0xF2 } },
+	{ "White", { .r = 0xF2, .g = 0xF2, .b = 0xF7 } },
+};
+
+static bool theme_equal(struct theme_rgb lhs, struct theme_rgb rhs)
+{
+	return (lhs.r == rhs.r) && (lhs.g == rhs.g) && (lhs.b == rhs.b);
+}
+
+uint8_t ui_settings_theme_index_for_color(struct theme_rgb theme)
+{
+	for (uint8_t i = 0; i < ARRAY_SIZE(themes); i++) {
+		if (theme_equal(theme, themes[i].color)) {
+			return i;
+		}
+	}
+
+	return 0U;
+}
+
+const char *ui_settings_theme_name_for_color(struct theme_rgb theme)
+{
+	for (uint8_t i = 0; i < ARRAY_SIZE(themes); i++) {
+		if (theme_equal(theme, themes[i].color)) {
+			return themes[i].name;
+		}
+	}
+
+	return "Custom";
+}
+
+static void draw_swatch(const struct ui_settings_item *item, lv_obj_t *container)
+{
+	const struct theme_rgb *color = item->user_data;
+
+	if (color == NULL) {
+		return;
+	}
+
+	lv_obj_set_style_bg_opa(container, LV_OPA_COVER, 0);
+	lv_obj_set_style_bg_color(container,
+				  lv_color_make(color->r, color->g, color->b), 0);
+}
+
+static uint8_t theme_get_count(struct ui_settings_page *page)
+{
+	ARG_UNUSED(page);
+	return ARRAY_SIZE(themes);
+}
+
+static void theme_get_item(struct ui_settings_page *page, uint8_t index,
+			   struct ui_settings_item *item)
+{
+	ARG_UNUSED(page);
+
+	item->icon = LV_SYMBOL_TINT;
+	item->title = themes[index].name;
+	item->draw = draw_swatch;
+	item->user_data = (void *)&themes[index].color;
+}
+
+static void theme_on_enter(struct ui_settings_page *page)
+{
+	page->selected = ui_settings_theme_current_index();
+}
+
+static void theme_on_select(struct ui_settings_page *page, uint8_t index)
+{
+	struct theme_rgb theme = themes[index].color;
+
+	ui_settings_theme_set_current(theme);
+	submit_theme_rgb_update_event(theme);
+	(void)ui_settings_controller_back();
+}
+
+static const struct ui_settings_page_ops theme_ops = {
+	.get_count = theme_get_count,
+	.get_item = theme_get_item,
+	.on_enter = theme_on_enter,
+	.on_select = theme_on_select,
+};
+
+struct ui_settings_page ui_settings_theme_page = {
+	.base = {
+		.ops = &theme_ops.base,
+	},
+	.ops = &theme_ops,
+	.title = LV_SYMBOL_TINT " Theme",
+	.hint = "Short: apply and return",
+};

src/usb_cdc_module.c

@@ -28,8 +28,7 @@ LOG_MODULE_REGISTER(MODULE, LOG_LEVEL_INF);
 #define USB_CDC_RX_RING_BUF_SIZE 256
 #define USB_CDC_TX_RING_BUF_SIZE 256
 #define USB_CDC_RX_CHUNK_SIZE 32
-#define USB_CDC_PROTO_RX_BUF_SIZE 128
-#define USB_CDC_PROTO_RX_FLUSH_DELAY K_MSEC(3)
+#define USB_CDC_PROTO_RX_BUF_SIZE PROTO_MAX_FRAME_LEN
 #define USB_CDC_EXPECTED_BAUDRATE 115200U
 
 enum usb_cdc_business_state {
@@ -47,7 +46,6 @@ struct usb_cdc_ctx {
 	struct ring_buf rx_ringbuf;
 	struct ring_buf tx_ringbuf;
 	struct k_work rx_work;
-	struct k_work_delayable rx_flush_work;
 	uint8_t proto_rx_buf[USB_CDC_PROTO_RX_BUF_SIZE];
 	bool usb_active;
 	size_t proto_rx_len;
@@ -74,15 +72,39 @@ static struct usb_cdc_ctx ctx = {
 		.ops = &lifecycle_ops,
 	},
 	.business = USB_CDC_BUS_OFFLINE,
-	.cdc_dev = DEVICE_DT_GET_ONE(zephyr_cdc_acm_uart),
+	.cdc_dev = DEVICE_DT_GET(DT_NODELABEL(cdc_acm_uart0)),
 	.usb_active = false,
 	.proto_rx_len = 0U,
 };
 
-#define proto_rx_buf ctx.proto_rx_buf
-
 static void validate_line_coding(void);
 
+static const char *usb_cdc_business_state_name(enum usb_cdc_business_state state)
+{
+	switch (state) {
+	case USB_CDC_BUS_OFFLINE:
+		return "BUS_OFFLINE";
+	case USB_CDC_WAIT_DTR:
+		return "WAIT_DTR";
+	case USB_CDC_SESSION_READY:
+		return "SESSION_READY";
+	default:
+		return "?";
+	}
+}
+
+static const char *proto_link_state_name(enum proto_transport_link_state state)
+{
+	switch (state) {
+	case PROTO_TRANSPORT_LINK_DOWN:
+		return "DOWN";
+	case PROTO_TRANSPORT_LINK_READY:
+		return "READY";
+	default:
+		return "?";
+	}
+}
+
 static bool lifecycle_is_ready(void)
 {
 	return module_lifecycle_is_running(&ctx.lc);
@@ -111,7 +133,6 @@ static void disable_uart_io(void)
 	uart_irq_rx_disable(ctx.cdc_dev);
 	uart_irq_tx_disable(ctx.cdc_dev);
 	ctx.proto_rx_len = 0U;
-	k_work_cancel_delayable(&ctx.rx_flush_work);
 	reset_ring_buffers();
 }
 
@@ -128,6 +149,7 @@ static void state_reconcile(enum module_lifecycle old_lifecycle,
 	if ((old_lifecycle == LC_RUNNING) &&
 	    (old_business == USB_CDC_SESSION_READY) &&
 	    (new_link == PROTO_TRANSPORT_LINK_DOWN)) {
+		LOG_INF("CDC reconcile: link READY -> DOWN, disable UART IO");
 		disable_uart_io();
 	}
 
@@ -135,6 +157,9 @@ static void state_reconcile(enum module_lifecycle old_lifecycle,
 	    (new_link == PROTO_TRANSPORT_LINK_READY)) {
 		int err;
 
+		LOG_INF("CDC reconcile: link DOWN -> READY (business:%s usb_active:%u lc:%s)",
+			usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+			module_lifecycle_name(ctx.lc.state));
 		validate_line_coding();
 
 		err = uart_line_ctrl_set(ctx.cdc_dev, UART_LINE_CTRL_DCD, 1);
@@ -151,6 +176,11 @@ static void state_reconcile(enum module_lifecycle old_lifecycle,
 	}
 
 	if (old_link != new_link) {
+		LOG_INF("CDC link %s -> %s (business:%s usb_active:%u lc:%s)",
+			proto_link_state_name(old_link),
+			proto_link_state_name(new_link),
+			usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+			module_lifecycle_name(ctx.lc.state));
 		submit_proto_transport_state_event(PROTO_TRANSPORT_USB_CDC,
 						   new_link);
 	}
@@ -166,11 +196,19 @@ static void business_state_set(enum usb_cdc_business_state new_state)
 	}
 
 	ctx.business = new_state;
+	LOG_INF("CDC business %s -> %s (usb_active:%u lc:%s)",
+		usb_cdc_business_state_name(old_business),
+		usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+		module_lifecycle_name(ctx.lc.state));
 	state_reconcile(old_lifecycle, old_business);
 }
 
 static void business_state_sync_from_usb(void)
 {
+	LOG_INF("CDC sync from USB (usb_active:%u business:%s lc:%s)",
+		ctx.usb_active, usb_cdc_business_state_name(ctx.business),
+		module_lifecycle_name(ctx.lc.state));
+
 	if (!ctx.usb_active) {
 		business_state_set(USB_CDC_BUS_OFFLINE);
 		return;
@@ -188,9 +226,14 @@ static void business_state_sync_from_usb(void)
 static void kick_tx(void)
 {
 	if (transport_link_state_get() != PROTO_TRANSPORT_LINK_READY) {
+		LOG_INF("CDC kick_tx skipped: link=%s business:%s usb_active:%u tx_rb:%u",
+			proto_link_state_name(transport_link_state_get()),
+			usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+			ring_buf_size_get(&ctx.tx_ringbuf));
 		return;
 	}
 
+	LOG_INF("CDC kick_tx: enable TX IRQ (tx_rb:%u)", ring_buf_size_get(&ctx.tx_ringbuf));
 	uart_irq_tx_enable(ctx.cdc_dev);
 }
 
@@ -232,6 +275,46 @@ static void validate_line_coding(void)
 #endif
 }
 
+static bool try_extract_frame(void)
+{
+	uint8_t *buf = ctx.proto_rx_buf;
+	size_t frame_len;
+	uint16_t magic;
+
+	if (ctx.proto_rx_len < PROTO_FRAME_HEADER_SIZE) {
+		return false;
+	}
+
+	magic = (uint16_t)buf[0] | ((uint16_t)buf[1] << 8);
+	if (magic != PROTO_FRAME_MAGIC) {
+		LOG_WRN("CDC invalid frame magic 0x%04x", magic);
+		memmove(buf, &buf[1], ctx.proto_rx_len - 1U);
+		ctx.proto_rx_len--;
+		return true;
+	}
+
+	if (buf[2] > PROTO_MAX_PAYLOAD_LEN) {
+		LOG_WRN("CDC invalid frame len:%u", buf[2]);
+		memmove(buf, &buf[1], ctx.proto_rx_len - 1U);
+		ctx.proto_rx_len--;
+		return true;
+	}
+
+	frame_len = PROTO_FRAME_HEADER_SIZE + buf[2];
+	if (ctx.proto_rx_len < frame_len) {
+		return false;
+	}
+
+	LOG_INF("CDC submit framed proto_rx len:%u", (unsigned int)frame_len);
+	(void)submit_proto_rx_event(PROTO_TRANSPORT_USB_CDC, buf, frame_len);
+
+	if (ctx.proto_rx_len > frame_len) {
+		memmove(buf, &buf[frame_len], ctx.proto_rx_len - frame_len);
+	}
+	ctx.proto_rx_len -= frame_len;
+	return true;
+}
+
 static void rx_work_handler(struct k_work *work)
 {
 	uint8_t buffer[USB_CDC_RX_CHUNK_SIZE];
@@ -242,39 +325,30 @@ static void rx_work_handler(struct k_work *work)
 		uint32_t len;
 		unsigned int key = irq_lock();
 
-			len = ring_buf_get(&ctx.rx_ringbuf, buffer, sizeof(buffer));
+		len = ring_buf_get(&ctx.rx_ringbuf, buffer, sizeof(buffer));
 		irq_unlock(key);
 
 		if (len == 0U) {
-			return;
+			break;
 		}
 
-		if ((ctx.proto_rx_len + len) > sizeof(proto_rx_buf)) {
-			LOG_WRN("Drop oversized CDC protobuf message len:%u",
+		LOG_INF("CDC rx_work pulled %u bytes from RX ring (proto_rx_len:%u)",
+			(unsigned int)len, (unsigned int)ctx.proto_rx_len);
+
+		if ((ctx.proto_rx_len + len) > sizeof(ctx.proto_rx_buf)) {
+			LOG_WRN("Drop oversized CDC framed data len:%u",
 				(uint32_t)(ctx.proto_rx_len + len));
 			ctx.proto_rx_len = 0U;
 		}
 
 		if (len > 0U) {
-			memcpy(&proto_rx_buf[ctx.proto_rx_len], buffer, len);
+			memcpy(&ctx.proto_rx_buf[ctx.proto_rx_len], buffer, len);
 			ctx.proto_rx_len += len;
-			k_work_reschedule(&ctx.rx_flush_work, USB_CDC_PROTO_RX_FLUSH_DELAY);
 		}
 	}
-}
 
-static void rx_flush_work_handler(struct k_work *work)
-{
-	ARG_UNUSED(work);
-
-	if ((transport_link_state_get() != PROTO_TRANSPORT_LINK_READY) ||
-	    (ctx.proto_rx_len == 0U)) {
-		return;
+	while (try_extract_frame()) {
 	}
-
-	(void)submit_proto_rx_event(PROTO_TRANSPORT_USB_CDC, proto_rx_buf,
-				    ctx.proto_rx_len);
-	ctx.proto_rx_len = 0U;
 }
 
 static void cdc_interrupt_handler(const struct device *dev, void *user_data)
@@ -304,6 +378,10 @@ static void cdc_interrupt_handler(const struct device *dev, void *user_data)
 						recv_len - (int)written);
 				}
 
+				LOG_INF("CDC IRQ RX recv:%d written:%u rx_rb:%u",
+					recv_len, (unsigned int)written,
+					ring_buf_size_get(&ctx.rx_ringbuf));
+
 				k_work_submit(&ctx.rx_work);
 			}
 		}
@@ -318,6 +396,7 @@ static void cdc_interrupt_handler(const struct device *dev, void *user_data)
 			irq_unlock(key);
 
 			if (len == 0U) {
+				LOG_INF("CDC IRQ TX ready but no pending data, disable TX IRQ");
 				uart_irq_tx_disable(dev);
 				continue;
 			}
@@ -330,6 +409,10 @@ static void cdc_interrupt_handler(const struct device *dev, void *user_data)
 				LOG_WRN("Drop %u CDC TX bytes",
 					(unsigned int)(len - (uint32_t)sent_len));
 			}
+
+			LOG_INF("CDC IRQ TX requested:%u sent:%d tx_rb_remain:%u",
+				(unsigned int)len, sent_len,
+				ring_buf_size_get(&ctx.tx_ringbuf));
 		}
 	}
 }
@@ -343,11 +426,13 @@ static int do_init(void)
 
 	reset_ring_buffers();
 	k_work_init(&ctx.rx_work, rx_work_handler);
-	k_work_init_delayable(&ctx.rx_flush_work, rx_flush_work_handler);
 	uart_irq_callback_set(ctx.cdc_dev, cdc_interrupt_handler);
 	ctx.business = USB_CDC_BUS_OFFLINE;
 	ctx.usb_active = false;
 	ctx.proto_rx_len = 0U;
+	LOG_INF("CDC init done (business:%s usb_active:%u lc:%s dev:%p)",
+		usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+		module_lifecycle_name(ctx.lc.state), ctx.cdc_dev);
 
 	return 0;
 }
@@ -359,6 +444,9 @@ static int do_start(void)
 
 static int do_stop(void)
 {
+	LOG_INF("CDC stop requested (business:%s usb_active:%u lc:%s)",
+		usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+		module_lifecycle_name(ctx.lc.state));
 	ctx.business = USB_CDC_BUS_OFFLINE;
 	return 0;
 }
@@ -379,6 +467,11 @@ static int apply_lifecycle(enum module_lifecycle target)
 		business_state_sync_from_usb();
 	}
 
+	LOG_INF("CDC lifecycle %s -> %s done (business:%s usb_active:%u link:%s)",
+		module_lifecycle_name(old_lifecycle), module_lifecycle_name(target),
+		usb_cdc_business_state_name(ctx.business), ctx.usb_active,
+		proto_link_state_name(transport_link_state_get()));
+
 	return 0;
 }
 
@@ -391,6 +484,10 @@ static bool handle_usb_state_event(const struct usb_state_event *event)
 	}
 
 	ctx.usb_active = new_usb_active;
+	LOG_INF("CDC usb_active -> %u from usb_state:%d (business:%s lc:%s)",
+		ctx.usb_active, event->state,
+		usb_cdc_business_state_name(ctx.business),
+		module_lifecycle_name(ctx.lc.state));
 	business_state_sync_from_usb();
 
 	return false;
@@ -404,7 +501,13 @@ static bool handle_usb_control_event(const struct usb_control_event *event)
 
 	switch (event->type) {
 	case USB_CONTROL_EVENT_CDC_LINE_STATE:
+		LOG_INF("CDC control event line_state dtr:%u usb_active:%u lc:%s business:%s",
+			event->data.cdc_line_state.dtr, ctx.usb_active,
+			module_lifecycle_name(ctx.lc.state),
+			usb_cdc_business_state_name(ctx.business));
 		if (!ctx.usb_active || !lifecycle_is_ready()) {
+			LOG_INF("CDC line_state ignored: usb_active:%u lc:%s",
+				ctx.usb_active, module_lifecycle_name(ctx.lc.state));
 			return false;
 		}
 
@@ -419,6 +522,13 @@ static bool handle_usb_control_event(const struct usb_control_event *event)
 		return false;
 
 	case USB_CONTROL_EVENT_CDC_LINE_CODING:
+		LOG_INF("CDC control event line_coding baud:%u data:%u stop:%u parity:%u flow:%u business:%s",
+			event->data.cdc_line_coding.baudrate,
+			event->data.cdc_line_coding.data_bits,
+			event->data.cdc_line_coding.stop_bits,
+			event->data.cdc_line_coding.parity,
+			event->data.cdc_line_coding.flow_ctrl,
+			usb_cdc_business_state_name(ctx.business));
 		if (event->data.cdc_line_coding.baudrate != 0U) {
 			LOG_INF("CDC baudrate %u",
 				event->data.cdc_line_coding.baudrate);
@@ -459,6 +569,10 @@ static bool handle_proto_tx_event(const struct proto_tx_event *event)
 	}
 
 	if (transport_link_state_get() != PROTO_TRANSPORT_LINK_READY) {
+		LOG_INF("CDC proto_tx ignored len:%u link:%s business:%s usb_active:%u",
+			(unsigned int)event->dyndata.size,
+			proto_link_state_name(transport_link_state_get()),
+			usb_cdc_business_state_name(ctx.business), ctx.usb_active);
 		return false;
 	}
 
@@ -467,6 +581,10 @@ static bool handle_proto_tx_event(const struct proto_tx_event *event)
 			       (uint32_t)event->dyndata.size);
 	irq_unlock(key);
 
+	LOG_INF("CDC proto_tx queued len:%u written:%u tx_rb:%u",
+		(unsigned int)event->dyndata.size, (unsigned int)written,
+		ring_buf_size_get(&ctx.tx_ringbuf));
+
 	if (written < event->dyndata.size) {
 		LOG_WRN("Drop %zu CDC TX bytes", event->dyndata.size - written);
 	}

src/usb_device_module.c

@@ -88,6 +88,54 @@ static struct usb_owner_ctx usb_ctx = {
 	.bus = USB_BUS_DISCONNECTED,
 };
 
+static const char *usb_stack_state_name(enum usb_stack_state state)
+{
+	switch (state) {
+	case USB_STACK_UNINITIALIZED:
+		return "UNINITIALIZED";
+	case USB_STACK_READY:
+		return "READY";
+	case USB_STACK_ENABLED:
+		return "ENABLED";
+	default:
+		return "?";
+	}
+}
+
+static const char *usb_bus_state_name(enum usb_bus_state state)
+{
+	switch (state) {
+	case USB_BUS_DISCONNECTED:
+		return "DISCONNECTED";
+	case USB_BUS_POWERED:
+		return "POWERED";
+	case USB_BUS_ACTIVE:
+		return "ACTIVE";
+	case USB_BUS_SUSPENDED:
+		return "SUSPENDED";
+	default:
+		return "?";
+	}
+}
+
+static const char *usb_public_state_name(enum usb_state state)
+{
+	switch (state) {
+	case USB_STATE_DISABLED:
+		return "DISABLED";
+	case USB_STATE_DISCONNECTED:
+		return "DISCONNECTED";
+	case USB_STATE_POWERED:
+		return "POWERED";
+	case USB_STATE_ACTIVE:
+		return "ACTIVE";
+	case USB_STATE_SUSPENDED:
+		return "SUSPENDED";
+	default:
+		return "?";
+	}
+}
+
 static inline enum usb_state usb_public_state_get(void)
 {
 	if (!module_lifecycle_is_running(&usb_ctx.lc) ||
@@ -111,12 +159,21 @@ static inline enum usb_state usb_public_state_get(void)
 
 static inline void usb_bus_set(enum usb_bus_state state)
 {
+	enum usb_bus_state old_state = usb_ctx.bus;
+
 	if (usb_ctx.bus == state) {
 		return;
 	}
 
 	usb_ctx.bus = state;
 
+	LOG_INF("USB bus %s -> %s (stack:%s lc:%s public:%s)",
+		usb_bus_state_name(old_state),
+		usb_bus_state_name(usb_ctx.bus),
+		usb_stack_state_name(usb_ctx.stack),
+		module_lifecycle_name(usb_ctx.lc.state),
+		usb_public_state_name(usb_public_state_get()));
+
 	if (module_lifecycle_is_initialized(&usb_ctx.lc)) {
 		submit_usb_state(usb_public_state_get());
 	}
@@ -168,6 +225,12 @@ static void usbd_msg_cb(struct usbd_context *const usbd_ctx,
 {
 	ARG_UNUSED(usbd_ctx);
 
+	LOG_INF("USBD msg:%s status:%u dev:%p stack:%s bus:%s lc:%s",
+		usbd_msg_type_string(msg->type), msg->status, msg->dev,
+		usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus),
+		module_lifecycle_name(usb_ctx.lc.state));
+
 	if (msg->type == USBD_MSG_CDC_ACM_CONTROL_LINE_STATE) {
 		uint32_t dtr = 0U;
 		int err = uart_line_ctrl_get(msg->dev, UART_LINE_CTRL_DTR, &dtr);
@@ -175,6 +238,7 @@ static void usbd_msg_cb(struct usbd_context *const usbd_ctx,
 		if (err) {
 			LOG_WRN("Failed to get CDC DTR (%d)", err);
 		} else {
+			LOG_INF("CDC control line state DTR:%u dev:%p", dtr, msg->dev);
 			submit_usb_control_cdc_line_state_event(msg->dev, dtr != 0U);
 		}
 
@@ -210,6 +274,8 @@ static void usbd_msg_cb(struct usbd_context *const usbd_ctx,
 		}
 #endif
 
+		LOG_INF("CDC line coding baud:%u data:%u stop:%u parity:%u flow:%u dev:%p",
+			baudrate, data_bits, stop_bits, parity, flow_ctrl, msg->dev);
 		submit_usb_control_cdc_line_coding_event(msg->dev, baudrate,
 						 data_bits, stop_bits,
 						 parity, flow_ctrl);
@@ -280,6 +346,10 @@ static int do_init(void)
 
 	usb_ctx.stack = USB_STACK_UNINITIALIZED;
 	usb_ctx.bus = USB_BUS_DISCONNECTED;
+	LOG_INF("USB init start (stack:%s bus:%s lc:%s)",
+		usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus),
+		module_lifecycle_name(usb_ctx.lc.state));
 	update_power_manager_restriction(false);
 
 	STRUCT_SECTION_FOREACH(usb_function_hook, hook) {
@@ -313,6 +383,11 @@ static int do_init(void)
 	}
 
 	usb_ctx.stack = USB_STACK_READY;
+	LOG_INF("USB init done (stack:%s bus:%s lc:%s public:%s)",
+		usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus),
+		module_lifecycle_name(usb_ctx.lc.state),
+		usb_public_state_name(usb_public_state_get()));
 
 	return 0;
 }
@@ -321,7 +396,13 @@ static int do_start(void)
 {
 	int err;
 
+	LOG_INF("USB start requested (stack:%s bus:%s lc:%s)",
+		usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus),
+		module_lifecycle_name(usb_ctx.lc.state));
+
 	if (usb_ctx.stack == USB_STACK_ENABLED) {
+		LOG_INF("USB start ignored: already enabled");
 		return 0;
 	}
 
@@ -334,11 +415,15 @@ static int do_start(void)
 
 		usb_ctx.stack = USB_STACK_ENABLED;
 		usb_ctx.bus = USB_BUS_POWERED;
+		LOG_INF("USB enabled without VBUS detect (stack:%s bus:%s)",
+			usb_stack_state_name(usb_ctx.stack),
+			usb_bus_state_name(usb_ctx.bus));
 		update_power_manager_restriction(true);
 		return 0;
 	}
 
 	if (usb_ctx.bus == USB_BUS_DISCONNECTED) {
+		LOG_INF("USB start deferred: waiting for VBUS");
 		return 0;
 	}
 
@@ -349,6 +434,8 @@ static int do_start(void)
 	}
 
 	usb_ctx.stack = USB_STACK_ENABLED;
+	LOG_INF("USB enabled (stack:%s bus:%s)", usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus));
 	update_power_manager_restriction(true);
 	return 0;
 }
@@ -357,6 +444,11 @@ static int do_stop(void)
 {
 	int err;
 
+	LOG_INF("USB stop requested (stack:%s bus:%s lc:%s)",
+		usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus),
+		module_lifecycle_name(usb_ctx.lc.state));
+
 	if (usb_ctx.stack == USB_STACK_ENABLED) {
 		err = usbd_disable(&blinky_usbd);
 		if (err) {
@@ -366,6 +458,8 @@ static int do_stop(void)
 	}
 
 	usb_ctx.stack = USB_STACK_READY;
+	LOG_INF("USB stopped (stack:%s bus:%s)", usb_stack_state_name(usb_ctx.stack),
+		usb_bus_state_name(usb_ctx.bus));
 	update_power_manager_restriction(false);
 	return 0;
 }
@@ -391,7 +485,7 @@ static bool handle_module_suspend_req_event(
 		return false;
 	}
 
-	if (module_set_lifecycle(&usb_ctx.lc, LC_SUSPENDED) == 0) {
+	if (module_set_lifecycle(&usb_ctx.lc, LC_STOPPED) == 0) {
 		submit_usb_state(usb_public_state_get());
 	}
 
@@ -402,7 +496,7 @@ static bool handle_module_resume_req_event(
 	const struct module_resume_req_event *event)
 {
 	if ((event->sink_module_id != MODULE_ID(MODULE)) ||
-	    (usb_ctx.lc.state != LC_SUSPENDED)) {
+	    (usb_ctx.lc.state != LC_STOPPED)) {
 		return false;
 	}