blackbeard420/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf synced 2025-03-09 17:19:09 -04:00
Code Issues Projects Releases Wiki Activity

fix(isp): isp_dvp raw->rgb needs a workaround

Browse Source
This commit is contained in:
Song Ruo Jing 2024-12-10 14:39:47 +08:00
parent 274321d82c
commit 706b5b483e
3 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
components/esp_driver_isp/src/isp_core.c
View File

@ -145,6 +145,10 @@ esp_err_t esp_isp_new_processor(const esp_isp_processor_cfg_t *proc_config, isp_
isp_ll_yuv_set_range(proc->hal.hw, proc_config->yuv_range); isp_ll_yuv_set_range(proc->hal.hw, proc_config->yuv_range);
} }
if (out_color_format.color_space == COLOR_SPACE_RGB && proc_config->input_data_source == ISP_INPUT_DATA_SOURCE_DVP) {
isp_ll_color_enable(proc->hal.hw, true); // workaround for DIG-474
}
proc->in_color_format = in_color_format; proc->in_color_format = in_color_format;
proc->out_color_format = out_color_format; proc->out_color_format = out_color_format;
proc->h_res = proc_config->h_res; proc->h_res = proc_config->h_res;

8
components/hal/esp32p4/include/hal/isp_ll.h
View File

@ -396,15 +396,15 @@ static inline bool isp_ll_set_output_data_color_format(isp_dev_t *hw, color_spac
case COLOR_PIXEL_RGB888: case COLOR_PIXEL_RGB888:
hw->cntl.isp_out_type = 2; hw->cntl.isp_out_type = 2;
hw->cntl.demosaic_en = 1; hw->cntl.demosaic_en = 1;
hw->cntl.rgb2yuv_en = 0; hw->cntl.rgb2yuv_en = 1;
hw->cntl.yuv2rgb_en = 0; hw->cntl.yuv2rgb_en = 1;
valid = true; valid = true;
break; break;
case COLOR_PIXEL_RGB565: case COLOR_PIXEL_RGB565:
hw->cntl.isp_out_type = 4; hw->cntl.isp_out_type = 4;
hw->cntl.demosaic_en = 1; hw->cntl.demosaic_en = 1;
hw->cntl.rgb2yuv_en = 0; hw->cntl.rgb2yuv_en = 1;
hw->cntl.yuv2rgb_en = 0; hw->cntl.yuv2rgb_en = 1;
valid = true; valid = true;
break; break;
default: default:

4
docs/en/api-reference/peripherals/isp.rst
View File

@ -589,6 +589,10 @@ After calling :cpp:func:`esp_isp_color_configure`, you need to enable the ISP co
Calling :cpp:func:`esp_isp_color_disable` does the opposite, that is, put the driver back to the **init** state. Calling :cpp:func:`esp_isp_color_disable` does the opposite, that is, put the driver back to the **init** state.
.. note::
When the ISP DVP peripheral is used with the output color format set to the RGB color space, :ref:`isp-color` is automatically enabled in the camera driver to ensure correct data output. The function :cpp:func:`esp_isp_color_disable` should never be called in this case, otherwise it may result in disarrayed camera data.
.. _isp-ccm-config: .. _isp-ccm-config:
Configure CCM Configure CCM