Enable GX8002 VAD audio path for EVT1#2511
Closed
wangli65535 wants to merge 5 commits intodev-nexfirmware-evt1from
Closed
Enable GX8002 VAD audio path for EVT1#2511wangli65535 wants to merge 5 commits intodev-nexfirmware-evt1from
wangli65535 wants to merge 5 commits intodev-nexfirmware-evt1from
Conversation
…tics mos_i2s_slave.c — 3 bug fixes: - Fix pingpong buffer logic: replace inverted-index selection with (current+1)%2 to correctly alternate RX DMA buffers and eliminate aliasing noise caused by re-queuing the still-active buffer - Fix I2S SLAVE clock config: switch to PCLK32M / MCK_DISABLED / RATIO_32X (clock supplied by GX8002 master; MCK output must be off) - Fix channel count: GX8002_PDM_CHANNELS 2→1 (LC3 encodes mono; previous value caused the encoder to receive 2x too many samples) - Add LOG_ERR/LOG_WRN for LC3 encode failures and frame-size mismatches main.c — BLE connection diagnostics: - Add on_le_param_updated: logs connection interval (interval × 1.25 ms), latency, and supervision timeout on every parameter update event - Add on_le_phy_updated: logs PHY mode (1M / 2M / Long Range) - Add on_le_data_len_updated: logs TX/RX data length and air time - Register all three callbacks in BT_CONN_CB_DEFINE app.overlay: - Add pinctrl-0 / pinctrl-names to the disabled pdm0 node to satisfy Zephyr DTS validation requirements docs/voice_latency_customer.md (new): - Customer-facing end-to-end voice latency analysis document - Covers device-side breakdown (50 ms steady-state / 132 ms first-trigger), BLE scheduling, and phone-side playback buffer - Includes optimization proposals (frames 5→1, BLE interval, app buffer) with quantified latency reduction estimates
- Added functions to manage the welcome screen state, including `display_is_welcome_screen_active` and `display_reset_protobuf_text_state`. - Improved welcome screen initialization and state restoration logic. - Updated `display_show_welcome_screen` to handle command enqueue failures. - Introduced new functions to ensure the readiness of the protobuf scene and to clear current display text. - Enhanced the handling of protobuf text updates to respect the welcome screen state. - Added logging for better traceability of protobuf message processing. - Integrated I2S initialization for the GX8002 driver. - Updated protobuf handler to ignore empty DisplayText while the welcome screen is active.
- Disable CONFIG_NORDIC_QSPI_NOR_XIP to prevent execution in place. - Increase CONFIG_LV_Z_VDB_SIZE from 40 to 42 for improved rendering performance.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
📋 PR Review Helper📱 Mobile App Build⏳ Waiting for build... 🕶️ ASG Client Build⏳ Waiting for build... 🔀 Test Locallygh pr checkout 2511 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Testing
Notes
This PR is intended for EVT1 GX8002 VAD path testing.