Skip to content

Add/esp32 spi sd - #626

Merged
yingtao450 merged 8 commits into
tuya:masterfrom
maidang-xing:add/esp32_spi_sd
Jul 3, 2026
Merged

Add/esp32 spi sd#626
yingtao450 merged 8 commits into
tuya:masterfrom
maidang-xing:add/esp32_spi_sd

Conversation

@maidang-xing

Copy link
Copy Markdown
Contributor

PR 描述/PR description

[在此详细描述 PR 的内容]/[Describe the PR content in detail here]

代码质量/Code Quality:

在本次拉取请求中,我已考虑以下事项 As part of this pull request, I've considered the following:

  • 确保代码注释和文档清晰,并使用英文注释以保证代码可读性。Ensure that the code comments and documentation are clear, and use English for comments to ensure code readability.
  • 确保文件头遵循文件头格式。Ensure that the file header follows the File Header Format.
  • 确保函数头遵循 Doxygen 格式。Ensure that function headers follow the Doxygen format as specified in Comments.
  • 已查阅 编码风格指南,并核查代码风格合规性,包括缩进、空格、命名规范及其他风格要求。 Reviewed the Coding Style Guide and verified code style compliance, including indentation, spacing, naming conventions, and other style guidelines.
  • 已使用代码格式化工具确保符合 TuyaOpen 编码规范。Have used the code-formatting source code formatting tool to ensure compliance with TuyaOpen coding standards.

- Fix SPI example pin defaults: CS=10->21, DC=13->40 for DNESP32S3
- Add XL9555 IO expander init (LCD power, reset, backlight)
- Add menuconfig options for IO expander pins
- Split pixel fill into DMA-sized chunks (<4092)
- Fix tdd_display_spi.c missing psram_mode initializer
- DNESP32S3: pinmux IO11/12/13/2 for SPI SD, tkl_fs_mount handles all bus init
- WAVESHARE: pinmux IO1/2/3, CS via TCA9554 EXIO7, SD and LCD use separate SPI hosts (SPI3 vs SPI2)
- Clean board code: no tkl_spi_init needed, only pinmux + fs mount
… and update board configurations for SD card and display initialization. Refactor SPI example to enhance clarity and functionality.
…ister_hardware and refactor example_spi.c for improved readability and structure.
…nts and maintaining consistent formatting throughout the file.
@yingtao450
yingtao450 merged commit f616c87 into tuya:master Jul 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants