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>
227 lines
7.6 KiB
Python
227 lines
7.6 KiB
Python
#!/usr/bin/env python
|
|
#
|
|
# SPDX-FileCopyrightText: 2025 Espressif Systems (Shanghai) CO LTD
|
|
# SPDX-License-Identifier: Apache-2.0
|
|
|
|
# check the host and co-processor version numbers against the value in
|
|
# idf_component.yml
|
|
# exit with 0 if ok
|
|
# exit with 1 if fail
|
|
|
|
import argparse
|
|
from datetime import date
|
|
import os
|
|
import re
|
|
import sys
|
|
|
|
# paths to files to check
|
|
yml_file = "idf_component.yml"
|
|
coprocessor_version_file = "slave/main/esp_hosted_coprocessor_fw_ver.h"
|
|
host_version_file = "host/esp_hosted_host_fw_ver.h"
|
|
|
|
# year starts from 2025
|
|
start_year = 2025
|
|
|
|
def get_idf_yml_version() -> (int, int, int):
|
|
# read the yml file
|
|
file_info = open(yml_file, "r")
|
|
info = file_info.read()
|
|
file_info.close()
|
|
|
|
# extract the version info
|
|
ver = re.search("^version: \"([0-9]+).([0-9]+).([0-9]+)\"", info)
|
|
#print("yml:", ver.group(1), ver.group(2), ver.group(3))
|
|
|
|
# return version info as a tuple (major, minor, patch)
|
|
return (ver.group(1), ver.group(2), ver.group(3))
|
|
|
|
def get_coprocessor_version() -> (int, int, int):
|
|
# read the coprocessor file
|
|
try:
|
|
file_info = open(coprocessor_version_file, "r")
|
|
except:
|
|
print("Coprocessor file open error: default to 0.0.0")
|
|
return (0, 0, 0)
|
|
info = file_info.read()
|
|
file_info.close()
|
|
|
|
# extract the version info
|
|
major_re = re.search("_VERSION_MAJOR_1 ([0-9]+)", info)
|
|
minor_re = re.search("_VERSION_MINOR_1 ([0-9]+)", info)
|
|
patch_re = re.search("_VERSION_PATCH_1 ([0-9]+)", info)
|
|
|
|
if ((not major_re) or (not minor_re) or (not patch_re)):
|
|
print("No coprocessor version info found: default to 0.0.0")
|
|
return (0, 0, 0)
|
|
|
|
#print("coprocessor:", major_re.group(1), minor_re.group(1), patch_re.group(1))
|
|
|
|
return (major_re.group(1), minor_re.group(1), patch_re.group(1))
|
|
|
|
def get_host_version() -> (int, int, int):
|
|
# read the host file
|
|
try:
|
|
file_info = open(host_version_file, "r")
|
|
except:
|
|
print("Host file open error: default to 0.0.0")
|
|
return (0, 0, 0)
|
|
info = file_info.read()
|
|
file_info.close()
|
|
|
|
# extract the version info
|
|
major_re = re.search("_VERSION_MAJOR_1 ([0-9]+)", info)
|
|
minor_re = re.search("_VERSION_MINOR_1 ([0-9]+)", info)
|
|
patch_re = re.search("_VERSION_PATCH_1 ([0-9]+)", info)
|
|
|
|
if ((not major_re) or (not minor_re) or (not patch_re)):
|
|
print("No host version info found: default to 0.0.0")
|
|
return (0, 0, 0)
|
|
|
|
#print("host:", major_re.group(1), minor_re.group(1), patch_re.group(1))
|
|
|
|
return (major_re.group(1), minor_re.group(1), patch_re.group(1))
|
|
|
|
# write common header using the provided file handle
|
|
def write_common_header(file_info):
|
|
year = date.today().year
|
|
|
|
file_info.write("/*\n")
|
|
if (year == start_year):
|
|
file_info.write(f" * SPDX-FileCopyrightText: {year} Espressif Systems (Shanghai) CO LTD\n")
|
|
else:
|
|
file_info.write(f" * SPDX-FileCopyrightText: {start_year}-{year} Espressif Systems (Shanghai) CO LTD\n")
|
|
file_info.write(" *\n")
|
|
file_info.write(" * SPDX-License-Identifier: Apache-2.0\n")
|
|
file_info.write(" *\n")
|
|
file_info.write(" * DO NOT MODIFY THIS FILE.\n")
|
|
file_info.write(" *\n")
|
|
file_info.write(" * tools/check_fw_versions.py generated this file.\n")
|
|
file_info.write(" *\n")
|
|
file_info.write(" * This file is autogenerated by a pre-commit hook.\n")
|
|
file_info.write(" * Version info here is populated from idf_component.yml\n");
|
|
file_info.write(" */\n")
|
|
|
|
# write footer for coprocessor file using the provided file handle
|
|
def write_coprocessor_footer(file_info):
|
|
file_info.write("\n")
|
|
file_info.write("/**\n")
|
|
file_info.write(" * Macro to convert version number into an integer\n")
|
|
file_info.write(" */\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_VAL(major, minor, patch) ((major << 16) | (minor << 8) | (patch))\n")
|
|
file_info.write("\n")
|
|
|
|
# write footer for host file using the provided file handle
|
|
def write_host_footer(file_info):
|
|
file_info.write("\n")
|
|
file_info.write("/**\n")
|
|
file_info.write(" * Macro to convert version number into an integer\n")
|
|
file_info.write(" */\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_VAL(major, minor, patch) ((major << 16) | (minor << 8) | (patch))\n")
|
|
file_info.write("\n")
|
|
file_info.write("/* Extract version components from version value */\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_MAJOR(ver) (((ver) >> 16) & 0xFF)\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_MINOR(ver) (((ver) >> 8) & 0xFF)\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_PATCH(ver) ((ver) & 0xFF)\n")
|
|
file_info.write("\n")
|
|
file_info.write("/* Format version tuple for printing */\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_PRINTF_ARGS(ver) \\\n")
|
|
file_info.write("\t(unsigned int)ESP_HOSTED_VERSION_MAJOR(ver), \\\n")
|
|
file_info.write("\t(unsigned int)ESP_HOSTED_VERSION_MINOR(ver), \\\n")
|
|
file_info.write("\t(unsigned int)ESP_HOSTED_VERSION_PATCH(ver)\n")
|
|
file_info.write("\n")
|
|
file_info.write("#define ESP_HOSTED_VERSION_PRINTF_FMT \"%u.%u.%u\"\n")
|
|
file_info.write("\n")
|
|
|
|
def set_coprocessor_version(version: tuple) -> int:
|
|
# write the coprocessor file
|
|
file_info = open(coprocessor_version_file, "w")
|
|
|
|
write_common_header(file_info)
|
|
|
|
file_info.write("#ifndef __ESP_HOSTED_COPROCESSOR_FW_VER_H__\n")
|
|
file_info.write("#define __ESP_HOSTED_COPROCESSOR_FW_VER_H__\n")
|
|
file_info.write("\n")
|
|
ver = version[0]
|
|
file_info.write(f"#define PROJECT_VERSION_MAJOR_1 {ver}\n")
|
|
ver = version[1]
|
|
file_info.write(f"#define PROJECT_VERSION_MINOR_1 {ver}\n")
|
|
ver = version[2]
|
|
file_info.write(f"#define PROJECT_VERSION_PATCH_1 {ver}\n")
|
|
write_coprocessor_footer(file_info)
|
|
file_info.write("#endif\n")
|
|
file_info.close()
|
|
return 0
|
|
|
|
def set_host_version(version: tuple) -> int:
|
|
# write the host file
|
|
file_info = open(host_version_file, "w")
|
|
|
|
write_common_header(file_info)
|
|
|
|
file_info.write("#ifndef __ESP_HOSTED_HOST_FW_VERSION_H__\n");
|
|
file_info.write("#define __ESP_HOSTED_HOST_FW_VERSION_H__\n");
|
|
file_info.write("\n")
|
|
ver = version[0]
|
|
file_info.write(f"#define ESP_HOSTED_VERSION_MAJOR_1 {ver}\n")
|
|
ver = version[1]
|
|
file_info.write(f"#define ESP_HOSTED_VERSION_MINOR_1 {ver}\n")
|
|
ver = version[2]
|
|
file_info.write(f"#define ESP_HOSTED_VERSION_PATCH_1 {ver}\n")
|
|
write_host_footer(file_info)
|
|
file_info.write("#endif\n")
|
|
file_info.close()
|
|
return 0
|
|
|
|
def check_slave_version(yml_version: tuple) -> int:
|
|
version = get_coprocessor_version()
|
|
if (version == yml_version):
|
|
return 0
|
|
else:
|
|
return 1
|
|
|
|
def check_host_version(yml_version: tuple) -> int:
|
|
version = get_host_version()
|
|
if (version == yml_version):
|
|
return 0
|
|
else:
|
|
return 1
|
|
|
|
def check(args) -> int:
|
|
ret = 0
|
|
yml_version = get_idf_yml_version()
|
|
update = args.update
|
|
force = args.force
|
|
|
|
if (check_slave_version(yml_version)):
|
|
print(f"Co-processor version check failed. Info in {coprocessor_version_file} different from {yml_file}.")
|
|
if (update or force):
|
|
print(f"{coprocessor_version_file} updated with correct version info.")
|
|
set_coprocessor_version(yml_version)
|
|
ret = 1
|
|
elif force:
|
|
print(f"Force updating version info in {coprocessor_version_file}.")
|
|
set_coprocessor_version(yml_version)
|
|
|
|
if (check_host_version(yml_version)):
|
|
print(f"Host version check failed. Info in {host_version_file} different from {yml_file}.")
|
|
if (update or force):
|
|
set_host_version(yml_version)
|
|
print(f"{host_version_file} updated with correct version info.")
|
|
ret = 1
|
|
elif force:
|
|
print(f"Force updating version info in {host_version_file}.")
|
|
set_host_version(yml_version)
|
|
|
|
return ret
|
|
|
|
if __name__ == '__main__':
|
|
parser = argparse.ArgumentParser(description = "Check, update version headers against component version")
|
|
parser.add_argument('-u', '--update', action = 'store_true',
|
|
help = "Updates version headers if different from component version")
|
|
parser.add_argument('-f', '--force', action = 'store_true',
|
|
help = "Force update version headers")
|
|
|
|
args = parser.parse_args()
|
|
|
|
sys.exit(check(args))
|