| T4 | <abhishek_0> meanwhile i have updated the docker image for gitlab-ci | 05:07 |
|---|---|---|
| T4 | <abhishek_0> https://gitlab.com/abhishek9650/sailfishos-wt88047-ci | 05:07 |
| T4 | <abhishek_0> check the latest tag | 05:07 |
| T4 | <abhishek_0> if anyone else wants to use it: abhishek9650/sfossdk:latest | 05:08 |
| *** ChanServ sets mode: +v T4 | 05:46 | |
| T4 | <abhishek_0> lol why was libwifi commented out https://paste.ubuntu.com/p/4cRcJDT2Hc/ | 05:54 |
| PragmaticAnarchi | hello | 06:03 |
| PragmaticAnarchi | before i start my journey turning my discusting zl1 android experience into a lovely linux in my pocket one logic indicates i ask first: does anyone know if a ports was atemptted for the leEco le pro 3 (zl1)? | 06:05 |
| PragmaticAnarchi | duckduckgo and xda reported nothing, so this is my last resort. | 06:06 |
| T4 | <abhishek_0> there was one guy on it, but never finished it | 06:09 |
| T4 | <meierrom> I'm not aware of any such port. Since lineage OS is supporting it and you can give a try. Won't be easy tho but it never is! 👍 | 07:34 |
| T4 | <meierrom> [Edit] I'm not aware of any such port. Since lineage OS is supporting it … you can give a try. Won't be easy tho but it never is! 👍 | 07:34 |
| Mister_Magister | @meierrom fun begins when there is no lineage port or any custom android of any kind | 08:06 |
| r0kk3rz | pfft, its easy lots of times | 08:07 |
| Mister_Magister | r0kk3rz: then halp | 08:07 |
| r0kk3rz | do you need help? | 08:08 |
| Mister_Magister | ye i have device tree made for twrp and need to convert it to lineage and make lineage work | 08:08 |
| r0kk3rz | oh nice | 08:08 |
| r0kk3rz | which device? | 08:09 |
| Mister_Magister | zenfone 5z | 08:09 |
| Mister_Magister | you said its easy so if you could help please | 08:09 |
| Mister_Magister | in android community nobody is gonna help you :/ | 08:09 |
| r0kk3rz | i was referring to @meierroms comment, ive never ported lineage before | 08:11 |
| r0kk3rz | but xda numptys can do it so how hard can it be | 08:11 |
| Mister_Magister | ah | 08:11 |
| abranson | piggz: I think gstreamer actually supports that colour format as part of its android support, so there may be some conversion functionality in there too | 08:50 |
| abranson | another alternative is that we stop misusing one format to wrap these opaque android ones, and actually label what's coming out properly | 08:51 |
| abranson | but that's quite a bit more work | 08:51 |
| T4 | <adampigg> abranson .... i only found an implementation in vlc so far ... but ill look in gst too | 08:53 |
| T4 | <adampigg> i piped it through the nv12 function but got color issues | 08:54 |
| abranson | piggz: https://github.com/GStreamer/gst-plugins-bad/blob/master/sys/androidmedia/gstamc.c#L2168 | 08:54 |
| abranson | i think that one was supported in gst-omx too | 08:58 |
| abranson | https://github.com/sailfishos/gst-omx/blob/master/rpm/0003-Add-GST_VIDEO_FORMAT_NV12_64Z32-to-the-video-format.patch | 08:59 |
| abranson | ah not quite the same sorry | 08:59 |
| r0kk3rz | how many color formats to they need?! | 09:00 |
| abranson | i think they're all variants that the hw decoders could do fastest | 09:01 |
| r0kk3rz | yeah no doubt theres an evolution of performance tweaks there | 09:01 |
| abranson | oh no wait that is the same. it tags QOMX_COLOR_FormatYUV420PackedSemiPlanar64x32Tile2m8ka as GST_VIDEO_FORMAT_NV12_64Z32, which gstreamer can convert to i420 itself. | 09:02 |
| T4 | <adampigg> abranson, so, best option is? | 09:02 |
| T4 | <adampigg> ah | 09:02 |
| abranson | piggz: you could try that. see if replacing the I420 with GST_VIDEO_FORMAT_NV12_64Z32 actually produces the correct colour | 09:02 |
| T4 | <adampigg> so, try and use gst version? | 09:03 |
| abranson | piggz: don't touch it - just copy it as with the current i420 case (that's for mtk) | 09:03 |
| T4 | <adampigg> inside gst-droid built in conversion? | 09:04 |
| abranson | yeah I think so. brainstorming a bit here though | 09:04 |
| T4 | <adampigg> dont worry, it wont happen until tonight anyway, have a think | 09:04 |
| T4 | <adampigg> so, gst-droid handles i420 and nv12 internally if not using droidmedia.....copy the i420 case and send it to another function? or add it to the if() and handle it the same? | 09:07 |
| abranson | adampigg: if you properly tag the frames coming out as GST_VIDEO_FORMAT_NV12_64Z32 then you won't have to use another function. gstreamer will insert a colour converter to give the browser the i420 it wants. | 09:08 |
| T4 | <adampigg> ok | 09:10 |
| abranson | so for a test, just replace everything i420 in gstdroidvdec.c with that format, and it should declare it as a source then gstreamer will do the rest | 09:13 |
| T4 | <abhishek_0> i am not getting any of this, but when you have solution for the colour issue, ping me :P | 09:14 |
| abranson | hmm, the current buffer copy does cropping too though. that might be tricky | 09:17 |
| abranson | that format looks crazy: "NV12 with 64x32 tiling in zigzag pattern" | 09:17 |
| abranson | no wonder that function was so long | 09:17 |
| Mister_Magister | is there somebody that knows about android porting? | 09:20 |
| T4 | <adampigg> abranson...yeah, i wondered/about the cropping, and assumed youd taken it into account :D | 09:23 |
| abranson | adampigg: well that's brainstorming for you :D | 09:28 |
| T4 | <adampigg> abranson, feel free to brainstorm for the next 6 hrs, when i will get to try it | 10:18 |
| T4 | <abhishek_0> :wondering: if we can automate the hal builds on gitlab-ci | 12:22 |
| r0kk3rz | why? | 12:24 |
| T4 | <abhishek_0> because why not, thinking of if we can take off all the load from the local machine | 12:29 |
| r0kk3rz | because generally once stuff is working the hal doesnt change much | 12:31 |
| r0kk3rz | with the exception of droidmedia maybe | 12:32 |
| r0kk3rz | otherwise you'd need to pack all the android source and build stuff into a container | 12:33 |
| T4 | <abhishek_0> probably not very efficient , ,, or i can just setup a runner | 12:36 |
| T4 | <abhishek_0> or maybe not,, $$ | 12:36 |
| r0kk3rz | generally my idea was just to run mic against whats in OBS | 12:37 |
| r0kk3rz | and set that up for all devices in OBS, so we can do bulk image updates | 12:38 |
| T4 | <abhishek_0> we can create a gitlab org | 12:39 |
| r0kk3rz | yeah something like that | 12:39 |
| T4 | <abhishek_0> i can set it up ,,, and we can add devices ,,, like lineageos does | 12:40 |
| T4 | <abhishek_0> i have updated the docker image, 2.2.1.18 builds fine | 12:42 |
| r0kk3rz | you just used zypper to update? | 13:14 |
| Mister_Magister | r0kk3rz: sfossdk? | 13:19 |
| T4 | <abhishek_0> r0kk3rz yeah | 13:49 |
| T4 | <meierrom> @Mister_Magister [is there somebody that knows about android por …], Mister_Magister: lineage OS people have likely more experience in Android porting than members of this group :) | 13:52 |
| Mister_Magister | @meierrom people there are more likely to help even if they dont have knowledge than anybody form android community | 13:53 |
| Mister_Magister | trust me | 13:53 |
| Mister_Magister | thats why im asking | 13:53 |
| * vknecht hesitates to ask los questions too ;-) | 14:11 | |
| wolfquin | noob question again; im getting the issue; (error: in sync --fetch-submodules: revision refs/tags/android-4.4.2_r2 in platform/prebuilts/tools not found) when trying to repo sync with (—fetch-submodules), im trying to sync hybris 11.0, and the kernel/device of gtelwifi(sm-t560)im using ubuntu chroot with Platform SDK of SailFishOS. and my manifest is the following:<?xml version="1.0" | 14:33 |
| wolfquin | encoding="UTF-8"?> | 14:33 |
| wolfquin | <manifest> | 14:33 |
| wolfquin | <project path="device/samsung/gtelwifi" | 14:33 |
| wolfquin | name="gtelwifi/android_device_samsung_gtelwifi" | 14:33 |
| wolfquin | remote="github" /> | 14:33 |
| wolfquin | <project path="kernel/samsung/gtelwifi" | 14:33 |
| wolfquin | name="gtelwifi/android_kernel_samsung_gtelwifi" | 14:33 |
| wolfquin | remote="github" /> | 14:33 |
| wolfquin | this is an unofficial build of LineageOS | 14:34 |
| mal | try without --fetch-submodules | 14:36 |
| wolfquin | 10.4Mfatal: Couldn't find remote ref refs/tags/cm-11.0-XNPH44S-bacon-5fa8c79c0b | 14:37 |
| wolfquin | should i just ignore it? | 14:38 |
| wolfquin | it seems it isn't downloading the kernel nor the device, must be something wrong with my manifest? | 14:39 |
| elros1 | try to set correct revision in your local manifest | 14:43 |
| mal | wolfquin: define revision for the projects in local manifest, in this case cm-11.0 | 14:43 |
| elros1 | :) | 14:43 |
| T4 | minlexx was removed by: minlexx | 14:43 |
| wolfquin | trying now, everything seems to work so far, at 24%. | 14:50 |
| T4 | <adampigg> abranson, on way home, any more brainstorming for me ;) | 14:53 |
| taixzo | mal: matissewifi? | 15:32 |
| mal | taixzo: I still haven't fixed the scaling issues | 15:44 |
| abranson | adampigg: only that you can probably use a videocrop element to do the cropping | 15:49 |
| piggz | abranson: say for the moment I ignore cropping....im in gstdroidvdec.c, where am i looking ? | 17:09 |
| piggz | has anyone replced a battery in a TBJ ? | 17:18 |
| T4 | faenil was added by: faenil | 17:22 |
| *** ChanServ sets mode: +v T4 | 17:24 | |
| elros1 | ohyash: quite smooth sfos on A31 https://raw.githubusercontent.com/elros34/freetab_8014/master/droid/Screenshot-18-09-18-18-06-13.png | 18:12 |
| piggz | mal: do you know where in gstdroidvdev i should tag the video as GST_VIDEO_FORMAT_NV12_64Z32 ? | 18:34 |
| piggz | im thinking in gst_droidvdec_data_available | 18:36 |
| mal | piggz: seems like a good place, somewhere before to color conversion function probably, you probably need also something in the color conversion so it will not fail with unknown format | 18:42 |
| piggz | mal: yes, i have that already | 18:43 |
| piggz | mal: im doing it wrong somewhere ... hitting an assert | 18:50 |
| piggz | gst_video_frame_map_id: assertion 'info->finfo->format == meta->format' failed | 18:50 |
| mal | piggz: fix call of gst_buffer_add_video_meta ? | 18:51 |
| mal | it sets format to GST_VIDEO_FORMAT_I420 | 18:52 |
| piggz | thats what i did | 18:52 |
| piggz | gst_buffer_add_video_meta (buff, GST_VIDEO_FRAME_FLAG_NONE, | 18:52 |
| piggz | GST_VIDEO_FORMAT_NV12_64Z32, width, height); | 18:52 |
| piggz | in 2 places | 18:52 |
| mal | where is the assert? | 18:54 |
| piggz | mal: elsewhere in gstreamer i think | 18:57 |
| piggz | mal: assert is here https://github.com/GStreamer/gst-plugins-base/blob/1.14/gst-libs/gst/video/video-frame.c#L90 | 19:26 |
| mal | piggz: the droidvdec says it's caps are for i420 so maybe that is the reason the other parts think it outputs only that | 19:28 |
| piggz | yes, i thought something like that | 19:28 |
| piggz | need abranson! | 19:28 |
| r0kk3rz | piggz: stuck? | 19:34 |
| mal | piggz: check dec->out_state->info | 19:36 |
| mal | in gst_droidvdec_data_available | 19:36 |
| mal | that is GstVideoInfo struct | 19:37 |
| Mister_Magister | dr_gogeta86_: ping | 19:41 |
| piggz | mal: i cant change that member as its const ... i'll print it out... | 19:49 |
| piggz | mal: format is 2, ie GST_VIDEO_FORMAT_I420 | 19:55 |
| mal | yep, so the expected format is that | 19:55 |
| piggz | mal: so, im back to the original....how to convert GST_VIDEO_FORMAT_NV12_64Z32 to GST_VIDEO_FORMAT_I420 | 20:01 |
| r0kk3rz | shuffle the bits around :) just like a rubics cube | 20:02 |
| piggz | r0kk3rz: yeah, shuffe the bits around in a fscking zigzag pattern :/ | 20:04 |
| r0kk3rz | yeah, like a rubics cube | 20:14 |
| r0kk3rz | get it looking right first and then we can try and optimise it | 20:16 |
| r0kk3rz | because it sounds like an 10000 step process to unzig the zags | 20:16 |
| piggz | r0kk3rz: bollocks to it ..... LLS works fine | 20:26 |
| mal | piggz: it's just that droidvdec is limited, maybe lls does it via built-in gstreamer codecs and stuff | 20:35 |
| r0kk3rz | lls would be outputting native buffers | 20:35 |
| mal | it probably leaves the conversion to other parts of gstreamer | 20:36 |
| piggz | i think it is browser that mandates i420 isnt it? | 20:37 |
| r0kk3rz | i dont think it converts it at all in normal case | 20:38 |
| piggz | now then, if i could justre-write www.videolan.org/developers/vlc/modules/codec/omxil/qcom.c | 20:40 |
| piggz | its certainly smaller than that huge function on stackexchange! | 20:40 |
| r0kk3rz | yeah it has functions | 20:41 |
| r0kk3rz | "nothing to be understood here, don't try" :D | 20:42 |
| r0kk3rz | "Second wtf: why this format at all" :D:D | 20:43 |
| piggz | i know, genius isnt it | 20:43 |
| r0kk3rz | this lets you know you're dealing with some heavy black magic | 20:44 |
| T4 | VeryOriginalUsername was removed by: VeryOriginalUsername | 20:48 |
| piggz | r0kk3rz: in that vlc code, what would pitch refer to? | 20:48 |
| r0kk3rz | hmmm | 20:50 |
| r0kk3rz | its read out of the inputted picture_t, whatever that is | 20:51 |
| piggz | https://github.com/videolan/vlc/blob/master/src/misc/picture.c#L160 | 20:53 |
| piggz | some function of width and pixel size | 20:53 |
| r0kk3rz | yes ok, dot pitch then | 20:54 |
| r0kk3rz | or maybe https://en.wikipedia.org/wiki/Stride_of_an_array | 20:55 |
| piggz | ah, stride i can work wth... | 20:56 |
| mal | piggz: that vlc function looks much more reasonable | 20:57 |
| piggz | mal: certainly | 20:57 |
| * piggz struggling to concentrate, when bttf-pt2 is on! | 20:58 | |
| piggz | so, the vlc function has a single pitch/stride .... i think we have 3 seperate strides, for y,u,v | 21:01 |
| piggz | mal: r0kk3rz: thats me for tonight ... i got a purple screen only | 21:31 |
| piggz | i tried as per https://bpaste.net/show/4b6a7d59caa7 , line 365 and 1322 | 21:32 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
