libera/#maemo-leste/ Thursday, 2023-04-06

sicelowhat full duplex is arno11 referring to?08:37
freemangordonboth parties able to be heard I guess08:37
siceloah, with the voice calls, thanks08:37
uvosheh touchscreen buttons just got some commercial backing some person from Wolfvision GmbH wants to use it and try submitting it upstream again10:20
uvosmaybe the input maintianers will care enough to at least look at the thing now10:20
freemangordonuvos: yeah, looks like what happened to input 'inhibit'11:23
freemangordonuvos: if mce modue opens some evdev device, does it somehow register that?11:33
freemangordonor, what about if input-ctrl enumerates all opened fds in /proc/$(pidof mce) and does not inhibit those?11:34
freemangordonok, seems we have   pointer_dev_list and   keyboard_dev_list11:37
uvos__right11:42
uvos__i think pointer_dev_list would be fine as a first aproximation11:43
uvos__these are the devices mce will itself disconnect from during display off11:43
uvos__but using that directly from a module is ugly11:44
freemangordonuvos__: so, lets confirm - my idea is to have 2 more helper functions in event-input.c that will give access to either   pointer_dev_list and   keyboard_dev_list directly or to some copy of those lists13:23
freemangordonthen every module will be able to enumerate input devices currently opened by mce and do whatever it wants to with this information13:24
freemangordonin particular - input-ctrl will inhibit everything that is not in keyboard_dev_list on display off13:24
uvos__sure13:24
uvos__just dont access the global from the module directly13:25
uvos__also lock not display off13:26
uvos__and be carefull about the one-click mess13:27
freemangordonsure (direct access)13:27
freemangordoncan;t parse - why on lock?13:27
uvos__the ts must remain active while the display is off but the deivce is not locked13:27
freemangordonf tklock is enabled, we don;t want TS to be inhibited13:27
uvos__otherwise touch wont reset the timers13:27
freemangordonok, I'll do what x11-ctrl does13:28
uvos__no no13:28
uvos__it dosent do this13:28
freemangordonit does13:28
uvos__no13:28
freemangordona13:28
freemangordonah, I see13:28
freemangordonwe disable with xorg, but not with mce13:28
uvos__because mce needs to get ts while display is off but not locked13:28
freemangordonmhm13:28
uvos__but xorg dosent13:28
uvos__right13:28
freemangordonright13:28
freemangordongot it13:28
freemangordonok, I'll see when current code suspends TS13:29
uvos__right13:29
freemangordonok13:29
freemangordonuvos__:   touchscreen_suspend_pipe?13:37
uvos__right13:39
uvos__altho this is a big mess in lock-tklock.c13:41
uvos__really both event-input should be able to figure it out itself based on the modes, but yes thats how it currently works13:42
freemangordonto me this is really better to have one place only that makes those decisions13:56
freemangordonotherwise we risk to be inconsistent (if every module draws its own conclusions)13:56
freemangordonnto to say the duplication of code13:57
freemangordons/say/mention/13:57
sicelodo we ever want to support tap-to-wake, like on N9/Harmattan? 😃14:01
siceloin that case i suppose ts would need to be enabled even when display off, or even when locked14:02
uvos__sicelo: well our current hw is very bad at this, as the power consumption of th ts is immense14:14
uvos__mapphone ts has a special mode for this, but in my expierance it dosnt work so great (and still uses quite a bit of power)14:14
uvos__so currently i would say we dont have any plans for tap to wake14:15
uvos__but yeah having it in one place would be good14:15
uvos__unfortionatly the suspention descisions are spread across tklock.c and are duplicated in lock-generic.c and are sotra repeaded agian in x11-ctrl14:16
uvos__anyhow yes use touchscreen_suspend_pipe, i take issue with its implementation on the sender end, its fine in principal14:19
uvos__and on the reciveing end14:19
uvos__imo you can also add the inhibiting to event-input instead of some special module if you like, as long as you provide a fallback path (to doing nothing) and a frendly info print if the kernel is to old.14:20
freemangordonuvos__: My gut feeling tells me a module is better17:54
siceloWizzup: n900's charge full 'issue' is not in kernel. it does emit POWER_SUPPLY_CAPACITY_LEVEL=Full at the right time. however upower doesn't use this property21:55
Wizzupsicelo: that's dumb, thanks for checking21:56
Wizzupmaybe we are missing a patch from spinal21:56
siceloare you sure it did work at some point in Leste?21:58
sicelore: upower - some months ago, i submitted https://gitlab.freedesktop.org/upower/upower/-/issues/21821:58
siceloi wish upower would properly support N900 and Droid 4 out of the box21:59
Wizzupsicelo: yes I think it did in leste22:09
OksanaaWhat are the chances of Pine phone with Maemo Leste being capable of running the Android app? https://www.flir.com/support-center/flir-one/flir-one-device-compatibility/22:10
Wizzupsicelo: no feedback huh :(22:10
OksanaaRequirement: arm64-v8a. Pinephone: 64 bit, is that alright?22:11
OksanaaRequirement: GPS and location. Pinephone: GPS might be working.22:12
OksanaaRequirement: microphone. Pinephone: audio might be working.22:12
OksanaaRequirement: USB OTG. Pinephone: Untested.22:13
uvosOksanaa: anbox works on pp22:47
uvosbut i doubt a ap that interfaces with custom hw will work22:47
xmnSo does waydroid22:47
uvosno on leste22:47
uvos*not22:47
xmnI saw anbox on leste. But don't remember how well it actually worked.22:48
Wizzupanbox is slow23:12
Wizzupwaydroid is kinda slow too, but not -that- slow23:12
freemangordonugh... Registering /dev/input/event1 as keyboard fd: -123:40
freemangordonevent1 -> ../../devices/platform/mapphone_touchscreen/input/input2/event123:40
freemangordonuvos: is that ok?23:40
freemangordonlike, this seems to keep TS enable, no?23:41
Wizzupmaybe this is ts buttons?23:41
freemangordonyes23:41
freemangordonbut is still TS23:42
freemangordonhmm23:42
freemangordon"mce: event-input: unbinding 2 touchscreen devices"23:42
freemangordonthere is more that's going on there it seems23:42
uvosyeah thats correct23:42
uvosmapphones have 2 ts23:42
freemangordonright23:43
uvosbeacause of the filtered touchscreen23:43
Wizzupright, but it's seen as keyboard23:43
freemangordonbut mce assume buttons as KBD, no?23:43
uvosno23:43
uvostouchscreen buttons creates 2 devices23:43
uvosone with the touch events with those filtered that land on the buttons23:43
uvosthis one is reccognized as a ts23:43
uvosand one with just the buttons23:44
uvosthis one is recognized as a keyboard23:44
freemangordonright23:44
freemangordonbut it is still TS23:44
freemangordonlrwxrwxrwx  1 root root 0 Apr  2 16:51 event1 -> ../../devices/platform/mapphone_touchscreen/input/input2/event123:44
freemangordonlrwxrwxrwx  1 root root 0 Apr  2 16:51 event2 -> ../../devices/platform/mapphone_touchscreen/input/input1/event223:44
uvosnope23:44
uvosits irellivant for purposes of pm23:44
freemangordonok23:44
freemangordonuvos: so, is this https://pastebin.com/yBp2BpGm ok then?23:45
uvosyes keeping the ts-buttons buttons event device open costs zero power23:46
freemangordonok23:46
freemangordonI will try to confirm23:46
uvosi mean i know how my own driver works :P23:49
freemangordonsure, but I am seeing too high power usage with only TS inhibited23:49
freemangordonso just want to verify it is not caused by TS buttons being active23:51
freemangordonuvos: 146453 for power_avg, 2 ssh sessions via wifi. is that ok?23:56
freemangordonlemme check when connected to gsm23:58

Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!