From ca2a029722592fd1d59c08e47ba7c1ae4e59f04e Mon Sep 17 00:00:00 2001 From: Song Ruo Jing Date: Wed, 24 Jun 2026 22:04:01 +0800 Subject: [PATCH] fix(dma2d): fix non-usable DMA2D_CSC_TX_SCRAMBLE option --- components/hal/esp32p4/include/hal/dma2d_ll.h | 2 +- components/hal/include/hal/dma2d_types.h | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/components/hal/esp32p4/include/hal/dma2d_ll.h b/components/hal/esp32p4/include/hal/dma2d_ll.h index de79e367a10..563ade6167a 100644 --- a/components/hal/esp32p4/include/hal/dma2d_ll.h +++ b/components/hal/esp32p4/include/hal/dma2d_ll.h @@ -1019,7 +1019,7 @@ static inline void dma2d_ll_tx_configure_color_space_conv(dma2d_dev_t *dev, uint input_sel = 7; break; case DMA2D_CSC_TX_SCRAMBLE: - input_sel = 2; // Or 3 + input_sel = 3; // only 3 bytes/pixel format is supported for scrambling proc_en = false; output_sel = 2; break; diff --git a/components/hal/include/hal/dma2d_types.h b/components/hal/include/hal/dma2d_types.h index b66c791deb4..44f1395c230 100644 --- a/components/hal/include/hal/dma2d_types.h +++ b/components/hal/include/hal/dma2d_types.h @@ -147,6 +147,7 @@ typedef enum { * @brief Enumeration of 2D-DMA pixel bytes scamble order in color space conversion * * Assuming the original pixel byte order (from MSB to LSB) is 2-1-0 . + * This scramble only fits 3 bytes/pixel format. For non-3 bytes/pixel format, the pixels will be messed up if scrambling is used. */ typedef enum { DMA2D_SCRAMBLE_ORDER_BYTE2_1_0, /*!< 2D-DMA pixel data scrambled as BYTE2-1-0 (no scramble) */