Login at Kodi Home

**herrnst** · 2012-11-01, 20:21

(2012-11-01, 19:58)jardes69 Wrote: Thanks for a quick response. I have downloaded and installed the addon just this morning.
I checked the code, corrections from 377b7d5 are there.

btw: Dimming on screensaver works OK

Ah, now I got it - suspending the whole application. Yeah thats yet not handled, need to check how to best do this.

Thanks for the feedback on the dimming Smile

jardes69 · 2012-11-01, 20:25

Yes, that's it!

I think it just need to check some kind of XBMC "suspend request" flag.

**herrnst** · 2012-11-01, 20:28

Hm...

@XBMC-Devs: Does anything like "xbmc.abortRequested" exists for this, like "xbmc.suspendRequest"?

jardes69 · 2012-11-01, 20:53

I found this:

### check if XBMC is shutting down
if xbmc.abortRequested:
log('XBMC abort requested, aborting')

**Martijn** · (This post was last modified: 2012-11-01, 20:55 by Martijn.)

(2012-11-01, 20:28)herrnst Wrote: Hm...

@XBMC-Devs: Does anything like "xbmc.abortRequested" exists for this, like "xbmc.suspendRequest"?

nope. not judging from the docs. guess you could make it a feature request for post-Frodo. Not sure how it is handled internally

jardes69 · 2012-11-01, 20:58

I do not know python, but can it be done like this?:

#main loop
while not xbmc.abortRequested:
settings_setup()
process_lcd() #lcd loop

g_lcdproc.SetBackLight(0)

**herrnst** · 2012-11-01, 21:32

(2012-11-01, 20:54)Martijn Wrote:
(2012-11-01, 20:28)herrnst Wrote: Hm...

@XBMC-Devs: Does anything like "xbmc.abortRequested" exists for this, like "xbmc.suspendRequest"?

nope. not judging from the docs. guess you could make it a feature request for post-Frodo. Not sure how it is handled internally

Ok, thanks for clearing. Guess this will get another option "Dim on Suspend/Shutdown" or so. Due to the extrastuff feature, some post-shutdown handling is required anyways, to clear things set by "option" LCDproc commands - currently, icons stay at their last state when unloading the addon Angel

**herrnst** · (This post was last modified: 2012-11-02, 00:58 by herrnst.)

@jardes69:

Can you please try to update to latest GIT master, then set "Dim on Shutdown" in the GUI settings to true, and then retry the suspend thing?

The addon now at least does shutdown handling, but am not sure if this has any effect on suspend operations. If it doesn't, according to Martijn something like that probably needs to be implemented in XBMCs addon interface first.

Edit: You also should see a "Shutting down" NOTICE originating from the addon in xbmc.log.

jardes69 · 2012-11-02, 11:10

Excellent, I will let you know!

**herrnst** · (This post was last modified: 2012-11-05, 00:48 by herrnst.)

@sphere: Thanks for your pointer to "xbmcgui.getCurrentWindowId()", works like a charm (even if for readability I had to self-define some interesting window ids Cool

).

@jardes69: Did you check the suspend stuff already?

@all: Please check if everything still works like it should since my last commits (2279eee, as I did some bigger rework on the InfoLabel retrieval stuff to make the code more readable.

@iMON/mdm166a users: Please check if especially the codec icons setting still works, as I implemented some pseudo-caching for this to save a bit on CPU cycles (don't determine icon/codec match if nothing changed). Still curious if mdm166a works at all Wink

BTW: Extrastuff handling as implemented like in Ticket#8981 is now complete Blush

Regards,
Daniel

jardes69 · 2012-11-05, 09:19

Daniel, I did not have time to check it yet, maybe tonight, I will definitelly let you know.

jardes69 · 2012-11-09, 00:07

I have just checked it. It does not dim the display when XBMC suspends.
The core LCD functionality does this though. Looks like it can not be done by a service addon.

**herrnst** · 2012-11-10, 14:26

Thanks for checking that! Honestly, I had that feeling it wouldn't work as intended Wink

I've posted a Feature Request on this in the Feature Request forum, maybe suspend handling can be done one day.

However, as a workaround, you might handle this on system level. IIRC, when the system is put to sleep, some events are triggered or scripts executed from some /etc/suspend.d dir (I personally do not use this, so someone else needs to provide correct directions on this). You could put a LCDproc unload/load handler there and configure LCDproc to turn off the display on unload, or even a script that connects to LCDproc, sets up a screen with higher priority and turns off the backlight afterwards. On resume, vice-versa needs to be done of course.

HTH,
Daniel

jardes69 · 2012-11-12, 09:05

Thanks for you reply. I have Lcdproc script for unloading/loading, but it does not switch the panel off. It's a good point, but I need to do some research. It's another option in my mind - if your script can check the keyboard, I can send a keypress in a sequence before the suspend command.

Jaro

**herrnst** · 2012-11-12, 21:10

Maybe you can also try:

put "Backlight=on" into your LCDd.conf (probably replacing Backlight=auto) - as a side effect this will disable backlight control from clients
On suspend, do a "LCDd stop" via the system. This should turn off the light.
On resume, do "LCDd start", again via the system. That should re-light the display. The LCDproc addon will reconnect after approx. 10 seconds.