libera/#maemo-leste/ Friday, 2024-05-03

Wizzupsicelo: cool, we should see if/when we can get new ofono00:38
Wizzuparno11: thanks for making this issue00:39
Wizzupis anyone else on weblate?02:00
WizzupI am not sure I have an account there02:00
Wizzupas in, I can make one, but we need one that runs the orga02:00
Wizzupok, the chimaera-experimental kernel works on my mz617 at least02:44
siceloWizzup: since 6.6 built successfully, would you also add the N900 charger/fuel gauge patches to it?11:30
Wizzupsure, I think we can do that, uvos said he would rebase on latest 6.6 lts, so it might be better to ask him, but I can try11:31
Wizzupiirc he said there was also maybe something missing11:31
WizzupI'm looking at getting all the translations updated/merged11:41
Wizzuphere's an example diff https://github.com/maemo-leste-translations/hildon-application-manager-l10n/pull/4/files11:41
Wizzupthinking about how to do this, do I just manually merge the pr, make a changelog entry, and so forth? I guess so, right?11:42
Wizzupanyone up for helping with this?11:42
uvos__sicelo: its in que12:13
uvos__Wizzup: sounds like a task than wants automation12:13
uvos__*that12:13
Wizzupmhm, maybe, yes12:26
Wizzupuvos__: but yeah I figured we might want to look at the translation diff too :D12:52
Wizzupoh actually the kernel didn't work on mz617 because we don't have btrfs built in13:13
Wizzupuvos__: if/when you rebase, please rebase on maemo-6.6-builddeb13:15
Wizzupor rather, use that as a base, or whatever :)13:17
arno11Wizzup: hi, with a new build, N900 needs a small change in dts to be able to use pwm-ir-tx13:34
arno11this way Pierogi should work OOTB iirc13:36
Wizzupwhat is the change?13:47
arno11just one line: pwm-ir-tx instead ir-rx5113:47
Wizzupok13:48
arno11let me doublecheck13:48
arno11n900-ir {13:51
arno11compatible = "pwm-ir-tx";13:51
arno11pwms = <0x131 0x00 0x66cc 0x00>;13:51
arno11oops sorry13:51
arno11indeed just one line to change13:51
Wizzupdoes this rely on other patches, or?13:54
Wizzupcurrently the kernel has:13:54
Wizzup    ir: n900-ir {13:54
Wizzup        compatible = "nokia,n900-ir";13:54
Wizzup        pwms = <&pwm9 0 26316 0>; /* 38000 Hz */13:54
Wizzup    };13:54
Wizzupso you want to change n900-ir to pwm-ir-tx ?13:54
Wizzupor am I missing something13:54
Wizzupoh, wait, I think you're talking about 6.1?13:55
arno11i was talking about 6.6, maybe too late for 6.1. yes the change is "pwm-ir-tx" instead of "nokia,n900-ir"14:04
Wizzupthe pwms is also different as you can see14:04
Wizzupalthough the hex is the same as the regular number14:04
arno11yes that's the same14:04
Wizzupso no other patch is necessary?14:05
arno11no other, iirc14:06
arno11at least in dts14:06
Wizzupwell, that's what I mean, does that pwm-ir-tx need patches, or14:09
arno11iirc one patch from fmg has been mergerd and another one is still waiting for test14:10
Wizzupright, but if it is merged to say 6.9 we miht need to carry it14:11
WizzupI don't mean to be annoying, I'm just trying to make sure we don't have to spend more time later finding them, so if you know what the patches are... :D14:11
arno11i understand :) (but irtx stuff was long time ago)14:13
arno11i cn have a look later today and LYK14:13
arno11*for patches14:14
Wizzupok, just the first sentence of the commit will help14:15
Wizzup6.1 was also a long time ago fwiw :P14:15
Wizzuparno11 this mmc queue requests change, is that beneficial for other devices too?17:14
Wizzupand should it not be set for both mmcblk0 and mmcblk1? or?17:14
Wizzupok yeah with btrfs supported added the latest experimental kernel works on mz61717:23
kivaWhat! Nokia 2780 Flip has PostmarketOS testing support: https://wiki.postmarketos.org/wiki/Nokia_2780_Flip_(nokia-weeknd)17:50
kivaMaemo support some day? ;)17:51
Wizzupyeah if mainline starts to work17:52
sicelomain problem would be that hildon is primarily a touch interface :-)18:04
kivaActually older hildon (in N810 with Maemo 4) had good keyboard only use support. Menus can be used with arrow keys..it should put back in Leste. I have missed it with Pinephone Keyboard.18:08
kivaor just make that mouse emulation to scroll-key of Nokia 2780.18:11
kivabut maybe better start from making SIM-pin query support keyboard of Pinephone. Starting terminal with ctrl+shift+x works after that already..so some kind of keyboard only works already.18:15
kivaAnd that reminds me that one older guy see me using Pinephone keyboard and he said: "at last phone with keyboard with enough big buttons"18:19
Wizzuphe must have big pockets :D18:20
kivaHe haven't problem with size, but he had big fingers and looking for phone that can be used with them and asked can Pinephone use without touching screen..I have to answer: No, but icons are big enough.18:27
Wizzupright18:27
kivabtw..Calendar widget crash (and disappear) if touching in Pinephone, but works if Calendar is already started.18:30
Wizzupit disappears from home?18:30
kivayes18:31
Wizzuphm18:32
arno11Wizzup: yes you're right for nr_requests (both mmcblk0 and mmcblk1)19:26
arno11and yes possibly beneficial on other devices19:26
arno11btw for irtx stuff, that's indeed for 6.9:19:28
arno11https://github.com/torvalds/linux/commit/12590d4d0e331d3cb9e6b3494515cd61c8a6624e19:28
arno11https://github.com/torvalds/linux/commit/363d0e56285e80cda997d41d94c22313b673557d19:28
arno11and the patch to test (last message from Sean):19:30
arno11https://lore.kernel.org/lkml/ZaK5wOHxf2CvzZrk@gofer.mess.org/T/19:30
Wizzupok, so we should pull all of these19:46
Wizzuparno11: the last message on that mailing list doesn't seem to contain a test for me19:54
Wizzupbut I can pull these commits into 6.619:54
Wizzuparno11: so those two and the dts change?19:56
Wizzuparno11: so just this:19:57
Wizzup ir: n900-ir {19:57
Wizzup-compatible = "nokia,n900-ir";19:57
Wizzup+compatible = "pwm-ir-tx";19:57
arno11compatible = "pwm-ir-tx" instead of compatible = "nokia,n900-ir"20:02
arno11that's it20:02
arno11and yes, the 2 commits are needed + dts change20:03
Wizzupok I'll put this in the experimental branch20:04
arno11ok and fo the patch to test see:20:05
arno11https://libera.irclog.whitequark.org/maemo-leste/2024-01-3020:05
Wizzupis pierogy already packaged?20:05
Wizzuparno11: why do I need the patch to test, just wondering20:05
Wizzupor maybe I am confused what you're saying / asking20:06
arno11yes sorry i'm a bit buzy and probably unclear as usual ;)20:06
arno11let's talk about the test later20:07
arno11*busy20:07
Wizzupok, but this test is userspace, or do you need kernel work done20:08
arno11pierogi is not already packaged but works localy20:08
arno11(kernel work to do: mark pwm as atomic)20:09
arno11bbl20:09
arno11Wizzup: (for the test) Sean asked to test something on the N900: set the atomic field like the raspberry pwm does in pwm-omap-dmtimer.c21:25
arno11https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/pwm/pwm-omap-dmtimer.c#n39521:25
arno11we need to add omap->chip.atomic = true;21:26
arno11(according to freemangordon)21:26
arno11just after chip->ops = &pwm_omap_dmtimer_ops;21:28
arno11(and if i understand correctly lol)21:28
arno11with all of that, i think we are ready to package pierogi and avoid kernel oops using it :P21:34
arno11i mean, dts change + 2 commits + atomic stuff21:44
Wizzuparno11: ok, so a test to see if things work well with the atomic field set?22:30
arno11Wizzup: yes22:34
Wizzupok, but we can also try it without, yes?22:40
WizzupI am just trying to see if I should build a kernel now or not22:40
Wizzupwith/without this additional code22:40
arno11good question22:42
arno11maybe we can try without first22:47

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