lvgl / lv_port_stm32f469_disco

LVGL ported to STM32F469 Discovery using SWSTM32 (Ac6) IDE
16 stars 10 forks source link

The project cannot be compiled as is, LVGL used is old #2

Closed MR-DOS closed 4 years ago

MR-DOS commented 4 years ago

Hi, the project cannot be compiled as is - one has to modify line 41 in "stm32f4xx_it.c" from #include "lvgl/lv_hal/lv_hal_tick.h" to #include "lvgl/src/lv_hal/lv_hal_tick.h"

Also, line 45 in "main.c" has to be commented out (the function called doesn't exist) while line 44 has to be uncommented.

The project is compatible only with version of LVGL up to v6.1.2 while current version is v7.0.2. Therefore the usability of this project for starting with LVGL is limited - one has to rewrite it to support the current versions.

kisvegabor commented 4 years ago

Hi,

I don't have this board so I can't update the project yet. :slightly_frowning_face:

Were you able to make it work? If so can you send a Pull request?

Besides, it'd be great to migrate to CubeIDE.

MR-DOS commented 4 years ago

Hi, believe it or not, but I don't know how to send a pull request. I'm not a programmer, but a HW engineer who suffers every time when having to deal with programming. All that is needed to make the project compilable for v6.1.2 is included in my initial report of the issue - only minor changes on 3 lines are needed. I will try to update it for the V7 branch, maybe then I will try to find out how to make a PL if it works.

kisvegabor commented 4 years ago

believe it or not, but I don't know how to send a pull request.

No problem :slightly_smiling_face: Are you interested in learning it? We have a short guide about it here.

MR-DOS commented 4 years ago

OK, so I tried replacing the old 6.x LVGL with 7.x LVGL. All it takes is checking out the current version along with the examples, correcting one broken link to the "lv_conf_checker.h" of LVGL to "lv_conf_internal.h" which does the same, it was just renamed. Then replacing the old "lv_conf.h" and "lv_ex_conf.h" with the new templates, uncommenting their content and setting the resolution to 800x480. It is also possible to very easily enable DMA2D, but id didn't result in any speed-up in my case...

kisvegabor commented 4 years ago

Thanks!

Can you attach you lv_conf.h? Just to be sure I don't miss any settings.

stale[bot] commented 4 years ago

This issue or pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 4 years ago

This issue or pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.