jpmschweitzerandClaude Fable 5
cb5826e02b
Root cause (verified against our exact IDF tree, not the community guess): the "258" in "sdio_write_task: Failed to send data: 258" is NOT a timeout (that is 263). 258 = 0x102 = ESP_ERR_INVALID_ARG. On the ESP32-P4, block- mode CMD53 writes require the SOURCE buffer to be 64-byte (cache-line) aligned; the IDF sdmmc driver rejects a misaligned source with INVALID_ARG BEFORE any bus activity. esp_hosts write loop then declares "Unrecoverable host sdio state" and reboots the whole P4. The audio TX payload is not 64-aligned, so streaming mic audio wedged on the very FIRST frame (which is exactly what we saw: listening -> instant Failed to send -> reboot). This also explains why buffer/queue/clock/retry tuning all did nothing: the write never reached the bus. And why our symptom was instant, not after ~100 writes (the community block-mode-desync theory) — it is the first misaligned buffer, every time. Fix: vendored esp_hosted 2.12.11 as an editable local component (overrides the registry copy) and bounce a misaligned TX payload through one aligned DMA scratch buffer in hosted_sdio_write_block (port_esp_hosted_host_sdio.c). TX is serialized by the bus lock so a single static bounce buffer is safe; freed in hosted_sdio_deinit. Host-only change — no C6 reflash. VERIFIED ON HARDWARE (autonomous self-test): 40s of continuous mic-audio upstream streaming — the traffic that previously wedged on the first frame — ran clean, zero timeouts, zero reboots. A guarded SDIO_TX_SELFTEST harness is kept (compiled out) for future SDIO stress testing. Credit: root cause + patch designed via multi-agent investigation; the precise 258=INVALID_ARG decode (correcting the upstream community timeout assumption) came from checking our actual esp_err.h. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
100 lines
2.5 KiB
C
100 lines
2.5 KiB
C
/*
|
|
* SPDX-FileCopyrightText: 2015-2025 Espressif Systems (Shanghai) CO LTD
|
|
*
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
*/
|
|
|
|
/*prevent recursive inclusion */
|
|
#ifndef __SERIAL_DRV_H
|
|
#define __SERIAL_DRV_H
|
|
|
|
#ifdef __cplusplus
|
|
extern "C" {
|
|
#endif
|
|
|
|
/** includes **/
|
|
#include "serial_ll_if.h"
|
|
|
|
/** Exported Functions **/
|
|
/*
|
|
* rpc_platform_init function initializes the rpc
|
|
* path data structures
|
|
* Input parameter
|
|
* None
|
|
* Returns
|
|
* SUCCESS(0) or FAILURE(-1) of above operation
|
|
*/
|
|
int rpc_platform_init(void);
|
|
|
|
/*
|
|
* rpc_platform_deinit function cleans up the rpc
|
|
* path library data structure
|
|
* Input parameter
|
|
* None
|
|
* Returns
|
|
* SUCCESS(0) or FAILURE(-1) of above operation
|
|
*/
|
|
int rpc_platform_deinit(void);
|
|
|
|
/*
|
|
* serial_drv_open function opens driver interface.
|
|
*
|
|
* Input parameter
|
|
* transport : Pointer to transport driver
|
|
* Returns
|
|
* serial_drv_handle : Driver Handle
|
|
*/
|
|
struct serial_drv_handle_t* serial_drv_open (const char* transport);
|
|
|
|
/*
|
|
* serial_drv_write function writes in_count bytes
|
|
* from buffer to driver interface
|
|
*
|
|
* Input parameter
|
|
* serial_drv_handle : Driver Handler
|
|
* buf : Data Buffer (Data written from buf to
|
|
* driver interface)
|
|
* in_count : Number of Bytes to be written
|
|
* Output parameter
|
|
* out_count : Number of Bytes written
|
|
*
|
|
* Returns
|
|
* SUCCESS(0) or FAILURE(-1) of above operation
|
|
*/
|
|
int serial_drv_write (struct serial_drv_handle_t* serial_drv_handle,
|
|
uint8_t* buf, int in_count, int* out_count);
|
|
|
|
/*
|
|
* serial_drv_read function gets buffer from serial driver
|
|
* after TLV parsing. output buffer is protobuf encoded
|
|
*
|
|
* Input parameter
|
|
* serial_drv_handle : Driver Handle
|
|
* Output parameter
|
|
* out_nbyte : Size of TLV parsed buffer
|
|
* Returns
|
|
* buf : Protocol encoded data Buffer
|
|
* caller will decode the protobuf
|
|
*/
|
|
|
|
uint8_t * serial_drv_read(struct serial_drv_handle_t *serial_drv_handle,
|
|
uint32_t *out_nbyte);
|
|
|
|
/*
|
|
* serial_drv_close function closes driver interface.
|
|
*
|
|
* Input parameter
|
|
* serial_drv_handle : Driver Handle
|
|
* Returns
|
|
* SUCCESS(0) or FAILURE(-1) of above operation
|
|
*/
|
|
|
|
int serial_drv_close (struct serial_drv_handle_t** serial_drv_handle);
|
|
|
|
|
|
#ifdef __cplusplus
|
|
}
|
|
#endif
|
|
|
|
#endif
|