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

3 participants