| rinigus | b100dian: you have tested camera2 droidmedia. does it work for nagara? can we choose cameras with it? | 08:21 |
|---|---|---|
| rinigus | I guess it is the only functionality that's missing with cameras | 08:22 |
| rinigus | apart of changing optical zoom on 1iv | 08:22 |
| rinigus | ... and motion compensation by camera, probably due to fake sensor manager in droidmedia | 08:29 |
| T42 | <b100dian> It doesn't work yet, need to see what the new logs say, after the fake package_native development. | 09:01 |
| T42 | <b100dian> No viewfinder, no list of cams so early failure probably | 09:01 |
| rinigus | @b100dian: which branch did you use? camera2 in droidmedia? that one is missing few fake services that were added recently. I am trying now to check whether merging camera2 with master branch would give better results. | 09:08 |
| *** Daanct12 is now known as Danct12 | 09:30 | |
| T42 | <elros34> I use 'setprop ctl.stop vendor.fps_hal' to stop android service. 'getprop | grep fps_hal'' shows it "stopping" but nothing happens, service is running and status is never set to 'stopped'. Is there anything I am missing here? | 09:44 |
| rinigus | b100dian: indeed, with camera2 something else is going on. getting logs similar to yours | 10:40 |
| rinigus | will revert for now for camera1 and start working on ofono volte plugin (aidl option) | 10:42 |
| T42 | <elros34> @edp_17 did try something like that? https://paste.opensuse.org/pastes/1261585c7e14 | 11:35 |
| T42 | <b100dian> rinigus: it wasnt merged with latest changes was gonna test that now again. | 11:43 |
| rinigus | b100dian: I tried with the merged changes and it didn't work for me. logcat was the same as yours at https://github.com/sailfishos-sony-nagara/main/issues/34#issuecomment-3091255594 | 11:44 |
| rinigus | but give it a try - maybe you are luckier :) | 11:45 |
| T42 | <edp_17> @elros34 : I've tried this: https://paste.ubuntu.com/p/BFSNRDQVVX/ | 12:52 |
| T42 | <edp_17> But I'll try that you suggested | 12:52 |
| T42 | <elros34> first you should build defconfig | 12:53 |
| T42 | <edp_17> Oh, okay. I've executed both and got this: https://paste.ubuntu.com/p/nGNzmDVcJn/ | 12:54 |
| mal | rinigus: I'll probably rebase camera2 and push it, need to see if I can fix some issues in it also | 12:59 |
| mal | rinigus: btw, I think the packagen_native issue is exactly the same that I had, and it's doable also via property, not sure yet which is better way | 13:06 |
| rinigus | mal: looking forward to your rebase with camera2. | 13:08 |
| rinigus | as for which way is better - I think it makes sense to be as close to android as possible. but its close to more philosophical discussion and maybe its better not to go there :) | 13:10 |
| mal | rinigus: in case you are interested check frameworks/av/media/libstagefright/MediaCodec.cpp, package_native use requested by connectFormatShaper which is called from createMediaMetrics via shapeMediaFormat and controlled via "debug.stagefright.enableshaping" property | 13:11 |
| mal | but since that is a debug.* property not sure if it's a good thing to use | 13:12 |
| T42 | <elros34> @edp_17 is this right after running make hybris-hal? | 13:16 |
| T42 | <elros34> ah typo, see missing space | 13:17 |
| T42 | <elros34> more than once | 13:17 |
| T42 | <elros34> and clean backports directory first, I remember building defconfig without proper ARCH can cause issues | 13:19 |
| rinigus | mal: interesting - those features in that source are exactly the ones that are called during video recording. as seen from the other side in package_native debug logs :) | 13:30 |
| mal | yes, so the issue is not limited to camera2, same is also in older camera | 13:34 |
| mal | I will do some investigation a bit later today | 13:35 |
| rinigus | indeed. I had it with older camera API | 13:36 |
| mal | if we go with fake package_native it would go into droidmedia like the others | 13:39 |
| mal | I have been thinking a bit or reworking the fake services in droidmedia, it's a bit stupid now | 13:40 |
| mal | *of reworking | 13:40 |
| rinigus | mal: then it has to be written using android classes. I have such version too which I did before libgbinder version. although, it wasn't processing any requests and this will have to be added (trivial to do) | 13:41 |
| rinigus | let me push it somewhere | 13:42 |
| mal | yeah, those are usually not difficult to do with android classes, I have done so many of those that it shouldn't take long | 13:42 |
| rinigus | mal: here you go - https://github.com/rinigus/fake-package-native-android | 13:47 |
| mal | thanks | 15:10 |
| T42 | <A_T_R> HI , my sfos port version 4.6.0.15 is working fine now , i wish to test my obs and for testing latest sfOS version 5.0.0.62 or any version ,what are the steps to follow ? | 15:30 |
| T42 | <A_T_R> here https://build.sailfishos.org/user/show/iAboothahir | 15:30 |
| mal | which device? | 15:34 |
| T42 | <A_T_R> asus max pro m1 ,codename x00td (X00TD) | 15:34 |
| mal | do the needed changes to https://build.sailfishos.org/project/show/nemo:devel:hw:asus:x00td | 15:35 |
| mal | so update the packages to correct versions and make sure the config packages etc are ok | 15:36 |
| T42 | <A_T_R> update droid-confifs to a specific sfos version and compile —droid-configs and upload droid-config-*.rpm ? | 15:37 |
| mal | droid-config package is not uploaded, it's built on obs, make sure your droid-config github has correct version and the just trigger build on obs | 15:38 |
| mal | I assume this is what you use https://github.com/SailfishOS-sdm660/droid-config-X00TD | 15:41 |
| T42 | <A_T_R> yes ,master branch | 15:41 |
| mal | is it missing some changes? | 15:42 |
| mal | seems to be rather old | 15:42 |
| T42 | <A_T_R> yes ,bit old - i am stuck at 4.6.0.15 | 15:42 |
| mal | so push the changes there | 15:42 |
| T42 | <A_T_R> about the submodule ? | 15:43 |
| T42 | <A_T_R> https://sailfishos.wiki/link/20#bkmrk-5.0.0.x use this recommended revision? | 15:43 |
| mal | yes | 15:44 |
| mal | and any other changes you might have locally | 15:44 |
| T42 | <A_T_R> as of now droid-config-X00TD locally has no changes | 15:45 |
| mal | it might work with current config submodule, not sure, maybe better to update the submodule to correct version | 15:45 |
| rinigus | when looking at available binder interfaces (aidl and hidl), nagara doesn't have IImsRadioResponse nor IImsRadioIndication. looks like ofono-binder-plugin-ext-qti is trying to use it though | 15:50 |
| rinigus | in logs, I get "BpBinder: Cannot do a user transaction on a system stability binder (vendor.qti.hardware.radio.ims.IImsRadioResponse) in a vendor stability context." | 15:50 |
| T42 | <A_T_R> yes bit old | 15:50 |
| T42 | <A_T_R> locally 4.6.0.15 | 15:50 |
| T42 | <A_T_R> droid-config -remote 4.5.0.21 (re @SailfishFreenodeIRCBridgeBot: <mal>seems to be rat...) | 15:50 |
| rinigus | hmm, will have to read code a bit more. maybe AIDL API is different... | 15:51 |
| mal | rinigus: gbinder_local_object_set_stability(callback_pointer, GBINDER_STABILITY_VINTF); before doing the call to which that callback pointer is given | 16:01 |
| mal | so in function doing the "open" binder call | 16:02 |
| mal | that is needed when using certain aidl interfaces | 16:02 |
| rinigus | mal: thanks!!! problem is that I don't see any vendor.qti.hardware.radio.ims.IImsRadioResponse in binder-list (/dev/binder or hwbinder). shouldn't it be there for such calls? | 16:03 |
| mal | yes | 16:03 |
| rinigus | but I can surely try | 16:03 |
| mal | what do you have in those? | 16:03 |
| mal | anything with ims? | 16:03 |
| rinigus | mal: https://dpaste.org/xDcUR/raw | 16:04 |
| rinigus | is there more in xperia 10v/iv? | 16:05 |
| mal | rinigus: you have those vendor.qti.hardware.radio.ims.IImsRadio/imsradio* in /dev/binder which are what you are looking for | 16:07 |
| mal | those response things don't show in binder-list, those are part of that vendor.qti.hardware.radio.ims.IImsRadio | 16:08 |
| mal | I haven't yet managed to get aidl ims working, I have been able to communicate with it and get responses and indications but no sign of it registering to volte | 16:09 |
| rinigus | thanks! so, I have to find AIDL for vendor.qti.hardware.radio.ims.IImsRadio and then see what would correspond to response... | 16:09 |
| rinigus | hmm, if mal hasn't done it - what chance do we have... | 16:10 |
| rinigus | OK, will try to dig into it. | 16:10 |
| rinigus | have you seen by any chance aidl definition for vendor.qti.hardware.radio.ims.IImsRadio? | 16:10 |
| rinigus | not that I looked for it | 16:11 |
| mal | it's just vendor.qti.hardware.radio.ims.IImsRadioResponse and similar for Indication | 16:11 |
| mal | I haven't seen any aidl files anywhere for that interface, I figured it out the ugly way | 16:12 |
| T42 | <edp_17> @elros34 , you were right, I didn't spot those typos. As I fixed them, the first one has built, but the second failed: https://paste.ubuntu.com/p/Gp2GWFm2nW/ | 16:46 |
| T42 | <elros34> python2 issue? | 16:49 |
| T42 | <edp_17> In HABUILD, the python —version gives me Python 3.8.5 | 16:52 |
| T42 | <elros34> too new | 16:55 |
| T42 | <edp_17> Hahaha. :) | 16:56 |
| rinigus | mal: just to make sure that I follow. I have AIDL service vendor.qti.hardware.radio.ims.IImsRadio/imsradio0 . under that, in LOS21, I can see interface vendor.qti.hardware.radio.ims.IImsRadio. so, if I understand you correctly, it also has interface vendor.qti.hardware.radio.ims.IImsRadio and maybe few others that are just not registered with binder. right? | 17:00 |
| mal | there is only that one interface, which inside contain the response and indication stuff | 17:46 |
| mal | you don't see the response and indication stuff in binder-list for any other interfaces either | 17:47 |
| rinigus | no, I don't see indeed. | 18:06 |
| rinigus | mal: in my understanding, what we see listed in binder-list are services. under each service, we can have interfaces. is that correct? | 18:15 |
| mal | well usually the names in binder-list are of the main interface | 18:21 |
| mal | afaik | 18:21 |
| rinigus | yes, true. but in just recent case - service was package_native and interface was android.content.pm.IPackageManagerNative | 18:23 |
| mal | ok | 18:32 |
| mal | yeah, I was thinking more of those hw side services | 18:32 |
| rinigus | mal: your tip with GBINDER_STABILITY_VINTF helped. looks to be going bit further. will study logs and read plugin code tomorrow | 18:47 |
| rinigus | we don't happen to have any libgbinder docs? | 18:47 |
| mal | just the api headers on github | 18:51 |
| mal | and test code in there also gives some hints about usage | 18:51 |
| mal | and of course other projects using it | 18:51 |
| rinigus | yep, was using those. will ask if I get too confused. | 19:00 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!