framebuffer: esp-box/esp-box-lite use framebuf height form Kconfig (BSP-331)

[espressif2022]

ESP-BSP Pull Request checklist

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

• Version of modified component bumped
• CI passing

Change description

In some scenarios, the remaining SRAM is insufficient, make the num of framebuf height to configurable.

[tore-espressif]

Hello @espressif2022 , you are right, the frame buffer is not easily configurable now.

We were thinking about making it configurable in runtime (as opposed to you suggestion with Kconfig).
We wanted to introduce new API, that would extend the current bsp_display_start() like this:

lv_disp_t *bsp_display_start_with_config(lvgl_port_cfg_t *cfg);

This would allow users to configure their esp_lvgl_port and framebuffer settings on runtim, what do you think?

cc @espzav

[espressif2022]

Hello @espressif2022 , you are right, the frame buffer is not easily configurable now.

We were thinking about making it configurable in runtime (as opposed to you suggestion with Kconfig). We wanted to introduce new API, that would extend the current bsp_display_start() like this:

lv_disp_t *bsp_display_start_with_config(lvgl_port_cfg_t *cfg);

This would allow users to configure their esp_lvgl_port and framebuffer settings on runtim, what do you think?

cc @espzav
@tore-espressif ，good suggestion.
This is needed for spi screens, probably useless for RGB screens.
This modification is only for the screen of SPI interface, will it affect the consistency of the interface(bsp_display_start())?

[tore-espressif]

This is needed for spi screens, probably useless for RGB screens.
This modification is only for the screen of SPI interface, will it affect the consistency of the interface(bsp_display_start())?

Yep, RGB screens are not yet supported by esp_lvgl_port , it is planned in following months.

So, unless this PR is not in a hurry, I'd suggest waiting a little. Then we can come up with API that would allow configuration of framebuffers for all LCDs (SPI, RGB...)

[espressif2022]

This is needed for spi screens, probably useless for RGB screens.
This modification is only for the screen of SPI interface, will it affect the consistency of the interface(bsp_display_start())?

Yep, RGB screens are not yet supported by esp_lvgl_port , it is planned in following months.

So, unless this PR is not in a hurry, I'd suggest waiting a little. Then we can come up with API that would allow configuration of framebuffers for all LCDs (SPI, RGB...)

You are thoughtful，but there's a little urgency now.
Recently, many developers urged us to update esp-box-lite, and now the examples of esp-box-lite have been componentized and are waiting to be released.
At present, on some examples, the RAM is insufficient, and rainmaker fails to run, so submit this PR.

[tore-espressif]

At present, on some examples, the RAM is insufficient, and rainmaker fails to run, so submit this PR.

Understood. I left one commit about using bool instead of int in Kconfig. Apart from that LGTM!

[tore-espressif]

@espressif2022 I rebased your branch on top of #188, which should fix the CI

[espressif2022]

@espressif2022 I rebased your branch on top of #188, which should fix the CI

ok.