libera/#maemo-leste/ Monday, 2022-06-06

uvosWizzup: no08:43
uvosi ment 5.16 as stated, i was mentioning that i have used this >5.15 version for quite some time and have split the 5.15 patch set in feature branches, not that we should switch to this version in particular08:43
uvossicelo: yes its eol08:44
Wizzupuvos: ok09:24
bencohhttps://www.hackster.io/reimunotmoe/notkia-f6e772 new target? :)11:01
bencoh(iiuc the current version has no modem)11:03
Wizzupnot a lot of ram11:05
Wizzupiirc11:05
bencohyeah, 64M, makes the n900 feel bloated :)11:05
uvosits less powerfull than the d4's modem :P11:08
Wizzupuvos: ok so my plan is the following:11:24
Wizzup1. make a simple py script to quickly restore specific 'groups' of registers11:24
Wizzup2. make some diff between before a call ,during a working call, and after a call (to restore them)11:24
Wizzup3. look into the differences, but only for handset for now11:24
Wizzupsome other thoughts I had, you said with the current sound driver design, the kernel turns off entire parts, can we just make some control (on/off) that keeps the whole tree on synthetically, and just set it as part of the ucm?11:25
Wizzupof course it's a hack, but it's way better than any kind of reg writing11:25
Wizzupuvos: like, for this part:11:32
Wizzup>The problem is the that the voice if is considered inactive by the kernel, this is because the kernel is not using this if at all. Because it isent, the modem is using it. this causes all the downstream nodes to be put in off state.11:32
Wizzupor is this what you tried earlier already, with forking the driver?11:32
uvoslol11:33
Wizzuphm?11:33
uvosnot you11:33
Wizzup:)11:33
bencohI'm fiddling with alsa drivers at work as well, and ... alsa can really be a pain sometimes, especially with badly designed drivers :/11:34
uvosi just realized that systemui, to play the alarm ringtone sends a dbus message to hildon-sv-notification-daemon wich then sends a dbus message to system ui to play the sound11:34
uvos"some control (on/off) that keeps the whole tree on synthetically" i tried doing that11:35
uvosbut i cant find a way to do so via the api avialable to the codec11:35
uvos(ie the only part we control)11:35
Wizzuphmm11:35
uvosbut i might have missed something11:35
uvosin general how this works is pretty cryptic to me, since alsa parses this big struct with string arguments11:36
uvosthat arnt documented anywhere11:36
uvosso you have to read quite a bit of code to figure out what anything dose11:36
Wizzupok, well, let me first fiddle a bit more with these register values and such11:36
Wizzupthe hack was just something that occured to me on my walk today11:36
uvosand hildon-sv-notification-daemon is owned by hildon-home11:40
uvosthese people where high11:40
Wizzupuvos: https://dpaste.com/6Z8DCHMB3.txt12:27
WizzupI need to check if I got the endianness correct12:27
Wizzupbut I think so?12:27
uvosi mean that makes sence12:37
uvos*sense12:37
uvoshose are the right regs12:37
Wizzupyeah, I also added a restore one that only restores audio regs12:44
Wizzupa restore one => a restore option12:45
Wizzuphopefully this will aid in debugging somehow12:45
Wizzupmaybe someone already wrote it , but I didn't have it :)12:45
Wizzupthis helps me not break my brain reading the reg defines12:45
Wizzuphttps://github.com/MerlijnWajer/droid4-cpcap-audio-reg-tool12:47
Wizzupit's here12:47
buZzhmmf, seems on d4 nowadays i get -all- sms stuck in tx_queue when i send it, even when i send it directly as a reply12:48
buZza reboot (and 1 tap on icd2 to get ofono to exist(?)) gets them sent12:49
WizzupbuZz: yes, this is again all and the same ofono problems12:49
WizzupI plan to look at that /after/ the audio things12:50
Wizzuptrust me I run into the same problems every day :)12:50
buZzah, right, np, it just seems new to me12:51
Wizzupbasically almost all ofono issues boil down to ofono not realizing the kernel sent it a message12:53
uvosoh god this is terrible12:53
Wizzupand then it gets it either way later or potentially never12:53
Wizzupthis is also why the internet activation sometimes just times out12:53
WizzupI have made a habit of typically activating the internet, and then 10 seconds later sent myself on my other number a sms12:53
Wizzupthat usually makes it activate right away12:54
Wizzupso yeah, this needs fixing12:54
uvosso to activate systemui hildon-sv-notification-daemon dlopens a systemui plugin (yes) and then uses it (so we now have 2 copies of the alarm subsystem of systemui running12:54
uvosand then the code is desinged sutch12:54
Wizzupuvos: freemangordon: might have some ideas/hints on this too, I think he knows how it works quite deeply12:55
uvosthat if it it cant decode the sound file (wich is provided by the user mind you) it just errors out12:55
uvos(and passes glib NULL in a bad place too causing a assert)12:55
uvosthis is why vibration dosent work12:55
uvosim cerntenly never going to use this alarm system for anything valuble after seeing how it works12:56
Wizzupit works well on the n900 for my for years12:56
Wizzupbut yeah, interesting :)12:56
Wizzupso what is our actual problem?12:56
Wizzupthe sound file or the glib null ptr?12:56
uvosthe decode function fails on any file i give it12:56
uvosnot sure why yet12:57
uvosalso this yeah12:57
uvosbut thats a side effect12:57
uvosof it having poor error checking12:57
uvosthe duplicated alarm system in systemui is only used in actdead btw12:58
uvos(same code running twice)12:58
Wizzupis something gstreamer missing?13:00
buZzah right, i was wondering, the /usr/share/sounds/ looked a bit empty, were the old sounds from nokia not released freely?13:01
Wizzupthey are not I think13:01
uvosit dosent use gstreamer13:01
uvosthose files would not help13:01
uvosit fails on any file13:01
uvoseven those13:01
Wizzupuvos: in any case, great work on digging in so far, sorry to hear the architecture is a bit meh...13:01
Wizzupok13:02
buZzWizzup: hmmm, so maybe we need a bunch of new sounds?13:02
uvosanyhow lunch13:02
uvosbuZz: yeah13:02
uvosbuZz: i suggested using the andoid asop soundset for now13:02
buZzis there a list of sounds we need?13:02
uvossince its apache213:02
buZzi was considering asking a musician13:02
uvosbuZz: sure13:02
uvosidk list13:02
uvosjust look on freemantle13:02
uvos*fremantle13:03
buZzhmm, i'll check around a bit13:03
buZzyeah , lemme just install it on my d4 :D apt install fremantle13:03
buZzhehe13:03
buZzhehe well, i can dig up a n900 i guess13:03
uvosor look at the pkgs13:03
buZznot really sure where they went after the last 4 moves13:03
uvosi gues13:03
buZzuvos: or enable inotify logging and run -all- the programs13:03
buZz:D13:03
buZzhaha13:03
buZzjk13:03
buZzactually, would be funny13:04
uvosalso very slow13:04
buZzsome auto-synth that makes a 'new sound' based on the filename of a unfound .wav13:04
Wizzupbrb lunch13:04
buZzeet smakelijk13:04
uvoswould be a derivatve work13:04
uvosithen13:04
uvosie a violation13:04
uvosi would just source some well licenced samples13:05
buZzhuh? i dont think so13:05
uvos(ex the android ones)13:05
uvosoh unfound13:05
uvosnvm yeah sure13:05
buZzi mean, a synthesizer can use anything as input, human turning knobs, ascii bytes from filename, etc13:05
buZzyeah13:05
uvosright13:05
buZzbuuuut, i think i'll just go make 'some' sounds , just a set of a couple13:06
buZzsomething with nice long stretched echos or something13:06
buZzas placeholder at least, for stuff i found missing13:06
buZzand just copy those over for as many as i need, so i can build a list13:06
buZzand later ask some musician to make a whole 'theme' just off the filenames13:07
uvossure yeah, that would be nice13:07
buZzimho leste should have its own sounds13:07
buZzlike the basic nokia sounds, triggering nerds worldwide into 'omg you're running leste??'13:07
buZz:D13:07
uvosi do like the ATT model of 500 telephone sphone as has default ringtone13:08
uvosso we do have that one covered13:08
buZznot sure if i ever heard that13:09
uvoshttp://uvos.xyz/maserati/videos/ringings.mkv13:10
uvosits not currently the default when you compile sphone on leste, becuse it uses libprofile there13:11
uvoswich has the non existant nokia tune as default ringtone13:11
uvosbut if you use sphone on anything else this is default13:11
buZzah well13:11
buZzmaybe i could imitate the nokia sounds on a kazoo?" :D13:12
buZzhahaha13:12
norayri didn't add dvorak layout last time, because i don't know how to name the dvorak layout.14:00
norayrin X11 that's again en_US but the variant is dvorak14:00
norayrand we in hildon input method plugins have14:00
norayrthe xml file named en_US14:01
norayrshould i do en_DV ? but that would mean wrong country.14:01
uvosso decoding works by passing a filenname to a com.nokia.NsvDecoder and then making assumptions that it saves a equivalent wav file in a magic folder (it dosent)15:05
uvosalso seams realy racey15:05
Wizzupuvos: hm... so the wav is to not have to decode in the future?15:05
Wizzupis that the thought?15:05
uvosyeah but no idea why they do this in a seperate process via dbus15:05
uvosjust call gstreamer youself15:06
uvosthey also dbus_g_proxy_begin_call15:06
uvosand then assume the file exists right after15:06
uvosi dont see how thats not really racey15:06
Wizzuphmmmmm15:06
Wizzupwhich code is this?15:06
uvosunless im missing somehting15:06
uvosnsv_decoder_decode in hildon-plugins-notify-sv15:07
Wizzupit looks like _nsv_decoder_decode_cb is called once the call is gone, which then call the real callback?15:08
uvossure15:09
Wizzupeven though _nsv_decoder_decode_finished_cb seems empty15:09
uvosexpect thats not how it works15:09
Wizzupbut I guess maybe that gets override somewhere15:09
uvosso the code that plays the sound assumes the wav exists allready15:09
uvosand fails hard when it dosent15:09
uvos(ie using NULL)15:09
uvosand the call to decode is issued by the ui15:09
Wizzupaha15:10
Wizzupmaybe the idea is to isolate errors in the decode sw so another process somehow15:11
Wizzupjust thinking out loud15:11
uvosyeah15:11
Wizzupthat is not bad per se15:11
uvosi dont know what process implements com.nokia.NsvDecoder atm15:11
Wizzupwant me to grep?15:12
uvosError: GDBus.Error:org.freedesktop.DBus.Error.NameHasNoOwner: Could not get PID of name 'com.nokia.NsvDecoder': no such name15:12
uvosnothing?15:12
uvosi gues15:12
uvosyeah please do15:12
Wizzupgrepping15:12
Wizzup./hildon-plugins-notify-sv/com.nokia.NsvDecoder.service.in:Name=com.nokia.NsvDecoder15:16
Wizzupthat is where the def is at least15:16
WizzupI see ./hildon-plugins-notify-sv/src/nsv-decoder-service.c15:16
Wizzupso that -should- be the service15:16
uvoshmm15:16
uvoswhy dose gdbus think its not reigsterd then15:16
Wizzup/usr/libexec/nsv-decoder-service seems to start ok15:17
Wizzupon my d4 at least15:17
Wizzup(no error)15:17
Wizzupor... hm15:17
Wizzupyeah, exits after a bit with exit code 015:17
Wizzupdo you see the call happen on dbus?15:17
uvosdbuis activation maybe15:17
uvossec15:18
Wizzupdbus-monitor will report if activation fails somehow15:18
uvosi dont see the call15:21
Wizzupdoes gdb get there?15:25
uvosno15:26
uvosim not sure what is supposed to trigger it atm15:26
Wizzupwhat if you just call the interface using dbus-send or so?15:27
Wizzupdo you know what the category and source/target file name should be?15:28
uvossure15:28
Wizzupmdbus2 com.nokia.NsvDecoder /com/nokia/NsvDecoder com.nokia.NsvDecoder.Decode "category" "/tmp/test.flac" "/tmp/decoded.wav"15:29
Wizzup()15:29
uvosdid it work?15:29
Wizzupsignal time=1654522055.922912 sender=:1.153 -> destination=(null destination) serial=9 path=/com/nokia/NsvDecoder; interface=com.nokia.NsvDecoder; member=ErrorDecoding string "category" string "/tmp/test.flac" string "/tmp/decoded.wav"15:29
Wizzup(nsv-decoder-service:9129): GStreamer-WARNING **: 16:28:42.426: 0.10-style raw audio caps are being created. Should be audio/x-raw,format=(string).. now.15:30
Wizzup(nsv-decoder-service:9129): GLib-GObject-WARNING **: 16:28:42.428: ../../../gobject/gsignal.c:2524: signal 'new-decoded-pad' is invalid for instance '0x669170' of type 'GstDecodeBin'15:30
Wizzupso if you run the program manually (from libexec) and then while it runs, run the mdbus2 stuff, you should see something similar15:30
Wizzup"decodebin2" has been renamed to "decodebin", with similar API. Note that there is no longer a "new-decoded-pad" signal, just use GstElement's "pad-added" signal instead (but don't forget to remove the 'gboolean last' argument from your old signal callback functino signature).15:32
Wizzupfrom: https://gstreamer.freedesktop.org/documentation/application-development/appendix/porting-1-0.html?gi-language=c15:32
Wizzupdo you want me to try to fix this on my d4?15:33
Wizzupor are you on it?15:33
uvosim afk ffor a couple of hours15:34
uvosill do it after15:34
WizzupI will look at it now15:34
uvosor you can if you like15:34
Wizzup(nsv-decoder-service:13166): GStreamer-WARNING **: 16:41:48.041: 0.10-style raw audio caps are being created. Should be audio/x-raw,format=(string).. now.15:43
Wizzup0:00:01.256317139 13166   0x63ffa0 WARN                 basesrc gstbasesrc.c:3583:gst_base_src_start_complete:<filesrc0> pad not activated yet15:43
Wizzup0:00:01.315917969 13166 0xb5b02a60 WARN                 flacdec gstflacdec.c:764:gst_flac_dec_handle_frame:<flacdec0> Lost sync, flushing decoder15:43
Wizzup0:00:01.321289063 13166 0xb5b02a60 WARN               baseparse gstbaseparse.c:3611:gst_base_parse_loop:<flacparse0> error: Internal data stream error.15:43
Wizzup0:00:01.322570801 13166 0xb5b02a60 WARN               baseparse gstbaseparse.c:3611:gst_base_parse_loop:<flacparse0> error: streaming stopped, reason not-linked (-1)15:43
Wizzup0:00:01.325286865 13166 0xb5b02a60 WARN                 basesrc gstbasesrc.c:2445:gst_base_src_update_length:<filesrc0> processing at or past EOS15:43
Wizzupwell, I'll look at what's up15:43
Wizzuphm, not sure if I am succeeding at making any gstreamer stuff work :)15:57
Wizzupso I think the problem is here:16:12
Wizzup0:00:02.305450440 13535 0xb5c02c60 INFO                GST_PADS gstpad.c:2378:gst_pad_link_prepare: trying to link decodebin0:src_0 and encoder-bin:sink16:12
Wizzup0:00:02.305908203 13535 0xb5c02c60 INFO                GST_PADS gstpad.c:2434:gst_pad_link_prepare: caps are incompatible16:12
Wizzupuvos: ok yeah it's simple I think16:20
lelMerlijnWajer opened a pull request: https://github.com/maemo-leste/hildon-plugins-notify-sv/pull/1 (nsv-decoder-task: fix problems in gst 1.0 port)16:25
Wizzupfreemangordon: https://github.com/maemo-leste/hildon-plugins-notify-sv/pull/116:25
Wizzupuvos: ^16:25
Wizzupnote that the file gets removed by something, I suppose the decode service itself, but it is there and the decoding seems to succeed16:25
Wizzupuvos: so I am pretty sure the UCM enable/disable sequences need to be amended with at least the 'Left' and 'Right' capture entries16:51
Wizzupuvos: should I make a PR for that?16:51
uvosWizzup: just do it :)17:30
WizzupI will once I know what else I need to change17:31
Wizzupuvos: lmk if the gst fixes the problem btw :)17:33
WizzupI definitely fixes decoding for me17:33
Wizzups/I/it/17:34
uvosill test continue on this later17:39
uvosi doubt it works tho17:39
uvosbut it will get further17:39
uvosit uses several x-maemo-* calls on pulse after17:40
uvosthose cant really work17:40
Wizzupyes, that's what I meant at the time wrt pulse17:41
uvossure yeah17:41
uvosbut it getting there is a big improvement allready17:42
Wizzupuvos: how did you get to these names:17:42
Wizzup        cset "name='Voice Loopback Switch' off"17:42
Wizzup        cset "name='Voice Playback Volume' 9"17:42
WizzupI don't see things named like this in amixer17:42
Wizzupamixer -c 0 specifically17:42
Wizzupis it just the mixer control + 'Volume'17:43
uvosamixer controls17:43
Wizzupty17:44
Wizzuphow would you know what maps to what?17:45
WizzupI can imagine that numid=17,iface=MIXER,name='Left Capture Route' could be the 'Left' mic that I am changing to 'Mic 2' or so17:45
Wizzupah, contents.17:45
freemangordon uvos: about x-maemo- things - what is the replacement?17:50
lelfreemangordon closed a pull request: https://github.com/maemo-leste/hildon-plugins-notify-sv/pull/1 (nsv-decoder-task: fix problems in gst 1.0 port)17:52
Wizzupwe have to made our pulse config17:53
Wizzupand add those things17:53
Wizzupor figure out an entire different way to do all of the routing that exists17:53
Wizzupin any case we're not there yet17:53
freemangordonWizzup: ah, so I didn;t get what uvos meant it seems17:53
freemangordonok17:53
freemangordondo you want me to do a changelog/release for hildon-plugins-notify-sv?17:54
siceloHehe, @alarm problems on  n900. That alarm is the ONLY alarm which can be relied on to wake one up even when device is off, in the entire mobile linux space (including Android, at least I've used about 6 different android phones/versions over the years, and they never could wake up for an alarm, unless I just had bad luck)17:55
Wizzupfreemangordon: yes, I think so17:55
freemangordonok, will do17:55
freemangordonsicelo: I was away, what problems to you mean?17:56
freemangordon*do you17:56
Wizzupit was regarding comments from uvos earlier I think17:56
Wizzup+ the gst stuff I fixed17:56
uvosthe implementaion of alarms is not so great17:56
uvosits fairly fragile17:57
uvosand pulls in a lot of stuff for farily limited reasons17:57
freemangordonmaybe I shall read the backlog about that17:58
sicelofreemangordon - I was reading backlog, yes.17:58
freemangordonuvos: unless you can summarize what you think is bad17:59
freemangordonotherwise I igree there are issues17:59
freemangordon*agree17:59
freemangordonone of them is that it is possible that you cannot stop an alarm immediately after you start it18:00
freemangordonalarm tone that is18:00
Wizzupuvos: btw18:00
freemangordonso deffinitely there is a field for improvement18:00
Wizzuphttp://dpaste.com/BMREQWKHT18:01
Wizzupthese are the exact bit-by-bit differences18:01
WizzupI think not all the bits are necessary for the calls18:01
Wizzupthe mic gain ones probably don't matter18:01
Wizzup(as they are just a bit diff)18:02
siceloYou're comparing on/with Android?18:02
WizzupCPCAP_REG_VAUDIOC + CPCAP_REG_CC are probably the 'kernel route' related ones18:02
Wizzupmaybe also the SDAC and SDACDI ones18:03
freemangordonuvos: but, besides that, alarm on n900 is what I am using to wake me up for the last 11 years. It has *NEVER* failed, no matter what18:04
bencoh:)18:05
freemangordonwell, not after I pushed alarms queue fix in CSSU :)18:06
norayrfreemangordon, may it be you know what to do if i'd like to create a dvorak layout?19:10
norayri didn't add dvorak layout last time, because i don't know how to name the dvorak layout.19:10
norayrin X11 that's again en_US but the variant is dvorak19:10
norayrand we in hildon input method plugins have the xml file named en_USthe xml file named en_US.19:10
norayrshould i do en_DV ? but that would mean wrong country.19:10
norayr(for some reason pasted twice)19:10
norayri mean my problem is the naming of the file.19:10
norayri don't see anything like 'variant' in our xml files.19:11
WizzupI don't think this is how it works19:22
Wizzupisn't there a layout for en_US ?19:22
uvosWizzup: him dosent do variants afaik19:28
uvosfreemangordon: it working on fremantle is a poor indicator of anything, since it was dead on arrival and never chainged.19:29
uvosfreemangordon: i just noticed that the alarm system needs a large amount of stuff to work via lots of dbus round trips and is not fault tollerant at all, because it either dose no error checking and ends up asserting in glib-critical because its using NULL pointers or it just bails19:31
uvosie: sound files missing? assert, decode /gst not working on user file? bail, new pulse audio so one of your modules dident load? just bail. systemui dosent respond in timeout? bail19:32
uvosetc19:32
norayrWizzup, yes there is a layout for en_US19:32
norayrbut that's a qwerty layout19:32
norayrit's really hard to type qwerty, especially on big pinephone on-screen kbd.19:33
norayrso i need my native dvorak layout.19:33
norayri am not sure how to name the xml file.19:33
uvosid really expect a alarm system to at least try and continue if it cant find a sound file or cant set the volume etc19:33
norayrwe have en_US19:33
norayrand en_GB19:33
norayrin X11, however that's19:33
norayrfile named us, and different layouts in it, for english in usa.19:35
norayrone of those is qwerty, one is dvorak.19:35
norayr(on a computer i use en_US(dvp) which means en_US, but the layout is programmer's dvorak.19:35
norayrlet's say in Bulgarian you have several variants:19:36
norayrbds, phonetic, bas_phonetic, bekl, latin19:36
norayrok i guess i'll just name the file somehow.19:37
norayren_dvorak, and let's see how it'll compile.19:37
Wizzupuvos: ok, with some hacks I reliably have call audio now upon calls19:40
Wizzupbut it's revealing some other problems, but that's all fine, teporary19:41
Wizzupnorayr: well, yes, variant is what you need to add, no?19:41
uvosWizzup: the problem is not his x layout19:42
uvosits the him vkb layout19:42
Wizzuphow does it work on fremantle?19:43
uvosno idea19:43
uvosdose it work?19:43
WizzupI don't know, I was never masochistic enough to try :P19:43
Wizzupso one of the funny things about sphone module order is that it will either do pulse first and then my hack, or reverse19:44
Wizzupbut ideally it would be pulse first, then the hack, but when the call ends, reverse order19:44
Wizzupbut I guess that's for another time19:44
Wizzupnorayr: you have read this right https://wiki.maemo.org/Remapping_keyboard#Multiple_Layouts19:44
Wizzupit might not be for the vkb either, not sure19:45
freemangordonuvos: I see. Well, happy path is more or less fine, so we can just add error checks/processing when needed19:45
freemangordonand fix the issue I mentioned earlier19:45
Wizzupuvos: so many of the registers to seem to get set in the kernel code, but somehow don't actually make it all the day?20:38
Wizzupall the way*20:38
Wizzupcpcap_voice_call seems to set CPCAP_BIT_VAUDIO_MODE1 at least20:39
uvosseting of registers is conditional to the nodes being active20:42
uvosthey are not20:42
uvosWizzup: hmm now trying to set a ringtone in the clockui cause it to segfault20:45
Wizzupuvos: gdb maybe?20:46
Wizzupuvos: right and that means that even cpcap_voice_call never gets called, right?20:46
uvosWizzup: that dose get called i think20:47
uvosWizzup: dpkg -S clock-ui20:47
uvosclock-ui: /usr/share/doc/clock-ui/copyright20:48
uvosclock-ui: /usr/share/doc/clock-ui/changelog.gz20:48
uvosclock-ui: /usr/share/doc/clock-ui20:48
uvos?20:48
Wizzupuvos: well if it gets called, it's not setting regs correctly20:49
Wizzupuvos: is that all? I see more20:49
uvosprobubly gets overwritten later20:49
uvosWizzup: yeah thats all here20:49
uvostheres soemthing wrong with the package20:49
Wizzupapt-cache show clock-ui20:50
Wizzup?20:50
uvosVersion: 0.8.0+2m7.120:50
uvosSHA256: eab0610d623379a70c06dc656a16f9386fa63ddba9e32789a31044d3a92dfb7c20:50
Wizzupsame here20:51
siceloI think that should be dpkg -L20:51
Wizzupoh yes20:52
Wizzupsomehow I missed that20:52
uvosoh right20:52
uvossilly me20:52
Wizzupuvos: so anything in /sys/kernel/debug/asoc/Mapphone\ Audio/cpcap-codec.0/dapm/ might keep the audio chain alive?20:53
Wizzupwell, if picked sensibly?20:53
uvosnot sure what you mean20:54
uvosthe base of the device needs to be one thats active20:54
uvosyou cant create one via the codec interface20:54
WizzupI guess my question is simply trying to understand how all of this works I guess20:54
Wizzupfor example I did see VAUDIO in that sysfs path20:54
uvosusually the base of the chain would be a dai20:54
uvosWizzup: it works21:04
uvos(alarm vibration + sound)21:05
uvosbut i had to set the sound file in gconf by hand21:05
Wizzupdid you change the pulse parts?21:09
uvosno21:09
uvospulse just returns sucess21:09
uvoserm ok...21:09
uvosbut not my problem :P21:09
uvosnow the segfault...21:09
Wizzupuvos: I can try to help if I can reproduce, so I need to set the file path and have my gst fix?21:17
uvosive got it21:17
uvosbut to repoduce you just have to try and change the ringtone21:17
freemangordonbactrace?21:18
uvosin clockui21:18
freemangordon*backtrace21:18
uvosim on it21:18
freemangordonok21:18
Wizzupis this profilesx?21:23
uvosno21:23
uvosprofilesx dosent set the ringtone21:24
uvosand the ringtone for alarms isent set in libprofile21:24
uvos(stupid)21:24
uvosits in gconf21:24
uvosits set by /usr/bin/worldclock21:24
Wizzupprofilesx can set 'ringing tone', but I guess you mean for alarm21:24
Wizzupright21:24
Wizzupok21:24
WizzupI don't think I get a segfault21:25
uvosin gdb it asserts21:25
WizzupI just changed it to something else21:25
Wizzuphmm21:26
uvostry chaging it backl21:26
Wizzupok21:26
uvosanyhow its tdialog.cpp line 13321:26
Wizzuploks like a crash yes21:26
uvosg_assert (player != NULL);21:26
Wizzupplaybin2 hmm21:26
uvosgts again21:26
uvos*gst21:27
uvos:P21:27
Wizzupplaybin2 does not exist21:27
Wizzupeither playbin (which is playbin2), or playbin321:27
uvosright should be esay21:27
uvosbut now i wonder why i was seing signal 11 before21:27
uvos(without gdb_21:27
uvos)21:27
Wizzupcould be g_assert icm qt not being happy21:28
Wizzupin combination with*21:28
Wizzupnot icm, sry21:28
Wizzupin general this really looks like it should use the qt gst stuff, not glib/gtk21:28
Wizzupbut for now maybe just try 'playbin'21:28
uvosyeah its building21:29
Wizzup:)21:29
freemangordonno idea why it was made like that21:31
freemangordonbut, this is not nokia code anyway21:31
Wizzupright21:31
uvosnot that nokia code would be better generally :P21:32
Wizzupit looks like there is a glib usage in two places21:32
freemangordonit does not make sense at all to do g_main_loop_run(loop); in Qt application to just play an audio file21:32
Wizzupmaybe 321:32
Wizzupthere is also: /* FIXME - try to not use glib */21:32
freemangordonthe ir QtMultimedia afaik21:32
uvosyes21:32
freemangordon*is21:32
freemangordonhehe21:32
uvosthat would be less code even21:33
freemangordonmhm21:33
uvosto use21:33
Wizzupit's just a bunch of g_strdup_printf+g_free21:33
uvosthat should be fine21:33
uvosif strange21:33
Wizzupand the rest is gst stuff it seems21:33
freemangordonok, maybe assign that to me21:33
uvosgrr my d4 hanged21:33
uvoswhile compileing21:33
Wizzupthere is also playbin2 usage in two places fwiw21:33
Wizzupuvos: hmmm interesting datapoint21:33
uvosWizzup: where else?21:34
Wizzup./alarmsndpick.cpp:    player = gst_element_factory_make ("playbin2", "Multimedia Player");21:34
Wizzup./tdialog.cpp:    player = gst_element_factory_make ("playbin2", "Multimedia Player");21:34
freemangordonit seems g_strdup_printf because of cloc_va_amount_days stuff21:36
freemangordonthere is no printf()-like function in Qt4 afaik21:37
uvosyou would use a stringstream21:37
uvosbut this is fine really21:38
uvosjust a bit strange21:38
freemangordonno, this is dgettext() that gives you format21:38
freemangordonso you must use printf()-like function21:39
Wizzupbut even using glib for str stuff is not too bad21:39
Wizzupit's the gst mainloop stuff that is bad21:39
freemangordonqt5 has asprintf() since 5.5, according to docs21:39
freemangordonright21:39
uvosdynamic format strings are a bit iky no?21:40
uvosbut ok21:40
freemangordonWizzup: do as you feel rigth - either port to QtMultimedia or create an issue and assign to me21:41
freemangordonQSound should do the job21:44
uvosno21:46
freemangordonno?21:46
uvosthey are arbitary format files21:46
freemangordonso?21:46
uvosQSound just plays wavs21:46
uvosanyhow its not hard21:46
Wizzupfreemangordon: I'll make an issue21:47
freemangordonok21:47
Wizzupuvos: have you seen this? https://www.kernel.org/doc/html/latest/sound/soc/codec-to-codec.html21:48
uvosQMediaPlayer would work21:49
uvosWizzup: no, but thats interesting21:49
lelMerlijnWajer opened an issue: https://github.com/maemo-leste/bugtracker/issues/619 (clock-ui: port gst code to Qt)21:49
Wizzupuvos: it looks like our usecase with the dsp21:49
Wizzupfreemangordon: did you rebuild the notify sv thing?21:49
Wizzupuvos: not sure if it will actually work for us, but it looks interesting21:52
WizzupIn order for this aforementioned route to get triggered, DAPM needs to find a valid endpoint which could be either a sink or source widget corresponding to playback and capture path respectively.21:53
uvosi mean now clock-ui works21:53
freemangordonWizzup: no, sorry, will do in tomorrow21:53
uvosbut it hangs until the audio file stops playing21:53
uvosobvious side effect of calling the main loop21:54
uvos...21:54
Wizzupuvos: ok, well, it's good for testing alarms, but we'll have to fix that soon :)21:54
Wizzupuvos: alt. can we make some fake Mux perhaps21:55
uvosyes21:55
uvosyes21:55
uvoshttps://doc.qt.io/qt-5/qaudio.html#Role-enum21:56
uvosinteresting21:56
uvosi wonder if this goes anyhwere21:57
* Wizzup stops reading alsa docs for now22:03
uvosanyhow it proted clockui to QMediaPlayer22:07
uvoss/it/i22:07
Wizzupported?22:08
Wizzupnice22:08
Wizzupwell you can close the bug report soon then :)22:08
Wizzupuvos: btw, did you chat with sre about the call audio dapm problems?22:09
uvosWizzup: only very briefly22:12
Wizzupok22:12
uvosWizzup: he was of the opinion that its impossible without switching from graph-card22:12
uvosand just fakeing a dai in your new graph-card'ish driver22:13
lelIMbackK opened a pull request: https://github.com/maemo-leste/clock-ui/pull/1 (switch to QMediaPlayer from gst)22:15
uvosthere22:15
uvoswith that and the gst patch we figured out before was required for decoding22:16
uvosalarms just work22:16
Wizzupuvos: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/sound/soc/samsung/neo1973_wm8753.c#n25622:16
Wizzupmaybe that is not graph card then :)22:17
lelMerlijnWajer closed a pull request: https://github.com/maemo-leste/clock-ui/pull/1 (switch to QMediaPlayer from gst)22:18
uvosWizzup: right thats a card driver not a codec driver22:18
uvosgraph-card is a card driver22:18
uvosofc22:18
Wizzupaha22:18
Wizzupok22:18
uvoswe use cpcap-codec, the codec driver22:19
uvoswith the generic graph-card card driver22:19
uvosi dont think we can make graph-card do what we want by messing around with the codec driver (or maybe we can see the alsa doc you posted)22:19
uvosbut we absoutly can if we just replace graph-card22:20
lelIMbackK closed an issue: https://github.com/maemo-leste/bugtracker/issues/619 (clock-ui: port gst code to Qt)22:21
lelIMbackK opened a pull request: https://github.com/maemo-leste/hildon-plugins-notify-sv/pull/2 (avoid passing NULL causing assert when fallback_sound_file dosent exist)22:26
lelIMbackK opened a pull request: https://github.com/maemo-leste/leste-config/pull/31 (Mapphones: up hifi volume a bit)22:34
uvosok22:35
uvosthats it22:35
uvosno kernel for today22:35
Wizzupnp :p22:48
Wizzupgood work22:48
lelMerlijnWajer closed a pull request: https://github.com/maemo-leste/leste-config/pull/31 (Mapphones: up hifi volume a bit)22:51
WizzupI'll rebuild the ucm later when I push my other changes22:51
Wizzupfreemangordon: I'll leave this one for you https://github.com/maemo-leste/hildon-plugins-notify-sv/pull/222:56
WizzupI am randomly seeing this:22:59
Wizzup[Mon Jun  6 23:57:21 2022] PVR_K: User requested SGX debug info22:59
Wizzup[Mon Jun  6 23:57:21 2022] PVR_K: SGX debug (SGX_DDK sgxddk 1.17@4948957)22:59
Wizzuphm: [167115.585] (EE) OMAP(0): ERROR: sgxSolidNextBatch: PVR2DBlt failed with error code: -2 (Device unavailable)22:59
Wizzupuvos: yeah alarm works23:05
Wizzupthe pattern is weird23:05
WizzupI snoozed it23:06
uvosyeah23:06
uvosit is23:06
uvosnot sure whats up with that23:06
Wizzuphm clock-ui still crashed for me23:06
Wizzuplet me see if it upgraded23:06
uvoswhy would it upgrade?23:06
uvoswe dident build it yet23:06
uvosuvos: Project ERROR: Unknown module(s) in QT: multimedia23:07
uvosah you did23:07
uvosand it failed23:07
Wizzupah23:07
Wizzupso we lack a pkg :)23:07
uvosyeah minor porblem23:08
Wizzuplmk what to add to control and I'll do it23:08
Wizzupno need ot pr23:08
uvoslibqt5multimedia5 to build-depends i gues23:09
uvosits a bit wierd libqt5multimedia5 dosent have a devel equivalent23:09
Wizzupqtmultimedia5-dev23:11
uvosright23:11
uvosthats probubly it23:11
uvosnice package nameing there23:12
Wizzup\o/23:12
Wizzupuvos: really nice that the alarm works23:12
uvosyeah23:12
Wizzup/usr/include/glib-2.0/glib/gtypes.h:32:10: fatal error: glibconfig.h: No such file or directory #include <glibconfig.h>23:16
uvosah yeah23:16
Wizzupwait, did you remove glib?23:16
uvosit still uses glib but that was just included by depens via gst23:17
uvosno23:17
Wizzupah23:17
Wizzupso do we need some glib dev?23:17
Wizzuppkg?23:17
Wizzuplibglib2.0-dev ?23:18
uvosshould be yes23:18
Wizzupuvos: btw I think there was a suggeston in the dapm somewhere that there could be a second almost dummy codec for this dai to dai without cpu thing23:20
Wizzupuvos: still a problem23:20
Wizzupuvos: I think some QT += stuff is required23:20
WizzupI guess PKGCONFIG23:21
uvoshmm23:21
uvosbut it builds locally23:21
uvosi dont se how that would change it23:21
WizzupPKGCONFIG += glib-2.023:21
uvos(building remote)23:21
Wizzuplet me just try this23:22
uvosa yes23:23
uvosclean building it local dosent work anymore23:23
Wizzupfigured23:23
Wizzupthat worked23:26
uvosgreat23:37
Wizzupwhat alarm sound do you use?23:38
Wizzupare the defaults installed?23:39
uvosyes23:39
Wizzupok, let me try it23:39
uvosshould play23:39
uvoswhen you select it23:39
konabedaboo-ba-bedaboo-ba-bedaboo-ba-be?23:39
uvosno23:40
konanot a terribly good alarm sound anyway :)23:40
Wizzupuvos: oh, didn't try, I usually have the device on silent :D23:41
Wizzupuvos: it might not even be that hard to switch away from graph card I think?23:46

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