Sunday, 2021-02-28

rinigusmal: good morning! as it looks like I need audiopolicyserver for camera, I have made a small service with it. while doing so, I android compiled it as 64-bit - same as it is on AOSP on my device. while we missed few lib links to apex, those were easy to add09:06
rinigusmissing links added in
rinigusadded service described in
rinigusresult: camera works as it is expected, as far as I can tell09:07
rinigusI can switch between modes (video->photo->video), shoot them.09:08
rinigusmal: please look into this approach and let me know what you think. I now wonder, what was the purpose of older miniaf.09:10
rinigusmal: also, whether we should follow AOSP approach and split cameraserver and audioserver without pushing it all into one. that would allow to follow AOSP libs closer, at least in terms of 32- && 64-bit09:12
piggzrinigus: doesnt a fake audiopolicy server already exist?09:12
piggzmal: after PlatformSDK upgrade, build no longer has the crypt issues, but still fails at the config.status step (or just after)09:13
riniguspiggz: it does as a part of minimedia.cpp. now, on my device (and I bet on pro1), audioserver is 64 bit; cameraserver 32 bit. as a result, I don't have some 32bit libs that are required for fake audiopolicy server in /system/lib09:14
piggzah, ok09:15
rinigusin addition, it turned out that on tama the order of ::instantiate was important.09:15
riniguswhen having audiopolicy before camera, it got all blocked with no camera available. so, it makes sense to separate these services09:15
riniguspiggz: do you happen to know where are miniaf sources?09:18
piggzrinigus: ?09:21
riniguspiggz thanks!09:24
swanuxSince yesterday I'm trying to solve my issue, and I tried another kernel config. The thing is, now I got this ( build error. Does anyone have any ideas?09:57
xmnpiggz Thanks for the PinePhone tip for sound. Although I could not find the pulseaudio dir. After the a update it started working!10:10
xmnever inching close to a usable OS for the PP. Cheers10:11
piggzxmn: yeah, just volume to sort <cough>jusa_</cough>10:43
piggzxmn: i recently updated pulseadudio and the ucm files, so was probably that10:43
xmnwell that was a nice surprise you gave me today :)10:48
xmnhow hard is the audio volume thing? Seem strange that its a thing. But I guess audio is in flux these days with pulseaudio & pipewire mix different distro10:49
piggzxmn: its the way sailfish does it ... there is a special pulseaudio pluging which manager the volume, and its that that doenst like our pulseaudio config for some reason10:55
piggzi suspect that if we had a volume app that controlled things individually like on a desktop, it would work10:56
piggz@eugenio_g7 ^^ was any special config needed on latte to get the alsa/UCM PA config to work with the meego volume plugin ?10:56
piggzi guess latte is a good reference .. and the new x86 port10:57
xmnhmm, sometime things that were thought to be smart turn out to cause issues later. This seems to be the case here.10:59
xmnthanks for sharing that info10:59
piggzxmn: the meego plugin does things like adjust the volume depending on the current use case, like call, media11:02
piggzthe volume controls on the phone are routed to that plugin iirc11:03
xmnI see. I did see that it seem the volume button are contextual. So if your in a music app then you get volume adjustment and if not then it default to ringtone volume.11:05
xmnSo piggz sailfish is using the meego plugin or something completely different11:05
piggzsailfish is using that plugin11:06
xmnI see11:06
piggzbut,the plugin seems to get confused with our config11:07
xmnso its a matter of finding the right combo of configs? Or something more you think?11:08
xmnClearly I'm not a programmer. :) just curious11:08
T42<eugenio_g7> piggz: what's the issue on pp? Only ringtone volume gets changed? I think we had that on latte too11:48
rinigus@eugenio_g7: on which SFOS version? I have to set volume to mute to be able to mute volume on 4.0.1/aarch64 tama. when some video is playing, it is possible to regulate volume11:53
T42<eugenio_g7> iirc it was pre 3.x on latte, but it was definitely an issue on our pulseaudio configs11:55
rinigus@eugenio_g7: OK, then it is something different11:57
T42<adampigg> @eugenio_g7 [piggz: what's the issue on pp? Only ringtone v …], No volume is changed at all13:18
T42Lucas %lastname% was added by: Lucas %lastname%14:27
riniguswhen we make a photo using stock camera, are we supposed to hear photo "shot" sound? same goes for starting and stopping video14:50
malrinigus: there should be some shutter sound14:56
rinigusmal: on my original (32 bit AOSP9) Tama port I don't have it. on AOSP10/aarch64 if I use MINIMEDIA_AUDIOPOLICYSERVICE_ENABLE - it is absent; if I use separate audiopolicy service, it is there14:58
rinigusis there a way to mute the sound if the user wishes to do so?14:59
T42<adampigg> wasnt there a privacy thing that said cameras had to make sounds nowadays?14:59
rinigus@adampigg: I think so. but it was about default, I think. although, I am not sure15:00
malrinigus: in some countries it's forbidden to have possibility to disable shutter sound15:15
rinigusmal: makes sense. then it is a bug in tama port (aosp9/arm32).15:17
T42DarknessHiddenorg was added by: DarknessHiddenorg15:18
*** rubdos_ is now known as rubdos19:36
T42SteffenPrz was added by: SteffenPrz19:55
riniguspiggz: do you remember why we specify _target_cpu in FPD? it is set to armv7hl and, as a result, doesn't really get picked up for aarch64 :)20:10
piggzrinigus: just seen, and replied, mal ^^20:11
piggzrinigus: im sure mal already has 64 bit patches for this20:11
riniguspiggz: nice! then will wait for mal reply. and then will try after 64 bit patches20:12
malrinigus: yes, I have 64-bit patches for piggz's fpd code20:16
malI just have forgotten to push those anywhere20:16
piggzmal: the usual behaviour is to send me a PR :D20:20
piggzthen i can nitpick it20:20
malpiggz: good that someone else also nitpicks :D20:21
piggzit would give me great pleasure to find a unnescessary newline20:21
malpiggz: rinigus:
piggzunfortunately i cant fault it20:55
riniguspiggz: this is a chance of a lifetime - make sure you check spaces as well21:38
rinigusmal: thank you for FPD PR. I'll compile it tomorrow night and hope to be able to test it on device as well22:07
malrinigus: thanks22:07

Generated by 2.17.1 by Marius Gedminas - find it at!