Openelec Build for OPI PC and 2 now with HW decoding

alien666dj

Hi jernej! First of all wants to thank you for your big work on openelec images, its great!!! But i have a question about hdmi cec, when i push power button on my tv LG remote the box goes in suspend mode, can i wake my orange pi pc from suspend mode by pressing the same button on my tv LG remote ? Thank you in advance for your help!

darqoq

jernej replied at 2016-8-30 22:16
This patch is necessary for power button to work. Obviously, it is not used only for H3. Without i ...

@jernej: I build OpenELEC using your code without "kodi-003-perform_suspend_instead_of_powerdown.patch"
Tested on Orange Pi PC:
(http://kodi.wiki/view/List_of_boolean_conditions)
(http://kodi.wiki/view/list_of_built-in_functions)
1. System.CanSuspend - works
2. Suspend() - works
3. System.CanPowerDown = works
4. PowerDown()

My build on default OpenELEC skin "Confluence" behaviour on "Close Menu"
(https://github.com/xbmc/skin.con ... onMenu.xml#L89-L185), shows:
1. Power off system -> same as shell commands "shutdown -h now", "poweroff"
after that Orange Pi PC is total power off, if you want to wake up board you have to unplug and plug power adapter
to start system again. If you want to unplug your board from power adapter - it is safest way to close system.
2. Custom shutdown timer
3. Suspend -> suspense system, after that Orange Pi PC is suspended, if you want to wake up board you have to use your
power button, but in this mode you shouldn't unplug from power adapter your board it is NOT safe way to close system.
4. Reboot -> same as shell command "reboot"

I think that this is better option, especially that still is possible to use power button to suspense/wake up system,
and also we have safe way to unplug board (currently doesn't exists from GUI), @jernej please consider to remove this this patch.

jernej

darqoq replied at 2016-9-1 16:43
@jernej: I build OpenELEC using your code without "kodi-003-perform_suspend_instead_of_powerdown.p ...

From what I understand, power button would then shutdown the system instead of going into suspend. Same goes for power button on IR remote. Those two cases are primary reason for that patch.

Dima

Suspend is much more useful for openelec. Especially considering the low power usage...

darqoq

jernej replied at 2016-9-1 17:51
From what I understand, power button would then shutdown the system instead of going into suspend. ...

I forget to test it, but yes, you are right :-(
Phisical power button will shutdown the system :-( (it also not good for me)
I have to try another way maybe I will modify default Confluence skin and LogindUPowerSyscall.cpp
to solve this problem,

darqoq

Dima replied at 2016-9-1 19:20
Suspend is much more useful for openelec. Especially considering the low power usage...

It is not true, more useful is has two option suspend/poweroff,
Power usage is not a problem, sometime I just need to safe shutdown system to unplug power adapter.

jernej

darqoq replied at 2016-9-1 20:15
I forget to test it, but yes, you are right :-(
Phisical power button will shutdown the system :-( ...

Just one small detail:
Both IR and power key transmits KEY_POWER event. Unfortunatelly, there is no KEY_SUSPEND or similar event present in kernel code.

giaur500

No! Power key on remote should suspend system. Power key again should wake up system again. Without need to reconnect power. If you want to completly power off, you should be able to do that from Kodi power menu. In this case power of should shutdown system completly (no LED light).

jernej

giaur500 replied at 2016-9-1 22:00
No! Power key on remote should suspend system. Power key again should wake up system again. Without  ...

I can assure you that it will stay that way. I was just talking about implementation detail and why a need for aforemention Kodi patch exists.

giaur500

jernej replied at 2016-9-2 05:03
I can assure you that it will stay that way. I was just talking about implementation detail and wh ...

Ok, but still unable to wake up with remote power key. This is biggest shame actually