diff options
author | William Wilgus <me.theuser@yahoo.com> | 2018-07-27 15:09:55 +0200 |
---|---|---|
committer | William Wilgus <me.theuser@yahoo.com> | 2018-07-27 15:09:55 +0200 |
commit | 400603abdfb4ba7566e0cae8dbed9268f06716dc (patch) | |
tree | 081559b876f26d6564d32470751b47a5814e5560 /apps/plugins/lua/rocklua.c | |
parent | dcd71e66bd5f0b521c9f98ebe83ff1f7abb62918 (diff) | |
download | rockbox-400603abdfb4ba7566e0cae8dbed9268f06716dc.tar.gz rockbox-400603abdfb4ba7566e0cae8dbed9268f06716dc.zip |
Lua -- Fix device hang when scroll function active on clear_screen
I previously noticed that manually clearing the framebuffer while scroll
function was active caused lua to crash
I could reproduce in sim and on device but I thought using the plugin
supplied rb->lcd_clear_screen was immune to this issue
Unfortunately some devices exhibit this behavior with the plugin function
as well
This patch adds rb->lcd_scroll_stop() before lcd_clear_screen at lua start-up
and to the supplied include file lcd.lua
Change-Id: I9800145e5c834ea27df5db5f1bca50b0d40faa49
Diffstat (limited to 'apps/plugins/lua/rocklua.c')
-rw-r--r-- | apps/plugins/lua/rocklua.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/apps/plugins/lua/rocklua.c b/apps/plugins/lua/rocklua.c index 5539618a43..48b5d62c37 100644 --- a/apps/plugins/lua/rocklua.c +++ b/apps/plugins/lua/rocklua.c | |||
@@ -160,6 +160,7 @@ enum plugin_status plugin_start(const void* parameter) | |||
160 | rocklua_openlibs(L); | 160 | rocklua_openlibs(L); |
161 | status = luaL_loadfile(L, filename); | 161 | status = luaL_loadfile(L, filename); |
162 | if (!status) { | 162 | if (!status) { |
163 | rb->lcd_scroll_stop(); /* rb doesn't like bg change while scroll */ | ||
163 | rb->lcd_clear_display(); | 164 | rb->lcd_clear_display(); |
164 | status = docall(L); | 165 | status = docall(L); |
165 | } | 166 | } |