lvgl / lv_drivers

TFT and touch pad drivers for LVGL embedded GUI library
https://docs.lvgl.io/master/porting/index.html
MIT License
291 stars 310 forks source link

refactor(win32drv): multiple display support #257

Closed wrgallo-navitas closed 1 year ago

wrgallo-navitas commented 1 year ago

Fixes: #254 #255 #256

Code-reviews are welcome

Co-Authored-By: wrgallo

MouriNaruto commented 1 year ago

I don‘t think having maximum display count limit is acceptable. Also, the window class should be reused as possible because windows ATOM objects have maximum count limit.

For keyboard support improvement, I think https://github.com/lvgl/lv_drivers/pull/258 is simpler than this PR and add mutex for improve the reliability.

In general, I want to provide a better solution than this PR.

Kenji Mouri

wrgallo-navitas commented 1 year ago

Closing this as it became a duplicate.

I don‘t think having maximum display count limit is acceptable. Also, the window class should be reused as possible because windows ATOM objects have maximum count limit.

For keyboard support improvement, I think #258 is simpler than this PR and add mutex for improve the reliability.

In general, I want to provide a better solution than this PR.

Kenji Mouri