libera/#maemo-leste/ Saturday, 2022-01-22

tmlinduvos: heh having hard time finding a droid4 los image to dump more gnss MPDTIME examples :) seems it's been years since i last downloaded los07:40
tmlind there should be los 14.1 site with the last set of nightly builds, maybe it was stargo's site on some german server? i think it had some experimental opengapps images as well07:43
sicelotmlind: http://droid.cs.fau.de/08:46
tmlindsicelo: thanks, that's it!08:54
siceloyw08:54
freemangordonuvos: ping09:18
freemangordonah, nevermind09:19
freemangordonno, this is buggy09:20
freemangordonhttps://github.com/maemo-leste/mce/blob/master/src/utils/powerkey.c#L53509:20
freemangordonhttps://github.com/maemo-leste/mce/blob/master/src/utils/powerkey.c#L54409:20
freemangordonthe condition on 544 can never be true09:21
freemangordonalso, what is the point of checking longpress timeout in key release at all?09:23
freemangordonisn't that triggered on timeout? like you press-and-hold and 1.5s later whatever the action connected to that happens?09:25
freemangordonalso, I think the issue with power menu appearing on double-press is that there is a flaw in the logic - shortpress action must not be executed immediately on key release, rather a 200-300 ms timer shall be started and upon its expiration the action shall be executed09:28
freemangordonyou can never detect double-press correctly otherwise09:29
freemangordonuvos: ^^^09:29
freemangordonah, you synthesize an event, thats why the check in release09:41
freemangordonstill, double-check is not needed09:42
uvosas per the original code sortpress action is executed immiatly on release09:51
uvosalso a 200ms timer would not help for correctness09:51
freemangordonI implemented that and am about to test09:51
uvosa timer the lengh of doublepress action is needeed09:52
freemangordonno, why?09:52
uvosto determine if its a double press or not09:52
freemangordonwe just need to detect double-press on time09:52
uvosyou have to let the time expire09:52
freemangordonok, lemme test it09:52
uvosotherwise you can get both single and double press executing09:52
uvosthe reason for the code that executes longpres on release09:53
freemangordonuvos: you implemented mode change check, no?09:53
uvosis that i first coded that with the intention of moving it to ev->value==209:53
freemangordonthat would prevent both single and double press executed09:53
uvosie use kernel keyrepeat time as long press09:53
uvos(possible mutliple repeats)09:53
freemangordonalsi, I don;t see any issue with long-press09:54
uvosso that mce stalling can not make a short pres long09:54
uvoshowever the mapphone cpcap input device dosent support repeat atm09:54
uvosan i wanted to investigate that09:54
uvosand then reativate that bit of code09:54
uvosand remove the timer09:54
uvos(depening on behavior on other devices too ofc009:54
freemangordonhaving timer is best, as otherwise you depend on repeat interval09:54
freemangordonanyway, lemme see iof my fix actually work09:55
uvosno the timer is really bad09:55
uvosif you set longpress time pretty short09:55
uvosit will often triger erroniously09:55
uvosand we can just wait n repeats09:55
uvosuntill the repeat event time is > longpress time09:55
uvosthat will give aproximately the right time09:55
uvoswithout a race09:55
uvosbetween the kernel event and mce stalling09:55
freemangordonok, my fix seems to behave correctly09:56
uvosyour fix is not a fix09:56
uvosif you just wait 200ms09:56
uvosthat only works if your double press is faster than 200ms09:56
freemangordonsure09:56
freemangordonthat's the whole point09:56
uvosalso i dont like 200ms latency at all09:56
freemangordonmy fix is really a fix as it behaves exactly as I wanted09:57
freemangordonI don;t see you having a better solution09:57
uvosthats a really bad standart of correctness09:57
freemangordonthe same as "also i dont like 200ms latency at all" without giving a reason09:58
freemangordonif you wish we can discuss what shall be the corect behaviour09:58
uvoswell theres two reasons right there09:58
freemangordonbut, how is your mouse double-click detected?09:58
uvos1. adding 200ms latency is bad, i use single click to lock so that dealys my lock by 200ms, you can easly notcie it, making the device slower is bad09:59
freemangordonuvos: again, how is mouse double-click detected?09:59
uvos2. adding 200ms latceny is arbirary as the doubleclick timeout is 1s and if the user doubleclicks >200ms and <1s the double action execution returns10:00
freemangordonsure we can make that a parameter10:00
freemangordonso you can make that to 0 for your use-case10:00
freemangordonagree?10:00
uvosi also dont think its great for opening the power menu10:01
freemangordonfor use-case where double-click locks the device, such a timeout is needed, otherwise there is an ugly power-menu blink right before the lock10:01
freemangordonofc 200 ms is arbitrary and is my first test value10:01
freemangordonI guess 150 might work as well10:01
uvosthe only correct value is 1s10:01
freemangordonwait, what?10:02
uvosthe only correct value is doublepress time10:02
uvosie 1s by default10:02
uvosofc this is way to mutch latenct10:02
uvoswitch is why nokia and i dident do this10:02
freemangordonwe cannot delay power menu for a second10:02
freemangordonso I don;t see why we discuss that10:03
freemangordonif you double-press for a second, well, you'll have power menu popped10:03
freemangordonbut it doesn;t make sense to double-press as quickly as you can and have it blinking10:03
uvosthe point is the 200ms timer is only userfull10:03
uvosif you doublepress <200ms10:03
freemangordonsure10:03
uvosif you doublepress <200ms10:04
freemangordonagree10:04
freemangordonbut usually you do10:04
freemangordonat least me10:04
uvosthen yout can set the the doublepresss timeout to <200ms10:04
uvosbut i think thas both to fast (for the user to doublpress reliably)10:04
freemangordonhow? by manually aditing config file?10:04
uvosand to slow (for the user to wait extra for single press action)10:04
uvosfreemangordon: ? by changeing the default and waiting for double press time to execute single press10:05
freemangordonno, sorry, wating 200 ms fo power-menu to appear is not slow10:05
uvosfreemangordon: that would at least be correct however the latency argument still exits just the same10:05
freemangordonuvos: ok, so far I see you only saying my patch is not the correct one, but I don;t understand what is the correct fix. *not* for your usecase, but for mine (and leste default behaviour)10:06
freemangordonand again, I will make that a parameter, so you can tweak your already tweaked system to behave exactly like without the fix10:07
freemangordonand you didn't answer my question how is mouse double-click detected10:07
uvosfreemangordon: ensureing that only one action is executed is only possible by wating for the timeout of the doublepress timer to see if the user presses again, due to causality10:09
uvosfreemangordon: both android power double press and mouse double-click work by the action of single click being required when doublpress is activated10:10
freemangordonand as I already explained that waiting one second for power menu to appear is terrible UX, so not an option10:10
uvoson android this is: from screen off: singe press opens turns on the display and doublepress opens the camera app10:11
freemangordonso, we don;t have a solution, but a workaround10:11
uvosofc the single press action is a prerequist of opening the camera app10:11
uvosfor mouse doubleclick:10:11
uvossingle click hilights the folder doubleclick opens it10:11
uvosthe actions both execute in double click and are not interfearing10:12
uvosfreemangordon: right the default maemo powerkey actions are fundamentaly broken ux10:12
uvosyou have to either wait10:12
uvosor blikc the power menu10:12
uvosboth are terrible10:12
uvosi asume they dident care10:12
uvosbecause they expected people to lock the device with the lock slider10:12
freemangordonor, have a workaround that mitigates it as much as possible10:13
uvosor change the actions to be consitant10:13
uvossingle press to lock10:13
uvosdouble to power off for instance are progressive like android, mouse10:13
freemangordonI don;t see how what I say contradicts to that10:14
freemangordonI want my device to behave not like android. period10:14
uvosyour workaround is wating10:14
freemangordonand?10:14
freemangordonI am not in a hurry10:14
uvosthats not like android10:14
uvosits progressive10:14
uvosin the action sense10:15
freemangordonby setting this timeout to zero, you will have your behaviour10:15
uvosdelaying 200ms is terrible ux10:15
freemangordonahrg10:15
uvosit makes it feal slow10:15
uvosyou can add this feature10:15
uvosand make the default 0ms10:15
uvosthats all ill accept10:15
freemangordonoh, so displaying half-rendered power menu for a split second is better?10:15
uvosyes it is10:16
freemangordonno10:16
uvosits ugly but at least its fast10:16
uvosie dosent get in the users way10:16
freemangordonah, so now you are "accepting" mce patches?10:16
uvosyes i am10:16
freemangordonplease have a look in git history10:16
uvosso?10:16
freemangordonof mce that is10:16
freemangordonsame10:16
freemangordonuvos: the main issue is not that power-menu is being displayed. the main issue is that power menu gets only half-rendered10:19
freemangordonand that is *UGLY*10:19
freemangordonhaving couple 100-200-300 ms delay on action is way better10:20
freemangordonot to say I don't really notice any difference between 0 and 200 ms for power menu to appear10:20
freemangordonyou already have a delay because of the dbus10:20
freemangordonalso, why is double-press timeout 1000ms?10:26
freemangordonand it it the same as medium delay?10:27
freemangordonthats FUBAR, really10:27
freemangordonah, medioum is for press-and-hold10:27
freemangordon*medium10:27
Wizzupmorning10:33
freemangordonmorning10:33
freemangordonWizzup: do you have any clue why double-press timeout in mce is 1s?10:40
dsc_morn10:40
freemangordonhmm, it seems it comes from Nokia10:40
freemangordonthat's the default though10:41
freemangordonand it is used10:42
Wizzupfreemangordon: not sure why it is 1s, I think we used mce, but one thing to keep in mind is that we probably don't want to delay single press for too long10:42
freemangordonsure10:43
WizzupI think we used fremantle default*10:43
freemangordondefine 'too long'10:43
Wizzupif people do want to use single press for locking, then delaying it adds latency for them10:43
freemangordonagree10:43
Wizzupbut I need to fully read backlog and just woke up :)10:43
freemangordonbut having a parameter will fix that for them10:43
freemangordoneither way they will have to chage the config from default10:44
freemangordonone more thing tho change should not be a big issue10:44
freemangordonhmm, I wonder if we can have "mce behaviour" packages10:45
freemangordonlike "maemo", "android", "whatever"10:45
WizzupI think that can just be a config file10:46
freemangordonumm, it is already, no? mce.ini10:46
Wizzupthere is mce.ini.d10:46
Wizzupwhere you can place files10:47
freemangordonyeah, this is what I meant10:47
Wizzupok10:47
freemangordonbut, to have a packages that install stuff there to change the behaviour from default10:47
Wizzupin any case, your fix is to delay power key menu showing while doing locking transition, and uvos wants power menu to not block, right? I think having a configurable timer for this not too bad if we can't think of another way to fix it10:48
WizzupI do agree that if the users want to lock then showing the power key menu before locking is silly and weird ux, but delaying single press to lock is also not ok - but it seems this can be solved with an extra option10:48
freemangordon:nod:10:49
Wizzupuvos: it seems to me if our current default is to have power key menu on single press and tklock on double press, having the value be 0 doesn't make that much sense, right?10:50
Wizzupuvos: I mean I lock my device way more often than that I use the powerkey menu (I really only use it to kill apps typically)10:50
freemangordonsame here10:50
Wizzupuvos: and yes that argues for making single press the lock, but that's beside the point :P10:50
Wizzupalso I believe this would also allow to have powerkey on doublepress, right?10:51
Wizzupif it doesn't enter lock that quickly (200ms) - if the user wants that10:51
Wizzup(or other configured value)10:51
freemangordonalso, I think we shall reduse double-press delay to some sane value10:52
freemangordon1000ms does not make sense10:52
freemangordonunless I am missing something10:52
Wizzupsome droid4 buttons are kinda poor10:52
freemangordon*reduce10:52
freemangordonyes, but one second?10:52
freemangordonnot critical though10:53
freemangordonhmm, it seems even 100ms is enough10:57
uvosyour just making another race11:04
uvosif you delay single press by 100ms11:05
uvosthen lock wins over power menu more often11:05
uvoswhy dont you: 1 figure out what the minimum resonable doubleclick timer is11:05
uvoswait with the signle press action for the doubleclick timer to expire11:06
uvosif and only if11:06
uvosthe set single press action and double press action require it11:06
uvosie delay be doubleclick timer if single action == power menu and double action == lock11:07
uvosor  single action == lock double action == power menu11:07
uvosthat way single action == lock double action == power menu also works11:07
freemangordonuvos: I was thinking about similar, but honestly, this is even more ugly hack than a delay11:07
uvosat least its correct11:08
uvosbehavior wise11:08
freemangordonalso, this does not contradicts with having a short press delay11:08
uvosand if the minumum resoable double press time is low11:08
freemangordon*contradict11:08
uvosits not any worse11:08
freemangordonalso, Wizzup said we shall keep double-press high because of HW issues11:08
freemangordonI cannot comment on that as I have only one device around11:09
uvosi dont think he is correct11:09
uvosi think the "HW issue" was mce stalling11:09
uvosthe time had to be long beacuse of the salls11:09
uvosbut dunno ofc11:09
freemangordonotherwise, as I already said, double-press of 1s is way too excessive to me11:09
freemangordonyeah, I think modem stalling the device was the issue11:10
uvosno11:10
freemangordonwell, quirks module11:10
uvosmce is vuernable to more stalls than just from modem11:10
freemangordonbut, we'll make most of them async, right?11:11
uvosdosent matter11:11
freemangordonhmm11:11
uvosusing a timer to detect longpress for instance is fundamentaly broken11:11
uvosas its a race between userspace geting the release event11:11
uvosand the timer expireing11:11
uvoseven is release is just 50ms behind press11:12
freemangordonok, but we don;t really need us precision there anyways11:12
uvosit might still arvie a long time after11:12
uvosits not us precission we are talking here11:12
freemangordonIIUC you fixed that yesterday, no?11:12
uvosnot for longpress11:13
freemangordonby checking the evnt times11:13
uvosas there is no keyrepeat11:13
freemangordonah11:13
uvosits impssible for longpress on d411:13
freemangordonwell, we have no option then11:13
uvosas the powerkey dosent repeat11:13
freemangordonso lets move on11:13
uvosright but thats a bug11:13
uvosin kernel11:13
freemangordonI am not sure I want power button to repeat11:14
freemangordonmaybe this is not a bug11:14
freemangordonbut made on purpose11:14
uvosno11:14
uvosthe volume key also dosent repeat11:14
uvosfor the same reason11:14
freemangordonok, won;t argue on that11:14
uvosand besides ofc you dont want the pwoer key to repeat11:14
uvosyou would just be using the events for timeing11:14
freemangordonanyway, ATM we have an issue with broken UX with default maemo behaviour.11:15
freemangordonmy patch mitigates that, albeit not being the proper fix11:16
uvosand i think its terrible11:16
uvosat least make a table11:16
uvosof action combinations11:16
uvosand dont even apply it to combinations that are not broken11:16
freemangordonhmm11:17
uvosie if doublpress = none11:17
uvosnever apply11:17
uvosif doublpress = poweroff and short = lokc11:17
freemangordonthis should be in mce.ini, right?11:17
uvosdont apply11:17
uvosetc11:17
uvos?11:18
uvosbased on whatever the varibales are set in powerkey.c11:18
uvosyes those are loaded from mce.ini(.d)11:18
freemangordonwhat about   PowerKeyShortDelayApply=lock,poweroff;poweroff,lock11:18
freemangordonjust an example11:19
freemangordonlets not hardcode, that's my point11:19
uvossure11:19
uvosbut please also handle the case11:19
uvosof short = lock double=powermenu11:19
uvosie this needs to wait the full delay11:19
freemangordonthat's why config option ;)11:19
uvosthats bad tho11:20
uvosit should just work11:20
uvoswhatever actions you set11:20
uvosno need to then figure out why this other delay needs to also be set11:20
freemangordonWizzup: ^^^ any comment11:20
freemangordonalso, I am going to change double-press timeout default to some sane value (like 300ms or so)11:21
uvosif you set it to 300ms11:21
uvosplease dont have an anditional 200ms timer at all11:21
freemangordonno, we still need it11:21
uvosthats just buggy for an extra 100ms11:21
freemangordonand breaks the ui11:21
uvoswhy?11:21
freemangordonUX11:21
freemangordonbecause you see half-rendered power menu for a split second11:22
uvosno if you have doublepress at 300ms11:22
uvosyou can wait the full 300ms if your going to wait 200ms anyhow11:22
uvosat that point its bad either way11:22
uvosat least 300ms is correct11:22
Wizzupsorry work mtg atm, I can read backlog11:22
freemangordonok11:22
freemangordonuvos: with that being a parameter, we can always set to 0 if needed11:22
freemangordonso, lemme write some actual code11:23
freemangordonlets have a discussion again once I have something to test, ok?11:23
uvosplease11:23
uvosthen add the parameter11:23
uvoscomment it out11:23
uvosin mce.ini11:23
uvosand make the default doublepress time11:23
freemangordonfor short delay? ok11:23
uvosyeah11:24
freemangordonsure11:24
freemangordonwhat default double-press time?11:24
freemangordon500ms?11:24
uvosidk try it out11:24
freemangordonok, I'll try a bit11:24
freemangordonsure11:24
uvosalso pease avoid applying it if its not nesscary11:24
uvosie at least dont if doublpress is disabled11:24
uvoswith action=none11:25
uvosbut handling other cases would be good to11:25
freemangordonyeah, we'll have only a couple of combos it is enabled for11:25
freemangordonas a config in mce.ini11:25
uvossure11:25
freemangordonok, ttyl, when I am ready11:25
Wizzupuvos: if you feel like testing, ofono 1.34 in experimental works ok for me (same bugs as before)11:36
lelMerlijnWajer renamed a repository: https://github.com/maemo-leste-upstream-forks/ofono-old13:27
lelMerlijnWajer renamed a repository: https://github.com/maemo-leste-upstream-forks/ofono13:36
Wizzupofono restarting upon upgrade is also kinda annoying :)13:55
sicelodebian/devuan has a way to protect selected daemons from upgrade restart13:58
lelMerlijnWajer edited a repository: https://github.com/maemo-leste-upstream-forks/ofono13:58
uvoswhy is ofono restarting annoying?14:00
siceloit might restart while you have a call, for example, or, disconnect you from 3G14:01
uvosi dont think it restarting during  call is an issue14:01
uvossince you wond be upgrading during a call14:01
uvosbut 3g ok14:01
sicelocalls come in when 'they' want :-)14:01
siceloso yes, you can have a call during an upgrade14:02
Wizzupright or when you rely on it for data14:02
uvoswell if you dont restart ofono, you have to restart later14:02
uvosand can miss a call then14:02
uvosso i dont buy the call thing14:02
Wizzupimho this is another daemon that should not restart until phone restart14:02
uvosbut 3g yes14:02
uvosi agree14:02
siceloWizzup: yes. makes sense14:02
Wizzupcan anyone figure out how to stop debian from doing this? all my google searched end up in d_hsystemd14:06
Wizzupdh_systemd*14:06
uvoswe do so for dsme i think14:06
uvosso look there maybe14:06
Wizzupah: https://askubuntu.com/questions/226227/how-to-manually-restart-services-affected-by-apt-upgrades14:07
Wizzupuvos: nah in dsme we hack around it14:07
Wizzupiirc 'stop' is just a no-op there14:07
Wizzuphttps://jpetazzo.github.io/2013/10/06/policy-rc-d-do-not-start-services-automatically/14:08
Wizzuplooks like this does too much though14:09
uvosits probubly a bad idea to not restart all services14:10
uvosmce for instance has relied on this in past multiple times14:10
Wizzupyeah I don't think we should prevent everything from restarting14:10
siceloWizzup: check how devuan does for ssh. at least on my debian system, it doesn't get auto-restarted (i get a prompt to decide instead). i think devuan has same mechanism14:10
Wizzupbut I think it might be possible to work on the policy stuff and not restart some daemons14:11
uvosssh has the capability to resore sessions on restart no?14:11
Wizzupas in I think policy-rc.d gets a initscript as param14:11
Wizzupuvos: no sshd just forks so the existing ones can just live14:11
Wizzupafaik14:11
uvosok14:12
uvosnot sure how it works exactly but restaring sshd causes it to have a new pid but the sessions dont disconnect14:12
Wizzupright they are forked and independent14:12
uvosok14:12
Wizzupafaik14:12
Wizzupin any case I think we can amend policy-rc.d with specific files to block restarts of say dsme14:12
Wizzuplooking at14:13
Wizzuphttps://people.debian.org/~hmh/invokerc.d-policyrc.d-specification.txt14:13
Wizzuprafael2k: new ofono is in droid414:38
rafael2kWizzup: yay!14:49
rafael2kI see mesa being updated too!!14:51
rafael2kGet:27 https://maedevu.maemo.org/leste beowulf-devel/main arm64 ofono arm64 1:1.34.0-1+2m7.1 [672 kB]14:51
rafael2k: )14:51
Wizzuprafael2k: yes, corruption problems are gone14:52
Wizzup(should be)14:52
tmlindfreemangordon: updated my d4 this morning, looks like the tearing with marina theme background scrolling in portrait mode is pretty much gone :)14:52
WizzupI don't believe we have the tearing fix from fmg in kernel yet fyi14:52
rafael2kWizzup: reaaaaally! Can I edit my xorg.conf and leave it "empty" again?14:52
Wizzuprafael2k: yes14:52
Wizzuprafael2k: please let me know14:52
rafael2kdoing it now!14:53
tmlindfreemangordon: fyi, i'm still seeing some tiny black ants here and there but rarely with the marina background scrolling in portrait mode test14:53
rafael2kWizzup: btw, where is the source of the new ofono?14:53
Wizzuprafael2k: https://github.com/maemo-leste-upstream-forks/ofono14:54
Wizzuprafael2k: maemo-ofono branch14:54
rafael2kfound - tks!14:54
Wizzuprafael2k: I am going outside for a bit, lmk if this works for you for say voice calls and stuff14:56
rafael2kI will, great work!!14:57
Wizzupyeah, I guess audio/ucm remains now14:58
Wizzupand lmk if X behaves too14:58
Wizzupwe can probably make it a bit faster btw, accel is a bit slower now but at least no flickering14:58
Wizzupbut at least it works now...14:58
rafael2ksphone needs to be rebuilt, it is still with that nasty bug when choosing a "recent" number to dial again14:59
Wizzupmaybe work with uvos on that one :)14:59
rafael2khe already fixed it14:59
* Wizzup bbiab14:59
Wizzuphe can issue rebuilds14:59
rafael2kah, ok14:59
* Wizzup afk14:59
Wizzupttyl14:59
rafael2kcalls work, all good ofono side15:00
tmlindnice :)15:00
rafael2know rebooting to test X15:01
rafael2kyay, firefox can play youtube videos!15:07
rafael2k: ))15:07
freemangordontmlind: TE patch is yet ot in leste kernel15:14
freemangordon*not15:14
freemangordonrafael2k: with sond?15:14
freemangordon*sound15:14
freemangordontmlind: and yeah, ants are still there15:15
tmlindfreemangordon: oh ok :)15:25
tmlindthat's why it's "almost gone"15:25
freemangordon:)15:29
rafael2kfreemangordon: sound has issues, I'll take a look right now16:07
freemangordongreat!16:08
lelfreemangordon opened a pull request: https://github.com/maemo-leste/mce/pull/53 (Workaround power key menu appearing for a split second)16:10
freemangordonuvos: Wizzup: ^^^16:11
rafael2kfreemangordon: with alsa sound is working all good... trouble starts when pulseaudio is on...16:15
freemangordonI see16:15
freemangordonWizzup: yeah, seems lock/unlock results in "MMU_Alloc: RA_Alloc of VMArena failed"16:16
freemangordonwill have a look tomorrow16:16
rafael2kI'll download mobian and see how stuff is working there... if needed we could upgrade some packages16:17
uvosrafael2k: sphone should be built allready16:17
uvos0.5.0 should contain the fix16:17
rafael2kuvos: strange, when I click in any "recent" number, the number does not go to the text field, and when I click dial, it crashes (but the call goes ahead)16:18
uvosrafael2k: hmm16:18
uvosok ill check it out16:18
uvosfreemangordon: please: 1. add tklock,menu to the list, 2. remove DEFAULT_POWER_SHORT_DELAY and make the dault short delay == DEFAULT_POWER_DOUBLE_DELAY16:18
rafael2ktks!16:18
uvosand i would recomment commenting PowerKeyShortDelay=250 out in mce.ini by default16:19
rafael2kuvos: ii  sphone:arm64                                   0.5.0+2m7                               arm64        Maemo Leste phone framework and GUI16:19
uvosso that  short delay == PowerKeyDoubleDelay16:19
uvosso that tklock,menu can work16:19
uvosotherwise its broken16:19
uvosactually short delay should be = PowerKeyDoubleDelay unless its also set in mce.ini16:21
uvosie not set set it to the same thing as doubleDelay16:21
freemangordonuvos: 1. it is already in th elist16:22
freemangordonand 2. better keep that separate, you can always make them equal16:23
uvosim not suggesting combining them16:23
uvosim am suggesting that if not set16:23
uvosie PowerKeyShortDelay is not in mce.ini16:23
uvosit should be set to doubleDelay16:23
freemangordonok, how is that going to help?16:24
freemangordonelaborate please, I don;t understand the point16:24
uvostklock,menu is not in the list16:24
uvosyou have menu,tklock16:24
freemangordonah16:24
uvostklock,menu needs PowerKeyDoubleDelay to be equal to PowerKeyShortDelay16:25
freemangordonbut, tklock is not listed as valid for short press16:25
uvosthere is not fakinging it16:25
uvosfreemangordon: thats wrong16:25
uvosfreemangordon: it works fine16:25
freemangordonok, but not in the comments in mce.ini16:25
uvossure we can fix the comments too16:25
freemangordonok, will do that16:25
uvosseting short to tklock has allways worked fine16:26
uvosanyhow16:26
uvosif you set short to tklock16:26
uvosand double to menu16:26
uvosyou cant acess the menu unless short delay == double dela16:26
uvosy16:26
uvosfaking it with a shorter timeout than double delay dosent work in this case16:26
freemangordonok, but does this work now?16:26
freemangordonlike - without that patch16:26
uvosit dosent setting tklok to single works16:26
uvosbut having double as menu at the same time dose not16:27
uvosyou can put power there16:27
uvosbut not menu16:27
uvosanyhow thats what this is supposed to fix too16:27
uvosso the thing is with the seperate timeout16:28
uvosthats fine16:28
uvosbut the default of the single timeout should be double delay16:29
uvosso that the whole thing works in any combination of actions16:29
freemangordonok, I'll add to the list, but will keep PowerKeyShortDelay=250 enabled, with a comment to make it equal to long delay if you want tklock,menu combo16:29
uvosdouble not long16:29
freemangordonyeah, double16:29
uvosi dont think PowerKeyShortDelay=250 is usefull really16:29
uvosif double is only 100ms more16:30
freemangordonit is16:30
uvosit makes it sill slightly buggy16:30
freemangordonat least on d416:30
uvossince there is a 100ms window16:30
freemangordonno, makes all the difference16:30
freemangordonofc those are preliminary values16:30
Wizzupnice to see this get fixed16:30
freemangordonif you test and suggest better ones, I am fine16:30
uvosanyhow the default should be double delay16:30
uvosdefault as in what happens when its not set in mce.ini16:31
uvossince thats the safe valuie16:31
uvosthat allways will work16:31
freemangordonI understand your point, but even that works fine16:31
freemangordonand I don;t wan't to delay single action for too long16:31
freemangordonok, lets see how it will behave with those values16:31
freemangordonwill gather some user feedback16:31
freemangordonand can change/tweak later16:32
freemangordonit is about config, no?16:32
uvosno16:32
uvosyou can have PowerKeyShortDelay=25016:32
uvosbut  PowerKeyShortDelay _is not set at all_16:32
freemangordonhmm?16:32
freemangordonsorry, can't parse16:32
uvosit should default to reading double16:32
freemangordonok, will do that change16:33
uvosrigh16:33
uvosgood16:33
uvosthat way16:33
Wizzupthanks guys16:33
uvosyou can have people just comment it out16:33
freemangordonyeah16:33
uvosif they use an action where they need to be the same16:33
freemangordondeal16:33
lelfreemangordon synchronize a pull request: https://github.com/maemo-leste/mce/pull/53 (Workaround power key menu appearing for a split second)16:48
freemangordonuvos: ^^^16:49
freemangordonif you are happy with this, please release so we can have it in -devel for a while16:53
freemangordonso we can gather some feedback from the users16:53
freemangordonalso from PP and n90016:54
freemangordon(two sides of the spectrum :) )16:54
rafael2kbtw, how did you fix pinephone rendering issues?18:19
Wizzupfreemangordon: uvos: this works https://dpaste.com/DTHHA4DEV.txt18:21
Wizzupsrry the last line should be sys.exit(0)18:22
Wizzup# dpkg-reconfigure ofono18:22
Wizzup[ ok ] Restarting Bluetooth DUN daemon: dundee.18:22
Wizzupinvoke-rc.d: policy-rc.d denied execution of restart.18:22
Wizzuprafael2k: we knew what the problem was kinda (egl buffer preserved not supported), enunes gave us a hack to add it to mesa, and we need to fix clutter to use buffer age later on, but meanwhile his hack works18:23
Wizzuprafael2k: we will also file a bug support to get egl buffer preserved added to mesa18:23
Wizzupfreemangordon: uvos: for reference here is a fixed one https://dpaste.com/6PF8QSAGY.txt18:23
Wizzupsicelo: ^^18:23
uvosWizzup: right18:24
sunshavi_ enunes the lima guy18:24
uvosbut that should be 2 policy files, one for ofono and one for dsme18:24
uvosbut you probubly know that18:24
Wizzupuvos: not sure what you mean, but debian only wants this one file, so I'd have to write that in python18:24
uvoshmm ok18:24
Wizzupuvos: we can also have this in leste-config18:24
uvospolicy-rc.d trew me off18:24
Wizzupmaybe it supports it as a file too, not sure18:24
Wizzupyeah I can double-check how it works18:25
uvosreally it should be packaged with the deamon18:25
uvosif at all possible18:25
uvosthe deamon package alone should not be broken18:25
WizzupI think this is something that is just missing in debian18:25
Wizzupjudging by how it works I don't think it supports directories18:25
uvosmaybe switch to systemd :P18:25
Wizzupsee:18:25
WizzupThe /usr/sbin/policy-rc.d file *must* be managed through the alternatives18:25
Wizzupsystem (/usr/sbin/update-alternatives) by any packages providing it.18:25
Wizzupok, bbiab18:26
Wizzupbut I think we can add this to leste-config ?18:26
uvoshmm thats mesy/hacky then18:26
Wizzup(maybe we can add a few more svcs)18:26
Wizzupwe can also have it only disallow 'restart' and accept 'start'18:26
Wizzupetc18:26
Wizzupbbiab :)18:27
Wizzupuvos: why is it messy, because other pkgs can't register it?18:49
Wizzupcan we think of a few more daemons we probably don't want to restart?18:50
rafael2kWizzup; cool!! tks a lot! PP usage in Maemo I would say it is really possible now! lets ask some donation to pine64 and a PP maemo version!  :P18:50
Wizzupicd2 maybe?18:50
uvosyeah idealy any signle package in leste works by itself18:50
uvosWizzup: depends18:50
uvosWizzup: how dose icd2 react to restart18:50
uvosWizzup: i gues it drops connections?18:50
Wizzupuvos: it's quite fine really, but you will lose connection18:50
Wizzupyeah18:50
uvosim not sure its a big deal, but the reason for not restarting ofono was dropping 3g so for consistancy18:51
uvosidealy ofc icd2 would do something smarter on restart18:51
WizzupI am not sure if I agree wit that per se wrt smarter, but that doesn't really matter18:58
Wizzuprafael2k: yeah would be good to check out keyboard usage18:59
Wizzuprafael2k: what do you mean wrt donation?18:59
sunshavi_https://drewdevault.com/2022/01/18/Pine64s-weird-priorities.html19:00
sunshavi_19:00
Wizzupsunshavi_: don't see it retweeted much19:17
sunshavi_Hi Wizzup19:17
sunshavi_In my opinion getting hw is very difficult also19:18
sunshavi_So what P64 does is awesome19:18
sunshavi_But some efforts could be coordinated. You guys do it a little bit with PMOS19:18
uvosnot really anymore19:19
sunshavi_Also there is no other company like AW. I hope PP-pro which is Rockchip based would give also great access to schematichs19:20
uvossince hildon is impossible to pacakge into a general use distrobution19:20
uvosfor various reasons19:20
Wizzupuvos: I think you're conflating generally working together and packaging our sw elsewhere19:20
uvoswell that would be the best bit of coperation between distrobutions19:21
Wizzupsunshavi_: well, I think we're doing a pretty good job at trying to mainline things, while we also work on our distro, we don't have manpower for much more, and getting something usable on our current targetted devices to me is more important than all using either ofono or modemmanager or something19:21
uvosotherwise we aslo dont really do mutch with pmos do we19:21
Wizzupuvos: they build on our kernel work for some devices :)19:21
sunshavi_Yes. I saw it. The work You are doing guys19:22
Wizzupand I'm pretty confident hildon will get packaged for it again at some point19:22
sunshavi_probably You are benefing also mesa+lima19:22
WizzupI think they benefit us more, but yeah :P19:22
Wizzupsure, it's about the interaction & fixing stuff19:22
sunshavi_well. Remember my main machine is an SBC with AW SOC which also happens to has lima mali-40019:23
sicelosunshavi_: i may as well declare that i've taken up maintainership of N900 on pmOS ;-)19:23
sunshavi_that's good sicelo19:23
sunshavi_more or less a month ago I was experimenting with a tablet an AW a72119:23
sicelowhat's AW? ah, allwinner19:24
sunshavi_I wanted it to boot from my combo {uSD-card-adapter-to-emmc}. So I was using an emmc19:24
sunshavi_I did not get it to work. If It would have worked. I think those guys on pmOS they are trying to support it also19:25
sunshavi_It just worked from a plain uSD19:26
sunshavi_But I am going to take to retake it in the future when time permits19:26
sunshavi_sicelo: I saw You were trying to calibrate n900 battery. Have You gotten it?19:28
sicelosunshavi_: one thing i would like to do is (if i had the device[s]) is - make Leste boot on some of the devices in pmOS community, if not all :-)19:28
sunshavi_that's Cool19:29
sicelosunshavi_: no, it was the d4. n900 you can calibrate battery with eyes closed since hw takes care of it on its own. and no, it's still uncalibrated. i gave up for the time being19:29
Wizzupsicelo: any clue how you plan to do that?19:29
sicelodo what? :-)19:30
sunshavi_I think calibrate d4 battery. or perhaps leste on pmOS-devices?19:30
siceloif you mean boot pmOS community devices with leste, i'd like to think it's going to be quite straightforward really. only terrible thing will be the power mgmt issues19:31
siceloas for calibrating d4, i really don't know. i was thinking to look at estimating capacity using voltage. iirc pavel does something like that in his unicsy_demo19:32
uvosupower dose this allready19:32
sunshavi_pmOs is better at power-management than maemo-leste?19:32
uvoswe just dont display it (only the bargraph dose show this estimation)19:32
Wizzupsicelo: - make Leste boot on some of the devices in pmOS community, if not all :-)19:32
uvoswrt callibration19:33
sicelosunshavi_: no.19:33
uvosi have no problem callibrating19:33
uvosby having the device be empty19:33
uvosbooting it19:33
uvosand then charging to full19:33
Wizzupsicelo: I mean, do you plan to package hildon or to use their kernel as base with leste on top?19:33
siceloWizzup: ok. the community devices are running mainline. and they all have working 3d... so should be quite easy if one has the device(s)19:33
uvosof you do have a really weak battery19:33
Wizzupsicelo: right, ok19:34
Wizzupsicelo: cool19:34
uvosmaybe lower the cutoff voltage for mce, it is quite high. it is so high because you can get into an unbootable situation if the voltage drops to low and the device shuts off before it gets to charge-mode19:34
uvosthere is only 100mv room between calibration and mce shuting down19:35
Wizzupmakes sense to me19:38
sicelouvos: the last time you were saying it won't calibrate because currently droid 4 doesn't do a proper shutdown, but dies. is that still the case?19:48
uvosso there is an oops19:49
uvosyeah19:49
uvosfor me it sometimes permanently hangs on shutdown19:49
uvosthat would cause callibration to faill to19:49
uvosso yes this is also still a problem19:49
uvosbut sometimes this dosent happen19:49
sicelookay19:50
uvosso yeah various issues19:51
uvoscalibration is destined to be very unrelaiable atm19:51
siceloanyway, i will see if i can run something from unicsy and log some of its output to a file. i know this will increase consumption, but at this point, i want to understand what's the battery status. atm this isn't possible19:51
uvossicelo: the best way to callibrate manualy19:52
uvosis to charge fullt19:52
uvos*fully19:52
uvostake a note of cat /sys/class/power_supply/battery/charge_counter19:52
uvosand then discharge to almost empty and take another note of cat /sys/class/power_supply/battery/charge_counter19:53
uvosor the other way around19:53
siceloYes, that's why i want to log to a file the entire time that the system is running, because shutdown is unreliable due to the oops you refer to. At least this way I'll have some values up to maybe a minute before it dies19:55
uvossicelo: well just do the reverse then19:55
uvostake a note of the value just after boot when the device boots from empty19:55
uvosand then put it on a charger and forget about it19:55
uvoswhenever you come back the difference in values is the capacity19:56
uvosyou can then save the value to callibration file in /var19:56
uvosthats it its callibrated19:56
siceloThe reverse is way harder imo, because d4 has its own boot from empty issues. Plus I want to know the discharge characteristics of this replacement battery19:57
uvosno it dosent anymore19:57
uvosmce shuts down early enough that its fine19:57
uvosin all cases19:57
uvosthe difference betwee charge and discharge capcaity is goning to be very small19:58
Wizzupso any other services besides ofono and dsme that we can think of?20:00
uvosi dont even really know if i agree with ofono20:01
uvosso no not really :P20:01
uvossphone has the opsit problem20:02
uvosit desperatly needs to be restarted on upgrade20:02
uvos(and currently is not)20:02
Wizzupright because it doesn't use init scripts (for obvious reasons)20:03
siceloWhy don't you do it in postinst?20:03
Wizzupright dsmetool could do that20:03
uvosdsmetool cant do it to my satisfaction afaik20:05
uvosand postins would not work either since its run as root20:05
uvosi gues i could write something that checks what sphone process is running20:06
uvosand then take a note of its uid/gid20:06
uvosand then kill it and restart it with the same uid/gid20:06
uvosbut makeing the env consistant is hard20:06
uvosat least i dont know how of the top of my head20:07
Wizzupuvos: dsmetool can run as user afaik20:10
Wizzupstart as rather20:10
Wizzupbut yeah all good, just saying that's how it -could- work20:10
uvosit can start user20:10
uvosi think this is broken20:10
uvosit needs to restart all sphone proceies for all uids its running for20:11
uvoswith proper dbus user bus and sutch20:11
uvos*start as user user20:11
Wizzupright20:12
Wizzupbtw, why does it need to be restarted upon update?20:12
uvosit comunicates with itself via dbus20:13
uvosthe interface is not stable atm20:13
uvosand not versioned either20:13
Wizzupright20:15
siceloi just thought how nice it would be to have a hildonized matrix client20:50
sicelommm, i wonder if telepathy has a matrix plugin20:50
Wizzupyes20:54
Wizzuptelepathy-tank20:54
Wizzupplease give it a try if you can20:54
Wizzup(in empathy or so)20:54
sicelooh, interesting! i'll have a look20:58
siceloWizzup: btw thanks for investigating the wifi issue on n900's wl1251. with that fix, iwd runs beautifully21:11
humpelstilzchen[Is Audio not working on pinephone a known issue?21:25
humpelstilzchen[alsa output works when pulseaudio is stopped, so probably pa issue21:25
Wizzupsicelo: good21:42
Wizzuphumpelstilzchen[: I think rafael2k is on it21:42
humpelstilzchen[ok21:56
lelsicelo opened a pull request: https://github.com/maemo-leste/droid4-battery-calibration/pull/1 (Correct the script path)23:34

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