Skip to content

Conversation

@Kainarx
Copy link
Contributor

@Kainarx Kainarx commented Dec 10, 2025

ESP-BSP Pull Request checklist

Note: For new BSPs create a PR with this link.

  • Version of modified component bumped
  • CI passing

Change description

Please describe your change here


Note

Migrates LCD drivers to the new DMA2D enable API, adds a Kconfig toggle for DMA2D, and updates component versions/IDF constraints.

  • Adopt esp_lcd_dpi_panel_enable_dma2d() for ESP_IDF_VERSION >= 6, retain .flags.use_dma2d = true for < 6 across ek79007, ili9881c, lt8912b (HDMI), and st7796 (incl. test)
  • Add CONFIG_BSP_LCD_USE_DMA2D in BSP Kconfigs to control DMA2D usage
  • Make HDMI dpi_configs mutable to set use_dma2d; set MIPI DSI phy_clk_src to 0 (driver default) in BSP display configs
  • Update macros in headers to gate legacy use_dma2d and introduce *_DPI_CONFIG_CF(...) variants without embedded DMA2D flags
  • Bump component versions: BSP esp32_p4_function_ev_board to 5.2.0; esp_lcd_ili9881c to 1.1.0; esp_lcd_lt8912b to 0.2.0; esp_lcd_st7796 to 1.4.0; require idf >= 5.4 for the ESP32-P4 BSP

Written by Cursor Bugbot for commit 1fc6818. This will update automatically on new commits. Configure here.

@Kainarx
Copy link
Contributor Author

Kainarx commented Dec 10, 2025

@suda-morris @espzav PTAL,thanks

@github-actions github-actions bot changed the title fix(lcd): use new mipi dma2d api fix(lcd): use new mipi dma2d api (BSP-755) Dec 10, 2025
@Kainarx Kainarx force-pushed the fix/update_mipi_dma2d_in_idf6.0 branch from 4d186b9 to 8c798f9 Compare December 10, 2025 08:50
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@Kainarx Kainarx force-pushed the fix/update_mipi_dma2d_in_idf6.0 branch from 8c798f9 to 0395ea9 Compare December 26, 2025 11:43
Copy link
Collaborator

@suda-morris suda-morris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Kainarx LGTM, please resolve the conflict as well.

@Kainarx Kainarx force-pushed the fix/update_mipi_dma2d_in_idf6.0 branch from 0395ea9 to 0fa1204 Compare December 30, 2025 06:14
@Kainarx Kainarx force-pushed the fix/update_mipi_dma2d_in_idf6.0 branch 2 times, most recently from 75965e0 to 27e424e Compare December 31, 2025 06:12
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@Kainarx Kainarx force-pushed the fix/update_mipi_dma2d_in_idf6.0 branch from 27e424e to 5a32408 Compare December 31, 2025 06:34
@Kainarx Kainarx force-pushed the fix/update_mipi_dma2d_in_idf6.0 branch from 5a32408 to 1fc6818 Compare December 31, 2025 11:44
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.

3 participants