libera/#maemo-leste/ Tuesday, 2021-10-12

lelIMbackK opened a pull request: https://github.com/maemo-leste-extras/simple-brightness-applet/pull/2 (port to new mce interfaces, removes all gconf dependancy)00:45
uvoshttps://github.com/maemo-leste-extras/simple-brightness-applet/pull/200:45
Wizzupuvos: thanks will try to pick these up tomorrow01:21
LjLhi people, new here, haven't used my N900 in a long time... is Leste in a state where it would be relatively usable with no major missing things on the N900? if not could i dual-boot it easily with classic Maemo? is the N900 even being targeted as viable anymore?04:15
* enyc meows LjL 04:53
* LjL meows back05:07
* joerg chases 5 dozen white mice towards enyc and LjL05:08
tmlindfreemangordon: so looks like we're missing the power domain handling for omap3, here's an initial patch for it muru.com/linux/n900/omap3-prm-gfx.patch06:05
tmlindthat get's modprobe of the pvr module working with no reverts for me, but there are other pvr kernel module related problems remaining06:05
tmlindit seems that the pvr module does internal powerdown for omap3/sgx530_125 and behaves somehow in a different way compared to omap406:06
tmlindor something like that, need to figure that out when i get a chance06:07
tmlindso pvrsrvinit will still fail for omap3 after the power domain patch06:07
tmlindlooks like somehow on omap3 PVRSRVSetDevicePowerStateKM fails to properly power up the device, the module is clocked as i can read the regs when enabling the module via sysfs manually06:39
tmlindoh well need to continue tonight06:39
freemangordontmlind: hmm, ok, maybe it counts on something being powered-up automatically.07:29
freemangordontmlind: yeah, with this patch pvrsrvctl --init --no-module hangs the device08:59
uvosLjL: "is Leste in a state where it would be relatively usable with no major missing things on the N900?" no absoulty not there are no working phone calls on n900 and lots of other stuff is also still missing11:32
uvosLjL: "is the N900 even being targeted as viable anymore?" yes it is, but our other 3 main supported devices are in a better state atm (particularly power managment is totaly broken on n900, there are sever bugs with the pvr driver and there is no routing for the n900s complex audio setup), but there is promesing work goning on right now on all of those problems so n900 should hopefully catch up soon11:35
uvosalso you need to temper your expectations with n900 wrt performance, its not going to run modern desktop applications like browsers anymore due to the limited ram.11:36
WizzupLjL: dual-booting is easy though12:02
WizzupI think once we get working phone call audio (+ audo integration) and sms I will personally switch though12:05
Wizzupdogfood all the way :)12:05
uvosbtw Wizzup is there a way to force the n900 to allways show the uboot menu12:08
uvoseven with the device closed?12:08
uvosi gues its hardcoded12:09
WizzupIt's check a gpio in u-boot12:09
Wizzupso it can be changed for sure12:09
lelIMbackK synchronize a pull request: https://github.com/maemo-leste-extras/profilesx/pull/1 (add support for touchscreen.vibration.enabled)12:17
uvosprofilesx works now12:17
uvoswrt adding removeing profiles12:17
Wizzupneat12:19
lelIMbackK synchronize a pull request: https://github.com/maemo-leste-extras/profilesx/pull/1 (add support for touchscreen.vibration.enabled)12:19
uvosid also take this package if youlike12:19
Wizzupgranted12:24
WizzupAlthough I think somethings needs to be done in jenkins as well12:24
Wizzupcc parazyd12:24
uvos"Also renames the cp applet to just Profiles, since its the only applet we have to change profile settings"12:25
uvoswhat do you think about this?12:25
uvosi found the control pannel profilesX name unseamly12:25
Wizzupyeah agreed12:27
uvosok i also added a new string: dgettext("osso-profiles", "profi_fi_touch_screen_vibrate")12:29
uvosnot sure where to add this12:29
Wizzuphttps://github.com/maemo-leste-translations/osso-profiles-l10n12:33
Wizzuphttps://github.com/maemo-leste-translations/osso-profiles-l10n/blob/master/po/osso-profiles.pot + https://github.com/maemo-leste-translations/osso-profiles-l10n/blob/master/po/en_US.po12:33
Wizzupand then hope others do the rest12:34
uvosi can do german at least :P12:34
* sicelo notes that reminding people that the N900 is underpowered for browsers is such a common theme here ... 12:47
Wizzuphehe12:48
parazyduvos, Wizzup: You need write permissions on the repo?12:52
Wizzupno, the question is how to add uvos to jenkins for that repo12:53
Wizzup(profilesx)12:53
parazydWe just add it to the bot's ACL12:53
Wizzupoh, so not in jenkins itself?12:53
Wizzupeither way I don't know where it's stored12:53
parazydDone12:54
parazydWizzup: Will pm12:54
parazydWe have a separation here12:55
parazydI see uvos prefers IRC, so I added him to the IRC ACL12:55
parazydOtherwise there's also the option to add to Jenkins' ACL, but then one has to use the Web UI to trigger builds.12:55
uvos"I see uvos prefers IRC, so I added him to the IRC ACL" heh thats a self fufilling Prophecy you only gave me IRC permissions to my other packages, ofc i use irc :P13:05
uvosnot that i mind13:05
WizzupI think that's another way of saying "the jenkins web ui is cumbersome to use" :)13:05
Wizzupalso to configure accounts13:05
uvosyeah its fine :)13:05
parazydYeah it sucks13:20
parazydWe also have python cli scripts to interface with it internally13:21
lelIMbackK closed a pull request: https://github.com/maemo-leste-extras/profilesx/pull/1 (add support for touchscreen.vibration.enabled)15:34
freemangordonuvos: that "profiles" rename is what is seen in control panel?17:47
Wizzupright17:48
freemangordonhow is that rename done? in .desktop file?17:49
freemangordonthis https://github.com/maemo-leste-extras/profilesx/pull/1/commits/6ce72179ab4820816a1de8bf87d4b7d6321ceef0 ?17:49
Wizzupyes, looks like it in the pr17:50
freemangordonthis is a hack17:50
freemangordonEnglish name should not be in the desktop file like that, IIUC17:51
freemangordonfor example sharing-accounts.desktop:17:52
freemangordonName=share_cpa_ti17:52
freemangordonX-Text-Domain=osso-sharing-ui17:53
Wizzupmakes sense, uvos^^17:55
uvosfreemangordon: hmm ok18:54
uvosfreemangordon: currently this string was never translated18:54
uvoswhat X-Text-Domain would you suggest to put this string?18:54
uvosits own18:54
uvosprofilsx dosent have one18:54
freemangordoncheck fremantle one18:54
uvosthis is an extras pacakge18:54
freemangordonsec18:55
uvosit wasent in fremantle as sutch18:55
uvosbtw this method violates xdg spec18:55
uvosfor translations18:55
uvosnot that it matters mutch18:55
freemangordonfremantle has "profiles"18:55
uvossince its a plugin18:55
uvosok18:56
freemangordonlemme check what is the package name18:56
uvoscould you check its desktop file18:56
freemangordonsure18:56
uvoson a running system simply18:56
freemangordonyes doing that as we speak :)18:56
freemangordonuvos: maemo-applet-profiles is the package name18:59
freemangordonI would recommend to download and have alook at it18:59
uvoswhere are fremantle repos?18:59
freemangordonthere is more that just transaltions that shall be done for a cpl applet replacement18:59
freemangordonsec19:00
freemangordonhttps://coderus.openrepos.net/n900mirror/downloads.maemo.nokia.com/fremantle/ssu/mr0/19:00
uvossure im also not sure profilesx is a permanent solution19:00
uvoswe should just have _something_ for now to set the ring tone and sutch19:00
freemangordonthen I'll RE the original applet19:00
uvossure if its better19:01
freemangordonuh, 37372 bytes19:01
freemangordonthis dude has lots of stuff in it19:01
uvosmaybe do it some other time then19:01
freemangordonyeah19:01
uvosprofilesx can serve19:01
freemangordonyep19:01
freemangordonuvos: btw, still, have a look at stock applet19:02
uvosok19:02
freemangordonwhat I meant was CUD and RFS scripts19:02
freemangordon(Clear-User-Data, Restore-Factory-Settings)19:03
freemangordonthose are important19:03
uvosnot sure why those are particulary important19:06
uvosbut ok19:06
uvoswould also want to change this to not depend on /home/user19:06
freemangordonbecause - OS is intended to be used on mobile devices, being able to restore to factory setting and to clear user data was there before even flashlight capability19:08
uvosanyhow i renamed the cp applet19:15
uvosparazyd: would you move profilesx to core and add it to the metapackages at your leisure19:16
uvosas it looks like its going to be our default profile applet for a while19:16
freemangordontmlind: who controls CM_xxx_SGX and PM_xxx_SGX regs and are we sure those have correct values? how could I verify?19:45
freemangordonI think I maybe should edit the driver to call EnableSGXClocks() on exit from probe and then verify if allregisters look ok19:51
freemangordonshall I verify other registers but PRCM?19:52
tmlindfreemangordon: the clocks should be ok, but the rate may need to be configured. the clocks are claimed by drivers/bus/ti-sysc.c and managed by runtime pm20:08
tmlindthe clock rate and parent clock can be configured with assigned-clock-parents etc in the dts like we did for droid420:09
freemangordonbut wrong rate should not lead to bus error but bad performance, no?20:09
tmlindtoo high clock rate would cause crashes for sure20:09
freemangordontmlind: also, I checked the rate (when it was working) and it was ok, 110666 or somesuch20:10
freemangordonI compared with fremantle20:10
tmlindok20:10
tmlindwhat is the source clock supposed to be20:10
freemangordonthe clock parent was same in either cases20:10
freemangordonnot sure, but I cam check20:11
freemangordon*can20:11
freemangordonshall I?20:11
tmlindsure makes sense, the clock rate can be found in /sys/kernel/debug/clk or something like that20:12
freemangordonyeah20:12
freemangordonthat's where I look20:12
freemangordon:)20:12
tmlindok, then checking the fremantle source clock can be done by dumping regs20:12
freemangordonI am on kernel-power20:13
freemangordonwhich exports clock tree in /sys/kernel/debug20:13
freemangordonthat's why I said that I compared the clocks and they are same20:13
tmlindok20:13
freemangordonroot@devuan-n900:/sys/kernel/debug/clk/sgx_fck# cat clk_parent20:16
freemangordoncore_d3_ck20:16
freemangordonroot@devuan-n900:/sys/kernel/debug/clk/sgx_fck# cat clk_rate20:16
freemangordon11066666620:16
freemangordontmlind: ^^^20:16
freemangordonl3_ick is parent of sgx_ick20:17
freemangordonclk_rate is 16600000020:17
freemangordontmlind: whall I disable off mode?20:18
freemangordon*shall20:18
tmlindshould not matter, but worth trying :)20:19
freemangordonok, what else could be then?20:19
tmlindso is fremantle also using 110666666 and core_d3_ck?20:20
freemangordonlemme check to be 100% sure20:20
tmlindok 110 sounds about right to me though20:21
tmlindon droid4 we have 307 looking at commit fab030ac99978d235ca3889262b28d93c8bccd3020:21
freemangordon/sys/kernel/debug/clock/virt_19_2m_ck/osc_sys_ck/sys_ck/dpll3_ck/dpll3_m2_ck/core_ck/sgx_fck20:23
freemangordon11066666620:23
freemangordonon fremantle20:23
tmlindok20:24
freemangordondo you need ick?20:24
tmlindnah20:24
freemangordon/sys/kernel/debug/clock/virt_19_2m_ck/osc_sys_ck/sys_ck/dpll3_ck/dpll3_m2_ck/core_ck/l3_ick/sgx_ick20:24
freemangordontoo late :)20:24
freemangordonhmm20:25
freemangordontmlind: on fremantle there is no core_d3_ck20:26
freemangordonor maybe it is just not important, as soon as the rate is fine20:27
freemangordonalso,20:27
freemangordonif we do not play with PM, it works fine20:27
tmlindyeah weird20:27
tmlindthe same clocks are enabled even without pm so those should be fine then20:28
uvosjust check the clock resigers after the hang over jtag?20:28
uvosthe pins on the back of the n900 have jtag right?20:28
freemangordonmaybe, but I dont; have the equipment to do that20:28
tmlindwell i force enabled sgx module via /sys and can read the regs just fine before prvsrvinit20:29
uvossure but the dirver might be messing with the clock regs in an unexpected way after prvsrvinit20:29
freemangordonpvrsvrinit does the same as pvrsvrctl?20:29
uvosi have no idea what the point of pvrsrv is even20:30
uvosfreemangordon: yes20:30
uvosit proububly uploads fw?20:30
tmlindit should not even know about the clock regs.. but maybe there's some pvr internal gate or multiplier20:30
freemangordonuvos: the driver does not touch clock regs, besides calling pm_xxx functions20:30
uvostmlind: it knows for sure20:30
freemangordonuvos: not really20:30
uvosi know the userspace knows the clock and decides behavior based on it20:30
freemangordonclocks are controlled by linux20:30
uvosno here me out20:31
uvoson ddk1.9 and the android blobs20:31
tmlinduvos: i don't think we have the pvr freq scaling enabled in the driver20:31
freemangordoneven of remantle, where you have freq scaling, it is done by the driver20:31
freemangordonit gathers performance data based on load that's reported by uKernel20:32
uvosso on d4 on the older blobs (icl android) there is wierd behavior where the driver changes the frequency based on load and the display refeshing20:32
freemangordonand decides to change opp20:32
uvosbut only if its set to exactly 307Mhz20:32
uvosif you set it to something else in linux20:32
uvosall this activity stops20:32
uvosthis code has to be _somewhere_20:33
uvosi faild to find it20:33
freemangordonyes, in the driver :)20:33
uvosok20:33
uvosthen i missed it while looking20:33
tmlindi guess it must be there buried into the 60k lines of ifdefs..20:33
freemangordonyeah20:33
freemangordonI can show it where it is in fremantle driver20:34
freemangordonit is not that different to what we use though20:34
tmlindwell the clock rate we can read in a loop from the regs while prvsrvinit is running20:34
uvoswell at the time i was concerned that pvrsrv uploads some fw and that sgx dose clock managment on device20:34
uvoslike modern desktop gpus20:34
tmlindthe driver certainly has all kinds of ioctl controls in place20:34
freemangordonno, sgx does what it has been told, at least for sgx530, AFAIK20:35
tmlindwould not be surprised if there's write to any reg ioctl there too20:35
freemangordoncould be20:35
freemangordonbut still the oops we get is a simple "resume from suspend" path20:36
freemangordonwhich behaves in exactly the same way with SUPPORT_ACTIVE_POWER_MANAGEMENT disabled, besides enabling/disabling clocks, IIUC20:37
freemangordontmlind: is it possible that there is some period after clocks being enabled but the PLL is still not stable?20:38
freemangordonor some other timeout needed to stabilize the clocks?20:38
tmlindsure it takes a while to lock, but the pll should be enabled and we just mult/div/gate it20:38
freemangordonyeah20:39
freemangordonwhat about those prcm regs? reset one?20:39
freemangordonis it possible that for some reason SGX is hold in reset?20:40
freemangordonand how to verify?20:40
uvosirrc there is a reset reg for sgx in omap420:40
uvosso just look at the omap3 datasheet and read it?20:41
freemangordonI think it is the same for omap3, reading through TRM as we speeck to find it20:41
freemangordon*speak20:41
freemangordonhmm, what about PM_PWSTCTRL_SGX20:42
freemangordon0x4830 6BE0 that is20:42
tmlindi don't think there's a rstctrl for sgx on omap3, if it was in reset no register access at all would work20:45
tmlindfreemangordon: the patch i posted this morning manages PM_PWSTCTRL_SGX20:45
tmlindhmm i guess i should check the 34xx trm for that reg though, that's from the dm3730 trm20:46
freemangordonI think it is the same20:48
freemangordonfrom 3430: PM_PWSTCTRL_SGX 0x4830 6BE020:49
freemangordonPM_PWSTST_SGX 0x4830 6BE420:49
tmlindlooks like the available modules are a bit different20:50
tmlindsorry available modes i mean20:50
freemangordonPOWERSTATE bits?20:51
tmlindyeah but we don't care about the inactive mode20:51
freemangordonhmm, in TRM I look at INACTIVE is defined for PM_PWSTST_SGX20:52
freemangordonthe readonly POWERSTATEST20:52
freemangordonso I am not sure what you mean in terms of PM_PWSTCTRL_SGX/POWERSTATE20:52
tmlindoh right, i was looking at the wrong reg..20:55
tmlindno statechange bit there it seems20:56
freemangordonhmm, what do you mean?20:59
freemangordonyou have INTRANSITION21:00
freemangordonhmm, seems we have /sys/kernel/debug/pm_genpd/prm_gfx with current_state 'off-0'21:03
tmlindsome prm have bit 4 for statechange, i doubt that matters in this case as the regs are readable21:04
tmlind/sys/kernel/debug/pm_genpd/prm_gfx21:04
freemangordondoes sysc (or whoever controls prcm state) waits for INTRANSITION to become zero?21:05
tmlindyeah the prm driver does21:06
tmlindmight be worth checking the /sys/kernel/debug/pm_genpd/prm_gfx state that it shows the real reg value and not the idle state value..21:07
freemangordonusingdevmem?21:08
freemangordonby using devmem?21:08
tmlindyeah21:09
freemangordonI have to find it first :)21:09
freemangordonok, not now, but tomorrow I will do a couple of experiments, unless you provide a fix by then :)21:10
freemangordonlike, recording the reg values with and without SUPPORT_ACTIVE_POWER_MANAGEMENT21:10
freemangordonbesides PRCM regs, what else shall I look at?21:11
tmlindchecking the values without SUPPORT_ACTIVE_POWER_MANAGEMENT seems like a good idea21:11
freemangordonok21:12
tmlindthe other things to look for are silly typos :)21:12
freemangordonwhere? :)21:12
freemangordonin the kernel source? :p21:12
tmlinddts, the patch i posted this morning, pvr changes affecting pm_runtime..21:13
freemangordonhmm, pvrsrvctl *DID NOT* crash this time21:15
freemangordonwith your patch that is21:15
tmlindweird21:15
freemangordonhowever, kmscube did21:15
freemangordonyes21:16
freemangordonbut, I disabled off mode21:16
tmlindthat sounds still like too high clock rate..21:16
freemangordonthis is controlled by tds, no?21:16
freemangordon*dts21:16
tmlindmaybe we have some cpuidle race with the PM_PWSTCTRL_SGX register access21:17
tmlindyeah the rate seems just fine though21:17
freemangordonyeah, rate seems fine21:17
tmlindhow about add some load to keep system from idling? like dd if=/dev/zero of=/dev/null &21:18
freemangordonalso, keep in mind we have issues where maybe half of the time device cannot boot21:18
freemangordonok, I can try that21:18
freemangordonnot now though, have to run21:18
freemangordonalso, I plan to weak the driver to try to leave the cloaks running after probe has finished21:19
freemangordonto see what is the actual state21:19
freemangordonanyway, ttyl21:19
tmlindok ttyl21:19
tmlindfreemangordon: so if omap3 or 34xx sgx ocp registers are broken like they seem based on what i noticed, maybe we should not enable SYS_USING_INTERRUPTS as that enables SGX_OCP_REGS_ENABLED, and seems SUPPORT_ACTIVE_POWER_MANAGEMENT depends on these..22:22
tmlindthere's also SGX_OCP_NO_INT_BYPASS option that might be for 36xx if the ocp reg got fixed or something22:23
tmlindanyways, ttyl22:23
freemangordonSYS_USING_INTERRUPTS interrupts alone (without SUPPORT_ACTIVE_POWER_MANAGEMENT) works perfectly fine22:23
freemangordonglmark score I posted yesterday was with SYS_USING_INTERRUPTS enabled22:24
freemangordontmlind: ^^^22:25
tmlindok, i guess they are also increasing in proc then..22:25
freemangordoncan't parse, sorry, could you rephrase22:25
freemangordonyou mean interrupt counter?22:25
tmlinduvos: adding SGX_OCP_NO_INT_BYPASS might enable some runtime gating as it tinkers with the sgx sysconfig reg22:26
tmlindfreemangordon: yeah so i'd assume fremantle has the sgx interrupts increasing? :)22:26
freemangordonyes, so is 1.17 with SYS_USING_INTERRUPTS enabled22:26
tmlindok22:27
freemangordon(13,53,07) freemangordon: root@devuan-n900:/sys/kernel/irq/37# cat per_cpu_count22:27
freemangordon(13,53,07) freemangordon: 6423422:27
freemangordon(13,53,07) freemangordon: root@devuan-n900:/sys/kernel/irq/37# cat per_cpu_count22:27
freemangordon(13,53,07) freemangordon: 6633622:27
freemangordonthis is while glmar is running22:27
freemangordon*glmark22:27
tmlindok22:27
tmlindprobably SUPPORT_ACTIVE_POWER_MANAGEMENT22:28
tmlindis for omap4 and later then22:28
tmlindmaybe omap3 assumes just gating using the sysconfig register or something22:29
freemangordonwhat I saw from the code, it just enables the code that calls pm_runtime_get()/pm_runtime_put()22:30
freemangordonso I doubt this is invalid for omap322:30
freemangordonolso, this seems to work fine in fremantle, where it calls clock_disable() instead22:31
freemangordon*also22:31
tmlindyeah ok, ttyl22:34

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