Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modules: LVGL: Add Support for Rounder Callback #69410

Conversation

ioannis-karachalios
Copy link
Contributor

@ioannis-karachalios ioannis-karachalios commented Feb 23, 2024

Add support for LVGL frame rounder callback function. Sometimes, either the underlying display controller and/or the LCD host controller might impose restrictions on frames resolution. For instance, the E1394AA65A display model should impose that either axis of a frame be multiple of 2 pixels or the Renesas LCD controller of the DA1469x SoC, should impose that the stride value, number of bytes between consecutive frame lines, be multiple of 4 bytes.

@zephyrbot zephyrbot added area: LVGL Light and Versatile Graphics Library Support platform: Renesas SmartBond Renesas Electronics Corporation, SmartBond labels Feb 23, 2024
@ioannis-karachalios
Copy link
Contributor Author

@danieldegrasse could you also provide your input on this PR?

boards/arm/da1469x_dk_pro/Kconfig.defconfig Outdated Show resolved Hide resolved
modules/lvgl/include/lvgl_display.h Outdated Show resolved Hide resolved
@ioannis-karachalios ioannis-karachalios force-pushed the module-lvgl-rounder-cb branch 2 times, most recently from a465928 to 9699c21 Compare February 26, 2024 19:01
faxe1008
faxe1008 previously approved these changes Feb 27, 2024
pdgendt
pdgendt previously approved these changes Feb 27, 2024
faxe1008
faxe1008 previously approved these changes Feb 27, 2024
@aescolar aescolar added the hwmv2-likely-conflict DNM until collab-hwmv2 has been merged label Feb 28, 2024
Copy link
Collaborator

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

Needs converting to hwmv2

@nordicjm nordicjm removed the hwmv2-likely-conflict DNM until collab-hwmv2 has been merged label Mar 4, 2024
@ioannis-karachalios
Copy link
Contributor Author

ioannis-karachalios commented Mar 4, 2024

Needs converting to hwmv2

Hi, it's not clear to me what this scheme is. In fact, I am now unable to boot with any sample code after the introduction of hwmv2.

@ioannis-karachalios
Copy link
Contributor Author

Needs converting to hwmv2

PR has been rebased

pdgendt
pdgendt previously approved these changes Mar 4, 2024
boards/renesas/da1469x_dk_pro/Kconfig.defconfig Outdated Show resolved Hide resolved
Add support for LVGL rounder callback function. Sometimes,
either the underlying display controller and/or the LCD host
controller might impose restrictions on frame resolution.
For instance, the E1394AA65A display model should impose that
either axis of a frame be multiple of 2 pixels or the Renesas
LCD controller of the DA1469x SoC, should impose that the
stride value, number of bytes between consecutive frame lines,
be multiple of 4 bytes.

Signed-off-by: Ioannis Karachalios <[email protected]>
@@ -0,0 +1,12 @@
# DA1469x series Development Kit Pro board configuration

# Copyright (c) 2022 Renesas Electronics Corporation
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit: did you mean 2024?

Add default LVGL Kconfig for rounder cb.

Signed-off-by: Ioannis Karachalios <[email protected]>
@nordicjm nordicjm removed their request for review March 4, 2024 14:24
@henrikbrixandersen henrikbrixandersen merged commit abc07d8 into zephyrproject-rtos:main Mar 4, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: LVGL Light and Versatile Graphics Library Support platform: Renesas SmartBond Renesas Electronics Corporation, SmartBond
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants