Skip to content

Conversation

@MrSumi
Copy link

@MrSumi MrSumi commented Sep 5, 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

Using 'CONFIG_SOC_LCDCAM_RGB_LCD_SUPPORTED ' and 'CONFIG_SOC_MIPI_DSI_SUPPORTED' instead of 'CONFIG_IDF_TARGET_ESP32S3' and 'CONFIG_IDF_TARGET_ESP32P4'.
To support ESP32P4 in RGB LCD panel.

@CLAassistant
Copy link

CLAassistant commented Sep 5, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot changed the title feat(esp_lvgl_port): support ESP32P4 in rgb lcd panel feat(esp_lvgl_port): support ESP32P4 in rgb lcd panel (BSP-713) Sep 7, 2025
Copy link
Collaborator

@espzav espzav left a comment

Choose a reason for hiding this comment

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

@MrSumi Thank you for this PR. I am sorry for late review. Please, rebase branch and update changelog file.
Thank you!

#include "freertos/semphr.h"

#if CONFIG_IDF_TARGET_ESP32S3 && ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 0)
#if (CONFIG_SOC_LCDCAM_RGB_LCD_SUPPORTED && ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 0))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
#if (CONFIG_SOC_LCDCAM_RGB_LCD_SUPPORTED && ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 0))
#if (SOC_LCDCAM_RGB_LCD_SUPPORTED && ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 0))

All SOC_* are without CONFIG_ prefix. Please change it.

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