Wednesday, 2021-04-21

MightyUmm guys is the matrix bridge broken?02:42
spiiroinpiggz: scoping the wakeup key thing: it looks like emitting it is used to solve a unique problem one specific device has -> I'm not sure whether there is anything generally usable/useful to do about it06:14
spiiroinif it is a spontaneous (vs. something user does) -> it probably should not wake up display06:15
spiiroinif the reason for emitting those events is to get out and stay out from suspend -> waking display conditionally might not even work06:16
spiiroinif the idea is just to block suspend -> it starts with wakup down. but when does the blocking end?06:16
spiiroinis the "key" is held "down" all the way until incoming call is handled? or is there down event immediately followed by up event?06:18
spiiroinif former, then blocking suspend while key is down = harmless enough to be handled in "core mce"06:19
spiiroinif the end of blocking needs to be deduced from something, like call state -> there is no obvious correlation between the keypress itself and what needs to be done06:20
spiiroini.e. "wakeup key means something about incoming call" is something that should be handled in device specific way06:21
spiiroinwhich could be for example a mce plugin used for this particular product only (could be reused of course if need arises)06:22
spiiroinbut if this is in a way part of cellular stack -> should mce even be involved with it? which is where standalone binary / patching something else than mce would come in picture06:24
T42<adampigg> Spiiroin: key up comes in immediately...then, anothet key down/up for every ring of the modem06:26
spiiroinhow far apart those are? or does it vary?06:27
T42<adampigg> A few seconds. .. 5ish probably06:27
T42<adampigg> For as long as phone is ringing06:28
spiiroinonce call state changes suspend is blocked due to that happening too06:29
spiiroinfor a while / each call state change06:29
spiiroinmight have been that is blocked for duration of incoming call / ringing06:30
T42<adampigg> Atm, im not getting the call state changed,  as ofono isnt picking up the ring while in suspend06:30
spiiroinI mean if ofono gets there -> babysitting ofono is needed until it handles the call06:31
spiiroinhave you checked timing between wakeup keypress and call state change?06:31
spiiroinbut probably using a automatically canceling wakelock for ~5 seconds should do it06:33
T42<adampigg> Spiiroin, well, there are a bunch of issues other ports are dealing with like the modem dissapearing from the usb bus while suspended. .. These things are still being workee on06:33
spiiroinis that usb wonkyness just incoming call phase or during call too?06:34
spiiroinbecause there is built-in assumption in mce that cpu can be put to sleep after call is activated06:35
spiiroinsuspend is blocked for initial 30s (or so) of active call, then suspend is allowed (=hack to get non-suspend-aware dtmf playback working)06:36
T42<adampigg> Just incoming i think.... The bus needs to be re enumerated before ofono can handle call06:36
spiiroinso, to scope whether/how/what works, we could try doing "block suspend for 5 seconds on wakup key down" in mce06:38
spiiroinif that is enough, I see no harm in putting and leaving such thing in mce06:39
T42<adampigg> cool ... if you want a read-up on the current state, see
spiiroin(if something else uses KEY_WAKEUP for some other reasons, this kind of patch does not affect it)06:40
T42<adampigg> this is separate from the wakup ... this is around fixing the USB06:40
spiiroinroughly -> case KEY_WAKEUP: mce_wakelock_obtain("key_wakeup", 5000); break06:45
spiiroinplus the adding of KEY_WAKEUP in probing array you already had06:46
piggzspiiroin: so, different to how I had it, converting to a gesture?06:46
piggzand ... when ofono if fixed to detect the call, that will wake up the screen instead?06:47
spiiroinpiggz: yes. if it is not user initiated -> it is not really a gesture/power key like thing06:47
T42<KathrineKarlsen> 𝗜𝗻 𝗺𝘆 𝗹𝗶𝗳𝗲, 𝗜 𝗵𝗮𝘃𝗲 𝗮𝗹𝘄𝗮𝘆𝘀 𝗯𝗲𝗲𝗻 𝘀𝘂𝗿𝗿𝗼𝘂𝗻𝗱𝗲𝗱 𝗯𝘆 𝗮 𝗹𝗼𝘁 𝗼𝗳 𝗮𝗺𝗮𝘇𝗶𝗻𝗴 𝗽𝗲𝗼𝗽𝗹𝗲. 𝗧𝗵𝗮𝗻𝗸𝘀 𝗳𝗼𝗿 𝗯𝗲𝗶𝗻𝗴 𝗼𝗻𝗲 𝗼𝗳 𝘁𝗵𝗲𝗺 𝗠𝗿 𝗠𝗶𝗰𝗵 𝗔𝗻𝗱𝗲𝗿𝘀𝗼𝗻,𝗵𝗲 𝗶𝘀 𝗮 𝗵𝗲𝗹𝗽𝗲𝗿06:47
spiiroinyes. incoming call = display policy special case -> display will unblack when/if proximity / cover sensor is not covered06:48
spiiroingesture is ignored if display can't be woken up at the time the event is received06:49
piggzok .. hopefully in the next few weeks there will be some work on ofono ... other distros use newer ofono/plugin so i will likely have to try and port to our version06:50
T42cryptoheals was added by: cryptoheals15:54
riniguspketo: sounds like "Xperia XZ2 (h8266 / h8266)" is not registered in the store. would you mind to check it and register if it is missing?17:14
rinigusit is one of the devices from Sony Tama port17:15
malrinigus: which ones do you support? is that just some variant of the same device you have done before or different device17:40
rinigusmal: there are six devices in sony tama (xz2/xz2c/xz3 x 1- and 2-sim variants). I support all of them, but not all were registered in the store17:41
rinigus... so, I get once in a while complaint that the store shows limited apps. then I grep irc archive and check if it was registered17:42
riniguswhen I asked to register all in one go, pketo preferred to register only ones that I had either access to or some user had access to17:43
rinigusso, no new port or variant in this respect17:43
piggzif anyone fancies being glitlab vertified for free
pketorinigus: h8266 added18:21
riniguspketo: thank you very much!18:27
T42nerd7473 was added by: nerd747323:43

Generated by 2.17.1 by Marius Gedminas - find it at!